* [PATCH v4 2/2] pci: pcie: add dependency info to Kconfig
2022-11-15 11:38 [PATCH v4 0/2] add hotplug depedency info Albert Zhou
2022-11-15 11:38 ` [PATCH v4 1/2] pci: hotplug: add dependency info to Kconfig Albert Zhou
@ 2022-11-15 11:38 ` Albert Zhou
2022-11-15 15:29 ` [PATCH v4 0/2] add hotplug depedency info Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Albert Zhou @ 2022-11-15 11:38 UTC (permalink / raw)
To: bhelgaas; +Cc: linux-pci, lukas, mika.westerberg
Thunderbolt/USB4 PCIe tunneling require the Hotplug feature. This is now
recorded in the help message for HOTPLUG_PCI_PCIE. Further, PCIEPORTBUS
and HOTPLUG_PCI_PCIE are defaulted to Y if USB4 is selected.
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Albert Zhou <albert.zhou.50@gmail.com>
---
drivers/pci/pcie/Kconfig | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/pci/pcie/Kconfig b/drivers/pci/pcie/Kconfig
index 788ac8df3f9d..32cc9a31e228 100644
--- a/drivers/pci/pcie/Kconfig
+++ b/drivers/pci/pcie/Kconfig
@@ -4,20 +4,22 @@
#
config PCIEPORTBUS
bool "PCI Express Port Bus support"
+ default y if USB4
help
This enables PCI Express Port Bus support. Users can then enable
support for Native Hot-Plug, Advanced Error Reporting, Power
Management Events, and Downstream Port Containment.
-
#
# Include service Kconfig here
#
config HOTPLUG_PCI_PCIE
bool "PCI Express Hotplug driver"
depends on HOTPLUG_PCI && PCIEPORTBUS
+ default y if USB4
help
Say Y here if you have a motherboard that supports PCI Express Native
- Hotplug
+ Hotplug. Thunderbolt/USB4 PCIe tunneling needs Native PCIe Hotplug to
+ be enabled
When in doubt, say N.
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v4 0/2] add hotplug depedency info
2022-11-15 11:38 [PATCH v4 0/2] add hotplug depedency info Albert Zhou
2022-11-15 11:38 ` [PATCH v4 1/2] pci: hotplug: add dependency info to Kconfig Albert Zhou
2022-11-15 11:38 ` [PATCH v4 2/2] pci: pcie: " Albert Zhou
@ 2022-11-15 15:29 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2022-11-15 15:29 UTC (permalink / raw)
To: Albert Zhou; +Cc: bhelgaas, linux-pci, lukas, mika.westerberg
On Tue, Nov 15, 2022 at 10:38:55PM +1100, Albert Zhou wrote:
> v3->v4: Add Mika's review tag.
>
> v2->v3: Dependency comment made more precise and technical, as suggested
> by Mika Westerberg.
>
> v1->v2: I added comments that PCIe cards with USB4 or Thunderbolt also
> require the hotplug feature. I also added the "default y if USB4" line
> to the relevant config options, as suggested by Lukas Wunner.
>
> Albert Zhou (2):
> pci: hotplug: add dependency info to Kconfig
> pci: pcie: add dependency info to Kconfig
>
> drivers/pci/hotplug/Kconfig | 4 +++-
> drivers/pci/pcie/Kconfig | 6 ++++--
> 2 files changed, 7 insertions(+), 3 deletions(-)
No need to repost just to add Reviewed-by or other tags. b4 does that
automatically.
I squashed these since it's really a single logical change.
I updated to follow the style conventions for subject lines, add
info to subject line, and drop extraneous whitespace change.
Applied to pci/hotplug for v6.2, thanks!
commit e67ad9354a9b ("PCI: pciehp: Enable by default if USB4 enabled")
Author: Albert Zhou <albert.zhou.50@gmail.com>
Date: Tue Nov 15 22:38:56 2022 +1100
PCI: pciehp: Enable by default if USB4 enabled
Thunderbolt/USB4 PCIe tunneling depends on native PCIe hotplug. Enable
pciehp by default if USB4 is enabled.
[bhelgaas: squash, update subject, commit logs, tidy whitespace]
Link: https://lore.kernel.org/r/20221115113857.35800-2-albert.zhou.50@gmail.com
Link: https://lore.kernel.org/r/20221115113857.35800-3-albert.zhou.50@gmail.com
Signed-off-by: Albert Zhou <albert.zhou.50@gmail.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
diff --git a/drivers/pci/hotplug/Kconfig b/drivers/pci/hotplug/Kconfig
index 840a84bb5ee2..48113b210cf9 100644
--- a/drivers/pci/hotplug/Kconfig
+++ b/drivers/pci/hotplug/Kconfig
@@ -6,11 +6,14 @@
menuconfig HOTPLUG_PCI
bool "Support for PCI Hotplug"
depends on PCI && SYSFS
+ default y if USB4
help
Say Y here if you have a motherboard with a PCI Hotplug controller.
This allows you to add and remove PCI cards while the machine is
powered up and running.
+ Thunderbolt/USB4 PCIe tunneling depends on native PCIe hotplug.
+
When in doubt, say N.
if HOTPLUG_PCI
diff --git a/drivers/pci/pcie/Kconfig b/drivers/pci/pcie/Kconfig
index 788ac8df3f9d..228652a59f27 100644
--- a/drivers/pci/pcie/Kconfig
+++ b/drivers/pci/pcie/Kconfig
@@ -4,6 +4,7 @@
#
config PCIEPORTBUS
bool "PCI Express Port Bus support"
+ default y if USB4
help
This enables PCI Express Port Bus support. Users can then enable
support for Native Hot-Plug, Advanced Error Reporting, Power
@@ -15,9 +16,12 @@ config PCIEPORTBUS
config HOTPLUG_PCI_PCIE
bool "PCI Express Hotplug driver"
depends on HOTPLUG_PCI && PCIEPORTBUS
+ default y if USB4
help
- Say Y here if you have a motherboard that supports PCI Express Native
- Hotplug
+ Say Y here if you have a motherboard that supports PCIe native
+ hotplug.
+
+ Thunderbolt/USB4 PCIe tunneling depends on native PCIe hotplug.
When in doubt, say N.
^ permalink raw reply related [flat|nested] 4+ messages in thread