In message 5BF78BCE8D9BF14A83F836BD9E3916BA23C0BD@blrms.slti.sanyo.co.in you wrote:
I am trying to write a driver for arm amba pl031 rtc module. i need to setup rtc controller to enable it. I need to call it during boot sequence. But in lib_arm/board.c we do not have a call to rtc_init func.
This is intentional. U-Boot does not initialize all hardware on the system, it initializes only such hardware which it needs for it's own operation.
To initialize the RTC, the "date reset" command will be used; this then includes the required init code.
i checked blackfin boot up sequence and they have something like below in uboot-1.3.3/lib_blackfin/board.c @ line 315
checkboard(); #if defined(CONFIG_RTC_BFIN) && defined(CONFIG_CMD_DATE) rtc_init(); #endif timer_init();
This is bad. This code violates the U-Boot design principles and shall be removed.
Mike, can you please take care to get rid of this code in the next release? Thanks in advance.
Shouldnt it be implemented for arm boot up sequence also.
No, definitely not.
Can i implement the same and send the patch to uboot. something like below in uboot-1.3.3/lib_arm/board.c @ line 267
No, please don't. I will reject any such patches.
Best regards,
Wolfgang Denk