* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
@ 2013-09-03 12:45 Przemyslaw Marczak
2013-09-09 8:04 ` Przemyslaw Marczak
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Przemyslaw Marczak @ 2013-09-03 12:45 UTC (permalink / raw)
To: u-boot
This change allow to use sd card on Goni the same like mmc 0.
SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
SD card is inited even if eMMC initialization fails.
Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
CC: Minkyu Kang <mk7.kang@samsung.com>
---
board/samsung/goni/goni.c | 31 ++++++++++++++++++++++++++++++-
1 file changed, 30 insertions(+), 1 deletion(-)
diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c
index 5b3d6ef..7098fe7 100644
--- a/board/samsung/goni/goni.c
+++ b/board/samsung/goni/goni.c
@@ -73,6 +73,7 @@ int checkboard(void)
int board_mmc_init(bd_t *bis)
{
int i;
+ int ret;
/* MASSMEMORY_EN: XMSMDATA7: GPJ2[7] output high */
s5p_gpio_direction_output(&s5pc110_gpio->j2, 7, 1);
@@ -95,7 +96,35 @@ int board_mmc_init(bd_t *bis)
s5p_gpio_set_drv(&s5pc110_gpio->g0, i, GPIO_DRV_4X);
}
- return s5p_mmc_init(0, 4);
+ ret = s5p_mmc_init(0, 4);
+ if (ret) {
+ puts("MMC: Failed to init MMC:0.\n");
+ }
+
+ /*
+ * SD card (T_FLASH) detect and init
+ * T_FLASH_DETECT: EINT28: GPH3[4] input mode
+ */
+ s5p_gpio_cfg_pin(&s5pc110_gpio->h3, 4, GPIO_INPUT);
+ s5p_gpio_set_pull(&s5pc110_gpio->h3, 4, GPIO_PULL_UP);
+
+ if (!s5p_gpio_get_value(&s5pc110_gpio->h3, 4)) {
+ for (i = 0; i < 7; i++) {
+ if (i == 2)
+ continue;
+ /* GPG2[0:6] special function 2 */
+ s5p_gpio_cfg_pin(&s5pc110_gpio->g2, i, 0x2);
+ /* GPG2[0:6] pull disable */
+ s5p_gpio_set_pull(&s5pc110_gpio->g2, i, GPIO_PULL_NONE);
+ /* GPG2[0:6] drv 4x */
+ s5p_gpio_set_drv(&s5pc110_gpio->g2, i, GPIO_DRV_4X);
+ }
+ if (s5p_mmc_init(2, 4))
+ puts("MMC: Failed to init SD card (MMC:2).\n");
+ }
+
+ /* SD card init is optional so print information when it fails only */
+ return ret;
}
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
2013-09-03 12:45 [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization Przemyslaw Marczak
@ 2013-09-09 8:04 ` Przemyslaw Marczak
2013-09-09 8:18 ` Jaehoon Chung
2013-09-09 10:51 ` Minkyu Kang
2 siblings, 0 replies; 6+ messages in thread
From: Przemyslaw Marczak @ 2013-09-09 8:04 UTC (permalink / raw)
To: u-boot
Hello Minkyu,
On 09/03/2013 02:45 PM, Przemyslaw Marczak wrote:
> This change allow to use sd card on Goni the same like mmc 0.
> SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
> SD card is inited even if eMMC initialization fails.
>
> Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> CC: Minkyu Kang <mk7.kang@samsung.com>
Could you review this patch, please?
Thank you.
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
2013-09-03 12:45 [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization Przemyslaw Marczak
2013-09-09 8:04 ` Przemyslaw Marczak
@ 2013-09-09 8:18 ` Jaehoon Chung
2013-09-09 10:51 ` Minkyu Kang
2 siblings, 0 replies; 6+ messages in thread
From: Jaehoon Chung @ 2013-09-09 8:18 UTC (permalink / raw)
To: u-boot
Hi,
On 09/03/2013 09:45 PM, Przemyslaw Marczak wrote:
> This change allow to use sd card on Goni the same like mmc 0.
> SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
> SD card is inited even if eMMC initialization fails.
>
> Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> CC: Minkyu Kang <mk7.kang@samsung.com>
> ---
> board/samsung/goni/goni.c | 31 ++++++++++++++++++++++++++++++-
> 1 file changed, 30 insertions(+), 1 deletion(-)
>
> diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c
> index 5b3d6ef..7098fe7 100644
> --- a/board/samsung/goni/goni.c
> +++ b/board/samsung/goni/goni.c
> @@ -73,6 +73,7 @@ int checkboard(void)
> int board_mmc_init(bd_t *bis)
> {
> int i;
> + int ret;
>
> /* MASSMEMORY_EN: XMSMDATA7: GPJ2[7] output high */
> s5p_gpio_direction_output(&s5pc110_gpio->j2, 7, 1);
> @@ -95,7 +96,35 @@ int board_mmc_init(bd_t *bis)
> s5p_gpio_set_drv(&s5pc110_gpio->g0, i, GPIO_DRV_4X);
> }
>
> - return s5p_mmc_init(0, 4);
> + ret = s5p_mmc_init(0, 4);
> + if (ret) {
> + puts("MMC: Failed to init MMC:0.\n");
> + }
> +
> + /*
> + * SD card (T_FLASH) detect and init
> + * T_FLASH_DETECT: EINT28: GPH3[4] input mode
> + */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->h3, 4, GPIO_INPUT);
> + s5p_gpio_set_pull(&s5pc110_gpio->h3, 4, GPIO_PULL_UP);
> +
> + if (!s5p_gpio_get_value(&s5pc110_gpio->h3, 4)) {
> + for (i = 0; i < 7; i++) {
> + if (i == 2)
> + continue;
> + /* GPG2[0:6] special function 2 */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->g2, i, 0x2);
> + /* GPG2[0:6] pull disable */
> + s5p_gpio_set_pull(&s5pc110_gpio->g2, i, GPIO_PULL_NONE);
> + /* GPG2[0:6] drv 4x */
> + s5p_gpio_set_drv(&s5pc110_gpio->g2, i, GPIO_DRV_4X);
> + }
> + if (s5p_mmc_init(2, 4))
> + puts("MMC: Failed to init SD card (MMC:2).\n");
Well, i think this is not good.
ret = s5p_mmc_init(2, 5)
if (ret) {
....
}
return ret;
Your checking is always returned "fail",if emmc is failed.
Best Regards,
Jaehoon Chung
> + }
> +
> + /* SD card init is optional so print information when it fails only */
> + return ret;
> }
> #endif
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
2013-09-03 12:45 [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization Przemyslaw Marczak
2013-09-09 8:04 ` Przemyslaw Marczak
2013-09-09 8:18 ` Jaehoon Chung
@ 2013-09-09 10:51 ` Minkyu Kang
2 siblings, 0 replies; 6+ messages in thread
From: Minkyu Kang @ 2013-09-09 10:51 UTC (permalink / raw)
To: u-boot
Dear Przemyslaw Marczak,
On 03/09/13 21:45, Przemyslaw Marczak wrote:
> This change allow to use sd card on Goni the same like mmc 0.
> SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
> SD card is inited even if eMMC initialization fails.
>
> Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> CC: Minkyu Kang <mk7.kang@samsung.com>
> ---
> board/samsung/goni/goni.c | 31 ++++++++++++++++++++++++++++++-
> 1 file changed, 30 insertions(+), 1 deletion(-)
>
> diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c
> index 5b3d6ef..7098fe7 100644
> --- a/board/samsung/goni/goni.c
> +++ b/board/samsung/goni/goni.c
> @@ -73,6 +73,7 @@ int checkboard(void)
> int board_mmc_init(bd_t *bis)
> {
> int i;
> + int ret;
>
> /* MASSMEMORY_EN: XMSMDATA7: GPJ2[7] output high */
> s5p_gpio_direction_output(&s5pc110_gpio->j2, 7, 1);
> @@ -95,7 +96,35 @@ int board_mmc_init(bd_t *bis)
> s5p_gpio_set_drv(&s5pc110_gpio->g0, i, GPIO_DRV_4X);
> }
>
> - return s5p_mmc_init(0, 4);
> + ret = s5p_mmc_init(0, 4);
> + if (ret) {
> + puts("MMC: Failed to init MMC:0.\n");
> + }
please remove brace.
> +
> + /*
> + * SD card (T_FLASH) detect and init
> + * T_FLASH_DETECT: EINT28: GPH3[4] input mode
> + */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->h3, 4, GPIO_INPUT);
> + s5p_gpio_set_pull(&s5pc110_gpio->h3, 4, GPIO_PULL_UP);
> +
> + if (!s5p_gpio_get_value(&s5pc110_gpio->h3, 4)) {
> + for (i = 0; i < 7; i++) {
> + if (i == 2)
> + continue;
please add blank line.
> + /* GPG2[0:6] special function 2 */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->g2, i, 0x2);
> + /* GPG2[0:6] pull disable */
> + s5p_gpio_set_pull(&s5pc110_gpio->g2, i, GPIO_PULL_NONE);
> + /* GPG2[0:6] drv 4x */
> + s5p_gpio_set_drv(&s5pc110_gpio->g2, i, GPIO_DRV_4X);
> + }
please add blank line.
> + if (s5p_mmc_init(2, 4))
> + puts("MMC: Failed to init SD card (MMC:2).\n");
> + }
> +
> + /* SD card init is optional so print information when it fails only */
maybe this comment should be located near s5p_mmc_init(2, 4).
> + return ret;
> }
> #endif
>
>
Thanks,
Minkyu Kang.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
@ 2013-09-09 12:50 Przemyslaw Marczak
2013-09-10 2:30 ` Jaehoon Chung
0 siblings, 1 reply; 6+ messages in thread
From: Przemyslaw Marczak @ 2013-09-09 12:50 UTC (permalink / raw)
To: u-boot
This change allow to use sd card on Goni the same like mmc 0.
SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
SD card is inited even if eMMC initialization fails.
Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
CC: Minkyu Kang <mk7.kang@samsung.com>
---
board/samsung/goni/goni.c | 33 +++++++++++++++++++++++++++++++--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c
index 5b3d6ef..d63e902 100644
--- a/board/samsung/goni/goni.c
+++ b/board/samsung/goni/goni.c
@@ -72,7 +72,7 @@ int checkboard(void)
#ifdef CONFIG_GENERIC_MMC
int board_mmc_init(bd_t *bis)
{
- int i;
+ int i, ret, ret_sd = 0;
/* MASSMEMORY_EN: XMSMDATA7: GPJ2[7] output high */
s5p_gpio_direction_output(&s5pc110_gpio->j2, 7, 1);
@@ -95,7 +95,36 @@ int board_mmc_init(bd_t *bis)
s5p_gpio_set_drv(&s5pc110_gpio->g0, i, GPIO_DRV_4X);
}
- return s5p_mmc_init(0, 4);
+ ret = s5p_mmc_init(0, 4);
+ if (ret)
+ puts("MMC: Failed to init MMC:0.\n");
+
+ /*
+ * SD card (T_FLASH) detect and init
+ * T_FLASH_DETECT: EINT28: GPH3[4] input mode
+ */
+ s5p_gpio_cfg_pin(&s5pc110_gpio->h3, 4, GPIO_INPUT);
+ s5p_gpio_set_pull(&s5pc110_gpio->h3, 4, GPIO_PULL_UP);
+
+ if (!s5p_gpio_get_value(&s5pc110_gpio->h3, 4)) {
+ for (i = 0; i < 7; i++) {
+ if (i == 2)
+ continue;
+
+ /* GPG2[0:6] special function 2 */
+ s5p_gpio_cfg_pin(&s5pc110_gpio->g2, i, 0x2);
+ /* GPG2[0:6] pull disable */
+ s5p_gpio_set_pull(&s5pc110_gpio->g2, i, GPIO_PULL_NONE);
+ /* GPG2[0:6] drv 4x */
+ s5p_gpio_set_drv(&s5pc110_gpio->g2, i, GPIO_DRV_4X);
+ }
+
+ ret_sd = s5p_mmc_init(2, 4)
+ if (ret_sd)
+ puts("MMC: Failed to init SD card (MMC:2).\n");
+ }
+
+ return ret | ret_sd;
}
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization.
2013-09-09 12:50 Przemyslaw Marczak
@ 2013-09-10 2:30 ` Jaehoon Chung
0 siblings, 0 replies; 6+ messages in thread
From: Jaehoon Chung @ 2013-09-10 2:30 UTC (permalink / raw)
To: u-boot
Dear Przemyslaw,
On 09/09/2013 09:50 PM, Przemyslaw Marczak wrote:
> This change allow to use sd card on Goni the same like mmc 0.
> SD card is mmc dev 1, so it can be used like this: "fatls mmc 1:2".
> SD card is inited even if eMMC initialization fails.
>
> Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> CC: Minkyu Kang <mk7.kang@samsung.com>
> ---
> board/samsung/goni/goni.c | 33 +++++++++++++++++++++++++++++++--
> 1 file changed, 31 insertions(+), 2 deletions(-)
>
> diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c
> index 5b3d6ef..d63e902 100644
> --- a/board/samsung/goni/goni.c
> +++ b/board/samsung/goni/goni.c
> @@ -72,7 +72,7 @@ int checkboard(void)
> #ifdef CONFIG_GENERIC_MMC
> int board_mmc_init(bd_t *bis)
> {
> - int i;
> + int i, ret, ret_sd = 0;
>
> /* MASSMEMORY_EN: XMSMDATA7: GPJ2[7] output high */
> s5p_gpio_direction_output(&s5pc110_gpio->j2, 7, 1);
> @@ -95,7 +95,36 @@ int board_mmc_init(bd_t *bis)
> s5p_gpio_set_drv(&s5pc110_gpio->g0, i, GPIO_DRV_4X);
> }
>
> - return s5p_mmc_init(0, 4);
> + ret = s5p_mmc_init(0, 4);
> + if (ret)
> + puts("MMC: Failed to init MMC:0.\n");
> +
> + /*
> + * SD card (T_FLASH) detect and init
> + * T_FLASH_DETECT: EINT28: GPH3[4] input mode
> + */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->h3, 4, GPIO_INPUT);
> + s5p_gpio_set_pull(&s5pc110_gpio->h3, 4, GPIO_PULL_UP);
> +
> + if (!s5p_gpio_get_value(&s5pc110_gpio->h3, 4)) {
> + for (i = 0; i < 7; i++) {
> + if (i == 2)
> + continue;
> +
> + /* GPG2[0:6] special function 2 */
> + s5p_gpio_cfg_pin(&s5pc110_gpio->g2, i, 0x2);
> + /* GPG2[0:6] pull disable */
> + s5p_gpio_set_pull(&s5pc110_gpio->g2, i, GPIO_PULL_NONE);
> + /* GPG2[0:6] drv 4x */
> + s5p_gpio_set_drv(&s5pc110_gpio->g2, i, GPIO_DRV_4X);
> + }
> +
> + ret_sd = s5p_mmc_init(2, 4)
> + if (ret_sd)
> + puts("MMC: Failed to init SD card (MMC:2).\n");
I think you can refer to trats.c.
Best Regards,
Jaehoon Chung
> + }
> +
> + return ret | ret_sd;
> }
> #endif
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-09-10 2:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-03 12:45 [U-Boot] [PATCH] arm:goni:mmc: Add sd card detection and initialization Przemyslaw Marczak
2013-09-09 8:04 ` Przemyslaw Marczak
2013-09-09 8:18 ` Jaehoon Chung
2013-09-09 10:51 ` Minkyu Kang
-- strict thread matches above, loose matches on Subject: below --
2013-09-09 12:50 Przemyslaw Marczak
2013-09-10 2:30 ` Jaehoon Chung
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox