northbridge/intel/haswell: Mask lower 20 bits of TOLUD and TOLM in hostbridge.asl

Lower 20bits of TOLUD and TOLM registers include 19 reserved bits and
1 lock bit. If lock bit is set, then systemagent.asl would end up
reporting the base address of low MMIO incorrectly i.e. off by 1.

This change masks the lower 20 bits of TOLUD and TOM registers when
exposing it in the ACPI tables to ensure that the base address of low
MMIO region is reported correctly.

Change-Id: I1fb52a42e84130d973e0970024e263f443aa0b89
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41977
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
This commit is contained in:
Furquan Shaikh 2020-06-01 13:27:16 -07:00 committed by Patrick Georgi
parent 181e2d445c
commit 506479d2a8
1 changed files with 6 additions and 2 deletions

View File

@ -425,13 +425,17 @@ Method (_CRS, 0, Serialized)
// Fix up PCI memory region // Fix up PCI memory region
// Start with Top of Lower Usable DRAM // Start with Top of Lower Usable DRAM
Local0 = ^MCHC.TLUD // Lower 20 bits of TOLUD register need to be masked since they contain lock and
// reserved bits.
Local0 = ^MCHC.TLUD & (0xfff << 20)
Local1 = ^MCHC.MEBA Local1 = ^MCHC.MEBA
// Check if ME base is equal // Check if ME base is equal
If (Local0 == Local1) { If (Local0 == Local1) {
// Use Top Of Memory instead // Use Top Of Memory instead
Local0 = ^MCHC.TOM // Lower 20 bits of TOM register need to be masked since they contain lock and
// reserved bits.
Local0 = ^MCHC.TOM & (0x7ffff << 20)
} }
PMIN = Local0 PMIN = Local0