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

[Derek John Clark <derekjohn.clark@gmail.com>]

On Sat, Jul 26, 2025 at 5:04 PM Armin Wolf <W_Armin@gmx.de> wrote:
>
> 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@gmail.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
>

Can do, thanks.
Derek

> > +Contact:     "Derek J. Clark" <derekjohn.clark@gmail.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@gmail.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@gmail.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@gmail.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@gmail.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@gmail.com>
> >   L:  platform-driver-x86@vger.kernel.org
> >   S:  Maintained
> > +F:   Documentation/ABI/testing/sysfs-platform-ayn-ec
> >   F:  drivers/platform/x86/ayn-ec.c
> >
> >   AZ6007 DVB DRIVER