Delete tifs DT node as part of fixup. TISCI API reported msmc_size, does not include 64KB reserved size for tifs aka MSMC comms memory, see documentation[0].
As part of fixup, original code uses TISCI API reported msmc_size as size for SRAM DT node. tifs node is similar to l3-cache, which should hold address above msmc_size, and should be deleted before passing control to OS. [0] https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/general/core.html?highlight=msmc#tisci-msg-query-msmc Reviewed-by: Neha Malcom Francis <[email protected]> Signed-off-by: Udit Kumar <[email protected]> --- RFC was sent to k.org with https://lore.kernel.org/all/[email protected]/ So planning to go ahead with option 1. Changelog: Changes in v3: - Collected R/B of v2 - Updated commit message typo Changes since v2: - moved tifs check above l3 as v2: https://lore.kernel.org/all/[email protected]/ v1: https://lore.kernel.org/all/[email protected]/ arch/arm/mach-k3/common_fdt.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-k3/common_fdt.c b/arch/arm/mach-k3/common_fdt.c index 3bdedd7b50..4a01671156 100644 --- a/arch/arm/mach-k3/common_fdt.c +++ b/arch/arm/mach-k3/common_fdt.c @@ -65,6 +65,7 @@ static int fdt_fixup_msmc_ram(void *blob, char *parent_path, char *node_name) subnode, addr, size); if (addr + size > msmc_size || !strncmp(fdt_get_name(blob, subnode, &len), "sysfw", 5) || + !strncmp(fdt_get_name(blob, subnode, &len), "tifs", 4) || !strncmp(fdt_get_name(blob, subnode, &len), "l3cache", 7)) { fdt_del_node(blob, subnode); debug("%s: deleting subnode %d\n", __func__, subnode); -- 2.34.1
