All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] Avoid needless CONFIG_TPL_BUILD checks for U-Boot proper build
@ 2026-09-08 17:04 Massimo Pegorer
  2026-09-08 17:04 ` [PATCH 1/2] arch: arm: rockchip: Skip needless CONFIG_TPL_BUILD check Massimo Pegorer
  2026-09-08 17:05 ` [PATCH 2/2] drivers: " Massimo Pegorer
  0 siblings, 2 replies; 3+ messages in thread
From: Massimo Pegorer @ 2026-09-08 17:04 UTC (permalink / raw)
  To: Simon Glass, u-boot
  Cc: Tom Rini, Ilias Apalodimas, Quentin Schulz, Kever Yang,
	Mattijs Korpershoek, Jonas Karlman, Neil Armstrong, Marek Vasut,
	Jörg Sommer, Carlo Caione, Alexey Charkov, Massimo Pegorer

Just check that CONFIG_XPL_BUILD is not set to select files for U-Boot
proper build. Indeed, CONFIG_TPL_BUILD being set implies by design that
CONFIG_XPL_BUILD is set too, therefore CONFIG_XPL_BUILD not set implies
that CONFIG_TPL_BUILD is not set either, making its check needless.

Signed-off-by: Massimo Pegorer <massimo.pegorer+oss@gmail.com>
---
Massimo Pegorer (2):
      arch: arm: rockchip: Skip needless CONFIG_TPL_BUILD check
      drivers: Skip needless CONFIG_TPL_BUILD check

 arch/arm/mach-rockchip/Makefile | 2 +-
 drivers/Makefile                | 2 +-
 drivers/bus/Makefile            | 2 +-
 drivers/mtd/Makefile            | 2 +-
 drivers/mtd/nand/Makefile       | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)
---
base-commit: b4ac71db13a33c4e6532c60a781ddf34b7f4375a
change-id: 20260908-xpl-8c3ee1f3549e

Best regards,
--  
Massimo Pegorer <massimo.pegorer+oss@gmail.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/2] arch: arm: rockchip: Skip needless CONFIG_TPL_BUILD check
  2026-09-08 17:04 [PATCH 0/2] Avoid needless CONFIG_TPL_BUILD checks for U-Boot proper build Massimo Pegorer
@ 2026-09-08 17:04 ` Massimo Pegorer
  2026-09-08 17:05 ` [PATCH 2/2] drivers: " Massimo Pegorer
  1 sibling, 0 replies; 3+ messages in thread
From: Massimo Pegorer @ 2026-09-08 17:04 UTC (permalink / raw)
  To: Simon Glass, u-boot
  Cc: Tom Rini, Ilias Apalodimas, Quentin Schulz, Kever Yang,
	Mattijs Korpershoek, Jonas Karlman, Neil Armstrong, Marek Vasut,
	Jörg Sommer, Carlo Caione, Alexey Charkov, Massimo Pegorer

Just check that CONFIG_XPL_BUILD is not set to select files for U-Boot
proper build. Indeed, CONFIG_TPL_BUILD being set implies by design that
CONFIG_XPL_BUILD is set too, therefore CONFIG_XPL_BUILD not set implies
that CONFIG_TPL_BUILD is not set either, making its check needless.

Fixes: bef9fdbed2e5 ("arch: Use CONFIG_XPL_BUILD instead of CONFIG_SPL_BUILD")
Signed-off-by: Massimo Pegorer <massimo.pegorer+oss@gmail.com>
---
 arch/arm/mach-rockchip/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-rockchip/Makefile b/arch/arm/mach-rockchip/Makefile
index d3bc0689f89..8148816c48c 100644
--- a/arch/arm/mach-rockchip/Makefile
+++ b/arch/arm/mach-rockchip/Makefile
@@ -15,7 +15,7 @@ obj-tpl-$(CONFIG_ROCKCHIP_PX30) += px30-board-tpl.o spl_common.o
 
 obj-spl-$(CONFIG_ROCKCHIP_RK3036) += rk3036-board-spl.o spl_common.o
 
