Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1427502 > unrolled thread

Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b

Started byDuc Dang <dhdang@apm.com>
First post2016-06-21 11:30 +0200
Last post2016-06-21 23:50 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b Duc Dang <dhdang@apm.com> - 2016-06-21 11:30 +0200
    Re: Warning when compiling DTB on X-Gene platform with dtc  1.4.1-g53bf130b Andrew Lunn <andrew@lunn.ch> - 2016-06-21 15:30 +0200
      Re: Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b Duc Dang <dhdang@apm.com> - 2016-06-21 23:50 +0200

#1427502 — Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b

FromDuc Dang <dhdang@apm.com>
Date2016-06-21 11:30 +0200
SubjectWarning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b
Message-ID<rMqam-3Th-17@gated-at.bofh.it>
Hi Rob,

I got warning on unit_address_vs_reg similar to this
https://lkml.org/lkml/2016/3/7/43 when using v4.7-rc2 dtc to build dtb
for X-Gene Mustang:

dhdang@xgene-next=>./scripts/dtc/dtc -O dtb -R 16 -o mustang.dtb
arch/arm64/boot/dts/apm/apm-mustang.dts
  Warning (unit_address_vs_reg): Node /soc/reboot@17000014 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
  Warning (unit_address_vs_reg): Node /gpio-keys/button@1 has a unit
name, but no reg property

There are nodes that I cannot change the node-name due to backward
compatible with old firmware reason (/soc/reboot@17000014 as an
example). Is there any other way to avoid the warning?

Regards,
Duc Dang.

[toc] | [next] | [standalone]


#1427738 — Re: Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b

FromAndrew Lunn <andrew@lunn.ch>
Date2016-06-21 15:30 +0200
SubjectRe: Warning when compiling DTB on X-Gene platform with dtc 1.4.1-g53bf130b
Message-ID<rMtUB-6nL-1@gated-at.bofh.it>
In reply to#1427502
On Tue, Jun 21, 2016 at 02:02:44AM -0700, Duc Dang wrote:
> Hi Rob,
> 
> I got warning on unit_address_vs_reg similar to this
> https://lkml.org/lkml/2016/3/7/43 when using v4.7-rc2 dtc to build dtb
> for X-Gene Mustang:
> 
> dhdang@xgene-next=>./scripts/dtc/dtc -O dtb -R 16 -o mustang.dtb
> arch/arm64/boot/dts/apm/apm-mustang.dts
>   Warning (unit_address_vs_reg): Node /soc/reboot@17000014 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
>   Warning (unit_address_vs_reg): Node /gpio-keys/button@1 has a unit
> name, but no reg property
> 
> There are nodes that I cannot change the node-name due to backward
> compatible with old firmware reason (/soc/reboot@17000014 as an
> example). Is there any other way to avoid the warning?

I think you have that backwards around. The Kernel needs to be
backwards compatible with old DT blobs. DT blobs don't generally need
to be backwards compatible to older kernels.

However, the kernel should however be happy with or without the unit
name. The unit name is pointless in this case.

	Andrew

[toc] | [prev] | [next] | [standalone]


#1428216

FromDuc Dang <dhdang@apm.com>
Date2016-06-21 23:50 +0200
Message-ID<rMBIu-2Rm-21@gated-at.bofh.it>
In reply to#1427738
On Tue, Jun 21, 2016 at 6:25 AM, Andrew Lunn <andrew@lunn.ch> wrote:
> On Tue, Jun 21, 2016 at 02:02:44AM -0700, Duc Dang wrote:
>> Hi Rob,
>>
>> I got warning on unit_address_vs_reg similar to this
>> https://lkml.org/lkml/2016/3/7/43 when using v4.7-rc2 dtc to build dtb
>> for X-Gene Mustang:
>>
>> dhdang@xgene-next=>./scripts/dtc/dtc -O dtb -R 16 -o mustang.dtb
>> arch/arm64/boot/dts/apm/apm-mustang.dts
>>   Warning (unit_address_vs_reg): Node /soc/reboot@17000014 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
>>   Warning (unit_address_vs_reg): Node /gpio-keys/button@1 has a unit
>> name, but no reg property
>>
>> There are nodes that I cannot change the node-name due to backward
>> compatible with old firmware reason (/soc/reboot@17000014 as an
>> example). Is there any other way to avoid the warning?
>
> I think you have that backwards around. The Kernel needs to be
> backwards compatible with old DT blobs. DT blobs don't generally need
> to be backwards compatible to older kernels.
>
> However, the kernel should however be happy with or without the unit
> name. The unit name is pointless in this case.

I should be more specific: The old firmware I am talking about is the bootloader
(U-Boot). U-Boot relies on the full node names to do device tree
fix-up, so changing
the node names will break the fix-up operation in already-released U-Boot.

>
>         Andrew

Regards,
Duc Dang.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web