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@xxxxxxxxxxxxxx> --- 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
