From 24837e75cb30cc6d8fbd86e5371f850699534975 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Thu, 11 Mar 2021 20:34:05 +0100 Subject: device/pciexp_device.c: Remove CPP guarding Let the linker do its job. This fixes building with !CONFIG_PCIEXP_HOTPLUG on some platforms. Change-Id: I46560722dcb5f1d902709e40b714ef092515b164 Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/51417 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Nico Huber --- src/device/pci_device.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'src/device/pci_device.c') diff --git a/src/device/pci_device.c b/src/device/pci_device.c index 161715fadd..d18b2f110b 100644 --- a/src/device/pci_device.c +++ b/src/device/pci_device.c @@ -882,14 +882,15 @@ static struct device_operations *get_pci_bridge_ops(struct device *dev) case PCI_EXP_TYPE_DOWNSTREAM: printk(BIOS_DEBUG, "%s subordinate bus PCI Express\n", dev_path(dev)); -#if CONFIG(PCIEXP_HOTPLUG) - u16 sltcap; - sltcap = pci_read_config16(dev, pciexpos + PCI_EXP_SLTCAP); - if (sltcap & PCI_EXP_SLTCAP_HPC) { - printk(BIOS_DEBUG, "%s hot-plug capable\n", dev_path(dev)); - return &default_pciexp_hotplug_ops_bus; + if (CONFIG(PCIEXP_HOTPLUG)) { + u16 sltcap; + sltcap = pci_read_config16(dev, pciexpos + PCI_EXP_SLTCAP); + if (sltcap & PCI_EXP_SLTCAP_HPC) { + printk(BIOS_DEBUG, "%s hot-plug capable\n", + dev_path(dev)); + return &default_pciexp_hotplug_ops_bus; + } } -#endif /* CONFIG(PCIEXP_HOTPLUG) */ return &default_pciexp_ops_bus; case PCI_EXP_TYPE_PCI_BRIDGE: printk(BIOS_DEBUG, "%s subordinate PCI\n", -- cgit v1.2.3