On 27/01/2023 21:37, Tom Rini wrote:
On Fri, Jan 27, 2023 at 04:51:41PM +0100, Safae Ouajih wrote:
On 27/01/2023 01:55, Simon Glass wrote:
Hi Safae,
On Thu, 26 Jan 2023 at 09:05, Safae Ouajih <soua...@baylibre.com> wrote:
test_abootimg is extended to include the testing of boot images
version 4. For this, boot.img and vendor_boot.img have been
generated using mkbootimg tool with setting the header
version to 4.
This tests:
- Getting the header version using abootimg
- Extracting the load address of the dtb
- Extracting the dtb start address in RAM
Running test:
$ ./test/py/test.py --bd sandbox --build -k test_abootimg
Signed-off-by: Safae Ouajih <soua...@baylibre.com>
---
test/py/tests/test_android/test_abootimg.py | 135 ++++++++++++++++++--
1 file changed, 122 insertions(+), 13 deletions(-)
Reviewed-by: Simon Glass <s...@chromium.org>
Question below
diff --git a/test/py/tests/test_android/test_abootimg.py
b/test/py/tests/test_android/test_abootimg.py
index 43a7099c46..a5c734b9c5 100644
--- a/test/py/tests/test_android/test_abootimg.py
+++ b/test/py/tests/test_android/test_abootimg.py
@@ -32,6 +32,23 @@ Now one can obtain original boot.img from this hex dump like
this:
$ xxd -r -p boot.img.gz.hex boot.img.gz
$ gunzip -9 boot.img.gz
+
+For boot image header version 4, these tests rely on two images that are
generated
+using the same steps above :
+
+1- boot.img :
+ $ mkbootimg --kernel ./kernel --ramdisk ./ramdisk.img \
+ --cmdline "cmdline test" --dtb ./dtb.img \
+ --os_version R --os_patch_level 2019-06-05 \
+ --header_version 4 --output ./boot.img
+
+2- vendor_boot.img
+ $ mkbootimg --kernel ./kernel --ramdisk ./ramdisk.img \
+ --cmdline "cmdline test" --dtb ./dtb.img \
+ --os_version R --os_patch_level 2019-06-05 \
+ --pagesize 4096 --vendor_ramdisk ./ramdisk.img \
+ --header_version 4 --vendor_boot ./vboot.img \
Is it possible for us to run these commands in the test, to avoid
hassle when we want to change the disk image? See test_ut.py
Hi Simon,
I think that it is possible to run these commands in the test. However,
this requires adding "mkbootimg" android tool with the last updates
Please update tools/docker/Dockerfile to include this tool then, so we
can run it in CI (or locally using the images).
Hello Tom,
I will add a patch to add that tool in a v3.
Thank you,
Best regards,
Safae
