[PATCH v7 06/12] binman: Support new op-tee binary format

[Jerome Forissier]

On 12/22/22 00:38, Simon Glass wrote:
> Hi Jerome,
> 
> On Wed, 21 Dec 2022 at 07:39, Jerome Forissier
> <jerome.forissier at linaro.org> wrote:
>>
>> Hi Simon,
>>
>> On 12/17/22 22:28, Simon Glass wrote:
>>> OP-TEE has a format with a binary header that can be used instead of the
>>> ELF file. With newer versions of OP-TEE this may be required on some
>>> platforms.
>>>
>>> Add support for this in binman. First, add a method to obtain the ELF
>>> sections from an entry, then use that in the FIT support. We then end up
>>> with the ability to support both types of OP-TEE files, depending on which
>>> one is passed in with the entry argument (TEE=xxx in the U-Boot build).
>>>
>>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>> ---
>>>
>>> Changes in v7:
>>> - Correct missing test coverage
>>>
>>> Changes in v6:
>>> - Update op-tee to support new v1 binary header
>>
>> Thanks for the update. I'm trying to test this. How am I supposed to
>> pass the BL31 and TEE binaries? Currently I build u-boot with:
>>
>>   BL31=/path/to/bl31.elf TEE=/path/to/tee.bin make -C u-boot CROSS_COMPILE=aarch64-linux-gnu- CC=aarch64-linux-gnu-gcc HOSTCC=gcc
>>
>> I applied the series onto the 'next' branch and built as usual but the board
>> failed to boot so I suspect I am missing something.
> 
> Thanks for testing.
> 
> I seem to be missing setting the U-Boot load address. Also TEE is
> supposed to be optional, I believe.
> 
> I pushed an updated series to u-boot-dm/fit-working so could you try
> that (v8)? Also could you please include your board and console output
> if/when it fails?

OK, I will reply in the v8 thread to avoid ambiguities.


Thanks,
-- 
Jerome