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.c...@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
