Commit c24b8d57 authored by Shreesha Rajashekar's avatar Shreesha Rajashekar Committed by Wolfram Sang

i2c: iproc: Extend I2C read up to 255 bytes

Add support to allow I2C master read transfer up to 255 bytes.
Signed-off-by: default avatarShreesha Rajashekar <shreesha@broadcom.com>
Signed-off-by: default avatarRayagonda Kokatanur <rayagonda.kokatanur@broadcom.com>
Signed-off-by: default avatarRay Jui <ray.jui@broadcom.com>
Signed-off-by: default avatarWolfram Sang <wsa@the-dreams.de>
parent f6ac28d6
...@@ -80,6 +80,10 @@ ...@@ -80,6 +80,10 @@
#define I2C_TIMEOUT_MSEC 50000 #define I2C_TIMEOUT_MSEC 50000
#define M_TX_RX_FIFO_SIZE 64 #define M_TX_RX_FIFO_SIZE 64
#define M_RX_FIFO_MAX_THLD_VALUE (M_TX_RX_FIFO_SIZE - 1)
#define M_RX_MAX_READ_LEN 255
#define M_RX_FIFO_THLD_VALUE 50
enum bus_speed_index { enum bus_speed_index {
I2C_SPD_100K = 0, I2C_SPD_100K = 0,
...@@ -102,17 +106,39 @@ struct bcm_iproc_i2c_dev { ...@@ -102,17 +106,39 @@ struct bcm_iproc_i2c_dev {
/* bytes that have been transferred */ /* bytes that have been transferred */
unsigned int tx_bytes; unsigned int tx_bytes;
/* bytes that have been read */
unsigned int rx_bytes;
unsigned int thld_bytes;
}; };
/* /*
* Can be expanded in the future if more interrupt status bits are utilized * Can be expanded in the future if more interrupt status bits are utilized
*/ */
#define ISR_MASK (BIT(IS_M_START_BUSY_SHIFT) | BIT(IS_M_TX_UNDERRUN_SHIFT)) #define ISR_MASK (BIT(IS_M_START_BUSY_SHIFT) | BIT(IS_M_TX_UNDERRUN_SHIFT)\
| BIT(IS_M_RX_THLD_SHIFT))
static void bcm_iproc_i2c_read_valid_bytes(struct bcm_iproc_i2c_dev *iproc_i2c)
{
struct i2c_msg *msg = iproc_i2c->msg;
/* Read valid data from RX FIFO */
while (iproc_i2c->rx_bytes < msg->len) {
if (!((readl(iproc_i2c->base + M_FIFO_CTRL_OFFSET) >> M_FIFO_RX_CNT_SHIFT)
& M_FIFO_RX_CNT_MASK))
break;
msg->buf[iproc_i2c->rx_bytes] =
(readl(iproc_i2c->base + M_RX_OFFSET) >>
M_RX_DATA_SHIFT) & M_RX_DATA_MASK;
iproc_i2c->rx_bytes++;
}
}
static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data) static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data)
{ {
struct bcm_iproc_i2c_dev *iproc_i2c = data; struct bcm_iproc_i2c_dev *iproc_i2c = data;
u32 status = readl(iproc_i2c->base + IS_OFFSET); u32 status = readl(iproc_i2c->base + IS_OFFSET);
u32 tmp;
status &= ISR_MASK; status &= ISR_MASK;
...@@ -136,8 +162,6 @@ static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data) ...@@ -136,8 +162,6 @@ static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data)
/* mark the last byte */ /* mark the last byte */
if (idx == msg->len - 1) { if (idx == msg->len - 1) {
u32 tmp;
val |= BIT(M_TX_WR_STATUS_SHIFT); val |= BIT(M_TX_WR_STATUS_SHIFT);
/* /*
...@@ -156,6 +180,32 @@ static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data) ...@@ -156,6 +180,32 @@ static irqreturn_t bcm_iproc_i2c_isr(int irq, void *data)
iproc_i2c->tx_bytes += tx_bytes; iproc_i2c->tx_bytes += tx_bytes;
} }
if (status & BIT(IS_M_RX_THLD_SHIFT)) {
struct i2c_msg *msg = iproc_i2c->msg;
u32 bytes_left;
bcm_iproc_i2c_read_valid_bytes(iproc_i2c);
bytes_left = msg->len - iproc_i2c->rx_bytes;
if (bytes_left == 0) {
/* finished reading all data, disable rx thld event */
tmp = readl(iproc_i2c->base + IE_OFFSET);
tmp &= ~BIT(IS_M_RX_THLD_SHIFT);
writel(tmp, iproc_i2c->base + IE_OFFSET);
} else if (bytes_left < iproc_i2c->thld_bytes) {
/* set bytes left as threshold */
tmp = readl(iproc_i2c->base + M_FIFO_CTRL_OFFSET);
tmp &= ~(M_FIFO_RX_THLD_MASK << M_FIFO_RX_THLD_SHIFT);
tmp |= (bytes_left << M_FIFO_RX_THLD_SHIFT);
writel(tmp, iproc_i2c->base + M_FIFO_CTRL_OFFSET);
iproc_i2c->thld_bytes = bytes_left;
}
/*
* bytes_left >= iproc_i2c->thld_bytes,
* hence no need to change the THRESHOLD SET.
* It will remain as iproc_i2c->thld_bytes itself
*/
}
if (status & BIT(IS_M_START_BUSY_SHIFT)) { if (status & BIT(IS_M_START_BUSY_SHIFT)) {
iproc_i2c->xfer_is_done = 1; iproc_i2c->xfer_is_done = 1;
complete(&iproc_i2c->done); complete(&iproc_i2c->done);
...@@ -253,7 +303,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, ...@@ -253,7 +303,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c,
{ {
int ret, i; int ret, i;
u8 addr; u8 addr;
u32 val; u32 val, tmp, val_intr_en;
unsigned int tx_bytes; unsigned int tx_bytes;
unsigned long time_left = msecs_to_jiffies(I2C_TIMEOUT_MSEC); unsigned long time_left = msecs_to_jiffies(I2C_TIMEOUT_MSEC);
...@@ -298,7 +348,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, ...@@ -298,7 +348,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c,
* transaction is done, i.e., the internal start_busy bit, transitions * transaction is done, i.e., the internal start_busy bit, transitions
* from 1 to 0. * from 1 to 0.
*/ */
val = BIT(IE_M_START_BUSY_SHIFT); val_intr_en = BIT(IE_M_START_BUSY_SHIFT);
/* /*
* If TX data size is larger than the TX FIFO, need to enable TX * If TX data size is larger than the TX FIFO, need to enable TX
...@@ -307,9 +357,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, ...@@ -307,9 +357,7 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c,
*/ */
if (!(msg->flags & I2C_M_RD) && if (!(msg->flags & I2C_M_RD) &&
msg->len > iproc_i2c->tx_bytes) msg->len > iproc_i2c->tx_bytes)
val |= BIT(IE_M_TX_UNDERRUN_SHIFT); val_intr_en |= BIT(IE_M_TX_UNDERRUN_SHIFT);
writel(val, iproc_i2c->base + IE_OFFSET);
/* /*
* Now we can activate the transfer. For a read operation, specify the * Now we can activate the transfer. For a read operation, specify the
...@@ -317,11 +365,27 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, ...@@ -317,11 +365,27 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c,
*/ */
val = BIT(M_CMD_START_BUSY_SHIFT); val = BIT(M_CMD_START_BUSY_SHIFT);
if (msg->flags & I2C_M_RD) { if (msg->flags & I2C_M_RD) {
iproc_i2c->rx_bytes = 0;
if (msg->len > M_RX_FIFO_MAX_THLD_VALUE)
iproc_i2c->thld_bytes = M_RX_FIFO_THLD_VALUE;
else
iproc_i2c->thld_bytes = msg->len;
/* set threshold value */
tmp = readl(iproc_i2c->base + M_FIFO_CTRL_OFFSET);
tmp &= ~(M_FIFO_RX_THLD_MASK << M_FIFO_RX_THLD_SHIFT);
tmp |= iproc_i2c->thld_bytes << M_FIFO_RX_THLD_SHIFT;
writel(tmp, iproc_i2c->base + M_FIFO_CTRL_OFFSET);
/* enable the RX threshold interrupt */
val_intr_en |= BIT(IE_M_RX_THLD_SHIFT);
val |= (M_CMD_PROTOCOL_BLK_RD << M_CMD_PROTOCOL_SHIFT) | val |= (M_CMD_PROTOCOL_BLK_RD << M_CMD_PROTOCOL_SHIFT) |
(msg->len << M_CMD_RD_CNT_SHIFT); (msg->len << M_CMD_RD_CNT_SHIFT);
} else { } else {
val |= (M_CMD_PROTOCOL_BLK_WR << M_CMD_PROTOCOL_SHIFT); val |= (M_CMD_PROTOCOL_BLK_WR << M_CMD_PROTOCOL_SHIFT);
} }
writel(val_intr_en, iproc_i2c->base + IE_OFFSET);
writel(val, iproc_i2c->base + M_CMD_OFFSET); writel(val, iproc_i2c->base + M_CMD_OFFSET);
time_left = wait_for_completion_timeout(&iproc_i2c->done, time_left); time_left = wait_for_completion_timeout(&iproc_i2c->done, time_left);
...@@ -353,17 +417,6 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, ...@@ -353,17 +417,6 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c,
return ret; return ret;
} }
/*
* For a read operation, we now need to load the data from FIFO
* into the memory buffer
*/
if (msg->flags & I2C_M_RD) {
for (i = 0; i < msg->len; i++) {
msg->buf[i] = (readl(iproc_i2c->base + M_RX_OFFSET) >>
M_RX_DATA_SHIFT) & M_RX_DATA_MASK;
}
}
return 0; return 0;
} }
...@@ -395,9 +448,8 @@ static const struct i2c_algorithm bcm_iproc_algo = { ...@@ -395,9 +448,8 @@ static const struct i2c_algorithm bcm_iproc_algo = {
.functionality = bcm_iproc_i2c_functionality, .functionality = bcm_iproc_i2c_functionality,
}; };
static const struct i2c_adapter_quirks bcm_iproc_i2c_quirks = { static struct i2c_adapter_quirks bcm_iproc_i2c_quirks = {
/* need to reserve one byte in the FIFO for the slave address */ .max_read_len = M_RX_MAX_READ_LEN,
.max_read_len = M_TX_RX_FIFO_SIZE - 1,
}; };
static int bcm_iproc_i2c_cfg_speed(struct bcm_iproc_i2c_dev *iproc_i2c) static int bcm_iproc_i2c_cfg_speed(struct bcm_iproc_i2c_dev *iproc_i2c)
......
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