[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