[PATCH v2 0/9] spl: atf: add support for LOAD_IMAGE_V2

Michael Walle michael at walle.cc
Fri Nov 20 14:48:47 CET 2020


Am 2020-11-20 14:35, schrieb Michal Simek:
> On 20. 11. 20 14:25, Michael Walle wrote:
>> Am 2020-11-20 14:16, schrieb Michal Simek:
>>> On 20. 11. 20 12:27, Michael Walle wrote:
>>>> Am 2020-11-20 12:15, schrieb Michal Simek:
[..]
>>>>> Please get this reviewed by people who are using current blX code.
>>>> 
>>>> What do you mean by blX code? Nobody is using this flow for now, 
>>>> i.e.
>>>>   spl -> bl31 -> u-boot
>>> 
>>> As I said I am not quite sure about it. I see that rockchip guys are
>>> using ATF and they also call that code in ATF. They should know which
>>> version the use.
>> 
>> NXP imx8 also uses u-boot -> bl31 as far as I know.
>> 
>>> Definitely check with them to ack your patches.
>> 
>> I've put Kever (who did the initial patches for rockchip) on CC for 
>> this.
>> But again they are using the old method. So lets assume they will 
>> ignore
>> this series (for whatever reason), won't it be accepted then? I mean
>> this series doesn't touch the old behavior, just adding a new one.
> 
> up to Priyanka or Tom.
> I have no problem with your patches because we are not using this 
> method
> and your patches doesn't break our SPL boot flow.

I haven't said this: I really appreciated your review and I was 
surprised
how fast it was. Thanks and maybe Kever will also have a look. We'll see 
;)

-michael


More information about the U-Boot mailing list