Re: [PATCH 2/2] leds: lp5024: Add the LP5024/18 RGB LED driver

[Pavel Machek]

Hi!

> >>With the "color" sysfs file it will make more sense to allow for user
> >>defined color palettes.
> >>
> >
> >I think defining these values in the device tree or acpi severely limits the devices
> >capabilities.  Especially in development phases.  If the knobs were exposed then the user space
> >can create new experiences.  The color definition should be an absolute color defined in the dt and
> >either the framework or user space needs to mix these appropriately.  IMO user space should set the policy
> >of the user experience and the dt/acpi needs to set the capabilities.
> >
> >I do like Pavels idea on defining the more standard binding pattern to "group" leds into a single group.
> >
> >Maybe the framework could take these groups and combine/group them into a single node with the groups colors.
> 
> There is still HSV approach [0] in store. One problem with proposed
> implementation is fixed algorithm of RGB <-> HSV color space conversion.
> Maybe allowing for some board specific adjustments in DT would add
> more flexibility.
> 
> [0] https://lkml.org/lkml/2017/8/31/255

Yes we could do HSV. Problem is that that we do not really have RGB
available. We do have integers for red, green and blue, but they do
not correspond to RGB colorspace.

Anyway, this should not be driver specific; all drivers should use one
interface.

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