RE: [PATCH v7 1/2] dt-bindings: leds: Add LTC3220 18 channel LED Driver

[Escala, Edelweise]

Hi Rob,

Thank you for the review

> > +$id:
> > +https://urldefense.com/v3/__http://devicetree.org/schemas/leds/adi,lt
> >
> +c3220.yaml*__;Iw!!A3Ni8CS0y2Y!4aXJppTOoC_2MFbECSJDI6G6jFfk0Mzl1jxDj-3
> > +eTsiwvSa_l0oZwoalOp3RV4_m2HYvJ1lOmwObA2BoJA$
> > +$schema:
> > +https://urldefense.com/v3/__http://devicetree.org/meta-schemas/core.y
> > +aml*__;Iw!!A3Ni8CS0y2Y!4aXJppTOoC_2MFbECSJDI6G6jFfk0Mzl1jxDj-
> 3eTsiwvS
> > +a_l0oZwoalOp3RV4_m2HYvJ1lOmwMCRVBFDg$
> > +
> > +title: Analog Devices LTC3220 LED Driver
> > +
> > +maintainers:
> > +  - Edelweise Escala <edelweise.escala@xxxxxxxxxx>
> > +
> > +description:
> 
> You need a '>' to maintain paragraphs.
> 

I will add '>' for the upcoming change

> > +  The LTC3220 is a multi-display LED driver, which contains a
> > + high-efficiency,  low-noise charge pump to provide power to up to 18 LED
> current sources.
> > +  The LEDs are individually configurable to 64-step linear brightness
> > + control,  blinking and gradation control via 2-wire I2C interface.
> > +
> > +  For more product information please see the link below
> > +    https://www.analog.com/en/products/ltc3220.html
> > +
> > +properties:
> > +  compatible:
> > +    const: adi,ltc3220
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  '#address-cells':
> > +    const: 1
> > +
> > +  '#size-cells':
> > +    const: 0
> > +
> > +  reset-gpios:
> > +    maxItems: 1
> > +
> > +patternProperties:
> > +  '^led@([1-9]|1[0-8])$':
> 
> Unit-addresses are hex.
> 

Will fix unit address to be hex.

Best Regards,
Edelweise Escala