OSDN Git Service

dt-bindings: dsa: Remove unnecessary #address/#size-cells
authorFabio Estevam <fabio.estevam@nxp.com>
Mon, 7 May 2018 12:17:51 +0000 (09:17 -0300)
committerDavid S. Miller <davem@davemloft.net>
Wed, 9 May 2018 00:28:44 +0000 (20:28 -0400)
If the example binding is used on a real dts file, the following DTC
warning is seen with W=1:

arch/arm/boot/dts/imx6q-b450v3.dtb: Warning (avoid_unnecessary_addr_size): /mdio-gpio/switch@0: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property

Remove unnecessary #address-cells/#size-cells to improve the binding
document examples.

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Documentation/devicetree/bindings/net/dsa/dsa.txt

index cfe8f64..3ceeb8d 100644 (file)
@@ -82,8 +82,6 @@ linked into one DSA cluster.
 
        switch0: switch0@0 {
                compatible = "marvell,mv88e6085";
-               #address-cells = <1>;
-               #size-cells = <0>;
                reg = <0>;
 
                dsa,member = <0 0>;
@@ -135,8 +133,6 @@ linked into one DSA cluster.
 
        switch1: switch1@0 {
                compatible = "marvell,mv88e6085";
-               #address-cells = <1>;
-               #size-cells = <0>;
                reg = <0>;
 
                dsa,member = <0 1>;
@@ -204,8 +200,6 @@ linked into one DSA cluster.
 
        switch2: switch2@0 {
                compatible = "marvell,mv88e6085";
-               #address-cells = <1>;
-               #size-cells = <0>;
                reg = <0>;
 
                dsa,member = <0 2>;