[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<TY3PR01MB11346C5187C5A0A3745B6F870869EA@TY3PR01MB11346.jpnprd01.prod.outlook.com>
Date: Thu, 29 Jan 2026 17:43:09 +0000
From: Biju Das <biju.das.jz@...renesas.com>
To: Conor Dooley <conor@...nel.org>, biju.das.au <biju.das.au@...il.com>
CC: "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
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.
Cheers,
Biju
Powered by blists - more mailing lists