On Thu, Mar 2, 2017 at 5:29 PM, Rob Herring <robh@xxxxxxxxxx> wrote: > On Mon, Feb 27, 2017 at 08:07:19PM +0100, Linus Walleij wrote: >> As we have a PCI driver for the Gemini, we should select >> MIGHT_HAVE_PCI. >> >> Cc: Janos Laube <janos.dev@xxxxxxxxx> >> Cc: Paulius Zaleckas <paulius.zaleckas@xxxxxxxxx> >> Cc: Hans Ulli Kroll <ulli.kroll@xxxxxxxxxxxxxx> >> Cc: Florian Fainelli <f.fainelli@xxxxxxxxx> >> Cc: Feng-Hsin Chiang <john453@xxxxxxxxxxxxxxxx> >> Cc: Greentime Hu <green.hu@xxxxxxxxx> >> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> >> --- >> ChangeLog v1->v2->v3: >> - No changes >> >> PCI maintainers: this is FYI only, I will funnel this to the ARM >> SoC tree once we are done with the PCI driver. >> --- >> arch/arm/mach-gemini/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/mach-gemini/Kconfig b/arch/arm/mach-gemini/Kconfig >> index a5ee5fbab796..21f89a4c16fe 100644 >> --- a/arch/arm/mach-gemini/Kconfig >> +++ b/arch/arm/mach-gemini/Kconfig >> @@ -6,6 +6,7 @@ menuconfig ARCH_GEMINI >> select FTTMR010_TIMER >> select GPIO_GEMINI >> select GPIOLIB >> + select MIGHT_HAVE_PCI > > Doesn't Multi-platform do this for you? Oh it does, I just missed that. > If not, IMO, we should do that > or kill this option. We don't do an option like this for I2C, SPI, > display, anything else (well, except GPIOLIB ;)). I killed ARCH_WANT_OPTIONAL_GPIOLIB at least that thing was really confusing. Maybe we should have MULTIPLATFORM select GPIOLIB too. Yours, Linus Walleij
