Attention is currently required from: Eric Lai, Felix Held, Kapil Porwal, Kyösti Mälkki, Lance Zhao, Lean Sheng Tan, Nico Huber, Subrata Banik, Tim Wawrzynczak.
Hello Eric Lai, Felix Held, Kapil Porwal, Kyösti Mälkki, Lance Zhao, Lean Sheng Tan, Nico Huber, Subrata Banik, Tim Wawrzynczak, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/76000?usp=email
to look at the new patch set (#8).
The following approvals got outdated and were removed: Code-Review+1 by Eric Lai, Code-Review+1 by Nico Huber, Code-Review+1 by Tim Wawrzynczak, Code-Review+2 by Kapil Porwal, Code-Review+2 by Kyösti Mälkki, Code-Review+2 by Lean Sheng Tan, Verified+1 by build bot (Jenkins)
Change subject: acpi.c: Swap XSDT and RSDT for adding/finding tables ......................................................................
acpi.c: Swap XSDT and RSDT for adding/finding tables
If ACPI is above 4G it's not possible to have a valid RSDT pointer in RSDP, therefore swap RSDT and XSDT. Both are always generated on x86. On other architectures RSDT is often skipped, e.g. aarch64. On top of that the OS looks at XSDT first. So unconditionally using XSDT and not RSDT is fine.
This also deal with the ACPI pointer being above 4G. This currently never happens with x86 platforms.
Signed-off-by: Arthur Heymans arthur@aheymans.xyz Change-Id: I6588676186faa896b6076f871d7f8f633db21e70 --- M src/acpi/acpi.c 1 file changed, 34 insertions(+), 34 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/76000/8