GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

drivers/i2c/tpm/cr50: Reduce max buffer size 

Reduce the static buffer size from the generic default 1260
down to 64 to match the max FIFO size for the cr50 hardware
and reduce the footprint of the driver.

BUG=chrome-os-partner:53336

Change-Id: I6f9f71d501b60299edad4b16cc553a85391a1866
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-on: https://review.coreboot.org/16664
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Duncan Laurie 2016-09-19 16:37:46 -07:00 committed by Patrick Georgi
parent 2ea13c8699
commit 3727a8d909
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/drivers/i2c/tpm/cr50.c
View File

@ -41,19 +41,19 @@
#include <timer.h> #include <timer.h>
#include "tpm.h" #include "tpm.h"
#define CR50_MAX_BURSTCOUNT 63
#define SLEEP_DURATION 60 /* in usec */ #define SLEEP_DURATION 60 /* in usec */
#define SLEEP_DURATION_LONG 210 /* in usec */ #define SLEEP_DURATION_LONG 210 /* in usec */
#define SLEEP_DURATION_SAFE 750 /* in usec */ #define SLEEP_DURATION_SAFE 750 /* in usec */
#define SLEEP_DURATION_PROBE_MS 1000 /* in msec */ #define SLEEP_DURATION_PROBE_MS 1000 /* in msec */
#define CR50_MAX_BUFSIZE 63
#define CR50_DID_VID 0x00281ae0L #define CR50_DID_VID 0x00281ae0L
struct tpm_inf_dev { struct tpm_inf_dev {
int bus; int bus;
unsigned int addr; unsigned int addr;
uint8_t buf[TPM_BUFSIZE + sizeof(uint8_t)]; uint8_t buf[CR50_MAX_BUFSIZE + sizeof(uint8_t)];
}; };
static struct tpm_inf_dev g_tpm_dev CAR_GLOBAL; static struct tpm_inf_dev g_tpm_dev CAR_GLOBAL;
@ -115,7 +115,7 @@ static int iic_tpm_write(uint8_t addr, uint8_t *buffer, size_t len)
if (tpm_dev->addr == 0) if (tpm_dev->addr == 0)
return -1; return -1;
if (len > TPM_BUFSIZE) if (len > CR50_MAX_BUFSIZE)
return -1; return -1;
/* Prepend the 'register address' to the buffer */ /* Prepend the 'register address' to the buffer */
@ -226,7 +226,7 @@ static int cr50_wait_burst_status(struct tpm_chip *chip, uint8_t mask,
/* Check if mask matches and burst is valid */ /* Check if mask matches and burst is valid */
if ((*status & mask) == mask && if ((*status & mask) == mask &&
*burst > 0 && *burst <= CR50_MAX_BURSTCOUNT) *burst > 0 && *burst <= CR50_MAX_BUFSIZE)
return 0; return 0;
udelay(SLEEP_DURATION_SAFE); udelay(SLEEP_DURATION_SAFE);
@ -310,9 +310,6 @@ static int cr50_tis_i2c_send(struct tpm_chip *chip, uint8_t *buf, size_t len)
uint8_t tpm_go[4] = { TPM_STS_GO }; uint8_t tpm_go[4] = { TPM_STS_GO };
struct stopwatch sw; struct stopwatch sw;
if (len > TPM_BUFSIZE)
return -1;
stopwatch_init_msecs_expire(&sw, 2000); stopwatch_init_msecs_expire(&sw, 2000);
/* Wait until TPM is ready for a command */ /* Wait until TPM is ready for a command */