summaryrefslogtreecommitdiffstats
path: root/src/soc/intel/skylake/elog.c
diff options
context:
space:
mode:
authorTim Wawrzynczak <twawrzynczak@chromium.org>2021-04-30 13:47:04 -0600
committerTim Wawrzynczak <twawrzynczak@chromium.org>2021-05-03 16:28:53 +0000
commitb1623f23c0095a7dce6c874271f977f197f4949e (patch)
treed810c99efbcb519ae4c727dab02816719cf611f3 /src/soc/intel/skylake/elog.c
parent93982c3a6ede0bdb8dd0e94597e4de7b09406947 (diff)
downloadcoreboot-b1623f23c0095a7dce6c874271f977f197f4949e.tar.gz
coreboot-b1623f23c0095a7dce6c874271f977f197f4949e.tar.bz2
coreboot-b1623f23c0095a7dce6c874271f977f197f4949e.zip
soc/intel/*: Update data types for variables holding PCH_DEVFN_* macros
The usage of `pci_devfn_t` here is misleading, as these intentionally store the `PCH_DEVFN_*` macros so they can be used across `smm` and `ramstage` without requiring the device model. Update to `unsigned int` instead, as `pci_devfn_t` implies the data is an MMCONF-compatible PCI devfn offset. Change-Id: Ic8880de984e6eceda4cbe141e118f3a5fdd672a2 Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/52808 Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/skylake/elog.c')
-rw-r--r--src/soc/intel/skylake/elog.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/skylake/elog.c b/src/soc/intel/skylake/elog.c
index c6be670332ca..1c8051735143 100644
--- a/src/soc/intel/skylake/elog.c
+++ b/src/soc/intel/skylake/elog.c
@@ -26,7 +26,7 @@ static void pch_log_gpio_gpe(u32 gpe0_sts, u32 gpe0_en, int start)
}
struct pme_status_info {
- pci_devfn_t devfn;
+ unsigned int devfn;
uint8_t reg_offset;
uint32_t elog_event;
};