[U-Boot-Users] U-boot ColdFire development

[Liew Tsi Chung-r5aahp]

Stefan,

Here is the test version for 5329 u-boot source at Freescale.
http://opensource.freescale.com/git?p=u-boot-coldfire.git;a=summary

Added:
board/freescale/M5329EVB/Makefile config.mk flash.c m5329evb.c and
u-boot.lds cpu/mcf532x/Makefile config.mk cpu.c cpu_init.c speed.c
start.S drivers/net/mcffec.c drivers/serial/mcfserial.c
include/asm-m68k/immap_5329.h m5329.h mcfrtc.h
include/configs/M5329EVB.h lib_m68k/interrupts.c rtc/rtc.c

Modified:
Makefile
MAKEALL
README
common/cmd_bdinfo.c cmd_mii.c
include/asm-m68k/byteorder.h fec.h io.h mcftimer.h mcfuart.h ptrace.h
u-boot.h lib_m68k/board.c time.c

I had placed the mcfserial.c under drivers/serial and mcffec.c under
drivers/net. The rtc.c is under rtc/.

1. I do have one more doubt about placing the following defines as we
discussed early.

> /* Timer */
> #define CONFIG_MCFTMR
> #ifdef CONFIG_MCFTMR
> #	define CFG_UDELAY_BASE	(0xFC070000)
> #	define CFG_TMR_BASE		(0xFC074000)
> #	define CFG_INTR_BASE	(0xFC048000)
> #	define CFG_TMRINTR_NO	(32)
> #	define CFG_TMRINTR_MASK	(1)
> #endif

If I place the #ifdef CONFIG_MCFTMR ... #endif in say immap_5329.h,
isn't that in time.c must include <asm/immap_5329.h>.

Time.c
#ifdef CONFIG_5329
#include <asm/immap_5329.h>
#endif

#ifdef CONFIG_5272
#include <asm/immap_5272.h>
#endif
...

Is there a way not to include <asm/immap_5xxx.h> in time.c each time a
new processor is added?

2. I seperated the MII feature from fec.c and placed in under
board/freescale/mxxxevb.c. Since the mii phy is board dependent not cpu
dependent, do you agree?

3. EHCI/OTG driver development. Do I need to modify the current
cmd_usb.c to include EHCI or create a new file cmd_hsusb.c?

Regards,
TsiChung