[PATCH v2 2/2] cmd: mvebu/bubt: move eMMC data-partition uboot from LBA-0 to 4096

[Stefan Roese]

On 10/26/23 09:10, Stefan Roese wrote:
> On 10/25/23 10:22, Josua Mayer wrote:
>> A38x bootrom only searches 2 sectors when booting from eMMC,
>> irregardless of data or boot partition: 0 & 4096.
>>
>> For eMMC boot partitions sector 0 is fine, but on data partition it
>> conflicts with MBR.
>>
>> Change bubt command default to 4096 for eMMC data partition only, to
>> allow using an MBR partition table on the eMMC data partition while also
>> booting from it.
>>
>> Signed-off-by: Josua Mayer <josua at solid-run.com>
> 
> Reviewed-by: Stefan Roese <sr at denx.de>

Applied to u-boot-marvell/master

Thanks,
Stefan

> Thanks,
> Stefan
> 
>> ---
>> V1 -> V2: fixed build without CONFIG_SUPPORT_EMMC_BOOT
>>
>>   cmd/mvebu/bubt.c | 11 +++++++++--
>>   1 file changed, 9 insertions(+), 2 deletions(-)
>>
>> diff --git a/cmd/mvebu/bubt.c b/cmd/mvebu/bubt.c
>> index ca24a5c1c4b..744b1c20aa8 100644
>> --- a/cmd/mvebu/bubt.c
>> +++ b/cmd/mvebu/bubt.c
>> @@ -240,9 +240,16 @@ static int mmc_burn_image(size_t image_size)
>>   #endif
>>       /* SD reserves LBA-0 for MBR and boots from LBA-1,
>> -     * MMC/eMMC boots from LBA-0
>> +     * MMC/eMMC boots from LBA-0 and LBA-4096
>>        */
>> -    start_lba = IS_SD(mmc) ? 1 : 0;
>> +    if (IS_SD(mmc))
>> +        start_lba = 1;
>> +#ifdef CONFIG_SUPPORT_EMMC_BOOT
>> +    else if (part)
>> +        start_lba = 0;
>> +#endif
>> +    else
>> +        start_lba = 4096;
>>   #ifdef CONFIG_BLK
>>       blk_count = image_size / mmc->write_bl_len;
>>       if (image_size % mmc->write_bl_len)
> 
> Viele Grüße,
> Stefan Roese
> 

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de