powerpc/44x: Force PCI on for CURRITUCK

Michael Ellerman patch-notifications at ellerman.id.au
Fri Feb 22 20:47:47 AEDT 2019


On Thu, 2019-02-07 at 02:49:35 UTC, Michael Ellerman wrote:
> The recent rework of PCI kconfig symbols exposed an existing bug in
> the CURRITUCK kconfig logic.
> 
> It selects PPC4xx_PCI_EXPRESS which depends on PCI, but PCI is user
> selectable and might be disabled, leading to a warning:
> 
>   WARNING: unmet direct dependencies detected for PPC4xx_PCI_EXPRESS
>     Depends on [n]: PCI [=n] && 4xx [=y]
>     Selected by [y]:
>     - CURRITUCK [=y] && PPC_47x [=y]
> 
> Prior to commit eb01d42a7778 ("PCI: consolidate PCI config entry in
> drivers/pci") PCI was enabled by default for currituck_defconfig so we
> didn't see the warning. The bad logic was still there, it just
> required someone disabling PCI in their .config to hit it.
> 
> Fix it by forcing PCI on for CURRITUCK, which seems was always the
> expectation anyway.
> 
> Fixes: eb01d42a7778 ("PCI: consolidate PCI config entry in drivers/pci")
> Reported-by: Randy Dunlap <rdunlap at infradead.org>
> Signed-off-by: Michael Ellerman <mpe at ellerman.id.au>
> Reviewed-by: Christoph Hellwig <hch at lst.de>

Applied to powerpc next.

https://git.kernel.org/powerpc/c/aa7150ba378650d0e9d84b8e4d805946

cheers


More information about the Linuxppc-dev mailing list