Re: [PATCH 1/2] pinctrl: remove unneeded #ifdef around declarations
From: Linus Walleij
Date: Tue Jun 25 2019 - 04:52:55 EST
thanks for your patch. For some reason I managed to pick up
patch 2 before patch 1. I applied this now with some fuzzing.
(Please check the result.)
On Thu, Jun 13, 2019 at 3:55 AM Masahiro Yamada
> What is the point in surrounding the whole of declarations with
> ifdef like this?
I don't know if it is generally good to have phrases posed as
questions in a commit message, we prefer to have statements
about the change not a polemic dialog.
> #ifdef CONFIG_FOO
> int foo(void);
> If CONFIG_FOO is not defined, all callers of foo() will fail
> with implicit declaration errors since the top Makefile adds
> -Werror-implicit-function-declaration to KBUILD_CFLAGS.
Maybe this flag was not in the top Makefile when the #ifdefs
> This breaks the build earlier when you are doing something wrong.
> That's it.
> Anyway, it will fail to link since the definition of foo() is not
> In summary, these ifdef are unneeded.
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
Pushing this to the zeroday builders and let's see what happens!