Thread (4 messages) 4 messages, 4 authors, 2025-03-13

RE: [PATCH net-next] dt-bindings: net: Define interrupt constraints for DWMAC vendor bindings

From: <hidden>
Date: 2025-03-11 02:53:56
Also in: imx, linux-amlogic, linux-devicetree, linux-mediatek, linux-rockchip, lkml

Hi,
-----Original Message-----
From: Prabhakar <prabhakar.csengg@gmail.com>
Sent: Sunday, March 9, 2025 9:33 AM
To: Andrew Lunn <andrew+netdev@lunn.ch>; David S. Miller
[off-list ref]; Eric Dumazet [off-list ref]; Jakub
Kicinski [off-list ref]; Paolo Abeni [off-list ref]; Rob Herring
[off-list ref]; Krzysztof Kozlowski [off-list ref]; Conor Dooley
[off-list ref]; Neil Armstrong [off-list ref]; Kevin
Hilman [off-list ref]; Jerome Brunet [off-list ref];
Martin Blumenstingl [off-list ref]; Shawn Guo
[off-list ref]; Sascha Hauer [off-list ref];
Pengutronix Kernel Team [off-list ref]; Fabio Estevam
[off-list ref]; Heiko Stuebner [off-list ref]; Maxime
Coquelin [off-list ref]; Alexandre Torgue
[off-list ref]; iwamatsu nobuhiro(岩松 信洋 ○DITC
□DIT○OST) [off-list ref]; Matthias Brugger
[off-list ref]; AngeloGioacchino Del Regno
[off-list ref]; G. Jaya Kumaran
[off-list ref]; Biao Huang
[off-list ref]; Clark Wang [off-list ref]; Linux
Team [off-list ref]; David Wu [off-list ref];
Christophe Roullier [off-list ref]; Russell King (Oracle)
[off-list ref]; netdev@vger.kernel.org
Cc: devicetree@vger.kernel.org; linux-amlogic@lists.infradead.org;
linux-kernel@vger.kernel.org; imx@lists.linux.dev;
linux-rockchip@lists.infradead.org;
linux-stm32@st-md-mailman.stormreply.com;
linux-mediatek@lists.infradead.org; Prabhakar
[off-list ref]; Lad Prabhakar
[off-list ref]
Subject: [PATCH net-next] dt-bindings: net: Define interrupt constraints for
DWMAC vendor bindings

From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

The `snps,dwmac.yaml` binding currently sets `maxItems: 3` for the
`interrupts` and `interrupt-names` properties, but vendor bindings selecting
`snps,dwmac.yaml` do not impose these limits.

Define constraints for `interrupts` and `interrupt-names` properties in
various DWMAC vendor bindings to ensure proper validation and consistency.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
Hi All,

Based on recent patch [0] which increases the interrupts to 11 and adds
`additionalItems: true` its good to have constraints to validate the schema. Ive
made the changes based on the DT binding doc and the users. Ive ran dt
binding checks to ensure the constraints are valid. Please let me know if you'd
like me to split this patch or if any of the constraints are incorrect, as I don't
have documentation for all of these platforms.

https://lore.kernel.org/all/20250308200921.1089980-2-prabhakar.mahadev-la
d.rj@bp.renesas.com/

Cheers, Prabhakar
---
 .../bindings/net/toshiba,visconti-dwmac.yaml           |  6 ++++++
Acked-by: Nobuhiro Iwamatsu <redacted>

Best regards,
  Nobuhiro
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help