[U-Boot] [PATCH 1/2] [NEXT] orion5x: fix relocation-incompatible code
Albert ARIBAUD
albert.aribaud at free.fr
Thu Sep 23 11:32:58 CEST 2010
Hi Prafulla,
Le 23/09/2010 11:16, Prafulla Wadaskar a écrit :
>> +/* Disable window 7, configure it for FLASH, enable it. */
>
> Can you pls fix the ordering here so that win6/7 appears last in the code?
Of course.
>> @@ -265,6 +283,8 @@ int arch_misc_init(void)
>> writel(ORION5X_MPP16_23, ORION5X_MPP_BASE+0x50);
>> writel(ORION5X_GPIO_OUT_ENABLE, ORION5X_GPIO_BASE+0x04);
>>
>> + /* initialize timer */
>> + timer_init_r();
>
> Using CONFIG_MISC_INIT_R in the board configuration makes more sense here
> And you can put this function in timer.c only and avoid exporting API through cpu.h
Ok for misc_init_r() rather than arch_misc_init(). However I think this
function is not timer-specific and thus belongs more to cpu.c than timer.c.
>> \ No newline at end of file
>
> Pls fix this
Will do.
> Regards..
> Prafulla . .
Thanks!
Amicalement,
--
Albert.
More information about the U-Boot
mailing list