[PATCH v1 3/5] arm: dts: k3-am6xx: Fix FIT image memory overlap in binman configurations

[Neha Malcom Francis]

On 29/07/25 18:18, Aristo Chen wrote:
> Fix memory overlaps in FIT image configurations for TI AM62x and AM64x
> PHYCore and SK boards.
> 
> The overlaps occurred in two categories:
> 
> 1. TI firmware stub images (tifsstub-hs, tifsstub-fs, tifsstub-gp):
>    These mutually exclusive firmware variants were incorrectly assigned
>    the same load address within FIT configurations, causing overlap

I think this will break TIFS stub functionality which uses the "os"
property to figure out the right stub to pick from the FIT. See [0]

While I agree that ideally FIT images have to have non-overlapping
addresses, we may have to find a solution for such use-cases prior to
adopting this series/rule.

Require Dhruva to comment further on this TIFS stub load address change.

>    detection to fail. Adjust addresses with 64KB spacing:
>    - tifsstub-hs: Keep original address
>    - tifsstub-fs: Move to +64KB offset
>    - tifsstub-gp: Move to +128KB offset
> 
> 2. Device tree overlay images (som-no-rtc, som-no-spi, som-no-eth):
>    These overlay files had insufficient spacing between load addresses,
>    causing actual memory overlaps. Increase spacing to 8KB boundaries
>    to accommodate overlay sizes safely.
> 
> An upcoming commit will validate if the memory region is overlapped
> 
> Signed-off-by: Aristo Chen <aristo.chen at canonical.com>
> ---
>  arch/arm/dts/k3-am625-phycore-som-binman.dtsi | 22 +++++++++----------
>  arch/arm/dts/k3-am625-sk-binman.dtsi          | 16 +++++++-------
>  .../dts/k3-am625-verdin-wifi-dev-binman.dtsi  | 16 +++++++-------
>  arch/arm/dts/k3-am62a-phycore-som-binman.dtsi | 22 +++++++++----------
>  arch/arm/dts/k3-am62a-sk-binman.dtsi          | 16 +++++++-------
>  arch/arm/dts/k3-am642-phycore-som-binman.dtsi |  6 ++---
>  6 files changed, 49 insertions(+), 49 deletions(-)
> 
> diff --git a/arch/arm/dts/k3-am625-phycore-som-binman.dtsi b/arch/arm/dts/k3-am625-phycore-som-binman.dtsi
> index a9bd5a2be84..4344cefeba3 100644
> --- a/arch/arm/dts/k3-am625-phycore-som-binman.dtsi
> +++ b/arch/arm/dts/k3-am625-phycore-som-binman.dtsi
> @@ -234,8 +234,8 @@
>  					arch = "arm32";
>  					compression = "none";
>  					os = "tifsstub-fs";
> -					load = <0x9dc00000>;
> -					entry = <0x9dc00000>;
> +					load = <0x9dc10000>;
> +					entry = <0x9dc10000>;
>  					blob-ext {

[...]

[0]
https://elixir.bootlin.com/u-boot/v2025.07/source/arch/arm/mach-k3/r5/common.c#L349

-- 
Thanking You
Neha Malcom Francis