Re: [PATCH v13 04/10] dt-bindings: mfd: add maxim,max77705

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

 



пн, 23 дек. 2024 г. в 18:52, Krzysztof Kozlowski <krzk@xxxxxxxxxx>:
(...)
> > +
> > +  leds:
> > +    type: object
> > +    additionalProperties: false
> > +    description:
> > +      Up to 4 LED channels supported.
> > +
> > +    patternProperties:
> > +      "^led@[0-3]$":
>
> 0-2? 1-3? Preferred is the first, because we index unit addresses from
> 0.

max77705 supports 4 led channels, i.e. rgbw, hence 0-3.

>
> Please move "patternProperties" after the "properties:" block.
> Compatible should be the first thing visible to the reader.
>

ok

(...)
> > +
> > +        patternProperties:
> > +          "^led@[0-3]$":
>
> Same - 0-2
>

max77705 supports 4 led channels, i.e. rgbw, hence 0-3.

-- 
Best regards and thanks for review,
Dzmitry





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux