On Mon, Oct 06, 2008 at 02:51:03PM +0300, Felipe Balbi wrote: > On Mon, Oct 06, 2008 at 02:17:23PM +0300, Tony Lindgren wrote: > > * Riku Voipio <riku.voipio@xxxxxx> [081006 14:09]: > > > On Mon, Oct 06, 2008 at 12:11:31PM +0300, Felipe Balbi wrote: > > > > This driver should be sitting together with the other > > > > led drivers. > > > > > > But this driver actually doesn't implement the led sysfs interface. > > > If the driver is changed to implement the led framework interface, > > > we break the existing n810 userland and expose only limited subset > > > of lp5521 features for userland.. > > > > Maybe this feature should be a distro specific hack on top of > > the mainline tree? > > the problem is that led api (afaict) doesn't really have abstraction for > rgb leds like the one we use. It can only handle blinky leds and on/off > transitions and we need more functionality. RGB can somewhat be handled, by creating three led devices: n810:red:foo n810:gree:foo n810:blue:foo blinking could also be done, but that would be suboptimal. Mainly we would lose all the cool fading etc stuff than can be programmed to run on lp5521 without main cpu intervention.. > This driver will probably loop around for a while until we find out that > led api needs to be extended. Or implement a minimal led api (off/brightness) as well as keep the custom interface for time being, if that's ok with Richard (the led subsystem maintainer). -- "rm -rf" only sounds scary if you don't have backups -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
