[coreboot-gerrit] Change in coreboot[master]: src/arch/x86/acpi.c: Change TPM2 ACPI Table to support CRB

[Christian Walter (Code Review)]

Christian Walter has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34333 )

Change subject: src/arch/x86/acpi.c: Change TPM2 ACPI Table to support CRB
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34333/7/src/arch/x86/acpi.c 
File src/arch/x86/acpi.c:

https://review.coreboot.org/c/coreboot/+/34333/7/src/arch/x86/acpi.c@90 
PS7, Line 90: 		printk(BIOS_ERR,
            : 		       "ACPI: Error: Could not add ACPI table, "
            : 		       "too many tables.\n");
> printk(BIOS_ERR, "ACPI: Error: Could not add ACPI table, too many tables. […]

ACK

https://review.coreboot.org/c/coreboot/+/34333/7/src/arch/x86/acpi.c@388 
PS7, Line 388: 0xfed40040
> TPM32(0x40)?

Ack