[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Michal Simek]

DTC 1.2 reports these warnings:
Warning (unit_address_vs_reg): Node /memory has a reg or ranges
property, but no unit name
Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
but no unit name
Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
but no reg property

This patch is fixing them.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynq-7000.dtsi       | 7 ++++---
 arch/arm/dts/zynq-microzed.dts    | 2 +-
 arch/arm/dts/zynq-picozed.dts     | 2 +-
 arch/arm/dts/zynq-topic-miami.dts | 2 +-
 arch/arm/dts/zynq-zc702.dts       | 2 +-
 arch/arm/dts/zynq-zc706.dts       | 2 +-
 arch/arm/dts/zynq-zc770-xm010.dts | 2 +-
 arch/arm/dts/zynq-zc770-xm011.dts | 2 +-
 arch/arm/dts/zynq-zc770-xm012.dts | 2 +-
 arch/arm/dts/zynq-zc770-xm013.dts | 2 +-
 arch/arm/dts/zynq-zed.dts         | 2 +-
 arch/arm/dts/zynq-zybo.dts        | 2 +-
 12 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/arch/arm/dts/zynq-7000.dtsi b/arch/arm/dts/zynq-7000.dtsi
index b618a3f484f0..6df0329cfc90 100644
--- a/arch/arm/dts/zynq-7000.dtsi
+++ b/arch/arm/dts/zynq-7000.dtsi
@@ -6,9 +6,10 @@
  *
  * SPDX-License-Identifier:	GPL-2.0+
  */
-/include/ "skeleton.dtsi"
 
 / {
+	#address-cells = <1>;
+	#size-cells = <1>;
 	compatible = "xlnx,zynq-7000";
 
 	cpus {
@@ -37,14 +38,14 @@
 		};
 	};
 
