[U-Boot] [PATCH 32/42] Convert CONFIG_SPL_PINCTRL_SUPPORT to Kconfig

Masahiro Yamada yamada.masahiro at socionext.com
Thu Aug 25 03:34:32 CEST 2016


2016-08-25 1:52 GMT+09:00 Simon Glass <sjg at chromium.org>:
> Move this option to Kconfig and tidy up existing uses.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  configs/chromebook_jerry_defconfig | 1 +
>  configs/evb-rk3288_defconfig       | 1 +
>  configs/fennec-rk3288_defconfig    | 1 +
>  configs/firefly-rk3288_defconfig   | 1 +
>  configs/miniarm-rk3288_defconfig   | 1 +
>  configs/popmetal-rk3288_defconfig  | 1 +
>  configs/rock2_defconfig            | 1 +
>  include/configs/rk3288_common.h    | 2 --
>  8 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/configs/chromebook_jerry_defconfig b/configs/chromebook_jerry_defconfig
> index d7f9f47..7ca2009 100644
> --- a/configs/chromebook_jerry_defconfig
> +++ b/configs/chromebook_jerry_defconfig
> @@ -13,6 +13,7 @@ CONFIG_SPL_DRIVERS_MISC_SUPPORT=y
>  CONFIG_SPL_I2C_SUPPORT=y
>  CONFIG_SPL_LIBCOMMON_SUPPORT=y
>  CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_SPL_PINCTRL_SUPPORT=y
>  CONFIG_HUSH_PARSER=y
>  CONFIG_CMD_BOOTZ=y
>  # CONFIG_CMD_IMLS is not set


Wait!

I see only one define in rk3288_common.h,
but I did not see any reference of it.

$ git grep CONFIG_SPL_PINCTRL_SUPPORT
include/configs/rk3288_common.h:#define CONFIG_SPL_PINCTRL_SUPPORT





I was surprised at this because we already have
the following in drivers/pinctrl/Kconfig:

config SPL_PINCTRL
         bool "Support pin controlloers in SPL"
         depends on SPL && SPL_DM
         help
           This option is an SPL-variant of the PINCTRL option.
           See the help of PINCTRL for details.




The right thing is to just remove
the unreferenced CONFIG_SPL_PINCTRL_SUPPORT.


-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list