[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260129-cherub-prenatal-7882029a9027@spud>
Date: Thu, 29 Jan 2026 17:46:46 +0000
From: Conor Dooley <conor@...nel.org>
To: Biju Das <biju.das.jz@...renesas.com>
Cc: "biju.das.au" <biju.das.au@...il.com>,
"linux-renesas-soc@...r.kernel.org" <linux-renesas-soc@...r.kernel.org>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Linus Walleij <linusw@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
"magnus.damm" <magnus.damm@...il.com>,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...renesas.com>,
"linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>
Subject: Re: [PATCH RESEND 1/9] dt-bindings: pinctrl: renesas: Document
reset-names
On Thu, Jan 29, 2026 at 05:46:01PM +0000, Conor Dooley wrote:
> 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@...nel.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@...renesas.com>
> > > >
> > > > All SoCs has multiple resets. Document reset-names property.
> > > >
> > > > Signed-off-by: Biju Das <biju.das.jz@...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@...rochip.com>
Actually, can you remove the dtsi changes from the binding patch? I
forgot they were in here.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists