From 120f1128442c11ef3032e382c8b2377380fb3cbc Mon Sep 17 00:00:00 2001 From: Duncan Laurie Date: Mon, 19 Sep 2016 19:20:06 -0700 Subject: Revert "drivers/i2c/tpm/cr50: Support interrupts for status" This reverts commit a5e419c51187d24818f056327746a18676fe3a20. --- src/drivers/i2c/tpm/Kconfig | 12 +--------- src/drivers/i2c/tpm/cr50.c | 57 +++++++-------------------------------------- src/drivers/i2c/tpm/tpm.h | 1 - 3 files changed, 9 insertions(+), 61 deletions(-) (limited to 'src/drivers/i2c/tpm') diff --git a/src/drivers/i2c/tpm/Kconfig b/src/drivers/i2c/tpm/Kconfig index a5ab0771c4..db77b956b9 100644 --- a/src/drivers/i2c/tpm/Kconfig +++ b/src/drivers/i2c/tpm/Kconfig @@ -2,14 +2,9 @@ config I2C_TPM bool "I2C TPM" depends on TPM || TPM2 -config MAINBOARD_HAS_I2C_TPM_CR50 - bool - default n - choice prompt "I2C TPM Driver" - default I2C_TPM_CR50 if MAINBOARD_HAS_I2C_TPM_CR50 - default I2C_TPM_GENERIC if !MAINBOARD_HAS_I2C_TPM_CR50 + default I2C_TPM_GENERIC depends on I2C_TPM config I2C_TPM_GENERIC @@ -30,11 +25,6 @@ config DRIVER_TPM_I2C_ADDR default 2 # FIXME, workaround for Kconfig BS depends on I2C_TPM -config DRIVER_TPM_I2C_IRQ - int "IRQ or GPE to use for TPM interrupt" - default -1 - depends on I2C_TPM - config DRIVER_I2C_TPM_ACPI bool "Generate I2C TPM ACPI device" default y if ARCH_X86 && I2C_TPM diff --git a/src/drivers/i2c/tpm/cr50.c b/src/drivers/i2c/tpm/cr50.c index d3c0efa2ac..37f0881980 100644 --- a/src/drivers/i2c/tpm/cr50.c +++ b/src/drivers/i2c/tpm/cr50.c @@ -41,9 +41,6 @@ #include #include "tpm.h" -#if IS_ENABLED(CONFIG_ARCH_X86) -#include -#endif #define CR50_MAX_BUFSIZE 63 #define CR50_TIMEOUT_LONG_MS 2000 /* Long timeout while waiting for TPM */ @@ -58,26 +55,6 @@ struct tpm_inf_dev { static struct tpm_inf_dev g_tpm_dev CAR_GLOBAL; -/* Wait for interrupt to indicate the TPM is ready */ -static int cr50_i2c_wait_tpm_ready(struct tpm_chip *chip) -{ - struct stopwatch sw; - - if (!chip->vendor.irq_status) { - /* Fixed delay if interrupt not supported */ - mdelay(CR50_TIMEOUT_SHORT_MS); - return 0; - } - - stopwatch_init_msecs_expire(&sw, 10 * CR50_TIMEOUT_SHORT_MS); - - while (!chip->vendor.irq_status(chip->vendor.irq)) - if (stopwatch_expired(&sw)) - return -1; - - return 0; -} - /* * cr50_i2c_read() - read from TPM register * @@ -100,10 +77,6 @@ static int cr50_i2c_read(struct tpm_chip *chip, uint8_t addr, if (tpm_dev->addr == 0) return -1; - /* Clear interrupt before starting transaction */ - if (chip->vendor.irq_status) - chip->vendor.irq_status(chip->vendor.irq); - /* Send the register address byte to the TPM */ if (i2c_write_raw(tpm_dev->bus, tpm_dev->addr, &addr, 1)) { printk(BIOS_ERR, "%s: Address write failed\n", __func__); @@ -111,8 +84,7 @@ static int cr50_i2c_read(struct tpm_chip *chip, uint8_t addr, } /* Wait for TPM to be ready with response data */ - if (cr50_i2c_wait_tpm_ready(chip) < 0) - return -1; + mdelay(CR50_TIMEOUT_SHORT_MS); /* Read response data from the TPM */ if (i2c_read_raw(tpm_dev->bus, tpm_dev->addr, buffer, len)) { @@ -151,10 +123,6 @@ static int cr50_i2c_write(struct tpm_chip *chip, tpm_dev->buf[0] = addr; memcpy(tpm_dev->buf + 1, buffer, len); - /* Clear interrupt before starting transaction */ - if (chip->vendor.irq_status) - chip->vendor.irq_status(chip->vendor.irq); - /* Send write request buffer with address */ if (i2c_write_raw(tpm_dev->bus, tpm_dev->addr, tpm_dev->buf, len + 1)) { printk(BIOS_ERR, "%s: Error writing to TPM\n", __func__); @@ -162,7 +130,9 @@ static int cr50_i2c_write(struct tpm_chip *chip, } /* Wait for TPM to be ready */ - return cr50_i2c_wait_tpm_ready(chip); + mdelay(CR50_TIMEOUT_SHORT_MS); + + return 0; } static int check_locality(struct tpm_chip *chip, int loc) @@ -410,7 +380,6 @@ static void cr50_vendor_init(struct tpm_chip *chip) chip->vendor.recv = &cr50_i2c_tis_recv; chip->vendor.send = &cr50_i2c_tis_send; chip->vendor.cancel = &cr50_i2c_tis_ready; - chip->vendor.irq = CONFIG_DRIVER_TPM_I2C_IRQ; } int tpm_vendor_probe(unsigned bus, uint32_t addr) @@ -465,18 +434,8 @@ int tpm_vendor_init(struct tpm_chip *chip, unsigned bus, uint32_t dev_addr) cr50_vendor_init(chip); - /* - * Interrupts are not supported this early in firmware, - * use use an arch-specific method to query for interrupt status. - */ - if (chip->vendor.irq > 0) { - if (IS_ENABLED(CONFIG_ARCH_X86)) { - /* Query GPE status for interrupt */ - chip->vendor.irq_status = &acpi_get_gpe; - } else { - chip->vendor.irq = -1; - } - } + /* Disable interrupts (not supported) */ + chip->vendor.irq = 0; if (request_locality(chip, 0) != 0) return -1; @@ -490,8 +449,8 @@ int tpm_vendor_init(struct tpm_chip *chip, unsigned bus, uint32_t dev_addr) goto out_err; } - printk(BIOS_DEBUG, "cr50 TPM 2.0 (i2c %u:0x%02x irq %d id 0x%x)\n", - bus, dev_addr, chip->vendor.irq, vendor >> 16); + printk(BIOS_DEBUG, "cr50 TPM %u:%02x (device-id 0x%X)\n", + tpm_dev->bus, tpm_dev->addr, vendor >> 16); chip->is_open = 1; return 0; diff --git a/src/drivers/i2c/tpm/tpm.h b/src/drivers/i2c/tpm/tpm.h index b6dda1af47..048c848de2 100644 --- a/src/drivers/i2c/tpm/tpm.h +++ b/src/drivers/i2c/tpm/tpm.h @@ -74,7 +74,6 @@ struct tpm_vendor_specific { uint8_t req_complete_val; uint8_t req_canceled; int irq; - int (*irq_status)(int irq); int (*recv)(struct tpm_chip *, uint8_t *, size_t); int (*send)(struct tpm_chip *, uint8_t *, size_t); void (*cancel)(struct tpm_chip *); -- cgit v1.2.3