Hi!
> > Hi!
> > 
> > > + led-controller@45 {
> > > +         compatible = "awinic,aw2013";
> > > +         reg = <0x45>;
> > > +         #address-cells = <1>;
> > > +         #size-cells = <0>;
> > > +
> > > +         vcc-supply = <&pm8937_l10>;
> > > +         vio-supply = <&pm8937_l5>;
> > > +
> > > +         led@0 {
> > > +                 reg = <0>;
> > > +                 function = LED_FUNCTION_STATUS;
> > > +                 led-max-microamp = <5000>;
> > > +                 color = <LED_COLOR_ID_RED>;
> > > +         };
> > > +
> > > +         led@1 {
> > > +                 reg = <1>;
> > > +                 function = LED_FUNCTION_STATUS;
> > > +                 led-max-microamp = <5000>;
> > > +                 color = <LED_COLOR_ID_GREEN>;
> > > +         };
> > > +
> > > +         led@2 {
> > > +                 reg = <2>;
> > > +                 function = LED_FUNCTION_STATUS;
> > > +                 led-max-microamp = <5000>;
> > > +                 color = <LED_COLOR_ID_BLUE>;
> > > +         };
> > > + };
> > > +};
> > 
> > That's single, 3-color LED, right? Please see LED multicolor support.
> As far as i know aw2013 driver does not have multicolor support.
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/leds/leds-aw2013.yaml

I believe that needs to be fixed before more bugs are added on top to
work around that problem...

...and before that bug is cemented in the ABI.

Best regards,
                                                                Pavel
-- 
I don't work for Nazis and criminals, and neither should you.
Boycott Putin, Trump, Netanyahu and Musk!

Attachment: signature.asc
Description: PGP signature

Reply via email to