On Mon, May 5, 2014 at 10:26 AM, Tom Rini trini@ti.com wrote:
- Drop DEBUG
- Drop defines we can use the default of.
- Drop CONFIG_GICV3 and related defines, we aren't using interrupts and do not need to configure the interrupt controller.
- Provide a larger malloc pool.
- Correct default locations for kernel / initrd / device tree
Signed-off-by: Tom Rini trini@ti.com
[...]
-#define CONFIG_GICV3
/*#define CONFIG_ARMV8_SWITCH_TO_EL1*/
/*#define CONFIG_SYS_GENERIC_BOARD*/ @@ -94,20 +90,11 @@ /* Generic Timer Definitions */ #define COUNTER_FREQUENCY (0x1800000) /* 24MHz */
-/* Generic Interrupt Controller Definitions */ -#ifdef CONFIG_GICV3 -#define GICD_BASE (0x2f000000) -#define GICR_BASE (0x2f100000) -#else -#define GICD_BASE (0x2C001000) -#define GICC_BASE (0x2C002000) -#endif
IIRC, these should be needed if secure to non-secure switching is done.
#define CONFIG_SYS_MEMTEST_START V2M_BASE #define CONFIG_SYS_MEMTEST_END (V2M_BASE + 0x80000000)
/* Size of malloc() pool */ -#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 128 * 1024) +#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + (8 << 20))
This seems like another candidate for some sane default value.
/* SMSC91C111 Ethernet Configuration */ #define CONFIG_SMC91111 1 @@ -121,7 +108,6 @@ #define CONFIG_CONS_INDEX 0
#define CONFIG_BAUDRATE 115200 -#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 } #define CONFIG_SYS_SERIAL0 V2M_UART0 #define CONFIG_SYS_SERIAL1 V2M_UART1
@@ -166,10 +152,9 @@
/* Initial environment variables */ #define CONFIG_EXTRA_ENV_SETTINGS \
"kernel_addr=0x200000\0" \"initrd_addr=0xa00000\0" \"initrd_size=0x2000000\0" \"fdt_addr=0x100000\0" \
"kernel_addr=0x80080000\0" \"initrd_addr=0x90000000\0" \"fdt_addr=0x88000000\0" \
Don't we want the _r variants here?
Rob