From: Christophe Ricard <christophe.ricard-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org
Cc: jean-luc.blanc-qxv4g6HH51o@public.gmane.org,
ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org,
tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
christophe-h.ricard-qxv4g6HH51o@public.gmane.org,
benoit.houyere-qxv4g6HH51o@public.gmane.org
Subject: [PATCH 02/10] tpm_tis: Extend low-level interface to support proper return codes
Date: Sun, 10 Apr 2016 23:22:58 +0200 [thread overview]
Message-ID: <1460323386-16892-3-git-send-email-christophe-h.ricard@st.com> (raw)
In-Reply-To: <1460323386-16892-1-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
Though the ioread/iowrite calls cannot fail, other implementations of this
interface might want to return error codes if their communication fails.
This follows the usual pattern of negative values representing errors and
zero representing success. Positive values are not used (yet).
Errors are passed back to the caller if possible. If the interface of a
function does not allow that, it tries to do the most sensible thing it
can, but this might also mean ignoring the error in this instance.
Signed-off-by: Alexander Steffen <Alexander.Steffen-d0qZbvYSIPpWk0Htik3J/w@public.gmane.org>
Signed-off-by: Christophe Ricard <christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
---
drivers/char/tpm/tpm.h | 35 ++++----
drivers/char/tpm/tpm_tis.c | 218 ++++++++++++++++++++++++++++++++++-----------
include/linux/tpm.h | 16 ++--
3 files changed, 188 insertions(+), 81 deletions(-)
diff --git a/drivers/char/tpm/tpm.h b/drivers/char/tpm/tpm.h
index a8f48a2..e93e19b 100644
--- a/drivers/char/tpm/tpm.h
+++ b/drivers/char/tpm/tpm.h
@@ -467,44 +467,41 @@ static inline void tpm_buf_append_u32(struct tpm_buf *buf, const u32 value)
tpm_buf_append(buf, (u8 *) &value2, 4);
}
-static inline void tpm_read_bytes(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *result)
+static inline int tpm_read_bytes(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *result)
{
- chip->lowlevel->read_bytes(chip, addr, len, result);
+ return chip->lowlevel->read_bytes(chip, addr, len, result);
}
-static inline u8 tpm_read8(struct tpm_chip *chip, u32 addr)
+static inline int tpm_read8(struct tpm_chip *chip, u32 addr, u8 *result)
{
- u8 result;
-
- chip->lowlevel->read_bytes(chip, addr, 1, &result);
- return result;
+ return chip->lowlevel->read_bytes(chip, addr, 1, result);
}
-static inline u16 tpm_read16(struct tpm_chip *chip, u32 addr)
+static inline int tpm_read16(struct tpm_chip *chip, u32 addr, u16 *result)
{
- return chip->lowlevel->read16(chip, addr);
+ return chip->lowlevel->read16(chip, addr, result);
}
-static inline u32 tpm_read32(struct tpm_chip *chip, u32 addr)
+static inline int tpm_read32(struct tpm_chip *chip, u32 addr, u32 *result)
{
- return chip->lowlevel->read32(chip, addr);
+ return chip->lowlevel->read32(chip, addr, result);
}
-static inline void tpm_write_bytes(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *value)
+static inline int tpm_write_bytes(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *value)
{
- chip->lowlevel->write_bytes(chip, addr, len, value);
+ return chip->lowlevel->write_bytes(chip, addr, len, value);
}
-static inline void tpm_write8(struct tpm_chip *chip, u32 addr, u8 value)
+static inline int tpm_write8(struct tpm_chip *chip, u32 addr, u8 value)
{
- chip->lowlevel->write_bytes(chip, addr, 1, &value);
+ return chip->lowlevel->write_bytes(chip, addr, 1, &value);
}
-static inline void tpm_write32(struct tpm_chip *chip, u32 addr, u32 value)
+static inline int tpm_write32(struct tpm_chip *chip, u32 addr, u32 value)
{
- chip->lowlevel->write32(chip, addr, value);
+ return chip->lowlevel->write32(chip, addr, value);
}
extern struct class *tpm_class;
diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c
index eb0b6802..ee5851e 100644
--- a/drivers/char/tpm/tpm_tis.c
+++ b/drivers/char/tpm/tpm_tis.c
@@ -134,8 +134,14 @@ static int wait_startup(struct tpm_chip *chip, int l)
{
unsigned long stop = jiffies + chip->timeout_a;
do {
- if (tpm_read8(chip, TPM_ACCESS(l)) &
- TPM_ACCESS_VALID)
+ int rc;
+ u8 access;
+
+ rc = tpm_read8(chip, TPM_ACCESS(l), &access);
+ if (rc < 0)
+ return rc;
+
+ if (access & TPM_ACCESS_VALID)
return 0;
msleep(TPM_TIMEOUT);
} while (time_before(jiffies, stop));
@@ -145,9 +151,14 @@ static int wait_startup(struct tpm_chip *chip, int l)
static int check_locality(struct tpm_chip *chip, int l)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
+ int rc;
+ u8 access;
+
+ rc = tpm_read8(chip, TPM_ACCESS(l), &access);
+ if (rc < 0)
+ return rc;
- if ((tpm_read8(chip, TPM_ACCESS(l)) &
- (TPM_ACCESS_ACTIVE_LOCALITY | TPM_ACCESS_VALID)) ==
+ if ((access & (TPM_ACCESS_ACTIVE_LOCALITY | TPM_ACCESS_VALID)) ==
(TPM_ACCESS_ACTIVE_LOCALITY | TPM_ACCESS_VALID))
return priv->locality = l;
@@ -156,7 +167,14 @@ static int check_locality(struct tpm_chip *chip, int l)
static void release_locality(struct tpm_chip *chip, int l, int force)
{
- if (force || (tpm_read8(chip, TPM_ACCESS(l)) &
+ int rc;
+ u8 access;
+
+ rc = tpm_read8(chip, TPM_ACCESS(l), &access);
+ if (rc < 0)
+ return;
+
+ if (force || (access &
(TPM_ACCESS_REQUEST_PENDING | TPM_ACCESS_VALID)) ==
(TPM_ACCESS_REQUEST_PENDING | TPM_ACCESS_VALID))
tpm_write8(chip, TPM_ACCESS(l), TPM_ACCESS_ACTIVE_LOCALITY);
@@ -172,8 +190,9 @@ static int request_locality(struct tpm_chip *chip, int l)
if (check_locality(chip, l) >= 0)
return l;
- tpm_write8(chip, TPM_ACCESS(l), TPM_ACCESS_REQUEST_USE);
-
+ rc = tpm_write8(chip, TPM_ACCESS(l), TPM_ACCESS_REQUEST_USE);
+ if (rc < 0)
+ return rc;
stop = jiffies + chip->timeout_a;
@@ -207,8 +226,14 @@ again:
static u8 tpm_tis_status(struct tpm_chip *chip)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
+ int rc;
+ u8 status;
- return tpm_read8(chip, TPM_STS(priv->locality));
+ rc = tpm_read8(chip, TPM_STS(priv->locality), &status);
+ if (rc < 0)
+ return 0;
+
+ return status;
}
static void tpm_tis_ready(struct tpm_chip *chip)
@@ -223,14 +248,23 @@ static int get_burstcount(struct tpm_chip *chip)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
unsigned long stop;
- int burstcnt;
+ int burstcnt, rc;
+ u8 value;
/* wait for burstcount */
/* which timeout value, spec has 2 answers (c & d) */
stop = jiffies + chip->timeout_d;
do {
- burstcnt = tpm_read8(chip, TPM_STS(priv->locality) + 1);
- burstcnt += tpm_read8(chip, TPM_STS(priv->locality) + 2) << 8;
+ rc = tpm_read8(chip, TPM_STS(priv->locality) + 1, &value);
+ if (rc < 0)
+ return rc;
+
+ burstcnt = value;
+ rc = tpm_read8(chip, TPM_STS(priv->locality) + 2, &value);
+ if (rc < 0)
+ return rc;
+
+ burstcnt += value << 8;
if (burstcnt)
return burstcnt;
msleep(TPM_TIMEOUT);
@@ -241,15 +275,19 @@ static int get_burstcount(struct tpm_chip *chip)
static int recv_data(struct tpm_chip *chip, u8 *buf, size_t count)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
- int size = 0, burstcnt;
+ int size = 0, burstcnt, rc;
while (size < count &&
wait_for_tpm_stat(chip,
TPM_STS_DATA_AVAIL | TPM_STS_VALID,
chip->timeout_c,
&priv->read_queue, true) == 0) {
burstcnt = min_t(int, get_burstcount(chip), count - size);
- tpm_read_bytes(chip, TPM_DATA_FIFO(priv->locality), burstcnt,
- buf + size);
+
+ rc = tpm_read_bytes(chip, TPM_DATA_FIFO(priv->locality),
+ burstcnt, buf + size);
+ if (rc < 0)
+ return rc;
+
size += burstcnt;
}
return size;
@@ -333,8 +371,11 @@ static int tpm_tis_send_data(struct tpm_chip *chip, u8 *buf, size_t len)
while (count < len - 1) {
burstcnt = min_t(int, get_burstcount(chip), len - count - 1);
- tpm_write_bytes(chip, TPM_DATA_FIFO(priv->locality),
- burstcnt, buf + count);
+ rc = tpm_write_bytes(chip, TPM_DATA_FIFO(priv->locality),
+ burstcnt, buf + count);
+ if (rc < 0)
+ goto out_err;
+
count += burstcnt;
wait_for_tpm_stat(chip, TPM_STS_VALID, chip->timeout_c,
@@ -347,7 +388,10 @@ static int tpm_tis_send_data(struct tpm_chip *chip, u8 *buf, size_t len)
}
/* write last byte */
- tpm_write8(chip, TPM_DATA_FIFO(priv->locality), buf[count]);
+ rc = tpm_write8(chip, TPM_DATA_FIFO(priv->locality), buf[count]);
+ if (rc < 0)
+ goto out_err;
+
wait_for_tpm_stat(chip, TPM_STS_VALID, chip->timeout_c,
&priv->int_queue, false);
status = tpm_tis_status(chip);
@@ -368,10 +412,14 @@ static void disable_interrupts(struct tpm_chip *chip)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u32 intmask;
+ int rc;
+
+ rc = tpm_read32(chip, TPM_INT_ENABLE(priv->locality), &intmask);
+ if (rc < 0)
+ intmask = 0;
- intmask = tpm_read32(chip, TPM_INT_ENABLE(priv->locality));
intmask &= ~TPM_GLOBAL_INT_ENABLE;
- tpm_write32(chip, TPM_INT_ENABLE(priv->locality), intmask);
+ rc = tpm_write32(chip, TPM_INT_ENABLE(priv->locality), intmask);
devm_free_irq(&chip->dev, priv->irq, chip);
priv->irq = 0;
@@ -395,7 +443,10 @@ static int tpm_tis_send_main(struct tpm_chip *chip, u8 *buf, size_t len)
return rc;
/* go and do it */
- tpm_write8(chip, TPM_STS(priv->locality), TPM_STS_GO);
+ rc = tpm_write8(chip, TPM_STS(priv->locality), TPM_STS_GO);
+ if (rc < 0)
+ goto out_err;
+
if (chip->flags & TPM_CHIP_FLAG_IRQ) {
ordinal = be32_to_cpu(*((__be32 *) (buf + 6)));
@@ -455,10 +506,12 @@ static const struct tis_vendor_timeout_override vendor_timeout_overrides[] = {
static bool tpm_tis_update_timeouts(struct tpm_chip *chip,
unsigned long *timeout_cap)
{
- int i;
+ int i, rc;
u32 did_vid;
- did_vid = tpm_read32(chip, TPM_DID_VID(0));
+ rc = tpm_read32(chip, TPM_DID_VID(0), &did_vid);
+ if (rc < 0)
+ return rc;
for (i = 0; i != ARRAY_SIZE(vendor_timeout_overrides); i++) {
if (vendor_timeout_overrides[i].did_vid != did_vid)
@@ -486,7 +539,11 @@ static int probe_itpm(struct tpm_chip *chip)
};
size_t len = sizeof(cmd_getticks);
bool rem_itpm = itpm;
- u16 vendor = tpm_read16(chip, TPM_DID_VID(0));
+ u16 vendor;
+
+ rc = tpm_read16(chip, TPM_DID_VID(0), &vendor);
+ if (rc < 0)
+ return rc;
/* probe only iTPMS */
if (vendor != TPM_VID_INTEL)
@@ -544,50 +601,56 @@ static const struct tpm_class_ops tpm_tis = {
.req_canceled = tpm_tis_req_canceled,
};
-static void tpm_mem_read_bytes(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *result)
+static int tpm_mem_read_bytes(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *result)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
while (len--)
*result++ = ioread8(priv->iobase + addr);
+ return 0;
}
-static void tpm_mem_write_bytes(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *value)
+static int tpm_mem_write_bytes(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *value)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
while (len--)
iowrite8(*value++, priv->iobase + addr);
+ return 0;
}
-static u16 tpm_mem_read16(struct tpm_chip *chip, u32 addr)
+static int tpm_mem_read16(struct tpm_chip *chip, u32 addr, u16 *result)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
- return ioread16(priv->iobase + addr);
+ *result = ioread16(priv->iobase + addr);
+ return 0;
}
-static void tpm_mem_write16(struct tpm_chip *chip, u32 addr, u16 value)
+static int tpm_mem_write16(struct tpm_chip *chip, u32 addr, u16 value)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
iowrite16(value, priv->iobase + addr);
+ return 0;
}
-static u32 tpm_mem_read32(struct tpm_chip *chip, u32 addr)
+static int tpm_mem_read32(struct tpm_chip *chip, u32 addr, u32 *result)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
- return ioread32(priv->iobase + addr);
+ *result = ioread32(priv->iobase + addr);
+ return 0;
}
-static void tpm_mem_write32(struct tpm_chip *chip, u32 addr, u32 value)
+static int tpm_mem_write32(struct tpm_chip *chip, u32 addr, u32 value)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
iowrite32(value, priv->iobase + addr);
+ return 0;
}
static const struct tpm_class_lowlevel tpm_mem = {
@@ -604,9 +667,11 @@ static irqreturn_t tis_int_handler(int dummy, void *dev_id)
struct tpm_chip *chip = dev_id;
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u32 interrupt;
- int i;
+ int i, rc;
- interrupt = tpm_read32(chip, TPM_INT_STATUS(priv->locality));
+ rc = tpm_read32(chip, TPM_INT_STATUS(priv->locality), &interrupt);
+ if (rc < 0)
+ return IRQ_NONE;
if (interrupt == 0)
return IRQ_NONE;
@@ -624,9 +689,11 @@ static irqreturn_t tis_int_handler(int dummy, void *dev_id)
wake_up_interruptible(&priv->int_queue);
/* Clear interrupts handled with TPM_EOI */
- tpm_write32(chip, TPM_INT_STATUS(priv->locality), interrupt);
+ rc = tpm_write32(chip, TPM_INT_STATUS(priv->locality), interrupt);
+ if (rc < 0)
+ return IRQ_NONE;
- tpm_read32(chip, TPM_INT_STATUS(priv->locality));
+ tpm_read32(chip, TPM_INT_STATUS(priv->locality), &interrupt);
return IRQ_HANDLED;
}
@@ -639,6 +706,8 @@ static int tpm_tis_probe_irq_single(struct tpm_chip *chip, u32 intmask,
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u8 original_int_vec;
+ int rc;
+ u32 int_status;
if (devm_request_irq(&chip->dev, irq, tis_int_handler, flags,
dev_name(&chip->dev), chip) != 0) {
@@ -648,16 +717,28 @@ static int tpm_tis_probe_irq_single(struct tpm_chip *chip, u32 intmask,
}
priv->irq = irq;
- original_int_vec = tpm_read8(chip, TPM_INT_VECTOR(priv->locality));
- tpm_write8(chip, TPM_INT_VECTOR(priv->locality), irq);
+ rc = tpm_read8(chip, TPM_INT_VECTOR(priv->locality), &original_int_vec);
+ if (rc < 0)
+ return rc;
+
+ rc = tpm_write8(chip, TPM_INT_VECTOR(priv->locality), irq);
+ if (rc < 0)
+ return rc;
+
+ rc = tpm_read32(chip, TPM_INT_STATUS(priv->locality), &int_status);
+ if (rc < 0)
+ return rc;
/* Clear all existing */
- tpm_write32(chip, TPM_INT_STATUS(priv->locality),
- tpm_read32(chip, TPM_INT_STATUS(priv->locality)));
+ rc = tpm_write32(chip, TPM_INT_STATUS(priv->locality), int_status);
+ if (rc < 0)
+ return rc;
/* Turn on */
- tpm_write32(chip, TPM_INT_ENABLE(priv->locality),
- intmask | TPM_GLOBAL_INT_ENABLE);
+ rc = tpm_write32(chip, TPM_INT_ENABLE(priv->locality),
+ intmask | TPM_GLOBAL_INT_ENABLE);
+ if (rc < 0)
+ return rc;
priv->irq_tested = false;
@@ -673,8 +754,10 @@ static int tpm_tis_probe_irq_single(struct tpm_chip *chip, u32 intmask,
* will call disable_irq which undoes all of the above.
*/
if (!(chip->flags & TPM_CHIP_FLAG_IRQ)) {
- tpm_write8(chip, TPM_INT_VECTOR(priv->locality),
- original_int_vec);
+ rc = tpm_write8(chip, TPM_INT_VECTOR(priv->locality),
+ original_int_vec);
+ if (rc < 0)
+ return rc;
return 1;
}
@@ -690,9 +773,11 @@ static void tpm_tis_probe_irq(struct tpm_chip *chip, u32 intmask)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u8 original_int_vec;
- int i;
+ int i, rc;
- original_int_vec = tpm_read8(chip, TPM_INT_VECTOR(priv->locality));
+ rc = tpm_read8(chip, TPM_INT_VECTOR(priv->locality), &original_int_vec);
+ if (rc < 0)
+ return;
if (!original_int_vec) {
if (IS_ENABLED(CONFIG_X86))
@@ -713,11 +798,17 @@ static void tpm_tis_remove(struct tpm_chip *chip)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u32 reg = TPM_INT_ENABLE(priv->locality);
+ u32 interrupt;
+ int rc;
if (chip->flags & TPM_CHIP_FLAG_TPM2)
tpm2_shutdown(chip, TPM2_SU_CLEAR);
- tpm_write32(chip, reg, ~TPM_GLOBAL_INT_ENABLE & tpm_read32(chip, reg));
+ rc = tpm_read32(chip, reg, &interrupt);
+ if (rc < 0)
+ interrupt = 0;
+
+ tpm_write32(chip, reg, ~TPM_GLOBAL_INT_ENABLE & interrupt);
release_locality(chip, priv->locality, 1);
}
@@ -725,6 +816,7 @@ static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
acpi_handle acpi_dev_handle)
{
u32 vendor, intfcaps, intmask;
+ u8 rid;
int rc, probe;
struct tpm_chip *chip;
struct priv_data *priv;
@@ -761,7 +853,10 @@ static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
}
/* Take control of the TPM's interrupt hardware and shut it off */
- intmask = tpm_read32(chip, TPM_INT_ENABLE(priv->locality));
+ rc = tpm_read32(chip, TPM_INT_ENABLE(priv->locality), &intmask);
+ if (rc < 0)
+ goto out_err;
+
intmask |= TPM_INTF_CMD_READY_INT | TPM_INTF_LOCALITY_CHANGE_INT |
TPM_INTF_DATA_AVAIL_INT | TPM_INTF_STS_VALID_INT;
intmask &= ~TPM_GLOBAL_INT_ENABLE;
@@ -776,12 +871,19 @@ static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
if (rc)
goto out_err;
- vendor = tpm_read32(chip, TPM_DID_VID(0));
+ rc = tpm_read32(chip, TPM_DID_VID(0), &vendor);
+ if (rc < 0)
+ goto out_err;
+
priv->manufacturer_id = vendor;
+ rc = tpm_read8(chip, TPM_RID(0), &rid);
+ if (rc < 0)
+ goto out_err;
+
dev_info(dev, "%s TPM (device-id 0x%X, rev-id %d)\n",
(chip->flags & TPM_CHIP_FLAG_TPM2) ? "2.0" : "1.2",
- vendor >> 16, tpm_read8(chip, TPM_RID(0)));
+ vendor >> 16, rid);
if (!itpm) {
probe = probe_itpm(chip);
@@ -797,7 +899,10 @@ static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
/* Figure out the capabilities */
- intfcaps = tpm_read32(chip, TPM_INTF_CAPS(priv->locality));
+ rc = tpm_read32(chip, TPM_INTF_CAPS(priv->locality), &intfcaps);
+ if (rc < 0)
+ goto out_err;
+
dev_dbg(dev, "TPM interface capabilities (0x%x):\n",
intfcaps);
if (intfcaps & TPM_INTF_BURST_COUNT_STATIC)
@@ -877,12 +982,17 @@ static void tpm_tis_reenable_interrupts(struct tpm_chip *chip)
{
struct priv_data *priv = dev_get_drvdata(&chip->dev);
u32 intmask;
+ int rc;
/* reenable interrupts that device may have lost or
BIOS/firmware may have disabled */
- tpm_write8(chip, TPM_INT_VECTOR(priv->locality), priv->irq);
+ rc = tpm_write8(chip, TPM_INT_VECTOR(priv->locality), priv->irq);
+ if (rc < 0)
+ return;
- intmask = tpm_read32(chip, TPM_INT_ENABLE(priv->locality));
+ rc = tpm_read32(chip, TPM_INT_ENABLE(priv->locality), &intmask);
+ if (rc < 0)
+ return;
intmask |= TPM_INTF_CMD_READY_INT
| TPM_INTF_LOCALITY_CHANGE_INT | TPM_INTF_DATA_AVAIL_INT
diff --git a/include/linux/tpm.h b/include/linux/tpm.h
index 19d5d426..e66b413 100644
--- a/include/linux/tpm.h
+++ b/include/linux/tpm.h
@@ -47,14 +47,14 @@ struct tpm_class_ops {
};
struct tpm_class_lowlevel {
- void (*read_bytes)(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *result);
- void (*write_bytes)(struct tpm_chip *chip, u32 addr, u16 len,
- u8 *value);
- u16 (*read16)(struct tpm_chip *chip, u32 addr);
- void (*write16)(struct tpm_chip *chip, u32 addr, u16 src);
- u32 (*read32)(struct tpm_chip *chip, u32 addr);
- void (*write32)(struct tpm_chip *chip, u32 addr, u32 src);
+ int (*read_bytes)(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *result);
+ int (*write_bytes)(struct tpm_chip *chip, u32 addr, u16 len,
+ u8 *value);
+ int (*read16)(struct tpm_chip *chip, u32 addr, u16 *result);
+ int (*write16)(struct tpm_chip *chip, u32 addr, u16 src);
+ int (*read32)(struct tpm_chip *chip, u32 addr, u32 *result);
+ int (*write32)(struct tpm_chip *chip, u32 addr, u32 src);
};
#if defined(CONFIG_TCG_TPM) || defined(CONFIG_TCG_TPM_MODULE)
--
2.1.4
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial! http://pubads.g.doubleclick.net/
gampad/clk?id=1444514301&iu=/ca-pub-7940484522588532
next prev parent reply other threads:[~2016-04-10 21:22 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-10 21:22 [PATCH 00/10] Rework of tpm_tis to share common logic accross phy's (lpc/spi/-i2c-) Christophe Ricard
[not found] ` <1460323386-16892-1-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-10 21:22 ` [PATCH 01/10] tpm_tis: Introduce intermediate layer for TPM access Christophe Ricard
[not found] ` <1460323386-16892-2-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-10 23:45 ` Jason Gunthorpe
2016-04-11 16:12 ` Jarkko Sakkinen
2016-04-10 21:22 ` Christophe Ricard [this message]
2016-04-10 21:22 ` [PATCH 03/10] tpm: Use read/write_bytes for drivers without more specialized methods Christophe Ricard
2016-04-10 21:23 ` [PATCH 04/10] tpm: Manage itpm workaround with tis specific data_expect bit Christophe Ricard
[not found] ` <1460323386-16892-5-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-10 23:42 ` Jason Gunthorpe
2016-04-10 21:23 ` [PATCH 05/10] tpm: tpm_tis: Add post_probe phy handler Christophe Ricard
[not found] ` <1460323386-16892-6-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-10 23:43 ` Jason Gunthorpe
[not found] ` <20160410234333.GB11226-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-04-11 20:22 ` Jason Gunthorpe
2016-04-10 21:23 ` [PATCH 06/10] tpm: Add include guards in tpm.h Christophe Ricard
2016-04-10 21:23 ` [PATCH 07/10] devicetree: Add infineon to vendor-prefix.txt Christophe Ricard
[not found] ` <1460323386-16892-8-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-11 19:47 ` Rob Herring
2016-04-10 21:23 ` [PATCH 08/10] devicetree: Add Trusted Computing Group " Christophe Ricard
[not found] ` <1460323386-16892-9-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-11 20:06 ` Rob Herring
2016-04-11 20:21 ` Jason Gunthorpe
[not found] ` <20160411202119.GA3357-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-04-11 22:37 ` Rob Herring
2016-04-10 21:23 ` [PATCH 09/10] tpm/tpm_tis: Split tpm_tis driver into a core and TCG TIS compliant phy Christophe Ricard
[not found] ` <1460323386-16892-10-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-04-10 23:54 ` Jason Gunthorpe
2016-04-10 21:23 ` [PATCH 10/10] tpm/tpm_tis_spi: Add support for spi phy Christophe Ricard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1460323386-16892-3-git-send-email-christophe-h.ricard@st.com \
--to=christophe.ricard-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org \
--cc=benoit.houyere-qxv4g6HH51o@public.gmane.org \
--cc=christophe-h.ricard-qxv4g6HH51o@public.gmane.org \
--cc=jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=jean-luc.blanc-qxv4g6HH51o@public.gmane.org \
--cc=tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.