linux-kernel - Re: [PATCH v3 4/4] platform/x86: (ayn-ec) Add AYN EC Platform Documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5e910391-5769-40dc-882d-90f79888c772@gmx.de>
Date: Sun, 27 Jul 2025 02:03:43 +0200
From: Armin Wolf <W_Armin@....de>
To: "Derek J. Clark" <derekjohn.clark@...il.com>,
 Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
 Hans de Goede <hansg@...nel.org>
Cc: Jean Delvare <jdelvare@...e.com>, Guenter Roeck <linux@...ck-us.net>,
 Alok Tiwari <alok.a.tiwari@...cle.com>,
 David Box <david.e.box@...ux.intel.com>,
 platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-hwmon@...r.kernel.org, linux-doc@...r.kernel.org
Subject: Re: [PATCH v3 4/4] platform/x86: (ayn-ec) Add AYN EC Platform
 Documentation

Am 26.07.25 um 22:40 schrieb Derek J. Clark:

> Adds ABI documentation for the ayn-ec platform driver
>
> Signed-off-by: Derek J. Clark <derekjohn.clark@...il.com>
> ---
>   .../ABI/testing/sysfs-platform-ayn-ec         | 59 +++++++++++++++++++
>   MAINTAINERS                                   |  1 +
>   2 files changed, 60 insertions(+)
>   create mode 100644 Documentation/ABI/testing/sysfs-platform-ayn-ec
>
> diff --git a/Documentation/ABI/testing/sysfs-platform-ayn-ec b/Documentation/ABI/testing/sysfs-platform-ayn-ec
> new file mode 100644
> index 000000000000..32cb6f7ca2fc
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-platform-ayn-ec
> @@ -0,0 +1,59 @@
> +What:		/sys/class/hwmon/hwmon[0-9]/pwm1_enable
> +Date:		July 2025
> +KernelVersion:	6.17

I think that this driver will miss 6.17, better use 6.18 instead. Also you do not have to
document standard hwmon sysfs attributes. For documenting the special PWM mode i suggest that
you add some documentation under Documentation/hwmon/.

Additionally please add the new files to the MAINTAINERS entry of the associated driver.

Thanks,
Armin Wolf

> +Contact:	"Derek J. Clark" <derekjohn.clark@...il.com>
> +Description:
> +		This sets the PWM fan mode of operation. Valid values are [0-3].
> +    Values [0-2] conform with standard hwmon operating modes. Value 3
> +    enables user defined fan curve settings.
> +
> +    Applies to AYN Loki and Tectoy Zeenix lines of handheld devices.
> +
> +What:		/sys/class/hwmon/hwmon[0-9]/pwm1_auto_point[1-5]_pwm
> +Date:		July 2025
> +KernelVersion:	6.17
> +Contact:	"Derek J. Clark" <derekjohn.clark@...il.com>
> +Description:
> +		This sets the PWM fan duty cycle for the given index of the fan curve.
> +    When the temperature reaches the corresponding pwm1_auto_point[1-5]_temp,
> +    the EC will automatically increase the fan duty cycle to the given value.
> +
> +    Values are [0-255]
> +
> +    Applies to AYN Loki and Tectoy Zeenix lines of handheld devices.
> +
> +What:		/sys/class/hwmon/hwmon[0-9]/pwm1_auto_point[1-5]_temp
> +Date:		July 2025
> +KernelVersion:	6.17
> +Contact:	"Derek J. Clark" <derekjohn.clark@...il.com>
> +Description:
> +		This sets the activation temperature for the given index of the fan curve.
> +    When the temperature reaches the given value, the EC will automatically
> +    increase the fan duty cycle to the corresponding pwm1_auto_point[1-5]_pwm
> +    value.
> +
> +    Values are [0-100]
> +
> +    Applies to AYN Loki and Tectoy Zeenix lines of handheld devices.
> +
> +What:		/sys/class/leds/ayn:rgb:joystick_rings/effect
> +Date:		July 2025
> +KernelVersion:	6.17
> +Contact:	"Derek J. Clark" <derekjohn.clark@...il.com>
> +Description:
> +		This controls the display effect of the RGB interface.
> +
> +    Values are monocolor or breathe.
> +
> +    Applies to AYN Loki and Tectoy Zeenix lines of handheld devices.
> +
> +What:		/sys/class/leds/ayn:rgb:joystick_rings/effect_index
> +Date:		July 2025
> +KernelVersion:	6.17
> +Contact:	"Derek J. Clark" <derekjohn.clark@...il.com>
> +Description:
> +		This displays the available options for the effect attribute.
> +
> +    Values are monocolor or breathe.
> +
> +    Applies to AYN Loki and Tectoy Zeenix lines of handheld devices.
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5b816883fe7d..199bebbffa0d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4039,6 +4039,7 @@ AYN PLATFORM EC DRIVER
>   M:	Derek J. Clark <derekjohn.clark@...il.com>
>   L:	platform-driver-x86@...r.kernel.org
>   S:	Maintained
> +F:	Documentation/ABI/testing/sysfs-platform-ayn-ec
>   F:	drivers/platform/x86/ayn-ec.c
>   
>   AZ6007 DVB DRIVER