linux-kernel - Re: [PATCH net-next v2 1/3] dt-bindings: net: dwmac: Increase 'maxItems' for 'interrupts' and 'interrupt-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: <CA+V-a8v=q1aN_+4jQz3=StAXYP1-==OX5xZDrGXe8WWR4XbCKQ@mail.gmail.com>
Date: Tue, 11 Mar 2025 08:50:34 +0000
From: "Lad, Prabhakar" <prabhakar.csengg@...il.com>
To: "Russell King (Oracle)" <rmk+kernel@...linux.org.uk>
Cc: Rob Herring <robh@...nel.org>, Andrew Lunn <andrew+netdev@...n.ch>, 
	"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, 
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Philipp Zabel <p.zabel@...gutronix.de>, Geert Uytterhoeven <geert+renesas@...der.be>, 
	Giuseppe Cavallaro <peppe.cavallaro@...com>, Jose Abreu <joabreu@...opsys.com>, 
	Alexandre Torgue <alexandre.torgue@...s.st.com>, netdev@...r.kernel.org, 
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, 
	linux-renesas-soc@...r.kernel.org, Biju Das <biju.das.jz@...renesas.com>, 
	Fabrizio Castro <fabrizio.castro.jz@...esas.com>, 
	Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: Re: [PATCH net-next v2 1/3] dt-bindings: net: dwmac: Increase
 'maxItems' for 'interrupts' and 'interrupt-names'

Hi Russell,

On Tue, Mar 11, 2025 at 7:16 AM Lad, Prabhakar
<prabhakar.csengg@...il.com> wrote:
>
> Hi Rob,
>
> Thank you for the review.
>
> On Mon, Mar 10, 2025 at 9:30 PM Rob Herring <robh@...nel.org> wrote:
> >
> > On Sat, Mar 08, 2025 at 08:09:19PM +0000, Prabhakar wrote:
> > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> > >
> > > Increase the `maxItems` value for the `interrupts` and `interrupt-names`
> > > properties to accommodate the Renesas RZ/V2H(P) SoC, which features the
> > > `snps,dwmac-5.20` IP with 11 interrupts.
> > >
> > > Also add `additionalItems: true` to allow specifying extra interrupts
> > > beyond the predefined ones. Update the `interrupt-names` property to
> > > allow specifying extra `interrupt-names`.
> > >
> > > Also refactor the optional `interrupt-names` property by consolidating
> > > repeated enums into a single enum list.
> > >
> > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> > > ---
> > > Note, for this change I will be sending a sperate patch for vendor
> > > bindings to add constraints.
> > >
> > > v1->v2
> > > - No change
> > > ---
> > >  Documentation/devicetree/bindings/net/snps,dwmac.yaml | 6 ++++--
> > >  1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> > > index 3f0aa46d798e..fad0d611a75c 100644
> > > --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> > > +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> > > @@ -114,6 +114,8 @@ properties:
> > >
> > >    interrupts:
> > >      minItems: 1
> > > +    maxItems: 11
> > > +    additionalItems: true
> > >      items:
> > >        - description: Combined signal for various interrupt events
> > >        - description: The interrupt to manage the remote wake-up packet detection
> > > @@ -122,11 +124,11 @@ properties:
> > >
> > >    interrupt-names:
> > >      minItems: 1
> > > +    maxItems: 11
> > > +    additionalItems: true
> > >      items:
> > >        - const: macirq
> > >        - enum: [eth_wake_irq, eth_lpi, sfty]
> > > -      - enum: [eth_wake_irq, eth_lpi, sfty]
> > > -      - enum: [eth_wake_irq, eth_lpi, sfty]
> >
> > I think this should be structured similar to the DWC PCIe binding where
> > we define all possible names, but not the order:
> >
> > minItems: 1
> > maxItems: 11
> > items:
> >   oneOf:
> >     - const: macirq
> >       description: ...
> >     - const: eth_wake_irq
> >       description: ...
> >     - pattern: '^rx-queue-[0-3]$'
> >       description: ...
> >     - pattern: '^tx-queue-[0-3]$'
> >       description: ...
> >
> > And so on. Move the descriptions from 'interrupts' and drop 'items' and
> > 'additionalItems' from it.
> >
> Thanks for the pointer, I'll do as suggested above.
>
> @Russel, are you OK from me to add rx-queue/tx-queue in this binding
Apologies for the typo in your name.

Cheers,
Prabhakar