Patch "tpm: delete the TPM_TIS_CLK_ENABLE flag" has been added to the 4.15-stable tree - Linux-stable-mirror

13 Mar 2018

This is a note to let you know that I've just added the patch titled
tpm: delete the TPM_TIS_CLK_ENABLE flag
to the 4.15-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git%3Ba=su...
The filename of the patch is:
     tpm-delete-the-tpm_tis_clk_enable-flag.patch
and it can be found in the queue-4.15 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let stable@vger.kernel.org know about it.
...
From 764325add6c2ad1641199edde7b2995495fc2d7c Mon Sep 17 00:00:00 2001
From: Javier Martinez Canillas javierm@redhat.com
Date: Mon, 25 Dec 2017 03:22:49 +0100
Subject: tpm: delete the TPM_TIS_CLK_ENABLE flag
From: Javier Martinez Canillas javierm@redhat.com
commit 764325add6c2ad1641199edde7b2995495fc2d7c upstream.
This flag is only used to warn if CLKRUN_EN wasn't disabled on Braswell
systems, but the only way this can happen is if the code is not correct.
So it's an unnecessary check that just makes the code harder to read.
Suggested-by: Jarkko Sakkinen jarkko.sakkinen@linux.intel.com
Signed-off-by: Javier Martinez Canillas javierm@redhat.com
Reviewed-by: Jarkko Sakkinen jarkko.sakkinen@linux.intel.com
Tested-by: Jarkko Sakkinen jarkko.sakkinen@linux.intel.com
Signed-off-by: Jarkko Sakkinen jarkko.sakkinen@linux.intel.com
Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org
---
 drivers/char/tpm/tpm_tis.c      |   15 ---------------
 drivers/char/tpm/tpm_tis_core.c |    2 --
 drivers/char/tpm/tpm_tis_core.h |    1 -
 3 files changed, 18 deletions(-)

--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -138,9 +138,6 @@ static int tpm_tcg_read_bytes(struct tpm
 {
    struct tpm_tis_tcg_phy *phy = to_tpm_tis_tcg_phy(data);
-	if (is_bsw() && !(data->flags & TPM_TIS_CLK_ENABLE))
-		WARN(1, "CLKRUN not enabled!\n");
-
    while (len--)
    	*result++ = ioread8(phy->iobase + addr);
@@ -152,9 +149,6 @@ static int tpm_tcg_write_bytes(struct tp
 {
    struct tpm_tis_tcg_phy *phy = to_tpm_tis_tcg_phy(data);
-	if (is_bsw() && !(data->flags & TPM_TIS_CLK_ENABLE))
-		WARN(1, "CLKRUN not enabled!\n");
-
    while (len--)
    	iowrite8(*value++, phy->iobase + addr);
@@ -165,9 +159,6 @@ static int tpm_tcg_read16(struct tpm_tis
 {
    struct tpm_tis_tcg_phy *phy = to_tpm_tis_tcg_phy(data);
-	if (is_bsw() && !(data->flags & TPM_TIS_CLK_ENABLE))
-		WARN(1, "CLKRUN not enabled!\n");
-
    *result = ioread16(phy->iobase + addr);
return 0;
@@ -177,9 +168,6 @@ static int tpm_tcg_read32(struct tpm_tis
 {
    struct tpm_tis_tcg_phy *phy = to_tpm_tis_tcg_phy(data);
-	if (is_bsw() && !(data->flags & TPM_TIS_CLK_ENABLE))
-		WARN(1, "CLKRUN not enabled!\n");
-
    *result = ioread32(phy->iobase + addr);
return 0;
@@ -189,9 +177,6 @@ static int tpm_tcg_write32(struct tpm_ti
 {
    struct tpm_tis_tcg_phy *phy = to_tpm_tis_tcg_phy(data);
-	if (is_bsw() && !(data->flags & TPM_TIS_CLK_ENABLE))
-		WARN(1, "CLKRUN not enabled!\n");
-
    iowrite32(value, phy->iobase + addr);
return 0;
--- a/drivers/char/tpm/tpm_tis_core.c
+++ b/drivers/char/tpm/tpm_tis_core.c
@@ -698,7 +698,6 @@ static void tpm_tis_clkrun_enable(struct
    	return;
if (value) {
-		data->flags |= TPM_TIS_CLK_ENABLE;
    	data->clkrun_enabled++;
    	if (data->clkrun_enabled > 1)
    		return;
@@ -729,7 +728,6 @@ static void tpm_tis_clkrun_enable(struct
    	 * sure LPC clock is running before sending any TPM command.
    	 */
    	outb(0xCC, 0x80);
-		data->flags &= ~TPM_TIS_CLK_ENABLE;
    }
 }
--- a/drivers/char/tpm/tpm_tis_core.h
+++ b/drivers/char/tpm/tpm_tis_core.h
@@ -86,7 +86,6 @@ enum tis_defaults {
enum tpm_tis_flags {
    TPM_TIS_ITPM_WORKAROUND		= BIT(0),
-	TPM_TIS_CLK_ENABLE		= BIT(1),
 };
struct tpm_tis_data {
Patches currently in stable-queue which might be from javierm@redhat.com are
queue-4.15/tpm-delete-the-tpm_tis_clk_enable-flag.patch
queue-4.15/tpm-only-attempt-to-disable-the-lpc-clkrun-if-is-already-enabled.patch

    