On Thu, Mar 02, 2017 at 01:03:10PM +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) > > v2: Applied Jarkko's Comments > > 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 | 160 > ++++++++++++++++++----------------------- > 1 file changed, 68 insertions(+), 92 deletions(-)
I'll apply these. /Jarkko ------------------------------------------------------------------------------ Announcing the Oxford Dictionaries API! The API offers world-renowned dictionary content that is easy and intuitive to access. Sign up for an account today to start using our lexical data to power your apps and projects. Get started today and enter our developer competition. http://sdm.link/oxford _______________________________________________ tpmdd-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
