[PATCH v6 09/12] arm: mvebu: enable working default boot support

[Stefan Roese]

On 16.04.20 07:52, Joel Johnson wrote:
> On 2020-04-14 03:33, Stefan Roese wrote:
>> Hi Joel,
>>
>> On 23.03.20 21:21, Joel Johnson wrote:
>>> With the move to driver model usage, ensure that the required driver
>>> support for SPI and MMC booting is available in SPL.
>>>
>>> Tested on SolidRun ClearFog devices.
>>>
>>> Signed-off-by: Joel Johnson <mrjoel at lixil.net>
>>> Reviewed-by: Stefan Roese <sr at denx.de>
>>>
>>> ---
>>
>> This patch generates multiple warnings on other MVEBU platforms. I've
>> dropped it for now in my upcoming pull request. Please rework the
>> problematic patches (missing in the pull request) and re-send.
>>
>> Thanks,
>> Stefan
> 
> Sure, I'm looking into the changes and will have a new series with the 
> reworked patches in the next couple of days. The warnings really 
> highlighted additional cleanup on mvebu booting and defaults in general 
> which I'm taking a look at.

Thanks.

> I'm not too familiar with the multi-target building, but have read up on 
> and am currently using the buildman tool with arm and aarch64 toolsets, 
> and building the entire mvebu target set (21 boards currently). Is this 
> the best way to do the testing and building across multiple boards or is 
> there a different way you'd like the testing to be done in the end?

buildman is the correct tool for this. It can also automatically
download all necessary toolchains for other architectures.

> Somewhat offtopic, what's the best location to which wiki issues should 
> be reported? The page on Patches [1] still describes the old MAKEALL 
> script.

This is outdated.

> I hadn't found the script originally so skipped that step since 
> it wasn't possible. This time, in digging further in source history I 
> found the commit that removed it and referenced buildman. It would be 
> useful for new contributors to update the wiki with a pointer to 
> buildman if that's the desired replacement tool.

Could you please work on the wiki with the updated infos? That would
be helpful.

Thanks,
Stefan