diff options
| author | Rajiv Andrade <srajiv@linux.vnet.ibm.com> | 2011-11-01 17:00:52 -0200 |
|---|---|---|
| committer | Rajiv Andrade <srajiv@linux.vnet.ibm.com> | 2011-11-16 09:57:11 -0200 |
| commit | b9e3238aa36db33aa0d0bd44ef85297c45627aac (patch) | |
| tree | 5c9d90539b5730df66d8dfa26f025b2be898a006 /drivers/char/tpm/tpm_tis.c | |
| parent | 2f592f2a7d74c66763a6903edd04109132674d73 (diff) | |
| download | op-kernel-dev-b9e3238aa36db33aa0d0bd44ef85297c45627aac.zip op-kernel-dev-b9e3238aa36db33aa0d0bd44ef85297c45627aac.tar.gz | |
TPM: fix transmit_cmd error logic
It's incorrect to assume that buffers returned by the TPM
10 bytes long are always error reports. This patches
parses the error field in its header instead. The error report
is now being printed using dev_err() instead of dev_dbg(), making
it easier for users to provide more detailed bug reports.
Signed-off-by: Rajiv Andrade <srajiv@linux.vnet.ibm.com>
Diffstat (limited to 'drivers/char/tpm/tpm_tis.c')
| -rw-r--r-- | drivers/char/tpm/tpm_tis.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c index 92f9f34e..10cc44c 100644 --- a/drivers/char/tpm/tpm_tis.c +++ b/drivers/char/tpm/tpm_tis.c @@ -29,8 +29,6 @@ #include <linux/freezer.h> #include "tpm.h" -#define TPM_HEADER_SIZE 10 - enum tis_access { TPM_ACCESS_VALID = 0x80, TPM_ACCESS_ACTIVE_LOCALITY = 0x20, |