-ifeq ($(CONFIG_XPL_BUILD)$(CONFIG_TPL_BUILD),)
+ifeq ($(CONFIG_XPL_BUILD),)
 
 # Always include boot_mode.o, as we bypass it (i.e. turn it off)
 # inside of boot_mode.c when CONFIG_ROCKCHIP_BOOT_MODE_REG is 0.  This way,

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] drivers: Skip needless CONFIG_TPL_BUILD check
  2026-09-08 17:04 [PATCH 0/2] Avoid needless CONFIG_TPL_BUILD checks for U-Boot proper build Massimo Pegorer
  2026-09-08 17:04 ` [PATCH 1/2] arch: arm: rockchip: Skip needless CONFIG_TPL_BUILD check Massimo Pegorer
@ 2026-09-08 17:05 ` Massimo Pegorer
  1 sibling, 0 replies; 3+ messages in thread
From: Massimo Pegorer @ 2026-09-08 17:05 UTC (permalink / raw)
  To: Simon Glass, u-boot
  Cc: Tom Rini, Ilias Apalodimas, Quentin Schulz, Kever Yang,
	Mattijs Korpershoek, Jonas Karlman, Neil Armstrong, Marek Vasut,
	Jörg Sommer, Carlo Caione, Alexey Charkov, Massimo Pegorer

Just check that CONFIG_XPL_BUILD is not set to select files for U-Boot
proper build. Indeed, CONFIG_TPL_BUILD being set implies by design that
CONFIG_XPL_BUILD is set too, therefore CONFIG_XPL_BUILD not set implies
that CONFIG_TPL_BUILD is not set either, making its check needless.

Fixes: 371dc068bbf5 ("drivers: Use CONFIG_XPL_BUILD instead of CONFIG_SPL_BUILD")
Signed-off-by: Massimo Pegorer <massimo.pegorer+oss@gmail.com>
---
 drivers/Makefile          | 2 +-
 drivers/bus/Makefile      | 2 +-
 drivers/mtd/Makefile      | 2 +-
 drivers/mtd/nand/Makefile | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 269c3db12e8..da37338fe1f 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -87,7 +87,7 @@ obj-$(CONFIG_TPL_MPC8XXX_INIT_DDR) += ddr/fsl/
 
 endif
 
-ifeq ($(CONFIG_XPL_BUILD)$(CONFIG_TPL_BUILD),)
+ifeq ($(CONFIG_XPL_BUILD),)
 
 obj-y += ata/
 obj-$(CONFIG_DM_DEMO) += demo/
diff --git a/drivers/bus/Makefile b/drivers/bus/Makefile
index 2d242bc2886..239d9a35fe4 100644
--- a/drivers/bus/Makefile
+++ b/drivers/bus/Makefile
@@ -3,7 +3,7 @@
 # Makefile for the bus drivers.
 #
 
-ifeq ($(CONFIG_XPL_BUILD)$(CONFIG_TPL_BUILD),)
+ifeq ($(CONFIG_XPL_BUILD),)
 obj-$(CONFIG_TI_PWMSS)	+= ti-pwmss.o
 obj-$(CONFIG_UNIPHIER_SYSTEM_BUS) += uniphier-system-bus.o
 endif
diff --git a/drivers/mtd/Makefile b/drivers/mtd/Makefile
index ce05e206073..e204f19ff8b 100644
--- a/drivers/mtd/Makefile
+++ b/drivers/mtd/Makefile
@@ -16,7 +16,7 @@ mtd-$(CONFIG_RENESAS_RPC_HF) += renesas_rpc_hf.o
 mtd-$(CONFIG_HBMC_AM654) += hbmc-am654.o
 
 # U-Boot build
-ifeq ($(CONFIG_XPL_BUILD)$(CONFIG_TPL_BUILD),)
+ifeq ($(CONFIG_XPL_BUILD),)
 
 ifneq ($(mtd-y),)
 obj-y += mtd.o
diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index c8169cf7390..cf53c5f601e 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0+
 
-ifeq ($(CONFIG_XPL_BUILD)$(CONFIG_TPL_BUILD),)
+ifeq ($(CONFIG_XPL_BUILD),)
 nandcore-objs := core.o bbt.o
 obj-$(CONFIG_MTD_NAND_CORE) += nandcore.o
 obj-$(CONFIG_MTD_RAW_NAND) += raw/

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-09-08 17:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-08 17:04 [PATCH 0/2] Avoid needless CONFIG_TPL_BUILD checks for U-Boot proper build Massimo Pegorer
2026-09-08 17:04 ` [PATCH 1/2] arch: arm: rockchip: Skip needless CONFIG_TPL_BUILD check Massimo Pegorer
2026-09-08 17:05 ` [PATCH 2/2] drivers: " Massimo Pegorer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.