summaryrefslogtreecommitdiff
path: root/src/soc/intel/broadwell
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2018-12-29 08:04:16 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2019-01-03 12:16:33 +0000
commite16c9df4542a40c176225e9db034e750eb957d5d (patch)
tree11b04c93bb55fd5b2e3eed47c131cc9f5a821ff2 /src/soc/intel/broadwell
parente2c653e049d0c4db6c2712658eaa5838dac73ee9 (diff)
downloadcoreboot-e16c9df4542a40c176225e9db034e750eb957d5d.tar.xz
soc/intel: Fix bad uses of __SIMPLE_DEVICE__
Cases of *dev = PCI_DEV(b,d,f) are invalid. Not caught because files only build with __SIMPLE_DEVICE__ defined. Remove cases of testing __SIMPLE_DEVICE__ in files that are not build for ramstage. Change-Id: If10a0efa187c9b1d9a5577008aa46f050f0aa309 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/30497 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'src/soc/intel/broadwell')
-rw-r--r--src/soc/intel/broadwell/smihandler.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/src/soc/intel/broadwell/smihandler.c b/src/soc/intel/broadwell/smihandler.c
index 64a315d30e..777c0d20a1 100644
--- a/src/soc/intel/broadwell/smihandler.c
+++ b/src/soc/intel/broadwell/smihandler.c
@@ -80,12 +80,8 @@ static void busmaster_disable_on_bus(int bus)
for (slot = 0; slot < 0x20; slot++) {
for (func = 0; func < 8; func++) {
u32 reg32;
-#if defined(__SIMPLE_DEVICE__)
- pci_devfn_t dev = PCI_DEV(bus, slot, func);
-#else
- struct device *dev = PCI_DEV(bus, slot, func);
-#endif
+ pci_devfn_t dev = PCI_DEV(bus, slot, func);
val = pci_read_config32(dev, PCI_VENDOR_ID);
if (val == 0xffffffff || val == 0x00000000 ||