diff options
author | Hauke Mehrtens <hauke@hauke-m.de> | 2020-02-29 16:36:21 +0100 |
---|---|---|
committer | Hauke Mehrtens <hauke@hauke-m.de> | 2020-03-12 09:28:23 +0100 |
commit | 988546cd136aab66493ab5564601b25164fb6ecb (patch) | |
tree | 57f6a9601632ece80e355423c2fb8f5d0874e7c6 /target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch | |
parent | 8e6a8a08d2a3f23ff8accb065818808209eeceaa (diff) | |
download | upstream-988546cd136aab66493ab5564601b25164fb6ecb.tar.gz upstream-988546cd136aab66493ab5564601b25164fb6ecb.tar.bz2 upstream-988546cd136aab66493ab5564601b25164fb6ecb.zip |
ipq40xx: Remove kernel 4.14 support
This target was switched to kernel 4.19 more than 6 months ago in commit
f342ffd300da ("treewide: kernel: bump some targets to 4.19") and now
with kernel 5.4 support being added it gets harder to support kernel
4.14 in addition to kernel 4.19 and 5.4.
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
Diffstat (limited to 'target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch')
-rw-r--r-- | target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch | 1154 |
1 files changed, 0 insertions, 1154 deletions
diff --git a/target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch b/target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch deleted file mode 100644 index 6d32882117..0000000000 --- a/target/linux/ipq40xx/patches-4.14/088-0013-i2c-qup-reorganization-of-driver-code-to-remove-poll.patch +++ /dev/null @@ -1,1154 +0,0 @@ -From 7545c7dba169c4c29ba5f6ab8706267a84c0febe Mon Sep 17 00:00:00 2001 -From: Abhishek Sahu <absahu@codeaurora.org> -Date: Mon, 12 Mar 2018 18:45:02 +0530 -Subject: [PATCH 13/13] i2c: qup: reorganization of driver code to remove - polling for qup v2 -MIME-Version: 1.0 -Content-Type: text/plain; charset=UTF-8 -Content-Transfer-Encoding: 8bit - -Following are the major issues in current driver code - -1. The current driver simply assumes the transfer completion - whenever its gets any non-error interrupts and then simply do the - polling of available/free bytes in FIFO. -2. The block mode is not working properly since no handling in - being done for OUT_BLOCK_WRITE_REQ and IN_BLOCK_READ_READ. -3. An i2c transfer can contain multiple message and QUP v2 - supports reconfiguration during run in which the mode should be same - for all the sub transfer. Currently the mode is being programmed - before every sub transfer which is functionally wrong. If one message - is less than FIFO length and other message is greater than FIFO - length, then transfers will fail. - -Because of above, i2c v2 transfers of size greater than 64 are failing -with following error message - - i2c_qup 78b6000.i2c: timeout for fifo out full - -To make block mode working properly and move to use the interrupts -instead of polling, major code reorganization is required. Following -are the major changes done in this patch - -1. Remove the polling of TX FIFO free space and RX FIFO available - bytes and move to interrupts completely. QUP has QUP_MX_OUTPUT_DONE, - QUP_MX_INPUT_DONE, OUT_BLOCK_WRITE_REQ and IN_BLOCK_READ_REQ - interrupts to handle FIFO’s properly so check all these interrupts. -2. Determine the mode for transfer before starting by checking - all the tx/rx data length in each message. The complete message can be - transferred either in DMA mode or Programmed IO by FIFO/Block mode. - in DMA mode, both tx and rx uses same mode but in PIO mode, the TX and - RX can be in different mode. -3. During write, For FIFO mode, TX FIFO can be directly written - without checking for FIFO space. For block mode, the QUP will generate - OUT_BLOCK_WRITE_REQ interrupt whenever it has block size of available - space. -4. During read, both TX and RX FIFO will be used. TX will be used - for writing tags and RX will be used for receiving the data. In QUP, - TX and RX can operate in separate mode so configure modes accordingly. -5. For read FIFO mode, wait for QUP_MX_INPUT_DONE interrupt which - will be generated after all the bytes have been copied in RX FIFO. For - read Block mode, QUP will generate IN_BLOCK_READ_REQ interrupts - whenever it has block size of available data. -6. Split the transfer in chunk of one QUP block size(256 bytes) - and schedule each block separately. QUP v2 supports reconfiguration - during run in which QUP can transfer multiple blocks without issuing a - stop events. -7. Port the SMBus block read support for new code changes. - -Signed-off-by: Abhishek Sahu <absahu@codeaurora.org> -Reviewed-by: Sricharan R <sricharan@codeaurora.org> -Signed-off-by: Wolfram Sang <wsa@the-dreams.de> ---- - drivers/i2c/busses/i2c-qup.c | 900 ++++++++++++++++++++--------------- - 1 file changed, 515 insertions(+), 385 deletions(-) - ---- a/drivers/i2c/busses/i2c-qup.c -+++ b/drivers/i2c/busses/i2c-qup.c -@@ -141,17 +141,40 @@ - #define DEFAULT_SRC_CLK 20000000 - - /* -+ * Max tags length (start, stop and maximum 2 bytes address) for each QUP -+ * data transfer -+ */ -+#define QUP_MAX_TAGS_LEN 4 -+/* Max data length for each DATARD tags */ -+#define RECV_MAX_DATA_LEN 254 -+/* TAG length for DATA READ in RX FIFO */ -+#define READ_RX_TAGS_LEN 2 -+ -+/* - * count: no of blocks - * pos: current block number - * tx_tag_len: tx tag length for current block - * rx_tag_len: rx tag length for current block - * data_len: remaining data length for current message -+ * cur_blk_len: data length for current block - * total_tx_len: total tx length including tag bytes for current QUP transfer - * total_rx_len: total rx length including tag bytes for current QUP transfer -+ * tx_fifo_data_pos: current byte number in TX FIFO word - * tx_fifo_free: number of free bytes in current QUP block write. -+ * rx_fifo_data_pos: current byte number in RX FIFO word - * fifo_available: number of available bytes in RX FIFO for current - * QUP block read -+ * tx_fifo_data: QUP TX FIFO write works on word basis (4 bytes). New byte write -+ * to TX FIFO will be appended in this data and will be written to -+ * TX FIFO when all the 4 bytes are available. -+ * rx_fifo_data: QUP RX FIFO read works on word basis (4 bytes). This will -+ * contains the 4 bytes of RX data. -+ * cur_data: pointer to tell cur data position for current message -+ * cur_tx_tags: pointer to tell cur position in tags -+ * tx_tags_sent: all tx tag bytes have been written in FIFO word -+ * send_last_word: for tx FIFO, last word send is pending in current block - * rx_bytes_read: if all the bytes have been read from rx FIFO. -+ * rx_tags_fetched: all the rx tag bytes have been fetched from rx fifo word - * is_tx_blk_mode: whether tx uses block or FIFO mode in case of non BAM xfer. - * is_rx_blk_mode: whether rx uses block or FIFO mode in case of non BAM xfer. - * tags: contains tx tag bytes for current QUP transfer -@@ -162,10 +185,20 @@ struct qup_i2c_block { - int tx_tag_len; - int rx_tag_len; - int data_len; -+ int cur_blk_len; - int total_tx_len; - int total_rx_len; -+ int tx_fifo_data_pos; - int tx_fifo_free; -+ int rx_fifo_data_pos; - int fifo_available; -+ u32 tx_fifo_data; -+ u32 rx_fifo_data; -+ u8 *cur_data; -+ u8 *cur_tx_tags; -+ bool tx_tags_sent; -+ bool send_last_word; -+ bool rx_tags_fetched; - bool rx_bytes_read; - bool is_tx_blk_mode; - bool is_rx_blk_mode; -@@ -198,6 +231,7 @@ struct qup_i2c_dev { - int out_blk_sz; - int in_blk_sz; - -+ int blk_xfer_limit; - unsigned long one_byte_t; - unsigned long xfer_timeout; - struct qup_i2c_block blk; -@@ -212,10 +246,10 @@ struct qup_i2c_dev { - - /* To check if this is the last msg */ - bool is_last; -- bool is_qup_v1; -+ bool is_smbus_read; - - /* To configure when bus is in run state */ -- int config_run; -+ u32 config_run; - - /* dma parameters */ - bool is_dma; -@@ -223,6 +257,8 @@ struct qup_i2c_dev { - bool use_dma; - unsigned int max_xfer_sg_len; - unsigned int tag_buf_pos; -+ /* The threshold length above which block mode will be used */ -+ unsigned int blk_mode_threshold; - struct dma_pool *dpool; - struct qup_i2c_tag start_tag; - struct qup_i2c_bam brx; -@@ -287,9 +323,6 @@ static irqreturn_t qup_i2c_interrupt(int - goto done; - } - -- if (!qup->is_qup_v1) -- goto done; -- - if (opflags & QUP_OUT_SVC_FLAG) { - writel(QUP_OUT_SVC_FLAG, qup->base + QUP_OPERATIONAL); - -@@ -416,108 +449,6 @@ static int qup_i2c_bus_active(struct qup - return ret; - } - --/** -- * qup_i2c_wait_ready - wait for a give number of bytes in tx/rx path -- * @qup: The qup_i2c_dev device -- * @op: The bit/event to wait on -- * @val: value of the bit to wait on, 0 or 1 -- * @len: The length the bytes to be transferred -- */ --static int qup_i2c_wait_ready(struct qup_i2c_dev *qup, int op, bool val, -- int len) --{ -- unsigned long timeout; -- u32 opflags; -- u32 status; -- u32 shift = __ffs(op); -- int ret = 0; -- -- len *= qup->one_byte_t; -- /* timeout after a wait of twice the max time */ -- timeout = jiffies + len * 4; -- -- for (;;) { -- opflags = readl(qup->base + QUP_OPERATIONAL); -- status = readl(qup->base + QUP_I2C_STATUS); -- -- if (((opflags & op) >> shift) == val) { -- if ((op == QUP_OUT_NOT_EMPTY) && qup->is_last) { -- if (!(status & I2C_STATUS_BUS_ACTIVE)) { -- ret = 0; -- goto done; -- } -- } else { -- ret = 0; -- goto done; -- } -- } -- -- if (time_after(jiffies, timeout)) { -- ret = -ETIMEDOUT; -- goto done; -- } -- usleep_range(len, len * 2); -- } -- --done: -- if (qup->bus_err || qup->qup_err) -- ret = (qup->bus_err & QUP_I2C_NACK_FLAG) ? -ENXIO : -EIO; -- -- return ret; --} -- --static void qup_i2c_set_write_mode_v2(struct qup_i2c_dev *qup, -- struct i2c_msg *msg) --{ -- /* Number of entries to shift out, including the tags */ -- int total = msg->len + qup->blk.tx_tag_len; -- -- total |= qup->config_run; -- -- if (total < qup->out_fifo_sz) { -- /* FIFO mode */ -- writel(QUP_REPACK_EN, qup->base + QUP_IO_MODE); -- writel(total, qup->base + QUP_MX_WRITE_CNT); -- } else { -- /* BLOCK mode (transfer data on chunks) */ -- writel(QUP_OUTPUT_BLK_MODE | QUP_REPACK_EN, -- qup->base + QUP_IO_MODE); -- writel(total, qup->base + QUP_MX_OUTPUT_CNT); -- } --} -- --static int check_for_fifo_space(struct qup_i2c_dev *qup) --{ -- int ret; -- -- ret = qup_i2c_change_state(qup, QUP_PAUSE_STATE); -- if (ret) -- goto out; -- -- ret = qup_i2c_wait_ready(qup, QUP_OUT_FULL, -- RESET_BIT, 4 * ONE_BYTE); -- if (ret) { -- /* Fifo is full. Drain out the fifo */ -- ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -- if (ret) -- goto out; -- -- ret = qup_i2c_wait_ready(qup, QUP_OUT_NOT_EMPTY, -- RESET_BIT, 256 * ONE_BYTE); -- if (ret) { -- dev_err(qup->dev, "timeout for fifo out full"); -- goto out; -- } -- -- ret = qup_i2c_change_state(qup, QUP_PAUSE_STATE); -- if (ret) -- goto out; -- } -- --out: -- return ret; --} -- - static void qup_i2c_write_tx_fifo_v1(struct qup_i2c_dev *qup) - { - struct qup_i2c_block *blk = &qup->blk; -@@ -560,60 +491,17 @@ static void qup_i2c_write_tx_fifo_v1(str - static void qup_i2c_set_blk_data(struct qup_i2c_dev *qup, - struct i2c_msg *msg) - { -- memset(&qup->blk, 0, sizeof(qup->blk)); -- -+ qup->blk.pos = 0; - qup->blk.data_len = msg->len; -- qup->blk.count = (msg->len + QUP_READ_LIMIT - 1) / QUP_READ_LIMIT; -- -- /* 4 bytes for first block and 2 writes for rest */ -- qup->blk.tx_tag_len = 4 + (qup->blk.count - 1) * 2; -- -- /* There are 2 tag bytes that are read in to fifo for every block */ -- if (msg->flags & I2C_M_RD) -- qup->blk.rx_tag_len = qup->blk.count * 2; --} -- --static int qup_i2c_send_data(struct qup_i2c_dev *qup, int tlen, u8 *tbuf, -- int dlen, u8 *dbuf) --{ -- u32 val = 0, idx = 0, pos = 0, i = 0, t; -- int len = tlen + dlen; -- u8 *buf = tbuf; -- int ret = 0; -- -- while (len > 0) { -- ret = check_for_fifo_space(qup); -- if (ret) -- return ret; -- -- t = (len >= 4) ? 4 : len; -- -- while (idx < t) { -- if (!i && (pos >= tlen)) { -- buf = dbuf; -- pos = 0; -- i = 1; -- } -- val |= buf[pos++] << (idx++ * 8); -- } -- -- writel(val, qup->base + QUP_OUT_FIFO_BASE); -- idx = 0; -- val = 0; -- len -= 4; -- } -- -- ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -- -- return ret; -+ qup->blk.count = DIV_ROUND_UP(msg->len, qup->blk_xfer_limit); - } - - static int qup_i2c_get_data_len(struct qup_i2c_dev *qup) - { - int data_len; - -- if (qup->blk.data_len > QUP_READ_LIMIT) -- data_len = QUP_READ_LIMIT; -+ if (qup->blk.data_len > qup->blk_xfer_limit) -+ data_len = qup->blk_xfer_limit; - else - data_len = qup->blk.data_len; - -@@ -630,9 +518,9 @@ static int qup_i2c_set_tags_smb(u16 addr - { - int len = 0; - -- if (msg->len > 1) { -+ if (qup->is_smbus_read) { - tags[len++] = QUP_TAG_V2_DATARD_STOP; -- tags[len++] = qup_i2c_get_data_len(qup) - 1; -+ tags[len++] = qup_i2c_get_data_len(qup); - } else { - tags[len++] = QUP_TAG_V2_START; - tags[len++] = addr & 0xff; -@@ -694,24 +582,6 @@ static int qup_i2c_set_tags(u8 *tags, st - return len; - } - --static int qup_i2c_issue_xfer_v2(struct qup_i2c_dev *qup, struct i2c_msg *msg) --{ -- int data_len = 0, tag_len, index; -- int ret; -- -- tag_len = qup_i2c_set_tags(qup->blk.tags, qup, msg); -- index = msg->len - qup->blk.data_len; -- -- /* only tags are written for read */ -- if (!(msg->flags & I2C_M_RD)) -- data_len = qup_i2c_get_data_len(qup); -- -- ret = qup_i2c_send_data(qup, tag_len, qup->blk.tags, -- data_len, &msg->buf[index]); -- qup->blk.data_len -= data_len; -- -- return ret; --} - - static void qup_i2c_bam_cb(void *data) - { -@@ -778,6 +648,7 @@ static int qup_i2c_bam_make_desc(struct - u32 i = 0, tlen, tx_len = 0; - u8 *tags; - -+ qup->blk_xfer_limit = QUP_READ_LIMIT; - qup_i2c_set_blk_data(qup, msg); - - blocks = qup->blk.count; -@@ -1026,7 +897,7 @@ static int qup_i2c_wait_for_complete(str - unsigned long left; - int ret = 0; - -- left = wait_for_completion_timeout(&qup->xfer, HZ); -+ left = wait_for_completion_timeout(&qup->xfer, qup->xfer_timeout); - if (!left) { - writel(1, qup->base + QUP_SW_RESET); - ret = -ETIMEDOUT; -@@ -1038,65 +909,6 @@ static int qup_i2c_wait_for_complete(str - return ret; - } - --static int qup_i2c_write_one_v2(struct qup_i2c_dev *qup, struct i2c_msg *msg) --{ -- int ret = 0; -- -- qup->msg = msg; -- qup->pos = 0; -- enable_irq(qup->irq); -- qup_i2c_set_blk_data(qup, msg); -- qup_i2c_set_write_mode_v2(qup, msg); -- -- ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -- if (ret) -- goto err; -- -- writel(qup->clk_ctl, qup->base + QUP_I2C_CLK_CTL); -- -- do { -- ret = qup_i2c_issue_xfer_v2(qup, msg); -- if (ret) -- goto err; -- -- ret = qup_i2c_wait_for_complete(qup, msg); -- if (ret) -- goto err; -- -- qup->blk.pos++; -- } while (qup->blk.pos < qup->blk.count); -- -- ret = qup_i2c_wait_ready(qup, QUP_OUT_NOT_EMPTY, RESET_BIT, ONE_BYTE); -- --err: -- disable_irq(qup->irq); -- qup->msg = NULL; -- -- return ret; --} -- --static void qup_i2c_set_read_mode_v2(struct qup_i2c_dev *qup, int len) --{ -- int tx_len = qup->blk.tx_tag_len; -- -- len += qup->blk.rx_tag_len; -- len |= qup->config_run; -- tx_len |= qup->config_run; -- -- if (len < qup->in_fifo_sz) { -- /* FIFO mode */ -- writel(QUP_REPACK_EN, qup->base + QUP_IO_MODE); -- writel(tx_len, qup->base + QUP_MX_WRITE_CNT); -- writel(len, qup->base + QUP_MX_READ_CNT); -- } else { -- /* BLOCK mode (transfer data on chunks) */ -- writel(QUP_INPUT_BLK_MODE | QUP_REPACK_EN, -- qup->base + QUP_IO_MODE); -- writel(tx_len, qup->base + QUP_MX_OUTPUT_CNT); -- writel(len, qup->base + QUP_MX_INPUT_CNT); -- } --} -- - static void qup_i2c_read_rx_fifo_v1(struct qup_i2c_dev *qup) - { - struct qup_i2c_block *blk = &qup->blk; -@@ -1120,104 +932,6 @@ static void qup_i2c_read_rx_fifo_v1(stru - blk->rx_bytes_read = true; - } - --static int qup_i2c_read_fifo_v2(struct qup_i2c_dev *qup, -- struct i2c_msg *msg) --{ -- u32 val; -- int idx, pos = 0, ret = 0, total, msg_offset = 0; -- -- /* -- * If the message length is already read in -- * the first byte of the buffer, account for -- * that by setting the offset -- */ -- if (qup_i2c_check_msg_len(msg) && (msg->len > 1)) -- msg_offset = 1; -- total = qup_i2c_get_data_len(qup); -- total -= msg_offset; -- -- /* 2 extra bytes for read tags */ -- while (pos < (total + 2)) { -- /* Check that FIFO have data */ -- ret = qup_i2c_wait_ready(qup, QUP_IN_NOT_EMPTY, -- SET_BIT, 4 * ONE_BYTE); -- if (ret) { -- dev_err(qup->dev, "timeout for fifo not empty"); -- return ret; -- } -- val = readl(qup->base + QUP_IN_FIFO_BASE); -- -- for (idx = 0; idx < 4; idx++, val >>= 8, pos++) { -- /* first 2 bytes are tag bytes */ -- if (pos < 2) -- continue; -- -- if (pos >= (total + 2)) -- goto out; -- msg->buf[qup->pos + msg_offset] = val & 0xff; -- qup->pos++; -- } -- } -- --out: -- qup->blk.data_len -= total; -- -- return ret; --} -- --static int qup_i2c_read_one_v2(struct qup_i2c_dev *qup, struct i2c_msg *msg) --{ -- int ret = 0; -- -- qup->msg = msg; -- qup->pos = 0; -- enable_irq(qup->irq); -- qup_i2c_set_blk_data(qup, msg); -- qup_i2c_set_read_mode_v2(qup, msg->len); -- -- ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -- if (ret) -- goto err; -- -- writel(qup->clk_ctl, qup->base + QUP_I2C_CLK_CTL); -- -- do { -- ret = qup_i2c_issue_xfer_v2(qup, msg); -- if (ret) -- goto err; -- -- ret = qup_i2c_wait_for_complete(qup, msg); -- if (ret) -- goto err; -- -- ret = qup_i2c_read_fifo_v2(qup, msg); -- if (ret) -- goto err; -- -- qup->blk.pos++; -- -- /* Handle SMBus block read length */ -- if (qup_i2c_check_msg_len(msg) && (msg->len == 1)) { -- if (msg->buf[0] > I2C_SMBUS_BLOCK_MAX) { -- ret = -EPROTO; -- goto err; -- } -- msg->len += msg->buf[0]; -- qup->pos = 0; -- qup_i2c_set_blk_data(qup, msg); -- /* set tag length for block read */ -- qup->blk.tx_tag_len = 2; -- qup_i2c_set_read_mode_v2(qup, msg->buf[0]); -- } -- } while (qup->blk.pos < qup->blk.count); -- --err: -- disable_irq(qup->irq); -- qup->msg = NULL; -- -- return ret; --} -- - static void qup_i2c_write_rx_tags_v1(struct qup_i2c_dev *qup) - { - struct i2c_msg *msg = qup->msg; -@@ -1404,13 +1118,434 @@ out: - return ret; - } - -+/* -+ * Configure registers related with reconfiguration during run and call it -+ * before each i2c sub transfer. -+ */ -+static void qup_i2c_conf_count_v2(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ u32 qup_config = I2C_MINI_CORE | I2C_N_VAL_V2; -+ -+ if (blk->is_tx_blk_mode) -+ writel(qup->config_run | blk->total_tx_len, -+ qup->base + QUP_MX_OUTPUT_CNT); -+ else -+ writel(qup->config_run | blk->total_tx_len, -+ qup->base + QUP_MX_WRITE_CNT); -+ -+ if (blk->total_rx_len) { -+ if (blk->is_rx_blk_mode) -+ writel(qup->config_run | blk->total_rx_len, -+ qup->base + QUP_MX_INPUT_CNT); -+ else -+ writel(qup->config_run | blk->total_rx_len, -+ qup->base + QUP_MX_READ_CNT); -+ } else { -+ qup_config |= QUP_NO_INPUT; -+ } -+ -+ writel(qup_config, qup->base + QUP_CONFIG); -+} -+ -+/* -+ * Configure registers related with transfer mode (FIFO/Block) -+ * before starting of i2c transfer. It will be called only once in -+ * QUP RESET state. -+ */ -+static void qup_i2c_conf_mode_v2(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ u32 io_mode = QUP_REPACK_EN; -+ -+ if (blk->is_tx_blk_mode) { -+ io_mode |= QUP_OUTPUT_BLK_MODE; -+ writel(0, qup->base + QUP_MX_WRITE_CNT); -+ } else { -+ writel(0, qup->base + QUP_MX_OUTPUT_CNT); -+ } -+ -+ if (blk->is_rx_blk_mode) { -+ io_mode |= QUP_INPUT_BLK_MODE; -+ writel(0, qup->base + QUP_MX_READ_CNT); -+ } else { -+ writel(0, qup->base + QUP_MX_INPUT_CNT); -+ } -+ -+ writel(io_mode, qup->base + QUP_IO_MODE); -+} -+ -+/* Clear required variables before starting of any QUP v2 sub transfer. */ -+static void qup_i2c_clear_blk_v2(struct qup_i2c_block *blk) -+{ -+ blk->send_last_word = false; -+ blk->tx_tags_sent = false; -+ blk->tx_fifo_data = 0; -+ blk->tx_fifo_data_pos = 0; -+ blk->tx_fifo_free = 0; -+ -+ blk->rx_tags_fetched = false; -+ blk->rx_bytes_read = false; -+ blk->rx_fifo_data = 0; -+ blk->rx_fifo_data_pos = 0; -+ blk->fifo_available = 0; -+} -+ -+/* Receive data from RX FIFO for read message in QUP v2 i2c transfer. */ -+static void qup_i2c_recv_data(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ int j; -+ -+ for (j = blk->rx_fifo_data_pos; -+ blk->cur_blk_len && blk->fifo_available; -+ blk->cur_blk_len--, blk->fifo_available--) { -+ if (j == 0) -+ blk->rx_fifo_data = readl(qup->base + QUP_IN_FIFO_BASE); -+ -+ *(blk->cur_data++) = blk->rx_fifo_data; -+ blk->rx_fifo_data >>= 8; -+ -+ if (j == 3) -+ j = 0; -+ else -+ j++; -+ } -+ -+ blk->rx_fifo_data_pos = j; -+} -+ -+/* Receive tags for read message in QUP v2 i2c transfer. */ -+static void qup_i2c_recv_tags(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ -+ blk->rx_fifo_data = readl(qup->base + QUP_IN_FIFO_BASE); -+ blk->rx_fifo_data >>= blk->rx_tag_len * 8; -+ blk->rx_fifo_data_pos = blk->rx_tag_len; -+ blk->fifo_available -= blk->rx_tag_len; -+} -+ -+/* -+ * Read the data and tags from RX FIFO. Since in read case, the tags will be -+ * preceded by received data bytes so -+ * 1. Check if rx_tags_fetched is false i.e. the start of QUP block so receive -+ * all tag bytes and discard that. -+ * 2. Read the data from RX FIFO. When all the data bytes have been read then -+ * set rx_bytes_read to true. -+ */ -+static void qup_i2c_read_rx_fifo_v2(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ -+ if (!blk->rx_tags_fetched) { -+ qup_i2c_recv_tags(qup); -+ blk->rx_tags_fetched = true; -+ } -+ -+ qup_i2c_recv_data(qup); -+ if (!blk->cur_blk_len) -+ blk->rx_bytes_read = true; -+} -+ -+/* -+ * Write bytes in TX FIFO for write message in QUP v2 i2c transfer. QUP TX FIFO -+ * write works on word basis (4 bytes). Append new data byte write for TX FIFO -+ * in tx_fifo_data and write to TX FIFO when all the 4 bytes are present. -+ */ -+static void -+qup_i2c_write_blk_data(struct qup_i2c_dev *qup, u8 **data, unsigned int *len) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ unsigned int j; -+ -+ for (j = blk->tx_fifo_data_pos; *len && blk->tx_fifo_free; -+ (*len)--, blk->tx_fifo_free--) { -+ blk->tx_fifo_data |= *(*data)++ << (j * 8); -+ if (j == 3) { -+ writel(blk->tx_fifo_data, -+ qup->base + QUP_OUT_FIFO_BASE); -+ blk->tx_fifo_data = 0x0; -+ j = 0; -+ } else { -+ j++; -+ } -+ } -+ -+ blk->tx_fifo_data_pos = j; -+} -+ -+/* Transfer tags for read message in QUP v2 i2c transfer. */ -+static void qup_i2c_write_rx_tags_v2(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ -+ qup_i2c_write_blk_data(qup, &blk->cur_tx_tags, &blk->tx_tag_len); -+ if (blk->tx_fifo_data_pos) -+ writel(blk->tx_fifo_data, qup->base + QUP_OUT_FIFO_BASE); -+} -+ -+/* -+ * Write the data and tags in TX FIFO. Since in write case, both tags and data -+ * need to be written and QUP write tags can have maximum 256 data length, so -+ * -+ * 1. Check if tx_tags_sent is false i.e. the start of QUP block so write the -+ * tags to TX FIFO and set tx_tags_sent to true. -+ * 2. Check if send_last_word is true. It will be set when last few data bytes -+ * (less than 4 bytes) are reamining to be written in FIFO because of no FIFO -+ * space. All this data bytes are available in tx_fifo_data so write this -+ * in FIFO. -+ * 3. Write the data to TX FIFO and check for cur_blk_len. If it is non zero -+ * then more data is pending otherwise following 3 cases can be possible -+ * a. if tx_fifo_data_pos is zero i.e. all the data bytes in this block -+ * have been written in TX FIFO so nothing else is required. -+ * b. tx_fifo_free is non zero i.e tx FIFO is free so copy the remaining data -+ * from tx_fifo_data to tx FIFO. Since, qup_i2c_write_blk_data do write -+ * in 4 bytes and FIFO space is in multiple of 4 bytes so tx_fifo_free -+ * will be always greater than or equal to 4 bytes. -+ * c. tx_fifo_free is zero. In this case, last few bytes (less than 4 -+ * bytes) are copied to tx_fifo_data but couldn't be sent because of -+ * FIFO full so make send_last_word true. -+ */ -+static void qup_i2c_write_tx_fifo_v2(struct qup_i2c_dev *qup) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ -+ if (!blk->tx_tags_sent) { -+ qup_i2c_write_blk_data(qup, &blk->cur_tx_tags, -+ &blk->tx_tag_len); -+ blk->tx_tags_sent = true; -+ } -+ -+ if (blk->send_last_word) -+ goto send_last_word; -+ -+ qup_i2c_write_blk_data(qup, &blk->cur_data, &blk->cur_blk_len); -+ if (!blk->cur_blk_len) { -+ if (!blk->tx_fifo_data_pos) -+ return; -+ -+ if (blk->tx_fifo_free) -+ goto send_last_word; -+ -+ blk->send_last_word = true; -+ } -+ -+ return; -+ -+send_last_word: -+ writel(blk->tx_fifo_data, qup->base + QUP_OUT_FIFO_BASE); -+} -+ -+/* -+ * Main transfer function which read or write i2c data. -+ * The QUP v2 supports reconfiguration during run in which multiple i2c sub -+ * transfers can be scheduled. -+ */ -+static int -+qup_i2c_conf_xfer_v2(struct qup_i2c_dev *qup, bool is_rx, bool is_first, -+ bool change_pause_state) -+{ -+ struct qup_i2c_block *blk = &qup->blk; -+ struct i2c_msg *msg = qup->msg; -+ int ret; -+ -+ /* -+ * Check if its SMBus Block read for which the top level read will be -+ * done into 2 QUP reads. One with message length 1 while other one is -+ * with actual length. -+ */ -+ if (qup_i2c_check_msg_len(msg)) { -+ if (qup->is_smbus_read) { -+ /* -+ * If the message length is already read in -+ * the first byte of the buffer, account for -+ * that by setting the offset -+ */ -+ blk->cur_data += 1; -+ is_first = false; -+ } else { -+ change_pause_state = false; -+ } -+ } -+ -+ qup->config_run = is_first ? 0 : QUP_I2C_MX_CONFIG_DURING_RUN; -+ -+ qup_i2c_clear_blk_v2(blk); -+ qup_i2c_conf_count_v2(qup); -+ -+ /* If it is first sub transfer, then configure i2c bus clocks */ -+ if (is_first) { -+ ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -+ if (ret) -+ return ret; -+ -+ writel(qup->clk_ctl, qup->base + QUP_I2C_CLK_CTL); -+ -+ ret = qup_i2c_change_state(qup, QUP_PAUSE_STATE); -+ if (ret) -+ return ret; -+ } -+ -+ reinit_completion(&qup->xfer); -+ enable_irq(qup->irq); -+ /* -+ * In FIFO mode, tx FIFO can be written directly while in block mode the -+ * it will be written after getting OUT_BLOCK_WRITE_REQ interrupt -+ */ -+ if (!blk->is_tx_blk_mode) { -+ blk->tx_fifo_free = qup->out_fifo_sz; -+ -+ if (is_rx) -+ qup_i2c_write_rx_tags_v2(qup); -+ else -+ qup_i2c_write_tx_fifo_v2(qup); -+ } -+ -+ ret = qup_i2c_change_state(qup, QUP_RUN_STATE); -+ if (ret) -+ goto err; -+ -+ ret = qup_i2c_wait_for_complete(qup, msg); -+ if (ret) -+ goto err; -+ -+ /* Move to pause state for all the transfers, except last one */ -+ if (change_pause_state) { -+ ret = qup_i2c_change_state(qup, QUP_PAUSE_STATE); -+ if (ret) -+ goto err; -+ } -+ -+err: -+ disable_irq(qup->irq); -+ return ret; -+} -+ -+/* -+ * Transfer one read/write message in i2c transfer. It splits the message into -+ * multiple of blk_xfer_limit data length blocks and schedule each -+ * QUP block individually. -+ */ -+static int qup_i2c_xfer_v2_msg(struct qup_i2c_dev *qup, int msg_id, bool is_rx) -+{ -+ int ret = 0; -+ unsigned int data_len, i; -+ struct i2c_msg *msg = qup->msg; -+ struct qup_i2c_block *blk = &qup->blk; -+ u8 *msg_buf = msg->buf; -+ -+ qup->blk_xfer_limit = is_rx ? RECV_MAX_DATA_LEN : QUP_READ_LIMIT; -+ qup_i2c_set_blk_data(qup, msg); -+ -+ for (i = 0; i < blk->count; i++) { -+ data_len = qup_i2c_get_data_len(qup); -+ blk->pos = i; -+ blk->cur_tx_tags = blk->tags; -+ blk->cur_blk_len = data_len; -+ blk->tx_tag_len = -+ qup_i2c_set_tags(blk->cur_tx_tags, qup, qup->msg); -+ -+ blk->cur_data = msg_buf; -+ -+ if (is_rx) { -+ blk->total_tx_len = blk->tx_tag_len; -+ blk->rx_tag_len = 2; -+ blk->total_rx_len = blk->rx_tag_len + data_len; -+ } else { -+ blk->total_tx_len = blk->tx_tag_len + data_len; -+ blk->total_rx_len = 0; -+ } -+ -+ ret = qup_i2c_conf_xfer_v2(qup, is_rx, !msg_id && !i, -+ !qup->is_last || i < blk->count - 1); -+ if (ret) -+ return ret; -+ -+ /* Handle SMBus block read length */ -+ if (qup_i2c_check_msg_len(msg) && msg->len == 1 && -+ !qup->is_smbus_read) { -+ if (msg->buf[0] > I2C_SMBUS_BLOCK_MAX) -+ return -EPROTO; -+ -+ msg->len = msg->buf[0]; -+ qup->is_smbus_read = true; -+ ret = qup_i2c_xfer_v2_msg(qup, msg_id, true); -+ qup->is_smbus_read = false; -+ if (ret) -+ return ret; -+ -+ msg->len += 1; -+ } -+ -+ msg_buf += data_len; -+ blk->data_len -= qup->blk_xfer_limit; -+ } -+ -+ return ret; -+} -+ -+/* -+ * QUP v2 supports 3 modes -+ * Programmed IO using FIFO mode : Less than FIFO size -+ * Programmed IO using Block mode : Greater than FIFO size -+ * DMA using BAM : Appropriate for any transaction size but the address should -+ * be DMA applicable -+ * -+ * This function determines the mode which will be used for this transfer. An -+ * i2c transfer contains multiple message. Following are the rules to determine -+ * the mode used. -+ * 1. Determine complete length, maximum tx and rx length for complete transfer. -+ * 2. If complete transfer length is greater than fifo size then use the DMA -+ * mode. -+ * 3. In FIFO or block mode, tx and rx can operate in different mode so check -+ * for maximum tx and rx length to determine mode. -+ */ -+static int -+qup_i2c_determine_mode_v2(struct qup_i2c_dev *qup, -+ struct i2c_msg msgs[], int num) -+{ -+ int idx; -+ bool no_dma = false; -+ unsigned int max_tx_len = 0, max_rx_len = 0, total_len = 0; -+ -+ /* All i2c_msgs should be transferred using either dma or cpu */ -+ for (idx = 0; idx < num; idx++) { -+ if (msgs[idx].len == 0) -+ return -EINVAL; -+ -+ if (msgs[idx].flags & I2C_M_RD) -+ max_rx_len = max_t(unsigned int, max_rx_len, -+ msgs[idx].len); -+ else -+ max_tx_len = max_t(unsigned int, max_tx_len, -+ msgs[idx].len); -+ -+ if (is_vmalloc_addr(msgs[idx].buf)) -+ no_dma = true; -+ -+ total_len += msgs[idx].len; -+ } -+ -+ if (!no_dma && qup->is_dma && -+ (total_len > qup->out_fifo_sz || total_len > qup->in_fifo_sz)) { -+ qup->use_dma = true; -+ } else { -+ qup->blk.is_tx_blk_mode = max_tx_len > qup->out_fifo_sz - -+ QUP_MAX_TAGS_LEN ? true : false; -+ qup->blk.is_rx_blk_mode = max_rx_len > qup->in_fifo_sz - -+ READ_RX_TAGS_LEN ? true : false; -+ } -+ -+ return 0; -+} -+ - static int qup_i2c_xfer_v2(struct i2c_adapter *adap, - struct i2c_msg msgs[], - int num) - { - struct qup_i2c_dev *qup = i2c_get_adapdata(adap); - int ret, idx = 0; -- unsigned int total_len = 0; - - qup->bus_err = 0; - qup->qup_err = 0; -@@ -1419,6 +1554,10 @@ static int qup_i2c_xfer_v2(struct i2c_ad - if (ret < 0) - goto out; - -+ ret = qup_i2c_determine_mode_v2(qup, msgs, num); -+ if (ret) -+ goto out; -+ - writel(1, qup->base + QUP_SW_RESET); - ret = qup_i2c_poll_state(qup, QUP_RESET_STATE); - if (ret) -@@ -1428,60 +1567,35 @@ static int qup_i2c_xfer_v2(struct i2c_ad - writel(I2C_MINI_CORE | I2C_N_VAL_V2, qup->base + QUP_CONFIG); - writel(QUP_V2_TAGS_EN, qup->base + QUP_I2C_MASTER_GEN); - -- if ((qup->is_dma)) { -- /* All i2c_msgs should be transferred using either dma or cpu */ -- for (idx = 0; idx < num; idx++) { -- if (msgs[idx].len == 0) { -- ret = -EINVAL; -- goto out; -- } -- -- if (is_vmalloc_addr(msgs[idx].buf)) -- break; -- -- total_len += msgs[idx].len; -- } -- -- if (idx == num && (total_len > qup->out_fifo_sz || -- total_len > qup->in_fifo_sz)) -- qup->use_dma = true; -+ if (qup_i2c_poll_state_i2c_master(qup)) { -+ ret = -EIO; -+ goto out; - } - -- idx = 0; -+ if (qup->use_dma) { -+ reinit_completion(&qup->xfer); -+ ret = qup_i2c_bam_xfer(adap, &msgs[0], num); -+ qup->use_dma = false; -+ } else { -+ qup_i2c_conf_mode_v2(qup); - -- do { -- if (msgs[idx].len == 0) { -- ret = -EINVAL; -- goto out; -- } -+ for (idx = 0; idx < num; idx++) { -+ qup->msg = &msgs[idx]; -+ qup->is_last = idx == (num - 1); - -- if (qup_i2c_poll_state_i2c_master(qup)) { -- ret = -EIO; -- goto out; -+ ret = qup_i2c_xfer_v2_msg(qup, idx, -+ !!(msgs[idx].flags & I2C_M_RD)); -+ if (ret) -+ break; - } -+ qup->msg = NULL; -+ } - -- qup->is_last = (idx == (num - 1)); -- if (idx) -- qup->config_run = QUP_I2C_MX_CONFIG_DURING_RUN; -- else -- qup->config_run = 0; -- -- reinit_completion(&qup->xfer); -- -- if (qup->use_dma) { -- ret = qup_i2c_bam_xfer(adap, &msgs[idx], num); -- qup->use_dma = false; -- break; -- } else { -- if (msgs[idx].flags & I2C_M_RD) -- ret = qup_i2c_read_one_v2(qup, &msgs[idx]); -- else -- ret = qup_i2c_write_one_v2(qup, &msgs[idx]); -- } -- } while ((idx++ < (num - 1)) && !ret); -+ if (!ret) -+ ret = qup_i2c_bus_active(qup, ONE_BYTE); - - if (!ret) -- ret = qup_i2c_change_state(qup, QUP_RESET_STATE); -+ qup_i2c_change_state(qup, QUP_RESET_STATE); - - if (ret == 0) - ret = num; -@@ -1545,6 +1659,7 @@ static int qup_i2c_probe(struct platform - u32 src_clk_freq = DEFAULT_SRC_CLK; - u32 clk_freq = DEFAULT_CLK_FREQ; - int blocks; -+ bool is_qup_v1; - - qup = devm_kzalloc(&pdev->dev, sizeof(*qup), GFP_KERNEL); - if (!qup) -@@ -1563,12 +1678,10 @@ static int qup_i2c_probe(struct platform - if (of_device_is_compatible(pdev->dev.of_node, "qcom,i2c-qup-v1.1.1")) { - qup->adap.algo = &qup_i2c_algo; - qup->adap.quirks = &qup_i2c_quirks; -- qup->is_qup_v1 = true; -- qup->write_tx_fifo = qup_i2c_write_tx_fifo_v1; -- qup->read_rx_fifo = qup_i2c_read_rx_fifo_v1; -- qup->write_rx_tags = qup_i2c_write_rx_tags_v1; -+ is_qup_v1 = true; - } else { - qup->adap.algo = &qup_i2c_algo_v2; -+ is_qup_v1 = false; - ret = qup_i2c_req_dma(qup); - - if (ret == -EPROBE_DEFER) -@@ -1694,14 +1807,31 @@ nodma: - ret = -EIO; - goto fail; - } -- qup->out_blk_sz = blk_sizes[size] / 2; -+ qup->out_blk_sz = blk_sizes[size]; - - size = QUP_INPUT_BLOCK_SIZE(io_mode); - if (size >= ARRAY_SIZE(blk_sizes)) { - ret = -EIO; - goto fail; - } -- qup->in_blk_sz = blk_sizes[size] / 2; -+ qup->in_blk_sz = blk_sizes[size]; -+ -+ if (is_qup_v1) { -+ /* -+ * in QUP v1, QUP_CONFIG uses N as 15 i.e 16 bits constitutes a -+ * single transfer but the block size is in bytes so divide the -+ * in_blk_sz and out_blk_sz by 2 -+ */ -+ qup->in_blk_sz /= 2; -+ qup->out_blk_sz /= 2; -+ qup->write_tx_fifo = qup_i2c_write_tx_fifo_v1; -+ qup->read_rx_fifo = qup_i2c_read_rx_fifo_v1; -+ qup->write_rx_tags = qup_i2c_write_rx_tags_v1; -+ } else { -+ qup->write_tx_fifo = qup_i2c_write_tx_fifo_v2; -+ qup->read_rx_fifo = qup_i2c_read_rx_fifo_v2; -+ qup->write_rx_tags = qup_i2c_write_rx_tags_v2; -+ } - - size = QUP_OUTPUT_FIFO_SIZE(io_mode); - qup->out_fifo_sz = qup->out_blk_sz * (2 << size); |