-	pmu {
+	pmu at f8891000 {
 		compatible = "arm,cortex-a9-pmu";
 		interrupts = <0 5 4>, <0 6 4>;
 		interrupt-parent = <&intc>;
 		reg = < 0xf8891000 0x1000 0xf8893000 0x1000 >;
 	};
 
-	regulator_vccpint: fixedregulator at 0 {
+	regulator_vccpint: fixedregulator {
 		compatible = "regulator-fixed";
 		regulator-name = "VCCPINT";
 		regulator-min-microvolt = <1000000>;
diff --git a/arch/arm/dts/zynq-microzed.dts b/arch/arm/dts/zynq-microzed.dts
index cb238cd5e7bb..2d07b92e9cff 100644
--- a/arch/arm/dts/zynq-microzed.dts
+++ b/arch/arm/dts/zynq-microzed.dts
@@ -18,7 +18,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-picozed.dts b/arch/arm/dts/zynq-picozed.dts
index 3408df8b8b3d..fea04ab45a72 100644
--- a/arch/arm/dts/zynq-picozed.dts
+++ b/arch/arm/dts/zynq-picozed.dts
@@ -16,7 +16,7 @@
 		serial0 = &uart1;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-topic-miami.dts b/arch/arm/dts/zynq-topic-miami.dts
index 8e988f92877c..aa05c4d368b2 100644
--- a/arch/arm/dts/zynq-topic-miami.dts
+++ b/arch/arm/dts/zynq-topic-miami.dts
@@ -21,7 +21,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc702.dts b/arch/arm/dts/zynq-zc702.dts
index 6585010f4bc8..478e9fd4ef25 100644
--- a/arch/arm/dts/zynq-zc702.dts
+++ b/arch/arm/dts/zynq-zc702.dts
@@ -21,7 +21,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc706.dts b/arch/arm/dts/zynq-zc706.dts
index d04880a2cdd3..8b0177bc512a 100644
--- a/arch/arm/dts/zynq-zc706.dts
+++ b/arch/arm/dts/zynq-zc706.dts
@@ -21,7 +21,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc770-xm010.dts b/arch/arm/dts/zynq-zc770-xm010.dts
index 33524cb6def5..42af313c13dd 100644
--- a/arch/arm/dts/zynq-zc770-xm010.dts
+++ b/arch/arm/dts/zynq-zc770-xm010.dts
@@ -25,7 +25,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc770-xm011.dts b/arch/arm/dts/zynq-zc770-xm011.dts
index 463b14b5ea15..7f08961491a6 100644
--- a/arch/arm/dts/zynq-zc770-xm011.dts
+++ b/arch/arm/dts/zynq-zc770-xm011.dts
@@ -23,7 +23,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc770-xm012.dts b/arch/arm/dts/zynq-zc770-xm012.dts
index 6cab83266770..699cd2c0fb1c 100644
--- a/arch/arm/dts/zynq-zc770-xm012.dts
+++ b/arch/arm/dts/zynq-zc770-xm012.dts
@@ -24,7 +24,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zc770-xm013.dts b/arch/arm/dts/zynq-zc770-xm013.dts
index d5bb4efcc58a..07e92b88fb0f 100644
--- a/arch/arm/dts/zynq-zc770-xm013.dts
+++ b/arch/arm/dts/zynq-zc770-xm013.dts
@@ -25,7 +25,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynq-zed.dts b/arch/arm/dts/zynq-zed.dts
index 4363a4fcaccb..0ac7532300f0 100644
--- a/arch/arm/dts/zynq-zed.dts
+++ b/arch/arm/dts/zynq-zed.dts
@@ -20,7 +20,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x20000000>;
 	};
diff --git a/arch/arm/dts/zynq-zybo.dts b/arch/arm/dts/zynq-zybo.dts
index f8dcf1d6995c..d59a3831352d 100644
--- a/arch/arm/dts/zynq-zybo.dts
+++ b/arch/arm/dts/zynq-zybo.dts
@@ -20,7 +20,7 @@
 		mmc0 = &sdhci0;
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x20000000>;
 	};
-- 
1.9.1

[U-Boot] [PATCH 2/2] ARM64: zynqmp: Remove DTC 1.2 warnings

[Michal Simek]

DTC 1.2 reports these warnings:
Warning (unit_address_vs_reg): Node /amba_apu has a reg or ranges
property, but no unit name
Warning (unit_address_vs_reg): Node /amba has a reg or ranges property,
but no unit name
Warning (unit_address_vs_reg): Node /amba/usb at fe200000 has a unit name,
but no reg property
Warning (unit_address_vs_reg): Node /amba/usb at fe300000 has a unit name,
but no reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-video0channel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-video1channel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-video2channel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-graphicschannel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-audio0channel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node
/amba/dma at fd4c0000/dma-audio1channel at fd4c0000 has a unit name, but no
reg property
Warning (unit_address_vs_reg): Node /memory has a reg or ranges
property, but no unit name

This patch is fixing them.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-ep108.dts            |  2 +-
 arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts |  2 +-
 arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts |  2 +-
 arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts |  2 +-
 arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts |  2 +-
 arch/arm/dts/zynqmp-zcu102.dts           |  2 +-
 arch/arm/dts/zynqmp.dtsi                 | 20 ++++++++++----------
 7 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/arch/arm/dts/zynqmp-ep108.dts b/arch/arm/dts/zynqmp-ep108.dts
index 0bbf9a759756..751717028b50 100644
--- a/arch/arm/dts/zynqmp-ep108.dts
+++ b/arch/arm/dts/zynqmp-ep108.dts
@@ -31,7 +31,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x40000000>;
 	};
diff --git a/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts b/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
index c68a41bea79a..f3bca60c7e8d 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
@@ -34,7 +34,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x80000000>, <0x8 0x00000000 0x0 0x80000000>;
 	};
diff --git a/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts b/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
index 3fdfcc8a11ae..09a114be1610 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
@@ -36,7 +36,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x80000000>, <0x8 0x00000000 0x0 0x80000000>;
 	};
diff --git a/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts b/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
index 03f1ad7934c1..1f03a94820e2 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
@@ -41,7 +41,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x80000000>, <0x8 0x00000000 0x0 0x80000000>;
 	};
