[PATCH v2 2/2] spl: mmc: Pass eMMC HW partition 7 through
Marek Vasut
marex at denx.de
Sat Mar 11 17:14:44 CET 2023
The eMMC HW partition 0 and 7 both mean USER HW partition. Use this as
a mean of propagating A/B copy selection within USER HW partition. The
spl_mmc_get_uboot_raw_sector() can detect that a USER HW partition is
in use and based on whether it is 0 or 7, select appropriate sector to
load from.
Signed-off-by: Fedor Ross <fedor.ross at ifm.com>
Signed-off-by: Marek Vasut <marex at denx.de>
---
Cc: Jaehoon Chung <jh80.chung at samsung.com>
Cc: Peng Fan <peng.fan at nxp.com>
---
V2: Initialize the part variable, else it is passed uninitialized to
mmc_load_image_raw_sector(..raw_sect + spl_mmc_raw_uboot_offset(part));
and that prevents pine64 from booting. Interestingly enough, there
is no warning emitted, which sounds like a compiler bug.
---
common/spl/spl_mmc.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
index 8a0c7331267..7a3fd3590dd 100644
--- a/common/spl/spl_mmc.c
+++ b/common/spl/spl_mmc.c
@@ -420,7 +420,7 @@ int spl_mmc_load(struct spl_image_info *spl_image,
static struct mmc *mmc;
u32 boot_mode;
int err = 0;
- __maybe_unused int hw_part = 0;
+ __maybe_unused int part = 0, hw_part = 0;
int mmc_dev;
/* Perform peripheral init only once for an mmc device */
@@ -445,11 +445,12 @@ int spl_mmc_load(struct spl_image_info *spl_image,
switch (boot_mode) {
case MMCSD_MODE_EMMCBOOT:
hw_part = spl_mmc_emmc_boot_partition(mmc);
+ part = hw_part == 7 ? 0 : hw_part;
if (CONFIG_IS_ENABLED(MMC_TINY))
- err = mmc_switch_part(mmc, hw_part);
+ err = mmc_switch_part(mmc, part);
else
- err = blk_dselect_hwpart(mmc_get_blk_desc(mmc), hw_part);
+ err = blk_dselect_hwpart(mmc_get_blk_desc(mmc), part);
if (err) {
#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT
@@ -479,7 +480,7 @@ int spl_mmc_load(struct spl_image_info *spl_image,
#endif
#ifdef CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
err = mmc_load_image_raw_sector(spl_image, bootdev, mmc,
- raw_sect + spl_mmc_raw_uboot_offset(hw_part));
+ raw_sect + spl_mmc_raw_uboot_offset(part));
if (!err)
return err;
#endif
--
2.39.2
More information about the U-Boot
mailing list