public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] arm: dts: vf6: fix DTC unit warnings
@ 2016-04-18  9:26 Heiko Schocher
  2016-04-18  9:42 ` Bhuvanchandra DV
  0 siblings, 1 reply; 2+ messages in thread
From: Heiko Schocher @ 2016-04-18  9:26 UTC (permalink / raw)
  To: u-boot

Fix following warnings for all vf6 based boards:

Warning (unit_address_vs_reg): Node /soc/ifc at 1530000/board-control at 3,0/mdio-mux-emi1 has a reg or ranges property, but no unit name
Warning (unit_address_vs_reg): Node /soc/aips-bus at 40000000 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /soc/aips-bus at 40080000 has a unit name, but no reg property

Signed-off-by: Heiko Schocher <hs@denx.de>
---
This warnings pop up with the DTC compiler:
$ /tmp/dtc/dtc -v
Version: DTC 1.4.1-gbeef80b8

This fixes the compile warnings for:
https://travis-ci.org/u-boot/u-boot/jobs/122880431

see:
https://travis-ci.org/hsdenx/u-boot/jobs/123238780


Changes in v2:
- added comment from bhuvanchandra.dv at toradex.com :
  used the changes Stefan Agner posted o LKML:
  https://lkml.org/lkml/2016/4/2/11

 arch/arm/dts/ls1021a-qds.dtsi | 2 +-
 arch/arm/dts/vf.dtsi          | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/dts/ls1021a-qds.dtsi b/arch/arm/dts/ls1021a-qds.dtsi
index ca9e835..4cb6337 100644
--- a/arch/arm/dts/ls1021a-qds.dtsi
+++ b/arch/arm/dts/ls1021a-qds.dtsi
@@ -138,7 +138,7 @@
 		device-width = <1>;
 		ranges = <0 3 0 0x100>;
 
-		mdio-mux-emi1 {
+		mdio-mux-emi1 at 54 {
 			compatible = "mdio-mux-mmioreg";
 			mdio-parent-bus = <&mdio0>;
 			#address-cells = <1>;
diff --git a/arch/arm/dts/vf.dtsi b/arch/arm/dts/vf.dtsi
index 1530d2f..c10c564 100644
--- a/arch/arm/dts/vf.dtsi
+++ b/arch/arm/dts/vf.dtsi
@@ -32,6 +32,7 @@
 			compatible = "fsl,aips-bus", "simple-bus";
 			#address-cells = <1>;
 			#size-cells = <1>;
+			reg = <0x40000000 0x00070000>;
 			ranges;
 
 			uart0: serial at 40027000 {
@@ -119,6 +120,7 @@
 			compatible = "fsl,aips-bus", "simple-bus";
 			#address-cells = <1>;
 			#size-cells = <1>;
+			reg = <0x40080000 0x0007f000>;
 			ranges;
 
 			uart4: serial at 400a9000 {
-- 
2.5.0

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

* [U-Boot] [PATCH v2] arm: dts: vf6: fix DTC unit warnings
  2016-04-18  9:26 [U-Boot] [PATCH v2] arm: dts: vf6: fix DTC unit warnings Heiko Schocher
@ 2016-04-18  9:42 ` Bhuvanchandra DV
  0 siblings, 0 replies; 2+ messages in thread
From: Bhuvanchandra DV @ 2016-04-18  9:42 UTC (permalink / raw)
  To: u-boot

On 04/18/2016 02:56 PM, Heiko Schocher wrote:
> Fix following warnings for all vf6 based boards:

Freescale Vybrid and ls1021xx are different SoCs, may be we can split 
this patchset.

Apart from that.
Reviewed-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
>
> Warning (unit_address_vs_reg): Node /soc/ifc at 1530000/board-control at 3,0/mdio-mux-emi1 has a reg or ranges property, but no unit name
> Warning (unit_address_vs_reg): Node /soc/aips-bus at 40000000 has a unit name, but no reg property
> Warning (unit_address_vs_reg): Node /soc/aips-bus at 40080000 has a unit name, but no reg property
>
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> This warnings pop up with the DTC compiler:
> $ /tmp/dtc/dtc -v
> Version: DTC 1.4.1-gbeef80b8
>
> This fixes the compile warnings for:
> https://travis-ci.org/u-boot/u-boot/jobs/122880431
>
> see:
> https://travis-ci.org/hsdenx/u-boot/jobs/123238780
>
>
> Changes in v2:
> - added comment from bhuvanchandra.dv at toradex.com :
>    used the changes Stefan Agner posted o LKML:
>    https://lkml.org/lkml/2016/4/2/11
>
>   arch/arm/dts/ls1021a-qds.dtsi | 2 +-
>   arch/arm/dts/vf.dtsi          | 2 ++
>   2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/dts/ls1021a-qds.dtsi b/arch/arm/dts/ls1021a-qds.dtsi
> index ca9e835..4cb6337 100644
> --- a/arch/arm/dts/ls1021a-qds.dtsi
> +++ b/arch/arm/dts/ls1021a-qds.dtsi
> @@ -138,7 +138,7 @@
>   		device-width = <1>;
>   		ranges = <0 3 0 0x100>;
>
> -		mdio-mux-emi1 {
> +		mdio-mux-emi1 at 54 {
>   			compatible = "mdio-mux-mmioreg";
>   			mdio-parent-bus = <&mdio0>;
>   			#address-cells = <1>;
> diff --git a/arch/arm/dts/vf.dtsi b/arch/arm/dts/vf.dtsi
> index 1530d2f..c10c564 100644
> --- a/arch/arm/dts/vf.dtsi
> +++ b/arch/arm/dts/vf.dtsi
> @@ -32,6 +32,7 @@
>   			compatible = "fsl,aips-bus", "simple-bus";
>   			#address-cells = <1>;
>   			#size-cells = <1>;
> +			reg = <0x40000000 0x00070000>;
>   			ranges;
>
>   			uart0: serial at 40027000 {
> @@ -119,6 +120,7 @@
>   			compatible = "fsl,aips-bus", "simple-bus";
>   			#address-cells = <1>;
>   			#size-cells = <1>;
> +			reg = <0x40080000 0x0007f000>;
>   			ranges;
>
>   			uart4: serial at 400a9000 {
>

-- 
Best regards,
Bhuvan

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

end of thread, other threads:[~2016-04-18  9:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-18  9:26 [U-Boot] [PATCH v2] arm: dts: vf6: fix DTC unit warnings Heiko Schocher
2016-04-18  9:42 ` Bhuvanchandra DV

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox