linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
@ 2020-04-09 20:24 Rob Herring
  2020-04-09 20:32 ` Guenter Roeck
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Rob Herring @ 2020-04-09 20:24 UTC (permalink / raw)
  To: devicetree
  Cc: linux-kernel, Maxime Ripard, Chen-Yu Tsai, Nuno Sá,
	Jean Delvare, Guenter Roeck, Thierry Reding, Jonathan Hunter,
	Lee Jones, Bjorn Andersson, David S. Miller, Matt Mackall,
	Herbert Xu, Nicolas Saenz Julienne, Florian Fainelli, Ray Jui,
	Scott Branden, bcm-kernel-feedback-list, Mark Brown, linux-hwmon,
	linux-tegra, linux-arm-msm, netdev, linux-crypto,
	linux-rpi-kernel, linux-spi, linux-usb

A recent update to dtc and changes to the default warnings introduced
some new warnings in the DT binding examples:

Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.example.dts:23.13-61:
 Warning (dma_ranges_format): /example-0/dram-controller@1c01000:dma-ranges: "dma-ranges" property has invalid length (12 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1)
Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.example.dts:17.22-28.11:
 Warning (unit_address_vs_reg): /example-0/fpga-axi@0: node has a unit name, but no reg or ranges property
Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dts:34.13-54:
 Warning (dma_ranges_format): /example-0/memory-controller@2c00000:dma-ranges: "dma-ranges" property has invalid length (24 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 2)
Documentation/devicetree/bindings/mfd/st,stpmic1.example.dts:19.15-79.11:
 Warning (unit_address_vs_reg): /example-0/i2c@0: node has a unit name, but no reg or ranges property
Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.example.dts:28.23-31.15:
 Warning (unit_address_vs_reg): /example-0/mdio@37000000/switch@10: node has a unit name, but no reg or ranges property
Documentation/devicetree/bindings/rng/brcm,bcm2835.example.dts:17.5-21.11:
 Warning (unit_address_vs_reg): /example-0/rng: node has a reg or ranges property, but no unit name
Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.example.dts:20.20-43.11:
 Warning (unit_address_vs_reg): /example-0/soc@0: node has a unit name, but no reg or ranges property
Documentation/devicetree/bindings/usb/ingenic,musb.example.dts:18.28-21.11:
 Warning (unit_address_vs_reg): /example-0/usb-phy@0: node has a unit name, but no reg or ranges property

Cc: Maxime Ripard <mripard@kernel.org>
Cc: Chen-Yu Tsai <wens@csie.org>
Cc: "Nuno Sá" <nuno.sa@analog.com>
Cc: Jean Delvare <jdelvare@suse.com>
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Jonathan Hunter <jonathanh@nvidia.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Matt Mackall <mpm@selenic.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Ray Jui <rjui@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: Mark Brown <broonie@kernel.org>
Cc: linux-hwmon@vger.kernel.org
Cc: linux-tegra@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org
Cc: netdev@vger.kernel.org
Cc: linux-crypto@vger.kernel.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-spi@vger.kernel.org
Cc: linux-usb@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
Will take this via the DT tree.

Rob

 .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |  6 +++
 .../bindings/hwmon/adi,axi-fan-control.yaml   |  2 +-
 .../nvidia,tegra186-mc.yaml                   | 41 +++++++++++--------
 .../devicetree/bindings/mfd/st,stpmic1.yaml   |  2 +-
 .../bindings/net/qcom,ipq8064-mdio.yaml       |  1 +
 .../devicetree/bindings/rng/brcm,bcm2835.yaml |  2 +-
 .../bindings/spi/qcom,spi-qcom-qspi.yaml      |  2 +-
 .../devicetree/bindings/usb/ingenic,musb.yaml |  2 +-
 8 files changed, 35 insertions(+), 23 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
index aa0738b4d534..e713a6fe4cf7 100644
--- a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
@@ -42,6 +42,10 @@ properties:
     description:
       See section 2.3.9 of the DeviceTree Specification.
 
+  '#address-cells': true
+
+  '#size-cells': true
+
 required:
   - "#interconnect-cells"
   - compatible
@@ -59,6 +63,8 @@ examples:
         compatible = "allwinner,sun5i-a13-mbus";
         reg = <0x01c01000 0x1000>;
         clocks = <&ccu CLK_MBUS>;
+        #address-cells = <1>;
+        #size-cells = <1>;
         dma-ranges = <0x00000000 0x40000000 0x20000000>;
         #interconnect-cells = <1>;
     };
diff --git a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
index 57a240d2d026..29bb2c778c59 100644
--- a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
+++ b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
@@ -47,7 +47,7 @@ required:
 
 examples:
   - |
-    fpga_axi: fpga-axi@0 {
+    fpga_axi: fpga-axi {
             #address-cells = <0x2>;
             #size-cells = <0x1>;
 
diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
index 12516bd89cf9..611bda38d187 100644
--- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
@@ -97,30 +97,35 @@ examples:
     #include <dt-bindings/clock/tegra186-clock.h>
     #include <dt-bindings/interrupt-controller/arm-gic.h>
 
-    memory-controller@2c00000 {
-        compatible = "nvidia,tegra186-mc";
-        reg = <0x0 0x02c00000 0x0 0xb0000>;
-        interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
-
+    bus {
         #address-cells = <2>;
         #size-cells = <2>;
 
-        ranges = <0x0 0x02c00000 0x02c00000 0x0 0xb0000>;
+        memory-controller@2c00000 {
+            compatible = "nvidia,tegra186-mc";
+            reg = <0x0 0x02c00000 0x0 0xb0000>;
+            interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
+
+            #address-cells = <2>;
+            #size-cells = <2>;
+
+            ranges = <0x0 0x02c00000 0x0 0x02c00000 0x0 0xb0000>;
 
-        /*
-         * Memory clients have access to all 40 bits that the memory
-         * controller can address.
-         */
-        dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
+            /*
+             * Memory clients have access to all 40 bits that the memory
+             * controller can address.
+             */
+            dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
 
-        external-memory-controller@2c60000 {
-            compatible = "nvidia,tegra186-emc";
-            reg = <0x0 0x02c60000 0x0 0x50000>;
-            interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
-            clocks = <&bpmp TEGRA186_CLK_EMC>;
-            clock-names = "emc";
+            external-memory-controller@2c60000 {
+                compatible = "nvidia,tegra186-emc";
+                reg = <0x0 0x02c60000 0x0 0x50000>;
+                interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
+                clocks = <&bpmp TEGRA186_CLK_EMC>;
+                clock-names = "emc";
 
-            nvidia,bpmp = <&bpmp>;
+                nvidia,bpmp = <&bpmp>;
+            };
         };
     };
 
diff --git a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
index d9ad9260e348..f88d13d70441 100644
--- a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
+++ b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
@@ -274,7 +274,7 @@ examples:
   - |
     #include <dt-bindings/mfd/st,stpmic1.h>
     #include <dt-bindings/interrupt-controller/arm-gic.h>
-    i2c@0 {
+    i2c {
       #address-cells = <1>;
       #size-cells = <0>;
       pmic@33 {
diff --git a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
index b9f90081046f..67df3fe861ee 100644
--- a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
+++ b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
@@ -48,6 +48,7 @@ examples:
 
         switch@10 {
             compatible = "qca,qca8337";
+            reg = <0x10>;
             /* ... */
         };
     };
diff --git a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
index 89ab67f20a7f..c147900f9041 100644
--- a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
+++ b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
@@ -39,7 +39,7 @@ additionalProperties: false
 
 examples:
   - |
-    rng {
+    rng@7e104000 {
         compatible = "brcm,bcm2835-rng";
         reg = <0x7e104000 0x10>;
         interrupts = <2 29>;
diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
index 0cf470eaf2a0..5c16cf59ca00 100644
--- a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
+++ b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
@@ -61,7 +61,7 @@ examples:
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
     #include <dt-bindings/interrupt-controller/arm-gic.h>
 
-    soc: soc@0 {
+    soc: soc {
         #address-cells = <2>;
         #size-cells = <2>;
 
diff --git a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
index 1d6877875077..c2d2ee43ba67 100644
--- a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
+++ b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
@@ -56,7 +56,7 @@ additionalProperties: false
 examples:
   - |
     #include <dt-bindings/clock/jz4740-cgu.h>
-    usb_phy: usb-phy@0 {
+    usb_phy: usb-phy {
       compatible = "usb-nop-xceiv";
       #phy-cells = <0>;
     };
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
  2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
@ 2020-04-09 20:32 ` Guenter Roeck
  2020-04-10  2:15 ` Bjorn Andersson
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2020-04-09 20:32 UTC (permalink / raw)
  To: Rob Herring
  Cc: devicetree, linux-kernel, Maxime Ripard, Chen-Yu Tsai,
	Nuno Sá,
	Jean Delvare, Thierry Reding, Jonathan Hunter, Lee Jones,
	Bjorn Andersson, David S. Miller, Matt Mackall, Herbert Xu,
	Nicolas Saenz Julienne, Florian Fainelli, Ray Jui, Scott Branden,
	bcm-kernel-feedback-list, Mark Brown, linux-hwmon, linux-tegra,
	linux-arm-msm, netdev, linux-crypto, linux-rpi-kernel, linux-spi,
	linux-usb

On Thu, Apr 09, 2020 at 02:24:58PM -0600, Rob Herring wrote:
> A recent update to dtc and changes to the default warnings introduced
> some new warnings in the DT binding examples:
> 
> Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.example.dts:23.13-61:
>  Warning (dma_ranges_format): /example-0/dram-controller@1c01000:dma-ranges: "dma-ranges" property has invalid length (12 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1)
> Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.example.dts:17.22-28.11:
>  Warning (unit_address_vs_reg): /example-0/fpga-axi@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dts:34.13-54:
>  Warning (dma_ranges_format): /example-0/memory-controller@2c00000:dma-ranges: "dma-ranges" property has invalid length (24 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 2)
> Documentation/devicetree/bindings/mfd/st,stpmic1.example.dts:19.15-79.11:
>  Warning (unit_address_vs_reg): /example-0/i2c@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.example.dts:28.23-31.15:
>  Warning (unit_address_vs_reg): /example-0/mdio@37000000/switch@10: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/rng/brcm,bcm2835.example.dts:17.5-21.11:
>  Warning (unit_address_vs_reg): /example-0/rng: node has a reg or ranges property, but no unit name
> Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.example.dts:20.20-43.11:
>  Warning (unit_address_vs_reg): /example-0/soc@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/usb/ingenic,musb.example.dts:18.28-21.11:
>  Warning (unit_address_vs_reg): /example-0/usb-phy@0: node has a unit name, but no reg or ranges property
> 
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Chen-Yu Tsai <wens@csie.org>
> Cc: "Nuno Sá" <nuno.sa@analog.com>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Matt Mackall <mpm@selenic.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Ray Jui <rjui@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: bcm-kernel-feedback-list@broadcom.com
> Cc: Mark Brown <broonie@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-tegra@vger.kernel.org
> Cc: linux-arm-msm@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-crypto@vger.kernel.org
> Cc: linux-rpi-kernel@lists.infradead.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>

For hwmon:

Acked-by: Guenter Roeck <linux@roeck-us.net>

> ---
> Will take this via the DT tree.
> 
> Rob
> 
>  .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |  6 +++
>  .../bindings/hwmon/adi,axi-fan-control.yaml   |  2 +-
>  .../nvidia,tegra186-mc.yaml                   | 41 +++++++++++--------
>  .../devicetree/bindings/mfd/st,stpmic1.yaml   |  2 +-
>  .../bindings/net/qcom,ipq8064-mdio.yaml       |  1 +
>  .../devicetree/bindings/rng/brcm,bcm2835.yaml |  2 +-
>  .../bindings/spi/qcom,spi-qcom-qspi.yaml      |  2 +-
>  .../devicetree/bindings/usb/ingenic,musb.yaml |  2 +-
>  8 files changed, 35 insertions(+), 23 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> index aa0738b4d534..e713a6fe4cf7 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> @@ -42,6 +42,10 @@ properties:
>      description:
>        See section 2.3.9 of the DeviceTree Specification.
>  
> +  '#address-cells': true
> +
> +  '#size-cells': true
> +
>  required:
>    - "#interconnect-cells"
>    - compatible
> @@ -59,6 +63,8 @@ examples:
>          compatible = "allwinner,sun5i-a13-mbus";
>          reg = <0x01c01000 0x1000>;
>          clocks = <&ccu CLK_MBUS>;
> +        #address-cells = <1>;
> +        #size-cells = <1>;
>          dma-ranges = <0x00000000 0x40000000 0x20000000>;
>          #interconnect-cells = <1>;
>      };
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> index 57a240d2d026..29bb2c778c59 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> @@ -47,7 +47,7 @@ required:
>  
>  examples:
>    - |
> -    fpga_axi: fpga-axi@0 {
> +    fpga_axi: fpga-axi {
>              #address-cells = <0x2>;
>              #size-cells = <0x1>;
>  
> diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> index 12516bd89cf9..611bda38d187 100644
> --- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> @@ -97,30 +97,35 @@ examples:
>      #include <dt-bindings/clock/tegra186-clock.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>  
> -    memory-controller@2c00000 {
> -        compatible = "nvidia,tegra186-mc";
> -        reg = <0x0 0x02c00000 0x0 0xb0000>;
> -        interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> -
> +    bus {
>          #address-cells = <2>;
>          #size-cells = <2>;
>  
> -        ranges = <0x0 0x02c00000 0x02c00000 0x0 0xb0000>;
> +        memory-controller@2c00000 {
> +            compatible = "nvidia,tegra186-mc";
> +            reg = <0x0 0x02c00000 0x0 0xb0000>;
> +            interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> +
> +            #address-cells = <2>;
> +            #size-cells = <2>;
> +
> +            ranges = <0x0 0x02c00000 0x0 0x02c00000 0x0 0xb0000>;
>  
> -        /*
> -         * Memory clients have access to all 40 bits that the memory
> -         * controller can address.
> -         */
> -        dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
> +            /*
> +             * Memory clients have access to all 40 bits that the memory
> +             * controller can address.
> +             */
> +            dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
>  
> -        external-memory-controller@2c60000 {
> -            compatible = "nvidia,tegra186-emc";
> -            reg = <0x0 0x02c60000 0x0 0x50000>;
> -            interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> -            clocks = <&bpmp TEGRA186_CLK_EMC>;
> -            clock-names = "emc";
> +            external-memory-controller@2c60000 {
> +                compatible = "nvidia,tegra186-emc";
> +                reg = <0x0 0x02c60000 0x0 0x50000>;
> +                interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> +                clocks = <&bpmp TEGRA186_CLK_EMC>;
> +                clock-names = "emc";
>  
> -            nvidia,bpmp = <&bpmp>;
> +                nvidia,bpmp = <&bpmp>;
> +            };
>          };
>      };
>  
> diff --git a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> index d9ad9260e348..f88d13d70441 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> @@ -274,7 +274,7 @@ examples:
>    - |
>      #include <dt-bindings/mfd/st,stpmic1.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> -    i2c@0 {
> +    i2c {
>        #address-cells = <1>;
>        #size-cells = <0>;
>        pmic@33 {
> diff --git a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> index b9f90081046f..67df3fe861ee 100644
> --- a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> +++ b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> @@ -48,6 +48,7 @@ examples:
>  
>          switch@10 {
>              compatible = "qca,qca8337";
> +            reg = <0x10>;
>              /* ... */
>          };
>      };
> diff --git a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> index 89ab67f20a7f..c147900f9041 100644
> --- a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> +++ b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> @@ -39,7 +39,7 @@ additionalProperties: false
>  
>  examples:
>    - |
> -    rng {
> +    rng@7e104000 {
>          compatible = "brcm,bcm2835-rng";
>          reg = <0x7e104000 0x10>;
>          interrupts = <2 29>;
> diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> index 0cf470eaf2a0..5c16cf59ca00 100644
> --- a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> +++ b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> @@ -61,7 +61,7 @@ examples:
>      #include <dt-bindings/clock/qcom,gcc-sdm845.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>  
> -    soc: soc@0 {
> +    soc: soc {
>          #address-cells = <2>;
>          #size-cells = <2>;
>  
> diff --git a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> index 1d6877875077..c2d2ee43ba67 100644
> --- a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> @@ -56,7 +56,7 @@ additionalProperties: false
>  examples:
>    - |
>      #include <dt-bindings/clock/jz4740-cgu.h>
> -    usb_phy: usb-phy@0 {
> +    usb_phy: usb-phy {
>        compatible = "usb-nop-xceiv";
>        #phy-cells = <0>;
>      };
> -- 
> 2.20.1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
  2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
  2020-04-09 20:32 ` Guenter Roeck
