[U-Boot] [PATCH] arm: delete unused macro CONFIG_ARCH_DEVICE_TREE

Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com Cc: Tom Warren twarren@nvidia.com Cc: Stephen Warren swarren@nvidia.com Cc: Rajeshwari Birje rajeshwari.s@samsung.com Cc: Inderpal Singh inderpal.singh@linaro.org ---
Hi.
I grepped the whole tree and found arch/arm/cpu/armv7/tegra124/config.mk include/configs/arndale.h include/configs/smdk5420.h define CONFIG_ARCH_DEVICE_TREE, but it is never used anywhere.
I cannot understand what this macro was added for. If I am doing a wrong thing, please stop me.
arch/arm/cpu/armv7/tegra124/config.mk | 10 ---------- include/configs/arndale.h | 2 -- include/configs/smdk5420.h | 2 -- 3 files changed, 14 deletions(-) delete mode 100644 arch/arm/cpu/armv7/tegra124/config.mk
diff --git a/arch/arm/cpu/armv7/tegra124/config.mk b/arch/arm/cpu/armv7/tegra124/config.mk deleted file mode 100644 index 2f1c645..0000000 --- a/arch/arm/cpu/armv7/tegra124/config.mk +++ /dev/null @@ -1,10 +0,0 @@ -# -# (C) Copyright 2013 -# NVIDIA Corporation <www.nvidia.com> -# (C) Copyright 2002 -# Gary Jennejohn, DENX Software Engineering, garyj@denx.de -# -# SPDX-License-Identifier: GPL-2.0+ -# - -CONFIG_ARCH_DEVICE_TREE := tegra124 diff --git a/include/configs/arndale.h b/include/configs/arndale.h index 9584d82..515facf 100644 --- a/include/configs/arndale.h +++ b/include/configs/arndale.h @@ -22,8 +22,6 @@ #define CONFIG_DISPLAY_CPUINFO #define CONFIG_DISPLAY_BOARDINFO
-/* Enable fdt support for Exynos5250 */ -#define CONFIG_ARCH_DEVICE_TREE exynos5250 #define CONFIG_OF_CONTROL #define CONFIG_OF_SEPARATE
diff --git a/include/configs/smdk5420.h b/include/configs/smdk5420.h index 447f8e5..b96eea8 100644 --- a/include/configs/smdk5420.h +++ b/include/configs/smdk5420.h @@ -17,8 +17,6 @@ #undef CONFIG_DEFAULT_DEVICE_TREE #define CONFIG_DEFAULT_DEVICE_TREE exynos5420-smdk5420
-#define CONFIG_ARCH_DEVICE_TREE exynos5420 - #define CONFIG_VAR_SIZE_SPL
#define CONFIG_SYS_SDRAM_BASE 0x20000000

On 02/24/2014 05:45 AM, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com
Commit description?
Acked-by: Stephen Warren swarren@nvidia.com
FWIW, this config option was used prior to:
8ad723acd7e9 dts/Makefile: don't define ARCH_CPU_DTS, BOARD_DTS 6e8e0311ccc7 dt: don't use ARCH_CPU_DTS ... 6c5be646b47e Tegra: fdt: Change /include/ to #include for C preprocessor
I evidently forgot to remove some of the places where Exynos boards defined CONFIG_ARCH_DEVICE_TREE, and when sending the Tegra124 patches recently, didn't notice that I didn't need the file that defined it there.

Hello Stephen
On Mon, 24 Feb 2014 10:26:46 -0700 Stephen Warren swarren@wwwdotorg.org wrote:
On 02/24/2014 05:45 AM, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com
Commit description?
This patch looked trivial so I did not write commit description. Sorry for my laziness..
Best Regards Masahiro

On Mon, Feb 24, 2014 at 09:45:48PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com Cc: Tom Warren twarren@nvidia.com Cc: Stephen Warren swarren@nvidia.com Cc: Rajeshwari Birje rajeshwari.s@samsung.com Cc: Inderpal Singh inderpal.singh@linaro.org
Applied to u-boot/master, thanks!
participants (3)
-
Masahiro Yamada
-
Stephen Warren
-
Tom Rini