[PATCH] kbuild: make implicit int to pointer conversion warning an error

Ahmad Fatoum a.fatoum at pengutronix.de
Tue Apr 30 06:15:40 PDT 2024


We are guilty of using ints as pointers in readl/writel, but the old
definition of these uses a cast to silence the warning.

Everywhere else there's no reason to allow this, so make this a hard
error always to weed out any instances we may still have.

Disallowing this has the added benefit of making it easier to change API
in barebox: We do not guarantee out-of-tree board code API stability,
but we want changes in API to result in build errors, not in warnings
that may be eaten up by a build system sitting in-between Kbuild and the
user.

While at it, we fix the whitespace for the surrounding lines.

Signed-off-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
---
 Makefile | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 8e262c17fd50..517b54bd9e6d 100644
--- a/Makefile
+++ b/Makefile
@@ -451,8 +451,9 @@ KBUILD_CPPFLAGS        := -D__KERNEL__ -D__BAREBOX__ $(LINUXINCLUDE) -fno-builti
 
 KBUILD_CFLAGS   := -Wall -Wundef -Werror=strict-prototypes -Wno-trigraphs \
 		   -fno-strict-aliasing -fno-common -fshort-wchar \
-                   -Werror=implicit-function-declaration -Werror=implicit-int \
-                   -Os -pipe -Wmissing-prototypes -std=gnu11
+		   -Werror=implicit-function-declaration -Werror=implicit-int \
+		   -Werror=int-conversion \
+		   -Os -pipe -Wmissing-prototypes -std=gnu11
 KBUILD_AFLAGS          := -D__ASSEMBLY__
 KBUILD_AFLAGS_KERNEL :=
 KBUILD_CFLAGS_KERNEL :=
-- 
2.39.2




More information about the barebox mailing list