@ 2020-04-10  2:15 ` Bjorn Andersson
  2020-04-14 14:13 ` Thierry Reding
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Bjorn Andersson @ 2020-04-10  2:15 UTC (permalink / raw)
  To: Rob Herring
  Cc: devicetree, linux-kernel, Maxime Ripard, Chen-Yu Tsai, Nuno S?,
	Jean Delvare, Guenter Roeck, Thierry Reding, Jonathan Hunter,
	Lee Jones, David S. Miller, Matt Mackall, Herbert Xu,
	Nicolas Saenz Julienne, Florian Fainelli, Ray Jui, Scott Branden,
	bcm-kernel-feedback-list, Mark Brown, linux-hwmon, linux-tegra,
	linux-arm-msm, netdev, linux-crypto, linux-rpi-kernel, linux-spi,
	linux-usb

On Thu 09 Apr 13:24 PDT 2020, Rob Herring wrote:

> A recent update to dtc and changes to the default warnings introduced
> some new warnings in the DT binding examples:
> 
> Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.example.dts:23.13-61:
>  Warning (dma_ranges_format): /example-0/dram-controller@1c01000:dma-ranges: "dma-ranges" property has invalid length (12 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1)
> Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.example.dts:17.22-28.11:
>  Warning (unit_address_vs_reg): /example-0/fpga-axi@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dts:34.13-54:
>  Warning (dma_ranges_format): /example-0/memory-controller@2c00000:dma-ranges: "dma-ranges" property has invalid length (24 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 2)
> Documentation/devicetree/bindings/mfd/st,stpmic1.example.dts:19.15-79.11:
>  Warning (unit_address_vs_reg): /example-0/i2c@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.example.dts:28.23-31.15:
>  Warning (unit_address_vs_reg): /example-0/mdio@37000000/switch@10: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/rng/brcm,bcm2835.example.dts:17.5-21.11:
>  Warning (unit_address_vs_reg): /example-0/rng: node has a reg or ranges property, but no unit name
> Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.example.dts:20.20-43.11:
>  Warning (unit_address_vs_reg): /example-0/soc@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/usb/ingenic,musb.example.dts:18.28-21.11:
>  Warning (unit_address_vs_reg): /example-0/usb-phy@0: node has a unit name, but no reg or ranges property
> 
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Chen-Yu Tsai <wens@csie.org>
> Cc: "Nuno Sá" <nuno.sa@analog.com>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Matt Mackall <mpm@selenic.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Ray Jui <rjui@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: bcm-kernel-feedback-list@broadcom.com
> Cc: Mark Brown <broonie@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-tegra@vger.kernel.org
> Cc: linux-arm-msm@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-crypto@vger.kernel.org
> Cc: linux-rpi-kernel@lists.infradead.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> ---
> Will take this via the DT tree.
> 
> Rob
> 
>  .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |  6 +++
>  .../bindings/hwmon/adi,axi-fan-control.yaml   |  2 +-
>  .../nvidia,tegra186-mc.yaml                   | 41 +++++++++++--------
>  .../devicetree/bindings/mfd/st,stpmic1.yaml   |  2 +-
>  .../bindings/net/qcom,ipq8064-mdio.yaml       |  1 +
>  .../devicetree/bindings/rng/brcm,bcm2835.yaml |  2 +-
>  .../bindings/spi/qcom,spi-qcom-qspi.yaml      |  2 +-
>  .../devicetree/bindings/usb/ingenic,musb.yaml |  2 +-
>  8 files changed, 35 insertions(+), 23 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> index aa0738b4d534..e713a6fe4cf7 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> @@ -42,6 +42,10 @@ properties:
>      description:
>        See section 2.3.9 of the DeviceTree Specification.
>  
> +  '#address-cells': true
> +
> +  '#size-cells': true
> +
>  required:
>    - "#interconnect-cells"
>    - compatible
> @@ -59,6 +63,8 @@ examples:
>          compatible = "allwinner,sun5i-a13-mbus";
>          reg = <0x01c01000 0x1000>;
>          clocks = <&ccu CLK_MBUS>;
> +        #address-cells = <1>;
> +        #size-cells = <1>;
>          dma-ranges = <0x00000000 0x40000000 0x20000000>;
>          #interconnect-cells = <1>;
>      };
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> index 57a240d2d026..29bb2c778c59 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> @@ -47,7 +47,7 @@ required:
>  
>  examples:
>    - |
> -    fpga_axi: fpga-axi@0 {
> +    fpga_axi: fpga-axi {
>              #address-cells = <0x2>;
>              #size-cells = <0x1>;
>  
> diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> index 12516bd89cf9..611bda38d187 100644
> --- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.yaml
> @@ -97,30 +97,35 @@ examples:
>      #include <dt-bindings/clock/tegra186-clock.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>  
> -    memory-controller@2c00000 {
> -        compatible = "nvidia,tegra186-mc";
> -        reg = <0x0 0x02c00000 0x0 0xb0000>;
> -        interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> -
> +    bus {
>          #address-cells = <2>;
>          #size-cells = <2>;
>  
> -        ranges = <0x0 0x02c00000 0x02c00000 0x0 0xb0000>;
> +        memory-controller@2c00000 {
> +            compatible = "nvidia,tegra186-mc";
> +            reg = <0x0 0x02c00000 0x0 0xb0000>;
> +            interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> +
> +            #address-cells = <2>;
> +            #size-cells = <2>;
> +
> +            ranges = <0x0 0x02c00000 0x0 0x02c00000 0x0 0xb0000>;
>  
> -        /*
> -         * Memory clients have access to all 40 bits that the memory
> -         * controller can address.
> -         */
> -        dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
> +            /*
> +             * Memory clients have access to all 40 bits that the memory
> +             * controller can address.
> +             */
> +            dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
>  
> -        external-memory-controller@2c60000 {
> -            compatible = "nvidia,tegra186-emc";
> -            reg = <0x0 0x02c60000 0x0 0x50000>;
> -            interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> -            clocks = <&bpmp TEGRA186_CLK_EMC>;
> -            clock-names = "emc";
> +            external-memory-controller@2c60000 {
> +                compatible = "nvidia,tegra186-emc";
> +                reg = <0x0 0x02c60000 0x0 0x50000>;
> +                interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> +                clocks = <&bpmp TEGRA186_CLK_EMC>;
> +                clock-names = "emc";
>  
> -            nvidia,bpmp = <&bpmp>;
> +                nvidia,bpmp = <&bpmp>;
> +            };
>          };
>      };
>  
> diff --git a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> index d9ad9260e348..f88d13d70441 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> @@ -274,7 +274,7 @@ examples:
>    - |
>      #include <dt-bindings/mfd/st,stpmic1.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> -    i2c@0 {
> +    i2c {
>        #address-cells = <1>;
>        #size-cells = <0>;
>        pmic@33 {
> diff --git a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> index b9f90081046f..67df3fe861ee 100644
> --- a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> +++ b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> @@ -48,6 +48,7 @@ examples:
>  
>          switch@10 {
>              compatible = "qca,qca8337";
> +            reg = <0x10>;
>              /* ... */
>          };
>      };
> diff --git a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> index 89ab67f20a7f..c147900f9041 100644
> --- a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> +++ b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> @@ -39,7 +39,7 @@ additionalProperties: false
>  
>  examples:
>    - |
> -    rng {
> +    rng@7e104000 {
>          compatible = "brcm,bcm2835-rng";
>          reg = <0x7e104000 0x10>;
>          interrupts = <2 29>;
> diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> index 0cf470eaf2a0..5c16cf59ca00 100644
> --- a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> +++ b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> @@ -61,7 +61,7 @@ examples:
>      #include <dt-bindings/clock/qcom,gcc-sdm845.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>  
> -    soc: soc@0 {
> +    soc: soc {
>          #address-cells = <2>;
>          #size-cells = <2>;
>  
> diff --git a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> index 1d6877875077..c2d2ee43ba67 100644
> --- a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> @@ -56,7 +56,7 @@ additionalProperties: false
>  examples:
>    - |
>      #include <dt-bindings/clock/jz4740-cgu.h>
> -    usb_phy: usb-phy@0 {
> +    usb_phy: usb-phy {
>        compatible = "usb-nop-xceiv";
>        #phy-cells = <0>;
>      };
> -- 
> 2.20.1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
  2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
  2020-04-09 20:32 ` Guenter Roeck
  2020-04-10  2:15 ` Bjorn Andersson
@ 2020-04-14 14:13 ` Thierry Reding
  2020-04-14 14:22 ` Sa, Nuno
  2020-04-14 17:06 ` Mark Brown
  4 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2020-04-14 14:13 UTC (permalink / raw)
  To: Rob Herring
  Cc: devicetree, linux-kernel, Maxime Ripard, Chen-Yu Tsai,
	Nuno Sá,
	Jean Delvare, Guenter Roeck, Jonathan Hunter, Lee Jones,
	Bjorn Andersson, David S. Miller, Matt Mackall, Herbert Xu,
	Nicolas Saenz Julienne, Florian Fainelli, Ray Jui, Scott Branden,
	bcm-kernel-feedback-list, Mark Brown, linux-hwmon, linux-tegra,
	linux-arm-msm, netdev, linux-crypto, linux-rpi-kernel, linux-spi,
	linux-usb

[-- Attachment #1: Type: text/plain, Size: 3357 bytes --]

On Thu, Apr 09, 2020 at 02:24:58PM -0600, Rob Herring wrote:
> A recent update to dtc and changes to the default warnings introduced
> some new warnings in the DT binding examples:
> 
> Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.example.dts:23.13-61:
>  Warning (dma_ranges_format): /example-0/dram-controller@1c01000:dma-ranges: "dma-ranges" property has invalid length (12 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1)
> Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.example.dts:17.22-28.11:
>  Warning (unit_address_vs_reg): /example-0/fpga-axi@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-mc.example.dts:34.13-54:
>  Warning (dma_ranges_format): /example-0/memory-controller@2c00000:dma-ranges: "dma-ranges" property has invalid length (24 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 2)
> Documentation/devicetree/bindings/mfd/st,stpmic1.example.dts:19.15-79.11:
>  Warning (unit_address_vs_reg): /example-0/i2c@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.example.dts:28.23-31.15:
>  Warning (unit_address_vs_reg): /example-0/mdio@37000000/switch@10: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/rng/brcm,bcm2835.example.dts:17.5-21.11:
>  Warning (unit_address_vs_reg): /example-0/rng: node has a reg or ranges property, but no unit name
> Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.example.dts:20.20-43.11:
>  Warning (unit_address_vs_reg): /example-0/soc@0: node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/usb/ingenic,musb.example.dts:18.28-21.11:
>  Warning (unit_address_vs_reg): /example-0/usb-phy@0: node has a unit name, but no reg or ranges property
> 
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Chen-Yu Tsai <wens@csie.org>
> Cc: "Nuno Sá" <nuno.sa@analog.com>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Matt Mackall <mpm@selenic.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Ray Jui <rjui@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: bcm-kernel-feedback-list@broadcom.com
> Cc: Mark Brown <broonie@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-tegra@vger.kernel.org
> Cc: linux-arm-msm@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-crypto@vger.kernel.org
> Cc: linux-rpi-kernel@lists.infradead.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Will take this via the DT tree.
> 
> Rob
> 
>  .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |  6 +++
>  .../bindings/hwmon/adi,axi-fan-control.yaml   |  2 +-
>  .../nvidia,tegra186-mc.yaml                   | 41 +++++++++++--------

Acked-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
  2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
                   ` (2 preceding siblings ...)
  2020-04-14 14:13 ` Thierry Reding
@ 2020-04-14 14:22 ` Sa, Nuno
  2020-04-14 17:06 ` Mark Brown
  4 siblings, 0 replies; 6+ messages in thread
From: Sa, Nuno @ 2020-04-14 14:22 UTC (permalink / raw)
  To: Rob Herring, devicetree
  Cc: linux-kernel, Maxime Ripard, Chen-Yu Tsai, Jean Delvare,
	Guenter Roeck, Thierry Reding, Jonathan Hunter, Lee Jones,
	Bjorn Andersson, David S. Miller, Matt Mackall, Herbert Xu,
	Nicolas Saenz Julienne, Florian Fainelli, Ray Jui, Scott Branden,
	bcm-kernel-feedback-list, Mark Brown, linux-hwmon, linux-tegra,
	linux-arm-msm, netdev, linux-crypto, linux-rpi-kernel, linux-spi,
	linux-usb


> From: Rob Herring <robh@kernel.org>
> Sent: Donnerstag, 9. April 2020 22:25
> To: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org; Maxime Ripard <mripard@kernel.org>;
> Chen-Yu Tsai <wens@csie.org>; Sa, Nuno <Nuno.Sa@analog.com>; Jean
> Delvare <jdelvare@suse.com>; Guenter Roeck <linux@roeck-us.net>;
> Thierry Reding <thierry.reding@gmail.com>; Jonathan Hunter
> <jonathanh@nvidia.com>; Lee Jones <lee.jones@linaro.org>; Bjorn
> Andersson <bjorn.andersson@linaro.org>; David S. Miller
> <davem@davemloft.net>; Matt Mackall <mpm@selenic.com>; Herbert Xu
> <herbert@gondor.apana.org.au>; Nicolas Saenz Julienne
> <nsaenzjulienne@suse.de>; Florian Fainelli <f.fainelli@gmail.com>; Ray Jui
> <rjui@broadcom.com>; Scott Branden <sbranden@broadcom.com>; bcm-
> kernel-feedback-list@broadcom.com; Mark Brown <broonie@kernel.org>;
> linux-hwmon@vger.kernel.org; linux-tegra@vger.kernel.org; linux-arm-
> msm@vger.kernel.org; netdev@vger.kernel.org; linux-
> crypto@vger.kernel.org; linux-rpi-kernel@lists.infradead.org; linux-
> spi@vger.kernel.org; linux-usb@vger.kernel.org
> Subject: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
> 
> 
> A recent update to dtc and changes to the default warnings introduced
> some new warnings in the DT binding examples:
> 
> Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-
> mbus.example.dts:23.13-61:
>  Warning (dma_ranges_format): /example-0/dram-controller@1c01000:dma-
> ranges: "dma-ranges" property has invalid length (12 bytes) (parent
> #address-cells == 1, child #address-cells == 2, #size-cells == 1)
> Documentation/devicetree/bindings/hwmon/adi,axi-fan-
> control.example.dts:17.22-28.11:
>  Warning (unit_address_vs_reg): /example-0/fpga-axi@0: node has a unit
> name, but no reg or ranges property
> Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-
> mc.example.dts:34.13-54:
>  Warning (dma_ranges_format): /example-0/memory-
> controller@2c00000:dma-ranges: "dma-ranges" property has invalid length
> (24 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells
> == 2)
> Documentation/devicetree/bindings/mfd/st,stpmic1.example.dts:19.15-
> 79.11:
>  Warning (unit_address_vs_reg): /example-0/i2c@0: node has a unit name,
> but no reg or ranges property
> Documentation/devicetree/bindings/net/qcom,ipq8064-
> mdio.example.dts:28.23-31.15:
>  Warning (unit_address_vs_reg): /example-0/mdio@37000000/switch@10:
> node has a unit name, but no reg or ranges property
> Documentation/devicetree/bindings/rng/brcm,bcm2835.example.dts:17.5-
> 21.11:
>  Warning (unit_address_vs_reg): /example-0/rng: node has a reg or ranges
> property, but no unit name
> Documentation/devicetree/bindings/spi/qcom,spi-qcom-
> qspi.example.dts:20.20-43.11:
>  Warning (unit_address_vs_reg): /example-0/soc@0: node has a unit name,
> but no reg or ranges property
> Documentation/devicetree/bindings/usb/ingenic,musb.example.dts:18.28-
> 21.11:
>  Warning (unit_address_vs_reg): /example-0/usb-phy@0: node has a unit
> name, but no reg or ranges property
> 
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Chen-Yu Tsai <wens@csie.org>
> Cc: "Nuno Sá" <nuno.sa@analog.com>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Matt Mackall <mpm@selenic.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Ray Jui <rjui@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: bcm-kernel-feedback-list@broadcom.com
> Cc: Mark Brown <broonie@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Cc: linux-tegra@vger.kernel.org
> Cc: linux-arm-msm@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-crypto@vger.kernel.org
> Cc: linux-rpi-kernel@lists.infradead.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---

Acked-by: Nuno Sá <nuno.sa@analog.com>

> Will take this via the DT tree.
> 
> Rob
> 
>  .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |  6 +++
>  .../bindings/hwmon/adi,axi-fan-control.yaml   |  2 +-
>  .../nvidia,tegra186-mc.yaml                   | 41 +++++++++++--------
>  .../devicetree/bindings/mfd/st,stpmic1.yaml   |  2 +-
>  .../bindings/net/qcom,ipq8064-mdio.yaml       |  1 +
>  .../devicetree/bindings/rng/brcm,bcm2835.yaml |  2 +-
>  .../bindings/spi/qcom,spi-qcom-qspi.yaml      |  2 +-
>  .../devicetree/bindings/usb/ingenic,musb.yaml |  2 +-
>  8 files changed, 35 insertions(+), 23 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-
> a10-mbus.yaml
> b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-
> mbus.yaml
> index aa0738b4d534..e713a6fe4cf7 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-
> mbus.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-
> mbus.yaml
> @@ -42,6 +42,10 @@ properties:
>      description:
>        See section 2.3.9 of the DeviceTree Specification.
> 
> +  '#address-cells': true
> +
> +  '#size-cells': true
> +
>  required:
>    - "#interconnect-cells"
>    - compatible
> @@ -59,6 +63,8 @@ examples:
>          compatible = "allwinner,sun5i-a13-mbus";
>          reg = <0x01c01000 0x1000>;
>          clocks = <&ccu CLK_MBUS>;
> +        #address-cells = <1>;
> +        #size-cells = <1>;
>          dma-ranges = <0x00000000 0x40000000 0x20000000>;
>          #interconnect-cells = <1>;
>      };
> diff --git a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-
> control.yaml b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-
> control.yaml
> index 57a240d2d026..29bb2c778c59 100644
> --- a/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adi,axi-fan-control.yaml
> @@ -47,7 +47,7 @@ required:
> 
>  examples:
>    - |
> -    fpga_axi: fpga-axi@0 {
> +    fpga_axi: fpga-axi {
>              #address-cells = <0x2>;
>              #size-cells = <0x1>;
> 
> diff --git a/Documentation/devicetree/bindings/memory-
> controllers/nvidia,tegra186-mc.yaml
> b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra186-
> mc.yaml
> index 12516bd89cf9..611bda38d187 100644
> --- a/Documentation/devicetree/bindings/memory-
> controllers/nvidia,tegra186-mc.yaml
> +++ b/Documentation/devicetree/bindings/memory-
> controllers/nvidia,tegra186-mc.yaml
> @@ -97,30 +97,35 @@ examples:
>      #include <dt-bindings/clock/tegra186-clock.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> 
> -    memory-controller@2c00000 {
> -        compatible = "nvidia,tegra186-mc";
> -        reg = <0x0 0x02c00000 0x0 0xb0000>;
> -        interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> -
> +    bus {
>          #address-cells = <2>;
>          #size-cells = <2>;
> 
> -        ranges = <0x0 0x02c00000 0x02c00000 0x0 0xb0000>;
> +        memory-controller@2c00000 {
> +            compatible = "nvidia,tegra186-mc";
> +            reg = <0x0 0x02c00000 0x0 0xb0000>;
> +            interrupts = <GIC_SPI 223 IRQ_TYPE_LEVEL_HIGH>;
> +
> +            #address-cells = <2>;
> +            #size-cells = <2>;
> +
> +            ranges = <0x0 0x02c00000 0x0 0x02c00000 0x0 0xb0000>;
> 
> -        /*
> -         * Memory clients have access to all 40 bits that the memory
> -         * controller can address.
> -         */
> -        dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
> +            /*
> +             * Memory clients have access to all 40 bits that the memory
> +             * controller can address.
> +             */
> +            dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
> 
> -        external-memory-controller@2c60000 {
> -            compatible = "nvidia,tegra186-emc";
> -            reg = <0x0 0x02c60000 0x0 0x50000>;
> -            interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> -            clocks = <&bpmp TEGRA186_CLK_EMC>;
> -            clock-names = "emc";
> +            external-memory-controller@2c60000 {
> +                compatible = "nvidia,tegra186-emc";
> +                reg = <0x0 0x02c60000 0x0 0x50000>;
> +                interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> +                clocks = <&bpmp TEGRA186_CLK_EMC>;
> +                clock-names = "emc";
> 
> -            nvidia,bpmp = <&bpmp>;
> +                nvidia,bpmp = <&bpmp>;
> +            };
>          };
>      };
> 
> diff --git a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> index d9ad9260e348..f88d13d70441 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stpmic1.yaml
> @@ -274,7 +274,7 @@ examples:
>    - |
>      #include <dt-bindings/mfd/st,stpmic1.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> -    i2c@0 {
> +    i2c {
>        #address-cells = <1>;
>        #size-cells = <0>;
>        pmic@33 {
> diff --git a/Documentation/devicetree/bindings/net/qcom,ipq8064-
> mdio.yaml b/Documentation/devicetree/bindings/net/qcom,ipq8064-
> mdio.yaml
> index b9f90081046f..67df3fe861ee 100644
> --- a/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> +++ b/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.yaml
> @@ -48,6 +48,7 @@ examples:
> 
>          switch@10 {
>              compatible = "qca,qca8337";
> +            reg = <0x10>;
>              /* ... */
>          };
>      };
> diff --git a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> index 89ab67f20a7f..c147900f9041 100644
> --- a/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> +++ b/Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml
> @@ -39,7 +39,7 @@ additionalProperties: false
> 
>  examples:
>    - |
> -    rng {
> +    rng@7e104000 {
>          compatible = "brcm,bcm2835-rng";
>          reg = <0x7e104000 0x10>;
>          interrupts = <2 29>;
> diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-
> qspi.yaml b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-
> qspi.yaml
> index 0cf470eaf2a0..5c16cf59ca00 100644
> --- a/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> +++ b/Documentation/devicetree/bindings/spi/qcom,spi-qcom-qspi.yaml
> @@ -61,7 +61,7 @@ examples:
>      #include <dt-bindings/clock/qcom,gcc-sdm845.h>
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> 
> -    soc: soc@0 {
> +    soc: soc {
>          #address-cells = <2>;
>          #size-cells = <2>;
> 
> diff --git a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> index 1d6877875077..c2d2ee43ba67 100644
> --- a/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ingenic,musb.yaml
> @@ -56,7 +56,7 @@ additionalProperties: false
>  examples:
>    - |
>      #include <dt-bindings/clock/jz4740-cgu.h>
> -    usb_phy: usb-phy@0 {
> +    usb_phy: usb-phy {
>        compatible = "usb-nop-xceiv";
>        #phy-cells = <0>;
>      };
> --
> 2.20.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples
  2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
                   ` (3 preceding siblings ...)
  2020-04-14 14:22 ` Sa, Nuno
@ 2020-04-14 17:06 ` Mark Brown
  4 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2020-04-14 17:06 UTC (permalink / raw)
  To: Rob Herring
  Cc: devicetree, linux-kernel, Maxime Ripard, Chen-Yu Tsai,
	Nuno Sá,
	Jean Delvare, Guenter Roeck, Thierry Reding, Jonathan Hunter,
	Lee Jones, Bjorn Andersson, David S. Miller, Matt Mackall,
	Herbert Xu, Nicolas Saenz Julienne, Florian Fainelli, Ray Jui,
	Scott Branden, bcm-kernel-feedback-list, linux-hwmon,
	linux-tegra, linux-arm-msm, netdev, linux-crypto,
	linux-rpi-kernel, linux-spi, linux-usb

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]

On Thu, Apr 09, 2020 at 02:24:58PM -0600, Rob Herring wrote:
> A recent update to dtc and changes to the default warnings introduced
> some new warnings in the DT binding examples:

Acked-by: Mark Brown <broonie@kernel.org>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-04-14 17:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-09 20:24 [PATCH] dt-bindings: Fix dtc warnings on reg and ranges in examples Rob Herring
2020-04-09 20:32 ` Guenter Roeck
2020-04-10  2:15 ` Bjorn Andersson
2020-04-14 14:13 ` Thierry Reding
2020-04-14 14:22 ` Sa, Nuno
2020-04-14 17:06 ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).