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

Simon Glass sjg at chromium.org
Thu Apr 29 18:09:58 CEST 2021


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?

> 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


More information about the U-Boot mailing list