[PATCH v1 1/5] binman: Fix FIT image overlap issues

[Neha Malcom Francis]

Hi Aristo,

On 29/07/25 18:18, Aristo Chen wrote:
> Fix three binman tests that has memory region overlap issue, the test
> cases needed to be updated to use non-overlapping memory layouts.
> 
> * Tests fixed:
>   - testFitFirmwareLoadables
>   - testFitSignSimple
>   - testFitSignNoSingatureNodes

nitpick: s/Singature/Signature
> 
> * Changes made:
>   Updated DTB test files to change U-Boot load addresses from 0x0 to
>   0x2000 to avoid overlapping with ATF in the 0x10-0xfc range:
>   - 276_fit_firmware_loadables.dts
>   - 340_fit_signature.dts
>   - 342_fit_signature.dts
> 
> An upcoming commit will validate if the memory region is overlapped
> 
> Signed-off-by: Aristo Chen <aristo.chen at canonical.com>
> ---
>  tools/binman/test/276_fit_firmware_loadables.dts | 4 ++--
>  tools/binman/test/340_fit_signature.dts          | 4 ++--
>  tools/binman/test/342_fit_signature.dts          | 4 ++--
>  3 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/tools/binman/test/276_fit_firmware_loadables.dts b/tools/binman/test/276_fit_firmware_loadables.dts
> index 2f79cdc9bb8..d344036a11a 100644
> --- a/tools/binman/test/276_fit_firmware_loadables.dts
> +++ b/tools/binman/test/276_fit_firmware_loadables.dts
> @@ -19,8 +19,8 @@
>  					arch = "arm64";
>  					os = "u-boot";
>  					compression = "none";
> -					load = <0x00000000>;
> -					entry = <0x00000000>;
> +					load = <0x00002000>;
> +					entry = <0x00002000>;
>  
>  					u-boot-nodtb {
>  					};
> diff --git a/tools/binman/test/340_fit_signature.dts b/tools/binman/test/340_fit_signature.dts
> index 9dce62e52de..1c25d52cba4 100644
> --- a/tools/binman/test/340_fit_signature.dts
> +++ b/tools/binman/test/340_fit_signature.dts
> @@ -20,8 +20,8 @@
>  					arch = "arm64";
>  					os = "u-boot";
>  					compression = "none";
> -					load = <0x00000000>;
> -					entry = <0x00000000>;
> +					load = <0x00002000>;
> +					entry = <0x00002000>;
>  
>  					u-boot-nodtb {
>  					};
> diff --git a/tools/binman/test/342_fit_signature.dts b/tools/binman/test/342_fit_signature.dts
> index 267105d0f68..2ac600b1c70 100644
> --- a/tools/binman/test/342_fit_signature.dts
> +++ b/tools/binman/test/342_fit_signature.dts
> @@ -20,8 +20,8 @@
>  					arch = "arm64";
>  					os = "u-boot";
>  					compression = "none";
> -					load = <0x00000000>;
> -					entry = <0x00000000>;
> +					load = <0x00002000>;
> +					entry = <0x00002000>;
>  
>  					u-boot-nodtb {
>  					};

Reviewed-by: Neha Malcom Francis <n-francis at ti.com>

-- 
Thanking You
Neha Malcom Francis