Skip to contentSkip to conversation

arm, imx6: add support for SD/MMC Manufacture Mode

PATCH New Jay Carlson 2 participants 2 messages Jun 22, 2018 – Jul 23, 2018

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 arm, imx6: add support for SD/MMC Manufacture Mode

This patch adds support for booting blank i.MX6 devices in SD/MMC Manufacture
Mode: a fallback mode which the boot ROM enters when no fuses or boot mode pins have been set, the USB serial downloader connection is inactive, and there is an SD card inserted into uSDHC1. The i.MX7 and i.MX8, which both have a Boot_SW_Info data structure which can be parsed to determine boot source, is unaffected by this patch.

---
 arch/arm/mach-imx/spl.c | 10 ++++++++++
 1 file changed, 10 insertions(+)
arch/arm/mach-imx/spl.c +10 -0
index a20b30d154..8ecae8605c 100644
@@ -43,6 +43,16 @@ u32 spl_boot_device(void)
 	 */
 	if (is_usbotg_phy_active())
 		return BOOT_DEVICE_BOARD;
+	
+	/**
+	 * To support SD/MMC Manufacture Mode, we check that BOOT_MODE == 0
+	 * and that BT_FUSE_SEL == 0. If this was disabled via
+	 * blowing DISABLE_SDMMC_MFG, or if SDMMC MFG mode failed,
+	 * we would be in USB download mode, which the previous line would 
+	 * have detected.
+	 */
+	if (((bmode >> 24) & 0x03) == 0x00 && (bmode >> 4) == 0x00)
+		return BOOT_DEVICE_MMC1;
 
 	/* BOOT_CFG1[7:4] - see IMX6DQRM Table 8-8 */
 	switch ((reg & IMX6_BMODE_MASK) >> IMX6_BMODE_SHIFT) {
Signature · 1 line
2.16.1.windows.4
#1 lore raw

PATCH v2 1/1 arm, imx6: add support for SD/MMC Manufacture Mode

Hi Jay,

Quoted 24 lines · Jay Carlson

This patch adds support for booting blank i.MX6 devices in SD/MMC Manufacture
Mode: a fallback mode which the boot ROM enters when no fuses or boot mode pins have been set, the USB serial downloader connection is inactive, and there is an SD card inserted into uSDHC1. The i.MX7 and i.MX8, which both have a Boot_SW_Info data structure which can be parsed to determine boot source, is unaffected by this patch.

---
 arch/arm/mach-imx/spl.c | 10 ++++++++++
 1 file changed, 10 insertions(+)
arch/arm/mach-imx/spl.c +3 -0
index a20b30d154..8ecae8605c 100644
@@ -43,6 +43,16 @@ u32 spl_boot_device(void)
 	 */
 	if (is_usbotg_phy_active())
 		return BOOT_DEVICE_BOARD;
+	
+	/**
+	 * To support SD/MMC Manufacture Mode, we check that BOOT_MODE == 0

arch/arm/mach-imx/spl.c u32 spl_boot_device(void) · line 58

Quoted 10 lines
arch/arm/mach-imx/spl.c +7 -0
+	 * and that BT_FUSE_SEL == 0. If this was disabled via
+	 * blowing DISABLE_SDMMC_MFG, or if SDMMC MFG mode failed,
+	 * we would be in USB download mode, which the previous line would 
+	 * have detected.
+	 */
+	if (((bmode >> 24) & 0x03) == 0x00 && (bmode >> 4) == 0x00)
+		return BOOT_DEVICE_MMC1;
 
 	/* BOOT_CFG1[7:4] - see IMX6DQRM Table 8-8 */
 	switch ((reg & IMX6_BMODE_MASK) >> IMX6_BMODE_SHIFT) {

This looks to me a fix for a custom board, but it does not seem that can be applied general for all devices. I understand that this works in your use case, where you have soldered eMMC and in factory you install the software via SD card. But this is one use case.

Your change goes into general code (mach-imx/spl.c), and then it can breaks many projects. You coul write your board_boot_order() in your boot code to set the order of the device to be booted.

Best regards,
Stefano Babic

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