drivers/tpm: Add firmware-power-managed DSD property

Introduce firmware-power-managed DSD ACPI property for TPM devices.
This property can be checked by the kernel TPM driver to override how
the TPM power states are managed. This is a tri-state flag, true,
false, or unset. So an enum used to keep the flag is unset by default.

When firmware-power-managed is true, the kernel driver will not send a
shutdown during s2idle/s0i3 suspend.

BUG=b:200578885
BRANCH=None
TEST=TPM shutdown is triggered on s0ix suspend on guybrush with patched
kernel

Change-Id: Ia48ead856fc0c6e637a2e07a5ecc58423f599c5b
Signed-off-by: Rob Barnes <robbarnes@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59479
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
Rob Barnes 2021-11-18 14:12:46 -07:00 committed by Karthik Ramasubramanian
parent b35acf9210
commit 5e3c454fbb
2 changed files with 23 additions and 0 deletions

View File

@ -11,6 +11,7 @@
static void i2c_tpm_fill_ssdt(const struct device *dev) static void i2c_tpm_fill_ssdt(const struct device *dev)
{ {
struct acpi_dp *dsd;
struct drivers_i2c_tpm_config *config = dev->chip_info; struct drivers_i2c_tpm_config *config = dev->chip_info;
const char *scope = acpi_device_scope(dev); const char *scope = acpi_device_scope(dev);
struct acpi_i2c i2c = { struct acpi_i2c i2c = {
@ -47,6 +48,21 @@ static void i2c_tpm_fill_ssdt(const struct device *dev)
acpigen_write_resourcetemplate_footer(); acpigen_write_resourcetemplate_footer();
/* _DSD, Device-Specific Data */
dsd = acpi_dp_new_table("_DSD");
switch (config->power_managed_mode) {
case TPM_FIRMWARE_POWER_MANAGED:
acpi_dp_add_integer(dsd, "firmware-power-managed", 1);
break;
case TPM_KERNEL_POWER_MANAGED:
acpi_dp_add_integer(dsd, "firmware-power-managed", 0);
break;
case TPM_DEFAULT_POWER_MANAGED:
default:
/* Leave firmware-power-managed unset */
}
acpi_dp_write(dsd);
acpigen_pop_len(); /* Device */ acpigen_pop_len(); /* Device */
acpigen_pop_len(); /* Scope */ acpigen_pop_len(); /* Scope */

View File

@ -3,6 +3,12 @@
#include <acpi/acpi_device.h> #include <acpi/acpi_device.h>
#include <device/i2c_simple.h> #include <device/i2c_simple.h>
enum tpm_power_managed_mode {
TPM_DEFAULT_POWER_MANAGED = 0,
TPM_FIRMWARE_POWER_MANAGED,
TPM_KERNEL_POWER_MANAGED,
};
struct drivers_i2c_tpm_config { struct drivers_i2c_tpm_config {
const char *hid; /* ACPI _HID (required) */ const char *hid; /* ACPI _HID (required) */
const char *desc; /* Device Description */ const char *desc; /* Device Description */
@ -10,4 +16,5 @@ struct drivers_i2c_tpm_config {
enum i2c_speed speed; /* Bus speed in Hz, default is I2C_SPEED_FAST */ enum i2c_speed speed; /* Bus speed in Hz, default is I2C_SPEED_FAST */
struct acpi_irq irq; /* Interrupt */ struct acpi_irq irq; /* Interrupt */
struct acpi_gpio irq_gpio; /* GPIO interrupt */ struct acpi_gpio irq_gpio; /* GPIO interrupt */
enum tpm_power_managed_mode power_managed_mode; /* TPM power managed mode */
}; };