* [U-Boot] [PATCH v3] ti_armv7_common.h: Fix U-Boot location on eMMC
@ 2016-04-20 9:05 Sam Protsenko
2016-04-20 14:38 ` Tom Rini
2016-04-26 0:16 ` [U-Boot] [U-Boot, " Tom Rini
0 siblings, 2 replies; 3+ messages in thread
From: Sam Protsenko @ 2016-04-20 9:05 UTC (permalink / raw)
To: u-boot
According to common eMMC partition table for Android boot (see
PARTS_DEFAULT definition in include/configs/dra7xx_evm.h), "bootloader"
partition (where u-boot.img is stored) starts at 256 KiB. Which is equal
to 512 sectors (as 1 MMC sector size is 512 bytes).
This patch fixes CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR constant so that
it points to correct address of "bootloader" partition and SPL is able to
read, parse and run u-boot.img correctly.
This change was originally done as part of patch [1] in omapzoom u-boot.
Without this patch, SPL fails to parse U-Boot header with next error:
mkimage signature not found - ih_magic = 4814325a
While at it, also fix U-Boot partition size, which is 384 KiB (as stated in
include/configs/dra7xx_evm.h).
[1] http://omapzoom.org/?p=repo/u-boot.git;a=commit;h=742b82d0c0aa0ed8096c2225a00e9f350212efa9
Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
---
Changes for v2:
- adjust also partition size to be 384 KiB
Changes for v3:
- keep U_BOOT_SECTOR value in hex for consistency
include/configs/ti_armv7_common.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
index 787d04a..0725d79 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -229,8 +229,8 @@
#endif
/* RAW SD card / eMMC locations. */
-#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR 0x300 /* address 0x60000 */
-#define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS 0x200 /* 256 KB */
+#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR 0x200 /* address 0x40000 */
+#define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS 0x300 /* 384 KB */
/* FAT sd card locations. */
#define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION 1
--
2.8.0.rc3
^ permalink raw reply related [flat|nested] 3+ messages in thread* [U-Boot] [PATCH v3] ti_armv7_common.h: Fix U-Boot location on eMMC
2016-04-20 9:05 [U-Boot] [PATCH v3] ti_armv7_common.h: Fix U-Boot location on eMMC Sam Protsenko
@ 2016-04-20 14:38 ` Tom Rini
2016-04-26 0:16 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2016-04-20 14:38 UTC (permalink / raw)
To: u-boot
On Wed, Apr 20, 2016 at 12:05:59PM +0300, Sam Protsenko wrote:
> According to common eMMC partition table for Android boot (see
> PARTS_DEFAULT definition in include/configs/dra7xx_evm.h), "bootloader"
> partition (where u-boot.img is stored) starts at 256 KiB. Which is equal
> to 512 sectors (as 1 MMC sector size is 512 bytes).
>
> This patch fixes CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR constant so that
> it points to correct address of "bootloader" partition and SPL is able to
> read, parse and run u-boot.img correctly.
>
> This change was originally done as part of patch [1] in omapzoom u-boot.
> Without this patch, SPL fails to parse U-Boot header with next error:
>
> mkimage signature not found - ih_magic = 4814325a
>
> While at it, also fix U-Boot partition size, which is 384 KiB (as stated in
> include/configs/dra7xx_evm.h).
>
> [1] http://omapzoom.org/?p=repo/u-boot.git;a=commit;h=742b82d0c0aa0ed8096c2225a00e9f350212efa9
>
> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
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/20160420/2948c12d/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [U-Boot, v3] ti_armv7_common.h: Fix U-Boot location on eMMC
2016-04-20 9:05 [U-Boot] [PATCH v3] ti_armv7_common.h: Fix U-Boot location on eMMC Sam Protsenko
2016-04-20 14:38 ` Tom Rini
@ 2016-04-26 0:16 ` Tom Rini
1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2016-04-26 0:16 UTC (permalink / raw)
To: u-boot
On Wed, Apr 20, 2016 at 12:05:59PM +0300, Semen Protsenko wrote:
> According to common eMMC partition table for Android boot (see
> PARTS_DEFAULT definition in include/configs/dra7xx_evm.h), "bootloader"
> partition (where u-boot.img is stored) starts at 256 KiB. Which is equal
> to 512 sectors (as 1 MMC sector size is 512 bytes).
>
> This patch fixes CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR constant so that
> it points to correct address of "bootloader" partition and SPL is able to
> read, parse and run u-boot.img correctly.
>
> This change was originally done as part of patch [1] in omapzoom u-boot.
> Without this patch, SPL fails to parse U-Boot header with next error:
>
> mkimage signature not found - ih_magic = 4814325a
>
> While at it, also fix U-Boot partition size, which is 384 KiB (as stated in
> include/configs/dra7xx_evm.h).
>
> [1] http://omapzoom.org/?p=repo/u-boot.git;a=commit;h=742b82d0c0aa0ed8096c2225a00e9f350212efa9
>
> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
> Reviewed-by: Tom Rini <trini@konsulko.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/20160425/dc300098/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-04-26 0:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-20 9:05 [U-Boot] [PATCH v3] ti_armv7_common.h: Fix U-Boot location on eMMC Sam Protsenko
2016-04-20 14:38 ` Tom Rini
2016-04-26 0:16 ` [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