[{"id":1739029,"web_url":"http://patchwork.ozlabs.org/comment/1739029/","msgid":"<E1ddlJy-00008f-CZ@mail.theobroma-systems.com>","list_archive_url":null,"date":"2017-08-04T22:43:54","subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> The hw can read up to 32 bytes at a time. If we need\n> more than one chunk, we have to enter the plain RX mode.\n> \n> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>\n> ---\n>  drivers/i2c/rk_i2c.c | 19 ++++++++++++++++---\n>  1 file changed, 16 insertions(+), 3 deletions(-)\n> \n\nAcked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xPMl70Z6lz9s1h\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat,  5 Aug 2017 08:57:39 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid DD8C2C21DFA; Fri,  4 Aug 2017 22:53:40 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id D4600C21F40;\n\tFri,  4 Aug 2017 22:48:12 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 84560C21F1F; Fri,  4 Aug 2017 22:44:01 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id 43BB4C21EBA\n\tfor <u-boot@lists.denx.de>; Fri,  4 Aug 2017 22:43:56 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:56678\n\thelo=vpn-10-11-0-14.lan) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1ddlJy-00008f-CZ; Sat, 05 Aug 2017 00:43:54 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","To":"Wadim Egorov <w.egorov@phytec.de>","In-Reply-To":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","References":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","Message-Id":"<E1ddlJy-00008f-CZ@mail.theobroma-systems.com>","Date":"Sat, 05 Aug 2017 00:43:54 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1751029,"web_url":"http://patchwork.ozlabs.org/comment/1751029/","msgid":"<alpine.OSX.2.21.1708181505080.27138@vpn-10-11-0-14.lan>","list_archive_url":null,"date":"2017-08-18T13:06:05","subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"On Thu, 3 Aug 2017, Wadim Egorov wrote:\n\n> The hw can read up to 32 bytes at a time. If we need\n> more than one chunk, we have to enter the plain RX mode.\n\nWhy does this need to be in 'plain RX' mode for more than 32 bytes?\nWhat happens if someone wants to write/transmit more than 32 bytes?\n\n>\n> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>\n> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>\n> ---\n> drivers/i2c/rk_i2c.c | 19 ++++++++++++++++---\n> 1 file changed, 16 insertions(+), 3 deletions(-)\n>\n> diff --git a/drivers/i2c/rk_i2c.c b/drivers/i2c/rk_i2c.c\n> index 8bc045a..68e6653 100644\n> --- a/drivers/i2c/rk_i2c.c\n> +++ b/drivers/i2c/rk_i2c.c\n> @@ -164,6 +164,7 @@ static int rk_i2c_read(struct rk_i2c *i2c, uchar chip, uint reg, uint r_len,\n> \tuint rxdata;\n> \tuint i, j;\n> \tint err;\n> +\tbool snd_chunk = false;\n>\n> \tdebug(\"rk_i2c_read: chip = %d, reg = %d, r_len = %d, b_len = %d\\n\",\n> \t      chip, reg, r_len, b_len);\n> @@ -184,15 +185,26 @@ static int rk_i2c_read(struct rk_i2c *i2c, uchar chip, uint reg, uint r_len,\n>\n> \twhile (bytes_remain_len) {\n> \t\tif (bytes_remain_len > RK_I2C_FIFO_SIZE) {\n> -\t\t\tcon = I2C_CON_EN | I2C_CON_MOD(I2C_MODE_TRX);\n> +\t\t\tcon = I2C_CON_EN;\n> \t\t\tbytes_xferred = 32;\n> \t\t} else {\n> -\t\t\tcon = I2C_CON_EN | I2C_CON_MOD(I2C_MODE_TRX) |\n> -\t\t\t\tI2C_CON_LASTACK;\n> +\t\t\t/*\n> +\t\t\t * The hw can read up to 32 bytes at a time. If we need\n> +\t\t\t * more than one chunk, send an ACK after the last byte.\n> +\t\t\t */\n> +\t\t\tcon = I2C_CON_EN | I2C_CON_LASTACK;\n> \t\t\tbytes_xferred = bytes_remain_len;\n> \t\t}\n> \t\twords_xferred = DIV_ROUND_UP(bytes_xferred, 4);\n>\n> +\t\t/*\n> +\t\t * make sure we are in plain RX mode if we read a second chunk\n> +\t\t */\n> +\t\tif (snd_chunk)\n> +\t\t\tcon |= I2C_CON_MOD(I2C_MODE_RX);\n> +\t\telse\n> +\t\t\tcon |= I2C_CON_MOD(I2C_MODE_TRX);\n> +\n> \t\twritel(con, &regs->con);\n> \t\twritel(bytes_xferred, &regs->mrxcnt);\n> \t\twritel(I2C_MBRFIEN | I2C_NAKRCVIEN, &regs->ien);\n> @@ -227,6 +239,7 @@ static int rk_i2c_read(struct rk_i2c *i2c, uchar chip, uint reg, uint r_len,\n> \t\t}\n>\n> \t\tbytes_remain_len -= bytes_xferred;\n> +\t\tsnd_chunk = true;\n> \t\tdebug(\"I2C Read bytes_remain_len %d\\n\", bytes_remain_len);\n> \t}\n>\n>","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xYk3K1Pt3z9sNc\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 18 Aug 2017 23:10:37 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid D903BC22019; Fri, 18 Aug 2017 13:08:28 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 4C220C21FE9;\n\tFri, 18 Aug 2017 13:08:26 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 75E61C22005; Fri, 18 Aug 2017 13:06:10 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id F200DC21FFE\n\tfor <u-boot@lists.denx.de>; Fri, 18 Aug 2017 13:06:09 +0000 (UTC)","from [86.59.122.178] (port=34189 helo=android.lan)\n\tby mail.theobroma-systems.com with esmtps\n\t(TLS1.2:RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.80) (envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1digyW-0002ko-Rg; Fri, 18 Aug 2017 15:06:08 +0200","from [10.11.0.14] (helo=vpn-10-11-0-14.lan)\n\tby android.lan with esmtp (Exim 4.84_2)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1digyW-00033U-8W; Fri, 18 Aug 2017 15:06:08 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","Date":"Fri, 18 Aug 2017 15:06:05 +0200 (CEST)","From":"Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","X-X-Sender":"ptomsich@vpn-10-11-0-14.lan","To":"Wadim Egorov <w.egorov@phytec.de>","In-Reply-To":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","Message-ID":"<alpine.OSX.2.21.1708181505080.27138@vpn-10-11-0-14.lan>","References":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","User-Agent":"Alpine 2.21 (OSX 202 2017-01-01)","MIME-Version":"1.0","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1752514,"web_url":"http://patchwork.ozlabs.org/comment/1752514/","msgid":"<210d48bf-4322-11bd-4471-393d860990cb@phytec.de>","list_archive_url":null,"date":"2017-08-21T08:32:54","subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","submitter":{"id":68736,"url":"http://patchwork.ozlabs.org/api/people/68736/","name":"Wadim Egorov","email":"w.egorov@phytec.de"},"content":"Am 18.08.2017 um 15:06 schrieb Philipp Tomsich:\n>\n>\n> On Thu, 3 Aug 2017, Wadim Egorov wrote:\n>\n>> The hw can read up to 32 bytes at a time. If we need\n>> more than one chunk, we have to enter the plain RX mode.\n>\n> Why does this need to be in 'plain RX' mode for more than 32 bytes?\n\nI am not sure why this need to be in receive only mode. Probably that's\nhow the I2C controller operates for more than 32 byte reads?\nAnyway, if you try to read more than 32 bytes from an EEPROM, you will\nget RXDATA registers filled with 0xFF in the following chunks without\nthis patch.\nSame thing is done in the linux i2c driver:\n \nhttps://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/i2c/busses/i2c-rk3x.c?h=v4.13-rc5#n322\n\n\n> What happens if someone wants to write/transmit more than 32 bytes?\n\nShould not affect other operations as the I2C_CON_MODE will be set\nproperly in rk_i2c_read/write() calls before every I2C transmit.\n\nRegards,\nWadim\n\n>\n>>\n>> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>\n>> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>\n>> ---\n>> drivers/i2c/rk_i2c.c | 19 ++++++++++++++++---\n>> 1 file changed, 16 insertions(+), 3 deletions(-)\n>>\n>> diff --git a/drivers/i2c/rk_i2c.c b/drivers/i2c/rk_i2c.c\n>> index 8bc045a..68e6653 100644\n>> --- a/drivers/i2c/rk_i2c.c\n>> +++ b/drivers/i2c/rk_i2c.c\n>> @@ -164,6 +164,7 @@ static int rk_i2c_read(struct rk_i2c *i2c, uchar\n>> chip, uint reg, uint r_len,\n>>     uint rxdata;\n>>     uint i, j;\n>>     int err;\n>> +    bool snd_chunk = false;\n>>\n>>     debug(\"rk_i2c_read: chip = %d, reg = %d, r_len = %d, b_len = %d\\n\",\n>>           chip, reg, r_len, b_len);\n>> @@ -184,15 +185,26 @@ static int rk_i2c_read(struct rk_i2c *i2c,\n>> uchar chip, uint reg, uint r_len,\n>>\n>>     while (bytes_remain_len) {\n>>         if (bytes_remain_len > RK_I2C_FIFO_SIZE) {\n>> -            con = I2C_CON_EN | I2C_CON_MOD(I2C_MODE_TRX);\n>> +            con = I2C_CON_EN;\n>>             bytes_xferred = 32;\n>>         } else {\n>> -            con = I2C_CON_EN | I2C_CON_MOD(I2C_MODE_TRX) |\n>> -                I2C_CON_LASTACK;\n>> +            /*\n>> +             * The hw can read up to 32 bytes at a time. If we need\n>> +             * more than one chunk, send an ACK after the last byte.\n>> +             */\n>> +            con = I2C_CON_EN | I2C_CON_LASTACK;\n>>             bytes_xferred = bytes_remain_len;\n>>         }\n>>         words_xferred = DIV_ROUND_UP(bytes_xferred, 4);\n>>\n>> +        /*\n>> +         * make sure we are in plain RX mode if we read a second chunk\n>> +         */\n>> +        if (snd_chunk)\n>> +            con |= I2C_CON_MOD(I2C_MODE_RX);\n>> +        else\n>> +            con |= I2C_CON_MOD(I2C_MODE_TRX);\n>> +\n>>         writel(con, &regs->con);\n>>         writel(bytes_xferred, &regs->mrxcnt);\n>>         writel(I2C_MBRFIEN | I2C_NAKRCVIEN, &regs->ien);\n>> @@ -227,6 +239,7 @@ static int rk_i2c_read(struct rk_i2c *i2c, uchar\n>> chip, uint reg, uint r_len,\n>>         }\n>>\n>>         bytes_remain_len -= bytes_xferred;\n>> +        snd_chunk = true;\n>>         debug(\"I2C Read bytes_remain_len %d\\n\", bytes_remain_len);\n>>     }\n>>\n>>","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xbRlh0LFxz9s7v\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 21 Aug 2017 18:33:03 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid C362EC21E36; Mon, 21 Aug 2017 08:32:59 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 7A02EC21E14;\n\tMon, 21 Aug 2017 08:32:57 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 88756C21E14; Mon, 21 Aug 2017 08:32:55 +0000 (UTC)","from root.phytec.de (mail.rapiddevelopmentkit.de [217.6.246.34])\n\tby lists.denx.de (Postfix) with ESMTP id 2BE23C21C49\n\tfor <u-boot@lists.denx.de>; Mon, 21 Aug 2017 08:32:55 +0000 (UTC)","from idefix.phytec.de (idefix.phytec.de [172.16.0.10])\n\tby root.phytec.de (Postfix) with ESMTP id 82BEFA0030A;\n\tMon, 21 Aug 2017 10:34:23 +0200 (CEST)","from [172.16.26.50] ([172.16.26.50])\n\tby idefix.phytec.de (IBM Domino Release 9.0.1FP7)\n\twith ESMTP id 2017082110325486-52947 ;\n\tMon, 21 Aug 2017 10:32:54 +0200 "],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE\n\tautolearn=unavailable autolearn_force=no version=3.4.0","To":"Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","References":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>\n\t<alpine.OSX.2.21.1708181505080.27138@vpn-10-11-0-14.lan>","From":"Wadim Egorov <w.egorov@phytec.de>","Message-ID":"<210d48bf-4322-11bd-4471-393d860990cb@phytec.de>","Date":"Mon, 21 Aug 2017 10:32:54 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<alpine.OSX.2.21.1708181505080.27138@vpn-10-11-0-14.lan>","X-MIMETrack":"Itemize by SMTP Server on Idefix/Phytec(Release 9.0.1FP7|August\n\t17, 2016) at 21.08.2017 10:32:54,\n\tSerialize by Router on Idefix/Phytec(Release 9.0.1FP7|August  17,\n\t2016) at\n\t21.08.2017 10:32:54, Serialize complete at 21.08.2017 10:32:54","X-TNEFEvaluated":"1","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1763108,"web_url":"http://patchwork.ozlabs.org/comment/1763108/","msgid":"<E1dp9sQ-0005RD-J0@mail.theobroma-systems.com>","list_archive_url":null,"date":"2017-09-05T09:10:34","subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> The hw can read up to 32 bytes at a time. If we need\n> more than one chunk, we have to enter the plain RX mode.\n> \n> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>\n> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>\n> ---\n>  drivers/i2c/rk_i2c.c | 19 ++++++++++++++++---\n>  1 file changed, 16 insertions(+), 3 deletions(-)\n> \n\nReviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xmgxZ3P7hz9sPs\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue,  5 Sep 2017 19:13:38 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 1B0CBC21F8E; Tue,  5 Sep 2017 09:11:48 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 7270AC21F7A;\n\tTue,  5 Sep 2017 09:11:46 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 77F7CC21F88; Tue,  5 Sep 2017 09:10:40 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id A8D34C21F2F\n\tfor <u-boot@lists.denx.de>; Tue,  5 Sep 2017 09:10:35 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:57122\n\thelo=vpn-10-11-0-14.lan) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1dp9sQ-0005RD-J0; Tue, 05 Sep 2017 11:10:34 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","To":"Wadim Egorov <w.egorov@phytec.de>","In-Reply-To":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","References":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","Message-Id":"<E1dp9sQ-0005RD-J0@mail.theobroma-systems.com>","Date":"Tue, 05 Sep 2017 11:10:34 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1763122,"web_url":"http://patchwork.ozlabs.org/comment/1763122/","msgid":"<E1dpA83-0005sj-TE@mail.theobroma-systems.com>","list_archive_url":null,"date":"2017-09-05T09:26:43","subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> The hw can read up to 32 bytes at a time. If we need\n> more than one chunk, we have to enter the plain RX mode.\n> \n> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>\n> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>\n> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>\n> ---\n>  drivers/i2c/rk_i2c.c | 19 ++++++++++++++++---\n>  1 file changed, 16 insertions(+), 3 deletions(-)\n> \n\nApplied to u-boot-rockchip, thanks!","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xmhGH0lPVz9s0g\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue,  5 Sep 2017 19:28:07 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid C702EC21F0B; Tue,  5 Sep 2017 09:27:04 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 1AA31C21F34;\n\tTue,  5 Sep 2017 09:26:47 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 80CD8C21DD7; Tue,  5 Sep 2017 09:26:45 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id 445A3C21D8B\n\tfor <u-boot@lists.denx.de>; Tue,  5 Sep 2017 09:26:45 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:57573\n\thelo=vpn-10-11-0-14.lan) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1dpA83-0005sj-TE; Tue, 05 Sep 2017 11:26:43 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich <philipp.tomsich@theobroma-systems.com>","To":"Wadim Egorov <w.egorov@phytec.de>","In-Reply-To":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","References":"<1501760891-13982-1-git-send-email-w.egorov@phytec.de>","Message-Id":"<E1dpA83-0005sj-TE@mail.theobroma-systems.com>","Date":"Tue, 05 Sep 2017 11:26:43 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] rockchip: i2c: fix >32 byte reads","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]