Hi Prafulla, On Wed, Dec 8, 2010 at 1:06 AM, Prafulla Wadaskar <[email protected]> wrote: > This patch adds commonly used macros for ARMADA100 based > baords, Also some code reshuffled and updated for typos and comments > > Signed-off-by: Prafulla Wadaskar <[email protected]> > --- > Changelog v3: > CONFIG_ARCH_CPU_INIT moved out of #ifdef > > Changelog v4: > CONFIG_MFP change to CONFIG_MARVELL_MFP > some whitespaces removed > > include/configs/mv-common.h | 65 ++++++++++++++++++++++++++++++++---------- > 1 files changed, 49 insertions(+), 16 deletions(-) > > diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h > index 0a76163..067527a 100644 > --- a/include/configs/mv-common.h > +++ b/include/configs/mv-common.h > @@ -39,6 +39,7 @@ > #define CONFIG_MARVELL 1 > #define CONFIG_ARM926EJS 1 /* Basic Architecture */ > > +/* ====> Kirkwood Platform Common Definations */ > #if defined(CONFIG_KIRKWOOD) > #define CONFIG_MD5 /* get_random_hex on krikwood needs MD5 support */ > #define CONFIG_KIRKWOOD_EGIGA_INIT /* Enable GbePort0/1 for kernel */ > @@ -54,27 +55,45 @@ > #define CONFIG_SYS_KWD_CONFIG > $(SRCTREE)/$(CONFIG_BOARDDIR)/kwbimage.cfg > #endif /* CONFIG_SYS_KWD_CONFIG */ > > -/* > - * CONFIG_SYS_TEXT_BASE can be defined in board specific header file, if > needed > - */ > -#ifndef CONFIG_SYS_TEXT_BASE > -#define CONFIG_SYS_TEXT_BASE 0x00600000 > -#endif /* CONFIG_SYS_TEXT_BASE */ > +/* Kirkwood has 2k of Security SRAM, use it for SP */ > +#define CONFIG_SYS_INIT_SP_ADDR 0xC8012000 > +#define CONFIG_NR_DRAM_BANKS_MAX 2 > > #define CONFIG_I2C_MVTWSI_BASE KW_TWSI_BASE > -#define MV_UART0_BASE KW_UART0_BASE > +#define MV_UART_CONSOLE_BASE KW_UART0_BASE
Could we just define the uart base as hard coding one here? For it must refer to the .h file in the arch/arm/include directory, which make no sense for the configure file. > #define MV_SATA_BASE KW_SATA_BASE > #define MV_SATA_PORT0_OFFSET KW_SATA_PORT0_OFFSET > #define MV_SATA_PORT1_OFFSET KW_SATA_PORT1_OFFSET > > +/* ====> ARMADA100 Platform Common Definations */ I think you probably need to think to seperate armada100 sepecific and kirkwood specific stuff from mv-common.h now. For those two kind of definition makes mv-common not so common as it is. > +#elif defined (CONFIG_ARMADA100) > + > +#define CONFIG_SYS_TCLK (14745600) /* NS16550 clk config > */ > +#define CONFIG_SYS_HZ_CLOCK (3250000) /* Timer Freq. 3.25MHZ */ > +#define CONFIG_MARVELL_MFP /* Enable mvmfp driver */ > +#define MV_UART_CONSOLE_BASE ARMD1_UART1_BASE > +#define CONFIG_SYS_NS16550_IER (1 << 6) /* Bit 6 in UART_IER register > + represents UART Unit Enable */ > +/* > + * There is no internal RAM in ARMADA100, using DRAM > + * TBD: dcache to be used for this > + */ > +#define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE - > 0x00200000) > +#define CONFIG_NR_DRAM_BANKS_MAX 2 > + > #else > -#error "Unsupported SoC" > +#error "Unsupported SoC Platform..." > #endif > > +/* > + * Custom CONFIG_SYS_TEXT_BASE can be done in <board>.h > + */ > +#ifndef CONFIG_SYS_TEXT_BASE > +#define CONFIG_SYS_TEXT_BASE 0x00600000 > +#endif /* CONFIG_SYS_TEXT_BASE */ > + > /* additions for new ARM relocation support */ > -#define CONFIG_SYS_SDRAM_BASE 0x00000000 > -/* Kirkwood has 2k of Security SRAM, use it for SP */ > -#define CONFIG_SYS_INIT_SP_ADDR 0xC8012000 > +#define CONFIG_SYS_SDRAM_BASE 0x00000000 > > /* > * CLKs configurations > @@ -88,7 +107,7 @@ > #define CONFIG_SYS_NS16550_SERIAL > #define CONFIG_SYS_NS16550_REG_SIZE (-4) > #define CONFIG_SYS_NS16550_CLK CONFIG_SYS_TCLK > -#define CONFIG_SYS_NS16550_COM1 MV_UART0_BASE > +#define CONFIG_SYS_NS16550_COM1 MV_UART_CONSOLE_BASE > > /* > * Serial Port configuration > @@ -156,25 +175,37 @@ > #define CONFIG_CMDLINE_EDITING > #define CONFIG_CONSOLE_INFO_QUIET /* some code reduction */ > #define CONFIG_ARCH_CPU_INIT /* call arch_cpu_init() */ > +#ifndef CONFIG_ARMADA100 /* will be removed latter */ > #define CONFIG_ARCH_MISC_INIT /* call arch_misc_init() */ > +#endif /* CONFIG_ARMADA100 */ > #define CONFIG_BOARD_EARLY_INIT_F /* call board_init_f for early inits */ > #define CONFIG_DISPLAY_CPUINFO /* Display cpu info */ > -#define CONFIG_NR_DRAM_BANKS 4 > #define CONFIG_STACKSIZE 0x00100000 /* regular stack- 1M */ > #define CONFIG_SYS_LOAD_ADDR 0x00800000 /* default load adr- 8M */ > -#define CONFIG_SYS_MEMTEST_START 0x00400000 /* 4M */ > -#define CONFIG_SYS_MEMTEST_END 0x007fffff /*(_8M -1) */ > +#define CONFIG_SYS_MEMTEST_START 0x00800000 /* 8M */ > +#define CONFIG_SYS_MEMTEST_END 0x00ffffff /*(_16M -1) */ > #define CONFIG_SYS_RESET_ADDRESS 0xffff0000 /* Rst Vector Adr */ > #define CONFIG_SYS_MAXARGS 16 /* max number of command args */ > > /* > + * DRAM Banks configuration, Custom config can be done in <board>.h > + */ > +#ifndef CONFIG_NR_DRAM_BANKS > +#define CONFIG_NR_DRAM_BANKS CONFIG_NR_DRAM_BANKS_MAX > +#else > +#if (CONFIG_NR_DRAM_BANKS > CONFIG_NR_DRAM_BANKS_MAX) > +#error CONFIG_NR_DRAM_BANKS Configurated more than available > +#endif > +#endif /* CONFIG_NR_DRAM_BANKS */ > + > +/* > * Ethernet Driver configuration > */ > #ifdef CONFIG_CMD_NET > #define CONFIG_CMD_MII > #define CONFIG_NETCONSOLE /* include NetConsole support */ > #define CONFIG_NET_MULTI /* specify more that one ports available */ > -#define CONFIG_MII /* expose smi ove miiphy interface */ > +#define CONFIG_MII /* expose smi ove miiphy interface */ > #define CONFIG_MVGBE /* Enable Marvell Gbe Controller Driver */ > #define CONFIG_SYS_FAULT_ECHO_LINK_DOWN /* detect link using phy */ > #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ > @@ -232,6 +263,7 @@ > /* > * File system > */ > +#ifndef CONFIG_ARMADA100 /* will be removed latter */ We may need think how to remove it later now... > #define CONFIG_CMD_EXT2 > #define CONFIG_CMD_JFFS2 > #define CONFIG_CMD_FAT > @@ -242,5 +274,6 @@ > #define CONFIG_MTD_PARTITIONS > #define CONFIG_CMD_MTDPARTS > #define CONFIG_LZO > +#endif /* CONFIG_ARMADA100 */ > > #endif /* _MV_COMMON_H */ > -- > 1.5.3.4 Best regards, Lei _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
