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.c...@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-fran...@ti.com>
--
Thanking You
Neha Malcom Francis
