mbox series

[0/4] PCI: Don't select Kconfig symbols by default

Message ID 20200415001244.144623-1-helgaas@kernel.org
Headers show
Series PCI: Don't select Kconfig symbols by default | expand

Message

Bjorn Helgaas April 15, 2020, 12:12 a.m. UTC
From: Bjorn Helgaas <bhelgaas@google.com>

A few Kconfig symbols snuck in with "default y".  In general we don't want
that because we don't want to bloat the kernel with unnecessary drivers.

Remove the ones that are optional.

There are a few left, but they depend on something else that seems like the
real choice, e.g., XEN_PCIDEV_FRONTEND depends on XEN and PCI_XGENE_MSI
depends on PCI_XGENE.

Bjorn Helgaas (4):
  PCI: dra7xx: Don't select CONFIG_PCI_DRA7XX_HOST by default
  PCI: keystone: Don't select CONFIG_PCI_KEYSTONE_HOST by default
  PCI/AER: Don't select CONFIG_PCIEAER by default
  PCI/ASPM: Don't select CONFIG_PCIEASPM by default

 drivers/pci/controller/dwc/Kconfig | 2 --
 drivers/pci/pcie/Kconfig           | 2 --
 2 files changed, 4 deletions(-)

Comments

Bjorn Helgaas April 24, 2020, 7:37 p.m. UTC | #1
On Tue, Apr 14, 2020 at 07:12:40PM -0500, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
> 
> A few Kconfig symbols snuck in with "default y".  In general we don't want
> that because we don't want to bloat the kernel with unnecessary drivers.
> 
> Remove the ones that are optional.
> 
> There are a few left, but they depend on something else that seems like the
> real choice, e.g., XEN_PCIDEV_FRONTEND depends on XEN and PCI_XGENE_MSI
> depends on PCI_XGENE.
> 
> Bjorn Helgaas (4):
>   PCI: dra7xx: Don't select CONFIG_PCI_DRA7XX_HOST by default
>   PCI: keystone: Don't select CONFIG_PCI_KEYSTONE_HOST by default
>   PCI/AER: Don't select CONFIG_PCIEAER by default
>   PCI/ASPM: Don't select CONFIG_PCIEASPM by default
> 
>  drivers/pci/controller/dwc/Kconfig | 2 --
>  drivers/pci/pcie/Kconfig           | 2 --
>  2 files changed, 4 deletions(-)

Applied with Sathy's reviewed-by on the AER patch and the tweak Rob
suggested to the dra7xx one to pci/kconfig for v5.8.