Re: [PATCH v2] Documentation: leds: Add description of LED Flash class extension

[Pavel Machek <pavel@ucw.cz>]

Hi!

> I can be 0 or 1. Let's make it more precise:
> 
> - flash_strobe - flash strobe state (RW):
> 			semantics on write:
> 				0: turn the flash LED off
> 				1: strobe the flash LED
> 			semantics on read:
> 				0: flash LED is off
> 				1: flash LED is strobing

Thanks.

> >>+	- available_sync_leds - list of sub-LEDs available for flash strobe
> >>+				synchronization (RO)
> >
> >"space separated"?
> 
> - available_sync_leds - space separated list of sub-LEDs available for
> 			flash strobe synchronization; each sub-LED is
> 			described in the form of chunks:
> 			[led_id: led_name]
> 
> 
> >So this will say something like "0 3 5"
> 
> Rather e.g.: [0: none] [1: max77693-led1] [2: max77693-led2]

No no, sorry, you can't do that. Sysfs is supposed to be one value per
file, and this is stretching it. (It would be also difficult to parse;
for example, you can reasonably have ":" in led name, and perhaps even
" " or "]"....
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html