diff --git a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
index d754f9f9040a..698e72e0c5d0 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
@@ -32,7 +32,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x80000000>, <0x8 0x00000000 0x0 0x80000000>;
 	};
diff --git a/arch/arm/dts/zynqmp-zcu102.dts b/arch/arm/dts/zynqmp-zcu102.dts
index de9960245664..b3f5eaacaefc 100644
--- a/arch/arm/dts/zynqmp-zcu102.dts
+++ b/arch/arm/dts/zynqmp-zcu102.dts
@@ -35,7 +35,7 @@
 		stdout-path = "serial0:115200n8";
 	};
 
-	memory {
+	memory at 0 {
 		device_type = "memory";
 		reg = <0x0 0x0 0x0 0x80000000>, <0x8 0x00000000 0x0 0x80000000>;
 	};
diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi
index 619450e1ba92..ac17c4723a13 100644
--- a/arch/arm/dts/zynqmp.dtsi
+++ b/arch/arm/dts/zynqmp.dtsi
@@ -243,7 +243,7 @@
 			     <1 10 0xf01>;
 	};
 
-	amba_apu: amba_apu {
+	amba_apu: amba_apu at 0 {
 		compatible = "simple-bus";
 		#address-cells = <2>;
 		#size-cells = <1>;
@@ -262,7 +262,7 @@
 		};
 	};
 
