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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 9 Oct 2022 02:35:46 +0300 From: Vladimir Oltean <olteanv@...il.com> To: Colin Foster <colin.foster@...advantage.com> Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, Russell King <linux@...linux.org.uk>, UNGLinuxDriver@...rochip.com, Alexandre Belloni <alexandre.belloni@...tlin.com>, Claudiu Manoil <claudiu.manoil@....com>, Paolo Abeni <pabeni@...hat.com>, Jakub Kicinski <kuba@...nel.org>, Eric Dumazet <edumazet@...gle.com>, "David S. Miller" <davem@...emloft.net>, Florian Fainelli <f.fainelli@...il.com>, Vivien Didelot <vivien.didelot@...il.com>, Andrew Lunn <andrew@...n.ch>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Rob Herring <robh+dt@...nel.org>, Lee Jones <lee@...nel.org> Subject: Re: [RFC v4 net-next 00/17] add support for the the vsc7512 internal copper phys On Sat, Oct 08, 2022 at 11:51:35AM -0700, Colin Foster wrote: > The only warning I know about is about the reg array being too long: > /Documentation/devicetree/bindings/net/dsa/mscc,ocelot.example.dtb: soc@0: ethernet-switch@0:reg: [[1895890944, 65536], [1896022016, 65536], [1896349696, 256], [1896742912, 65536], [1897791488, 256], [1897857024, 256], [1897922560, 256], [1897988096, 256], [1898053632, 256], [1898119168, 256], [1898184704, 256], [1898250240, 256], [1898315776, 256], [1898381312, 256], [1898446848, 256], [1904214016, 524288], [1904738304, 65536], [1896087552, 65536], [1896153088, 65536], [1896218624, 65536]] is too long > > Also I know there is still ongoing discussion regarding what the documentation > should reflect. I hope I'm not getting ahead of myself by submitting this > RFC at this time. This is because 'reg' is longer than the schema was told it would be. net/dsa/mscc,ocelot.yaml defines 'reg' with maxItems: 1, whereas net/mscc,vsc7514-switch.yaml lists out all targets. At this stage I wonder whether it wouldn't be better for you to patch up net/mscc,vsc7514-switch.yaml with support for your compatible string and reference dsa.yaml, rather than net/dsa/mscc,ocelot.yaml which should perhaps be rebranded to make it clear it only applies to NXP integrations of Microchip switches. I also expressed something along these lines yesterday https://lore.kernel.org/netdev/20221007231009.qgcirfezgib5vu6y@skbuf/ and I'd wait for a little bit more, to give Krzysztof a chance to respond and share his thoughts.
Powered by blists - more mailing lists