Hi Simon, On Mon, Nov 25, 2019 at 12:12 PM Simon Glass <s...@chromium.org> wrote: > > If these phases are used we typically want to enable pinctrl in then, so > that pad setup and GPIO access are possible. > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > Changes in v5: > - Correct build error in chromebook_samus_tpl > > Changes in v4: None > Changes in v3: None > Changes in v2: None > > arch/Kconfig | 2 ++ > configs/chromebook_samus_tpl_defconfig | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/arch/Kconfig b/arch/Kconfig > index 54de91afb3..ae9c93ed7b 100644 > --- a/arch/Kconfig > +++ b/arch/Kconfig > @@ -193,6 +193,7 @@ config X86 > imply SPL_OF_LIBFDT > imply SPL_DRIVERS_MISC_SUPPORT > imply SPL_GPIO_SUPPORT > + imply SPL_PINCTRL > imply SPL_LIBCOMMON_SUPPORT > imply SPL_LIBGENERIC_SUPPORT > imply SPL_SERIAL_SUPPORT > @@ -206,6 +207,7 @@ config X86 > imply TPL_DM > imply TPL_DRIVERS_MISC_SUPPORT > imply TPL_GPIO_SUPPORT > + imply TPL_PINCTRL > imply TPL_LIBCOMMON_SUPPORT > imply TPL_LIBGENERIC_SUPPORT > imply TPL_SERIAL_SUPPORT > diff --git a/configs/chromebook_samus_tpl_defconfig > b/configs/chromebook_samus_tpl_defconfig > index fc6ceeac70..44e6d33181 100644 > --- a/configs/chromebook_samus_tpl_defconfig > +++ b/configs/chromebook_samus_tpl_defconfig > @@ -73,6 +73,8 @@ CONFIG_SYS_I2C_DW=y > CONFIG_TPL_MISC=y > CONFIG_CROS_EC=y > CONFIG_CROS_EC_LPC=y > +# CONFIG_SPL_PINCTRL is not set > +# CONFIG_TPL_PINCTRL is not set
If we have to disable these 2 options for Samus, I wonder why we imply these 2 options in arch/Kconfig for x86? And how about other x86 boards? Do they need unset these 2 options, eg: QEMU x86_64? > CONFIG_SYS_NS16550=y > CONFIG_SOUND=y > CONFIG_SOUND_I8254=y > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot