Re: [PATCH 1/2] pinctrl: remove unneeded #ifdef around declarations
From: Linus Walleij
Date: Tue Jun 25 2019 - 04:52:55 EST
Hi Masahiro,
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
<yamada.masahiro@xxxxxxxxxxxxx> wrote:
> 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);
> #endif
>
> 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
where introduced?
> This breaks the build earlier when you are doing something wrong.
> That's it.
Good idea.
> Anyway, it will fail to link since the definition of foo() is not
> compiled.
>
> 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!
Yours,
Linus Walleij