Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62493 )
Change subject: drivers/pc80/tpm/tis.c: Use fixed TPM ACPI path ......................................................................
drivers/pc80/tpm/tis.c: Use fixed TPM ACPI path
Windows 11 installer expects the TPM to reside under \_SB_.PCI0 in ACPI device hierarchy, otherwise the TPM is not detected. Hardcode the path to fix the issue.
TEST=Boot Windows 11 on Clevo NV41MZ and see the TPM is detected correctly
TEST=Boot Ubuntu 20.04 on Clevo NV41MZ and see the TPM is detected correctly
Signed-off-by: Michał Żygowski michal.zygowski@3mdeb.com Signed-off-by: Michał Kopeć michal.kopec@3mdeb.com Change-Id: If0b3136e3eb8eb1bb132132a5f3a7034bdd3b424 Reviewed-on: https://review.coreboot.org/c/coreboot/+/62493 Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/drivers/pc80/tpm/tis.c 1 file changed, 2 insertions(+), 6 deletions(-)
Approvals: build bot (Jenkins): Verified Michał Żygowski: Looks good to me, approved
diff --git a/src/drivers/pc80/tpm/tis.c b/src/drivers/pc80/tpm/tis.c index 455bcf9..cf46f76 100644 --- a/src/drivers/pc80/tpm/tis.c +++ b/src/drivers/pc80/tpm/tis.c @@ -779,12 +779,8 @@ #if CONFIG(HAVE_ACPI_TABLES) static void lpc_tpm_fill_ssdt(const struct device *dev) { - const char *path = acpi_device_path(dev->bus->dev); - - if (!path) { - path = "\_SB_.PCI0.LPCB"; - printk(BIOS_DEBUG, "Using default TPM ACPI path: '%s'\n", path); - } + /* Windows 11 requires the following path for TPM to be detected */ + const char *path = "\_SB_.PCI0";
/* Device */ acpigen_write_scope(path);