]> git.kernelconcepts.de Git - karo-tx-uboot.git/blobdiff - drivers/tpm/tpm_tis_i2c.c
tpm: tpm_tis_i2c: Tidy up delays
[karo-tx-uboot.git] / drivers / tpm / tpm_tis_i2c.c
index 39652a977c7402e7a339bf3ce45303420f44cf0c..4b2ef94aa7d2780f459f6fca19c4b8d548b202d9 100644 (file)
@@ -43,7 +43,7 @@ static const char * const chip_name[] = {
 static struct tpm_chip g_chip;
 
 /*
- * iic_tpm_read() - read from TPM register
+ * tpm_tis_i2c_read() - read from TPM register
  * @addr: register address to read from
  * @buffer: provided by caller
  * @len: number of bytes to read
@@ -56,7 +56,7 @@ static struct tpm_chip g_chip;
  *
  * Return -EIO on error, 0 on success.
  */
-static int iic_tpm_read(u8 addr, u8 *buffer, size_t len)
+static int tpm_tis_i2c_read(u8 addr, u8 *buffer, size_t len)
 {
        int rc;
        int count;
@@ -68,7 +68,7 @@ static int iic_tpm_read(u8 addr, u8 *buffer, size_t len)
                        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);
                }
                if (rc)
                        return -rc;
@@ -78,7 +78,7 @@ static int iic_tpm_read(u8 addr, u8 *buffer, size_t len)
                 * 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 */
@@ -95,20 +95,20 @@ static int iic_tpm_read(u8 addr, u8 *buffer, size_t len)
                        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' */
-       udelay(SLEEP_DURATION);
+       udelay(SLEEP_DURATION_US);
        if (rc)
                return -rc;
 
        return 0;
 }
 
