Re: [PATCH 0/3] tpm/tpm_crb: implement power management.

[Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>]

On Wed, Sep 07, 2016 at 02:32:30PM +0300, Tomas Winkler wrote:
> The overall platform ability to enter a low power state is also
> conditioned on the ability of a tpm device to go to idle state.
> This series should provide this feature. 
> 
> Unfortunately, there is a HW bug on Intel PTT devices on Skylake,
> Kabylake, and Broxton devices, where certain registers lost retention
> during TPM idle state. Hence this implementation takes this into
> consideration and implement the feature based only on access to
> registers that retain their state. This still conforms to the spec
> and should be correct also on non Intle devices.
> 
> This path series should be applied on top of the series:
> 'Small fixes and cleanups for tpm_crb'

Thanks Tomas. I'll head on to testing tomorrow.

> Tomas Winkler (3):
>   tpm/tpm_crb: implement tpm crb idle state
>   tmp/tpm_crb: fix Intel PTT hw bug during idle state
>   tpm/tpm_crb: cache cmd_size register value.
> 
>  drivers/char/tpm/tpm-interface.c |  21 +++++++
>  drivers/char/tpm/tpm_crb.c       | 124 ++++++++++++++++++++++++++++++++++-----
>  include/linux/tpm.h              |   3 +-
>  3 files changed, 132 insertions(+), 16 deletions(-)
> 
> -- 
> 2.7.4

/Jarkko

------------------------------------------------------------------------------