diff mbox series

[v3,1/3] spl: mmc: support uboot image offset on main partition

Message ID 64d8ffc3379ce93437113d8f6df513fa6d46e8c3.1579072090.git.baruch@tkos.co.il
State Accepted
Commit e42afd8416fea64725eddf26be4ec3a0397778d6
Headers show
Series [v3,1/3] spl: mmc: support uboot image offset on main partition | expand

Commit Message

Baruch Siach Jan. 15, 2020, 7:08 a.m. UTC
On Armada 38x platforms the ROM code loads SPL from offset 0 of eMMC
hardware boot partitions. When there are no boot partitions (i.e. SD
card) the ROM skips the first sector that usually contains the (logical)
partition table. Since the generated .kwb image contains the main U-Boot
image in a fixed location (0x140 sectors by default), we end up with the
main U-Boot image in offset of 1 sector. The current workaround is to
manually set CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR to 0x141 to
compensate for that.

This patch uses the run-time detected boot partition to determine the
right offset of the main U-Boot partition. The generated .kwb image is
now compatible with both eMMC boot partition, and SD card main data
partition.

Signed-off-by: Baruch Siach <baruch at tkos.co.il>
---
Travis CI: Passed
https://travis-ci.com/baruchsiach-travis/u-boot/builds/144357746

v3: No change

v2: Rebase on top of current master
---
 common/spl/Kconfig   | 12 ++++++++++++
 common/spl/spl_mmc.c | 15 +++++++++++++--
 2 files changed, 25 insertions(+), 2 deletions(-)

Comments

Stefan Roese Jan. 16, 2020, 9:57 a.m. UTC | #1
On 15.01.20 08:08, Baruch Siach wrote:
> On Armada 38x platforms the ROM code loads SPL from offset 0 of eMMC
> hardware boot partitions. When there are no boot partitions (i.e. SD
> card) the ROM skips the first sector that usually contains the (logical)
> partition table. Since the generated .kwb image contains the main U-Boot
> image in a fixed location (0x140 sectors by default), we end up with the
> main U-Boot image in offset of 1 sector. The current workaround is to
> manually set CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR to 0x141 to
> compensate for that.
> 
> This patch uses the run-time detected boot partition to determine the
> right offset of the main U-Boot partition. The generated .kwb image is
> now compatible with both eMMC boot partition, and SD card main data
> partition.
> 
> Signed-off-by: Baruch Siach <baruch at tkos.co.il>

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan
Stefan Roese Jan. 16, 2020, 2:27 p.m. UTC | #2
On 15.01.20 08:08, Baruch Siach wrote:
> On Armada 38x platforms the ROM code loads SPL from offset 0 of eMMC
> hardware boot partitions. When there are no boot partitions (i.e. SD
> card) the ROM skips the first sector that usually contains the (logical)
> partition table. Since the generated .kwb image contains the main U-Boot
> image in a fixed location (0x140 sectors by default), we end up with the
> main U-Boot image in offset of 1 sector. The current workaround is to
> manually set CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR to 0x141 to
> compensate for that.
> 
> This patch uses the run-time detected boot partition to determine the
> right offset of the main U-Boot partition. The generated .kwb image is
> now compatible with both eMMC boot partition, and SD card main data
> partition.
> 
> Signed-off-by: Baruch Siach <baruch at tkos.co.il>

Applied to u-boot-marvell/master

Thanks,
Stefan
diff mbox series

Patch

diff --git a/common/spl/Kconfig b/common/spl/Kconfig
index 61488daa3c3d..784742a3a218 100644
--- a/common/spl/Kconfig
+++ b/common/spl/Kconfig
@@ -329,6 +329,18 @@  config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
 	  Address on the MMC to load U-Boot from, when the MMC is being used
 	  in raw mode. Units: MMC sectors (1 sector = 512 bytes).
 
+config SYS_MMCSD_RAW_MODE_U_BOOT_DATA_PART_OFFSET
+	hex "U-Boot main hardware partition image offset"
+	depends on SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
+	default 0x0
+	help
+	  On some platforms SPL location depends on hardware partition. The ROM
+	  code skips the MBR sector when loading SPL from main hardware data
+	  partition. This adds offset to the main U-Boot image. Set this symbol
+	  to the number of skipped sectors.
+
+	  If unsure, leave the default.
+
 config SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION
 	bool "MMC Raw mode: by partition"
 	help
diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
index 2ede096e61c8..3e6a17c110cd 100644
--- a/common/spl/spl_mmc.c
+++ b/common/spl/spl_mmc.c
@@ -49,6 +49,16 @@  static ulong h_spl_load_read(struct spl_load_info *load, ulong sector,
 	return blk_dread(mmc_get_blk_desc(mmc), sector, count, buf);
 }
 
+static __maybe_unused unsigned long spl_mmc_raw_uboot_offset(int part)
+{
+#if IS_ENABLED(CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR)
+	if (part == 0)
+		return CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_DATA_PART_OFFSET;
+#endif
+
+	return 0;
+}
+
 static __maybe_unused
 int mmc_load_image_raw_sector(struct spl_image_info *spl_image,
 			      struct mmc *mmc, unsigned long sector)
@@ -325,7 +335,7 @@  int spl_mmc_load(struct spl_image_info *spl_image,
 	static struct mmc *mmc;
 	u32 boot_mode;
 	int err = 0;
-	__maybe_unused int part;
+	__maybe_unused int part = 0;
 
 	/* Perform peripheral init only once */
 	if (!mmc) {
@@ -391,7 +401,8 @@  int spl_mmc_load(struct spl_image_info *spl_image,
 			return err;
 #endif
 #ifdef CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
-		err = mmc_load_image_raw_sector(spl_image, mmc, raw_sect);
+		err = mmc_load_image_raw_sector(spl_image, mmc,
+				raw_sect + spl_mmc_raw_uboot_offset(part));
 		if (!err)
 			return err;
 #endif