arm: ls1021atwr: Rework local commands to not be included in SPL
authorTom Rini <trini@konsulko.com>
Wed, 3 Jan 2018 14:01:33 +0000 (09:01 -0500)
committerYork Sun <york.sun@nxp.com>
Mon, 15 Jan 2018 20:44:24 +0000 (12:44 -0800)
Move some of the code for the "lane_bank" and "cpld" code local
commands so that they are not built for SPL as they can only be
used in full U-Boot.  This means we can mark a few functions as
static as well now.

Cc: Alison Wang <alison.wang@freescale.com>
Cc: Sumit Garg <sumit.garg@nxp.com>
Cc: York Sun <york.sun@nxp.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Alison Wang <alison.wang@nxp.com>
Tested-by: Alison Wang <alison.wang@nxp.com>
Reviewed-by: York Sun <york.sun@nxp.com>
board/freescale/ls1021atwr/ls1021atwr.c

index 2da0677..622a500 100644 (file)
@@ -92,9 +92,7 @@ struct cpld_data {
 };
 
 #if !defined(CONFIG_QSPI_BOOT) && !defined(CONFIG_SD_BOOT_QSPI)
-static void convert_serdes_mux(int type, int need_reset);
-
-void cpld_show(void)
+static void cpld_show(void)
 {
        struct cpld_data *cpld_data = (void *)(CONFIG_SYS_CPLD_BASE);
 
@@ -292,6 +290,47 @@ int board_eth_init(bd_t *bis)
 }
 
 #if !defined(CONFIG_QSPI_BOOT) && !defined(CONFIG_SD_BOOT_QSPI)
+static void convert_serdes_mux(int type, int need_reset)
+{
+       char current_serdes;
+       struct cpld_data *cpld_data = (void *)(CONFIG_SYS_CPLD_BASE);
+
+       current_serdes = cpld_data->serdes_mux;
+
+       switch (type) {
+       case LANEB_SATA:
+               current_serdes &= ~MASK_LANE_B;
+               break;
+       case LANEB_SGMII1:
+               current_serdes |= (MASK_LANE_B | MASK_SGMII | MASK_LANE_C);
+               break;
+       case LANEC_SGMII1:
+               current_serdes &= ~(MASK_LANE_B | MASK_SGMII | MASK_LANE_C);
+               break;
+       case LANED_SGMII2:
+               current_serdes |= MASK_LANE_D;
+               break;
+       case LANEC_PCIEX1:
+               current_serdes |= MASK_LANE_C;
+               break;
+       case (LANED_PCIEX2 | LANEC_PCIEX1):
+               current_serdes |= MASK_LANE_C;
+               current_serdes &= ~MASK_LANE_D;
+               break;
+       default:
+               printf("CPLD serdes MUX: unsupported MUX type 0x%x\n", type);
+               return;
+       }
+
+       cpld_data->soft_mux_on |= CPLD_SET_MUX_SERDES;
+       cpld_data->serdes_mux = current_serdes;
+
+       if (need_reset == 1) {
+               printf("Reset board to enable configuration\n");
+               cpld_data->system_rst = CONFIG_RESET;
+       }
+}
+
 int config_serdes_mux(void)
 {
        struct ccsr_gur __iomem *gur = (void *)(CONFIG_SYS_FSL_GUTS_ADDR);
@@ -584,7 +623,8 @@ u16 flash_read16(void *addr)
        return (((val) >> 8) & 0x00ff) | (((val) << 8) & 0xff00);
 }
 
-#if !defined(CONFIG_QSPI_BOOT) && !defined(CONFIG_SD_BOOT_QSPI)
+#if !defined(CONFIG_QSPI_BOOT) && !defined(CONFIG_SD_BOOT_QSPI) \
+       && !defined(CONFIG_SPL_BUILD)
 static void convert_flash_bank(char bank)
 {
        struct cpld_data *cpld_data = (void *)(CONFIG_SYS_CPLD_BASE);
@@ -645,48 +685,7 @@ U_BOOT_CMD(
 
 );
 
-static void convert_serdes_mux(int type, int need_reset)
-{
-       char current_serdes;
-       struct cpld_data *cpld_data = (void *)(CONFIG_SYS_CPLD_BASE);
-
-       current_serdes = cpld_data->serdes_mux;
-
-       switch (type) {
-       case LANEB_SATA:
-               current_serdes &= ~MASK_LANE_B;
-               break;
-       case LANEB_SGMII1:
-               current_serdes |= (MASK_LANE_B | MASK_SGMII | MASK_LANE_C);
-               break;
-       case LANEC_SGMII1:
-               current_serdes &= ~(MASK_LANE_B | MASK_SGMII | MASK_LANE_C);
-               break;
-       case LANED_SGMII2:
-               current_serdes |= MASK_LANE_D;
-               break;
-       case LANEC_PCIEX1:
-               current_serdes |= MASK_LANE_C;
-               break;
-       case (LANED_PCIEX2 | LANEC_PCIEX1):
-               current_serdes |= MASK_LANE_C;
-               current_serdes &= ~MASK_LANE_D;
-               break;
-       default:
-               printf("CPLD serdes MUX: unsupported MUX type 0x%x\n", type);
-               return;
-       }
-
-       cpld_data->soft_mux_on |= CPLD_SET_MUX_SERDES;
-       cpld_data->serdes_mux = current_serdes;
-
-       if (need_reset == 1) {
-               printf("Reset board to enable configuration\n");
-               cpld_data->system_rst = CONFIG_RESET;
-       }
-}
-
-void print_serdes_mux(void)
+static void print_serdes_mux(void)
 {
        char current_serdes;
        struct cpld_data *cpld_data = (void *)(CONFIG_SYS_CPLD_BASE);