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?h...
Reviewed-by: Neha Malcom Francis n-francis@ti.com Signed-off-by: Udit Kumar u-kumar1@ti.com --- RFC was sent to k.org with https://lore.kernel.org/all/20230506100826.1525641-1-u-kumar1@ti.com/ 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/20230420081128.3617214-1-u-kumar1@ti.com/ v1: https://lore.kernel.org/all/20230419061352.3156023-1-u-kumar1@ti.com/
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);