Re: [U-Boot] [PATCH 1/3] arm: baltos: remove TI board leftover

2016-12-01 Thread Tom Rini
On Thu, Dec 01, 2016 at 12:52:16PM +0100, yegorsli...@googlemail.com wrote:

> From: Yegor Yefremov 
> 
> Remove unneeded pinmux configurations and TI EEPROM struct.
> 
> Signed-off-by: Yegor Yefremov 

Reviewed-by: Tom Rini 

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/3] arm: baltos: remove TI board leftover

2016-12-01 Thread yegorslists
From: Yegor Yefremov 

Remove unneeded pinmux configurations and TI EEPROM struct.

Signed-off-by: Yegor Yefremov 
---
 board/vscom/baltos/board.c |  1 -
 board/vscom/baltos/board.h | 55 
 board/vscom/baltos/mux.c   | 69 --
 3 files changed, 125 deletions(-)

diff --git a/board/vscom/baltos/board.c b/board/vscom/baltos/board.c
index 27056e1..dbf79a5 100644
--- a/board/vscom/baltos/board.c
+++ b/board/vscom/baltos/board.c
@@ -415,7 +415,6 @@ int board_eth_init(bd_t *bis)
int rv, n = 0;
uint8_t mac_addr[6];
uint32_t mac_hi, mac_lo;
-   __maybe_unused struct am335x_baseboard_id header;
 
/*
 * Note here that we're using CPSW1 since that has a 1Gbit PHY while
diff --git a/board/vscom/baltos/board.h b/board/vscom/baltos/board.h
index bcdb648..40ddd90 100644
--- a/board/vscom/baltos/board.h
+++ b/board/vscom/baltos/board.h
@@ -11,24 +11,6 @@
 #ifndef _BOARD_H_
 #define _BOARD_H_
 
-/*
- * TI AM335x parts define a system EEPROM that defines certain sub-fields.
- * We use these fields to in turn see what board we are on, and what
- * that might require us to set or not set.
- */
-#define HDR_NO_OF_MAC_ADDR 3
-#define HDR_ETH_ALEN   6
-#define HDR_NAME_LEN   8
-
-struct am335x_baseboard_id {
-   unsigned int  magic;
-   char name[HDR_NAME_LEN];
-   char version[4];
-   char serial[12];
-   char config[32];
-   char mac_addr[HDR_NO_OF_MAC_ADDR][HDR_ETH_ALEN];
-};
-
 typedef struct _BSP_VS_HWPARAM// v1.0
 {
uint32_t Magic;
@@ -41,37 +23,6 @@ typedef struct _BSP_VS_HWPARAM// v1.0
uint8_t MAC3[6];// WL1271 WLAN
 } __attribute__ ((packed)) BSP_VS_HWPARAM;
 
-static inline int board_is_bone(struct am335x_baseboard_id *header)
-{
-   return !strncmp(header->name, "A335BONE", HDR_NAME_LEN);
-}
-
-static inline int board_is_bone_lt(struct am335x_baseboard_id *header)
-{
-   return !strncmp(header->name, "A335BNLT", HDR_NAME_LEN);
-}
-
-static inline int board_is_evm_sk(struct am335x_baseboard_id *header)
-{
-   return !strncmp("A335X_SK", header->name, HDR_NAME_LEN);
-}
-
-static inline int board_is_idk(struct am335x_baseboard_id *header)
-{
-   return !strncmp(header->config, "SKU#02", 6);
-}
-
-static inline int board_is_gp_evm(struct am335x_baseboard_id *header)
-{
-   return !strncmp("A33515BB", header->name, HDR_NAME_LEN);
-}
-
-static inline int board_is_evm_15_or_later(struct am335x_baseboard_id *header)
-{
-   return (board_is_gp_evm(header) &&
-   strncmp("1.5", header->version, 3) <= 0);
-}
-
 /*
  * We have three pin mux functions that must exist.  We must be able to enable
  * uart0, for initial output and i2c0 to read the main EEPROM.  We then have a
@@ -79,12 +30,6 @@ static inline int board_is_evm_15_or_later(struct 
am335x_baseboard_id *header)
  * is required on the board.
  */
 void enable_uart0_pin_mux(void);
-void enable_uart1_pin_mux(void);
-void enable_uart2_pin_mux(void);
-void enable_uart3_pin_mux(void);
-void enable_uart4_pin_mux(void);
-void enable_uart5_pin_mux(void);
-void enable_i2c0_pin_mux(void);
 void enable_i2c1_pin_mux(void);
 void enable_board_pin_mux(void);
 #endif
diff --git a/board/vscom/baltos/mux.c b/board/vscom/baltos/mux.c
index 8783b25..94410ae 100644
--- a/board/vscom/baltos/mux.c
+++ b/board/vscom/baltos/mux.c
@@ -27,36 +27,6 @@ static struct module_pin_mux uart0_pin_mux[] = {
{-1},
 };
 
-static struct module_pin_mux uart1_pin_mux[] = {
-   {OFFSET(uart1_rxd), (MODE(0) | PULLUP_EN | RXACTIVE)},  /* UART1_RXD */
-   {OFFSET(uart1_txd), (MODE(0) | PULLUDEN)},  /* UART1_TXD */
-   {-1},
-};
-
-static struct module_pin_mux uart2_pin_mux[] = {
-   {OFFSET(spi0_sclk), (MODE(1) | PULLUP_EN | RXACTIVE)},  /* UART2_RXD */
-   {OFFSET(spi0_d0), (MODE(1) | PULLUDEN)},/* UART2_TXD */
-   {-1},
-};
-
-static struct module_pin_mux uart3_pin_mux[] = {
-   {OFFSET(spi0_cs1), (MODE(1) | PULLUP_EN | RXACTIVE)},   /* UART3_RXD */
-   {OFFSET(ecap0_in_pwm0_out), (MODE(1) | PULLUDEN)},  /* UART3_TXD */
-   {-1},
-};
-
-static struct module_pin_mux uart4_pin_mux[] = {
-   {OFFSET(gpmc_wait0), (MODE(6) | PULLUP_EN | RXACTIVE)}, /* UART4_RXD */
-   {OFFSET(gpmc_wpn), (MODE(6) | PULLUDEN)},   /* UART4_TXD */
-   {-1},
-};
-
-static struct module_pin_mux uart5_pin_mux[] = {
-   {OFFSET(lcd_data9), (MODE(4) | PULLUP_EN | RXACTIVE)},  /* UART5_RXD */
-   {OFFSET(lcd_data8), (MODE(4) | PULLUDEN)},  /* UART5_TXD */
-   {-1},
-};
-
 static struct module_pin_mux mmc0_pin_mux[] = {
{OFFSET(mmc0_dat3), (MODE(0) | RXACTIVE | PULLUP_EN)},  /* MMC0_DAT3 */
{OFFSET(mmc0_dat2), (MODE(0) | RXACTIVE | PULLUP_EN)},  /* MMC0_DAT2 */
@@ -68,14 +38,6 @@ static struct module_pin_mux mmc0_pin_mux[] = {
{-1},
 };
 
-static struct module_pin_mux i2c0