[coreboot-gerrit] Change in coreboot[master]: driver/pc80/tpm: Runtime generate ACPI table for TPM driver

[Tobias Diedrich (Code Review)]

Tobias Diedrich has posted comments on this change. ( https://review.coreboot.org/17425 )

Change subject: driver/pc80/tpm: Runtime generate ACPI table for TPM driver
......................................................................

Patch Set 12:
> (1 comment)
>
> In ticket #102 [1], it’s claimed that this commit introduced a
> regression.
>
> The change-set I6e65b37507c111af1c0c56822327d5497f58f085 also
> mentions this commit.

"ACPI Error: [_SB_.PCI0.LPCB] Namespace lookup failure, AE_NOT_FOUND (20160422/dswload-210)"
sounds like  the default path _SB.PCI0.LPCB does not match the actual path on the board?
e.g. in https://review.coreboot.org/#/c/18402/ I added the plumbing to make the path correct for AMD SB800 (for pcengines APU1).
This would probably need a similar patch for the affected board or alternatively make the default "_SB.PCI0.LPCB" configurable in Kconfig instead of hardcoded.