dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn
@ 2025-10-23 20:01 Nathan Chancellor
  2025-10-23 20:03 ` Arnd Bergmann
  2025-10-27 21:59 ` Nicolas Schier
  0 siblings, 2 replies; 3+ messages in thread
From: Nathan Chancellor @ 2025-10-23 20:01 UTC (permalink / raw)
  To: Nathan Chancellor, Nicolas Schier
  Cc: Rasmus Villemoes, Arnd Bergmann, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, linux-kbuild,
	linux-kernel, dri-devel

Since commit e88ca24319e4 ("kbuild: consolidate warning flags in
scripts/Makefile.extrawarn"), scripts/Makefile.extrawarn contains all
warnings for the main kernel build, not just warnings enabled by the
values for W=. Rename it to scripts/Makefile.warn to make it clearer
that this Makefile is where all Kbuild warning handling should exist.

Signed-off-by: Nathan Chancellor <nathan@kernel.org>
---
 Makefile                                      | 2 +-
 drivers/gpu/drm/Makefile                      | 2 +-
 scripts/{Makefile.extrawarn => Makefile.warn} | 0
 3 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 79b631075d33..96ca0f17d158 100644
--- a/Makefile
+++ b/Makefile
@@ -1084,7 +1084,7 @@ KBUILD_CPPFLAGS += $(call cc-option,-fmacro-prefix-map=$(srcroot)/=)
 endif
 
 # include additional Makefiles when needed
-include-y			:= scripts/Makefile.extrawarn
+include-y			:= scripts/Makefile.warn
 include-$(CONFIG_DEBUG_INFO)	+= scripts/Makefile.debug
 include-$(CONFIG_DEBUG_INFO_BTF)+= scripts/Makefile.btf
 include-$(CONFIG_KASAN)		+= scripts/Makefile.kasan
diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
index 4b2f7d794275..d1a102c4b80a 100644
--- a/drivers/gpu/drm/Makefile
+++ b/drivers/gpu/drm/Makefile
@@ -6,7 +6,7 @@
 CFLAGS-$(CONFIG_DRM_USE_DYNAMIC_DEBUG)	+= -DDYNAMIC_DEBUG_MODULE
 
 # Unconditionally enable W=1 warnings locally
-# --- begin copy-paste W=1 warnings from scripts/Makefile.extrawarn
+# --- begin copy-paste W=1 warnings from scripts/Makefile.warn
 subdir-ccflags-y += -Wextra -Wunused -Wno-unused-parameter
 subdir-ccflags-y += $(call cc-option, -Wrestrict)
 subdir-ccflags-y += -Wmissing-format-attribute
diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.warn
similarity index 100%
rename from scripts/Makefile.extrawarn
rename to scripts/Makefile.warn

---
base-commit: ac1280211e1c41704c756fd1bc5512f92010b3f0
change-id: 20251023-rename-scripts-makefile-extrawarn-c494df9d0dba

Best regards,
--  
Nathan Chancellor <nathan@kernel.org>


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

* Re: [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn
  2025-10-23 20:01 [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn Nathan Chancellor
@ 2025-10-23 20:03 ` Arnd Bergmann
  2025-10-27 21:59 ` Nicolas Schier
  1 sibling, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2025-10-23 20:03 UTC (permalink / raw)
  To: Nathan Chancellor, Nicolas Schier
  Cc: Rasmus Villemoes, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, Dave Airlie, Simona Vetter, linux-kbuild,
	linux-kernel, dri-devel

On Thu, Oct 23, 2025, at 22:01, Nathan Chancellor wrote:
> Since commit e88ca24319e4 ("kbuild: consolidate warning flags in
> scripts/Makefile.extrawarn"), scripts/Makefile.extrawarn contains all
> warnings for the main kernel build, not just warnings enabled by the
> values for W=. Rename it to scripts/Makefile.warn to make it clearer
> that this Makefile is where all Kbuild warning handling should exist.
>
> Signed-off-by: Nathan Chancellor <nathan@kernel.org>

Makes sense,

Acked-by: Arnd Bergmann <arnd@arndb.de>

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

* Re: [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn
  2025-10-23 20:01 [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn Nathan Chancellor
  2025-10-23 20:03 ` Arnd Bergmann
@ 2025-10-27 21:59 ` Nicolas Schier
  1 sibling, 0 replies; 3+ messages in thread
From: Nicolas Schier @ 2025-10-27 21:59 UTC (permalink / raw)
  To: Nathan Chancellor
  Cc: Nicolas Schier, Rasmus Villemoes, Arnd Bergmann,
	Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
	Simona Vetter, linux-kbuild, linux-kernel, dri-devel

On Thu, 23 Oct 2025 22:01:36 +0200, Nathan Chancellor wrote:
> Since commit e88ca24319e4 ("kbuild: consolidate warning flags in
> scripts/Makefile.extrawarn"), scripts/Makefile.extrawarn contains all
> warnings for the main kernel build, not just warnings enabled by the
> values for W=. Rename it to scripts/Makefile.warn to make it clearer
> that this Makefile is where all Kbuild warning handling should exist.
> 
> 
> [...]

Applied to kbuild-next, thanks!

[1/1] kbuild: Rename Makefile.extrawarn to Makefile.warn
      https://git.kernel.org/kbuild/c/24722b62

Please note that commit hashes might change in case of issues with
kbuild-next.

Best regards,
-- 
Nicolas


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

end of thread, other threads:[~2025-10-27 22:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-23 20:01 [PATCH] kbuild: Rename Makefile.extrawarn to Makefile.warn Nathan Chancellor
2025-10-23 20:03 ` Arnd Bergmann
2025-10-27 21:59 ` Nicolas Schier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).