[U-Boot] [PATCH 3/7] warp7: include: configs: Differentiate bootscript address from loadaddr

[Pierre-Jean Texier]

Hi Bryan,

Le 20/05/2019 à 10:33, Bryan O'Donoghue a écrit :
>
>
> On 15/05/2019 20:13, Pierre-Jean Texier wrote:
>> Regarding your question, it seems that is now the standard on many 
>> platforms [2].
>> In fact, instead of keeping a custom environment, it is better to use 
>> a more
>> generic approach by switching to disto config (with boot.scr, 
>> extlinux, and so on).
>>
>> For sure, we can customized the CONFIG_BOOTCOMMAND variable in patch.
>
> Right.
>
> This smells like a bigger change to me then, since we have to account 
> for how we transition current users from their u-boot environment to 
> distroboot.
>
> It sounds to me like a series of patches around this one topic.
>
> So, I think it is worthwhile exploring that set of changes but, I do 
> think we should do it separately to this set :)

Indeed, it seems you're right.
I'll send this later, means once your patches are integrated ;).
I am convinced that it's better to work on your series first.

Just a quick remark, can you share more informations
about the setup ? so I can test your series on my side ;)

Pierre-Jean

>
> ---
> bod