Sanjeev Premi wrote:
> The board-types defined in struct omap3_sysinfo seem to be
> unused. The function display_board_info() is passed
> board type as an argument; which is ignored.
> 
> This patch removes all uses of board-type, related definitions
> and functions.
> 
> Signed-off-by: Sanjeev Premi <pr...@ti.com>

Signed-off-by: Dirk Behme <dirk.be...@googlemail.com>
Tested-by: Dirk Behme <dirk.be...@googlemail.com>

> ---
>  board/omap3/beagle/beagle.h            |    2 --
>  board/omap3/evm/evm.h                  |    2 --
>  board/omap3/overo/overo.h              |    2 --
>  board/omap3/pandora/pandora.h          |    2 --
>  board/omap3/zoom1/zoom1.h              |    2 --
>  cpu/arm_cortexa8/omap3/board.c         |    5 +----
>  cpu/arm_cortexa8/omap3/sys_info.c      |   19 +------------------
>  include/asm-arm/arch-omap3/omap3.h     |    8 --------
>  include/asm-arm/arch-omap3/sys_proto.h |    5 +----
>  9 files changed, 3 insertions(+), 44 deletions(-)
> 
> diff --git a/board/omap3/beagle/beagle.h b/board/omap3/beagle/beagle.h
> index d66f159..5d45f01 100644
> --- a/board/omap3/beagle/beagle.h
> +++ b/board/omap3/beagle/beagle.h
> @@ -24,8 +24,6 @@
>  #define _BEAGLE_H_
>  
>  const omap3_sysinfo sysinfo = {
> -     SDP_3430_V1,
> -     SDP_3430_V2,
>       DDR_STACKED,
>       "OMAP3 Beagle board",
>  #if defined(CONFIG_ENV_IS_IN_ONENAND)
> diff --git a/board/omap3/evm/evm.h b/board/omap3/evm/evm.h
> index 199824f..4a82a7f 100644
> --- a/board/omap3/evm/evm.h
> +++ b/board/omap3/evm/evm.h
> @@ -24,8 +24,6 @@
>  #define _EVM_H_
>  
>  const omap3_sysinfo sysinfo = {
> -     OMAP3EVM_V1,
> -     OMAP3EVM_V2,
>       DDR_DISCRETE,
>       "OMAP3 EVM board",
>  #if defined(CONFIG_ENV_IS_IN_ONENAND)
> diff --git a/board/omap3/overo/overo.h b/board/omap3/overo/overo.h
> index e8ccc1d..b595f6a 100644
> --- a/board/omap3/overo/overo.h
> +++ b/board/omap3/overo/overo.h
> @@ -24,8 +24,6 @@
>  #define _OVERO_H_
>  
>  const omap3_sysinfo sysinfo = {
> -     SDP_3430_V1,
> -     SDP_3430_V2,
>       DDR_STACKED,
>       "Gumstix Overo board",
>  #if defined(CONFIG_ENV_IS_IN_ONENAND)
> diff --git a/board/omap3/pandora/pandora.h b/board/omap3/pandora/pandora.h
> index dd09f12..8f0838c 100644
> --- a/board/omap3/pandora/pandora.h
> +++ b/board/omap3/pandora/pandora.h
> @@ -24,8 +24,6 @@
>  #define _PANDORA_H_
>  
>  const omap3_sysinfo sysinfo = {
> -     SDP_3430_V1,
> -     SDP_3430_V2,
>       DDR_STACKED,
>       "OMAP3 Pandora",
>       "NAND",
> diff --git a/board/omap3/zoom1/zoom1.h b/board/omap3/zoom1/zoom1.h
> index bc8fba8..4f49a41 100644
> --- a/board/omap3/zoom1/zoom1.h
> +++ b/board/omap3/zoom1/zoom1.h
> @@ -28,8 +28,6 @@
>  #define _BOARD_ZOOM1_H_
>  
>  const omap3_sysinfo sysinfo = {
> -     SDP_3430_V1,
> -     SDP_3430_V2,
>       DDR_STACKED,
>       "OMAP3 Zoom MDK Rev 1",
>       "NAND",
> diff --git a/cpu/arm_cortexa8/omap3/board.c b/cpu/arm_cortexa8/omap3/board.c
> index 15ea936..9318e22 100644
> --- a/cpu/arm_cortexa8/omap3/board.c
> +++ b/cpu/arm_cortexa8/omap3/board.c
> @@ -272,11 +272,8 @@ int dram_init(void)
>  {
>       DECLARE_GLOBAL_DATA_PTR;
>       unsigned int size0 = 0, size1 = 0;
> -     u32 btype;
>  
> -     btype = get_board_type();
> -
> -     display_board_info(btype);
> +     display_board_info();
>  
>       /*
>        * If a second bank of DDR is attached to CS1 this is
> diff --git a/cpu/arm_cortexa8/omap3/sys_info.c 
> b/cpu/arm_cortexa8/omap3/sys_info.c
> index b385b91..80f6e5e 100644
> --- a/cpu/arm_cortexa8/omap3/sys_info.c
> +++ b/cpu/arm_cortexa8/omap3/sys_info.c
> @@ -130,23 +130,6 @@ u32 get_sdr_cs_offset(u32 cs)
>       return offset;
>  }
>  
> -/***********************************************************************
> - * get_board_type() - get board type based on current production stats.
> - *  - NOTE-1-: 2 I2C EEPROMs will someday be populated with proper info.
> - *    when they are available we can get info from there.  This should
> - *    be correct of all known boards up until today.
> - *  - NOTE-2- EEPROMs are populated but they are updated very slowly.  To
> - *    avoid waiting on them we will use ES version of the chip to get info.
> - *    A later version of the FPGA migth solve their speed issue.
> - ************************************************************************/
> -u32 get_board_type(void)
> -{
> -     if (get_cpu_rev() == CPU_3430_ES2)
> -             return sysinfo.board_type_v2;
> -     else
> -             return sysinfo.board_type_v1;
> -}
> -
>  /***************************************************************************
>   *  get_gpmc0_base() - Return current address hardware will be
>   *     fetching from. The below effectively gives what is correct, its a bit
> @@ -188,7 +171,7 @@ u32 get_board_rev(void)
>  /*********************************************************************
>   *  display_board_info() - print banner with board info.
>   *********************************************************************/
> -void display_board_info(u32 btype)
> +void display_board_info(void)
>  {
>       char *cpu_s, *mem_s, *sec_s;
>  
> diff --git a/include/asm-arm/arch-omap3/omap3.h 
> b/include/asm-arm/arch-omap3/omap3.h
> index 8c9656f..d0d2737 100644
> --- a/include/asm-arm/arch-omap3/omap3.h
> +++ b/include/asm-arm/arch-omap3/omap3.h
> @@ -177,14 +177,6 @@ typedef struct gpio {
>  #define WIDTH_8BIT           0x0000
>  #define WIDTH_16BIT          0x1000  /* bit pos for 16 bit in gpmc */
>  
> -/* SDP definitions according to FPGA Rev. Is this OK?? */
> -#define SDP_3430_V1          0x1
> -#define SDP_3430_V2          0x2
> -
> -/* EVM definitions */
> -#define OMAP3EVM_V1          0x1
> -#define OMAP3EVM_V2          0x2
> -
>  /* I2C power management companion definitions */
>  #define PWRMGT_ADDR_ID1              0x48
>  #define PWRMGT_ADDR_ID2              0x49
> diff --git a/include/asm-arm/arch-omap3/sys_proto.h 
> b/include/asm-arm/arch-omap3/sys_proto.h
> index 8b554bb..1bcec04 100644
> --- a/include/asm-arm/arch-omap3/sys_proto.h
> +++ b/include/asm-arm/arch-omap3/sys_proto.h
> @@ -22,8 +22,6 @@
>  #define _SYS_PROTO_H_
>  
>  typedef struct {
> -     u32 board_type_v1;
> -     u32 board_type_v2;
>       u32 mtype;
>       char *board_string;
>       char *nand_string;
> @@ -46,8 +44,7 @@ u32 get_sysboot_value(void);
>  u32 is_gpmc_muxed(void);
>  u32 get_gpmc0_type(void);
>  u32 get_gpmc0_width(void);
> -u32 get_board_type(void);
> -void display_board_info(u32);
> +void display_board_info(void);
>  u32 get_sdr_cs_size(u32);
>  u32 get_sdr_cs_offset(u32);
>  u32 is_running_in_sdram(void);

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

Reply via email to