[PATCH 10/10] arm: mvebu: clearfog: don't assume MMC booting

Stefan Roese sr at denx.de
Mon Jan 13 12:42:24 CET 2020


Hi Baruch,

On 13.01.20 12:40, Baruch Siach wrote:
> Hi Stefan,
> 
> On Mon, Jan 13 2020, Stefan Roese wrote:
>> On 12.01.20 17:34, Baruch Siach wrote:
>>> On Sun, Jan 12 2020, Joel Johnson wrote:
>>>> On 2020-01-12 03:49, Baruch Siach wrote:
>>>>> On Sat, Jan 11 2020, Joel Johnson wrote:
>>>>>> +config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
>>>>>> +	hex "Address on the MMC to load U-Boot from"
>>>>>> +	default 0x141
>>>>>> +	depends on MVEBU_SPL_BOOT_DEVICE_MMC
>>>>>
>>>>> You might want to take a look at the series linked below, which
>>>>> conflicts with this patch:
>>>>>
>>>>>     https://patchwork.ozlabs.org/project/uboot/list/?series=121894
>>>>
>>>> Indeed, the dynamic runtime approach seems much nicer for handling the MMCSD
>>>> offset business, including relative vs. absolute device offset. I only have
>>>> non-eMMC units so can't test it, but I'm all for dropping the
>>>> SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR portion of this patch in favor of that patch,
>>>> in which leaving the defconfig entry does no harm. Any indication of when that
>>>> patch will be included? It seems to have been stagnant for a while.
>>>
>>> Indeed. I'm waiting patiently.
>>
>> I have missed addressing this patch series for quite some time, sorry.
>> Could you please rebase this patchset on top of current mainline as it
>> doesn't apply any more. And please do a compile run for all targets as
>> well (Travis). I'll try to push it upstream in this merge window.
> 
> Thanks. I'll work on that tomorrow.

Thanks.
  
> While you are at it can you also push this fix:
> 
>    https://patchwork.ozlabs.org/patch/1205597/
> 
> Clearfog can't boot v2020.01 clearfog_defconfig without it :-(.

Its on my list. I'll pull it with the other patch with my next pull
request.

Thanks,
Stefan


More information about the U-Boot mailing list