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

[nobuhiro1.iwamatsu at toshiba.co.jp]

Hi,

> -----Original Message-----
> From: Prabhakar <prabhakar.csengg at gmail.com>
> Sent: Sunday, March 9, 2025 9:33 AM
> To: Andrew Lunn <andrew+netdev at lunn.ch>; David S. Miller
> <davem at davemloft.net>; Eric Dumazet <edumazet at google.com>; Jakub
> Kicinski <kuba at kernel.org>; Paolo Abeni <pabeni at redhat.com>; Rob Herring
> <robh at kernel.org>; Krzysztof Kozlowski <krzk+dt at kernel.org>; Conor Dooley
> <conor+dt at kernel.org>; Neil Armstrong <neil.armstrong at linaro.org>; Kevin
> Hilman <khilman at baylibre.com>; Jerome Brunet <jbrunet at baylibre.com>;
> Martin Blumenstingl <martin.blumenstingl at googlemail.com>; Shawn Guo
> <shawnguo at kernel.org>; Sascha Hauer <s.hauer at pengutronix.de>;
> Pengutronix Kernel Team <kernel at pengutronix.de>; Fabio Estevam
> <festevam at gmail.com>; Heiko Stuebner <heiko at sntech.de>; Maxime
> Coquelin <mcoquelin.stm32 at gmail.com>; Alexandre Torgue
> <alexandre.torgue at foss.st.com>; iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ
> □ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu at toshiba.co.jp>; Matthias Brugger
> <matthias.bgg at gmail.com>; AngeloGioacchino Del Regno
> <angelogioacchino.delregno at collabora.com>; G. Jaya Kumaran
> <vineetha.g.jaya.kumaran at intel.com>; Biao Huang
> <biao.huang at mediatek.com>; Clark Wang <xiaoning.wang at nxp.com>; Linux
> Team <linux-imx at nxp.com>; David Wu <david.wu at rock-chips.com>;
> Christophe Roullier <christophe.roullier at foss.st.com>; Russell King (Oracle)
> <rmk+kernel at armlinux.org.uk>; netdev at vger.kernel.org
> Cc: devicetree at vger.kernel.org; linux-amlogic at lists.infradead.org;
> linux-kernel at vger.kernel.org; imx at lists.linux.dev;
> linux-rockchip at lists.infradead.org;
> linux-stm32 at st-md-mailman.stormreply.com;
> linux-mediatek at lists.infradead.org; Prabhakar
> <prabhakar.csengg at gmail.com>; Lad Prabhakar
> <prabhakar.mahadev-lad.rj at bp.renesas.com>
> Subject: [PATCH net-next] dt-bindings: net: Define interrupt constraints for
> DWMAC vendor bindings
> 
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj at 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 at 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 at bp.renesas.com/
> 
> Cheers, Prabhakar
> ---

>  .../bindings/net/toshiba,visconti-dwmac.yaml           |  6 ++++++

Acked-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu at toshiba.co.jp>

Best regards,
  Nobuhiro