Re: [PATCH RESEND 1/9] dt-bindings: pinctrl: renesas: Document reset-names

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2537 bytes --]

On Thu, Jan 29, 2026 at 05:43:09PM +0000, Biju Das wrote:
> Hi Conor,
> 
> Thanks for the feedback.
> 
> > -----Original Message-----
> > From: Conor Dooley <conor@kernel.org>
> > Sent: 29 January 2026 17:41
> > Subject: Re: [PATCH RESEND 1/9] dt-bindings: pinctrl: renesas: Document reset-names
> > 
> > On Thu, Jan 29, 2026 at 10:16:36AM +0000, Biju wrote:
> > > From: Biju Das <biju.das.jz@bp.renesas.com>
> > >
> > > All SoCs has multiple resets. Document reset-names property.
> > >
> > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > > ---
> > >  .../bindings/pinctrl/renesas,rzg2l-pinctrl.yaml   | 15 +++++++++++++++
> > >  arch/arm64/boot/dts/renesas/r9a07g043.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a07g044.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a07g054.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a08g045.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a09g047.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a09g056.dtsi        |  1 +
> > >  arch/arm64/boot/dts/renesas/r9a09g057.dtsi        |  1 +
> > >  8 files changed, 22 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> > b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> > > index 00c05243b9a4..fbbba53cde9b 100644
> > > --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> > > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> > > @@ -78,6 +78,16 @@ properties:
> > >            - description: PFC main reset
> > >            - description: Reset for the control register related to WDTUDFCA and WDTUDFFCM pins
> > >
> > > +  reset-names:
> > > +    oneOf:
> > > +      - items:
> > > +          - const: rstn
> > > +          - const: port
> > > +          - const: spare
> > > +      - items:
> > > +          - const: main
> > > +          - const: error
> > > +
> > >  additionalProperties:
> > >    anyOf:
> > >      - type: object
> > > @@ -152,10 +162,14 @@ allOf:
> > >        properties:
> > >          resets:
> > >            maxItems: 2
> > > +        reset-names:
> > > +          maxItems: 2
> > >      else:
> > >        properties:
> > >          resets:
> > >            minItems: 3
> > > +        reset-names:
> > > +          maxItems: 3
> > 
> > This is minItems, no?
> 
> Oops, Will fix it in next version.

w/ minItems
Acked-by: Conor Dooley <conor.dooley@microchip.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]