* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
@ 2016-04-15 10:58 Heiko Schocher
2016-04-15 14:09 ` Bhuvanchandra DV
2016-04-18 15:31 ` Tom Rini
0 siblings, 2 replies; 6+ messages in thread
From: Heiko Schocher @ 2016-04-15 10:58 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
arch/arm/dts/ls1021a-qds.dtsi | 2 +-
arch/arm/dts/vf.dtsi | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
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..99f3269 100644
--- a/arch/arm/dts/vf.dtsi
+++ b/arch/arm/dts/vf.dtsi
@@ -28,7 +28,7 @@
compatible = "simple-bus";
ranges;
- aips0: aips-bus at 40000000 {
+ aips0: aips-bus_40000000 {
compatible = "fsl,aips-bus", "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
@@ -115,7 +115,7 @@
};
};
- aips1: aips-bus at 40080000 {
+ aips1: aips-bus_40080000 {
compatible = "fsl,aips-bus", "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
--
2.5.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
2016-04-15 10:58 [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings Heiko Schocher
@ 2016-04-15 14:09 ` Bhuvanchandra DV
2016-04-18 4:53 ` Heiko Schocher
2016-04-18 15:31 ` Tom Rini
1 sibling, 1 reply; 6+ messages in thread
From: Bhuvanchandra DV @ 2016-04-15 14:09 UTC (permalink / raw)
To: u-boot
On 04/15/2016 04:28 PM, Heiko Schocher wrote:
> 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
May be this fix[1] which Stefan Agner submitted lately is also
applicable here.
[1] https://lkml.org/lkml/2016/4/2/11
>
> 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
>
>
> arch/arm/dts/ls1021a-qds.dtsi | 2 +-
> arch/arm/dts/vf.dtsi | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> 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..99f3269 100644
> --- a/arch/arm/dts/vf.dtsi
> +++ b/arch/arm/dts/vf.dtsi
> @@ -28,7 +28,7 @@
> compatible = "simple-bus";
> ranges;
>
> - aips0: aips-bus at 40000000 {
> + aips0: aips-bus_40000000 {
> compatible = "fsl,aips-bus", "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> @@ -115,7 +115,7 @@
> };
> };
>
> - aips1: aips-bus at 40080000 {
> + aips1: aips-bus_40080000 {
> compatible = "fsl,aips-bus", "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
>
--
Best regards,
Bhuvan
^ permalink raw reply [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
2016-04-15 14:09 ` Bhuvanchandra DV
@ 2016-04-18 4:53 ` Heiko Schocher
2016-04-18 8:37 ` Bhuvanchandra DV
0 siblings, 1 reply; 6+ messages in thread
From: Heiko Schocher @ 2016-04-18 4:53 UTC (permalink / raw)
To: u-boot
Hello Bhuvanchandra DV,
Am 15.04.2016 um 16:09 schrieb Bhuvanchandra DV:
> On 04/15/2016 04:28 PM, Heiko Schocher wrote:
>> 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
>
> May be this fix[1] which Stefan Agner submitted lately is also applicable here.
>
> [1] https://lkml.org/lkml/2016/4/2/11
Yes, better (as it adds the reg property). I hoped for such a comment,
as I cannot search for all missing reg properties!
post a v2 soon.
Thanks!
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
2016-04-18 4:53 ` Heiko Schocher
@ 2016-04-18 8:37 ` Bhuvanchandra DV
2016-04-18 9:22 ` Heiko Schocher
0 siblings, 1 reply; 6+ messages in thread
From: Bhuvanchandra DV @ 2016-04-18 8:37 UTC (permalink / raw)
To: u-boot
On 04/18/2016 10:23 AM, Heiko Schocher wrote:
> Hello Bhuvanchandra DV,
>
> Am 15.04.2016 um 16:09 schrieb Bhuvanchandra DV:
>> On 04/15/2016 04:28 PM, Heiko Schocher wrote:
>>> 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
>>
>> May be this fix[1] which Stefan Agner submitted lately is also
>> applicable here.
>>
>> [1] https://lkml.org/lkml/2016/4/2/11
>
> Yes, better (as it adds the reg property). I hoped for such a comment,
> as I cannot search for all missing reg properties!
>
> post a v2 soon.
Sure! will submit the patch.
>
> Thanks!
>
> bye,
> Heiko
--
Best regards,
Bhuvan
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
2016-04-18 8:37 ` Bhuvanchandra DV
@ 2016-04-18 9:22 ` Heiko Schocher
0 siblings, 0 replies; 6+ messages in thread
From: Heiko Schocher @ 2016-04-18 9:22 UTC (permalink / raw)
To: u-boot
Hello Bhuvanchandra DV,
Am 18.04.2016 um 10:37 schrieb Bhuvanchandra DV:
> On 04/18/2016 10:23 AM, Heiko Schocher wrote:
>> Hello Bhuvanchandra DV,
>>
>> Am 15.04.2016 um 16:09 schrieb Bhuvanchandra DV:
>>> On 04/15/2016 04:28 PM, Heiko Schocher wrote:
>>>> 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
>>>
>>> May be this fix[1] which Stefan Agner submitted lately is also
>>> applicable here.
>>>
>>> [1] https://lkml.org/lkml/2016/4/2/11
>>
>> Yes, better (as it adds the reg property). I hoped for such a comment,
>> as I cannot search for all missing reg properties!
>>
>> post a v2 soon.
>
> Sure! will submit the patch.
Worked in the changes from the post on lkml, I post a v2, thanks!
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings
2016-04-15 10:58 [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings Heiko Schocher
2016-04-15 14:09 ` Bhuvanchandra DV
@ 2016-04-18 15:31 ` Tom Rini
1 sibling, 0 replies; 6+ messages in thread
From: Tom Rini @ 2016-04-18 15:31 UTC (permalink / raw)
To: u-boot
On Fri, Apr 15, 2016 at 12:58:15PM +0200, Heiko Schocher wrote:
> Fix following warnings for all vf6 based boards:
We're going to defer on this patch for now, 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/20160418/da69bcc9/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-04-18 15:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-15 10:58 [U-Boot] [PATCH] arm: dts: vf6: fix DTC unit warnings Heiko Schocher
2016-04-15 14:09 ` Bhuvanchandra DV
2016-04-18 4:53 ` Heiko Schocher
2016-04-18 8:37 ` Bhuvanchandra DV
2016-04-18 9:22 ` Heiko Schocher
2016-04-18 15:31 ` Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox