Commit c8f1f2e9 authored Apr 04, 2023 by Rob Herring Committed by Jakub Kicinski Apr 05, 2023

dt-bindings: net: ethernet-switch: Make "#address-cells/#size-cells" required



The schema doesn't allow for a single (unaddressed) ethernet port node
nor does a single port switch make much sense. So if there's always
multiple child nodes, "#address-cells" and "#size-cells" should be
required.

Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Link: https://lore.kernel.org/r/20230404204213.635773-1-robh@kernel.org


Signed-off-by: Jakub Kicinski <kuba@kernel.org>

parent acd11255

Documentation/devicetree/bindings/net/ethernet-switch.yaml

+4 −0

Original line number	Diff line number	Diff line
		@@ -40,6 +40,10 @@ patternProperties:
		type: object
		description: Ethernet switch ports

		required:
		- "#address-cells"
		- "#size-cells"

		oneOf:
		- required:
		- ports

Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml

+3 −1

Original line number	Diff line number	Diff line
		@@ -53,6 +53,8 @@ examples:
		reg = <0x10>;

		ports {
		#address-cells = <1>;
		#size-cells = <0>;
		/* ... */
		};
		};