Hi Tim,
Sorry for the late reply.

On 08/05/14 08:16, Tim Harvey wrote:
Add a common spl.c file to support boot device functions needed for SPL
such as detecting the boot device.

Signed-off-by: Tim Harvey <thar...@gateworks.com>
---
v3:
  - added Freescale copyright

v2:
  - re-base on top of Masahiro Yamada's consolidation patch:
    http://patchwork.ozlabs.org/patch/341817/
  - moved spl_boot_device() and get_boot_mode() into own common imx spl.c file
  - use mem-mapped structure for smbr reg access
---

[..snip..]

+               }
+               break;
+       /* SD/eSD: 8.5.3, Table 8-15  */
+       case 0x4:
+       case 0x5:
+               return BOOT_DEVICE_MMC1;
+       /* MMC/eMMC: 8.5.3 */
+       case 0x6:
+       case 0x7:
+               return BOOT_DEVICE_MMC1;
+       /* NAND Flash: 8.5.2 */
+       case 0x8 ... 0xf:
+               return BOOT_DEVICE_NAND;
+       }
+       return BOOT_DEVICE_NONE;
+}
+#endif
+
+#if defined(CONFIG_SPL_MMC_SUPPORT)
+/* called from spl_mmc to see type of boot mode for storage (RAW or FAT) */
+u32 spl_boot_mode(void)
+{
+       switch (spl_boot_device()) {
+       /* for MMC return either RAW or FAT mode */
+       case BOOT_DEVICE_MMC1:
+       case BOOT_DEVICE_MMC2:

spl_boot_device() returns only BOOT_DEVICE_MMC1. Was the MMC/eMMC case
supposed to return a BOOT_DEVICE_MMC2?

Aside from this nitpick,
Acked-by: Nikita Kiryanov <nik...@compulab.co.il>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to