Re: [PATCH v3 1/2] dt-bindings: leds: add cznic,turris-omnia-leds binding

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

 



On Sun, 12 Jul 2020 23:27:07 +0200
Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx> wrote:

> > +            multi-led@0 {
> > +                reg = <0>;
> > +                color = <LED_COLOR_ID_MULTI>;
> > +                function = LED_FUNCTION_POWER;  
> 
> Please provide child nodes for each color LED. Let's stick
> to the bindings closely and not make any deviations from
> the beginning.

Why? It would make sense if there were devices using this controller
having other configuration, but on Omnia, all LEDs are RGB.

Also, if I do this, should I also make the driver check in the probe
function whether the per-channel child nodes are correct? Eg. if they
are always three: one for red, one for green and one for blue? Or
should the driver ignore this and only the device tree binding specify
it?

Because the way the driver is written now, it only registers
multi-color RGB LEDs.

Marek



[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