RE: [PATCH RESEND 1/9] dt-bindings: pinctrl: renesas: Document reset-names
From: Biju Das
Date: Thu Jan 29 2026 - 12:43:26 EST
Hi Conor,
Thanks for the feedback.
> -----Original Message-----
> From: Conor Dooley <conor@xxxxxxxxxx>
> 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@xxxxxxxxxxxxxx>
> >
> > All SoCs has multiple resets. Document reset-names property.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> > .../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.
Cheers,
Biju