[PATCH v2 04/13] gpio: Introduce CONFIG_ONLY_GENERIC_GPIO to cleanup #ifdefs

[Masami Hiramatsu]

Hi Simon,

2021年4月30日(金) 1:10 Simon Glass <sjg at chromium.org>:
>
> Hi Masami,
>
> On Fri, 16 Apr 2021 at 16:38, Masami Hiramatsu
> <masami.hiramatsu at linaro.org> wrote:
> >
> > Many architecture do not have specific asm/arch/gpio.h, so instead
> > of adding !defined(CONFIG_ARCH_xxx), introduce CONFIG_ONLY_GENERIC_GPIO
>
> This seems OK, but I think GPIO_GENERIC_ONLY is a better name, since
> it uses the GPIO prefix.
>
> I would also prefer to have a 'positive' option, but I suspect that
> might be a pain to do?

Would you mean making it something like CONFIG_GPIO_EXTRA_HEADER ?

I think it is also possible. My concern is if I missed any arch which
should say y that :P.

Thank you,

>
> > and select it.
> >
> > Signed-off-by: Masami Hiramatsu <masami.hiramatsu at linaro.org>
> > ---
> >  arch/arm/Kconfig             |   17 +++++++++++++++++
> >  arch/arm/include/asm/gpio.h  |    8 +-------
> >  board/cortina/common/Kconfig |    1 +
> >  3 files changed, 19 insertions(+), 7 deletions(-)
> >
>
> [..]
>
> Regards,
> Simon



--
Masami Hiramatsu