Skip to content

Commit

Permalink
dt-bindings: dsa: Remove unnecessary #address/#size-cells
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Fabio Estevam authored and David S. Miller committed May 9, 2018
1 parent 2b999ba commit 53a7bdf
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions Documentation/devicetree/bindings/net/dsa/dsa.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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>;
Expand Down Expand Up @@ -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>;
Expand Down Expand Up @@ -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>;
Expand Down

0 comments on commit 53a7bdf

Please sign in to comment.