[PATCH] arm64: configs: Remove obsolete TI config

Aashvij Shenai a-shenai at ti.com
Mon Jan 20 09:50:42 CET 2025


This config is causing conflicts with how fdtfile variable is
initialized.
For K3 devices, CONFIG_DEFAULT_DEVICE_TREE= "ti/k3-<board>.dtb".
With CONFIG_TI_FDT_FOLDER_PATH also prefixing "ti", fdtfile is then
"ti/ti/k3-<board>.dtb". This variable is updated when fitImage is
booted and fails to boot due to the parsing error "ti/ti/".

Given that there are no other users of this config other than K3 for
now, it is being removed.

Since am64x, j721e and j721s2 also define a DEFAULT_FDT_FILE, update
them to conform to the DEFAULT_DEVICE_TREE standard.

Signed-off-by: Aashvij Shenai <a-shenai at ti.com>
---

 board/beagle/beagleboneai64/beagleboneai64.c |  3 +--
 board/beagle/beagleplay/beagleplay.c         |  3 +--
 board/ti/am64x/evm.c                         |  4 ++--
 board/ti/common/Kconfig                      | 11 -----------
 board/ti/common/fdt_ops.c                    |  9 +++------
 board/ti/j721e/evm.c                         |  6 +++---
 board/ti/j721s2/evm.c                        |  4 ++--
 7 files changed, 12 insertions(+), 28 deletions(-)

