Re: [PATCH v4 7/9] PCI: Make pci_match_one_device() match on ID instead of device

On Fri Jul 24, 2026 at 11:29 PM BST, Bjorn Helgaas wrote:

On Thu, Jul 23, 2026 at 11:00:46PM +0100, Gary Guo wrote:

quoted

PCI dynamic ID needs to match IDs against a new ID to see if it already
exists. Existing APIs can only match IDs against devices, causing the
dynamic ID insertion code to create a temporary device only for matching
purpose. Rename pci_match_one_device() to pci_match_one_id() so it can be
used for this purpose instead; add a pci_id_from_device() helper to make it
easy to convert users.

s/causing the ... code to create/so the ... code creates/
s/purpose/purposes/

quoted

Similarly, convert pci_match_id() to do_pci_match_id(). But keep the
existing API because there are many users.

Reviewed-by: Danilo Krummrich <dakr@kernel.org>
Signed-off-by: Gary Guo <gary@garyguo.net>

Reviewed-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Bjorn Helgaas <bhelgaas@google.com>

What's the merge strategy?  If these all need to be merged together,
feel free to include the PCI patches whereever you merge the series.

I think the series should be merged together, as I consider the driver changes
included in this series being "fixing the users".

The series should probably be taken via the PCI tree. I suppose one can also
argue that the lifetime issue with device IDs isn't unique to PCI bus so this
can also be considered driver-core material, but given that Greg has already
taken the USB fixes via the USB tree the only unfixed bus left is PCI.

So unless Danilo thinks otherwise I'd suggest the series be taken as a whole via
PCI.

Best,
Gary

quoted

---
 drivers/pci/pci-driver.c | 38 ++++++++++++++++++++++++++++----------
 drivers/pci/pci.h        | 43 +++++++++++++++++++++++++++++++------------
 drivers/pci/search.c     |  8 +++++---
 3 files changed, 64 insertions(+), 25 deletions(-)