* [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1
@ 2016-08-14 19:34 Fabio Estevam
2016-08-14 19:34 ` [PATCH 2/3] ARM: dts: imx23: Fix build warnings " Fabio Estevam
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Fabio Estevam @ 2016-08-14 19:34 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
Fix the following build warnings with W=1:
Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/skeleton.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/skeleton.dtsi b/arch/arm/boot/dts/skeleton.dtsi
index b41d241..a20da0a 100644
--- a/arch/arm/boot/dts/skeleton.dtsi
+++ b/arch/arm/boot/dts/skeleton.dtsi
@@ -9,5 +9,5 @@
#size-cells = <1>;
chosen { };
aliases { };
- memory { device_type = "memory"; reg = <0 0>; };
+ memory at 0 { device_type = "memory"; reg = <0 0>; };
};
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] ARM: dts: imx23: Fix build warnings with W=1
2016-08-14 19:34 [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1 Fabio Estevam
@ 2016-08-14 19:34 ` Fabio Estevam
2016-08-14 19:34 ` [PATCH 3/3] ARM: dts: imx28: " Fabio Estevam
2016-08-14 20:24 ` [PATCH 1/3] ARM: dts: skeleton: Fix build warning " Stefan Wahren
2 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2016-08-14 19:34 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
Fix the following build warnings with W=1:
Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
While at it, also pass the 'device_type' property as recommended by the
ePAPR specification.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/imx23-evk.dts | 3 ++-
arch/arm/boot/dts/imx23-olinuxino.dts | 3 ++-
arch/arm/boot/dts/imx23-sansa.dts | 3 ++-
arch/arm/boot/dts/imx23-stmp378x_devb.dts | 3 ++-
arch/arm/boot/dts/imx23-xfi3.dts | 3 ++-
5 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index 57e2997..bd06402 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -16,7 +16,8 @@
model = "Freescale i.MX23 Evaluation Kit";
compatible = "fsl,imx23-evk", "fsl,imx23";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx23-olinuxino.dts b/arch/arm/boot/dts/imx23-olinuxino.dts
index a8b1c53..109f516 100644
--- a/arch/arm/boot/dts/imx23-olinuxino.dts
+++ b/arch/arm/boot/dts/imx23-olinuxino.dts
@@ -19,7 +19,8 @@
model = "i.MX23 Olinuxino Low Cost Board";
compatible = "olimex,imx23-olinuxino", "fsl,imx23";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
diff --git a/arch/arm/boot/dts/imx23-sansa.dts b/arch/arm/boot/dts/imx23-sansa.dts
index 4ec32f4..9cf40d3 100644
--- a/arch/arm/boot/dts/imx23-sansa.dts
+++ b/arch/arm/boot/dts/imx23-sansa.dts
@@ -48,7 +48,8 @@
model = "SanDisk Sansa Fuze+";
compatible = "sandisk,sansa_fuze_plus", "fsl,imx23";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
diff --git a/arch/arm/boot/dts/imx23-stmp378x_devb.dts b/arch/arm/boot/dts/imx23-stmp378x_devb.dts
index 455169e..aab0293 100644
--- a/arch/arm/boot/dts/imx23-stmp378x_devb.dts
+++ b/arch/arm/boot/dts/imx23-stmp378x_devb.dts
@@ -16,7 +16,8 @@
model = "Freescale STMP378x Development Board";
compatible = "fsl,stmp378x-devb", "fsl,imx23";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
diff --git a/arch/arm/boot/dts/imx23-xfi3.dts b/arch/arm/boot/dts/imx23-xfi3.dts
index 025cf94..2b5df8d 100644
--- a/arch/arm/boot/dts/imx23-xfi3.dts
+++ b/arch/arm/boot/dts/imx23-xfi3.dts
@@ -48,7 +48,8 @@
model = "Creative ZEN X-Fi3";
compatible = "creative,x-fi3", "fsl,imx23";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] ARM: dts: imx28: Fix build warnings with W=1
2016-08-14 19:34 [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1 Fabio Estevam
2016-08-14 19:34 ` [PATCH 2/3] ARM: dts: imx23: Fix build warnings " Fabio Estevam
@ 2016-08-14 19:34 ` Fabio Estevam
2016-08-15 16:01 ` Rob Herring
2016-08-14 20:24 ` [PATCH 1/3] ARM: dts: skeleton: Fix build warning " Stefan Wahren
2 siblings, 1 reply; 7+ messages in thread
From: Fabio Estevam @ 2016-08-14 19:34 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
Fix the following build warnings with W=1:
Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
While at it, also pass the 'device_type' property as recommended by the
ePAPR specification.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/imx28-apf28.dts | 3 ++-
arch/arm/boot/dts/imx28-apx4devkit.dts | 3 ++-
arch/arm/boot/dts/imx28-cfa10036.dts | 3 ++-
arch/arm/boot/dts/imx28-duckbill.dts | 3 ++-
arch/arm/boot/dts/imx28-eukrea-mbmx283lc.dts | 3 ++-
arch/arm/boot/dts/imx28-eukrea-mbmx287lc.dts | 3 ++-
arch/arm/boot/dts/imx28-evk.dts | 3 ++-
arch/arm/boot/dts/imx28-m28.dtsi | 3 ++-
arch/arm/boot/dts/imx28-m28cu3.dts | 3 ++-
arch/arm/boot/dts/imx28-sps1.dts | 3 ++-
arch/arm/boot/dts/imx28-tx28.dts | 3 ++-
11 files changed, 22 insertions(+), 11 deletions(-)
diff --git a/arch/arm/boot/dts/imx28-apf28.dts b/arch/arm/boot/dts/imx28-apf28.dts
index 070e59c..b6976fb 100644
--- a/arch/arm/boot/dts/imx28-apf28.dts
+++ b/arch/arm/boot/dts/imx28-apf28.dts
@@ -16,7 +16,8 @@
model = "Armadeus Systems APF28 module";
compatible = "armadeus,imx28-apf28", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-apx4devkit.dts b/arch/arm/boot/dts/imx28-apx4devkit.dts
index 1092b76..ce6115e 100644
--- a/arch/arm/boot/dts/imx28-apx4devkit.dts
+++ b/arch/arm/boot/dts/imx28-apx4devkit.dts
@@ -5,7 +5,8 @@
model = "Bluegiga APX4 Development Kit";
compatible = "bluegiga,apx4devkit", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
diff --git a/arch/arm/boot/dts/imx28-cfa10036.dts b/arch/arm/boot/dts/imx28-cfa10036.dts
index 570aa33..d016a3b 100644
--- a/arch/arm/boot/dts/imx28-cfa10036.dts
+++ b/arch/arm/boot/dts/imx28-cfa10036.dts
@@ -16,7 +16,8 @@
model = "Crystalfontz CFA-10036 Board";
compatible = "crystalfontz,cfa10036", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-duckbill.dts b/arch/arm/boot/dts/imx28-duckbill.dts
index ce1a7ef..b5bc3f5 100644
--- a/arch/arm/boot/dts/imx28-duckbill.dts
+++ b/arch/arm/boot/dts/imx28-duckbill.dts
@@ -16,7 +16,8 @@
model = "I2SE Duckbill";
compatible = "i2se,duckbill", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-eukrea-mbmx283lc.dts b/arch/arm/boot/dts/imx28-eukrea-mbmx283lc.dts
index 7c1572c..28dab6d 100644
--- a/arch/arm/boot/dts/imx28-eukrea-mbmx283lc.dts
+++ b/arch/arm/boot/dts/imx28-eukrea-mbmx283lc.dts
@@ -23,7 +23,8 @@
model = "Eukrea Electromatique MBMX283LC";
compatible = "eukrea,mbmx283lc", "eukrea,mbmx28lc", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x04000000>;
};
};
diff --git a/arch/arm/boot/dts/imx28-eukrea-mbmx287lc.dts b/arch/arm/boot/dts/imx28-eukrea-mbmx287lc.dts
index e773144..08125e5 100644
--- a/arch/arm/boot/dts/imx28-eukrea-mbmx287lc.dts
+++ b/arch/arm/boot/dts/imx28-eukrea-mbmx287lc.dts
@@ -22,7 +22,8 @@
model = "Eukrea Electromatique MBMX287LC";
compatible = "eukrea,mbmx287lc", "eukrea,mbmx283lc", "eukrea,mbmx28lc", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
};
diff --git a/arch/arm/boot/dts/imx28-evk.dts b/arch/arm/boot/dts/imx28-evk.dts
index a5ba669..4cd8216 100644
--- a/arch/arm/boot/dts/imx28-evk.dts
+++ b/arch/arm/boot/dts/imx28-evk.dts
@@ -16,7 +16,8 @@
model = "Freescale i.MX28 Evaluation Kit";
compatible = "fsl,imx28-evk", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-m28.dtsi b/arch/arm/boot/dts/imx28-m28.dtsi
index 214bb15..1bcba90 100644
--- a/arch/arm/boot/dts/imx28-m28.dtsi
+++ b/arch/arm/boot/dts/imx28-m28.dtsi
@@ -15,7 +15,8 @@
model = "DENX M28";
compatible = "denx,m28", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-m28cu3.dts b/arch/arm/boot/dts/imx28-m28cu3.dts
index 2df63be..c4c4cc7 100644
--- a/arch/arm/boot/dts/imx28-m28cu3.dts
+++ b/arch/arm/boot/dts/imx28-m28cu3.dts
@@ -16,7 +16,8 @@
model = "MSR M28CU3";
compatible = "msr,m28cu3", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-sps1.dts b/arch/arm/boot/dts/imx28-sps1.dts
index 0ce3cb8..5ddfb16 100644
--- a/arch/arm/boot/dts/imx28-sps1.dts
+++ b/arch/arm/boot/dts/imx28-sps1.dts
@@ -16,7 +16,8 @@
model = "SchulerControl GmbH, SC SPS 1";
compatible = "schulercontrol,imx28-sps1", "fsl,imx28";
- memory {
+ memory at 40000000 {
+ device_type = "memory";
reg = <0x40000000 0x08000000>;
};
diff --git a/arch/arm/boot/dts/imx28-tx28.dts b/arch/arm/boot/dts/imx28-tx28.dts
index 0ebbc83..b4d4dbb 100644
--- a/arch/arm/boot/dts/imx28-tx28.dts
+++ b/arch/arm/boot/dts/imx28-tx28.dts
@@ -35,7 +35,8 @@
usbotg = &usb0;
};
- memory {
+ memory at 0 {
+ device_type = "memory";
reg = <0 0>; /* will be filled in by U-Boot */
};
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1
2016-08-14 19:34 [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1 Fabio Estevam
2016-08-14 19:34 ` [PATCH 2/3] ARM: dts: imx23: Fix build warnings " Fabio Estevam
2016-08-14 19:34 ` [PATCH 3/3] ARM: dts: imx28: " Fabio Estevam
@ 2016-08-14 20:24 ` Stefan Wahren
2016-08-14 20:46 ` Fabio Estevam
2 siblings, 1 reply; 7+ messages in thread
From: Stefan Wahren @ 2016-08-14 20:24 UTC (permalink / raw)
To: linux-arm-kernel
Hi Fabio,
> Fabio Estevam <festevam@gmail.com> hat am 14. August 2016 um 21:34
> geschrieben:
>
>
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> Fix the following build warnings with W=1:
>
> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but
> no unit name
according to this thread [1] this patch shouldn't be applied yet. Patch 2 and 3
are okay.
Btw: Do you have any comments about this patch series [2]?
[1] -
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-March/418430.html
[2] - http://marc.info/?l=linux-arm-kernel&m=147048830916087
>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> ---
> arch/arm/boot/dts/skeleton.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/skeleton.dtsi b/arch/arm/boot/dts/skeleton.dtsi
> index b41d241..a20da0a 100644
> --- a/arch/arm/boot/dts/skeleton.dtsi
> +++ b/arch/arm/boot/dts/skeleton.dtsi
> @@ -9,5 +9,5 @@
> #size-cells = <1>;
> chosen { };
> aliases { };
> - memory { device_type = "memory"; reg = <0 0>; };
> + memory at 0 { device_type = "memory"; reg = <0 0>; };
> };
> --
> 1.9.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1
2016-08-14 20:24 ` [PATCH 1/3] ARM: dts: skeleton: Fix build warning " Stefan Wahren
@ 2016-08-14 20:46 ` Fabio Estevam
0 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2016-08-14 20:46 UTC (permalink / raw)
To: linux-arm-kernel
Hi Stefan,
On Sun, Aug 14, 2016 at 5:24 PM, Stefan Wahren <stefan.wahren@i2se.com> wrote:
> according to this thread [1] this patch shouldn't be applied yet. Patch 2 and 3
> are okay.
Ok, thanks. So patch 1/3 can be discarded then.
> Btw: Do you have any comments about this patch series [2]?
I tried to fix this sometime ago, but never had a chance to address
the suggestions from Ulf. So better wait for the comments from the mmc
folks.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] ARM: dts: imx28: Fix build warnings with W=1
2016-08-14 19:34 ` [PATCH 3/3] ARM: dts: imx28: " Fabio Estevam
@ 2016-08-15 16:01 ` Rob Herring
2016-08-15 16:07 ` Fabio Estevam
0 siblings, 1 reply; 7+ messages in thread
From: Rob Herring @ 2016-08-15 16:01 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Aug 14, 2016 at 2:34 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> Fix the following build warnings with W=1:
>
> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
>
> While at it, also pass the 'device_type' property as recommended by the
> ePAPR specification.
I'm surprised it worked without unless you were relying on
skeleton.dtsi. If that's the case, please remove the dependency on
skeleton.dtsi while at it.
Rob
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] ARM: dts: imx28: Fix build warnings with W=1
2016-08-15 16:01 ` Rob Herring
@ 2016-08-15 16:07 ` Fabio Estevam
0 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2016-08-15 16:07 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 15, 2016 at 1:01 PM, Rob Herring <robh+dt@kernel.org> wrote:
> On Sun, Aug 14, 2016 at 2:34 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> From: Fabio Estevam <fabio.estevam@nxp.com>
>>
>> Fix the following build warnings with W=1:
>>
>> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
>>
>> While at it, also pass the 'device_type' property as recommended by the
>> ePAPR specification.
>
> I'm surprised it worked without unless you were relying on
> skeleton.dtsi. If that's the case, please remove the dependency on
> skeleton.dtsi while at it.
Yes, imx28.dtsi includes skeleton.dtsi. Will remove this dependency
and submit a v2.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-08-15 16:07 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-14 19:34 [PATCH 1/3] ARM: dts: skeleton: Fix build warning with W=1 Fabio Estevam
2016-08-14 19:34 ` [PATCH 2/3] ARM: dts: imx23: Fix build warnings " Fabio Estevam
2016-08-14 19:34 ` [PATCH 3/3] ARM: dts: imx28: " Fabio Estevam
2016-08-15 16:01 ` Rob Herring
2016-08-15 16:07 ` Fabio Estevam
2016-08-14 20:24 ` [PATCH 1/3] ARM: dts: skeleton: Fix build warning " Stefan Wahren
2016-08-14 20:46 ` Fabio Estevam
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).