Re: [PATCH v1 leds-next 0/3] Add Turris 1.x LED controller support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

> this adds support for the LED controller of the 8 RGB LEDs on the front
> side of CZ.NIC's Turris 1.x router.
> 
> This is the first version, please review, especially my solution of the
> HW trigger problem.
> 
> At first I wanted to create a classic led-trigger for the HW triggers
> for these LEDs, but because of the RGB model I went with the solution where
> there are three separate LEDs (R, G and B) for each RGB LED, and the HW
> trigger controls the whole RGB group, so setting HW trigger on one channel
> should set it on all three channels.
> 
> Another solution would be to create 8 LEDs, where their color could be
> changed via a special attribute file named "color", and a classic led-trigger
> could be used for implementing the HW trigger. This solution though would
> not allow to use kernel SW triggers for the each color channel
> separately.

So... there was extensive discussion abour rgb leds. You may want to
look at thread "leds: lp5024: Add the LP5024/18 RGB LED..."

Currently, we use single device per LED channel. If you want driver in
quickly, just do that.

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

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux