Commit 6c9f0ce0 authored by Javier Martinez Canillas's avatar Javier Martinez Canillas Committed by Jarkko Sakkinen

tpm: only attempt to disable the LPC CLKRUN if is already enabled

Commit 5e572cab ("tpm: Enable CLKRUN protocol for Braswell systems")
added logic in the TPM TIS driver to disable the Low Pin Count CLKRUN
signal during TPM transactions.

Unfortunately this breaks other devices that are attached to the LPC bus
like for example PS/2 mouse and keyboards.

One flaw with the logic is that it assumes that the CLKRUN is always
enabled, and so it unconditionally enables it after a TPM transaction.

But it could be that the CLKRUN# signal was already disabled in the LPC
bus and so after the driver probes, CLKRUN_EN will remain enabled which
may break other devices that are attached to the LPC bus but don't have
support for the CLKRUN protocol.

Fixes: 5e572cab ("tpm: Enable CLKRUN protocol for Braswell systems")
Signed-off-by: default avatarJavier Martinez Canillas <javierm@redhat.com>
Tested-by: default avatarJames Ettle <james@ettle.org.uk>
Tested-by: default avatarJeffery Miller <jmiller@neverware.com>
Reviewed-by: default avatarJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Tested-by: default avatarJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Signed-off-by: default avatarJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
parent 5fc0bead
...@@ -759,7 +759,8 @@ static void tpm_tis_clkrun_enable(struct tpm_chip *chip, bool value) ...@@ -759,7 +759,8 @@ static void tpm_tis_clkrun_enable(struct tpm_chip *chip, bool value)
struct tpm_tis_data *data = dev_get_drvdata(&chip->dev); struct tpm_tis_data *data = dev_get_drvdata(&chip->dev);
u32 clkrun_val; u32 clkrun_val;
if (!IS_ENABLED(CONFIG_X86) || !is_bsw()) if (!IS_ENABLED(CONFIG_X86) || !is_bsw() ||
!data->ilb_base_addr)
return; return;
if (value) { if (value) {
...@@ -818,6 +819,7 @@ int tpm_tis_core_init(struct device *dev, struct tpm_tis_data *priv, int irq, ...@@ -818,6 +819,7 @@ int tpm_tis_core_init(struct device *dev, struct tpm_tis_data *priv, int irq,
u32 vendor; u32 vendor;
u32 intfcaps; u32 intfcaps;
u32 intmask; u32 intmask;
u32 clkrun_val;
u8 rid; u8 rid;
int rc, probe; int rc, probe;
struct tpm_chip *chip; struct tpm_chip *chip;
...@@ -843,6 +845,13 @@ int tpm_tis_core_init(struct device *dev, struct tpm_tis_data *priv, int irq, ...@@ -843,6 +845,13 @@ int tpm_tis_core_init(struct device *dev, struct tpm_tis_data *priv, int irq,
ILB_REMAP_SIZE); ILB_REMAP_SIZE);
if (!priv->ilb_base_addr) if (!priv->ilb_base_addr)
return -ENOMEM; return -ENOMEM;
clkrun_val = ioread32(priv->ilb_base_addr + LPC_CNTRL_OFFSET);
/* Check if CLKRUN# is already not enabled in the LPC bus */
if (!(clkrun_val & LPC_CLKRUN_EN)) {
iounmap(priv->ilb_base_addr);
priv->ilb_base_addr = NULL;
}
} }
if (chip->ops->clk_enable != NULL) if (chip->ops->clk_enable != NULL)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment