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

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: 
 <TY7PR01MB148181E9FA1336D0D3CE9275A92D12@TY7PR01MB14818.jpnprd01.prod.outlook.com>
Date: Tue, 11 Mar 2025 02:51:37 +0000
From: <nobuhiro1.iwamatsu@...hiba.co.jp>
To: <prabhakar.csengg@...il.com>, <andrew+netdev@...n.ch>,
        <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>,
        <pabeni@...hat.com>, <robh@...nel.org>, <krzk+dt@...nel.org>,
        <conor+dt@...nel.org>, <neil.armstrong@...aro.org>,
        <khilman@...libre.com>, <jbrunet@...libre.com>,
        <martin.blumenstingl@...glemail.com>, <shawnguo@...nel.org>,
        <s.hauer@...gutronix.de>, <kernel@...gutronix.de>,
        <festevam@...il.com>, <heiko@...ech.de>, <mcoquelin.stm32@...il.com>,
        <alexandre.torgue@...s.st.com>, <matthias.bgg@...il.com>,
        <angelogioacchino.delregno@...labora.com>,
        <vineetha.g.jaya.kumaran@...el.com>, <biao.huang@...iatek.com>,
        <xiaoning.wang@....com>, <linux-imx@....com>,
        <david.wu@...k-chips.com>, <christophe.roullier@...s.st.com>,
        <rmk+kernel@...linux.org.uk>, <netdev@...r.kernel.org>
CC: <devicetree@...r.kernel.org>, <linux-amlogic@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>, <imx@...ts.linux.dev>,
        <linux-rockchip@...ts.infradead.org>,
        <linux-stm32@...md-mailman.stormreply.com>,
        <linux-mediatek@...ts.infradead.org>,
        <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: RE: [PATCH net-next] dt-bindings: net: Define interrupt constraints
 for DWMAC vendor bindings

Hi,

> -----Original Message-----
> From: Prabhakar <prabhakar.csengg@...il.com>
> Sent: Sunday, March 9, 2025 9:33 AM
> To: 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>; Rob Herring
> <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley
> <conor+dt@...nel.org>; Neil Armstrong <neil.armstrong@...aro.org>; Kevin
> Hilman <khilman@...libre.com>; Jerome Brunet <jbrunet@...libre.com>;
> Martin Blumenstingl <martin.blumenstingl@...glemail.com>; Shawn Guo
> <shawnguo@...nel.org>; Sascha Hauer <s.hauer@...gutronix.de>;
> Pengutronix Kernel Team <kernel@...gutronix.de>; Fabio Estevam
> <festevam@...il.com>; Heiko Stuebner <heiko@...ech.de>; Maxime
> Coquelin <mcoquelin.stm32@...il.com>; Alexandre Torgue
> <alexandre.torgue@...s.st.com>; iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ
> □ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@...hiba.co.jp>; Matthias Brugger
> <matthias.bgg@...il.com>; AngeloGioacchino Del Regno
> <angelogioacchino.delregno@...labora.com>; G. Jaya Kumaran
> <vineetha.g.jaya.kumaran@...el.com>; Biao Huang
> <biao.huang@...iatek.com>; Clark Wang <xiaoning.wang@....com>; Linux
> Team <linux-imx@....com>; David Wu <david.wu@...k-chips.com>;
> Christophe Roullier <christophe.roullier@...s.st.com>; Russell King (Oracle)
> <rmk+kernel@...linux.org.uk>; netdev@...r.kernel.org
> Cc: devicetree@...r.kernel.org; linux-amlogic@...ts.infradead.org;
> linux-kernel@...r.kernel.org; imx@...ts.linux.dev;
> linux-rockchip@...ts.infradead.org;
> linux-stm32@...md-mailman.stormreply.com;
> linux-mediatek@...ts.infradead.org; Prabhakar
> <prabhakar.csengg@...il.com>; Lad Prabhakar
> <prabhakar.mahadev-lad.rj@...renesas.com>
> Subject: [PATCH net-next] dt-bindings: net: Define interrupt constraints for
> DWMAC vendor bindings
> 
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@...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@...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@...renesas.com/
> 
> Cheers, Prabhakar
> ---

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

Acked-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@...hiba.co.jp>

Best regards,
  Nobuhiro