[PATCH] cmd/ti: Update Kconfig logic and Makefile recursion logic

- Add some dependencies to CMD_DDR3 as this is only valid on some platforms (which tend to select it as well). - The proper gate for going in to cmd/ti is not CONFIG_TI_COMMON_CMD_OPTIONS as nothing under there is controlled by that symbol but the general TI architecture options.
Signed-off-by: Tom Rini trini@konsulko.com --- cmd/Makefile | 7 +++++-- cmd/ti/Kconfig | 1 + cmd/ti/Makefile | 4 ---- 3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/cmd/Makefile b/cmd/Makefile index 9bebf321c397..be13e288ae28 100644 --- a/cmd/Makefile +++ b/cmd/Makefile @@ -235,6 +235,11 @@ obj-$(CONFIG_X86) += x86/ obj-$(CONFIG_CMD_MESON) += meson/
obj-$(CONFIG_ARCH_MVEBU) += mvebu/ + +# TI +obj-$(CONFIG_ARCH_KEYSTONE) += ti/ +obj-$(CONFIG_ARCH_K3) += ti/ +obj-$(CONFIG_ARCH_OMAP2PLUS) += ti/ endif # !CONFIG_SPL_BUILD
obj-$(CONFIG_$(SPL_)CMD_TLV_EEPROM) += tlv_eeprom.o @@ -244,8 +249,6 @@ obj-y += nvedit.o
obj-$(CONFIG_CMD_BCM_EXT_UTILS) += broadcom/
-obj-$(CONFIG_TI_COMMON_CMD_OPTIONS) += ti/ - filechk_data_gz = (echo "static const char data_gz[] ="; cat $< | scripts/bin2c; echo ";")
filechk_data_size = \ diff --git a/cmd/ti/Kconfig b/cmd/ti/Kconfig index db557445a890..9442c9993c19 100644 --- a/cmd/ti/Kconfig +++ b/cmd/ti/Kconfig @@ -2,6 +2,7 @@ menu "TI specific command line interface"
config CMD_DDR3 bool "command for verifying DDR features" + depends on ARCH_KEYSTONE || DRA7XX help Support for testing ddr3 on TI platforms. This command supports memory verification, memory comapre and ecc diff --git a/cmd/ti/Makefile b/cmd/ti/Makefile index 045593396b70..87f85b2c2323 100644 --- a/cmd/ti/Makefile +++ b/cmd/ti/Makefile @@ -1,9 +1,5 @@ # SPDX-License-Identifier: GPL-2.0+ # Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/
-obj- += dummy.o - -ifndef CONFIG_SPL_BUILD obj-$(CONFIG_CMD_DDR3) += ddr3.o obj-$(CONFIG_CMD_PD) += pd.o -endif

On Sat, Oct 07, 2023 at 03:36:58PM -0400, Tom Rini wrote:
- Add some dependencies to CMD_DDR3 as this is only valid on some platforms (which tend to select it as well).
- The proper gate for going in to cmd/ti is not CONFIG_TI_COMMON_CMD_OPTIONS as nothing under there is controlled by that symbol but the general TI architecture options.
Signed-off-by: Tom Rini trini@konsulko.com
Applied to u-boot/master, thanks!
participants (1)
-
Tom Rini