Hi Krzysztof,
On Tue, 30 Jan 2024 at 17:49, Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> wrote:
>
> On 30/01/2024 13:08, Naresh Solanki wrote:
> > Update maxItems to 60 for gpio-reserved-ranges to allow multiple gpio
> > reserved ranges.
> > Add input-enable property to allow configuring a pin as input.
> > Rearrange allOf
>
> Why?
allOf is aligned with other implementations.
>
> > Update example.
> >
> > Signed-off-by: Naresh Solanki <naresh.solanki@xxxxxxxxxxxxx>
> > ---
> > .../bindings/pinctrl/cypress,cy8c95x0.yaml | 28 +++++++++++++++----
> > 1 file changed, 23 insertions(+), 5 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > index 7f30ec2f1e54..89ce0cb68834 100644
> > --- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > +++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > @@ -45,7 +45,8 @@ properties:
> > maxItems: 1
> >
> > gpio-reserved-ranges:
> > - maxItems: 1
> > + minItems: 1
> > + maxItems: 60
> >
> > vdd-supply:
> > description:
> > @@ -85,6 +86,8 @@ patternProperties:
> >
> > bias-disable: true
> >
> > + input-enable: true
> > +
> > output-high: true
> >
> > output-low: true
> > @@ -101,6 +104,9 @@ patternProperties:
> >
> > additionalProperties: false
> >
> > +allOf:
> > + - $ref: pinctrl.yaml#
> > +
>
> That's not a correct placement.
ok. Will remove this specific change.
>
> > required:
> > - compatible
> > - reg
> > @@ -112,9 +118,6 @@ required:
> >
> > additionalProperties: false
> >
> > -allOf:
> > - - $ref: pinctrl.yaml#
> > -
> > examples:
> > - |
> > #include <dt-bindings/interrupt-controller/arm-gic.h>
> > @@ -133,6 +136,21 @@ examples:
> > interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> > interrupt-controller;
> > vdd-supply = <&p3v3>;
> > - gpio-reserved-ranges = <5 1>;
> > + gpio-reserved-ranges = <1 2>, <6 1>, <10 1>, <15 1>;
> > +
> > + pinctrl-0 = <&U62160_pins>, <&U62160_ipins>;
> > + pinctrl-names = "default";
> > + U62160_pins: cfg-pins {
> > + pins = "gp03", "gp16", "gp20", "gp50", "gp51";
>
> Your indentation is totally broken.
sorry I missed checking on this before sending the patch for review. will fix
it to use two space for indentation.
Regards,
Naresh
>
>
> Best regards,
> Krzysztof
>
