summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2020-11-12 21:09:56 +0100
committerArthur Heymans <arthur@aheymans.xyz>2020-11-20 10:18:25 +0000
commit054026cdb8dc4f6a2cebf63ba5f64be738f324ad (patch)
tree76e59f5f8ead1bda54ceffcc78a16de4e781a096
parenta1f65bed10d41e41c684cd5e109b9883560e36f2 (diff)
downloadcoreboot-054026cdb8dc4f6a2cebf63ba5f64be738f324ad.tar.xz
soc/intel/apollolake: use P2SB function to generate DMAR IOAPIC
Change-Id: If088d5bf701310e54b14965145229627f3a50417 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/47535 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
-rw-r--r--src/soc/intel/apollolake/acpi.c12
1 files changed, 2 insertions, 10 deletions
diff --git a/src/soc/intel/apollolake/acpi.c b/src/soc/intel/apollolake/acpi.c
index 8609c7c8e2..d0e7a73a3f 100644
--- a/src/soc/intel/apollolake/acpi.c
+++ b/src/soc/intel/apollolake/acpi.c
@@ -185,22 +185,14 @@ static unsigned long soc_fill_dmar(unsigned long current)
/* DEFVTBAR has to be set and enabled. */
if (defvtbar && defvten) {
tmp = current;
- /*
- * P2SB may already be hidden. There's no clear rule, when.
- * It is needed to get bus, device and function for IOAPIC and
- * HPET device which is stored in P2SB device. So unhide it to
- * get the info and hide it again when done.
- */
- p2sb_unhide();
- struct device *p2sb_dev = pcidev_path_on_root(PCH_DEVFN_P2SB);
- uint16_t ibdf = pci_read_config16(p2sb_dev, PCH_P2SB_IBDF);
+ union p2sb_bdf ibdf = p2sb_get_ioapic_bdf();
union p2sb_bdf hbdf = p2sb_get_hpet_bdf();
p2sb_hide();
current += acpi_create_dmar_drhd(current,
DRHD_INCLUDE_PCI_ALL, 0, defvtbar);
current += acpi_create_dmar_ds_ioapic(current,
- 2, ibdf >> 8, PCI_SLOT(ibdf), PCI_FUNC(ibdf));
+ 2, ibdf.bus, ibdf.dev, ibdf.fn);
current += acpi_create_dmar_ds_msi_hpet(current,
0, hbdf.bus, hbdf.dev, hbdf.fn);
acpi_dmar_drhd_fixup(tmp, current);