[U-Boot] [PATCH 5/7] sunxi: arm64: correct usage of DT node address in FIT generation

Andre Przywara andre.przywara at arm.com
Tue Dec 19 08:56:24 UTC 2017


Hi Simon,

thanks for going through this!

On 19/12/17 04:24, Simon Glass wrote:
> Hi Andre,
> 
> On 3 December 2017 at 19:05, Andre Przywara <andre.przywara at arm.com> wrote:
>> The DT spec demands a unit-address in a node name to match the "reg"
>> property in that node. Newer dtc versions will throw warnings if this is
>> not the case.
>> Adjust the FIT build script for 64-bit Allwinner boards to remove the
>> bogus addresses from the node names and avoid the warnings.
>> This avoids a warning with recent versions of the dtc tool.
>>
>> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
>> ---
>>  board/sunxi/mksunxi_fit_atf.sh | 16 ++++++++--------
>>  1 file changed, 8 insertions(+), 8 deletions(-)
>>
>> diff --git a/board/sunxi/mksunxi_fit_atf.sh b/board/sunxi/mksunxi_fit_atf.sh
>> index b1d6e0e16a..36abe9efed 100755
>> --- a/board/sunxi/mksunxi_fit_atf.sh
>> +++ b/board/sunxi/mksunxi_fit_atf.sh
>> @@ -21,7 +21,7 @@ cat << __HEADER_EOF
>>         #address-cells = <1>;
>>
>>         images {
>> -               uboot at 1 {
>> +               uboot {
>>                         description = "U-Boot (64-bit)";
>>                         data = /incbin/("u-boot-nodtb.bin");
>>                         type = "standalone";
>> @@ -29,7 +29,7 @@ cat << __HEADER_EOF
>>                         compression = "none";
>>                         load = <0x4a000000>;
>>                 };
>> -               atf at 1 {
>> +               atf {
>>                         description = "ARM Trusted Firmware";
>>                         data = /incbin/("$BL31");
>>                         type = "firmware";
>> @@ -44,7 +44,7 @@ cnt=1
>>  for dtname in $*
>>  do
>>         cat << __FDT_IMAGE_EOF
>> -               fdt@$cnt {
>> +               fdt_$cnt {
>>                         description = "$(basename $dtname .dtb)";
>>                         data = /incbin/("$dtname");
>>                         type = "flat_dt";
>> @@ -57,7 +57,7 @@ done
>>  cat << __CONF_HEADER_EOF
>>         };
>>         configurations {
>> -               default = "config at 1";
>> +               default = "config_1";
> 
> Have you changed from hyphen to underscore here?

Arrgh, indeed. I think that was the first version of the patch, before I
changed to '-'.
Sorry for that, will fix it.

Cheers,
Andre.

> 
>>
>>  __CONF_HEADER_EOF
>>
>> @@ -65,11 +65,11 @@ cnt=1
>>  for dtname in $*
>>  do
>>         cat << __CONF_SECTION_EOF
>> -               config@$cnt {
>> +               config_$cnt {
>>                         description = "$(basename $dtname .dtb)";
>> -                       firmware = "uboot at 1";
>> -                       loadables = "atf at 1";
>> -                       fdt = "fdt@$cnt";
>> +                       firmware = "uboot";
>> +                       loadables = "atf";
>> +                       fdt = "fdt_$cnt";
>>                 };
>>  __CONF_SECTION_EOF
>>         cnt=$((cnt+1))
>> --
>> 2.14.1
>>
> 
> Regards,
> Simon
> 


More information about the U-Boot mailing list