]> www.infradead.org Git - users/dwmw2/linux.git/commit
PCI: cpqphp: Simplify PCI_ScanBusForNonBridge()
authorIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Tue, 22 Oct 2024 09:11:40 +0000 (12:11 +0300)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 22 Oct 2024 15:06:59 +0000 (10:06 -0500)
commit5a02413a4586a7cfa10b7380377138e66db9df4b
tree80396455e0ec193434587cb3711f9694264c3dc7
parentde2cdf110a39a660e810979e480761fed9842436
PCI: cpqphp: Simplify PCI_ScanBusForNonBridge()

PCI_ScanBusForNonBridge() has two loops, one to search for non-bridges and
a second to look for bridges. The second loop has hints in a debug print it
should do recursion for buses underneath the bridge, but no recursion is
attempted.

Since the second loop is quite useless in its current form, just eliminate
it. This code hasn't been touched for very long time so either it's unused
or the missing parts are not important enough for anyone to attempt to add
them.

Leave only a warning print and comment about the missing recursion for the
unlikely case that somebody comes across the lack of functionality. In any
case, search whether an endpoint exists downstream of a bridge sounds
generic enough to belong to core so if the functionality is to be extended
it should probably be moved into PCI core.

Link: https://lore.kernel.org/r/20241022091140.3504-5-ilpo.jarvinen@linux.intel.com
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/hotplug/cpqphp_pci.c