Re: [PATCH 0/5] Fix whole native SPI TPM driver

From	Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 0/5] Fix whole native SPI TPM driver
Date	2017-02-22 22:20 +0100
Message-ID	<tdMun-7Lo-39@gated-at.bofh.it> (permalink)
References	<tbwN4-fF-17@gated-at.bofh.it>
Organization	Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

Show all headers | View raw

On Thu, Feb 16, 2017 at 04:08:21PM +0000, Peter Huewe wrote:
> During our testing it showed that unfortunately the whole native spi tpm driver
> was more or less non-functional since it was merged, e.g. the wrong byte for
> waitstate handling was used and transfers larger than 64 bytes did not work at all.
> 
> This was probably caused by the merging of the different approaches back then,
> as the initial RFC patch did not have these problems, and also my sudden lack
> of time/commitment back then.
> I'm sorry that the final driver code went untested for that long time.
> 
> This patch set fixes these issues one by one.
> In order to avoid duplication the read/write function was consolidated to one
> transfer function, so we do not have to apply the same fix at two locations.
> Maybe consider squashing it - we splitted it for easier review.
>  
> Affected Kernels: 4.8, 4.9, 4.10
> Patchset was tested on Raspberry Pi2 with SLB9670 (TPM1.2 and TPM2.0)
> 
> Peter Huewe (5):
>   tpm_tis_spi: Use single function to transfer data
>   tpm_tis_spi: Abort transfer when too many wait states are signaled
>   tpm_tis_spi: Check correct byte for wait state indicator
>   tpm_tis_spi: Remove limitation of transfers to MAX_SPI_FRAMESIZE bytes
>   tpm_tis_spi: Add small delay after last transfer
> 
>  drivers/char/tpm/tpm_tis_spi.c | 163 +++++++++++++++++------------------------
>  1 file changed, 69 insertions(+), 94 deletions(-)
> 
> -- 
> 2.7.4
> 

I also found type errors a whilea ago:

https://patchwork.kernel.org/patch/9547993/

Do this take care of these issues? 

/JArkko

Thread

[PATCH 0/5] Fix whole native SPI TPM driver Peter Huewe <peter.huewe@infineon.com> - 2017-02-16 17:10 +0100
  [PATCH 3/5] tpm_tis_spi: Check correct byte for wait state indicator Peter Huewe <peter.huewe@infineon.com> - 2017-02-16 17:10 +0100
    Re: [PATCH 3/5] tpm_tis_spi: Check correct byte for wait state  indicator Christophe Ricard <christophe.ricard@gmail.com> - 2017-02-17 06:20 +0100
      Re: [PATCH 3/5] tpm_tis_spi: Check correct byte for wait state indicator Peter Huewe <peterhuewe@gmx.de> - 2017-02-17 08:20 +0100
    Re: [PATCH 3/5] tpm_tis_spi: Check correct byte for wait state  indicator Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-02-24 14:20 +0100
  [PATCH 1/5] tpm_tis_spi: Use single function to transfer data Peter Huewe <peter.huewe@infineon.com> - 2017-02-16 17:20 +0100
    Re: [PATCH 1/5] tpm_tis_spi: Use single function to transfer data Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-02-24 14:00 +0100
  Re: [PATCH 0/5] Fix whole native SPI TPM driver Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-02-22 22:20 +0100
    Re: [PATCH 0/5] Fix whole native SPI TPM driver Peter Huewe <peterhuewe@gmx.de> - 2017-02-23 00:00 +0100
      Re: [PATCH 0/5] Fix whole native SPI TPM driver Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-02-24 16:20 +0100

csiph-web