diff mbox

[U-Boot,V2] imx: MX6DQ{P}/DL:SABRESD Fix bmode eMMC failure

Message ID 1454126022-4314-1-git-send-email-van.freenix@gmail.com
State Awaiting Upstream
Delegated to: Stefano Babic
Headers show

Commit Message

Peng Fan Jan. 30, 2016, 3:53 a.m. UTC 
From: Ye Li <ye.li@nxp.com>

The BOOTCFG value used by bmode for SABRESD eMMC boot are actually for SD card.
Fixed the value to correct one.

Signed-off-by: Ye Li <ye.li@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Stefano Babic <sbabic@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
---

Change V2:
 Fix commit log, following Fabio's comments.

 board/freescale/mx6sabresd/mx6sabresd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Fabio Estevam Jan. 30, 2016, 2:34 p.m. UTC | #1 
On Sat, Jan 30, 2016 at 1:53 AM, Peng Fan <van.freenix@gmail.com> wrote:
> From: Ye Li <ye.li@nxp.com>
>
> The BOOTCFG value used by bmode for SABRESD eMMC boot are actually for SD card.
> Fixed the value to correct one.
>
> Signed-off-by: Ye Li <ye.li@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Reviewed-by: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>

Acked-by: Fabio Estevam <fabio.estevam@nxp.com>
Stefano Babic Feb. 2, 2016, 8:22 p.m. UTC | #2 
On 30/01/2016 04:53, Peng Fan wrote:
> From: Ye Li <ye.li@nxp.com>
> 
> The BOOTCFG value used by bmode for SABRESD eMMC boot are actually for SD card.
> Fixed the value to correct one.
> 
> Signed-off-by: Ye Li <ye.li@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Reviewed-by: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>
> ---
> 
> Change V2:
>  Fix commit log, following Fabio's comments.
> 
>  board/freescale/mx6sabresd/mx6sabresd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/board/freescale/mx6sabresd/mx6sabresd.c b/board/freescale/mx6sabresd/mx6sabresd.c
> index d20953d..42f89ab 100644
> --- a/board/freescale/mx6sabresd/mx6sabresd.c
> +++ b/board/freescale/mx6sabresd/mx6sabresd.c
> @@ -638,7 +638,7 @@ static const struct boot_mode board_boot_modes[] = {
>  	{"sd2",	 MAKE_CFGVAL(0x40, 0x28, 0x00, 0x00)},
>  	{"sd3",	 MAKE_CFGVAL(0x40, 0x30, 0x00, 0x00)},
>  	/* 8 bit bus width */
> -	{"emmc", MAKE_CFGVAL(0x40, 0x38, 0x00, 0x00)},
> +	{"emmc", MAKE_CFGVAL(0x60, 0x58, 0x00, 0x00)},
>  	{NULL,	 0},
>  };
>  #endif
> 
Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic
diff mbox

Patch

diff --git a/board/freescale/mx6sabresd/mx6sabresd.c b/board/freescale/mx6sabresd/mx6sabresd.c
index d20953d..42f89ab 100644
--- a/board/freescale/mx6sabresd/mx6sabresd.c
+++ b/board/freescale/mx6sabresd/mx6sabresd.c
@@ -638,7 +638,7 @@  static const struct boot_mode board_boot_modes[] = {
 	{"sd2",	 MAKE_CFGVAL(0x40, 0x28, 0x00, 0x00)},
 	{"sd3",	 MAKE_CFGVAL(0x40, 0x30, 0x00, 0x00)},
 	/* 8 bit bus width */
-	{"emmc", MAKE_CFGVAL(0x40, 0x38, 0x00, 0x00)},
+	{"emmc", MAKE_CFGVAL(0x60, 0x58, 0x00, 0x00)},
 	{NULL,	 0},
 };
 #endif