Re: [PATCH v4 0/2] add hotplug depedency info

[Bjorn Helgaas <helgaas@kernel.org>]

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.