Re: [PATCH v13 RESEND 3/4] docs: ABI: Document LP5812 LED sysfs interfaces
From: Nam Tran
Date: Tue Aug 19 2025 - 12:34:57 EST
On Sun, 17 Aug 2025, Randy Dunlap wrote:
> > +What: /sys/class/leds/led_<id>/lod_lsd
> > +Date: July 2025
> > +KernelVersion: 6.17
> > +Contact: Nam Tran <trannamatk@xxxxxxxxx>
> > +Description:
> > + 0 0 mean no lod and lsd fault detected, 1 1 mean lod and lsd fault detected (RO)
>
> At first the "0 0" and "1 1" confused me (thought it was a typo),
> but I think what you are showing here is a sysfs file with 2 values, right?
> That used to be discouraged (or even nacked), although I don't know the
> current policy on that.
The lod_lsd sysfs file currently reports two values: the first value is the LOD
(LED open detection) fault status, the first value is the LOD (LED open detection)
fault status.
I followed this approach to keep the two related fault bits in a single file, but
I'm open to splitting them into separate sysfs entries if that is preferred.
Best regards,
Nam Tran