[{"id":1766289,"web_url":"http://patchwork.ozlabs.org/comment/1766289/","msgid":"<CA+M3ks4y0u=ozKiq=xHFvcKJcBjDZDQysU6Mac+i1nEGTW6xOg@mail.gmail.com>","list_archive_url":null,"date":"2017-09-11T12:55:19","subject":"Re: [RESEND PATCH v3 0/5] Add support for the STM32F7 I2C","submitter":{"id":66874,"url":"http://patchwork.ozlabs.org/api/people/66874/","name":"Benjamin Gaignard","email":"benjamin.gaignard@linaro.org"},"content":"2017-09-01 9:34 GMT+02:00 Pierre-Yves MORDRET <pierre-yves.mordret@st.com>:\n> This patchset adds support for the I2C controller embedded in STM32F7xx SoC.\n> It enables I2C transfer in interrupt mode with Standard-mode, Fast-mode and\n> Fast-mode+ bus speed.\n\nHi Wolfram,\n\nI notice that those patches aren't in pull request for 4.14 and not in\ni2c-next branch.\nWhat can we do to progress in this topic? Does issues/remarks still\nneed to be fixed in this code ?\n\nThanks for your advices,\nBenjamin\n\n> ---\n>  Version history:\n>     v3:\n>         * Move stm32f7_i2c_match above stm32f7_i2c_driver\n>         * of_device_get_match_data instead of of_match_device\n>         * Improve I2C Speed DT gathering\n>         * dev_err into dev_dbg for Arbitration loss\n>         * Remove useless space aligned\n>\n>     v2:\n>         * Implement an I2C timings computation algorithm instead of static\n>           values(bindings). Algorithm uses generic I2C SCL Falling/Rising\n>           bindings and System clock to compute its timings.\n>         * I2C Device Tree Update\n> ---\n> Pierre-Yves MORDRET (5):\n>   dt-bindings: i2c-stm32: Document the STM32F7 I2C bindings\n>   i2c: i2c-stm32f4: use generic definition of speed enum\n>   i2c: i2c-stm32f7: add driver\n>   ARM: dts: stm32: Add I2C1 support for STM32F746 SoC\n>   ARM: dts: stm32: Add I2C1 support for STM32F746 eval board\n>\n>  .../devicetree/bindings/i2c/i2c-stm32.txt          |  29 +-\n>  arch/arm/boot/dts/stm32746g-eval.dts               |   8 +\n>  arch/arm/boot/dts/stm32f746.dtsi                   |  22 +\n>  drivers/i2c/busses/Kconfig                         |  10 +\n>  drivers/i2c/busses/Makefile                        |   1 +\n>  drivers/i2c/busses/i2c-stm32.h                     |  20 +\n>  drivers/i2c/busses/i2c-stm32f4.c                   |  18 +-\n>  drivers/i2c/busses/i2c-stm32f7.c                   | 974 +++++++++++++++++++++\n>  8 files changed, 1068 insertions(+), 14 deletions(-)\n>  create mode 100644 drivers/i2c/busses/i2c-stm32.h\n>  create mode 100644 drivers/i2c/busses/i2c-stm32f7.c\n>\n> --\n> 2.7.4\n>\n>\n> _______________________________________________\n> linux-arm-kernel mailing list\n> linux-arm-kernel@lists.infradead.org\n> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel","headers":{"Return-Path":"<devicetree-owner@vger.kernel.org>","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"fTsHno41\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrSZv5wwWz9s7B\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tMon, 11 Sep 2017 22:55:35 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752000AbdIKMzW (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tMon, 11 Sep 2017 08:55:22 -0400","from mail-io0-f170.google.com ([209.85.223.170]:36224 \"EHLO\n\tmail-io0-f170.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751774AbdIKMzV (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Mon, 11 Sep 2017 08:55:21 -0400","by mail-io0-f170.google.com with SMTP id d16so25381950ioj.3\n\tfor <devicetree@vger.kernel.org>;\n\tMon, 11 Sep 2017 05:55:21 -0700 (PDT)","by 10.157.80.140 with HTTP; Mon, 11 Sep 2017 05:55:19 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc:content-transfer-encoding;\n\tbh=01X+qECgzdXFN/eTFfePsmQl1LROrdjfg9/mGYNips8=;\n\tb=fTsHno41rdVICapwjnZB0nvwRqjxNChuRaxt80+uX0z45rL2cYbbeHDjyqcfLEkk6V\n\tZAOvwhXjnW5xcRn3JBfTs3MjNSqRVHf3XQijhgE1yNDFUsnGK9J6VI46rClLJo+6bpsw\n\tm/+cQVm4kbt8am2ZqhFgVNsAyx2ClWKX4uWss=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc:content-transfer-encoding;\n\tbh=01X+qECgzdXFN/eTFfePsmQl1LROrdjfg9/mGYNips8=;\n\tb=eYw6HPpPekFg0SLi8BpJ50GiziEhveDVyBtat5zbbnyvaL3inFEVfzPqt6S00CltOp\n\tgNKNwNytDF4keDw0b6z55GmvZT4weKo9EMsh3fi2SkB7uuZg8H0aBnOlBysfPD03N44K\n\taqzs0xnMwkb7s+MA3D8Lqoz8OIN9vM8BiUjCXEx2Vvt+1eUMIu7QiTyuzxjfycympxRc\n\torYfq+UB2UOXoemn55DK7x3HZ4OOSneOFI4nZoKRAwqYnr8rC0BivWcjJF/R4B0yj0P0\n\tRomriEe1wCrrDlfXNQFqMlSn7h1Z7HlDl7iyc6+gnF37Oddaym5wrawadF3JKAISfFFL\n\t+jUA==","X-Gm-Message-State":"AHPjjUjLwOPjbM+WFB360p4Hp/v9UqPINWxWhBUC4jm3KpWp0keRZkoF\n\th9NPJuDK66Ea8zENJgaOjTWQxbqO1kDE","X-Google-Smtp-Source":"AOwi7QAqU3sL97FS9xB4eBwWzCwo0s6a+Vl7BC5Y300QikQY0CJgMyjy+oQb6ZiTjCjUMB7Ew2DxfTefSyok8rW1uiA=","X-Received":"by 10.202.3.213 with SMTP id 204mr12016015oid.205.1505134520423; \n\tMon, 11 Sep 2017 05:55:20 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1504251255-20469-1-git-send-email-pierre-yves.mordret@st.com>","References":"<1504251255-20469-1-git-send-email-pierre-yves.mordret@st.com>","From":"Benjamin Gaignard <benjamin.gaignard@linaro.org>","Date":"Mon, 11 Sep 2017 14:55:19 +0200","Message-ID":"<CA+M3ks4y0u=ozKiq=xHFvcKJcBjDZDQysU6Mac+i1nEGTW6xOg@mail.gmail.com>","Subject":"Re: [RESEND PATCH v3 0/5] Add support for the STM32F7 I2C","To":"Pierre-Yves MORDRET <pierre-yves.mordret@st.com>","Cc":"Wolfram Sang <wsa@the-dreams.de>, Rob Herring <robh+dt@kernel.org>,\n\tMark Rutland <mark.rutland@arm.com>,\n\tMaxime Coquelin <mcoquelin.stm32@gmail.com>,\n\tAlexandre Torgue <alexandre.torgue@st.com>,\n\tRussell King <linux@armlinux.org.uk>,\n\tlinux-i2c@vger.kernel.org, devicetree@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org,\n\tLinux Kernel Mailing List <linux-kernel@vger.kernel.org>","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<devicetree.vger.kernel.org>","X-Mailing-List":"devicetree@vger.kernel.org"}},{"id":1768194,"web_url":"http://patchwork.ozlabs.org/comment/1768194/","msgid":"<20170913212639.2zyvg24364ovwfxh@ninjato>","list_archive_url":null,"date":"2017-09-13T21:26:39","subject":"Re: [RESEND PATCH v3 3/5] i2c: i2c-stm32f7: add driver","submitter":{"id":22495,"url":"http://patchwork.ozlabs.org/api/people/22495/","name":"Wolfram Sang","email":"wsa@the-dreams.de"},"content":"Hi,\n\nthanks for this driver!\n\n> +/**\n> + * struct stm32f7_i2c_spec - private i2c specification timing\n> + * @rate: I2C bus speed (Hz)\n> + * @rate_min: 80% of I2C bus speed (Hz)\n> + * @rate_max: 120% of I2C bus speed (Hz)\n\nYou would generate a clock which is higher than the requested one?\nThis is highly unusual. Any special reason?\n\n> + * @fall_max: Max fall time of both SDA and SCL signals (ns)\n> + * @rise_max: Max rise time of both SDA and SCL signals (ns)\n> + * @hddat_min: Min data hold time (ns)\n> + * @vddat_max: Max data valid time (ns)\n> + * @sudat_min: Min data setup time (ns)\n> + * @l_min: Min low period of the SCL clock (ns)\n> + * @h_min: Min high period of the SCL clock (ns)\n> + */\n> +static struct stm32f7_i2c_spec i2c_specs[] = {\n> +\t[STM32_I2C_SPEED_STANDARD] = {\n> +\t\t.rate = 100000,\n> +\t\t.rate_min = 8000,\n\nThis is not 80%. Typo?\n\n> +\t\t.rate_max = 120000,\n> +\t\t.fall_max = 300,\n> +\t\t.rise_max = 1000,\n> +\t\t.hddat_min = 0,\n> +\t\t.vddat_max = 3450,\n> +\t\t.sudat_min = 250,\n> +\t\t.l_min = 4700,\n> +\t\t.h_min = 4000,\n> +\t},\n\n...\n\n> +\t/*\n> +\t * Among Prescaler possibilities discovered above figures out SCL Low\n> +\t * and High Period. Provided:\n> +\t * - SCL Low Period has to be higher than Low Period of tehs SCL Clock\n\ntehs?\n\n> +\t *   defined by I2C Specification. I2C Clock has to be lower than\n> +\t *   (SCL Low Period - Analog/Digital filters) / 4.\n> +\t * - SCL High Period has to be lower than High Period of the SCL Clock\n> +\t *   defined by I2C Specification\n> +\t * - I2C Clock has to be lower than SCL High Period\n> +\t */\n\n...\n\n> +\t/* NACK received */\n> +\tif (status & STM32F7_I2C_ISR_NACKF) {\n> +\t\tdev_dbg(i2c_dev->dev, \"<%s>: Receive NACK\\n\", __func__);\n> +\t\twritel_relaxed(STM32F7_I2C_ICR_NACKCF, base + STM32F7_I2C_ICR);\n> +\t\tf7_msg->result = -EBADE;\n\n-ENXIO (see Documentation/i2c/fault-codes)\n\n...\n\n> +\ttimeout = wait_for_completion_timeout(&i2c_dev->complete,\n> +\t\t\t\t\t      i2c_dev->adap.timeout);\n> +\tret = f7_msg->result;\n> +\n> +\tif (!timeout) {\n> +\t\tdev_dbg(i2c_dev->dev, \"Access to slave 0x%x timed out\\n\",\n> +\t\t\ti2c_dev->msg->addr);\n> +\t\tret = -ETIMEDOUT;\n> +\t}\n\nCould you rename the variable to time_left? It looks strange, basically:\n\n\tif (!timeout)\n\t\treturn -ETIMEDOUT\n\n...\n\n> +\tadap->retries = 0;\n\nWhy no retries when you check for arbitration lost?\n\n> +\tadap->algo = &stm32f7_i2c_algo;\n> +\tadap->dev.parent = &pdev->dev;\n> +\tadap->dev.of_node = pdev->dev.of_node;\n> +\n> +\tinit_completion(&i2c_dev->complete);\n> +\n> +\tret = i2c_add_adapter(adap);\n> +\tif (ret) {\n> +\t\tdev_err(&pdev->dev, \"Failed to add adapter\\n\");\n\nPlease remove, the core will print info when adding fails.\n\n\nRest looks good!\n\nThanks,\n\n   Wolfram","headers":{"Return-Path":"<devicetree-owner@vger.kernel.org>","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=<UNKNOWN>)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xsvql6ghyz9s81\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 07:26:43 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751137AbdIMV0m (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tWed, 13 Sep 2017 17:26:42 -0400","from sauhun.de ([88.99.104.3]:43526 \"EHLO pokefinder.org\"\n\trhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP\n\tid S1751121AbdIMV0l (ORCPT <rfc822;devicetree@vger.kernel.org>);\n\tWed, 13 Sep 2017 17:26:41 -0400","from localhost (p54B333DB.dip0.t-ipconnect.de [84.179.51.219])\n\tby pokefinder.org (Postfix) with ESMTPSA id 21A9B2C3256;\n\tWed, 13 Sep 2017 23:26:40 +0200 (CEST)"],"Date":"Wed, 13 Sep 2017 23:26:39 +0200","From":"Wolfram Sang <wsa@the-dreams.de>","To":"Pierre-Yves MORDRET <pierre-yves.mordret@st.com>","Cc":"Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>,\n\tMaxime Coquelin <mcoquelin.stm32@gmail.com>,\n\tAlexandre Torgue <alexandre.torgue@st.com>,\n\tRussell King <linux@armlinux.org.uk>,\n\tlinux-i2c@vger.kernel.org, devicetree@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org","Subject":"Re: [RESEND PATCH v3 3/5] i2c: i2c-stm32f7: add driver","Message-ID":"<20170913212639.2zyvg24364ovwfxh@ninjato>","References":"<1504251255-20469-1-git-send-email-pierre-yves.mordret@st.com>\n\t<1504251255-20469-4-git-send-email-pierre-yves.mordret@st.com>","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha256;\n\tprotocol=\"application/pgp-signature\"; boundary=\"3nttgcsxruquswtt\"","Content-Disposition":"inline","In-Reply-To":"<1504251255-20469-4-git-send-email-pierre-yves.mordret@st.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<devicetree.vger.kernel.org>","X-Mailing-List":"devicetree@vger.kernel.org"}},{"id":1768404,"web_url":"http://patchwork.ozlabs.org/comment/1768404/","msgid":"<63ecfac9-990b-e6c5-d9bf-7708bacc10d2@st.com>","list_archive_url":null,"date":"2017-09-14T08:11:11","subject":"Re: [RESEND PATCH v3 3/5] i2c: i2c-stm32f7: add driver","submitter":{"id":71499,"url":"http://patchwork.ozlabs.org/api/people/71499/","name":"Pierre Yves MORDRET","email":"pierre-yves.mordret@st.com"},"content":"On 09/13/2017 11:26 PM, Wolfram Sang wrote:\n> Hi,\n> \n> thanks for this driver!\n> \n>> +/**\n>> + * struct stm32f7_i2c_spec - private i2c specification timing\n>> + * @rate: I2C bus speed (Hz)\n>> + * @rate_min: 80% of I2C bus speed (Hz)\n>> + * @rate_max: 120% of I2C bus speed (Hz)\n> \n> You would generate a clock which is higher than the requested one?\n> This is highly unusual. Any special reason?\n\nWell. I allow the clock to be higher than expected.\nLooking at I2C spec again it turns out the mode specifies the max: no overshoot\nof the clock. I will lock max to 100% then.\nWill be fixed\n\n> \n>> + * @fall_max: Max fall time of both SDA and SCL signals (ns)\n>> + * @rise_max: Max rise time of both SDA and SCL signals (ns)\n>> + * @hddat_min: Min data hold time (ns)\n>> + * @vddat_max: Max data valid time (ns)\n>> + * @sudat_min: Min data setup time (ns)\n>> + * @l_min: Min low period of the SCL clock (ns)\n>> + * @h_min: Min high period of the SCL clock (ns)\n>> + */\n>> +static struct stm32f7_i2c_spec i2c_specs[] = {\n>> +\t[STM32_I2C_SPEED_STANDARD] = {\n>> +\t\t.rate = 100000,\n>> +\t\t.rate_min = 8000,\n> \n> This is not 80%. Typo?\n\nYep. This is a typo\n\n> \n>> +\t\t.rate_max = 120000,\n>> +\t\t.fall_max = 300,\n>> +\t\t.rise_max = 1000,\n>> +\t\t.hddat_min = 0,\n>> +\t\t.vddat_max = 3450,\n>> +\t\t.sudat_min = 250,\n>> +\t\t.l_min = 4700,\n>> +\t\t.h_min = 4000,\n>> +\t},\n> \n> ...\n> \n>> +\t/*\n>> +\t * Among Prescaler possibilities discovered above figures out SCL Low\n>> +\t * and High Period. Provided:\n>> +\t * - SCL Low Period has to be higher than Low Period of tehs SCL Clock\n> \n> tehs?\n\nOops.\n\n> \n>> +\t *   defined by I2C Specification. I2C Clock has to be lower than\n>> +\t *   (SCL Low Period - Analog/Digital filters) / 4.\n>> +\t * - SCL High Period has to be lower than High Period of the SCL Clock\n>> +\t *   defined by I2C Specification\n>> +\t * - I2C Clock has to be lower than SCL High Period\n>> +\t */\n> \n> ...\n> \n>> +\t/* NACK received */\n>> +\tif (status & STM32F7_I2C_ISR_NACKF) {\n>> +\t\tdev_dbg(i2c_dev->dev, \"<%s>: Receive NACK\\n\", __func__);\n>> +\t\twritel_relaxed(STM32F7_I2C_ICR_NACKCF, base + STM32F7_I2C_ICR);\n>> +\t\tf7_msg->result = -EBADE;\n> \n> -ENXIO (see Documentation/i2c/fault-codes)\n\nOK\n\n> \n> ...\n> \n>> +\ttimeout = wait_for_completion_timeout(&i2c_dev->complete,\n>> +\t\t\t\t\t      i2c_dev->adap.timeout);\n>> +\tret = f7_msg->result;\n>> +\n>> +\tif (!timeout) {\n>> +\t\tdev_dbg(i2c_dev->dev, \"Access to slave 0x%x timed out\\n\",\n>> +\t\t\ti2c_dev->msg->addr);\n>> +\t\tret = -ETIMEDOUT;\n>> +\t}\n> \n> Could you rename the variable to time_left? It looks strange, basically:\n> \n> \tif (!timeout)\n> \t\treturn -ETIMEDOUT\n> \n\nokay\n\n> ...\n> \n>> +\tadap->retries = 0;\n> \n> Why no retries when you check for arbitration lost?\n> \n>> +\tadap->algo = &stm32f7_i2c_algo;\n>> +\tadap->dev.parent = &pdev->dev;\n>> +\tadap->dev.of_node = pdev->dev.of_node;\n>> +\n>> +\tinit_completion(&i2c_dev->complete);\n>> +\n>> +\tret = i2c_add_adapter(adap);\n>> +\tif (ret) {\n>> +\t\tdev_err(&pdev->dev, \"Failed to add adapter\\n\");\n> \n> Please remove, the core will print info when adding fails.\n> \n\nI will\n\n> \n> Rest looks good!\n\nGreat !\n\n> \n> Thanks,\n> \n>    Wolfram\n> \n\nThanks\n--\nTo unsubscribe from this list: send the line \"unsubscribe devicetree\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at  http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"<devicetree-owner@vger.kernel.org>","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=<UNKNOWN>)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xtB8X3TF8z9sP1\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 18:12:12 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751255AbdINIMK (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tThu, 14 Sep 2017 04:12:10 -0400","from mx08-00178001.pphosted.com ([91.207.212.93]:35146 \"EHLO\n\tmx07-00178001.pphosted.com\" rhost-flags-OK-OK-OK-FAIL)\n\tby vger.kernel.org with ESMTP id S1751206AbdINIMJ (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Thu, 14 Sep 2017 04:12:09 -0400","from pps.filterd (m0046660.ppops.net [127.0.0.1])\n\tby mx08-.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id\n\tv8E898vS026290; Thu, 14 Sep 2017 10:11:24 +0200","from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35])\n\tby mx08-00178001.pphosted.com with ESMTP id 2cv5e55xef-1\n\t(version=TLSv1 cipher=ECDHE-RSA-AES256-SHA bits=256 verify=NOT);\n\tThu, 14 Sep 2017 10:11:24 +0200","from zeta.dmz-eu.st.com (zeta.dmz-eu.st.com [164.129.230.9])\n\tby beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 8DA8534;\n\tThu, 14 Sep 2017 08:11:22 +0000 (GMT)","from Webmail-eu.st.com (sfhdag5node2.st.com [10.75.127.14])\n\tby zeta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 5F12A15BA;\n\tThu, 14 Sep 2017 08:11:22 +0000 (GMT)","from [10.201.23.236] (10.75.127.45) by SFHDAG5NODE2.st.com\n\t(10.75.127.14) with Microsoft SMTP Server (TLS) id 15.0.1178.4;\n\tThu, 14 Sep 2017 10:11:21 +0200"],"Subject":"Re: [RESEND PATCH v3 3/5] i2c: i2c-stm32f7: add driver","To":"Wolfram Sang <wsa@the-dreams.de>","CC":"Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>,\n\tMaxime Coquelin <mcoquelin.stm32@gmail.com>,\n\tAlexandre Torgue <alexandre.torgue@st.com>,\n\tRussell King <linux@armlinux.org.uk>,\n\t<linux-i2c@vger.kernel.org>, <devicetree@vger.kernel.org>,\n\t<linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>","References":"<1504251255-20469-1-git-send-email-pierre-yves.mordret@st.com>\n\t<1504251255-20469-4-git-send-email-pierre-yves.mordret@st.com>\n\t<20170913212639.2zyvg24364ovwfxh@ninjato>","From":"Pierre Yves MORDRET <pierre-yves.mordret@st.com>","Message-ID":"<63ecfac9-990b-e6c5-d9bf-7708bacc10d2@st.com>","Date":"Thu, 14 Sep 2017 10:11:11 +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":"<20170913212639.2zyvg24364ovwfxh@ninjato>","Content-Type":"text/plain; charset=\"windows-1252\"","Content-Language":"en-US","Content-Transfer-Encoding":"7bit","X-Originating-IP":"[10.75.127.45]","X-ClientProxiedBy":"SFHDAG1NODE3.st.com (10.75.127.3) To SFHDAG5NODE2.st.com\n\t(10.75.127.14)","X-Proofpoint-Virus-Version":"vendor=fsecure engine=2.50.10432:, ,\n\tdefinitions=2017-09-14_02:, , signatures=0","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<devicetree.vger.kernel.org>","X-Mailing-List":"devicetree@vger.kernel.org"}}]