On Wed, 11 Oct 2023, Masahiro Yamada <[email protected]> wrote: > On Wed, Oct 11, 2023 at 4:29 PM Jani Nikula <[email protected]> wrote: >> >> We enable a bunch more compiler warnings than the kernel >> defaults. However, they've drifted to become a unique set of warnings, >> and have increasingly fallen behind from the W=1 set. >> >> Align with the W=1 warnings from scripts/Makefile.extrawarn for clarity, >> by copy-pasting them with s/KBUILD_CFLAGS/subdir-ccflags-y/ to make it >> easier to compare in the future. >> >> Some of the -Wextra warnings do need to be disabled, just like in >> Makefile.extrawarn, but take care to not disable them for W=2 or W=3 >> builds, depending on the warning. >> >> v2: Add back some -Wextra warning disables (Nathan) >> >> Cc: Arnd Bergmann <[email protected]> >> Cc: Nick Desaulniers <[email protected]> >> Cc: Nathan Chancellor <[email protected]> >> Cc: Masahiro Yamada <[email protected]> >> Reviewed-by: Nathan Chancellor <[email protected]> >> Signed-off-by: Jani Nikula <[email protected]> > > Reviewed-by: Masahiro Yamada <[email protected]>
Thanks for the reviews everyone... ...though we missed some s/KBUILD_CFLAGS/subdir-ccflags-y/ in v2. Fixed while applying. BR, Jani. -- Jani Nikula, Intel
