Commit d3ff6ce4 authored by Daniel Kurtz's avatar Daniel Kurtz Committed by Jean Delvare

i2c-i801: Enable IRQ for byte_by_byte transactions

Byte-by-byte transactions are used primarily for accessing I2C devices
with an SMBus controller.  For these transactions, for each byte that is
read or written, the SMBus controller generates a BYTE_DONE IRQ.  The isr
reads/writes the next byte, and clears the IRQ flag to start the next byte.
On the penultimate IRQ, the isr also sets the LAST_BYTE flag.

There is no locking around the cmd/len/count/data variables, since the
I2C adapter lock ensures there is never multiple simultaneous transactions
for the same device, and the driver thread never accesses these variables
while interrupts might be occurring.

The end result is faster I2C block read and write transactions.

Note: This patch has only been tested and verified by doing I2C read and
write block transfers on Cougar Point 6 Series PCH, as well as I2C read
block transfers on ICH5.
Signed-off-by: default avatarDaniel Kurtz <djkurtz@chromium.org>
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
parent 29b60854
...@@ -166,6 +166,13 @@ struct i801_priv { ...@@ -166,6 +166,13 @@ struct i801_priv {
/* isr processing */ /* isr processing */
wait_queue_head_t waitq; wait_queue_head_t waitq;
u8 status; u8 status;
/* Command state used by isr for byte-by-byte block transactions */
u8 cmd;
bool is_read;
int count;
int len;
u8 *data;
}; };
static struct pci_driver i801_driver; static struct pci_driver i801_driver;
...@@ -373,14 +380,60 @@ static int i801_block_transaction_by_block(struct i801_priv *priv, ...@@ -373,14 +380,60 @@ static int i801_block_transaction_by_block(struct i801_priv *priv,
return 0; return 0;
} }
static void i801_isr_byte_done(struct i801_priv *priv)
{
if (priv->is_read) {
/* For SMBus block reads, length is received with first byte */
if (((priv->cmd & 0x1c) == I801_BLOCK_DATA) &&
(priv->count == 0)) {
priv->len = inb_p(SMBHSTDAT0(priv));
if (priv->len < 1 || priv->len > I2C_SMBUS_BLOCK_MAX) {
dev_err(&priv->pci_dev->dev,
"Illegal SMBus block read size %d\n",
priv->len);
/* FIXME: Recover */
priv->len = I2C_SMBUS_BLOCK_MAX;
} else {
dev_dbg(&priv->pci_dev->dev,
"SMBus block read size is %d\n",
priv->len);
}
priv->data[-1] = priv->len;
}
/* Read next byte */
if (priv->count < priv->len)
priv->data[priv->count++] = inb(SMBBLKDAT(priv));
else
dev_dbg(&priv->pci_dev->dev,
"Discarding extra byte on block read\n");
/* Set LAST_BYTE for last byte of read transaction */
if (priv->count == priv->len - 1)
outb_p(priv->cmd | SMBHSTCNT_LAST_BYTE,
SMBHSTCNT(priv));
} else if (priv->count < priv->len - 1) {
/* Write next byte, except for IRQ after last byte */
outb_p(priv->data[++priv->count], SMBBLKDAT(priv));
}
/* Clear BYTE_DONE to continue with next byte */
outb_p(SMBHSTSTS_BYTE_DONE, SMBHSTSTS(priv));
}
/* /*
* i801 signals transaction completion with one of these interrupts: * There are two kinds of interrupts:
* INTR - Success *
* DEV_ERR - Invalid command, NAK or communication timeout * 1) i801 signals transaction completion with one of these interrupts:
* BUS_ERR - SMI# transaction collision * INTR - Success
* FAILED - transaction was canceled due to a KILL request * DEV_ERR - Invalid command, NAK or communication timeout
* When any of these occur, update ->status and wake up the waitq. * BUS_ERR - SMI# transaction collision
* ->status must be cleared before kicking off the next transaction. * FAILED - transaction was canceled due to a KILL request
* When any of these occur, update ->status and wake up the waitq.
* ->status must be cleared before kicking off the next transaction.
*
* 2) For byte-by-byte (I2C read/write) transactions, one BYTE_DONE interrupt
* occurs for each byte of a byte-by-byte to prepare the next byte.
*/ */
static irqreturn_t i801_isr(int irq, void *dev_id) static irqreturn_t i801_isr(int irq, void *dev_id)
{ {
...@@ -397,6 +450,9 @@ static irqreturn_t i801_isr(int irq, void *dev_id) ...@@ -397,6 +450,9 @@ static irqreturn_t i801_isr(int irq, void *dev_id)
if (status != 0x42) if (status != 0x42)
dev_dbg(&priv->pci_dev->dev, "irq: status = %02x\n", status); dev_dbg(&priv->pci_dev->dev, "irq: status = %02x\n", status);
if (status & SMBHSTSTS_BYTE_DONE)
i801_isr_byte_done(priv);
/* /*
* Clear irq sources and report transaction result. * Clear irq sources and report transaction result.
* ->status must be cleared before the next transaction is started. * ->status must be cleared before the next transaction is started.
...@@ -443,6 +499,21 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, ...@@ -443,6 +499,21 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv,
else else
smbcmd = I801_BLOCK_DATA; smbcmd = I801_BLOCK_DATA;
if (priv->features & FEATURE_IRQ) {
priv->is_read = (read_write == I2C_SMBUS_READ);
if (len == 1 && priv->is_read)
smbcmd |= SMBHSTCNT_LAST_BYTE;
priv->cmd = smbcmd | SMBHSTCNT_INTREN;
priv->len = len;
priv->count = 0;
priv->data = &data->block[1];
outb_p(priv->cmd | SMBHSTCNT_START, SMBHSTCNT(priv));
wait_event(priv->waitq, (status = priv->status));
priv->status = 0;
return i801_check_post(priv, status);
}
for (i = 1; i <= len; i++) { for (i = 1; i <= len; i++) {
if (i == len && read_write == I2C_SMBUS_READ) if (i == len && read_write == I2C_SMBUS_READ)
smbcmd |= SMBHSTCNT_LAST_BYTE; smbcmd |= SMBHSTCNT_LAST_BYTE;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment