[PATCH] configs: j7200: correct mmc offset

Nishanth Menon nm at ti.com
Wed May 3 16:45:40 CEST 2023


On 10:58-20230503, Udit Kumar wrote:
> This patch corrects the MMC raw mode sector offset as
> per documentation.
> 
> https://software-dl.ti.com/jacinto7/esd/processor-sdk-linux-j7200/08_06_00_11/exports/docs/j7200/linux/Foundational_Components/U-Boot/UG-Memory.html
> 
> Section: eMMC layout

Please drop these TI specific documentation. Instead ADD documentation
into u-boot source explaining these eMMC layout.

NOTE: emmc raw mode for boot partition is one way of using SPL. it is
also very restrictive. u-boot and TIFS, OPTEE all packed into EMMC boot
partition has limits to which it can scale.

On beagle family for example, we choose this is tooo constraining and
chose to go down a different path where the uda partition in fs mode is
used for u-boot, tfa, optee etc.. allowing more maintainable system
instead of having to constantly refactoring the image offsets and sizes
involved.
> 
> Without this correct offset eMMC boot with UDA partition will fail.
Where this fails is looking for the next segment of u-boot image in boot
partition, not UDA.

> 
> Fixes: f8c1e893c82 (configs: j7200_evm_a72: Add Initial suppot)
> Fixes: 02dff65efe7 (configs: j7200_evm_r5: Add initial support)
> Fixes: 360c7f46f39 (configs: Add configs for J7200 High Security EVM)
> 
> Way to test with eMMC UDA partition
> 
> => mmc dev 0 0
> => fatload mmc 1 ${loadaddr} tiboot3.bin
> => mmc write ${loadaddr} 0x0 0x800
> => fatload mmc 1 ${loadaddr} tispl.bin
> => mmc write ${loadaddr} 0x800 0x1000
> => fatload mmc 1 ${loadaddr} u-boot.img
> => mmc write ${loadaddr} 0x1800 0x2000
> => mmc partconf 0 1 7 1
> => mmc bootbus 0 2 0 0
> 
> Cc: Bhavya Kapoor <b-kapoor at ti.com>
> Cc: Diwakar Dhyani <d-dhyani at ti.com>
> Cc: KEERTHY <j-keerthy at ti.com>
> Signed-off-by: Udit Kumar <u-kumar1 at ti.com>
> ---
>  configs/j7200_evm_a72_defconfig    | 2 +-
>  configs/j7200_evm_r5_defconfig     | 2 +-
>  configs/j7200_hs_evm_a72_defconfig | 2 +-
>  configs/j7200_hs_evm_r5_defconfig  | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/configs/j7200_evm_a72_defconfig b/configs/j7200_evm_a72_defconfig
> index fa5ea2aecd..8a810f8f48 100644
> --- a/configs/j7200_evm_a72_defconfig
> +++ b/configs/j7200_evm_a72_defconfig
> @@ -46,7 +46,7 @@ CONFIG_SPL_STACK_R=y
>  CONFIG_SYS_SPL_MALLOC=y
>  CONFIG_SYS_SPL_MALLOC_SIZE=0x800000
>  CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y
> -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x1400
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x1800
>  CONFIG_SPL_DMA=y
>  CONFIG_SPL_ENV_SUPPORT=y
>  CONFIG_SPL_FS_LOAD_PAYLOAD_NAME="u-boot.img"
> diff --git a/configs/j7200_evm_r5_defconfig b/configs/j7200_evm_r5_defconfig
> index 00ec48b83b..908cfaf402 100644
> --- a/configs/j7200_evm_r5_defconfig
> +++ b/configs/j7200_evm_r5_defconfig
> @@ -43,7 +43,7 @@ CONFIG_CUSTOM_SYS_SPL_MALLOC_ADDR=0x84000000
>  CONFIG_SYS_SPL_MALLOC_SIZE=0x1000000
>  CONFIG_SPL_EARLY_BSS=y
>  CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y
> -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x400
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x800
>  CONFIG_SPL_DMA=y
>  CONFIG_SPL_ENV_SUPPORT=y
>  CONFIG_SPL_FS_EXT4=y
> diff --git a/configs/j7200_hs_evm_a72_defconfig b/configs/j7200_hs_evm_a72_defconfig
> index d9560727ed..c234a58a7a 100644
> --- a/configs/j7200_hs_evm_a72_defconfig
> +++ b/configs/j7200_hs_evm_a72_defconfig
> @@ -47,7 +47,7 @@ CONFIG_SPL_STACK_R=y
>  CONFIG_SYS_SPL_MALLOC=y
>  CONFIG_SYS_SPL_MALLOC_SIZE=0x800000
>  CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y
> -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x1400
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x1800
>  CONFIG_SPL_DMA=y
>  CONFIG_SPL_ENV_SUPPORT=y
>  CONFIG_SPL_FS_LOAD_PAYLOAD_NAME="u-boot.img"
> diff --git a/configs/j7200_hs_evm_r5_defconfig b/configs/j7200_hs_evm_r5_defconfig
> index 94a6523f06..74015db1af 100644
> --- a/configs/j7200_hs_evm_r5_defconfig
> +++ b/configs/j7200_hs_evm_r5_defconfig
> @@ -43,7 +43,7 @@ CONFIG_CUSTOM_SYS_SPL_MALLOC_ADDR=0x84000000
>  CONFIG_SYS_SPL_MALLOC_SIZE=0x1000000
>  CONFIG_SPL_EARLY_BSS=y
>  CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y
> -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x400
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x800
>  CONFIG_SPL_DMA=y
>  CONFIG_SPL_ENV_SUPPORT=y
>  CONFIG_SPL_FS_EXT4=y
> -- 
> 2.34.1
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list