[U-Boot] [RFC PATCH 2/2] sunxi: add "fel" boot target

Hans de Goede
2015-09-14T11:46:57Z

Hi,

On 14-09-15 13:42, Hans de Goede wrote:

>> Supporting both boot.scr and uEnv.txt for FEL boot seems to be
>> reasonably simple to me. You can even do it in a single patch
>> series. As Hans suggests, please take care of the boot.scr case
>> first. Then maybe introduce uEnv.txt support with an additional
>> patch.
>
> I'm still not convinced that support uEnv.txt is necessary at all,
> but I agree that if we this having this done through extra fields,
> rather then through a union would be better.

p.s.

As for the version byte vs minor/major scheme discussion. Why not
simply declare that we will always be backwards compatible, and
only ever add new fields ? Then we can simply add version checks
around those new fields, and have both back and forward compatibility.

In this case the single byte should suffice nicely.

Regards,

Hans