public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] kbuild: Fix unnecessary rebuilding DT base+overlay targets
@ 2026-01-14 18:46 Rob Herring (Arm)
  2026-01-14 19:16 ` David Heidelberg
  0 siblings, 1 reply; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-01-14 18:46 UTC (permalink / raw)
  To: Saravana Kannan, Nathan Chancellor, Nicolas Schier
  Cc: David Heidelberg, devicetree, linux-kbuild, linux-kernel

Commit 5d0cac427d42 ("kbuild: Support directory targets for building
DTBs") causes base+overlay targets to be rebuilt every time. The problem
is due to the added check on $(MAKECMDGOALS) which now testing again
doesn't even seem to be needed. Revert that portion of the commit.

Fixes: 5d0cac427d42 ("kbuild: Support directory targets for building DTBs")
Reported-by: David Heidelberg <david@ixit.cz>
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 scripts/Makefile.dtbs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/scripts/Makefile.dtbs b/scripts/Makefile.dtbs
index f5d51b8c2eeb..b2d85b9a4a7a 100644
--- a/scripts/Makefile.dtbs
+++ b/scripts/Makefile.dtbs
@@ -23,10 +23,7 @@ dtb-y           := $(addprefix $(obj)/, $(dtb-y))
 multi-dtb-y     := $(addprefix $(obj)/, $(multi-dtb-y))
 real-dtb-y      := $(addprefix $(obj)/, $(real-dtb-y))
 
-dtb-targets := %.dtb %.dtbo dtbs dtbs_check
-ifneq ($(findstring /dts/,$(MAKECMDGOALS))$(filter $(dtb-targets),$(MAKECMDGOALS)),)
 always-y        += $(dtb-y)
-endif
 
 targets         += $(real-dtb-y)
 
-- 
2.51.0


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

* Re: [PATCH] kbuild: Fix unnecessary rebuilding DT base+overlay targets
  2026-01-14 18:46 [PATCH] kbuild: Fix unnecessary rebuilding DT base+overlay targets Rob Herring (Arm)
@ 2026-01-14 19:16 ` David Heidelberg
  0 siblings, 0 replies; 2+ messages in thread
From: David Heidelberg @ 2026-01-14 19:16 UTC (permalink / raw)
  To: Rob Herring (Arm), Saravana Kannan, Nathan Chancellor,
	Nicolas Schier
  Cc: devicetree, linux-kbuild, linux-kernel

On 14/01/2026 19:46, Rob Herring (Arm) wrote:
> Commit 5d0cac427d42 ("kbuild: Support directory targets for building
> DTBs") causes base+overlay targets to be rebuilt every time. The problem
> is due to the added check on $(MAKECMDGOALS) which now testing again
> doesn't even seem to be needed. Revert that portion of the commit.
> 
> Fixes: 5d0cac427d42 ("kbuild: Support directory targets for building DTBs")
> Reported-by: David Heidelberg <david@ixit.cz>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>   scripts/Makefile.dtbs | 3 ---
>   1 file changed, 3 deletions(-)
> 

Whoa, that was quick! Thanks.

Tested-by: David Heidelberg <david@ixit.cz>

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

end of thread, other threads:[~2026-01-14 19:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-14 18:46 [PATCH] kbuild: Fix unnecessary rebuilding DT base+overlay targets Rob Herring (Arm)
2026-01-14 19:16 ` David Heidelberg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox