On Wed, Feb 18, 2015 at 11:47:47PM +0100, Pavel Machek wrote:
> 
> On Wed 2015-02-18 17:20:22, Jacek Anaszewski wrote:
> > Add a documentation of LED Flash class specific sysfs attributes.
> > 
> > Signed-off-by: Jacek Anaszewski <[email protected]>
> > Acked-by: Kyungmin Park <[email protected]>
> > Cc: Bryan Wu <[email protected]>
> > Cc: Richard Purdie <[email protected]>
> 
> NAK-ed-by: Pavel Machek
> 
> > +What:              /sys/class/leds/<led>/available_sync_leds
> > +Date:              February 2015
> > +KernelVersion:     3.20
> > +Contact:   Jacek Anaszewski <[email protected]>
> > +Description:       read/write
> > +           Space separated list of LEDs available for flash strobe
> > +           synchronization, displayed in the format:
> > +
> > +           led1_id.led1_name led2_id.led2_name led3_id.led3_name etc.
> 
> Multiple values per file, with all the problems we had in /proc. I
> assume led_id is an integer? What prevents space or dot in led name?

Very good point. How about using a newline instead? That'd be a little bit
easier to parse, too.

-- 
Sakari Ailus
e-mail: [email protected]     XMPP: [email protected]
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to