On 08/02/2013 03:26 PM, Tom Rini wrote:
> Update omap5_common.h to use ti_armv7_common.h, and in turn update
> dra7xx_evm.h and omap5_uevm.h slightly.  The biggest changes here are

The commit headline does not indicate anything about dra7xx.  So dra7xx dev's 
might over look this change.

> that IP blocks which exist on the platform, and had clocks enabled,
> now have the drivers being built as well.
>
> Signed-off-by: Tom Rini <tr...@ti.com>
> ---
>  include/configs/dra7xx_evm.h   |   11 +--
>  include/configs/omap5_common.h |  188 
> +++++-----------------------------------
>  include/configs/omap5_uevm.h   |    8 +-
>  3 files changed, 31 insertions(+), 176 deletions(-)
>
> diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
> index 51be73d..58786ff 100644
> --- a/include/configs/dra7xx_evm.h
> +++ b/include/configs/dra7xx_evm.h
> @@ -12,21 +12,18 @@
>  #ifndef __CONFIG_DRA7XX_EVM_H
>  #define __CONFIG_DRA7XX_EVM_H
>  
> -/* High Level Configuration Options */
> -#define CONFIG_DRA7XX                /* in a TI DRA7XX core */
> -#define CONFIG_ENV_IS_NOWHERE                /* For now. */
> -
> -#include <configs/omap5_common.h>
> +#define CONFIG_DRA7XX
>  
> -#define CONFIG_SYS_PROMPT            "DRA752 EVM # "
> +#define CONFIG_ENV_IS_NOWHERE                /* For now. */
>  
> +#define CONSOLEDEV                   "ttyO0"
>  #define CONFIG_CONS_INDEX            1
>  #define CONFIG_SYS_NS16550_COM1              UART1_BASE
>  #define CONFIG_BAUDRATE                      115200
>  
>  #define CONFIG_SYS_OMAP_ABE_SYSCK
>  
> -#define CONSOLEDEV           "ttyO0"
> +#include <configs/omap5_common.h>
>  
>  /* CPSW Ethernet */
>  #define CONFIG_CMD_NET
> diff --git a/include/configs/omap5_common.h b/include/configs/omap5_common.h
> index 7dd56cf..469a081 100644
> --- a/include/configs/omap5_common.h
> +++ b/include/configs/omap5_common.h
> @@ -14,106 +14,51 @@
>  #ifndef __CONFIG_OMAP5_COMMON_H
>  #define __CONFIG_OMAP5_COMMON_H
>  
> -/*
> - * High Level Configuration Options
> - */
> -#define CONFIG_OMAP  /* in a TI OMAP core */
> -#define CONFIG_OMAP54XX      /* which is a 54XX */
> -#define CONFIG_OMAP_GPIO
> -
> -/* Get CPU defs */
> -#include <asm/arch/cpu.h>
> -#include <asm/arch/omap.h>
> -
> -/* Display CPU and Board Info */
> +#define CONFIG_OMAP54XX
>  #define CONFIG_DISPLAY_CPUINFO
>  #define CONFIG_DISPLAY_BOARDINFO
> -
>  #define CONFIG_MISC_INIT_R
> +#define CONFIG_ARCH_CPU_INIT
>  
> -#define CONFIG_OF_LIBFDT
> -#define CONFIG_CMD_BOOTZ
> +#define CONFIG_SYS_CACHELINE_SIZE    64
>  
> -#define CONFIG_CMDLINE_TAG /* enable passing of ATAGs */
> -#define CONFIG_SETUP_MEMORY_TAGS
> -#define CONFIG_INITRD_TAG
> +/* Use General purpose timer 1 */
> +#define CONFIG_SYS_TIMERBASE         GPT2_BASE
> +
> +#define CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS
> +
> +/* Defines for SDRAM init */
> +#ifndef CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS
> +#define CONFIG_SYS_AUTOMATIC_SDRAM_DETECTION
> +#define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
> +#endif
> +
> +#ifndef CONFIG_SPL_BUILD
> +#define CONFIG_PALMAS_POWER
> +#endif
> +
> +#include <asm/arch/cpu.h>
> +#include <asm/arch/omap.h>
>  
> -/*
> - * Size of malloc() pool
> - * Total Size Environment - 128k
> - * Malloc - add 256k
> - */
>  #define CONFIG_ENV_SIZE                      (128 << 10)
> -#define CONFIG_SYS_MALLOC_LEN                (CONFIG_ENV_SIZE + (256 << 10))
> -/* Vector Base */
> -#define CONFIG_SYS_CA9_VECTOR_BASE   SRAM_ROM_VECT_BASE
>  
> -/*
> - * Hardware drivers
> - */
> +#include <configs/ti_armv7_common.h>
>  
>  /*
> - * serial port - NS16550 compatible
> + * Hardware drivers
>   */
> -#define V_NS16550_CLK                        48000000
> -
>  #define CONFIG_SYS_NS16550
>  #define CONFIG_SYS_NS16550_SERIAL
>  #define CONFIG_SYS_NS16550_REG_SIZE  (-4)
> -#define CONFIG_SYS_NS16550_CLK               V_NS16550_CLK
> -
> -/* CPU */
> -#define CONFIG_ARCH_CPU_INIT
> -
> -/* I2C  */
> -#define CONFIG_HARD_I2C
> -#define CONFIG_SYS_I2C_SPEED         100000
> -#define CONFIG_SYS_I2C_SLAVE         1
> -#define CONFIG_DRIVER_OMAP34XX_I2C
> -#define CONFIG_I2C_MULTI_BUS
> -
> -/* MMC */
> -#define CONFIG_GENERIC_MMC
> -#define CONFIG_MMC
> -#define CONFIG_OMAP_HSMMC
> -#define CONFIG_DOS_PARTITION
> -
> -#define CONFIG_SYS_CONSOLE_IS_IN_ENV
> -
> -/* Flash */
> -#define CONFIG_SYS_NO_FLASH
> -
> -/* Cache */
> -#define CONFIG_SYS_CACHELINE_SIZE    64
> -#define CONFIG_SYS_CACHELINE_SHIFT   6
> +#define CONFIG_SYS_NS16550_CLK               48000000
>  
> -/* commands to include */
> -#include <config_cmd_default.h>
> -
> -/* Enabled commands */
> -#define CONFIG_CMD_EXT2              /* EXT2 Support                 */
> -#define CONFIG_CMD_FAT               /* FAT support                  */
> -#define CONFIG_CMD_I2C               /* I2C serial bus support       */
> -#define CONFIG_CMD_MMC               /* MMC support                  */
> -
> -/* Disabled commands */
> +/* Per-SoC commands */
>  #undef CONFIG_CMD_NET
>  #undef CONFIG_CMD_NFS
> -#undef CONFIG_CMD_FPGA               /* FPGA configuration Support   */
> -#undef CONFIG_CMD_IMLS               /* List all found images        */
>  
>  /*
>   * Environment setup
>   */
> -
> -#define CONFIG_BOOTDELAY     3
> -#define CONFIG_ENV_VARS_UBOOT_CONFIG
> -#define CONFIG_CMD_FS_GENERIC
> -#define CONFIG_CMD_EXT2
> -#define CONFIG_CMD_EXT4
> -
> -#define CONFIG_ENV_OVERWRITE
> -
>  #ifndef PARTS_DEFAULT
>  #define PARTS_DEFAULT
>  #endif
> @@ -178,96 +123,11 @@
>               "fi; " \
>       "fi"
>  
> -#define CONFIG_AUTO_COMPLETE         1
> -
> -/*
> - * Miscellaneous configurable options
> - */
> -
> -#define CONFIG_SYS_LONGHELP  /* undef to save memory */
> -#define CONFIG_SYS_HUSH_PARSER       /* use "hush" command parser */
> -#define CONFIG_SYS_CBSIZE            256
> -/* Print Buffer Size */
> -#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> -                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> -#define CONFIG_SYS_MAXARGS           16
> -/* Boot Argument Buffer Size */
> -#define CONFIG_SYS_BARGSIZE          (CONFIG_SYS_CBSIZE)
> -
> -/*
> - * memtest setup
> - */
> -#define CONFIG_SYS_MEMTEST_START     0x80000000
> -#define CONFIG_SYS_MEMTEST_END               (CONFIG_SYS_MEMTEST_START + (32 
> << 20))
> -
> -/* Default load address */
> -#define CONFIG_SYS_LOAD_ADDR         0x80000000
> -
> -/* Use General purpose timer 1 */
> -#define CONFIG_SYS_TIMERBASE         GPT2_BASE
> -#define CONFIG_SYS_PTV                       2       /* Divisor: 2^(PTV+1) 
> => 8 */
> -#define CONFIG_SYS_HZ                        1000
> -
> -/*
> - * SDRAM Memory Map
> - * Even though we use two CS all the memory
> - * is mapped to one contiguous block
> - */
> -#define CONFIG_NR_DRAM_BANKS 1
> -
> -#define CONFIG_SYS_SDRAM_BASE                0x80000000
> -#define CONFIG_SYS_INIT_SP_ADDR         (NON_SECURE_SRAM_END - \
> -                                      GENERATED_GBL_DATA_SIZE)
> -
> -#define CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS
> -
> -/* Defines for SDRAM init */
> -#ifndef CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS
> -#define CONFIG_SYS_AUTOMATIC_SDRAM_DETECTION
> -#define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
> -#endif
> -
> -#ifndef CONFIG_SPL_BUILD
> -#define CONFIG_PALMAS_POWER
> -#endif
>  
>  /* Defines for SPL */
> -#define CONFIG_SPL
> -#define CONFIG_SPL_FRAMEWORK
>  #define CONFIG_SPL_TEXT_BASE         0x40300350
>  #define CONFIG_SPL_MAX_SIZE          0x19000 /* 100K */
> -#define CONFIG_SPL_STACK             CONFIG_SYS_INIT_SP_ADDR
>  #define CONFIG_SPL_DISPLAY_PRINT
> -
> -#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR      0x300 /* address 
> 0x60000 */
> -#define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS   0x200 /* 256 KB */
> -#define CONFIG_SYS_MMC_SD_FAT_BOOT_PARTITION 1
> -#define CONFIG_SPL_FAT_LOAD_PAYLOAD_NAME     "u-boot.img"
> -
> -#define CONFIG_SPL_LIBCOMMON_SUPPORT
> -#define CONFIG_SPL_LIBDISK_SUPPORT
> -#define CONFIG_SPL_I2C_SUPPORT
> -#define CONFIG_SPL_MMC_SUPPORT
> -#define CONFIG_SPL_FAT_SUPPORT
> -#define CONFIG_SPL_LIBGENERIC_SUPPORT
> -#define CONFIG_SPL_SERIAL_SUPPORT
>  #define CONFIG_SPL_LDSCRIPT "$(CPUDIR)/omap-common/u-boot-spl.lds"
>  
> -/*
> - * 64 bytes before this address should be set aside for u-boot.img's
> - * header. That is 80E7FFC0--0x80E80000 should not be used for any
> - * other needs.
> - */
> -#define CONFIG_SYS_TEXT_BASE         0x80E80000
> -
> -/*
> - * BSS and malloc area 64MB into memory to allow enough
> - * space for the kernel at the beginning of memory
> - */
> -#define CONFIG_SPL_BSS_START_ADDR    0x84000000
> -#define CONFIG_SPL_BSS_MAX_SIZE              0x100000        /* 1 MB */
> -#define CONFIG_SYS_SPL_MALLOC_START  0x84100000
> -#define CONFIG_SYS_SPL_MALLOC_SIZE   0x100000        /* 1 MB */
> -#define CONFIG_SPL_GPIO_SUPPORT
> -
>  #endif /* __CONFIG_OMAP5_COMMON_H */
> diff --git a/include/configs/omap5_uevm.h b/include/configs/omap5_uevm.h
> index 47568e9..d10c2b5 100644
> --- a/include/configs/omap5_uevm.h
> +++ b/include/configs/omap5_uevm.h
> @@ -17,8 +17,6 @@
>       "uuid_disk=${uuid_gpt_disk};" \
>       "name=rootfs,start=2MiB,size=-,uuid=${uuid_gpt_rootfs}"
>  
> -#include <configs/omap5_common.h>
> -
>  #define CONFIG_CONS_INDEX            3
>  #define CONFIG_SYS_NS16550_COM3              UART3_BASE
>  #define CONFIG_BAUDRATE                      115200
> @@ -42,9 +40,9 @@
>  #define CONFIG_SYS_I2C_TCA642X_BUS_NUM 4
>  #define CONFIG_SYS_I2C_TCA642X_ADDR 0x22
>  
> -#define CONFIG_SYS_PROMPT            "OMAP5432 uEVM # "
> -
>  #define CONSOLEDEV           "ttyO2"
> -
>  #define CONFIG_OMAP_PLATFORM_RESET_TIME_MAX_USEC     16296
> +
> +#include <configs/omap5_common.h>
> +
>  #endif /* __CONFIG_OMAP5_EVM_H */
Would it be better to split this into two patches? one for OMAP5 and one for 
dra7xx?



Dan

-- 
------------------
Dan Murphy

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to