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