public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
@ 2016-04-21  1:48 Fabio Estevam
  2016-04-26 15:58 ` Fabio Estevam
  2016-05-03  1:05 ` [U-Boot] [U-Boot, " Tom Rini
  0 siblings, 2 replies; 8+ messages in thread
From: Fabio Estevam @ 2016-04-21  1:48 UTC (permalink / raw)
  To: u-boot

From: Fabio Estevam <fabio.estevam@nxp.com>

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 <fabio.estevam@nxp.com>
---
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(-)

diff --git a/include/configs/mx6ul_14x14_evk.h b/include/configs/mx6ul_14x14_evk.h
index 2fff34a..76929ac 100644
--- a/include/configs/mx6ul_14x14_evk.h
+++ b/include/configs/mx6ul_14x14_evk.h
@@ -46,7 +46,6 @@
 #define CONFIG_SYS_FSL_USDHC_NUM	2
 #endif
 
-#define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */
 #endif
 
 /* I2C configs */
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-21  1:48 [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT Fabio Estevam
@ 2016-04-26 15:58 ` Fabio Estevam
  2016-04-26 16:04   ` Tom Rini
  2016-04-26 16:11   ` Stefano Babic
  2016-05-03  1:05 ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 2 replies; 8+ messages in thread
From: Fabio Estevam @ 2016-04-26 15:58 UTC (permalink / raw)
  To: u-boot

Tom/Stefano,

On Wed, Apr 20, 2016 at 10:48 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> 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 <fabio.estevam@nxp.com>

Could this one be applied for 2016.05?

Thanks

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-26 15:58 ` Fabio Estevam
@ 2016-04-26 16:04   ` Tom Rini
  2016-04-26 16:33     ` Stefano Babic
  2016-04-26 16:11   ` Stefano Babic
  1 sibling, 1 reply; 8+ messages in thread
From: Tom Rini @ 2016-04-26 16:04 UTC (permalink / raw)
  To: u-boot

On Tue, Apr 26, 2016 at 12:58:05PM -0300, Fabio Estevam wrote:
> Tom/Stefano,
> 
> On Wed, Apr 20, 2016 at 10:48 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > From: Fabio Estevam <fabio.estevam@nxp.com>
> >
> > 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 <fabio.estevam@nxp.com>
> 
> 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?

-- 
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>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-26 15:58 ` Fabio Estevam
  2016-04-26 16:04   ` Tom Rini
@ 2016-04-26 16:11   ` Stefano Babic
  1 sibling, 0 replies; 8+ messages in thread
From: Stefano Babic @ 2016-04-26 16:11 UTC (permalink / raw)
  To: u-boot

On 26/04/2016 17:58, Fabio Estevam wrote:
> Tom/Stefano,
> 
> On Wed, Apr 20, 2016 at 10:48 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> From: Fabio Estevam <fabio.estevam@nxp.com>
>>
>> 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 <fabio.estevam@nxp.com>
> 
> Could this one be applied for 2016.05?

I pick it up

Stefano


-- 
=====================================================================
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
=====================================================================

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-26 16:04   ` Tom Rini
@ 2016-04-26 16:33     ` Stefano Babic
  2016-05-03  0:59       ` Fabio Estevam
  0 siblings, 1 reply; 8+ messages in thread
From: Stefano Babic @ 2016-04-26 16:33 UTC (permalink / raw)
  To: u-boot

Hi Tom,

On 26/04/2016 18:04, Tom Rini wrote:
> On Tue, Apr 26, 2016 at 12:58:05PM -0300, Fabio Estevam wrote:
>> Tom/Stefano,
>>
>> On Wed, Apr 20, 2016 at 10:48 PM, Fabio Estevam <festevam@gmail.com> wrote:
>>> From: Fabio Estevam <fabio.estevam@nxp.com>
>>>
>>> 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 <fabio.estevam@nxp.com>
>>
>> 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

-- 
=====================================================================
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
=====================================================================

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-26 16:33     ` Stefano Babic
@ 2016-05-03  0:59       ` Fabio Estevam
  2016-05-03  1:05         ` Tom Rini
  0 siblings, 1 reply; 8+ messages in thread
From: Fabio Estevam @ 2016-05-03  0:59 UTC (permalink / raw)
  To: u-boot

Tom,

On Tue, Apr 26, 2016 at 1:33 PM, Stefano Babic <sbabic@denx.de> wrote:

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

Are you applying this one directly?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-05-03  0:59       ` Fabio Estevam
@ 2016-05-03  1:05         ` Tom Rini
  0 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-05-03  1:05 UTC (permalink / raw)
  To: u-boot

On Mon, May 02, 2016 at 09:59:33PM -0300, Fabio Estevam wrote:
> Tom,
> 
> On Tue, Apr 26, 2016 at 1:33 PM, Stefano Babic <sbabic@denx.de> wrote:
> 
> > Fine with me if you want to pick it up directly without PR.
> 
> Are you applying this one directly?

Thanks for the reminder.

-- 
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>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [U-Boot, v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT
  2016-04-21  1:48 [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT Fabio Estevam
  2016-04-26 15:58 ` Fabio Estevam
@ 2016-05-03  1:05 ` Tom Rini
  1 sibling, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-05-03  1:05 UTC (permalink / raw)
  To: u-boot

On Wed, Apr 20, 2016 at 10:48:13PM -0300, Fabio Estevam wrote:

> From: Fabio Estevam <fabio.estevam@nxp.com>
> 
> 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 <fabio.estevam@nxp.com>

Applied to u-boot/master, thanks!

-- 
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>

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2016-05-03  1:05 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-21  1:48 [U-Boot] [PATCH v2] mx6ul_evk: Remove CONFIG_SUPPORT_EMMC_BOOT Fabio Estevam
2016-04-26 15:58 ` Fabio Estevam
2016-04-26 16:04   ` Tom Rini
2016-04-26 16:33     ` Stefano Babic
2016-05-03  0:59       ` Fabio Estevam
2016-05-03  1:05         ` Tom Rini
2016-04-26 16:11   ` Stefano Babic
2016-05-03  1:05 ` [U-Boot] [U-Boot, " Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox