Use a _US suffix for microseconds and a _MS suffic for milliseconds. Move
all timeouts and delays into one place. Use mdelay() instead of udelay()
where appropriate.
Signed-off-by: Simon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
rc = dm_i2c_write(g_chip.dev, 0, (uchar *)&addrbuf, 1);
if (rc == 0)
break; /* Success, break to skip sleep */
rc = dm_i2c_write(g_chip.dev, 0, (uchar *)&addrbuf, 1);
if (rc == 0)
break; /* Success, break to skip sleep */
- udelay(SLEEP_DURATION);
+ udelay(SLEEP_DURATION_US);
* retrieving the data
*/
for (count = 0; count < MAX_COUNT; count++) {
* retrieving the data
*/
for (count = 0; count < MAX_COUNT; count++) {
- udelay(SLEEP_DURATION);
+ udelay(SLEEP_DURATION_US);
rc = dm_i2c_read(g_chip.dev, 0, buffer, len);
if (rc == 0)
break; /* success, break to skip sleep */
rc = dm_i2c_read(g_chip.dev, 0, buffer, len);
if (rc == 0)
break; /* success, break to skip sleep */
rc = dm_i2c_read(g_chip.dev, addr, buffer, len);
if (rc == 0)
break; /* break here to skip sleep */
rc = dm_i2c_read(g_chip.dev, addr, buffer, len);
if (rc == 0)
break; /* break here to skip sleep */
- udelay(SLEEP_DURATION);
+ udelay(SLEEP_DURATION_US);
}
}
/* Take care of 'guard time' */
}
}
/* Take care of 'guard time' */
- udelay(SLEEP_DURATION);
+ udelay(SLEEP_DURATION_US);
}
static int tpm_tis_i2c_write_generic(u8 addr, u8 *buffer, size_t len,
}
static int tpm_tis_i2c_write_generic(u8 addr, u8 *buffer, size_t len,
- unsigned int sleep_time, u8 max_count)
+ unsigned int sleep_time_us, u8 max_count)
rc = dm_i2c_write(g_chip.dev, addr, buffer, len);
if (rc == 0)
break; /* Success, break to skip sleep */
rc = dm_i2c_write(g_chip.dev, addr, buffer, len);
if (rc == 0)
break; /* Success, break to skip sleep */
}
/* take care of 'guard time' */
}
/* take care of 'guard time' */
*/
static int tpm_tis_i2c_write(u8 addr, u8 *buffer, size_t len)
{
*/
static int tpm_tis_i2c_write(u8 addr, u8 *buffer, size_t len)
{
- return tpm_tis_i2c_write_generic(addr, buffer, len, SLEEP_DURATION,
- MAX_COUNT);
+ return tpm_tis_i2c_write_generic(addr, buffer, len, SLEEP_DURATION_US,
+ MAX_COUNT);
*/
static int tpm_tis_i2c_write_long(u8 addr, u8 *buffer, size_t len)
{
*/
static int tpm_tis_i2c_write_long(u8 addr, u8 *buffer, size_t len)
{
- return tpm_tis_i2c_write_generic(addr, buffer, len, SLEEP_DURATION_LONG,
- MAX_COUNT_LONG);
+ return tpm_tis_i2c_write_generic(addr, buffer, len,
+ SLEEP_DURATION_LONG_US,
+ MAX_COUNT_LONG);
}
static int tpm_tis_i2c_check_locality(struct tpm_chip *chip, int loc)
}
static int tpm_tis_i2c_check_locality(struct tpm_chip *chip, int loc)
do {
if (tpm_tis_i2c_check_locality(chip, loc) >= 0)
return loc;
do {
if (tpm_tis_i2c_check_locality(chip, loc) >= 0)
return loc;
- udelay(TPM_TIMEOUT * 1000);
+ mdelay(TPM_TIMEOUT_MS);
} while (get_timer(start) < stop);
return -1;
} while (get_timer(start) < stop);
return -1;
if (burstcnt)
return burstcnt;
if (burstcnt)
return burstcnt;
- udelay(TPM_TIMEOUT * 1000);
+ mdelay(TPM_TIMEOUT_MS);
} while (get_timer(start) < stop);
return -EBUSY;
} while (get_timer(start) < stop);
return -EBUSY;
start = get_timer(0);
stop = timeout;
do {
start = get_timer(0);
stop = timeout;
do {
- udelay(TPM_TIMEOUT * 1000);
+ mdelay(TPM_TIMEOUT_MS);
*status = tpm_tis_i2c_status(chip);
if ((*status & mask) == mask)
return 0;
*status = tpm_tis_i2c_status(chip);
if ((*status & mask) == mask)
return 0;
* The TPM needs some time to clean up here,
* so we sleep rather than keeping the bus busy
*/
* The TPM needs some time to clean up here,
* so we sleep rather than keeping the bus busy
*/
tpm_tis_i2c_release_locality(chip, chip->locality, 0);
return size;
tpm_tis_i2c_release_locality(chip, chip->locality, 0);
return size;
* The TPM needs some time to clean up here,
* so we sleep rather than keeping the bus busy
*/
* The TPM needs some time to clean up here,
* so we sleep rather than keeping the bus busy
*/
tpm_tis_i2c_release_locality(chip, chip->locality, 0);
return rc;
tpm_tis_i2c_release_locality(chip, chip->locality, 0);
return rc;
chip->irq = 0;
/* Default timeouts - these could move to the device tree */
chip->irq = 0;
/* Default timeouts - these could move to the device tree */
- chip->timeout_a = TIS_SHORT_TIMEOUT;
- chip->timeout_b = TIS_LONG_TIMEOUT;
- chip->timeout_c = TIS_SHORT_TIMEOUT;
- chip->timeout_d = TIS_SHORT_TIMEOUT;
+ chip->timeout_a = TIS_SHORT_TIMEOUT_MS;
+ chip->timeout_b = TIS_LONG_TIMEOUT_MS;
+ chip->timeout_c = TIS_SHORT_TIMEOUT_MS;
+ chip->timeout_d = TIS_SHORT_TIMEOUT_MS;
chip->req_complete_mask = TPM_STS_DATA_AVAIL | TPM_STS_VALID;
chip->req_complete_val = TPM_STS_DATA_AVAIL | TPM_STS_VALID;
chip->req_canceled = TPM_STS_COMMAND_READY;
chip->req_complete_mask = TPM_STS_DATA_AVAIL | TPM_STS_VALID;
chip->req_complete_val = TPM_STS_DATA_AVAIL | TPM_STS_VALID;
chip->req_canceled = TPM_STS_COMMAND_READY;
rc = -ECANCELED;
goto out;
}
rc = -ECANCELED;
goto out;
}
- udelay(TPM_TIMEOUT * 1000);
+ mdelay(TPM_TIMEOUT_MS);
} while (get_timer(start) < stop);
tpm_tis_i2c_ready(chip);
} while (get_timer(start) < stop);
tpm_tis_i2c_ready(chip);
#include <linux/types.h>
enum tpm_timeout {
#include <linux/types.h>
enum tpm_timeout {
- TPM_TIMEOUT = 5, /* msecs */
+ TPM_TIMEOUT_MS = 5,
+ TIS_SHORT_TIMEOUT_MS = 750,
+ TIS_LONG_TIMEOUT_MS = 2000,
+ SLEEP_DURATION_US = 60,
+ SLEEP_DURATION_LONG_US = 210,
};
/* Size of external transmit buffer (used in tpm_transmit)*/
};
/* Size of external transmit buffer (used in tpm_transmit)*/
*/
#define MAX_COUNT_LONG 50
*/
#define MAX_COUNT_LONG 50
-#define SLEEP_DURATION 60 /* in usec */
-#define SLEEP_DURATION_LONG 210 /* in usec */
-
#define TPM_HEADER_SIZE 10
enum tis_access {
#define TPM_HEADER_SIZE 10
enum tis_access {
TPM_STS_DATA_EXPECT = 0x08,
};
TPM_STS_DATA_EXPECT = 0x08,
};
-enum tis_defaults {
- TIS_SHORT_TIMEOUT = 750, /* ms */
- TIS_LONG_TIMEOUT = 2000, /* ms */
-};
-
/* expected value for DIDVID register */
#define TPM_TIS_I2C_DID_VID_9635 0x000b15d1L
#define TPM_TIS_I2C_DID_VID_9645 0x001a15d1L
/* expected value for DIDVID register */
#define TPM_TIS_I2C_DID_VID_9635 0x000b15d1L
#define TPM_TIS_I2C_DID_VID_9645 0x001a15d1L
/* Extended error numbers from linux (see errno.h) */
#define ECANCELED 125 /* Operation Canceled */
/* Extended error numbers from linux (see errno.h) */
#define ECANCELED 125 /* Operation Canceled */
-/* Timer frequency. Corresponds to msec timer resolution*/
+/* Timer frequency. Corresponds to msec timer resolution */
#define HZ 1000
#define TPM_MAX_ORDINAL 243
#define HZ 1000
#define TPM_MAX_ORDINAL 243