[PATCH v7 14/16] test: py: vboot: add test for global image signature

[Philippe REYNES]

Hi Tom,


Le 25/03/2022 à 18:11, Tom Rini a écrit :
> On Mon, Mar 14, 2022 at 03:57:43PM +0100, Philippe Reynes wrote:
>
>> Adds test units for the pre-load header signature.
>>
>> Signed-off-by: Philippe Reynes <philippe.reynes at softathome.com>
>> ---
>>   test/py/tests/test_vboot.py                   | 145 ++++++++++++++++--
>>   test/py/tests/vboot/sandbox-binman-pss.dts    |  25 +++
>>   test/py/tests/vboot/sandbox-binman.dts        |  24 +++
>>   .../tests/vboot/sandbox-u-boot-global-pss.dts |  28 ++++
>>   test/py/tests/vboot/sandbox-u-boot-global.dts |  27 ++++
>>   test/py/tests/vboot/simple-images.its         |  36 +++++
>>   6 files changed, 269 insertions(+), 16 deletions(-)
>>   create mode 100644 test/py/tests/vboot/sandbox-binman-pss.dts
>>   create mode 100644 test/py/tests/vboot/sandbox-binman.dts
>>   create mode 100644 test/py/tests/vboot/sandbox-u-boot-global-pss.dts
>>   create mode 100644 test/py/tests/vboot/sandbox-u-boot-global.dts
>>   create mode 100644 test/py/tests/vboot/simple-images.its
> This leads to CI failures such as:
> https://source.denx.de/u-boot/u-boot/-/jobs/412046
> and I guess maybe we need something new in test/py/requirements.txt
> maybe?

I think that this issue should be fixed with this patch :

https://patchwork.ozlabs.org/project/uboot/patch/20220127140314.10264-2-philippe.reynes@softathome.com/


> The rest of the series seems fine, btw.
>
great, thanks a lot


Regards,

Philippe