Hi Josh,

sorry for beeing late. I'd like to apply this one but it does not
cleanly. Could you please rebase on current master?

Andreas

On 19.11.14 12:03, Josh Wu wrote:
> Create a new configuration file: at91-sama5_common.h. Which include the
> configurations that reused by all SAMA5 chips.
> 
> In the command header, we add the CMD_SETEXPR.
> 
> NOTE: NOR flash definition should be put before including the common header.
> 
> Signed-off-by: Josh Wu <josh...@atmel.com>
> ---
>  include/configs/at91-sama5_common.h | 87 
> +++++++++++++++++++++++++++++++++++++
>  include/configs/sama5d3_xplained.h  | 77 ++------------------------------
>  include/configs/sama5d3xek.h        | 83 +++--------------------------------
>  include/configs/sama5d4_xplained.h  | 74 ++-----------------------------
>  include/configs/sama5d4ek.h         | 75 ++------------------------------
>  5 files changed, 103 insertions(+), 293 deletions(-)
>  create mode 100644 include/configs/at91-sama5_common.h
> 
> diff --git a/include/configs/at91-sama5_common.h 
> b/include/configs/at91-sama5_common.h
> new file mode 100644
> index 0000000..4fab41d
> --- /dev/null
> +++ b/include/configs/at91-sama5_common.h
> @@ -0,0 +1,87 @@
> +/*
> + * Common part of configuration settings for the SAMA5 board.
> + *
> + * Copyright (C) 2014 Atmel Corporation
> + *                 Josh Wu <josh...@atmel.com>
> + *
> + * SPDX-License-Identifier:  GPL-2.0+
> + */
> +
> +#ifndef __CONFIG_AT91_SAMA5_H
> +#define __CONFIG_AT91_SAMA5_H
> +
> +#include <asm/hardware.h>
> +
> +#define CONFIG_SYS_TEXT_BASE         0x26f00000
> +
> +/* ARM asynchronous clock */
> +#define CONFIG_SYS_AT91_SLOW_CLOCK      32768
> +#define CONFIG_SYS_AT91_MAIN_CLOCK      12000000 /* from 12 MHz crystal */
> +
> +#define CONFIG_ARCH_CPU_INIT
> +
> +#ifndef CONFIG_SPL_BUILD
> +#define CONFIG_SKIP_LOWLEVEL_INIT
> +#endif
> +
> +#define CONFIG_BOARD_EARLY_INIT_F
> +#define CONFIG_DISPLAY_CPUINFO
> +
> +#define CONFIG_CMD_BOOTZ
> +#define CONFIG_OF_LIBFDT             /* Device Tree support */
> +
> +#define CONFIG_SYS_GENERIC_BOARD
> +
> +/* general purpose I/O */
> +#define CONFIG_AT91_GPIO
> +
> +#define CONFIG_BOOTDELAY             3
> +
> +/*
> + * BOOTP options
> + */
> +#define CONFIG_BOOTP_BOOTFILESIZE
> +#define CONFIG_BOOTP_BOOTPATH
> +#define CONFIG_BOOTP_GATEWAY
> +#define CONFIG_BOOTP_HOSTNAME
> +
> +/*
> + * Command line configuration.
> + */
> +#include <config_cmd_default.h>
> +#undef CONFIG_CMD_FPGA
> +#undef CONFIG_CMD_IMI
> +#undef CONFIG_CMD_LOADS
> +#define CONFIG_CMD_PING
> +#define CONFIG_CMD_DHCP
> +#define CONFIG_CMD_SETEXPR
> +
> +#ifdef CONFIG_SYS_USE_MMC
> +#define CONFIG_BOOTARGS                                                      
> \
> +     "console=ttyS0,115200 earlyprintk "                             \
> +     "root=/dev/mmcblk0p2 rw rootwait"
> +#else
> +#define CONFIG_BOOTARGS                                                      
> \
> +     "console=ttyS0,115200 earlyprintk "                             \
> +     "mtdparts=atmel_nand:256k(bootstrap)ro,512k(uboot)ro,"          \
> +     "256K(env),256k(evn_redundent),256k(spare),"                    \
> +     "512k(dtb),6M(kernel)ro,-(rootfs) "                             \
> +     "rootfstype=ubifs ubi.mtd=7 root=ubi0:rootfs"
> +#endif
> +
> +#define CONFIG_BAUDRATE                      115200
> +
> +#define CONFIG_SYS_PROMPT            "U-Boot> "
> +#define CONFIG_SYS_CBSIZE            256
> +#define CONFIG_SYS_MAXARGS           16
> +#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> +                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> +#define CONFIG_SYS_LONGHELP
> +#define CONFIG_CMDLINE_EDITING
> +#define CONFIG_AUTO_COMPLETE
> +#define CONFIG_SYS_HUSH_PARSER
> +
> +/* Size of malloc() pool */
> +#define CONFIG_SYS_MALLOC_LEN                (4 * 1024 * 1024)
> +
> +#endif
> \ No newline at end of file
> diff --git a/include/configs/sama5d3_xplained.h 
> b/include/configs/sama5d3_xplained.h
> index d24d600..75c3be6 100644
> --- a/include/configs/sama5d3_xplained.h
> +++ b/include/configs/sama5d3_xplained.h
> @@ -10,30 +10,10 @@
>  #ifndef __CONFIG_H
>  #define __CONFIG_H
>  
> -#include <asm/hardware.h>
> -
> -#define CONFIG_SYS_TEXT_BASE         0x26f00000
> -
> -/* ARM asynchronous clock */
> -#define CONFIG_SYS_AT91_SLOW_CLOCK      32768
> -#define CONFIG_SYS_AT91_MAIN_CLOCK      12000000 /* from 12 MHz crystal */
> -
> -#define CONFIG_ARCH_CPU_INIT
> -
> -#ifndef CONFIG_SPL_BUILD
> -#define CONFIG_SKIP_LOWLEVEL_INIT
> -#endif
> -
> -#define CONFIG_BOARD_EARLY_INIT_F
> -#define CONFIG_DISPLAY_CPUINFO
> -
> -#define CONFIG_CMD_BOOTZ
> -#define CONFIG_OF_LIBFDT             /* Device Tree support */
> -
> -#define CONFIG_SYS_GENERIC_BOARD
> +/* No NOR flash, this definition should put before common header */
> +#define CONFIG_SYS_NO_FLASH
>  
> -/* general purpose I/O */
> -#define CONFIG_AT91_GPIO
> +#include "at91-sama5_common.h"
>  
>  /* serial console */
>  #define CONFIG_ATMEL_USART
> @@ -51,29 +31,6 @@
>   */
>  #define ATMEL_PMC_UHP                        AT91SAM926x_PMC_UHP
>  
> -#define CONFIG_BOOTDELAY             3
> -
> -/*
> - * BOOTP options
> - */
> -#define CONFIG_BOOTP_BOOTFILESIZE
> -#define CONFIG_BOOTP_BOOTPATH
> -#define CONFIG_BOOTP_GATEWAY
> -#define CONFIG_BOOTP_HOSTNAME
> -
> -/* No NOR flash */
> -#define CONFIG_SYS_NO_FLASH
> -
> -/*
> - * Command line configuration.
> - */
> -#include <config_cmd_default.h>
> -#undef CONFIG_CMD_FPGA
> -#undef CONFIG_CMD_IMI
> -#undef CONFIG_CMD_LOADS
> -#define CONFIG_CMD_PING
> -#define CONFIG_CMD_DHCP
> -
>  /* SDRAM */
>  #define CONFIG_NR_DRAM_BANKS         1
>  #define CONFIG_SYS_SDRAM_BASE           ATMEL_BASE_DDRCS
> @@ -180,34 +137,6 @@
>  #define CONFIG_ENV_IS_NOWHERE
>  #endif
>  
> -#ifdef CONFIG_SYS_USE_MMC
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "root=/dev/mmcblk0p2 rw rootwait"
> -#else
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "mtdparts=atmel_nand:256k(bootstrap)ro,512k(uboot)ro,"          \
> -     "256K(env),256k(evn_redundent),256k(spare),"                    \
> -     "512k(dtb),6M(kernel)ro,-(rootfs) "                             \
> -     "rootfstype=ubifs ubi.mtd=7 root=ubi0:rootfs"
> -#endif
> -
> -#define CONFIG_BAUDRATE                      115200
> -
> -#define CONFIG_SYS_PROMPT            "U-Boot> "
> -#define CONFIG_SYS_CBSIZE            256
> -#define CONFIG_SYS_MAXARGS           16
> -#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> -                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> -#define CONFIG_SYS_LONGHELP
> -#define CONFIG_CMDLINE_EDITING
> -#define CONFIG_AUTO_COMPLETE
> -#define CONFIG_SYS_HUSH_PARSER
> -
> -/* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN                (4 * 1024 * 1024)
> -
>  /* SPL */
>  #define CONFIG_SPL_FRAMEWORK
>  #define CONFIG_SPL_TEXT_BASE         0x300000
> diff --git a/include/configs/sama5d3xek.h b/include/configs/sama5d3xek.h
> index dfbf3cb..ee0fddb 100644
> --- a/include/configs/sama5d3xek.h
> +++ b/include/configs/sama5d3xek.h
> @@ -13,30 +13,11 @@
>  #ifndef __CONFIG_H
>  #define __CONFIG_H
>  
> -#include <asm/hardware.h>
> -
> -#define CONFIG_SYS_TEXT_BASE         0x26f00000
> -
> -/* ARM asynchronous clock */
> -#define CONFIG_SYS_AT91_SLOW_CLOCK      32768
> -#define CONFIG_SYS_AT91_MAIN_CLOCK      12000000 /* from 12 MHz crystal */
> -
> -#define CONFIG_ARCH_CPU_INIT
> -
> -#ifndef CONFIG_SPL_BUILD
> -#define CONFIG_SKIP_LOWLEVEL_INIT
> -#endif
> -
> -#define CONFIG_BOARD_EARLY_INIT_F
> -#define CONFIG_DISPLAY_CPUINFO
> -
> -#define CONFIG_CMD_BOOTZ
> -#define CONFIG_OF_LIBFDT             /* Device Tree support */
> -
> -#define CONFIG_SYS_GENERIC_BOARD
> -
> -/* general purpose I/O */
> -#define CONFIG_AT91_GPIO
> +/*
> + * If has No NOR flash, please put the definition: CONFIG_SYS_NO_FLASH
> + * before the common header.
> + */
> +#include "at91-sama5_common.h"
>  
>  /* serial console */
>  #define CONFIG_ATMEL_USART
> @@ -69,40 +50,16 @@
>  /* board specific (not enough SRAM) */
>  #define CONFIG_SAMA5D3_LCD_BASE              0x23E00000
>  
> -#define CONFIG_BOOTDELAY             3
> -
> -/*
> - * BOOTP options
> - */
> -#define CONFIG_BOOTP_BOOTFILESIZE
> -#define CONFIG_BOOTP_BOOTPATH
> -#define CONFIG_BOOTP_GATEWAY
> -#define CONFIG_BOOTP_HOSTNAME
> -
>  /* NOR flash */
> -#define CONFIG_CMD_FLASH
> -
> -#ifdef CONFIG_CMD_FLASH
> +#ifndef CONFIG_SYS_NO_FLASH
>  #define CONFIG_FLASH_CFI_DRIVER
>  #define CONFIG_SYS_FLASH_CFI
>  #define CONFIG_SYS_FLASH_PROTECTION
>  #define CONFIG_SYS_FLASH_BASE                0x10000000
>  #define CONFIG_SYS_MAX_FLASH_SECT    131
>  #define CONFIG_SYS_MAX_FLASH_BANKS   1
> -#else
> -#define CONFIG_SYS_NO_FLASH
>  #endif
>  
> -/*
> - * Command line configuration.
> - */
> -#include <config_cmd_default.h>
> -#undef CONFIG_CMD_FPGA
> -#undef CONFIG_CMD_IMI
> -#undef CONFIG_CMD_LOADS
> -#define CONFIG_CMD_PING
> -#define CONFIG_CMD_DHCP
> -
>  /* SDRAM */
>  #define CONFIG_NR_DRAM_BANKS         1
>  #define CONFIG_SYS_SDRAM_BASE           ATMEL_BASE_DDRCS
> @@ -226,34 +183,6 @@
>  #define CONFIG_ENV_IS_NOWHERE
>  #endif
>  
> -#ifdef CONFIG_SYS_USE_MMC
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "root=/dev/mmcblk0p2 rw rootwait"
> -#else
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "mtdparts=atmel_nand:256k(bootstrap)ro,512k(uboot)ro,"          \
> -     "256K(env),256k(evn_redundent),256k(spare),"                    \
> -     "512k(dtb),6M(kernel)ro,-(rootfs) "                             \
> -     "rootfstype=ubifs ubi.mtd=7 root=ubi0:rootfs"
> -#endif
> -
> -#define CONFIG_BAUDRATE                      115200
> -
> -#define CONFIG_SYS_PROMPT            "U-Boot> "
> -#define CONFIG_SYS_CBSIZE            256
> -#define CONFIG_SYS_MAXARGS           16
> -#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> -                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> -#define CONFIG_SYS_LONGHELP
> -#define CONFIG_CMDLINE_EDITING
> -#define CONFIG_AUTO_COMPLETE
> -#define CONFIG_SYS_HUSH_PARSER
> -
> -/* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN                (1024 * 1024)
> -
>  /* SPL */
>  #define CONFIG_SPL_FRAMEWORK
>  #define CONFIG_SPL_TEXT_BASE         0x300000
> diff --git a/include/configs/sama5d4_xplained.h 
> b/include/configs/sama5d4_xplained.h
> index 104edef..26af695 100644
> --- a/include/configs/sama5d4_xplained.h
> +++ b/include/configs/sama5d4_xplained.h
> @@ -10,57 +10,16 @@
>  #ifndef __CONFIG_H
>  #define __CONFIG_H
>  
> -#include <asm/hardware.h>
> -
> -#define CONFIG_SYS_TEXT_BASE         0x26f00000
> -
> -/* ARM asynchronous clock */
> -#define CONFIG_SYS_AT91_SLOW_CLOCK      32768
> -#define CONFIG_SYS_AT91_MAIN_CLOCK      12000000 /* from 12 MHz crystal */
> -
> -#define CONFIG_ARCH_CPU_INIT
> -
> -#define CONFIG_SKIP_LOWLEVEL_INIT
> -#define CONFIG_BOARD_EARLY_INIT_F
> -#define CONFIG_DISPLAY_CPUINFO
> -
> -#define CONFIG_CMD_BOOTZ
> -#define CONFIG_OF_LIBFDT             /* Device Tree support */
> -
> -#define CONFIG_SYS_GENERIC_BOARD
> +/* No NOR flash, this definition should put before common header */
> +#define CONFIG_SYS_NO_FLASH
>  
> -/* general purpose I/O */
> -#define CONFIG_AT91_GPIO
> +#include "at91-sama5_common.h"
>  
>  /* serial console */
>  #define CONFIG_ATMEL_USART
>  #define CONFIG_USART_BASE            ATMEL_BASE_USART3
>  #define CONFIG_USART_ID                      ATMEL_ID_USART3
>  
> -#define CONFIG_BOOTDELAY             3
> -
> -/*
> - * BOOTP options
> - */
> -#define CONFIG_BOOTP_BOOTFILESIZE
> -#define CONFIG_BOOTP_BOOTPATH
> -#define CONFIG_BOOTP_GATEWAY
> -#define CONFIG_BOOTP_HOSTNAME
> -
> -/* No NOR flash */
> -#define CONFIG_SYS_NO_FLASH
> -
> -/*
> - * Command line configuration.
> - */
> -#include <config_cmd_default.h>
> -#undef CONFIG_CMD_FPGA
> -#undef CONFIG_CMD_IMI
> -#undef CONFIG_CMD_LOADS
> -#define CONFIG_CMD_PING
> -#define CONFIG_CMD_DHCP
> -#define CONFIG_CMD_SETEXPR
> -
>  /* SDRAM */
>  #define CONFIG_NR_DRAM_BANKS         1
>  #define CONFIG_SYS_SDRAM_BASE           ATMEL_BASE_DDRCS
> @@ -185,32 +144,5 @@
>                               "bootz 0x22000000 - 0x21000000"
>  #endif
>  
> -#ifdef CONFIG_SYS_USE_MMC
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "root=/dev/mmcblk0p2 rw rootwait"
> -#else
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "mtdparts=atmel_nand:256k(bootstrap)ro,512k(uboot)ro,"          \
> -     "256K(env),256k(evn_redundent),256k(spare),"                    \
> -     "512k(dtb),6M(kernel)ro,-(rootfs) "                             \
> -     "rootfstype=ubifs ubi.mtd=7 root=ubi0:rootfs"
> -#endif
> -
> -#define CONFIG_BAUDRATE                      115200
> -
> -#define CONFIG_SYS_PROMPT            "U-Boot> "
> -#define CONFIG_SYS_CBSIZE            256
> -#define CONFIG_SYS_MAXARGS           16
> -#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> -                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> -#define CONFIG_SYS_LONGHELP
> -#define CONFIG_CMDLINE_EDITING
> -#define CONFIG_AUTO_COMPLETE
> -#define CONFIG_SYS_HUSH_PARSER
> -
> -/* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN                (4 * 1024 * 1024)
>  
>  #endif
> diff --git a/include/configs/sama5d4ek.h b/include/configs/sama5d4ek.h
> index cbdb3a2..15a92af 100644
> --- a/include/configs/sama5d4ek.h
> +++ b/include/configs/sama5d4ek.h
> @@ -10,57 +10,16 @@
>  #ifndef __CONFIG_H
>  #define __CONFIG_H
>  
> -#include <asm/hardware.h>
> -
> -#define CONFIG_SYS_TEXT_BASE         0x26f00000
> -
> -/* ARM asynchronous clock */
> -#define CONFIG_SYS_AT91_SLOW_CLOCK      32768
> -#define CONFIG_SYS_AT91_MAIN_CLOCK      12000000 /* from 12 MHz crystal */
> -
> -#define CONFIG_ARCH_CPU_INIT
> -
> -#define CONFIG_SKIP_LOWLEVEL_INIT
> -#define CONFIG_BOARD_EARLY_INIT_F
> -#define CONFIG_DISPLAY_CPUINFO
> -
> -#define CONFIG_CMD_BOOTZ
> -#define CONFIG_OF_LIBFDT             /* Device Tree support */
> -
> -#define CONFIG_SYS_GENERIC_BOARD
> +/* No NOR flash, this definition should put before common header */
> +#define CONFIG_SYS_NO_FLASH
>  
> -/* general purpose I/O */
> -#define CONFIG_AT91_GPIO
> +#include "at91-sama5_common.h"
>  
>  /* serial console */
>  #define CONFIG_ATMEL_USART
>  #define CONFIG_USART_BASE            ATMEL_BASE_USART3
>  #define      CONFIG_USART_ID                 ATMEL_ID_USART3
>  
> -#define CONFIG_BOOTDELAY             3
> -
> -/*
> - * BOOTP options
> - */
> -#define CONFIG_BOOTP_BOOTFILESIZE
> -#define CONFIG_BOOTP_BOOTPATH
> -#define CONFIG_BOOTP_GATEWAY
> -#define CONFIG_BOOTP_HOSTNAME
> -
> -/* No NOR flash */
> -#define CONFIG_SYS_NO_FLASH
> -
> -/*
> - * Command line configuration.
> - */
> -#include <config_cmd_default.h>
> -#undef CONFIG_CMD_FPGA
> -#undef CONFIG_CMD_IMI
> -#undef CONFIG_CMD_LOADS
> -#define CONFIG_CMD_PING
> -#define CONFIG_CMD_DHCP
> -#define CONFIG_CMD_SETEXPR
> -
>  /* SDRAM */
>  #define CONFIG_NR_DRAM_BANKS         1
>  #define CONFIG_SYS_SDRAM_BASE           ATMEL_BASE_DDRCS
> @@ -181,34 +140,8 @@
>  #define CONFIG_BOOTCOMMAND   "fatload mmc 0:1 0x21000000 sama5d4ek.dtb; " \
>                               "fatload mmc 0:1 0x22000000 zImage; " \
>                               "bootz 0x22000000 - 0x21000000"
> -#endif
> -
> -#ifdef CONFIG_SYS_USE_MMC
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "root=/dev/mmcblk0p2 rw rootwait"
>  #else
> -#define CONFIG_BOOTARGS                                                      
> \
> -     "console=ttyS0,115200 earlyprintk "                             \
> -     "mtdparts=atmel_nand:256k(bootstrap)ro,512k(uboot)ro,"          \
> -     "256K(env),256k(evn_redundent),256k(spare),"                    \
> -     "512k(dtb),6M(kernel)ro,-(rootfs) "                             \
> -     "rootfstype=ubifs ubi.mtd=7 root=ubi0:rootfs"
> +#define CONFIG_ENV_IS_NOWHERE
>  #endif
>  
> -#define CONFIG_BAUDRATE                      115200
> -
> -#define CONFIG_SYS_PROMPT            "U-Boot> "
> -#define CONFIG_SYS_CBSIZE            256
> -#define CONFIG_SYS_MAXARGS           16
> -#define CONFIG_SYS_PBSIZE            (CONFIG_SYS_CBSIZE + \
> -                                     sizeof(CONFIG_SYS_PROMPT) + 16)
> -#define CONFIG_SYS_LONGHELP
> -#define CONFIG_CMDLINE_EDITING
> -#define CONFIG_AUTO_COMPLETE
> -#define CONFIG_SYS_HUSH_PARSER
> -
> -/* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN                (4 * 1024 * 1024)
> -
>  #endif
> 
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to