Re: outdated documentation for lp5523 LED driver?

[Pavel Machek]

On Fri 2016-07-15 09:40:52, Pali Rohár wrote:
> On Friday 15 July 2016 09:32:14 Pavel Machek wrote:
> > Hi!
> > 
> > Documentation says that load/leds are only visible in the load mode,
> > but that does not seem to be true (kernel 4.4.0 on N900).
> > 
> > # 1) Legacy interface - enginex_mode, enginex_load and enginex_leds
> > #  Control interface for the engines:
> > #    x is 1 .. 3
> > #      enginex_mode : disabled, load, run
> > #      enginex_load : microcode load (visible only in load mode)
> > #      enginex_leds : led mux control (visible only in load mode)
> > 
> > pavel@n900:/sys/class/leds/lp5523:r/device$ ls -al engine*
> > -rw-r--r-- 1 root root 4096 Jul  6 22:47 engine1_leds
> > --w------- 1 root root 4096 Jul  6 22:47 engine1_load
> > -rw-r--r-- 1 root root 4096 Jul  6 23:41 engine1_mode
> > -rw-r--r-- 1 root root 4096 Jul  6 22:47 engine2_leds
> > --w------- 1 root root 4096 Jul  6 22:47 engine2_load
> > -rw-r--r-- 1 root root 4096 Jul  6 23:41 engine2_mode
> > -rw-r--r-- 1 root root 4096 Jul  6 22:47 engine3_leds
> > --w------- 1 root root 4096 Jul  6 22:47 engine3_load
> > -rw-r--r-- 1 root root 4096 Jul  6 22:47 engine3_mode
> > pavel@n900:/sys/class/leds/lp5523:r/device$ grep . engine*mode
> > engine1_mode:run
> > engine2_mode:load
> > engine3_mode:run
> > pavel@n900:/sys/class/leds/lp5523:r/device$
> 
> On 2.6.28 kernel is present only legacy interface and for this version
> is above documentation correct. When engine is in "run" or "disabled",
> then sysfs nodes _load and _leds are invisible.

Well, I don't think 2.6.28 is suitable kernel to compare
against... and I don't think hiding sysfs attributes makes any sense.

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html