diff options
author | Klaus Heinrich Kiwi <klaus@linux.vnet.ibm.com> | 2021-03-31 10:23:13 -0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-04-06 22:37:23 +0100 |
commit | 900f8676e5d3919e6e668c0958eac870e866bdff (patch) | |
tree | a827c2427aa40bbf3068b00d726cb16cb5add690 /meta/lib/oeqa | |
parent | 3d13fc1f255e5a1729f9a0bfa37fb17750ba9f71 (diff) | |
download | poky-900f8676e5d3919e6e668c0958eac870e866bdff.tar.gz |
uboot: Fixes SPL verified boot on corner cases
* The kernel-fitimage class adds a do_assemble_fitimage_initramfs task
regardless of INITRAMFS_IMAGE_BUNDLE setting, which in some cases can
result in that task running after do_uboot_assemble_fitimage and
overwriting the u-boot-spl.dtb file with the pristine version (without
public key). Fix this by making do_uboot_assemble_fitimage dependant
on both do_assemble_fitimage_* tasks, regardless of the aforementioned
setting.
* Adjust 'type' and 'os' on the U-boot fitimage its script so that
mkimage/dumpimage can recognize them.
* Move the deployment of the u-boot-spl-nodtb files outside of
concat_spl_dtb_helper(), so that we can better isolate the scenarios
of creating an (unsigned) U-Boot fitimage versus also signing it. This
prevents some stale files from being deployed in the images directory.
* Remove any u-boot-fitImage and u-boot-its files from build tree, in
case the build tree is being reused across bitbake calls.
(From OE-Core rev: dc26d35e0935f30af55a3d2cb5c501d1b5c35437)
Signed-off-by: Klaus Heinrich Kiwi <klaus@linux.vnet.ibm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa')
-rw-r--r-- | meta/lib/oeqa/selftest/cases/fitimage.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/lib/oeqa/selftest/cases/fitimage.py b/meta/lib/oeqa/selftest/cases/fitimage.py index b911fded74..815ee48c05 100644 --- a/meta/lib/oeqa/selftest/cases/fitimage.py +++ b/meta/lib/oeqa/selftest/cases/fitimage.py | |||
@@ -293,7 +293,7 @@ FIT_SIGN_INDIVIDUAL = "1" | |||
293 | # different architectures. | 293 | # different architectures. |
294 | its_field_check = [ | 294 | its_field_check = [ |
295 | 'description = "A model description";', | 295 | 'description = "A model description";', |
296 | 'type = "uboot";', | 296 | 'type = "standalone";', |
297 | 'load = <0x80080000>;', | 297 | 'load = <0x80080000>;', |
298 | 'entry = <0x80080000>;', | 298 | 'entry = <0x80080000>;', |
299 | 'default = "conf";', | 299 | 'default = "conf";', |
@@ -376,7 +376,7 @@ UBOOT_MKIMAGE_SIGN_ARGS = "-c 'a smart U-Boot comment'" | |||
376 | # different architectures. | 376 | # different architectures. |
377 | its_field_check = [ | 377 | its_field_check = [ |
378 | 'description = "A model description";', | 378 | 'description = "A model description";', |
379 | 'type = "uboot";', | 379 | 'type = "standalone";', |
380 | 'load = <0x80080000>;', | 380 | 'load = <0x80080000>;', |
381 | 'entry = <0x80080000>;', | 381 | 'entry = <0x80080000>;', |
382 | 'default = "conf";', | 382 | 'default = "conf";', |