[U-Boot] About generic board

[Simon Glass]

+Mailing list

On 8 May 2015 at 00:56,  <uboot at andestech.com> wrote:
> Dear Simon Glass,
>
>
>
> This is Rick.
>

Hi Rick,

>
>
> I am U-Boot architecture and board maintainer of ndsd32.
>
>
>
> I have read your README.generic-board and trying to deprecat
> arch/nds32/lib/board.c in favour of common/board_f.c and common/board_r.c
>
>
>
> May I ask you some questions about that ?
>
>
>
>
>
> Question 1
>
> How can let nds32 become the supported architecture in relative code of
> common/board_f.c as below
>
>
>
> #if defined(CONFIG_ARM) || defined(CONFIG_MIPS) || defined(CONFIG_BLACKFIN)
> // I want add || defined(CONFIG_NDS32)
>  timer_init,  /* initialize timer */
> #endif
>
>
>
> #if defined(CONFIG_ARM) || defined(CONFIG_X86) || defined(CONFIG_MICROBLAZE)
> || defined(CONFIG_AVR32)    // I want add || defined(CONFIG_NDS32)
>  dram_init,  /* configure available RAM banks */
> #endif
>
>
>
> Question 2
>
> How can let nds32 become the supported architecture in relative code of
> common/board_r.c as below
>
>
>
> static int initr_reloc_global_data(void)
> {
> #ifdef __ARM__                                                    // I want
> add defined(__ARM__) || defined(__NDS32__)
>  monitor_flash_len = _end - __image_copy_start;
> #elif !defined(CONFIG_SANDBOX) && !defined(CONFIG_NIOS2)
>  monitor_flash_len = (ulong)&__init_end - gd->relocaddr;
> #endif
>
>
>
> Above are the generic code that I think I can not modify basically. So I get
> stuck here.
>
>
>
> Shall I ask you to modify for me ?
>
> Or I can modify the relative generic code into my patch by myself and send
> to you later for reviewing and approving ?

It's fine to modify it - but please do it in a separate patch. Thanks
for taking this on.

>
>
>
> Thank you,
>
>
>
> Best Regards,
>
>
>
> Rick
>
>

Regards,
Simon