From c0733e1639bc97cd1774c556edd6bb6526876529 Mon Sep 17 00:00:00 2001 From: Kyösti Mälkki Date: Sun, 14 Feb 2021 06:58:39 +0200 Subject: ACPI: Use common OperationRegion for PCI_MMCONF MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: Iadb4c3c77ecda4df8e48415d246e769ede2ce86d Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/c/coreboot/+/50648 Reviewed-by: Angel Pons Tested-by: build bot (Jenkins) --- src/soc/amd/stoneyridge/acpi/pci_int.asl | 4 +--- src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl | 3 +-- 2 files changed, 2 insertions(+), 5 deletions(-) (limited to 'src/soc') diff --git a/src/soc/amd/stoneyridge/acpi/pci_int.asl b/src/soc/amd/stoneyridge/acpi/pci_int.asl index 0f1feef05a..e905c9738c 100644 --- a/src/soc/amd/stoneyridge/acpi/pci_int.asl +++ b/src/soc/amd/stoneyridge/acpi/pci_int.asl @@ -1,8 +1,6 @@ /* SPDX-License-Identifier: GPL-2.0-only */ - /* PCIe Configuration Space for CONFIG_MMCONF_BUS_NUMBER busses */ - OperationRegion(PCFG, SystemMemory, PCBA, PCLN) /* Each bus consumes 1MB */ - Field(PCFG, ByteAcc, NoLock, Preserve) { + Field(PCFG, ByteAcc, NoLock, Preserve) { /* Byte offsets are computed using the following technique: * ((bus number + 1) * ((device number * 8) * 4096)) + register offset * The 8 comes from 8 functions per device, and 4096 bytes per function config space diff --git a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl index 40e882c004..4580b772f0 100644 --- a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl +++ b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl @@ -284,8 +284,7 @@ Field( SMIC, ByteAcc, NoLock, Preserve) { PGA3, 8 , } -OperationRegion(FCFG, SystemMemory, PCBA, 0x01000000) -Field(FCFG, DwordAcc, NoLock, Preserve) +Field(PCFG, DwordAcc, NoLock, Preserve) { /* XHCI */ Offset(0x00080010), /* Base address */ -- cgit v1.2.3