[U-Boot] [PATCH 2/2] imx: mx6slevk: Add support for USDHC1 and USDHC3 slots

2014-10-13 Thread Ye . Li
There are three SD/MMC sockets on mx6slevk boards. Implements the
full support for them.
The default boot socket is USDHC2, so the MMC environment is set
to that device.

Signed-off-by: Ye.Li b37...@freescale.com
---
 board/freescale/mx6slevk/mx6slevk.c |  101 --
 include/configs/mx6slevk.h  |6 ++-
 2 files changed, 100 insertions(+), 7 deletions(-)

diff --git a/board/freescale/mx6slevk/mx6slevk.c 
b/board/freescale/mx6slevk/mx6slevk.c
index a990b4c..265811b 100644
--- a/board/freescale/mx6slevk/mx6slevk.c
+++ b/board/freescale/mx6slevk/mx6slevk.c
@@ -51,6 +51,23 @@ static iomux_v3_cfg_t const uart1_pads[] = {
MX6_PAD_UART1_RXD__UART1_RXD | MUX_PAD_CTRL(UART_PAD_CTRL),
 };
 
+static iomux_v3_cfg_t const usdhc1_pads[] = {
+   /* 8 bit SD */
+   MX6_PAD_SD1_CLK__USDHC1_CLK | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_CMD__USDHC1_CMD | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT0__USDHC1_DAT0 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT1__USDHC1_DAT1 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT2__USDHC1_DAT2 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT3__USDHC1_DAT3 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT4__USDHC1_DAT4 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT5__USDHC1_DAT5 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT6__USDHC1_DAT6 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD1_DAT7__USDHC1_DAT7 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+
+   /*CD pin*/
+   MX6_PAD_KEY_ROW7__GPIO_4_7 | MUX_PAD_CTRL(NO_PAD_CTRL),
+};
+
 static iomux_v3_cfg_t const usdhc2_pads[] = {
MX6_PAD_SD2_CLK__USDHC2_CLK | MUX_PAD_CTRL(USDHC_PAD_CTRL),
MX6_PAD_SD2_CMD__USDHC2_CMD | MUX_PAD_CTRL(USDHC_PAD_CTRL),
@@ -58,6 +75,21 @@ static iomux_v3_cfg_t const usdhc2_pads[] = {
MX6_PAD_SD2_DAT1__USDHC2_DAT1 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
MX6_PAD_SD2_DAT2__USDHC2_DAT2 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
MX6_PAD_SD2_DAT3__USDHC2_DAT3 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+
+   /*CD pin*/
+   MX6_PAD_SD2_DAT7__GPIO_5_0 | MUX_PAD_CTRL(NO_PAD_CTRL),
+};
+
+static iomux_v3_cfg_t const usdhc3_pads[] = {
+   MX6_PAD_SD3_CLK__USDHC3_CLK | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD3_CMD__USDHC3_CMD | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD3_DAT0__USDHC3_DAT0 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD3_DAT1__USDHC3_DAT1 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD3_DAT2__USDHC3_DAT2 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+   MX6_PAD_SD3_DAT3__USDHC3_DAT3 | MUX_PAD_CTRL(USDHC_PAD_CTRL),
+
+   /*CD pin*/
+   MX6_PAD_REF_CLK_32K__GPIO_3_22 | MUX_PAD_CTRL(NO_PAD_CTRL),
 };
 
 static iomux_v3_cfg_t const fec_pads[] = {
@@ -103,21 +135,78 @@ static void setup_iomux_fec(void)
gpio_set_value(ETH_PHY_RESET, 1);
 }
 
-static struct fsl_esdhc_cfg usdhc_cfg[1] = {
-   {USDHC2_BASE_ADDR},
+#define USDHC1_CD_GPIO IMX_GPIO_NR(4, 7)
+#define USDHC2_CD_GPIO IMX_GPIO_NR(5, 0)
+#define USDHC3_CD_GPIO IMX_GPIO_NR(3, 22)
+
+static struct fsl_esdhc_cfg usdhc_cfg[3] = {
+   {USDHC1_BASE_ADDR},
+   {USDHC2_BASE_ADDR, 0, 4},
+   {USDHC3_BASE_ADDR, 0, 4},
 };
 
 int board_mmc_getcd(struct mmc *mmc)
 {
-   return 1;   /* Assume boot SD always present */
+   struct fsl_esdhc_cfg *cfg = (struct fsl_esdhc_cfg *)mmc-priv;
+   int ret = 0;
+
+   switch (cfg-esdhc_base) {
+   case USDHC1_BASE_ADDR:
+   ret = !gpio_get_value(USDHC1_CD_GPIO);
+   break;
+   case USDHC2_BASE_ADDR:
+   ret = !gpio_get_value(USDHC2_CD_GPIO);
+   break;
+   case USDHC3_BASE_ADDR:
+   ret = !gpio_get_value(USDHC3_CD_GPIO);
+   break;
+   }
+
+   return ret;
 }
 
 int board_mmc_init(bd_t *bis)
 {
-   imx_iomux_v3_setup_multiple_pads(usdhc2_pads, ARRAY_SIZE(usdhc2_pads));
+   int i;
+
+   /*
+* According to the board_mmc_init() the following map is done:
+* (U-boot device node)(Physical Port)
+* mmc0USDHC1
+* mmc1USDHC2
+* mmc2USDHC3
+*/
+   for (i = 0; i  CONFIG_SYS_FSL_USDHC_NUM; i++) {
+   switch (i) {
+   case 0:
+   imx_iomux_v3_setup_multiple_pads(
+   usdhc1_pads, ARRAY_SIZE(usdhc1_pads));
+   gpio_direction_input(USDHC1_CD_GPIO);
+   usdhc_cfg[0].sdhc_clk = mxc_get_clock(MXC_ESDHC_CLK);
+   break;
+   case 1:
+   imx_iomux_v3_setup_multiple_pads(
+   usdhc2_pads, ARRAY_SIZE(usdhc2_pads));
+   gpio_direction_input(USDHC2_CD_GPIO);
+   usdhc_cfg[1].sdhc_clk = mxc_get_clock(MXC_ESDHC2_CLK);
+   break;
+   case 2:
+   

Re: [U-Boot] [PATCH 2/2] imx: mx6slevk: Add support for USDHC1 and USDHC3 slots

2014-10-13 Thread Fabio Estevam
On Mon, Oct 13, 2014 at 4:51 AM, Ye.Li b37...@freescale.com wrote:

  int board_mmc_init(bd_t *bis)
  {
 -   imx_iomux_v3_setup_multiple_pads(usdhc2_pads, 
 ARRAY_SIZE(usdhc2_pads));
 +   int i;
 +
 +   /*
 +* According to the board_mmc_init() the following map is done:
 +* (U-boot device node)(Physical Port)
 +* mmc0USDHC1
 +* mmc1USDHC2
 +* mmc2USDHC3
 +*/
 +   for (i = 0; i  CONFIG_SYS_FSL_USDHC_NUM; i++) {
 +   switch (i) {
 +   case 0:
 +   imx_iomux_v3_setup_multiple_pads(
 +   usdhc1_pads, ARRAY_SIZE(usdhc1_pads));
 +   gpio_direction_input(USDHC1_CD_GPIO);
 +   usdhc_cfg[0].sdhc_clk = mxc_get_clock(MXC_ESDHC_CLK);
 +   break;
 +   case 1:
 +   imx_iomux_v3_setup_multiple_pads(
 +   usdhc2_pads, ARRAY_SIZE(usdhc2_pads));
 +   gpio_direction_input(USDHC2_CD_GPIO);
 +   usdhc_cfg[1].sdhc_clk = mxc_get_clock(MXC_ESDHC2_CLK);
 +   break;
 +   case 2:
 +   imx_iomux_v3_setup_multiple_pads(
 +   usdhc3_pads, ARRAY_SIZE(usdhc3_pads));
 +   gpio_direction_input(USDHC3_CD_GPIO);
 +   usdhc_cfg[2].sdhc_clk = mxc_get_clock(MXC_ESDHC3_CLK);
 +   break;
 +   default:
 +   printf(Warning: you configured more USDHC 
 controllers
 +   (%d) than supported by the board\n, i + 1);
 +   return 0;

No, you should 'return -EINVAL' here.

 +   }
 +
 +   if (fsl_esdhc_initialize(bis, usdhc_cfg[i]))
 +   printf(Warning: failed to initialize mmc dev 
 %d\n, i);
 +   }

ret = fsl_esdhc_initialize(bis, usdhc_cfg[i])
if (ret) {
   printf(Warning: failed to initialize mmc dev %d\n, i);
   return ret;
}
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot