linux.git/drivers/char/tpm, branch v3.11

tpm/tpm_i2c_infineon: Remove unused header file

2013-06-27T22:47:43+00:00

This driver does not use any module parameters anymore,
so the inclusion of the header file can be removed.

Signed-off-by: Peter Huewe

tpm: tpm_i2c_infinion: Don't modify i2c_client->driver

2013-06-27T22:47:43+00:00

The I2C client driver is not supposed to modify the client's driver pointer,
this is handled by the I2C core.

Signed-off-by: Lars-Peter Clausen 
Signed-off-by: Peter Huewe

tpm: fix regression caused by section type conflict of tpm_dev_release() in ppc builds

2013-05-27T20:09:09+00:00

The 8119807 commit reintroduced a regression
(error: __ksymtab_tpm_dev_release causes a section type conflict) that was fixed by commit
cbb2ed4.
Fix it for good by adding the prototype to tpm.h so sparse doesn't
complain about it anymore.

Reported-by: Tony Camuso 
Signed-off-by: Peter Huewe

tpm: move TPM_DIGEST_SIZE defintion

2013-05-21T17:25:02+00:00

IMA requires access to TPM_DIGEST_SIZE definition.  This patch
moves the definition to .

Signed-off-by: Mimi Zohar 
Signed-off-by: Kent Yoder

tpm_tis: missing platform_driver_unregister() on error in init_tis()

2013-05-21T17:24:48+00:00

Add the missing platform_driver_unregister() before return
from init_tis() in the device register error handling case.

Signed-off-by: Wei Yongjun 
Signed-off-by: Kent Yoder

TPM: Retry SaveState command in suspend path

2013-04-17T14:31:32+00:00

If the TPM has already been sent a SaveState command before the driver
is loaded it may have problems sending that same command again later.

This issue is seen with the Chromebook Pixel due to a firmware bug in
the legacy mode boot path which is sending the SaveState command
before booting the kernel.  More information is available at
http://crbug.com/203524

This change introduces a retry of the SaveState command in the suspend
path in order to work around this issue.  A future firmware update
should fix this but this is also a trivial workaround in the driver
that has no effect on systems that do not show this problem.

When this does happen the TPM responds with a non-fatal TPM_RETRY code
that is defined in the specification:

  The TPM is too busy to respond to the command immediately, but the
  command could be resubmitted at a later time.  The TPM MAY return
  TPM_RETRY for any command at any time.

It can take several seconds before the TPM will respond again.  I
measured a typical time between 3 and 4 seconds and the timeout is set
at a safe 5 seconds.

It is also possible to reproduce this with commands via /dev/tpm0.
The bug linked above has a python script attached which can be used to
test for this problem.  I tested a variety of TPMs from Infineon,
Nuvoton, Atmel, and STMicro but was only able to reproduce this with
LPC and I2C TPMs from Infineon.

The TPM specification only loosely defines this behavior:

  TPM Main Level 2 Part 3 v1.2 r116, section 3.3. TPM_SaveState:
  The TPM MAY declare all preserved values invalid in response to any
  command other than TPM_Init.

  TCG PC Client BIOS Spec 1.21 section 8.3.1.
  After issuing a TPM_SaveState command, the OS SHOULD NOT issue TPM
  commands before transitioning to S3 without issuing another
  TPM_SaveState command.

  TCG PC Client TIS 1.21, section 4. Power Management:
  The TPM_SaveState command allows a Static OS to indicate to the TPM
  that the platform may enter a low power state where the TPM will be
  required to enter into the D3 power state.  The use of the term "may"
  is significant in that there is no requirement for the platform to
  actually enter the low power state after sending the TPM_SaveState
  command.  The software may, in fact, send subsequent commands after
  sending the TPM_SaveState command.

Change-Id: I52b41e826412688e5b6c8ddd3bb16409939704e9
Signed-off-by: Duncan Laurie 
Signed-off-by: Kent Yoder

tpm/tpm_i2c_infineon: Add small comment about return value of __i2c_transfer

2013-04-12T17:17:29+00:00

Kent Yoder indicated that the code might be a bit clearer with a comment
here, so this patch adds a small explanation of the code.

Signed-off-by: Peter Huewe 
Signed-off-by: Kent Yoder

tpm/tpm_i2c_infineon.c: Add OF attributes type and name to the of_device_id table entries

2013-04-12T17:17:29+00:00

As the subject says.
It's probably a good idea to have these fields populated.

Signed-off-by: Peter Huewe 
Signed-off-by: Kent Yoder

tpm_i2c_stm_st33: Remove duplicate inclusion of header files

2013-04-12T17:17:29+00:00

module.h and sched.h were included twice.

Signed-off-by: Sachin Kamat 
Signed-off-by: Kent Yoder

tpm: Add support for new Infineon I2C TPM (SLB 9645 TT 1.2 I2C)

2013-04-12T17:17:28+00:00

This driver adds support for Infineon's new SLB 9645 TT 1.2 I2C TPMs,
which supports clockstretching, combined reads and a bus speed of
up to 400khz. The device also has a new device id.

The driver works now also fine with device trees, so you can
instantiate your device by adding:
 +       tpm {
 +               compatible = "infineon,slb9645tt";
 +               reg = <0x20>;
 +       };
 for SLB 9645 devices or

 +       tpm {
 +               compatible = "infineon,slb9635tt";
 +               reg = <0x20>;
 +       };
 for SLB 9635 devices

to your device tree.
tpm_i2c_infineon is also retained as a compatible id as a fallback to
slb9635 protocol.

The driver was tested on Beaglebone.

Signed-off-by: Peter Huewe 
Signed-off-by: Kent Yoder