Commit 197e0da1 authored by Ilpo Järvinen's avatar Ilpo Järvinen Committed by Bjorn Helgaas

x86/pci: Use PCI_HEADER_TYPE_* instead of literals

Replace 0x7f and 0x80 literals with PCI_HEADER_TYPE_* defines.

Link: https://lore.kernel.org/r/20231124090919.23687-1-ilpo.jarvinen@linux.intel.comSigned-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
parent e585a37e
...@@ -259,10 +259,9 @@ static u32 __init search_agp_bridge(u32 *order, int *valid_agp) ...@@ -259,10 +259,9 @@ static u32 __init search_agp_bridge(u32 *order, int *valid_agp)
order); order);
} }
/* No multi-function device? */
type = read_pci_config_byte(bus, slot, func, type = read_pci_config_byte(bus, slot, func,
PCI_HEADER_TYPE); PCI_HEADER_TYPE);
if (!(type & 0x80)) if (!(type & PCI_HEADER_TYPE_MFD))
break; break;
} }
} }
......
...@@ -779,13 +779,13 @@ static int __init check_dev_quirk(int num, int slot, int func) ...@@ -779,13 +779,13 @@ static int __init check_dev_quirk(int num, int slot, int func)
type = read_pci_config_byte(num, slot, func, type = read_pci_config_byte(num, slot, func,
PCI_HEADER_TYPE); PCI_HEADER_TYPE);
if ((type & 0x7f) == PCI_HEADER_TYPE_BRIDGE) { if ((type & PCI_HEADER_TYPE_MASK) == PCI_HEADER_TYPE_BRIDGE) {
sec = read_pci_config_byte(num, slot, func, PCI_SECONDARY_BUS); sec = read_pci_config_byte(num, slot, func, PCI_SECONDARY_BUS);
if (sec > num) if (sec > num)
early_pci_scan_bus(sec); early_pci_scan_bus(sec);
} }
if (!(type & 0x80)) if (!(type & PCI_HEADER_TYPE_MFD))
return -1; return -1;
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment