summaryrefslogtreecommitdiffstats
path: root/arch
diff options
context:
space:
mode:
authorTomas Winkler <tomas.winkler@intel.com>2018-06-28 18:13:33 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2018-09-19 22:43:38 +0200
commite5d857d5f1fc0893a2efa46a7acdec1ef67363d7 (patch)
tree98bc1c4eab8a4280de55dc6d9d7f36f0a47e093d /arch
parent64def6f35348f67c0ae99c38f5bd506fd396a1a2 (diff)
downloadlinux-stable-e5d857d5f1fc0893a2efa46a7acdec1ef67363d7.tar.gz
linux-stable-e5d857d5f1fc0893a2efa46a7acdec1ef67363d7.tar.bz2
linux-stable-e5d857d5f1fc0893a2efa46a7acdec1ef67363d7.zip
tpm: separate cmd_ready/go_idle from runtime_pm
commit 627448e85c766587f6fdde1ea3886d6615081c77 upstream. Fix tpm ptt initialization error: tpm tpm0: A TPM error (378) occurred get tpm pcr allocation. We cannot use go_idle cmd_ready commands via runtime_pm handles as with the introduction of localities this is no longer an optional feature, while runtime pm can be not enabled. Though cmd_ready/go_idle provides a power saving, it's also a part of TPM2 protocol and should be called explicitly. This patch exposes cmd_read/go_idle via tpm class ops and removes runtime pm support as it is not used by any driver. When calling from nested context always use both flags: TPM_TRANSMIT_UNLOCKED and TPM_TRANSMIT_RAW. Both are needed to resolve tpm spaces and locality request recursive calls to tpm_transmit(). TPM_TRANSMIT_RAW should never be used standalone as it will fail on double locking. While TPM_TRANSMIT_UNLOCKED standalone should be called from non-recursive locked contexts. New wrappers are added tpm_cmd_ready() and tpm_go_idle() to streamline tpm_try_transmit code. tpm_crb no longer needs own power saving functions and can drop using tpm_pm_suspend/resume. This patch cannot be really separated from the locality fix. Fixes: 888d867df441 (tpm: cmd_ready command can be issued only after granting locality) Cc: stable@vger.kernel.org Fixes: 888d867df441 (tpm: cmd_ready command can be issued only after granting locality) Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> Tested-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> Reviewed-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>
Diffstat (limited to 'arch')
0 files changed, 0 insertions, 0 deletions