u-boot.git/tools, branch v2021.10

Revert "mkeficapsule: Remove dtb related options"

2021-09-18T09:46:56+00:00

This reverts commit f86caab058ff062ce72b24cd1ab9ec1253cc1352.

Signed-off-by: Simon Glass

tools: k3_fit_atf: Fix DM binary FIT load addresses

2021-09-11T00:10:20+00:00

The DM binary runs on the MCU R5F Core0 after R5 SPL on J721E and J7200
SoCs. The binary is built alongside the TFA, OPTEE and A72 SPL binaries
and included in the tispl.bin FIT image. The R5 SPL loads the DM binary
at 0xA0000000 address, based on the value used in the FIT image build
script. The DM binary though is an ELF image and not a regular binary
file, and so is processed further to load the actual program segments
using the U-Boot's standard ELF loader library.

The DM binary does leverage a certain portion of DDR for its program
segments, and typically reserves 16 MB of DDR at 0xA0000000 with the
1st MB used for IPC between Linux and the remote processor, and
remaining memory for firmware segments. This can cause an incomplete
loading of the program segments if the DM binary is larger than 1 MB,
due to overlap of the initial loaded binary and the actual program
segments.

Fix this by using the address 0x89000000, which matches the current
"addr_mcur5f0_0load" env variable used by R5 SPL before the DM firmware
inclusion into the tispl.bin.

Fixes: df5363a67f35 ("tools: k3_fit_atf: add DM binary to the FIT image")
Signed-off-by: Suman Anna

tools: kwbimage: Remove comment about unimplemented register headers in v1 images

2021-09-01T06:09:24+00:00

Support for register headers in v1 images was implemented in commit
02ba70ad6822 ("tools: kwbimage: Add support for DATA command also for v1
images"). So remove old comment.

Signed-off-by: Pali Rohár 
Fixes: 02ba70ad6822 ("tools: kwbimage: Add support for DATA command also for v1 images")
Reviewed-by: Stefan Roese

kwbimage: check return value of image_get_csk_index

2021-09-01T06:09:24+00:00

image_get_csk_index() may return -1 in case of an error. Don't use this
value as index.

This resolves Coverity CID 338488
Memory - illegal accesses  (NEGATIVE_RETURNS)

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Stefan Roese 
Reviewed-by: Pali Rohár

kwbimage: check fopen() return value

2021-09-01T06:09:24+00:00

Always check the return value of fopen().

This resolves Coverity CID 338491:
Null pointer dereferences (NULL_RETURNS)

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Stefan Roese 
Reviewed-by: Pali Rohár

tools: kwbimage: Use IBR_HDR_* constants instead of raw numbers

2021-09-01T06:07:04+00:00

There are already IBR_HDR_* constants for these numbers, so use them.

Signed-off-by: Pali Rohár 
Reviewed-by: Stefan Roese

tools: kwbimage: Verify size of image data

2021-09-01T06:07:04+00:00

Part of image data is 4 byte checksum, so every image must contain at least
4 bytes. Verify it to prevent memory corruptions.

Signed-off-by: Pali Rohár 
Reviewed-by: Stefan Roese

tools: kwbimage: Verify size of v0 image header

2021-09-01T06:07:04+00:00

Check that extended image header size is not larger than file size.

Signed-off-by: Pali Rohár 
Reviewed-by: Stefan Roese

tools: kwbimage: Verify supported image version

2021-09-01T06:07:04+00:00

Only image versions 0 and 1 are supported. Verify it in
kwbimage_verify_header() function.

Signed-off-by: Pali Rohár 
Reviewed-by: Stefan Roese

dtoc: Correct the intarray-widening test case

2021-08-08T17:27:27+00:00

This case was intended to check that widening an int array with an int
does nothing. Fix it.

Reported-by: Walter Lozano 
Signed-off-by: Simon Glass 
Reviewed-by: Walter Lozano