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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot