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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <TY3PR01MB11346221862DCD11BB6D6FFA0869EA@TY3PR01MB11346.jpnprd01.prod.outlook.com>
Date: Thu, 29 Jan 2026 18:09:16 +0000
From: Biju Das <biju.das.jz@...renesas.com>
To: Conor Dooley <conor@...nel.org>
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

Hi Conor,

> -----Original Message-----
> From: Conor Dooley <conor@...nel.org>
> Sent: 29 January 2026 17:47
> 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-pinctr
> > > > > l.yaml
> > > > b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.
> > > > yaml
> > > > > index 00c05243b9a4..fbbba53cde9b 100644
> > > > > ---
> > > > > a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctr
> > > > > l.yaml
> > > > > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pi
> > > > > +++ nctrl.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.

OK, I will make it separate in next version

Cheers,
Biju