diff options
author | Keith Hui <buurin@gmail.com> | 2020-04-20 21:21:33 -0400 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-07-06 06:26:55 +0000 |
commit | 8ba85deb8f249a17ffa3e53b4c772ec783a3bdb8 (patch) | |
tree | 7157dfe480d141c9706e5d50a429aa9334fe1abd | |
parent | 562279e6caf6609c599ec97f65947c0e94a1085f (diff) | |
download | coreboot-8ba85deb8f249a17ffa3e53b4c772ec783a3bdb8.tar.gz coreboot-8ba85deb8f249a17ffa3e53b4c772ec783a3bdb8.tar.bz2 coreboot-8ba85deb8f249a17ffa3e53b4c772ec783a3bdb8.zip |
nb/intel/i440bx: Refactor ACPI code
Bring DRB7 OpRegion and top-of-memory indicator inside NB device.
Use more concise ASL 2.0 syntax for TOM calculations.
Change-Id: I2c74ef30a9bb48e02154f963b1ca3a4f5f3004df
Signed-off-by: Keith Hui <buurin@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41049
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rw-r--r-- | src/northbridge/intel/i440bx/acpi/sb_pci0_crs.asl | 29 |
1 files changed, 13 insertions, 16 deletions
diff --git a/src/northbridge/intel/i440bx/acpi/sb_pci0_crs.asl b/src/northbridge/intel/i440bx/acpi/sb_pci0_crs.asl index fe66f3931142..ce71aedaeef7 100644 --- a/src/northbridge/intel/i440bx/acpi/sb_pci0_crs.asl +++ b/src/northbridge/intel/i440bx/acpi/sb_pci0_crs.asl @@ -1,21 +1,19 @@ /* SPDX-License-Identifier: GPL-2.0-only */ -/* i440bx Northbridge */ +/* i440bx Northbridge resources that sits on \_SB.PCI0 */ Device (NB) { Name(_ADR, 0x00000000) OperationRegion(PCIC, PCI_Config, 0x00, 0x100) -} - -Field (NB.PCIC, AnyAcc, NoLock, Preserve) -{ - Offset (0x67), // DRB7 - DRB7, 8, -} - -Method(TOM1, 0) { - /* Multiply by 8MB to get TOM */ - Return(ShiftLeft(DRB7, 23)) + Field (PCIC, ByteAcc, NoLock, Preserve) + { + Offset (0x67), // DRB7 + DRB7, 8, + } + Method(TOM1, 0) { + /* Multiply by 8MB to get TOM */ + Return(DRB7 << 23) + } } Method(_CRS, 0) { @@ -60,10 +58,9 @@ Method(_CRS, 0) { * 32bit (0x00000000 - TOM1) will wrap and give the same * result as 64bit (0x100000000 - TOM1). */ - Store(TOM1, MM1B) - ShiftLeft(0x10000000, 4, Local0) - Subtract(Local0, TOM1, Local0) - Store(Local0, MM1L) + MM1B = \_SB.PCI0.NB.TOM1 + Local0 = 0x10000000 << 4 + MM1L = Local0 - MM1B Return(TMP) } |