[U-Boot] [PATCH v9 05/10] nds32/ag101: lowlevel_init.S of ag101

Macpaul Lin macpaul at gmail.com
Mon May 2 17:37:49 CEST 2011


Hi Wolfgang,

> Patch changelog belongs _below_ the "---" line, not above.

Sorry for this mistaken, will fix in Patch v10

>>  arch/nds32/cpu/n1213/ag101/lowlevel_init.S |  160 ++++++++++++++++++++++++++++
>>  1 files changed, 160 insertions(+), 0 deletions(-)
>>  create mode 100644 arch/nds32/cpu/n1213/ag101/lowlevel_init.S
>
> This file alone makes no sense.  Pluease submit as a single commit
> together with the other board code.

The single file is intend to be reviewed and fix up easyly because I
know there must be some problem of this lowlevel_init.S.
I think this should go with the SoC. So I will merge it into ag101 SoC
level related code.

> ...
>> +     /*
>> +      * relocation, copy ROM code to SDRAM (current at 0x10000000)
>> +      */
>> +     li      $r4, CONFIG_SYS_RELO_ADDR               ! 0x10000000
>> +     li      $r5, 0x0
>> +     la      $r1, relo_base
>> +     sub     $r2, $r0, $r1
>> +     sethi   $r6, hi20(andesboot_end)
>> +     ori     $r6, $r6, lo12(andesboot_end)
>> +     add     $r6, $r6, $r2
>> +1:
>> +     lwi     $r7, [$r5]
>> +     swi     $r7, [$r4]
>> +     addi    $r5, $r5, #4
>> +     addi    $r4, $r4, #4
>> +     blt     $r5, $r6, 1b
>
> You are using  non-standard and completely undocumented CONFIG option
> here, CONFIG_SYS_RELO_ADDR.  Please don't.  I think when adding new
> architecture support we should avaid errors done with other
> architectures in the past (see ARM, MIPS, ...).

Sure.

> Please implement proper relocation to a dynamically determined
> address at the upper end of RAM. Keep in mind that the intention is
> to have only a single lib/board.c isnatead of a collection of
> slightly different arch-specific arch/*/board.c versions.
>
> Best regards,
>
> Wolfgang Denk

Will try to fix it soon.
Thanks.

-- 
Best regards,
Macpaul Lin


More information about the U-Boot mailing list