-static int iic_tpm_write_generic(u8 addr, u8 *buffer, size_t len,
-               unsigned int sleep_time, u8 max_count)
+static int tpm_tis_i2c_write_generic(u8 addr, u8 *buffer, size_t len,
+                                    unsigned int sleep_time_us, u8 max_count)
 {
        int rc = 0;
        int count;
@@ -117,11 +117,11 @@ static int iic_tpm_write_generic(u8 addr, u8 *buffer, size_t len,
                rc = dm_i2c_write(g_chip.dev, addr, buffer, len);
                if (rc == 0)
                        break;  /* Success, break to skip sleep */
-               udelay(sleep_time);
+               udelay(sleep_time_us);
        }
 
        /* take care of 'guard time' */
-       udelay(sleep_time);
+       udelay(sleep_time_us);
        if (rc)
                return -rc;
 
@@ -129,7 +129,7 @@ static int iic_tpm_write_generic(u8 addr, u8 *buffer, size_t len,
 }
 
 /*
- * iic_tpm_write() - write to TPM register
+ * tpm_tis_i2c_write() - write to TPM register
  * @addr: register address to write to
  * @buffer: containing data to be written
  * @len: number of bytes to write
@@ -140,33 +140,34 @@ static int iic_tpm_write_generic(u8 addr, u8 *buffer, size_t len,
  * NOTE: TPM is big-endian for multi-byte values. Multi-byte
  * values have to be swapped.
  *
- * NOTE: use this function instead of the iic_tpm_write_generic function.
+ * NOTE: use this function instead of the tpm_tis_i2c_write_generic function.
  *
  * Return -EIO on error, 0 on success
  */
-static int iic_tpm_write(u8 addr, u8 *buffer, size_t len)
+static int tpm_tis_i2c_write(u8 addr, u8 *buffer, size_t len)
 {
-       return iic_tpm_write_generic(addr, buffer, len, SLEEP_DURATION,
-                       MAX_COUNT);
+       return tpm_tis_i2c_write_generic(addr, buffer, len, SLEEP_DURATION_US,
+                                        MAX_COUNT);
 }
 
 /*
  * This function is needed especially for the cleanup situation after
  * sending TPM_READY
  */
-static int iic_tpm_write_long(u8 addr, u8 *buffer, size_t len)
+static int tpm_tis_i2c_write_long(u8 addr, u8 *buffer, size_t len)
 {
-       return iic_tpm_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 check_locality(struct tpm_chip *chip, int loc)
+static int tpm_tis_i2c_check_locality(struct tpm_chip *chip, int loc)
 {
        const u8 mask = TPM_ACCESS_ACTIVE_LOCALITY | TPM_ACCESS_VALID;
        u8 buf;
        int rc;
 
-       rc = iic_tpm_read(TPM_ACCESS(loc), &buf, 1);
+       rc = tpm_tis_i2c_read(TPM_ACCESS(loc), &buf, 1);
        if (rc < 0)
                return rc;
 
@@ -178,30 +179,31 @@ static int check_locality(struct tpm_chip *chip, int loc)
        return -1;
 }
 
-static void release_locality(struct tpm_chip *chip, int loc, int force)
+static void tpm_tis_i2c_release_locality(struct tpm_chip *chip, int loc,
+                                        int force)
 {
        const u8 mask = TPM_ACCESS_REQUEST_PENDING | TPM_ACCESS_VALID;
        u8 buf;
 
-       if (iic_tpm_read(TPM_ACCESS(loc), &buf, 1) < 0)
+       if (tpm_tis_i2c_read(TPM_ACCESS(loc), &buf, 1) < 0)
                return;
 
        if (force || (buf & mask) == mask) {
                buf = TPM_ACCESS_ACTIVE_LOCALITY;
-               iic_tpm_write(TPM_ACCESS(loc), &buf, 1);
+               tpm_tis_i2c_write(TPM_ACCESS(loc), &buf, 1);
        }
 }
 
-static int request_locality(struct tpm_chip *chip, int loc)
+static int tpm_tis_i2c_request_locality(struct tpm_chip *chip, int loc)
 {
        unsigned long start, stop;
        u8 buf = TPM_ACCESS_REQUEST_USE;
        int rc;
 
-       if (check_locality(chip, loc) >= 0)
+       if (tpm_tis_i2c_check_locality(chip, loc) >= 0)
                return loc;  /* We already have the locality */
 
-       rc = iic_tpm_write(TPM_ACCESS(loc), &buf, 1);
+       rc = tpm_tis_i2c_write(TPM_ACCESS(loc), &buf, 1);
        if (rc)
                return rc;
 
@@ -209,9 +211,9 @@ static int request_locality(struct tpm_chip *chip, int loc)
        start = get_timer(0);
        stop = chip->timeout_a;
        do {
-               if (check_locality(chip, loc) >= 0)
+               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;
@@ -222,7 +224,7 @@ static u8 tpm_tis_i2c_status(struct tpm_chip *chip)
        /* NOTE: Since i2c read may fail, return 0 in this case --> time-out */
        u8 buf;
 
-       if (iic_tpm_read(TPM_STS(chip->locality), &buf, 1) < 0)
+       if (tpm_tis_i2c_read(TPM_STS(chip->locality), &buf, 1) < 0)
                return 0;
        else
                return buf;
@@ -236,12 +238,12 @@ static void tpm_tis_i2c_ready(struct tpm_chip *chip)
        u8 buf = TPM_STS_COMMAND_READY;
 
        debug("%s\n", __func__);
-       rc = iic_tpm_write_long(TPM_STS(chip->locality), &buf, 1);
+       rc = tpm_tis_i2c_write_long(TPM_STS(chip->locality), &buf, 1);
        if (rc)
                debug("%s: rc=%d\n", __func__, rc);
 }
 
-static ssize_t get_burstcount(struct tpm_chip *chip)
+static ssize_t tpm_tis_i2c_get_burstcount(struct tpm_chip *chip)
 {
        unsigned long start, stop;
        ssize_t burstcnt;
@@ -254,21 +256,21 @@ static ssize_t get_burstcount(struct tpm_chip *chip)
        do {
                /* Note: STS is little endian */
                addr = TPM_STS(chip->locality) + 1;
-               if (iic_tpm_read(addr, buf, 3) < 0)
+               if (tpm_tis_i2c_read(addr, buf, 3) < 0)
                        burstcnt = 0;
                else
                        burstcnt = (buf[2] << 16) + (buf[1] << 8) + buf[0];
 
                if (burstcnt)
                        return burstcnt;
-               udelay(TPM_TIMEOUT * 1000);
+               mdelay(TPM_TIMEOUT_MS);
        } while (get_timer(start) < stop);
 
        return -EBUSY;
 }
 
-static int wait_for_stat(struct tpm_chip *chip, u8 mask, unsigned long timeout,
-               int *status)
+static int tpm_tis_i2c_wait_for_stat(struct tpm_chip *chip, u8 mask,
+                                    unsigned long timeout, int *status)
 {
        unsigned long start, stop;
 
@@ -280,7 +282,7 @@ static int wait_for_stat(struct tpm_chip *chip, u8 mask, unsigned long timeout,
        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;
@@ -289,14 +291,14 @@ static int wait_for_stat(struct tpm_chip *chip, u8 mask, unsigned long timeout,
        return -ETIME;
 }
 
-static int recv_data(struct tpm_chip *chip, u8 *buf, size_t count)
+static int tpm_tis_i2c_recv_data(struct tpm_chip *chip, u8 *buf, size_t count)
 {
        size_t size = 0;
        ssize_t burstcnt;
        int rc;
 
        while (size < count) {
-               burstcnt = get_burstcount(chip);
+               burstcnt = tpm_tis_i2c_get_burstcount(chip);
 
                /* burstcount < 0 -> tpm is busy */
                if (burstcnt < 0)
@@ -306,7 +308,7 @@ static int recv_data(struct tpm_chip *chip, u8 *buf, size_t count)
                if (burstcnt > (count - size))
                        burstcnt = count - size;
 
-               rc = iic_tpm_read(TPM_DATA_FIFO(chip->locality),
+               rc = tpm_tis_i2c_read(TPM_DATA_FIFO(chip->locality),
                                &(buf[size]), burstcnt);
                if (rc == 0)
                        size += burstcnt;
@@ -326,7 +328,7 @@ static int tpm_tis_i2c_recv(struct tpm_chip *chip, u8 *buf, size_t count)
        }
 
        /* Read first 10 bytes, including tag, paramsize, and result */
-       size = recv_data(chip, buf, TPM_HEADER_SIZE);
+       size = tpm_tis_i2c_recv_data(chip, buf, TPM_HEADER_SIZE);
        if (size < TPM_HEADER_SIZE) {
                error("Unable to read header\n");
                goto out;
@@ -340,15 +342,16 @@ static int tpm_tis_i2c_recv(struct tpm_chip *chip, u8 *buf, size_t count)
                goto out;
        }
 
-       size += recv_data(chip, &buf[TPM_HEADER_SIZE],
-                       expected - TPM_HEADER_SIZE);
+       size += tpm_tis_i2c_recv_data(chip, &buf[TPM_HEADER_SIZE],
+                                     expected - TPM_HEADER_SIZE);
        if (size < expected) {
                error("Unable to read remainder of result\n");
                size = -ETIME;
                goto out;
        }
 
-       wait_for_stat(chip, TPM_STS_VALID, chip->timeout_c, &status);
+       tpm_tis_i2c_wait_for_stat(chip, TPM_STS_VALID, chip->timeout_c,
+                                 &status);
        if (status & TPM_STS_DATA_AVAIL) {  /* Retry? */
                error("Error left over data\n");
                size = -EIO;
@@ -361,8 +364,8 @@ out:
         * The TPM needs some time to clean up here,
         * so we sleep rather than keeping the bus busy
         */
-       udelay(2000);
-       release_locality(chip, chip->locality, 0);
+       mdelay(2);
+       tpm_tis_i2c_release_locality(chip, chip->locality, 0);
 
        return size;
 }
@@ -379,20 +382,20 @@ static int tpm_tis_i2c_send(struct tpm_chip *chip, u8 *buf, size_t len)
        if (len > TPM_DEV_BUFSIZE)
                return -E2BIG;  /* Command is too long for our tpm, sorry */
 
-       if (request_locality(chip, 0) < 0)
+       if (tpm_tis_i2c_request_locality(chip, 0) < 0)
                return -EBUSY;
 
        status = tpm_tis_i2c_status(chip);
        if ((status & TPM_STS_COMMAND_READY) == 0) {
                tpm_tis_i2c_ready(chip);
-               if (wait_for_stat(chip, TPM_STS_COMMAND_READY,
-                                 chip->timeout_b, &status) < 0) {
+               if (tpm_tis_i2c_wait_for_stat(chip, TPM_STS_COMMAND_READY,
+                                             chip->timeout_b, &status) < 0) {
                        rc = -ETIME;
                        goto out_err;
                }
        }
 
-       burstcnt = get_burstcount(chip);
+       burstcnt = tpm_tis_i2c_get_burstcount(chip);
 
        /* burstcount < 0 -> tpm is busy */
        if (burstcnt < 0)
@@ -408,7 +411,7 @@ static int tpm_tis_i2c_send(struct tpm_chip *chip, u8 *buf, size_t len)
                        burstcnt = CONFIG_TPM_TIS_I2C_BURST_LIMITATION;
 #endif /* CONFIG_TPM_TIS_I2C_BURST_LIMITATION */
 
-               rc = iic_tpm_write(TPM_DATA_FIFO(chip->locality),
+               rc = tpm_tis_i2c_write(TPM_DATA_FIFO(chip->locality),
                                &(buf[count]), burstcnt);
                if (rc == 0)
                        count += burstcnt;
@@ -418,8 +421,9 @@ static int tpm_tis_i2c_send(struct tpm_chip *chip, u8 *buf, size_t len)
                                rc = -EIO;
                                goto out_err;
                        }
-                       rc = wait_for_stat(chip, TPM_STS_VALID,
-                                          chip->timeout_c, &status);
+                       rc = tpm_tis_i2c_wait_for_stat(chip, TPM_STS_VALID,
+                                                      chip->timeout_c,
+                                                      &status);
                        if (rc)
                                goto out_err;
 
@@ -431,7 +435,7 @@ static int tpm_tis_i2c_send(struct tpm_chip *chip, u8 *buf, size_t len)
        }
 
        /* Go and do it */
-       iic_tpm_write(TPM_STS(chip->locality), &sts, 1);
+       tpm_tis_i2c_write(TPM_STS(chip->locality), &sts, 1);
        debug("done\n");
 
        return len;
@@ -443,8 +447,8 @@ out_err:
         * The TPM needs some time to clean up here,
         * so we sleep rather than keeping the bus busy
         */
-       udelay(2000);
-       release_locality(chip, chip->locality, 0);
+       mdelay(2);
+       tpm_tis_i2c_release_locality(chip, chip->locality, 0);
 
        return rc;
 }
@@ -476,21 +480,21 @@ static int tpm_tis_i2c_init(struct udevice *dev)
        /* Disable interrupts (not supported) */
        chip->irq = 0;
 
-       /* Default timeouts */
-       chip->timeout_a = TIS_SHORT_TIMEOUT;
-       chip->timeout_b = TIS_LONG_TIMEOUT;
-       chip->timeout_c = TIS_SHORT_TIMEOUT;
-       chip->timeout_d = TIS_SHORT_TIMEOUT;
+       /* Default timeouts - these could move to the device tree */
+       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;
 
-       if (request_locality(chip, 0) < 0)
+       if (tpm_tis_i2c_request_locality(chip, 0) < 0)
                return  -ENODEV;
 
        /* Read four bytes from DID_VID register */
-       if (iic_tpm_read(TPM_DID_VID(0), (uchar *)&vendor, 4) < 0) {
-               release_locality(chip, 0, 1);
+       if (tpm_tis_i2c_read(TPM_DID_VID(0), (uchar *)&vendor, 4) < 0) {
+               tpm_tis_i2c_release_locality(chip, 0, 1);
                return -EIO;
        }
 
@@ -519,8 +523,8 @@ static int tpm_tis_i2c_init(struct udevice *dev)
 }
 
 /* Returns max number of milliseconds to wait */
-static unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip,
-               u32 ordinal)
+static unsigned long tpm_tis_i2c_calc_ordinal_duration(struct tpm_chip *chip,
+                                                      u32 ordinal)
 {
        int duration_idx = TPM_UNDEFINED;
        int duration = 0;
@@ -542,7 +546,7 @@ static unsigned long tpm_calc_ordinal_duration(struct tpm_chip *chip,
                return duration;
 }
 
-static ssize_t tpm_transmit(const unsigned char *buf, size_t bufsiz)
+static ssize_t tpm_tis_i2c_transmit(const unsigned char *buf, size_t bufsiz)
 {
        int rc;
        u32 count, ordinal;
@@ -575,7 +579,7 @@ static ssize_t tpm_transmit(const unsigned char *buf, size_t bufsiz)
                goto out_recv;
 
        start = get_timer(0);
-       stop = tpm_calc_ordinal_duration(chip, ordinal);
+       stop = tpm_tis_i2c_calc_ordinal_duration(chip, ordinal);
        do {
                debug("waiting for status... %ld %ld\n", start, stop);
                u8 status = tpm_tis_i2c_status(chip);
@@ -590,7 +594,7 @@ static ssize_t tpm_transmit(const unsigned char *buf, size_t bufsiz)
                        rc = -ECANCELED;
                        goto out;
                }
-               udelay(TPM_TIMEOUT * 1000);
+               mdelay(TPM_TIMEOUT_MS);
        } while (get_timer(start) < stop);
 
        tpm_tis_i2c_ready(chip);
@@ -608,34 +612,13 @@ out:
        return rc;
 }
 
-static int tpm_open_dev(struct udevice *dev)
-{
-       int rc;
-
-       debug("%s: start\n", __func__);
-       if (g_chip.is_open)
-               return -EBUSY;
-       rc = tpm_tis_i2c_init(dev);
-       if (rc < 0)
-               g_chip.is_open = 0;
-       return rc;
-}
-
-static void tpm_close(void)
-{
-       if (g_chip.is_open) {
-               release_locality(&g_chip, g_chip.locality, 1);
-               g_chip.is_open = 0;
-       }
-}
-
 /**
  * Decode TPM configuration.
  *
  * @param dev  Returns a configuration of TPM device
  * @return 0 if ok, -1 on error
  */
-static int tpm_decode_config(struct tpm_chip *chip)
+static int tpm_tis_i2c_decode_config(struct tpm_chip *chip)
 {
        const void *blob = gd->fdt_blob;
        struct udevice *bus;
@@ -678,7 +661,7 @@ static int tpm_decode_config(struct tpm_chip *chip)
        }
        /*
         * TODO(sjg@chromium.org): Older TPMs will need to use the older method
-        * in iic_tpm_read() so the offset length needs to be 0 here.
+        * in tpm_tis_i2c_read() so the offset length needs to be 0 here.
         */
        ret = i2c_get_chip(bus, chip_addr, 1, &chip->dev);
        if (ret) {
@@ -695,7 +678,7 @@ int tis_init(void)
        if (g_chip.inited)
                return 0;
 
-       if (tpm_decode_config(&g_chip))
+       if (tpm_tis_i2c_decode_config(&g_chip))
                return -1;
 
        debug("%s: done\n", __func__);
@@ -712,7 +695,12 @@ int tis_open(void)
        if (!g_chip.inited)
                return -1;
 
-       rc = tpm_open_dev(g_chip.dev);
+       debug("%s: start\n", __func__);
+       if (g_chip.is_open)
+               return -EBUSY;
+       rc = tpm_tis_i2c_init(g_chip.dev);
+       if (rc < 0)
+               g_chip.is_open = 0;
 
        return rc;
 }
@@ -722,7 +710,10 @@ int tis_close(void)
        if (!g_chip.inited)
                return -1;
 
-       tpm_close();
+       if (g_chip.is_open) {
+               tpm_tis_i2c_release_locality(&g_chip, g_chip.locality, 1);
+               g_chip.is_open = 0;
+       }
 
        return 0;
 }
@@ -741,7 +732,7 @@ int tis_sendrecv(const uint8_t *sendbuf, size_t sbuf_size,
 
        memcpy(buf, sendbuf, sbuf_size);
 
-       len = tpm_transmit(buf, sbuf_size);
+       len = tpm_tis_i2c_transmit(buf, sbuf_size);
 
        if (len < 10) {
                *rbuf_len = 0;