[{"id":1769982,"web_url":"http://patchwork.ozlabs.org/comment/1769982/","msgid":"<20170918083629.qn4dlrmk7ffipfsz@dell>","list_archive_url":null,"date":"2017-09-18T08:36:29","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":12720,"url":"http://patchwork.ozlabs.org/api/people/12720/","name":"Lee Jones","email":"lee.jones@linaro.org"},"content":"On Sun, 17 Sep 2017, Linus Walleij wrote:\n\n> This converts the GPIO-based I2C-driver to using GPIO\n> descriptors instead of the old global numberspace-based\n> GPIO interface. We:\n\n[STUFF]\n\n> - The MFD Silicon Motion SM501 is a special case. It dynamically\n>   spawns an I2C bus off the MFD using sm501_create_subdev().\n>   We use an approach to dynamically create a machine descriptor\n>   table and attach this to the \"SM501-LOW\" or \"SM501-HIGH\"\n>   gpiochip. We use chip-local offsets to grab the right lines.\n>   We can get rid of two local static inline helpers as part\n>   of this refactoring.\n> \n> Cc: Steven Miao <realmz6@gmail.com>\n> Cc: Ralf Baechle <ralf@linux-mips.org>\n> Cc: Lee Jones <lee.jones@linaro.org>\n> Cc: Guenter Roeck <linux@roeck-us.net>\n> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>\n> Cc: Magnus Damm <magnus.damm@gmail.com>\n> Cc: Ben Dooks <ben.dooks@codethink.co.uk>\n> Cc: Heiko Schocher <hs@denx.de>\n> Acked-by: Olof Johansson <olof@lixom.net>\n> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>\n> ---\n> ChangeLog v1->v2:\n> - Fix a minor typo in error path (scl was sda from copy-paste)\n> - Collected Olof's ACK\n> \n> Steven (Blackfin): requesting ACK for Wolfram to take this patch.\n> Ralf (MIPS): requesting ACK for Wolfram to take this patch.\n> Lee: requesting ACK for Wolfram to take this patch.\n\nThis ...\n\n> SM501 users: requesting Tested-by on this patch.\n\n... loosely depends on this (until it doesn't).","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"ukgih/L2\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"JLZhB2E+\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xwfWL1dPfz9rxm\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tMon, 18 Sep 2017 18:37:02 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtrY2-0007U5-P6; Mon, 18 Sep 2017 08:36:58 +0000","from mail-wr0-x236.google.com ([2a00:1450:400c:c0c::236])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtrXy-0007NB-19 for linux-arm-kernel@lists.infradead.org;\n\tMon, 18 Sep 2017 08:36:56 +0000","by mail-wr0-x236.google.com with SMTP id z39so5547211wrb.8\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tMon, 18 Sep 2017 01:36:33 -0700 (PDT)","from dell ([2.27.167.120]) by smtp.gmail.com with ESMTPSA id\n\tf5sm4929078wmg.10.2017.09.18.01.36.30\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tMon, 18 Sep 2017 01:36:31 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=hxBopa5Zmx2Bt5TuzFtLJTv3w4hT7ua2eK1pClKK394=;\n\tb=ukgih/L2+Tqqqe\n\tngstd+mXxskGfYHCtCEoRtCWF5R5yb/INHQxobBerDY0ulO9ewZxiRCj+NUFaADL8lUYOcXZOPl7E\n\tFv2o74HBuDDzhzK1GeP4GeX9EJjX8IL2rFQZRexvO/pleXECs5fhK9Aj10q3/vVJZFJbLoMQCQgD0\n\tghuBXYa3WpjP3L/flSWDddNmmJkWY+/FGyWGtbnKmTFYdq9jA/1tndZUTSeveZuJGSZ4A76Lr/U9A\n\tETbTi4dMY4gu9soGs2rDm1q1m5ucyZ+HQpEl6i3JaCpeou2aLe8PuS9a8SoGZs5lSG9LiNcCA5RQ8\n\t7dDxsYCetazLsSmI5ZLw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:content-transfer-encoding:in-reply-to\n\t:user-agent; bh=HstyGEfXQC8OQ5uowO99SVT9LXS5lGU+bP+avqJvkho=;\n\tb=JLZhB2E+sOXsJEOZKeJ2VeRIjnmOq2axVAZGsRknBM8gppVHBtufqA2QM5WDbnM5Z5\n\trWoeVRHsYty+07EU956A6zOA+uSPocIqVm+FHJEKLx9/hKgkWkLq06/18qbcJRyzX1X/\n\tJgKTu6HxK2+mfISL3yAWipl9pOWYomM8gcWu0="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:content-transfer-encoding\n\t:in-reply-to:user-agent;\n\tbh=HstyGEfXQC8OQ5uowO99SVT9LXS5lGU+bP+avqJvkho=;\n\tb=ME8X8+0ZF5WItZG70+4ipebzBi7c8zSrK7LWkq6fIELJojvRSXtm6MTVbOVr5JFKgn\n\trWb6lVUZBsROhOfkAitOdlhChXjXOVqXPWY1+dueeudmFYt4baEZkgy1sXXHxb0V/Bd7\n\tfKW3VqSGIhUg/VupoNfYGJtt0/v9rVh0ZV7VmMeb7Min2BzKlzaJpDJwE7qae/gFsGM2\n\tBKHK8sNeF6qKx/Gupwa1q1oOjxu1vyolRitzlp0iUPrVaN7YrplglboTM7djxEtFSatO\n\tlSl1YrTHdDl4jn2nc7lx7QvXI68FVGMndUkX+7klw4G9qlIG4iWyM6INhkccxl5u8QE/\n\txCeA==","X-Gm-Message-State":"AHPjjUgdu8IlVxS+Iw1KCPC/uf5XurVBD1JRUtAm8o6nwdiG1UyV4VUn\n\t5endnw4y1P2gTdgs","X-Google-Smtp-Source":"ADKCNb46KVzRCH0ZDlSgOoPdhuC0HUFD1XIZsZpQKX3DWKHyrmgXK0HYvEkj/8vHbwf/jHf5SZ2IFQ==","X-Received":"by 10.223.195.144 with SMTP id\n\tp16mr30841274wrf.123.1505723792017; \n\tMon, 18 Sep 2017 01:36:32 -0700 (PDT)","Date":"Mon, 18 Sep 2017 09:36:29 +0100","From":"Lee Jones <lee.jones@linaro.org>","To":"Linus Walleij <linus.walleij@linaro.org>","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","Message-ID":"<20170918083629.qn4dlrmk7ffipfsz@dell>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170917093906.16325-2-linus.walleij@linaro.org>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170918_013654_513902_D8950D3D ","X-CRM114-Status":"GOOD (  12.73  )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.0 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c0c:0:0:0:236 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"linux-mips@linux-mips.org, Guenter Roeck <linux@roeck-us.net>, Wolfram\n\tSang <wsa@the-dreams.de>, Steven Miao <realmz6@gmail.com>, Magnus Damm\n\t<magnus.damm@gmail.com>, adi-buildroot-devel@lists.sourceforge.net, \n\tRalf Baechle <ralf@linux-mips.org>, Ville =?iso-8859-1?q?Syrj=E4l=E4?=\n\t<ville.syrjala@linux.intel.com>,  Ben Dooks <ben.dooks@codethink.co.uk>,\n\tGeert Uytterhoeven <geert@linux-m68k.org>, linux-i2c@vger.kernel.org, \n\tHeiko Schocher <hs@denx.de>, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1770428,"web_url":"http://patchwork.ozlabs.org/comment/1770428/","msgid":"<CACRpkdb8jDRZxuTUNoosLORqDbF4PP4RSHj3avtUqzvW2RwpmQ@mail.gmail.com>","list_archive_url":null,"date":"2017-09-18T19:00:18","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":7055,"url":"http://patchwork.ozlabs.org/api/people/7055/","name":"Linus Walleij","email":"linus.walleij@linaro.org"},"content":"On Mon, Sep 18, 2017 at 10:36 AM, Lee Jones <lee.jones@linaro.org> wrote:\n> On Sun, 17 Sep 2017, Linus Walleij wrote:\n\n>> Lee: requesting ACK for Wolfram to take this patch.\n>\n> This ...\n>\n>> SM501 users: requesting Tested-by on this patch.\n>\n> ... loosely depends on this (until it doesn't).\n\nYeah I did my best to scout around the commit logs\nto figure out who's been contributing to the SM501 and\nusing it, I hope someone will step up and test it.\n\nLinus","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"QmHgbVsp\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"DHjg//ou\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xwwM76kXYz9s78\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 05:00:51 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1du1Hk-0002oq-EH; Mon, 18 Sep 2017 19:00:48 +0000","from mail-io0-x234.google.com ([2607:f8b0:4001:c06::234])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1du1He-0002jP-LU for linux-arm-kernel@lists.infradead.org;\n\tMon, 18 Sep 2017 19:00:46 +0000","by mail-io0-x234.google.com with SMTP id h66so4536669ioh.11\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tMon, 18 Sep 2017 12:00:22 -0700 (PDT)","by 10.79.164.78 with HTTP; Mon, 18 Sep 2017 12:00:18 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=bPFuk0hQnGNkB/2WBg2PDiA/7O3Kv982uQA4t1FWnk8=;\n\tb=QmHgbVsp23A+4l\n\t2W8CZ6oxSYt02hDKg7v5qRzYOQvaG+Yns/aM9xH/jltgEzkkOmgqS+FO1EDADPXAJy1thU3U/terr\n\to0HYN+Hkb+T3LT3OyESYU7ROaTpHaXe1WRolWpAe2hoFD8QRKTZwIV1Khw9ZmASTHGsvKLgc+bDyu\n\tDlvbD44Wsv0dI4xWfUFfKDlPaB58yGn06gEYvL3Eyc7FqMLYQlKf3Jogr1E81U0B2eKzAFnxDHiyk\n\teQ7pXov664v459z9BFAo3aT8F0qH2psz8GkjB3vkql60+mPSfa35+3FBtXkI9IOFu+QnraE7ff3tz\n\tBzHIV9fC6NoB2co8onSg==;","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; bh=6nLP9K73TIQJcdlvY3qC0XE//y6fHOI4ylTAtAJv73g=;\n\tb=DHjg//ouDjQD2AWZ347gq5CbmdiRtsSUOg0WUSXdr00L/6aNYhIfHPMsyqsVW2fbgt\n\tSf/n6op+vPLct1lxxyv65VhqNgCzGYuicQG/n0aiXU7vC/4BJ6snQ/fB6oOJV9FxmG4T\n\t/EjrS1oflj07Utzw95yO1yNxMKNj/oY3Q0fas="],"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;\n\tbh=6nLP9K73TIQJcdlvY3qC0XE//y6fHOI4ylTAtAJv73g=;\n\tb=cmd/A8ZtQHdRvsNkRcC4lnLyRtccQ8j3dcCmcOHAolN/nOYuGpDizhQmI/TD2LdQtX\n\thCdIU8pXg99xRSvf5+Y4XHZP/V4hzb1MULN3Q6/7aAXehllEHK20Y3iPFu2ks/gKe9Ur\n\tuS3brgESRztxOaGNFLZA9enjmb7aUhgD/VXwO3qCin5Tw57QALDd/izMXC/lbT8M57Ij\n\tUhmuAkYyhNAmyjDGAPJb0f131UUBDa1LAPqZajcYwjuf9UjZnw8KUG942uO8LjFlr03g\n\tNKyU64ebSfSSxj16asx+B/EuBOjpquIb+iSLsM+32AAN4A1sCm87zCRbPJA23A0SckEY\n\tkvgw==","X-Gm-Message-State":"AHPjjUhOzza7LYPerM3twvaAWVfCUkL6D0vGf6revafwpWu7L/Be143Y\n\tAMqqk5k5ZQqPyI7MHzl4pZ+kfn9BUc/Cy0v75JMiXA==","X-Google-Smtp-Source":"AOwi7QDRKe3IPmSNudMnBNNYHuv25Gl5oB9lL8fgLGdI/xWqnVKBOjikV/zFKoZxYOhdMxNHoHtPucU+K8oKEWQX8uI=","X-Received":"by 10.107.197.198 with SMTP id\n\tv189mr21945457iof.94.1505761219864; \n\tMon, 18 Sep 2017 12:00:19 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170918083629.qn4dlrmk7ffipfsz@dell>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>\n\t<20170918083629.qn4dlrmk7ffipfsz@dell>","From":"Linus Walleij <linus.walleij@linaro.org>","Date":"Mon, 18 Sep 2017 21:00:18 +0200","Message-ID":"<CACRpkdb8jDRZxuTUNoosLORqDbF4PP4RSHj3avtUqzvW2RwpmQ@mail.gmail.com>","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","To":"Lee Jones <lee.jones@linaro.org>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170918_120043_030763_BD208B3D ","X-CRM114-Status":"GOOD (  10.64  )","X-Spam-Score":"-2.7 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.7 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/,\n\tlow\n\ttrust [2607:f8b0:4001:c06:0:0:0:234 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Linux MIPS <linux-mips@linux-mips.org>,\n\tGuenter Roeck <linux@roeck-us.net>, Wolfram Sang <wsa@the-dreams.de>,\n\tSteven Miao <realmz6@gmail.com>, Magnus Damm <magnus.damm@gmail.com>,\n\tadi-buildroot-devel@lists.sourceforge.net, Ralf Baechle\n\t<ralf@linux-mips.org>, =?utf-8?b?VmlsbGUgU3lyasOkbMOk?=\n\t<ville.syrjala@linux.intel.com>,  Ben Dooks <ben.dooks@codethink.co.uk>,\n\tGeert Uytterhoeven <geert@linux-m68k.org>, \"linux-i2c@vger.kernel.org\"\n\t<linux-i2c@vger.kernel.org>,  Heiko Schocher <hs@denx.de>,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1770782,"web_url":"http://patchwork.ozlabs.org/comment/1770782/","msgid":"<42702eb3-d1dd-5dd0-dbd6-ac163bcc27c2@codethink.co.uk>","list_archive_url":null,"date":"2017-09-19T09:19:52","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":14023,"url":"http://patchwork.ozlabs.org/api/people/14023/","name":"Ben Dooks","email":"ben.dooks@codethink.co.uk"},"content":"On 18/09/17 20:00, Linus Walleij wrote:\n> On Mon, Sep 18, 2017 at 10:36 AM, Lee Jones <lee.jones@linaro.org> wrote:\n>> On Sun, 17 Sep 2017, Linus Walleij wrote:\n> \n>>> Lee: requesting ACK for Wolfram to take this patch.\n>>\n>> This ...\n>>\n>>> SM501 users: requesting Tested-by on this patch.\n>>\n>> ... loosely depends on this (until it doesn't).\n> \n> Yeah I did my best to scout around the commit logs\n> to figure out who's been contributing to the SM501 and\n> using it, I hope someone will step up and test it.\n\nI don't know if anything I have available working has an\nSM501 on it. We did a couple of boards with S3C2440 and\nan SM501 connected via external IO. However these have\nnot been supported in years (since Simtec)\n\nI don't even think I have the PCI SM501 kit, that may have\nended up either at Simtec or one of the other engineers.","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"UtYfDEMk\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xxHRV0f3qz9sBZ\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 19:20:54 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duEhy-0007Ot-7n; Tue, 19 Sep 2017 09:20:46 +0000","from imap1.codethink.co.uk ([176.9.8.82])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duEhs-0007IE-Mm for linux-arm-kernel@lists.infradead.org;\n\tTue, 19 Sep 2017 09:20:44 +0000","from [185.98.148.236] (helo=[0.0.0.0])\n\tby imap1.codethink.co.uk with esmtpsa (Exim 4.84_2 #1 (Debian))\n\tid 1duEh7-0005KT-Ug; Tue, 19 Sep 2017 10:19:54 +0100"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:\n\tContent-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive:\n\tList-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From:\n\tReferences:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=8sir7khCrRY8bCi5oWCh4bGFvvc/O8YkL7G4Fp+9QZw=;\n\tb=UtYfDEMkGRkbKlIkKir7oVQW1\n\tyw2QCTXwKW1uqdzQZiMemVx0QaoNvhP9CG0RSjDPE62e+1vEoJiIg6hPLc5Is1UeJczghcG7S5qHj\n\tSMKMEOX3WXkHpejOFSS6W+Qd1I+3XX8Vf13FUXbSuGIn/wBs1EWuf64zMiYhMroBFCNAG3rGo/0dw\n\t7bP/2VsWefgJMQgAw2oWPnkhxrvCpPb8aGs1rblS0riDvhJjX0vTnv3eYe0vUSF1N+kFXHmrsF148\n\tDMiqcsxS0BRs5M1FYUqwbHm9Ng+Bkb8q4KVHlNL09sE5E+u7hKwk/X1lm4XtzkJno19rmhiYJKBPG\n\twnLQYhWWA==;","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","To":"Linus Walleij <linus.walleij@linaro.org>,\n\tLee Jones <lee.jones@linaro.org>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>\n\t<20170918083629.qn4dlrmk7ffipfsz@dell>\n\t<CACRpkdb8jDRZxuTUNoosLORqDbF4PP4RSHj3avtUqzvW2RwpmQ@mail.gmail.com>","From":"Ben Dooks <ben.dooks@codethink.co.uk>","Organization":"Codethink Limited.","Message-ID":"<42702eb3-d1dd-5dd0-dbd6-ac163bcc27c2@codethink.co.uk>","Date":"Tue, 19 Sep 2017 10:19:52 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<CACRpkdb8jDRZxuTUNoosLORqDbF4PP4RSHj3avtUqzvW2RwpmQ@mail.gmail.com>","Content-Language":"en-GB","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170919_022040_915596_67D58E45 ","X-CRM114-Status":"GOOD (  12.98  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Linux MIPS <linux-mips@linux-mips.org>,\n\tGuenter Roeck <linux@roeck-us.net>, Wolfram Sang <wsa@the-dreams.de>,\n\tSteven Miao <realmz6@gmail.com>, Magnus Damm <magnus.damm@gmail.com>,\n\tadi-buildroot-devel@lists.sourceforge.net, Ralf Baechle\n\t<ralf@linux-mips.org>, =?utf-8?b?VmlsbGUgU3lyasOkbMOk?=\n\t<ville.syrjala@linux.intel.com>, Geert Uytterhoeven\n\t<geert@linux-m68k.org>, \"linux-i2c@vger.kernel.org\"\n\t<linux-i2c@vger.kernel.org>,  Heiko Schocher <hs@denx.de>,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1778132,"web_url":"http://patchwork.ozlabs.org/comment/1778132/","msgid":"<CACRpkdZ=BZHp3mjccUYaPTuMbXgwGSErvLprnp4j0H+7C5NYJQ@mail.gmail.com>","list_archive_url":null,"date":"2017-10-02T01:05:22","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":7055,"url":"http://patchwork.ozlabs.org/api/people/7055/","name":"Linus Walleij","email":"linus.walleij@linaro.org"},"content":"On Sun, Sep 17, 2017 at 11:39 AM, Linus Walleij\n<linus.walleij@linaro.org> wrote:\n\n> - The MFD Silicon Motion SM501 is a special case. It dynamically\n>   spawns an I2C bus off the MFD using sm501_create_subdev().\n>   We use an approach to dynamically create a machine descriptor\n>   table and attach this to the \"SM501-LOW\" or \"SM501-HIGH\"\n>   gpiochip. We use chip-local offsets to grab the right lines.\n>   We can get rid of two local static inline helpers as part\n>   of this refactoring.\n(...)\n> SM501 users: requesting Tested-by on this patch.\n\nPaging Simtec (if it reaches anyone), Vincent Sanders,\nTeddy Wang at Silicon Motion:\n\nDoes any of you have an \"Anubis\" board so you can test GPIO\non this board before/after this patch and see if it checks out right?\n\nI guess it's this board:\nhttp://www.simtec.co.uk/products/BBD20EUROA/\n\nDoes anyone know of a commercially obtainable product using\nSM501 with reasonable mainline Linux support so I can test it myself?\n\nGetting a bit desperate...\n\nYours,\nLinus Walleij","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"nwQXB7ki\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"TAgPkHMr\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y53rt4KsNz9t3s\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tMon,  2 Oct 2017 12:06:19 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dypBS-0001bj-Rl; Mon, 02 Oct 2017 01:06:10 +0000","from mail-io0-x22f.google.com ([2607:f8b0:4001:c06::22f])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dypB2-0001Xp-9o for linux-arm-kernel@lists.infradead.org;\n\tMon, 02 Oct 2017 01:05:47 +0000","by mail-io0-x22f.google.com with SMTP id e189so3633115ioa.4\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tSun, 01 Oct 2017 18:05:23 -0700 (PDT)","by 10.79.6.195 with HTTP; Sun, 1 Oct 2017 18:05:22 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=pWlQ/fWFMhBuvoiMZRki92NqPWwgDuh9yMbtkd3JtEA=;\n\tb=nwQXB7ki9PFjj0\n\tpOGZ7HI0rVqkepkAL66jONHI6avaXUKwyT+uRF31zIMtYpZ69gMjD70IiPUTW/YNAxcUepfALjLEM\n\tSwBxpVtIUCl10qTUssCaciXtUCEDyEM/IuB+1T/P9bx2QrycUb2s3m5Oubwcw6zPDj3RxK8IhadCc\n\tLo7INFp/+4u/l2SYPFpuXX6nEWPv1OD2AOKUoPmUdWXAEgHlvs5B5Xjb+PaToZAuHtthq+vMb85iP\n\tFflpwV9GRwZSZ3o2bl8Zs9P+5DkDXufh/7H0RMfMWLWssiNELclBpu3itv/AwKRLJirerozISQ29m\n\tBNXPsmOUnpEdFbBnySTg==;","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; bh=YdiEDqkHt02u1pb+caHosMt4TmWpdDZco/TUnhb0heI=;\n\tb=TAgPkHMrPOtP77ntmXVAS51q+I8Sb3dBhvMcqIlMkXPYMGcLoiiN9/J6K9Ek9m2zd9\n\tHMRxgwD/pacKWklQjKlHNbCM7wUM2hEer1zQKd8K2zZcOaUpa2iRDzSxdY6Rob65Puv9\n\tihfA5+qnQuzH2PuP6bPoKlYSK65jARamf8UwU="],"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;\n\tbh=YdiEDqkHt02u1pb+caHosMt4TmWpdDZco/TUnhb0heI=;\n\tb=tvIIvVVta+oDnmKNwtK9DeJbgPB9+i7SaNbkN8IWQqZOls1gH06ebY0032cTjLXB46\n\tT4T8xy6XZMeAk7+QaNbKDXQeZhrXddUiMwhqd4hHYGEr/jIZseRjdOYufgIl4E9go/m2\n\tWUi1b0+Ea5wK4pVOgDsiW0xdMPqStiFNY94LPyOKxKQId8ggHl+RGiEQimG1nY5R4/LK\n\tiI4dDqIpclPRO+CQC37vLFySzd/xvTdQmvFDyU+A3XWBAKcaTn4l/EqmRTy9S6mCPIqy\n\ttM4BhxXJ0pak75TVGCXsePBoWpvKyUL8vJr0DHZMlIrwdIvnN9EjFBEMfvQWeYhD9c5J\n\tntuQ==","X-Gm-Message-State":"AMCzsaVpEgbuhekebNdfhZ/IhIKB712/d4ubAOzRZR/VQDgUelcgWvcY\n\t903fHB5jVJxRDYQz7/NN14U6kwCh28t66eHXxXh+SQ==","X-Google-Smtp-Source":"AOwi7QAIUn2aBibG+Lblm9J7oscafQAUvCRhGeVzP9Y3Z7UcZ1AxLvXSkIGkOtBKlOve6pBzJjRLrq0IDtVvAAngcE4=","X-Received":"by 10.107.22.65 with SMTP id 62mr22943857iow.269.1506906322843; \n\tSun, 01 Oct 2017 18:05:22 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170917093906.16325-2-linus.walleij@linaro.org>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>","From":"Linus Walleij <linus.walleij@linaro.org>","Date":"Mon, 2 Oct 2017 03:05:22 +0200","Message-ID":"<CACRpkdZ=BZHp3mjccUYaPTuMbXgwGSErvLprnp4j0H+7C5NYJQ@mail.gmail.com>","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","To":"Wolfram Sang <wsa@the-dreams.de>, \n\t\"linux-i2c@vger.kernel.org\" <linux-i2c@vger.kernel.org>,\n\tSimtec Linux Team <linux@simtec.co.uk>, \n\tVincent Sanders <vincent.sanders@collabora.co.uk>,\n\tVincent Sanders <vince@kyllikki.org>, \n\tTeddy Wang <teddy.wang@siliconmotion.com>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171001_180544_391650_8532FDDB ","X-CRM114-Status":"GOOD (  12.03  )","X-Spam-Score":"-2.7 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.7 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/,\n\tlow\n\ttrust [2607:f8b0:4001:c06:0:0:0:22f listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Linux MIPS <linux-mips@linux-mips.org>, Steven Miao <realmz6@gmail.com>, \n\tLinus Walleij <linus.walleij@linaro.org>,\n\tMagnus Damm <magnus.damm@gmail.com>, \n\tadi-buildroot-devel@lists.sourceforge.net,\n\tRalf Baechle <ralf@linux-mips.org>, =?utf-8?b?VmlsbGUgU3lyasOkbMOk?=\n\t<ville.syrjala@linux.intel.com>,  Ben Dooks <ben.dooks@codethink.co.uk>,\n\tGeert Uytterhoeven\n\t<geert@linux-m68k.org>, Guenter Roeck <linux@roeck-us.net>, Heiko\n\tSchocher <hs@denx.de>, Lee Jones <lee.jones@linaro.org>, \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1778437,"web_url":"http://patchwork.ozlabs.org/comment/1778437/","msgid":"<3100653b-21b7-0a84-6379-7c38e18e2dd6@codethink.co.uk>","list_archive_url":null,"date":"2017-10-02T16:11:00","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":14023,"url":"http://patchwork.ozlabs.org/api/people/14023/","name":"Ben Dooks","email":"ben.dooks@codethink.co.uk"},"content":"On 02/10/17 02:05, Linus Walleij wrote:\n> On Sun, Sep 17, 2017 at 11:39 AM, Linus Walleij\n> <linus.walleij@linaro.org> wrote:\n> \n>> - The MFD Silicon Motion SM501 is a special case. It dynamically\n>>    spawns an I2C bus off the MFD using sm501_create_subdev().\n>>    We use an approach to dynamically create a machine descriptor\n>>    table and attach this to the \"SM501-LOW\" or \"SM501-HIGH\"\n>>    gpiochip. We use chip-local offsets to grab the right lines.\n>>    We can get rid of two local static inline helpers as part\n>>    of this refactoring.\n> (...)\n>> SM501 users: requesting Tested-by on this patch.\n> \n> Paging Simtec (if it reaches anyone), Vincent Sanders,\n> Teddy Wang at Silicon Motion:\n\nI'm fairly sure Simtec is not currently active. See below.\nI have no idea if the SM501 is even available for current designs.\n\n> Does any of you have an \"Anubis\" board so you can test GPIO\n> on this board before/after this patch and see if it checks out right?\n> \n> I guess it's this board:\n> http://www.simtec.co.uk/products/BBD20EUROA/\n\nI think that's fairly close.\nWe did have a couple of other designs done for clients, but they may\nnot have been available to the general public.\n\n> \n> Does anyone know of a commercially obtainable product using\n> SM501 with reasonable mainline Linux support so I can test it myself?\n> \n> Getting a bit desperate...\n\nAt the moment I have no time to go through the boxes of ex-Simtec stuff\nthat I still have... quite a lot of my old Simtec stuff has already been\nrecycled as it has been over five years since leaving.\n\n> Yours,\n> Linus Walleij\n>","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"p7bq2ycq\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y5Rxq4pVgz9t5s\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tTue,  3 Oct 2017 03:11:59 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dz3K0-0002CW-9V; Mon, 02 Oct 2017 16:11:56 +0000","from imap1.codethink.co.uk ([176.9.8.82])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dz3Jv-00027S-My for linux-arm-kernel@lists.infradead.org;\n\tMon, 02 Oct 2017 16:11:53 +0000","from [185.98.148.236] (helo=[0.0.0.0])\n\tby imap1.codethink.co.uk with esmtpsa (Exim 4.84_2 #1 (Debian))\n\tid 1dz3J9-0000oD-72; Mon, 02 Oct 2017 17:11:03 +0100"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:\n\tContent-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive:\n\tList-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From:\n\tReferences:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=yvqieOROg3lotHr4OdAOVB6/rm6Tsql6n1dkoVzpxJI=;\n\tb=p7bq2ycq/hBqDj2teSF1tusYA\n\tpG2pxz5s21RUgIYUWFYnVXojtKoFLsJukI9OdpIbwZ2EoUv7yefJg39FMv2FLaBgN8VEEBynCrR2x\n\t7uhtsOL+YXDjdbGiMIL9jAWiO9KxjsGYHUYjghMlLyYGmZSiFMeOW1byTiTAEnWgTgHGuek3PJdFy\n\ta/jcv7Xeq5Ndf2XboIykED+4jTgIv5j4Qs7khXEDY1zKAFoe2hcfQ+8zYJx+wpoMcztScFphNsOh7\n\t0lbzOAW6lH7Ri9IWDSOwhDOJffBsPCm1iVaTYzeaL0SUPDYLdAy8nfSOfNHtuvuZpWtc0qxfLKECs\n\tSOKnQQhqg==;","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","To":"Linus Walleij <linus.walleij@linaro.org>, Wolfram Sang\n\t<wsa@the-dreams.de>, \"linux-i2c@vger.kernel.org\"\n\t<linux-i2c@vger.kernel.org>, Simtec Linux Team <linux@simtec.co.uk>, \n\tVincent Sanders <vincent.sanders@collabora.co.uk>,\n\tVincent Sanders <vince@kyllikki.org>,\n\tTeddy Wang <teddy.wang@siliconmotion.com>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>\n\t<CACRpkdZ=BZHp3mjccUYaPTuMbXgwGSErvLprnp4j0H+7C5NYJQ@mail.gmail.com>","From":"Ben Dooks <ben.dooks@codethink.co.uk>","Organization":"Codethink Limited.","Message-ID":"<3100653b-21b7-0a84-6379-7c38e18e2dd6@codethink.co.uk>","Date":"Mon, 2 Oct 2017 17:11:00 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<CACRpkdZ=BZHp3mjccUYaPTuMbXgwGSErvLprnp4j0H+7C5NYJQ@mail.gmail.com>","Content-Language":"en-GB","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171002_091151_929857_9B673BDD ","X-CRM114-Status":"GOOD (  17.25  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Linux MIPS <linux-mips@linux-mips.org>, Steven Miao <realmz6@gmail.com>, \n\tMagnus Damm <magnus.damm@gmail.com>,\n\tadi-buildroot-devel@lists.sourceforge.net, Ralf Baechle\n\t<ralf@linux-mips.org>, =?utf-8?b?VmlsbGUgU3lyasOkbMOk?=\n\t<ville.syrjala@linux.intel.com>, Geert Uytterhoeven\n\t<geert@linux-m68k.org>, Guenter Roeck <linux@roeck-us.net>, Heiko\n\tSchocher <hs@denx.de>, Lee Jones <lee.jones@linaro.org>, \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1779670,"web_url":"http://patchwork.ozlabs.org/comment/1779670/","msgid":"<20171004105703.qmahtqieimw4kn3p@dell>","list_archive_url":null,"date":"2017-10-04T10:57:03","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":12720,"url":"http://patchwork.ozlabs.org/api/people/12720/","name":"Lee Jones","email":"lee.jones@linaro.org"},"content":"On Sun, 17 Sep 2017, Linus Walleij wrote:\n\n> This converts the GPIO-based I2C-driver to using GPIO\n> descriptors instead of the old global numberspace-based\n> GPIO interface. We:\n> \n> - Convert the driver to unconditionally grab two GPIOs\n>   from the device by index 0 (SDA) and 1 (SCL) which\n>   will work fine with device tree and descriptor tables.\n>   The existing device trees will continue to work just\n>   like before, but without any roundtrip through the\n>   global numberspace.\n> \n> - Brutally convert all boardfiles still passing global\n>   GPIOs by registering descriptor tables associated with\n>   the devices instead so this driver does not need to keep\n>   supporting passing any GPIO numbers as platform data.\n> \n> There is no stepwise approach as elegant as this, I\n> strongly prefer this big hammer over any antsteps for this\n> conversion. This way the old GPIO numbers go away and\n> NEVER COME BACK.\n> \n> Special conversion for the different boards utilizing\n> I2C-GPIO:\n> \n> - EP93xx (arch/arm/mach-ep93xx): pretty straight forward as\n>   all boards were using the same two GPIO lines, just define\n>   these two in a lookup table for \"i2c-gpio\" and register\n>   these along with the device. None of them define any\n>   other platform data so just pass NULL as platform data.\n>   This platform selects GPIOLIB so all should be smooth.\n>   The pins appear on a gpiochip for bank \"G\" as pins 1 (SDA)\n>   and 0 (SCL).\n> \n> - IXP4 (arch/arm/mach-ixp4): descriptor tables have to\n>   be registered for each board separately. They all use\n>   \"IXP4XX_GPIO_CHIP\" so it is pretty straight forward.\n>   Most board define no other platform data than SCL/SDA\n>   so they can drop the #include of <linux/i2c-gpio.h> and\n>   assign NULL to platform data.\n> \n>   The \"goramo_mlr\" (Goramo Multilink Router) board is a bit\n>   worrisome: it implements its own I2C bit-banging in the\n>   board file, and optionally registers an I2C serial port,\n>   but claims the same GPIO lines for itself in the board file.\n>   This is not going to work: there will be competition for the\n>   GPIO lines, so delete the optional extra I2C bus instead, no\n>   I2C devices are registered on it anyway, there are just hints\n>   that it may contain an EEPROM that may be accessed from\n>   userspace. This needs to be fixed up properly by the serial\n>   clock using I2C emulation so drop a note in the code.\n> \n> - KS8695 board acs5k (arch/arm/mach-ks8695/board-acs5.c)\n>   has some platform data in addition to the pins so it needs to\n>   be kept around sans GPIO lines. Its GPIO chip is named\n>   \"KS8695\" and the arch selects GPIOLIB.\n> \n> - PXA boards (arch/arm/mach-pxa/*) use some of the platform\n>   data so it needs to be preserved here. The viper board even\n>   registers two GPIO I2Cs. The gpiochip is named \"gpio-pxa\" and\n>   the arch selects GPIOLIB.\n> \n> - SA1100 Simpad (arch/arm/mach-sa1100/simpad.c) defines a GPIO\n>   I2C bus, and the arch selects GPIOLIB.\n> \n> - Blackfin boards (arch/blackfin/bf533 etc) for these I assume\n>   their I2C GPIOs refer to the local gpiochip defined in\n>   arch/blackfin/kernel/bfin_gpio.c names \"BFIN-GPIO\".\n>   The arch selects GPIOLIB. The boards get spiked with\n>   IF_ENABLED(I2C_GPIO) but that is a side effect of it\n>   being like that already (I would just have Kconfig select\n>   I2C_GPIO and get rid of them all.) I also delete any\n>   platform data set to 0 as it will get that value anyway\n>   from static declartions of platform data.\n> \n> - The MIPS selects GPIOLIB and the Alchemy machine is using\n>   two local GPIO chips, one of them has a GPIO I2C. We need\n>   to adjust the local offset from the global number space here.\n>   The ATH79 has a proper GPIO driver in drivers/gpio/gpio-ath79.c\n>   and AFAICT the chip is named \"ath79-gpio\" and the PB44\n>   PCF857x expander spawns from this on GPIO 1 and 0. The latter\n>   board only use the platform data to specify pins so it can be\n>   cut altogether after this.\n> \n> - The MFD Silicon Motion SM501 is a special case. It dynamically\n>   spawns an I2C bus off the MFD using sm501_create_subdev().\n>   We use an approach to dynamically create a machine descriptor\n>   table and attach this to the \"SM501-LOW\" or \"SM501-HIGH\"\n>   gpiochip. We use chip-local offsets to grab the right lines.\n>   We can get rid of two local static inline helpers as part\n>   of this refactoring.\n> \n> Cc: Steven Miao <realmz6@gmail.com>\n> Cc: Ralf Baechle <ralf@linux-mips.org>\n> Cc: Lee Jones <lee.jones@linaro.org>\n> Cc: Guenter Roeck <linux@roeck-us.net>\n> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>\n> Cc: Magnus Damm <magnus.damm@gmail.com>\n> Cc: Ben Dooks <ben.dooks@codethink.co.uk>\n> Cc: Heiko Schocher <hs@denx.de>\n> Acked-by: Olof Johansson <olof@lixom.net>\n> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>\n> ---\n> ChangeLog v1->v2:\n> - Fix a minor typo in error path (scl was sda from copy-paste)\n> - Collected Olof's ACK\n> \n> Steven (Blackfin): requesting ACK for Wolfram to take this patch.\n> Ralf (MIPS): requesting ACK for Wolfram to take this patch.\n> Lee: requesting ACK for Wolfram to take this patch.\n> SM501 users: requesting Tested-by on this patch.\n> ---\n>  arch/arm/mach-ep93xx/core.c                  |  39 ++++----\n>  arch/arm/mach-ep93xx/edb93xx.c               |  15 +--\n>  arch/arm/mach-ep93xx/include/mach/platform.h |   4 +-\n>  arch/arm/mach-ep93xx/simone.c                |  12 +--\n>  arch/arm/mach-ep93xx/snappercl15.c           |  12 +--\n>  arch/arm/mach-ep93xx/vision_ep9307.c         |   7 +-\n>  arch/arm/mach-ixp4xx/avila-setup.c           |  17 +++-\n>  arch/arm/mach-ixp4xx/dsmg600-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/fsg-setup.c             |  16 +++-\n>  arch/arm/mach-ixp4xx/goramo_mlr.c            |  24 ++---\n>  arch/arm/mach-ixp4xx/ixdp425-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/nas100d-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/nslu2-setup.c           |  16 +++-\n>  arch/arm/mach-ks8695/board-acs5k.c           |  13 ++-\n>  arch/arm/mach-pxa/palmz72.c                  |  12 ++-\n>  arch/arm/mach-pxa/viper.c                    |  27 +++++-\n>  arch/arm/mach-sa1100/simpad.c                |  12 ++-\n>  arch/blackfin/mach-bf533/boards/blackstamp.c |  19 +++-\n>  arch/blackfin/mach-bf533/boards/ezkit.c      |  18 +++-\n>  arch/blackfin/mach-bf533/boards/stamp.c      |  18 +++-\n>  arch/blackfin/mach-bf561/boards/ezkit.c      |  18 +++-\n>  arch/mips/alchemy/board-gpr.c                |  19 +++-\n>  arch/mips/ath79/mach-pb44.c                  |  16 +++-\n>  drivers/i2c/busses/i2c-gpio.c                | 134 +++++++++++++--------------\n>  drivers/mfd/sm501.c                          |  49 +++++-----\n\nAcked-by: Lee Jones <lee.jones@linaro.org>","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"nJDSOf6c\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"IMOK+hQJ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y6Xt663gfz9t2W\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed,  4 Oct 2017 21:57:34 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dzhMp-0004BG-5j; Wed, 04 Oct 2017 10:57:31 +0000","from mail-wm0-x229.google.com ([2a00:1450:400c:c09::229])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dzhMl-00046l-Mi for linux-arm-kernel@lists.infradead.org;\n\tWed, 04 Oct 2017 10:57:30 +0000","by mail-wm0-x229.google.com with SMTP id b189so18082601wmd.4\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tWed, 04 Oct 2017 03:57:07 -0700 (PDT)","from dell ([2.27.167.120]) by smtp.gmail.com with ESMTPSA id\n\th33sm13528445edh.70.2017.10.04.03.57.04\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 04 Oct 2017 03:57:05 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Z/I1YgSiMtsL5sT6wvlEmJ6Sg0o0eBkYtCTbggq5RrU=;\n\tb=nJDSOf6c4Jef44\n\t2PPCFzoRtoEAreg28Tcyv36vEc4DByo/zJvTJY1k0cHW2v01bdMZdra7z/agOiPzuA1jldLucGxeP\n\tsYh5fu7kFzy3NywJQkJecB5vKhV0BOL0mCIfmHrt43BbNqd7P7RwXnpGLSmFwVAaZV7de3BuUJuVb\n\tWNG1rMc56ZgkwSZ3Yesw2Kd9UpVNu1GYx1szbbEIc4voz9MK8ZFN56ZMHdpVxckBrMwDLBrsD5OvK\n\tnxhjV3+hdeigQLgFnYjd/HscYMoxRaFAn0T8iDwUpFMax44SqYf//3Rvp4cwEu5AQnJ5cv/Aj/Tij\n\tyZl9rjzGIb9UtMVwCsdg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:content-transfer-encoding:in-reply-to\n\t:user-agent; bh=g/+1E/FvCLLpqRNlBJJtsv9jHbbXVzj0tq0l82riPAI=;\n\tb=IMOK+hQJpd7T01oxR8jmVwL2i2Q7aAE4o6bOj/bOJsvxkoojcj6TcXznMbMmqSF65c\n\tiCreH+7exoi0V5lc0GTFa0x3C7Gh3VR9CMPbaOO7mERnHGtocFeXZZxe340VDGOWV4N5\n\tbP3fch/f3tCfqgmrsnvkRR4297C7ecvS/4NqQ="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:content-transfer-encoding\n\t:in-reply-to:user-agent;\n\tbh=g/+1E/FvCLLpqRNlBJJtsv9jHbbXVzj0tq0l82riPAI=;\n\tb=A9J1aUqgpJ3p6628ct2Lj/uevh0qe4fvdQqvhqUQiilNENUnXs7hFwflAlrg0YLY5k\n\t0qsUZ5cOqXiAYaM0/qNNxzgtjs55WhwmDBLE6W4MbafMmbPOAPPFsx7xphkX+K4bnpON\n\twrKZe+Tt+1dW54IAR4WZeKK7LEoKqgbxVsi7NBCLfEyr6DnCURA+1LRyTIXwEsAQyMSY\n\tWDCtDhdi7PPwZjhmRuTLvCOcUXMPPB8sQmSf7TB1nFEiWs22SNlTAEuryRGT4/ud8dk3\n\tIAu/lI5Jk4xWwBtxNsJlwD8Ron/zDwVd4fIv3SDkT/1R2K5o8tR+/ifWBpNq+48Q/gH8\n\th4JQ==","X-Gm-Message-State":"AHPjjUg3dBhmEUspkyMM/r0MFjAxvqt4EdvVnwm73/nWaJfUnY2/ZIVr\n\tQa5h5YS3pO+pICJa2q9Bh/gnbA==","X-Google-Smtp-Source":"AOwi7QAXmf4V5xxqGsEyyv14J9FEIqG9vZruV1b3Am2sycy/02/yTVIx6yk1xd/lTdpZEhMmPxPIbg==","X-Received":"by 10.80.212.40 with SMTP id t40mr28803272edh.67.1507114625862; \n\tWed, 04 Oct 2017 03:57:05 -0700 (PDT)","Date":"Wed, 4 Oct 2017 11:57:03 +0100","From":"Lee Jones <lee.jones@linaro.org>","To":"Linus Walleij <linus.walleij@linaro.org>","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","Message-ID":"<20171004105703.qmahtqieimw4kn3p@dell>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170917093906.16325-2-linus.walleij@linaro.org>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171004_035728_073569_51D82576 ","X-CRM114-Status":"GOOD (  30.21  )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.0 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"linux-mips@linux-mips.org, Guenter Roeck <linux@roeck-us.net>, Wolfram\n\tSang <wsa@the-dreams.de>, Steven Miao <realmz6@gmail.com>, Magnus Damm\n\t<magnus.damm@gmail.com>, adi-buildroot-devel@lists.sourceforge.net, \n\tRalf Baechle <ralf@linux-mips.org>, Ville =?iso-8859-1?q?Syrj=E4l=E4?=\n\t<ville.syrjala@linux.intel.com>,  Ben Dooks <ben.dooks@codethink.co.uk>,\n\tGeert Uytterhoeven <geert@linux-m68k.org>, linux-i2c@vger.kernel.org, \n\tHeiko Schocher <hs@denx.de>, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1782888,"web_url":"http://patchwork.ozlabs.org/comment/1782888/","msgid":"<20171009142403.GA17971@linux-mips.org>","list_archive_url":null,"date":"2017-10-09T14:24:03","subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","submitter":{"id":297,"url":"http://patchwork.ozlabs.org/api/people/297/","name":"Ralf Baechle","email":"ralf@linux-mips.org"},"content":"On Sun, Sep 17, 2017 at 11:39:00AM +0200, Linus Walleij wrote:\n\n> This converts the GPIO-based I2C-driver to using GPIO\n> descriptors instead of the old global numberspace-based\n> GPIO interface. We:\n> \n> - Convert the driver to unconditionally grab two GPIOs\n>   from the device by index 0 (SDA) and 1 (SCL) which\n>   will work fine with device tree and descriptor tables.\n>   The existing device trees will continue to work just\n>   like before, but without any roundtrip through the\n>   global numberspace.\n> \n> - Brutally convert all boardfiles still passing global\n>   GPIOs by registering descriptor tables associated with\n>   the devices instead so this driver does not need to keep\n>   supporting passing any GPIO numbers as platform data.\n> \n> There is no stepwise approach as elegant as this, I\n> strongly prefer this big hammer over any antsteps for this\n> conversion. This way the old GPIO numbers go away and\n> NEVER COME BACK.\n> \n> Special conversion for the different boards utilizing\n> I2C-GPIO:\n> \n> - EP93xx (arch/arm/mach-ep93xx): pretty straight forward as\n>   all boards were using the same two GPIO lines, just define\n>   these two in a lookup table for \"i2c-gpio\" and register\n>   these along with the device. None of them define any\n>   other platform data so just pass NULL as platform data.\n>   This platform selects GPIOLIB so all should be smooth.\n>   The pins appear on a gpiochip for bank \"G\" as pins 1 (SDA)\n>   and 0 (SCL).\n> \n> - IXP4 (arch/arm/mach-ixp4): descriptor tables have to\n>   be registered for each board separately. They all use\n>   \"IXP4XX_GPIO_CHIP\" so it is pretty straight forward.\n>   Most board define no other platform data than SCL/SDA\n>   so they can drop the #include of <linux/i2c-gpio.h> and\n>   assign NULL to platform data.\n> \n>   The \"goramo_mlr\" (Goramo Multilink Router) board is a bit\n>   worrisome: it implements its own I2C bit-banging in the\n>   board file, and optionally registers an I2C serial port,\n>   but claims the same GPIO lines for itself in the board file.\n>   This is not going to work: there will be competition for the\n>   GPIO lines, so delete the optional extra I2C bus instead, no\n>   I2C devices are registered on it anyway, there are just hints\n>   that it may contain an EEPROM that may be accessed from\n>   userspace. This needs to be fixed up properly by the serial\n>   clock using I2C emulation so drop a note in the code.\n> \n> - KS8695 board acs5k (arch/arm/mach-ks8695/board-acs5.c)\n>   has some platform data in addition to the pins so it needs to\n>   be kept around sans GPIO lines. Its GPIO chip is named\n>   \"KS8695\" and the arch selects GPIOLIB.\n> \n> - PXA boards (arch/arm/mach-pxa/*) use some of the platform\n>   data so it needs to be preserved here. The viper board even\n>   registers two GPIO I2Cs. The gpiochip is named \"gpio-pxa\" and\n>   the arch selects GPIOLIB.\n> \n> - SA1100 Simpad (arch/arm/mach-sa1100/simpad.c) defines a GPIO\n>   I2C bus, and the arch selects GPIOLIB.\n> \n> - Blackfin boards (arch/blackfin/bf533 etc) for these I assume\n>   their I2C GPIOs refer to the local gpiochip defined in\n>   arch/blackfin/kernel/bfin_gpio.c names \"BFIN-GPIO\".\n>   The arch selects GPIOLIB. The boards get spiked with\n>   IF_ENABLED(I2C_GPIO) but that is a side effect of it\n>   being like that already (I would just have Kconfig select\n>   I2C_GPIO and get rid of them all.) I also delete any\n>   platform data set to 0 as it will get that value anyway\n>   from static declartions of platform data.\n> \n> - The MIPS selects GPIOLIB and the Alchemy machine is using\n>   two local GPIO chips, one of them has a GPIO I2C. We need\n>   to adjust the local offset from the global number space here.\n>   The ATH79 has a proper GPIO driver in drivers/gpio/gpio-ath79.c\n>   and AFAICT the chip is named \"ath79-gpio\" and the PB44\n>   PCF857x expander spawns from this on GPIO 1 and 0. The latter\n>   board only use the platform data to specify pins so it can be\n>   cut altogether after this.\n> \n> - The MFD Silicon Motion SM501 is a special case. It dynamically\n>   spawns an I2C bus off the MFD using sm501_create_subdev().\n>   We use an approach to dynamically create a machine descriptor\n>   table and attach this to the \"SM501-LOW\" or \"SM501-HIGH\"\n>   gpiochip. We use chip-local offsets to grab the right lines.\n>   We can get rid of two local static inline helpers as part\n>   of this refactoring.\n> \n> Cc: Steven Miao <realmz6@gmail.com>\n> Cc: Ralf Baechle <ralf@linux-mips.org>\n> Cc: Lee Jones <lee.jones@linaro.org>\n> Cc: Guenter Roeck <linux@roeck-us.net>\n> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>\n> Cc: Magnus Damm <magnus.damm@gmail.com>\n> Cc: Ben Dooks <ben.dooks@codethink.co.uk>\n> Cc: Heiko Schocher <hs@denx.de>\n> Acked-by: Olof Johansson <olof@lixom.net>\n> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>\n> ---\n> ChangeLog v1->v2:\n> - Fix a minor typo in error path (scl was sda from copy-paste)\n> - Collected Olof's ACK\n> \n> Steven (Blackfin): requesting ACK for Wolfram to take this patch.\n> Ralf (MIPS): requesting ACK for Wolfram to take this patch.\n> Lee: requesting ACK for Wolfram to take this patch.\n> SM501 users: requesting Tested-by on this patch.\n> ---\n>  arch/arm/mach-ep93xx/core.c                  |  39 ++++----\n>  arch/arm/mach-ep93xx/edb93xx.c               |  15 +--\n>  arch/arm/mach-ep93xx/include/mach/platform.h |   4 +-\n>  arch/arm/mach-ep93xx/simone.c                |  12 +--\n>  arch/arm/mach-ep93xx/snappercl15.c           |  12 +--\n>  arch/arm/mach-ep93xx/vision_ep9307.c         |   7 +-\n>  arch/arm/mach-ixp4xx/avila-setup.c           |  17 +++-\n>  arch/arm/mach-ixp4xx/dsmg600-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/fsg-setup.c             |  16 +++-\n>  arch/arm/mach-ixp4xx/goramo_mlr.c            |  24 ++---\n>  arch/arm/mach-ixp4xx/ixdp425-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/nas100d-setup.c         |  16 +++-\n>  arch/arm/mach-ixp4xx/nslu2-setup.c           |  16 +++-\n>  arch/arm/mach-ks8695/board-acs5k.c           |  13 ++-\n>  arch/arm/mach-pxa/palmz72.c                  |  12 ++-\n>  arch/arm/mach-pxa/viper.c                    |  27 +++++-\n>  arch/arm/mach-sa1100/simpad.c                |  12 ++-\n>  arch/blackfin/mach-bf533/boards/blackstamp.c |  19 +++-\n>  arch/blackfin/mach-bf533/boards/ezkit.c      |  18 +++-\n>  arch/blackfin/mach-bf533/boards/stamp.c      |  18 +++-\n>  arch/blackfin/mach-bf561/boards/ezkit.c      |  18 +++-\n>  arch/mips/alchemy/board-gpr.c                |  19 +++-\n>  arch/mips/ath79/mach-pb44.c                  |  16 +++-\n\nFor MIPS:\n\nAcked-by: Ralf Baechle <ralf@linux-mips.org>\n\n  Ralf","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Htt8KwX/\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y9jF44pmbz9t6N\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tTue, 10 Oct 2017 01:24:56 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1YzE-0005uu-EI; Mon, 09 Oct 2017 14:24:52 +0000","from eddie.linux-mips.org ([148.251.95.138]\n\thelo=cvs.linux-mips.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1Yz9-0005l3-D8 for linux-arm-kernel@lists.infradead.org;\n\tMon, 09 Oct 2017 14:24:50 +0000","from localhost.localdomain ([127.0.0.1]:59342 \"EHLO linux-mips.org\"\n\trhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP\n\tid S23992641AbdJIOYXQ6wwn (ORCPT\n\t<rfc822;linux-arm-kernel@lists.infradead.org>);\n\tMon, 9 Oct 2017 16:24:23 +0200","from h7.dl5rb.org.uk (localhost [127.0.0.1])\n\tby h7.dl5rb.org.uk (8.15.2/8.14.8) with ESMTP id v99EOD85018101;\n\tMon, 9 Oct 2017 16:24:13 +0200","(from ralf@localhost)\n\tby h7.dl5rb.org.uk (8.15.2/8.15.2/Submit) id v99EO3c9018084;\n\tMon, 9 Oct 2017 16:24:03 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=lFlpE7NEyv5pFIcPeU4SIhvk0OWU9FocgPA/m4A5kgM=;\n\tb=Htt8KwX/fBZCuK\n\teq3mFYTWNA74OTzcTlgl3lY9bdc1GlLHXPuFyexMSUX8DcBDWyu3miWrP4N4NFdgQY5gJqzeda+RZ\n\tcX1Yk2POG1PC7AByzY/J00YMbN2Hw8kszmlLzSJsb9LlB2Nn/SzAzAAsCFPjOuZ4NR60jTE4avXJ6\n\tRi0+vdaWdSPcYxtgUo90mciOF886WjaPWVE5ULHHQ8AiFpnGg27zPMtwQoFIp5WhmK1z5PigQH4h7\n\tW3BOLp45RRugGvpgJCkDNsRXFflq2rAQUbIQGJaXfvOWqjhVgm5tbv3IgtUodJ+JHBlDTzi84A4mA\n\tWLOM0pZGZl3Hm4+UNWWA==;","Date":"Mon, 9 Oct 2017 16:24:03 +0200","From":"Ralf Baechle <ralf@linux-mips.org>","To":"Linus Walleij <linus.walleij@linaro.org>","Subject":"Re: [PATCH 1/7] i2c: gpio: Convert to use descriptors","Message-ID":"<20171009142403.GA17971@linux-mips.org>","References":"<20170917093906.16325-1-linus.walleij@linaro.org>\n\t<20170917093906.16325-2-linus.walleij@linaro.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170917093906.16325-2-linus.walleij@linaro.org>","User-Agent":"Mutt/1.9.1 (2017-09-22)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171009_072447_814647_34AC9C12 ","X-CRM114-Status":"GOOD (  31.06  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [148.251.95.138 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"linux-mips@linux-mips.org, Guenter Roeck <linux@roeck-us.net>, Wolfram\n\tSang <wsa@the-dreams.de>, Steven Miao <realmz6@gmail.com>, Magnus Damm\n\t<magnus.damm@gmail.com>, adi-buildroot-devel@lists.sourceforge.net, Ville\n\t=?iso-8859-1?q?Syrj?= =?iso-8859-1?q?=E4l=E4?=\n\t<ville.syrjala@linux.intel.com>,  Ben Dooks <ben.dooks@codethink.co.uk>,\n\tGeert Uytterhoeven <geert@linux-m68k.org>, linux-i2c@vger.kernel.org, \n\tHeiko Schocher <hs@denx.de>, Lee Jones <lee.jones@linaro.org>, \n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]