Hi! > >>>> This patch introduces 4 new attributes: > >>>> master_fader_leds > >>>> master_fader1 > >>>> master_fader2 > >>>> master_fader3 > >>>> > >>>> Fo example, to map channel 0,6 to master_fader1, > >>>> map channel 1,7 to master_fader2, > >>>> map channel 2,8 to master_fader3, and > >>>> map channel 3,4,5 to none > >>> > >>> > > >>>> > >>>> echo "123000123" > master_fader_leds > >>> > >>> > >>> I propose to add ABI documentation for this driver. It already exposes > >>> many custom attributes but I can't find documentation for them. > >> > >> > >> I agree with Jacek's comment but it would be better if we could see the > >> description in lp5523 driver > >> documentation(Documentation/leds/leds-lp5523.txt). > > > > > > Right, I missed that file. > > > > > > Toshi, > > I merged this patch into my tree with Milo and Jacek Acks. But please > follow their advice and provide a patch to update those document > files. Are you sure this is the right interface? lp5523 is turing-complete. Yes, it can multiply brightness values by a constant. It can also do morse code on its own, and compute prime numbers. Why is this useful addition to kernel<->user interface? Can we do better interface that works on more than one chip? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in