Commit de2cdf11 authored by Ilpo Järvinen's avatar Ilpo Järvinen Committed by Bjorn Helgaas
Browse files

PCI: cpqphp: Use define to read class/revision dword 

Replace literal 0x08 with PCI_CLASS_REVISION.

Link: https://lore.kernel.org/r/20241022091140.3504-4-ilpo.jarvinen@linux.intel.com


Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
parent 752430d1

drivers/pci/hotplug/cpqphp_pci.c

+2 −2
Original line number Diff line number Diff line
@@ -199,7 +199,7 @@ static int PCI_ScanBusForNonBridge(struct controller *ctrl, u8 bus_num, u8 *dev_ 
		/* Scan for access first */

		if (!pci_bus_read_dev_vendor_id(ctrl->pci_bus, tdevice, &work, 0))

			continue;

		ret = pci_bus_read_config_dword(ctrl->pci_bus, tdevice, 0x08, &work);

		ret = pci_bus_read_config_dword(ctrl->pci_bus, tdevice, PCI_CLASS_REVISION, &work);

		if (ret)

			continue;

		dbg("Looking for nonbridge bus_num %d dev_num %d\n", bus_num, tdevice);
@@ -214,7 +214,7 @@ static int PCI_ScanBusForNonBridge(struct controller *ctrl, u8 bus_num, u8 *dev_ 
		/* Scan for access first */

		if (!pci_bus_read_dev_vendor_id(ctrl->pci_bus, tdevice, &work, 0))

			continue;

		ret = pci_bus_read_config_dword(ctrl->pci_bus, tdevice, 0x08, &work);

		ret = pci_bus_read_config_dword(ctrl->pci_bus, tdevice, PCI_CLASS_REVISION, &work);

		if (ret)

			continue;

		dbg("Looking for bridge bus_num %d dev_num %d\n", bus_num, tdevice);