Attention is currently required from: Arthur Heymans, Christian Walter, Jérémy Compostella, Patrick Rudolph, Shuo Liu.
Shuo Liu has uploaded a new patch set (#7) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/81276?usp=email )
The following approvals got outdated and were removed: Code-Review+1 by Arthur Heymans, Code-Review+1 by Jérémy Compostella, Code-Review+2 by Shuo Liu, Verified+1 by build bot (Jenkins)
Change subject: drivers/pc80/tpm: Support Xeon-SP ......................................................................
drivers/pc80/tpm: Support Xeon-SP
On Xeon-SP the first PCI domain isn't named PCI0 and thus the TPM ACPI code fails to evaluate. Instead of hardcoding an ACPI path use the domain name set in the SoC chip_ops.
TEST=Build and boot on intel/archercity CRB
Below issue is fixed,
ACPI BIOS Error (bug): Could not resolve symbol [_SB.PCI0], AE_NOT_FOUND (20200925/dswload2-163) ACPI Error: AE_NOT_FOUND, During name lookup/catalog (20200925/psobject-221) ACPI: Skipping parse of AML opcode: OpcodeName unavailable (0x0010)
Change-Id: I6130b937ec81bbde78d2619eb859ecf354fc3854 Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/drivers/pc80/tpm/tis.c 1 file changed, 12 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/76/81276/7