summaryrefslogtreecommitdiffstats
path: root/drivers/char
diff options
context:
space:
mode:
authorJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>2016-02-18 22:11:29 +0200
committerJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>2016-02-20 09:59:33 +0200
commit99cda8cb4639de81cde785b5bab9bc52e916e594 (patch)
treeecbfedd396719c122b192d7df402224dfe477d65 /drivers/char
parent30f9c8c9e2ea37473a51354e9e492580a40661ce (diff)
downloadtalos-op-linux-99cda8cb4639de81cde785b5bab9bc52e916e594.tar.gz
talos-op-linux-99cda8cb4639de81cde785b5bab9bc52e916e594.zip
tpm_crb: tpm2_shutdown() must be called before tpm_chip_unregister()
Wrong call order. Reported-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> Fixes: 74d6b3ceaa17 Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/char')
-rw-r--r--drivers/char/tpm/tpm_crb.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/char/tpm/tpm_crb.c b/drivers/char/tpm/tpm_crb.c
index 9eb04044c385..a12b31940344 100644
--- a/drivers/char/tpm/tpm_crb.c
+++ b/drivers/char/tpm/tpm_crb.c
@@ -344,11 +344,11 @@ static int crb_acpi_remove(struct acpi_device *device)
struct device *dev = &device->dev;
struct tpm_chip *chip = dev_get_drvdata(dev);
- tpm_chip_unregister(chip);
-
if (chip->flags & TPM_CHIP_FLAG_TPM2)
tpm2_shutdown(chip, TPM2_SU_CLEAR);
+ tpm_chip_unregister(chip);
+
return 0;
}
OpenPOWER on IntegriCloud