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

[Stefan Roese]

Hi Baruch,

On 14.01.20 13:55, Baruch Siach wrote:
> Hi Stefan,
> 
> On Mon, Jan 13, 2020 at 07:48:34AM +0100, 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.
> 
> The original patch series applies cleanly on top of current master as of
> commit 88366b96ee0d86 ("travis: Switch to QEMU v4.2.0 globally"), and runs
> successfully. It only trivially conflicts with the clearfog_defconfig
> SPL_DM_GPIO fix (http://patchwork.ozlabs.org/patch/1205597/). Would you like
> me to rebase on to of that?

Yes, please.
  
>> And please do a compile run for all targets as well (Travis). I'll try to
>> push it upstream in this merge window.
> 
> buildman for arm and aarch64 is currently running.

All platforms please. I suggest to use Travis for this. Be prepared that
this takes a few hours though.

Thanks,
Stefan