mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

PATCH Superseded Fabio Estevam 3 participants 8 messages Apr 21, 2016 – May 3, 2016

lore

Share & tools

lore

Fabio Estevam

author

2016-04-21 01:48 UTC #0 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

From: Fabio Estevam <[email protected]>

mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:

U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###

When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Keep using SPL. Fix the boot issue by removing CONFIG_SUPPORT_EMMC_BOOT

 include/configs/mx6ul_14x14_evk.h | 1 -
 1 file changed, 1 deletion(-)

include/configs/mx6ul_14x14_evk.h +0 -1

index 2fff34a..76929ac 100644
@@ -46,7 +46,6 @@
 #define CONFIG_SYS_FSL_USDHC_NUM	2
 #endif
 
-#define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */
 #endif
 
 /* I2C configs */

Signature · 1 line

1.9.1

Fabio Estevam

author

2016-04-26 15:58 UTC #1 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

Tom/Stefano,

Quoted 12 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>
mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:
U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###

Quoted 5 lines

When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.
Signed-off-by Fabio Estevam <[email protected]>

Could this one be applied for 2016.05?

Thanks

Tom Rini

2016-04-26 16:04 UTC #2 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

Quoted 15 lines · Fabio Estevam

Tom/Stefano,
Quoted 12 lines · Fabio Estevam
From: Fabio Estevam <[email protected]>
mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:
U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###

Quoted 7 lines

Quoted 5 lines
When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.
Signed-off-by Fabio Estevam <[email protected]>
Could this one be applied for 2016.05?

Fine with me. Do you want to make a PR or should I take it Stefano, if you have no objections?

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160426/33a81b78/attachment.sig>

Stefano Babic

2016-04-26 16:33 UTC #3 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

Hi Tom,

Quoted 20 lines · Tom Rini

Quoted 19 lines · Fabio Estevam
Tom/Stefano,
Quoted 16 lines · Fabio Estevam
From: Fabio Estevam <[email protected]>
mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:
U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###
When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.

Quoted 6 lines

Quoted 3 lines
Quoted 1 line
Signed-off-by Fabio Estevam <[email protected]>
Could this one be applied for 2016.05?
Fine with me. Do you want to make a PR or should I take it Stefano, if you have no objections?

Fine with me if you want to pick it up directly without PR.

Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Fabio Estevam

author

2016-05-03 00:59 UTC #4 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

Tom,

Quoted 1 line · Stefano Babic

Fine with me if you want to pick it up directly without PR.

Are you applying this one directly?

Tom Rini

2016-05-03 01:05 UTC #5 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

Quoted 7 lines · Fabio Estevam

Tom,
Quoted 1 line · Stefano Babic
Fine with me if you want to pick it up directly without PR.
Are you applying this one directly?

Thanks for the reminder.

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160502/1ede1abb/attachment.sig>

Stefano Babic

2016-04-26 16:11 UTC #6 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

In reply to Fabio Estevam

Quoted 15 lines · Fabio Estevam

Tom/Stefano,
Quoted 12 lines · Fabio Estevam
From: Fabio Estevam <[email protected]>
mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:
U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###

Quoted 7 lines

Quoted 5 lines
When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.
Signed-off-by Fabio Estevam <[email protected]>
Could this one be applied for 2016.05?

I pick it up

Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Tom Rini

2016-05-03 01:05 UTC #7 lore raw

PATCH v2 1/1 mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT

In reply to Fabio Estevam

Quoted 12 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>
mx6ul_evk does not come with a eMMC populated, so we should not define CONFIG_SUPPORT_EMMC_BOOT as it causes SPL to not be able to boot some brands of SD cards, such as SanDisk microSD HC - 8GB:
U-Boot SPL 2016.05-rc1-28384-g108f841 (Apr 19 2016 - 11:19:11) Trying to boot from MMC1
spl: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###

Quoted 5 lines

When CONFIG_SUPPORT_EMMC_BOOT is defined spl_boot_mode() returns MMCSD_MODE_EMMCBOOT, so remove this option to have a reliable boot via SD card.
Signed-off-by Fabio Estevam <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160502/65eda860/attachment.sig>