Re: [PATCH v3 3/7] PCI: Drop pci_is_thunderbolt_attached()

From: Ilpo Järvinen
Date: Thu Nov 16 2023 - 07:52:08 EST


On Tue, 14 Nov 2023, Mario Limonciello wrote:

> All callers have switched to dev_is_removable() for detecting
> hotpluggable PCIe devices.
>
> Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx>
> ---
> v2->v3:
> * No changes
> ---
> include/linux/pci.h | 22 ----------------------
> 1 file changed, 22 deletions(-)
>
> diff --git a/include/linux/pci.h b/include/linux/pci.h
> index 60ca768bc867..1fbca2bd92e8 100644
> --- a/include/linux/pci.h
> +++ b/include/linux/pci.h
> @@ -2645,28 +2645,6 @@ static inline bool pci_ari_enabled(struct pci_bus *bus)
> return bus->self && bus->self->ari_enabled;
> }
>
> -/**
> - * pci_is_thunderbolt_attached - whether device is on a Thunderbolt daisy chain
> - * @pdev: PCI device to check
> - *
> - * Walk upwards from @pdev and check for each encountered bridge if it's part
> - * of a Thunderbolt controller. Reaching the host bridge means @pdev is not
> - * Thunderbolt-attached. (But rather soldered to the mainboard usually.)
> - */
> -static inline bool pci_is_thunderbolt_attached(struct pci_dev *pdev)
> -{
> - struct pci_dev *parent = pdev;
> -
> - if (pdev->is_thunderbolt)
> - return true;
> -
> - while ((parent = pci_upstream_bridge(parent)))
> - if (parent->is_thunderbolt)
> - return true;
> -
> - return false;
> -}
> -
> #if defined(CONFIG_PCIEPORTBUS) || defined(CONFIG_EEH)
> void pci_uevent_ers(struct pci_dev *pdev, enum pci_ers_result err_type);
> #endif
>

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>

--
i.