diff --git a/board/beagle/beagleboneai64/beagleboneai64.c b/board/beagle/beagleboneai64/beagleboneai64.c
index 1e43d1c1bd2..e8d07f1f95f 100644
--- a/board/beagle/beagleboneai64/beagleboneai64.c
+++ b/board/beagle/beagleboneai64/beagleboneai64.c
@@ -73,8 +73,7 @@ int board_late_init(void)
 {
 	char fdtfile[50];
 
-	snprintf(fdtfile, sizeof(fdtfile), "%s/%s.dtb",
-		 CONFIG_TI_FDT_FOLDER_PATH, CONFIG_DEFAULT_DEVICE_TREE);
+	snprintf(fdtfile, sizeof(fdtfile), "%s.dtb", CONFIG_DEFAULT_DEVICE_TREE);
 
 	env_set("fdtfile", fdtfile);
 
diff --git a/board/beagle/beagleplay/beagleplay.c b/board/beagle/beagleplay/beagleplay.c
index a21f09e3122..fae69b37585 100644
--- a/board/beagle/beagleplay/beagleplay.c
+++ b/board/beagle/beagleplay/beagleplay.c
@@ -69,8 +69,7 @@ int board_late_init(void)
 {
 	char fdtfile[50];
 
-	snprintf(fdtfile, sizeof(fdtfile), "%s/%s.dtb",
-		 CONFIG_TI_FDT_FOLDER_PATH, CONFIG_DEFAULT_DEVICE_TREE);
+	snprintf(fdtfile, sizeof(fdtfile), "%s.dtb", CONFIG_DEFAULT_DEVICE_TREE);
 
 	env_set("fdtfile", fdtfile);
 
diff --git a/board/ti/am64x/evm.c b/board/ti/am64x/evm.c
index 00b8317d6bf..c0033db2078 100644
--- a/board/ti/am64x/evm.c
+++ b/board/ti/am64x/evm.c
@@ -217,8 +217,8 @@ int checkboard(void)
 
 #ifdef CONFIG_BOARD_LATE_INIT
 static struct ti_fdt_map ti_am64_evm_fdt_map[] = {
-	{"am64x_gpevm", "k3-am642-evm.dtb"},
-	{"am64x_skevm", "k3-am642-sk.dtb"},
+	{"am64x_gpevm", "ti/k3-am642-evm.dtb"},
+	{"am64x_skevm", "ti/k3-am642-sk.dtb"},
 	{ /* Sentinel. */ }
 };
 
diff --git a/board/ti/common/Kconfig b/board/ti/common/Kconfig
index de44e4de211..b8b691aa06c 100644
--- a/board/ti/common/Kconfig
+++ b/board/ti/common/Kconfig
@@ -50,14 +50,3 @@ config TI_COMMON_CMD_OPTIONS
 	imply CMD_TIME
 	imply CMD_USB if USB
 
-config TI_FDT_FOLDER_PATH
-	string "Location of Folder path where dtb is present"
-	default "ti/davinci" if ARCH_DAVINCI
-	default "ti/keystone" if ARCH_KEYSTONE
-	default "ti/omap" if ARCH_OMAP2PLUS
-	default "ti" if ARCH_K3
-	depends on ARCH_DAVINCI || ARCH_KEYSTONE || ARCH_OMAP2PLUS || ARCH_K3
-	help
-	   Folder path for kernel device tree default.
-	   This is used along with fdtfile path to locate the kernel
-	   device tree blob.
diff --git a/board/ti/common/fdt_ops.c b/board/ti/common/fdt_ops.c
index 8a3300993ed..47df5726ff0 100644
--- a/board/ti/common/fdt_ops.c
+++ b/board/ti/common/fdt_ops.c
@@ -35,17 +35,14 @@ void ti_set_fdt_env(const char *board_name, struct ti_fdt_map *fdt_map)
 		 */
 #ifdef CONFIG_DEFAULT_FDT_FILE
 		if (strlen(CONFIG_DEFAULT_FDT_FILE)) {
-			snprintf(fdtfile, sizeof(fdtfile), "%s/%s",
-				 CONFIG_TI_FDT_FOLDER_PATH, CONFIG_DEFAULT_FDT_FILE);
+			snprintf(fdtfile, sizeof(fdtfile), "%s", CONFIG_DEFAULT_FDT_FILE);
 		} else
 #endif
 		{
-			snprintf(fdtfile, sizeof(fdtfile), "%s/%s.dtb",
-				 CONFIG_TI_FDT_FOLDER_PATH, CONFIG_DEFAULT_DEVICE_TREE);
+			snprintf(fdtfile, sizeof(fdtfile), "%s.dtb", CONFIG_DEFAULT_DEVICE_TREE);
 		}
 	} else {
-		snprintf(fdtfile, sizeof(fdtfile), "%s/%s", CONFIG_TI_FDT_FOLDER_PATH,
-			 fdt_file_name);
+		snprintf(fdtfile, sizeof(fdtfile), "%s", fdt_file_name);
 	}
 
 	env_set("fdtfile", fdtfile);
diff --git a/board/ti/j721e/evm.c b/board/ti/j721e/evm.c
index 6221be9dcff..c54816e9e18 100644
--- a/board/ti/j721e/evm.c
+++ b/board/ti/j721e/evm.c
@@ -395,9 +395,9 @@ static int probe_daughtercards(void)
 
 #ifdef CONFIG_BOARD_LATE_INIT
 static struct ti_fdt_map ti_j721e_evm_fdt_map[] = {
-	{"j721e", "k3-j721e-common-proc-board.dtb"},
-	{"j721e-sk", "k3-j721e-sk.dtb"},
-	{"j7200", "k3-j7200-common-proc-board.dtb"},
+	{"j721e", "ti/k3-j721e-common-proc-board.dtb"},
+	{"j721e-sk", "ti/k3-j721e-sk.dtb"},
+	{"j7200", "ti/k3-j7200-common-proc-board.dtb"},
 	{ /* Sentinel. */ }
 };
 static void setup_board_eeprom_env(void)
diff --git a/board/ti/j721s2/evm.c b/board/ti/j721s2/evm.c
index 2cfeb3bec6c..6627e0ce466 100644
--- a/board/ti/j721s2/evm.c
+++ b/board/ti/j721s2/evm.c
@@ -116,8 +116,8 @@ int checkboard(void)
 }
 
 static struct ti_fdt_map ti_j721s2_evm_fdt_map[] = {
-	{"j721s2", "k3-j721s2-common-proc-board.dtb"},
-	{"am68-sk", "k3-am68-sk-base-board.dtb"},
+	{"j721s2", "ti/k3-j721s2-common-proc-board.dtb"},
+	{"am68-sk", "ti/k3-am68-sk-base-board.dtb"},
 	{ /* Sentinel. */ }
 };
 
-- 
2.34.1



More information about the U-Boot mailing list