-	amba: amba {
+	amba: amba at 0 {
 		compatible = "simple-bus";
 		u-boot,dm-pre-reloc;
 		#address-cells = <2>;
@@ -800,7 +800,7 @@
 			power-domains = <&pd_uart1>;
 		};
 
-		usb0: usb at fe200000 {
+		usb0: usb0 {
 			#address-cells = <2>;
 			#size-cells = <1>;
 			status = "disabled";
@@ -821,7 +821,7 @@
 			};
 		};
 
-		usb1: usb at fe300000 {
+		usb1: usb1 {
 			#address-cells = <2>;
 			#size-cells = <1>;
 			status = "disabled";
@@ -938,22 +938,22 @@
 			clock-names = "axi_clk";
 			dma-channels = <6>;
 			#dma-cells = <1>;
-			dma-video0channel at fd4c0000 {
+			dma-video0channel {
 				compatible = "xlnx,video0";
 			};
-			dma-video1channel at fd4c0000 {
+			dma-video1channel {
 				compatible = "xlnx,video1";
 			};
-			dma-video2channel at fd4c0000 {
+			dma-video2channel {
 				compatible = "xlnx,video2";
 			};
-			dma-graphicschannel at fd4c0000 {
+			dma-graphicschannel {
 				compatible = "xlnx,graphics";
 			};
-			dma-audio0channel at fd4c0000 {
+			dma-audio0channel {
 				compatible = "xlnx,audio0";
 			};
-			dma-audio1channel at fd4c0000 {
+			dma-audio1channel {
 				compatible = "xlnx,audio1";
 			};
 		};
-- 
1.9.1

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Tom Rini]

On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:

> DTC 1.2 reports these warnings:
> Warning (unit_address_vs_reg): Node /memory has a reg or ranges
> property, but no unit name
> Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
> but no unit name
> Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
> but no reg property
> 
> This patch is fixing them.
> 
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Have you addressed these in Linux as well?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161111/78c32705/attachment.sig>

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Michal Simek]

Hi,

2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:

> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
>
> > DTC 1.2 reports these warnings:
> > Warning (unit_address_vs_reg): Node /memory has a reg or ranges
> > property, but no unit name
> > Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
> > but no unit name
> > Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
> > but no reg property
> >
> > This patch is fixing them.
> >
> > Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>
> Have you addressed these in Linux as well?
>
>
Going to do so next week.
Both should be in sync.

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Tom Rini]

On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
> Hi,
> 
> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
> 
> > On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
> >
> > > DTC 1.2 reports these warnings:
> > > Warning (unit_address_vs_reg): Node /memory has a reg or ranges
> > > property, but no unit name
> > > Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
> > > but no unit name
> > > Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
> > > but no reg property
> > >
> > > This patch is fixing them.
> > >
> > > Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> >
> > Have you addressed these in Linux as well?
>
> Going to do so next week.
> Both should be in sync.

Agreed.  I'd prefer to pull them in via Linux but at least once they're
on their way into the right tree for upstream we can bring them in here
as well (as I want to ensure future re-syncs are easy).  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161111/efd94431/attachment.sig>

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Michal Simek]

On 11.11.2016 18:33, Tom Rini wrote:
> On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
>> Hi,
>>
>> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
>>
>>> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
>>>
>>>> DTC 1.2 reports these warnings:
>>>> Warning (unit_address_vs_reg): Node /memory has a reg or ranges
>>>> property, but no unit name
>>>> Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
>>>> but no unit name
>>>> Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
>>>> but no reg property
>>>>
>>>> This patch is fixing them.
>>>>
>>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>>>
>>> Have you addressed these in Linux as well?
>>
>> Going to do so next week.
>> Both should be in sync.
> 
> Agreed.  I'd prefer to pull them in via Linux but at least once they're
> on their way into the right tree for upstream we can bring them in here
> as well (as I want to ensure future re-syncs are easy).  Thanks!
> 

I have sent them to Linux too.
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467965.html
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467967.html
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467968.html

For our case we have more dtses in u-boot especially for arm64 than in
Linux.
Also u-boot,dm-pre-reloc AFAIK hasn't been validated via dt mailing
list. I see one thread in 2015 without any resolution.
That's why taking stuff from Linux is sort of goal but nothing what we
can do without additional work.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161115/9692e03d/attachment.sig>

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Tom Rini]

On Tue, Nov 15, 2016 at 03:16:51PM +0100, Michal Simek wrote:
> On 11.11.2016 18:33, Tom Rini wrote:
> > On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
> >> Hi,
> >>
> >> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
> >>
> >>> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
> >>>
> >>>> DTC 1.2 reports these warnings:
> >>>> Warning (unit_address_vs_reg): Node /memory has a reg or ranges
> >>>> property, but no unit name
> >>>> Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
> >>>> but no unit name
> >>>> Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
> >>>> but no reg property
> >>>>
> >>>> This patch is fixing them.
> >>>>
> >>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> >>>
> >>> Have you addressed these in Linux as well?
> >>
> >> Going to do so next week.
> >> Both should be in sync.
> > 
> > Agreed.  I'd prefer to pull them in via Linux but at least once they're
> > on their way into the right tree for upstream we can bring them in here
> > as well (as I want to ensure future re-syncs are easy).  Thanks!
> > 
> 
> I have sent them to Linux too.
> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467965.html
> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467967.html
> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467968.html

OK.

> For our case we have more dtses in u-boot especially for arm64 than in
> Linux.

Intentional or just boards that have not yet made it there?

> Also u-boot,dm-pre-reloc AFAIK hasn't been validated via dt mailing
> list. I see one thread in 2015 without any resolution.

Yes, this is something that might get a re-visit someday, but is indeed
local to U-Boot only for now.

> That's why taking stuff from Linux is sort of goal but nothing what we
> can do without additional work.

So long as everything relevant to upstream has been posted upstream (and
you're saying it is, so good, I'm happy) then we're in good shape here.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161115/a4f2149f/attachment.sig>

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Michal Simek]

On 15.11.2016 15:22, Tom Rini wrote:
> On Tue, Nov 15, 2016 at 03:16:51PM +0100, Michal Simek wrote:
>> On 11.11.2016 18:33, Tom Rini wrote:
>>> On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
>>>> Hi,
>>>>
>>>> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
>>>>
>>>>> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
>>>>>
>>>>>> DTC 1.2 reports these warnings:
>>>>>> Warning (unit_address_vs_reg): Node /memory has a reg or ranges
>>>>>> property, but no unit name
>>>>>> Warning (unit_address_vs_reg): Node /pmu has a reg or ranges property,
>>>>>> but no unit name
>>>>>> Warning (unit_address_vs_reg): Node /fixedregulator at 0 has a unit name,
>>>>>> but no reg property
>>>>>>
>>>>>> This patch is fixing them.
>>>>>>
>>>>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>>>>>
>>>>> Have you addressed these in Linux as well?
>>>>
>>>> Going to do so next week.
>>>> Both should be in sync.
>>>
>>> Agreed.  I'd prefer to pull them in via Linux but at least once they're
>>> on their way into the right tree for upstream we can bring them in here
>>> as well (as I want to ensure future re-syncs are easy).  Thanks!
>>>
>>
>> I have sent them to Linux too.
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467965.html
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467967.html
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/467968.html
> 
> OK.
> 
>> For our case we have more dtses in u-boot especially for arm64 than in
>> Linux.
> 
> Intentional or just boards that have not yet made it there?

Simply because getting these stuff to U-Boot is easier then to Linux.

>> Also u-boot,dm-pre-reloc AFAIK hasn't been validated via dt mailing
>> list. I see one thread in 2015 without any resolution.
> 
> Yes, this is something that might get a re-visit someday, but is indeed
> local to U-Boot only for now.

Adding this to chosen node with list of IPs should be IMHO reasonable
solution.

>> That's why taking stuff from Linux is sort of goal but nothing what we
>> can do without additional work.
> 
> So long as everything relevant to upstream has been posted upstream (and
> you're saying it is, so good, I'm happy) then we're in good shape here.

Trying to keep all stuff in sync as much as possible and definitely we
need to do more Linux upstreaming.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161115/a83e71c9/attachment.sig>

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Masahiro Yamada]

Hi.


2016-11-16 4:01 GMT+09:00 Michal Simek <monstr@monstr.eu>:
> On 15.11.2016 15:22, Tom Rini wrote:
>> On Tue, Nov 15, 2016 at 03:16:51PM +0100, Michal Simek wrote:
>>> On 11.11.2016 18:33, Tom Rini wrote:
>>>> On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
>>>>> Hi,
>>>>>
>>>>> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
>>>>>
>>>>>> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
>>>>>>
>>>>>>> DTC 1.2 reports these warnings:



DTC 1.2 ?

Do you use such old DTC?



The top-level Makefile of U-Boot requires DTC 1.4 or newer,
so you cannot use DTC 1.2 from U-Boot build system.


checkdtc:
       @if test $(call dtc-version) -lt 0104; then \
       echo '*** Your dtc is too old, please upgrade to dtc 1.4 or newer'; \
       false; \
       fi



Did you run DTC directly, and were you hit by a problem?



-- 
Best Regards
Masahiro Yamada

[U-Boot] [PATCH 1/2] ARM: zynq: Remove DTC 1.2 warnings

[Michal Simek]

On 16.11.2016 01:34, Masahiro Yamada wrote:
> Hi.
> 
> 
> 2016-11-16 4:01 GMT+09:00 Michal Simek <monstr@monstr.eu>:
>> On 15.11.2016 15:22, Tom Rini wrote:
>>> On Tue, Nov 15, 2016 at 03:16:51PM +0100, Michal Simek wrote:
>>>> On 11.11.2016 18:33, Tom Rini wrote:
>>>>> On Fri, Nov 11, 2016 at 06:32:15PM +0100, Michal Simek wrote:
>>>>>> Hi,
>>>>>>
>>>>>> 2016-11-11 17:04 GMT+01:00 Tom Rini <trini@konsulko.com>:
>>>>>>
>>>>>>> On Fri, Nov 11, 2016 at 01:26:05PM +0100, Michal Simek wrote:
>>>>>>>
>>>>>>>> DTC 1.2 reports these warnings:
> 
> 
> 
> DTC 1.2 ?
> 
> Do you use such old DTC?
> 
> 
> 
> The top-level Makefile of U-Boot requires DTC 1.4 or newer,
> so you cannot use DTC 1.2 from U-Boot build system.
> 
> 
> checkdtc:
>        @if test $(call dtc-version) -lt 0104; then \
>        echo '*** Your dtc is too old, please upgrade to dtc 1.4 or newer'; \
>        false; \
>        fi
> 
> 
> 
> Did you run DTC directly, and were you hit by a problem?

That was typo - it is 1.4.2 dtc.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161116/de7e0616/attachment.sig>