[U-Boot] [PATCH v2 2/3] arm: mvebu: clearfog: set uboot image SD card offset

[Stefan Roese]

Hi Baruch,

On 06.08.19 08:20, Stefan Roese wrote:
> On 29.07.19 09:12, Baruch Siach wrote:
>> Armada 38x ROM skips the first SD card offset when loading SPL. This
>> affects the location of the main U-Boot image. SPL MMC code now supports
>> U-Boot image offset based on run-time detection of the boot partition.
>> Use this feature to make the same generated image support both SD card
>> and eMMC boot partition.
>>
>> Signed-off-by: Baruch Siach <baruch at tkos.co.il>
>> ---
>>    configs/clearfog_defconfig | 2 +-
>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configs/clearfog_defconfig b/configs/clearfog_defconfig
>> index b7b886be4f7a..3609445c1e95 100644
>> --- a/configs/clearfog_defconfig
>> +++ b/configs/clearfog_defconfig
>> @@ -22,7 +22,7 @@ CONFIG_SYS_CONSOLE_INFO_QUIET=y
>>    # CONFIG_DISPLAY_BOARDINFO is not set
>>    CONFIG_DISPLAY_BOARDINFO_LATE=y
>>    CONFIG_SPL_TEXT_BASE=0x40000030
>> -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x141
>> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_DATA_PART_OFFSET=0x1
> 
> While looking at this patch, shouldn't this new Kconfig symbol
> include "SECTOR" for better transparency? And why "DATA"?
> Perhaps use CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR_OFFSET?

Any updates on this?

Thanks,
Stefan