[{"id":1760153,"web_url":"http://patchwork.ozlabs.org/comment/1760153/","msgid":"<CAPDyKFogy=L=BDKtMhKCN0nMLi-Yy5SHgAhB_=jog+CX91breA@mail.gmail.com>","list_archive_url":null,"date":"2017-08-30T13:13:57","subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","submitter":{"id":21036,"url":"http://patchwork.ozlabs.org/api/people/21036/","name":"Ulf Hansson","email":"ulf.hansson@linaro.org"},"content":"On 28 August 2017 at 16:29, Jerome Brunet <jbrunet@baylibre.com> wrote:\n> The patchset features several bugfixes, rework and upgrade for the\n> meson-gx MMC driver.\n>\n> The main goal is to improve readability and enable new high speed\n> modes, such as eMMC DDR52 and sdcard UHS modes up to SDR50 (100Mhz)\n>\n> SDR104 is not working with a few cards on the p200 and the\n> libretech-cc. I suspect that 200Mhz might be a bit too fast for the PCB\n> of these boards, adding noise to the signal and eventually breaking\n> the communication with some cards. The same cards are working well on a\n> laptop or the nanopi-k2 at 200Mhz.\n>\n> This series has been tested on gxbb-p200, gxbb-nanopi-k2 and\n> gxl-s905x-libretech-cc\n>\n> Changes since v2 [1]:\n> * Drop patches 1 to 3: Applied.\n> * Drop patch 4: Debug stuff which should not have been sent.\n> * Added fix to previous patch 3:\n>   If the clock register is not initialized before registering the clk\n>   with CCF, the framework will complain about an illegal divider value.\n>   This had gone unnoticed because it was later fixed by the clock init\n>   rework.\n>\n>   Ulf, I know it is getting late but it would be nice if patch #1 of\n>   this v3 could go with 3 patches you already applied. The rest can\n>   wait for the following cycle.\n\nI decided to pick them all, so applied for next!\n\nFor patch1, I decided to update the changelog a bit, avoiding to have\na specific fixes tag, becasue the fix might not end up having the same\nhash, when/if I rebase my branch.\n\nThanks!\n\nKind regards\nUffe\n\n>\n> Changes since v1 [0]:\n> * Reorder patches to have fixes first, then rework and finally\n>   enhancements.\n> * Use CCF to manage clock phases\n>\n> [0]: https://lkml.kernel.org/r/20170804174353.16486-1-jbrunet@baylibre.com\n> [1]: https://lkml.kernel.org/r/20170821160301.21899-1-jbrunet@baylibre.com\n>\n> Jerome Brunet (13):\n>   mmc: meson-gx: initialize sane clk default before clock register\n>   mmc: meson-gx: cfg init overwrite values\n>   mmc: meson-gx: rework set_ios function\n>   mmc: meson-gx: rework clk_set function\n>   mmc: meson-gx: rework clock init function\n>   mmc: meson-gx: fix dual data rate mode frequencies\n>   mmc: meson-gx: work around clk-stop issue\n>   mmc: meson-gx: simplify interrupt handler\n>   mmc: meson-gx: implement card_busy callback\n>   mmc: meson-gx: use CCF to handle the clock phases\n>   mmc: meson-gx: implement voltage switch callback\n>   mmc: meson-gx: change default tx phase\n>   mmc: meson-gx: rework tuning function\n>\n>  drivers/mmc/host/meson-gx-mmc.c | 710 ++++++++++++++++++++++++++++------------\n>  1 file changed, 493 insertions(+), 217 deletions(-)\n>\n> --\n> 2.9.5\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 header.b=\"Vvo2qLzi\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=infradead.org header.i=@infradead.org\n\theader.b=\"bQnEnQ5d\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"DP6XSujC\"; 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 3xj5xF00Bpz9sN7\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 23:30:56 +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 1dn352-0008SQ-PF; Wed, 30 Aug 2017 13:30:52 +0000","from casper.infradead.org ([2001:8b0:10b:1236::1])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn34X-00086S-CB for linux-arm-kernel@bombadil.infradead.org;\n\tWed, 30 Aug 2017 13:30:21 +0000","from mail-it0-x22d.google.com ([2607:f8b0:4001:c0b::22d])\n\tby casper.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn2p2-0000Oy-IV for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 13:14:22 +0000","by mail-it0-x22d.google.com with SMTP id t134so968283ita.0\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tWed, 30 Aug 2017 06:13:59 -0700 (PDT)","by 10.2.96.38 with HTTP; Wed, 30 Aug 2017 06:13:57 -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=u+zjYcen4IgPl3KtoFn9kiD9PjCoQRtarJCpsVMY6do=;\n\tb=Vvo2qLziaWKO5l\n\tUR7umQaJS/gjF5teNjxnx871BAmD9GKfIcKTUKDLYNccoUiN3Alg9U8LKXY0vouZARQvWa8zed5fx\n\t6VokxBf38WW1XHt5Afw6r72L2pJ1S58krwZCvjYMXQ/46Y9N2bfp2O3Y+/DIdH8TDoMfamkzPIM7H\n\tDSwP7Q5cFKU1ALNTbyMDpq4DnPih2CAfB/JldKYWU+wpibEHkSKXKGM7ZNS92RRpAltIxuxq5blZV\n\tKo+DC4UDw911zX8+puHkyX/Tv5sLWCVbxXB+I0JN86ntSa3Igtk4JhAr0Mpfxs23l+whp7JzmFm3j\n\tgUZebb1eHi8q0xN9pAtA==;","v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=infradead.org; s=casper.20170209;\n\th=Content-Type:Cc:To:Subject:Message-ID:\n\tDate:From:References:In-Reply-To:MIME-Version:Sender:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:\n\tList-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;\n\tbh=4Bs6mvGtIdz0XLi5EBWH5KqQfKRpZmtaafscErVwha8=;\n\tb=bQnEnQ5dlcC4AnH/FgcGqi7HI\n\t3w6Q9begUMzpxe8N+XWFCvI41nAK5OKPPPkXP69i76lD4F51IParWPxKngHIZf8D/yPN1/BBrk5l6\n\tajBwUbZNMJ5+IYv7m8q+NbMs/73Ak5oOhHxuKmcrB4s78EVtVpvbfphlwqN7axEDntDSTj8HXELPy\n\tV+Q1DAF1Xin1vtm1Qz95ReG+rIoFyME/1cKSIhVWaygqrLOrBSWeDUjGmulKJGxAsMzGXnV4nK2+W\n\t0l2UPmMhEanblucwfL+JC85jH7tS36fb/TMtBVUeSiKwtFO/wbkGL0QxXiun/KoZdw9q16PwDFeeC\n\tlgGdkDKwg==;","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=4Bs6mvGtIdz0XLi5EBWH5KqQfKRpZmtaafscErVwha8=;\n\tb=DP6XSujCb/vHFxeJYRIjoQpTFOQ+sdiuvZ5YvM22U6pFlvK1W4NhyDkNtTVR+9HbDj\n\t4D/mUvn3scrig5BiKcWM4JJvtwr74QZ2FJePZXt9yQEvxV1IL7UTtkdD7OyjEOwQCoep\n\t4q8JCuQaVhPrFoJ5+mak9QtjfZg9x5erIJCjQ="],"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=4Bs6mvGtIdz0XLi5EBWH5KqQfKRpZmtaafscErVwha8=;\n\tb=rJ74+V5DmOj9/GgvogRPuczQgv/WruckpI/e5QeLBIfIejeTFeOE5lrPGQ7I1RJEms\n\t01cRAoMvRQQw4ck63XLeknoazyyEjaFUiIYtV69zstabCl7mYwxzhjfVVZkmLGVD/NQJ\n\thNXIXbGCk1yA1Tptt8obNwQkbUXlpEESQye+GVLyDoOkWa6F6p9CzI8vPGmw0ctnYf/c\n\tnj78qCDT1S6jFtKuhmU3tiOTkmvPP56RP1PYU77ag5hfkOW2kmRpuAVov42r94gEfxIm\n\tLGWNXwqIAjEy8Zni7LoFn7dC5aVnjIozTqYhBGGduICfPD+NZ8hNCDO3HOYF++WoJcsY\n\thGtA==","X-Gm-Message-State":"AHYfb5jzf7YnammmNBIZQ7gy2nbqrMkeqn+/+hkQ7pQdNIr+/zl4pAs9\n\t9dzFwr1ak4c2uiaKpo8U5Mle61i3nA2t","X-Received":"by 10.36.173.11 with SMTP id c11mr1451789itf.106.1504098838066; \n\tWed, 30 Aug 2017 06:13:58 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170828142915.27020-1-jbrunet@baylibre.com>","References":"<20170828142915.27020-1-jbrunet@baylibre.com>","From":"Ulf Hansson <ulf.hansson@linaro.org>","Date":"Wed, 30 Aug 2017 15:13:57 +0200","Message-ID":"<CAPDyKFogy=L=BDKtMhKCN0nMLi-Yy5SHgAhB_=jog+CX91breA@mail.gmail.com>","Subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","To":"Jerome Brunet <jbrunet@baylibre.com>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_141420_756762_0CEED9F6 ","X-CRM114-Status":"GOOD (  25.73  )","X-Spam-Score":"-2.7 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on casper.infradead.org summary:\n\tContent analysis details:   (-2.7 points, 5.0 required)\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:c0b:0:0:0:22d 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_AU Message has a valid DKIM or DK signature from\n\tauthor's domain\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature","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":"Kevin Hilman <khilman@baylibre.com>,\n\t\"linux-mmc@vger.kernel.org\" <linux-mmc@vger.kernel.org>,\n\t\"linux-kernel@vger.kernel.org\" <linux-kernel@vger.kernel.org>,\n\tCarlo Caione <carlo@caione.org>,\n\t\"open list:ARM/Amlogic Meson...\" <linux-amlogic@lists.infradead.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":1760416,"web_url":"http://patchwork.ozlabs.org/comment/1760416/","msgid":"<7hr2vsdgj4.fsf@baylibre.com>","list_archive_url":null,"date":"2017-08-30T19:03:43","subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","submitter":{"id":68189,"url":"http://patchwork.ozlabs.org/api/people/68189/","name":"Kevin Hilman","email":"khilman@baylibre.com"},"content":"Hi Ulf,\n\nUlf Hansson <ulf.hansson@linaro.org> writes:\n\n> On 28 August 2017 at 16:29, Jerome Brunet <jbrunet@baylibre.com> wrote:\n>> The patchset features several bugfixes, rework and upgrade for the\n>> meson-gx MMC driver.\n>>\n>> The main goal is to improve readability and enable new high speed\n>> modes, such as eMMC DDR52 and sdcard UHS modes up to SDR50 (100Mhz)\n>>\n>> SDR104 is not working with a few cards on the p200 and the\n>> libretech-cc. I suspect that 200Mhz might be a bit too fast for the PCB\n>> of these boards, adding noise to the signal and eventually breaking\n>> the communication with some cards. The same cards are working well on a\n>> laptop or the nanopi-k2 at 200Mhz.\n>>\n>> This series has been tested on gxbb-p200, gxbb-nanopi-k2 and\n>> gxl-s905x-libretech-cc\n>>\n>> Changes since v2 [1]:\n>> * Drop patches 1 to 3: Applied.\n>> * Drop patch 4: Debug stuff which should not have been sent.\n>> * Added fix to previous patch 3:\n>>   If the clock register is not initialized before registering the clk\n>>   with CCF, the framework will complain about an illegal divider value.\n>>   This had gone unnoticed because it was later fixed by the clock init\n>>   rework.\n>>\n>>   Ulf, I know it is getting late but it would be nice if patch #1 of\n>>   this v3 could go with 3 patches you already applied. The rest can\n>>   wait for the following cycle.\n>\n> I decided to pick them all, so applied for next!\n>\n\nJust to double-check, so I can plan for the DT patches\naccordingly... you are queuing this for v4.14, right?\n\nKevin","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=\"iauaxoB4\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=baylibre-com.20150623.gappssmtp.com\n\theader.i=@baylibre-com.20150623.gappssmtp.com header.b=\"zvHhERQo\"; \n\tdkim-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 3xjFKz5Cnlz9sN5\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 31 Aug 2017 05:04:22 +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 1dn8Hh-0006oY-RA; Wed, 30 Aug 2017 19:04:17 +0000","from mail-pg0-x22d.google.com ([2607:f8b0:400e:c05::22d])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn8He-0006ha-2Q for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 19:04:16 +0000","by mail-pg0-x22d.google.com with SMTP id b8so22354425pgn.5\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tWed, 30 Aug 2017 12:03:52 -0700 (PDT)","from localhost (c-98-203-232-209.hsd1.wa.comcast.net.\n\t[98.203.232.209]) by smtp.gmail.com with ESMTPSA id\n\tx19sm11160845pfa.49.2017.08.30.12.03.49\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 30 Aug 2017 12:03:49 -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:MIME-Version:Message-ID:In-Reply-To:\n\tDate:References:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Z3Bhce+VEt3tWZApUMalyQbM8Zlek72hCMVNLiacFQQ=;\n\tb=iauaxoB45FO3/s\n\tlc2HRQXCy8++77ApSpbMLtvLD+ul83lSegGIFah0uNvTb/71+lNoZprrsKKao6yijEREoBMQLxthZ\n\tfgrVcrEi0fjRH5aWjmh5mpxXPGD1T2LhhX5mL1zIL3HUV8D6Y0pOhDFL6WZObDjJJeN6+Jj6A64wQ\n\tSK4WJtZ3Vh5ND2iITJ9kf82zeIT7aZqiDvE3H9BllV18IOy39kwvcLf6olL/9XcNeqOGKBgwZGDpA\n\tRN+w6Vu1PS+Os5XtTdvKLcGWf3ONB97s+bKGKVWeIbe4M+HEO+6F5EVpYJRLqPu4iycqmSZIpQ5tq\n\tbK9oSEScFxVBS96D6awQ==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=baylibre-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:organization:references:date:in-reply-to\n\t:message-id:user-agent:mime-version;\n\tbh=p6SfwsfuCddcBqQMwtLJyjAZbN3Zf/8LqUP9MIW+faI=;\n\tb=zvHhERQof9kUH+f+OZUb2OCQPWUnSlyNkVcjIVrF10EpDRdOSmUfjMkrhi8XIDrrZ8\n\tyOCHIQbDgV+JobnHvtnrp4OMQMuKz+lB8ZUBOKw8Jab5B57dqCJu1UL3pdqzsBe6Wups\n\tm4uwwSY90XnD3wkdLxCPZ3BwMQgp+rokRmBfpiYQ7dcormhqyJok5AjHlV0/IJePrZL8\n\tp36EfUmFGehXT/crCkq0mmX1uUnxbdbji7V5acebqGEw8BfNHNxwQJI/qLO3Xovhxc+W\n\tEy2fZJQI9eOqRFAa+Zfocd2DrX8WOU4IQdYnP87uNy29O+7KunVIT0Y6CBdzKCo4UoAE\n\tH9cQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:organization:references:date\n\t:in-reply-to:message-id:user-agent:mime-version;\n\tbh=p6SfwsfuCddcBqQMwtLJyjAZbN3Zf/8LqUP9MIW+faI=;\n\tb=lhs9YxmDpBLJERwmQN+yvaxtLTip67aapONH5UxucknJD20rHao8OVAObIPoK3hOks\n\teFVGgK0D+ssn6e0+kWPepeo60QxuMJf3RwhhXpoRyllNFqkSbikQqz1DrPNBpf8OUfpg\n\t5PGPGsjg4J5ZmdeHfX6UFit2nNoUdAqKMP35uZ/fy3c0gmTe8+rhsHwKt4S8KnBDDsF0\n\tpPyZbbHLIl4iaja7IVnwjASLqNdFriN5kHs8/5ZL7NNRqaPbRmNXhWALZMWS5oe2QqGW\n\t6qY1J3x8HEVPMQEdX0L1LO1FGmWCt7aCbyvRZtT+kfCsTr1JHPCoBr1cKX8/RreEFjr4\n\tRCiw==","X-Gm-Message-State":"AHYfb5gKOQ47Fdx8eOTrg2M4d7qxN145UyT8dvcRuSwcGlvLlrqzMknj\n\tsCndS5G23FehMz4c","X-Received":"by 10.98.204.27 with SMTP id a27mr2613550pfg.58.1504119831112;\n\tWed, 30 Aug 2017 12:03:51 -0700 (PDT)","From":"Kevin Hilman <khilman@baylibre.com>","To":"Ulf Hansson <ulf.hansson@linaro.org>","Subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","Organization":"BayLibre","References":"<20170828142915.27020-1-jbrunet@baylibre.com>\n\t<CAPDyKFogy=L=BDKtMhKCN0nMLi-Yy5SHgAhB_=jog+CX91breA@mail.gmail.com>","Date":"Wed, 30 Aug 2017 12:03:43 -0700","In-Reply-To":"<CAPDyKFogy=L=BDKtMhKCN0nMLi-Yy5SHgAhB_=jog+CX91breA@mail.gmail.com>\n\t(Ulf Hansson's message of \"Wed, 30 Aug 2017 15:13:57 +0200\")","Message-ID":"<7hr2vsdgj4.fsf@baylibre.com>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_120414_153118_20846CD9 ","X-CRM114-Status":"GOOD (  14.68  )","X-Spam-Score":"-1.2 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.2 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 [2607:f8b0:400e:c05:0:0:0:22d listed in] [list.dnswl.org]\n\t0.7 SPF_SOFTFAIL SPF: sender does not match SPF record (softfail)\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","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-mmc@vger.kernel.org\" <linux-mmc@vger.kernel.org>,\n\t\"linux-kernel@vger.kernel.org\" <linux-kernel@vger.kernel.org>,\n\tCarlo Caione <carlo@caione.org>,\n\t\"open list:ARM/Amlogic Meson...\" <linux-amlogic@lists.infradead.org>, \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>, \n\tJerome Brunet <jbrunet@baylibre.com>","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":1760824,"web_url":"http://patchwork.ozlabs.org/comment/1760824/","msgid":"<CAPDyKFpTkqH4qvLAraWxfxm=LOuWZv7P9nrznJ+o81y+9hDjjA@mail.gmail.com>","list_archive_url":null,"date":"2017-08-31T10:46:12","subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","submitter":{"id":21036,"url":"http://patchwork.ozlabs.org/api/people/21036/","name":"Ulf Hansson","email":"ulf.hansson@linaro.org"},"content":"On 30 August 2017 at 21:03, Kevin Hilman <khilman@baylibre.com> wrote:\n> Hi Ulf,\n>\n> Ulf Hansson <ulf.hansson@linaro.org> writes:\n>\n>> On 28 August 2017 at 16:29, Jerome Brunet <jbrunet@baylibre.com> wrote:\n>>> The patchset features several bugfixes, rework and upgrade for the\n>>> meson-gx MMC driver.\n>>>\n>>> The main goal is to improve readability and enable new high speed\n>>> modes, such as eMMC DDR52 and sdcard UHS modes up to SDR50 (100Mhz)\n>>>\n>>> SDR104 is not working with a few cards on the p200 and the\n>>> libretech-cc. I suspect that 200Mhz might be a bit too fast for the PCB\n>>> of these boards, adding noise to the signal and eventually breaking\n>>> the communication with some cards. The same cards are working well on a\n>>> laptop or the nanopi-k2 at 200Mhz.\n>>>\n>>> This series has been tested on gxbb-p200, gxbb-nanopi-k2 and\n>>> gxl-s905x-libretech-cc\n>>>\n>>> Changes since v2 [1]:\n>>> * Drop patches 1 to 3: Applied.\n>>> * Drop patch 4: Debug stuff which should not have been sent.\n>>> * Added fix to previous patch 3:\n>>>   If the clock register is not initialized before registering the clk\n>>>   with CCF, the framework will complain about an illegal divider value.\n>>>   This had gone unnoticed because it was later fixed by the clock init\n>>>   rework.\n>>>\n>>>   Ulf, I know it is getting late but it would be nice if patch #1 of\n>>>   this v3 could go with 3 patches you already applied. The rest can\n>>>   wait for the following cycle.\n>>\n>> I decided to pick them all, so applied for next!\n>>\n>\n> Just to double-check, so I can plan for the DT patches\n> accordingly... you are queuing this for v4.14, right?\n\nYes, correct!\n\nKind regards\nUffe","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=\"u+Ghaj2L\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"eSGqojhG\"; 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 3xjfFF2w9wz9s7F\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 31 Aug 2017 20:46:41 +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 1dnMzd-0001a7-Vp; Thu, 31 Aug 2017 10:46:37 +0000","from mail-io0-x231.google.com ([2607:f8b0:4001:c06::231])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dnMzZ-0001Xu-Ij for linux-arm-kernel@lists.infradead.org;\n\tThu, 31 Aug 2017 10:46:35 +0000","by mail-io0-x231.google.com with SMTP id b2so5327211iof.0\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tThu, 31 Aug 2017 03:46:13 -0700 (PDT)","by 10.2.96.38 with HTTP; Thu, 31 Aug 2017 03:46:12 -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=ax2IBl5PpBADW/ITi6zsEBNpYCDpNNHTYDY4prG8gHA=;\n\tb=u+Ghaj2L1DEjFX\n\trFooqGzp7dBd3gYdOGOFTjUzpnU9Xk7MO236pbIEQoRndCRRaTkpH0JoQe407XE0ERTHzUUKjAhSA\n\tz0rdu5K8sgR6FQ4MPzCkMrSP0d0KFDNnquLObcxrRUpZNe26WguM70p04zBru0ldJnCnXzpubtebh\n\txnaX7TNpD7HxBUP71AEvFlKjzWTxiELhx7mS8W93ao8vPvYkYqmagAkKdJZuQsnZLF2TenlIbNR2A\n\tUjyDa56nicu7JNyjh0pvNxa/g79Hj3WpvHmdFXhLfd46w7/ZpjPCiDnAvlSots3T40OHoSLjvUNyU\n\tBqrN48KRMCvqlgyo7s0w==;","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=fNL7/ZI1bU47+o1qWNWfCMEWHDgDUaXjPWTDYA9iDuA=;\n\tb=eSGqojhGE6C8XaToAd/K+8QyFvg5ZQNaSkyzytLoA30vUFi6gSdjC/vVI7AUAv5i03\n\tKyZSyBFOX90uLZi9wdt1i2RxHNrDIOL7QCBYT8e2oH+tv6ewLJTciJnR49YyMxFzqRZS\n\t3mMpLhwtanw77hP7Td8IpsBcu3tv3toJS4jpw="],"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=fNL7/ZI1bU47+o1qWNWfCMEWHDgDUaXjPWTDYA9iDuA=;\n\tb=EkO4KyV55Juz6relLiOW6/aLVfkqatcLf2DTeooR2uCj7kdh3QCO/kbWrMJlsnH/cu\n\trG3o0js46MrgW/sDIebqBEWmJJH3lOUGe+LpNE6DZLRqgnCgi0doyIV/BpECozg71OvW\n\tBAS0EG/8d9dUopymMzxXoLwTuI4Q6foAu4+fDCUggPXv980RQrDjGG4jKQu+NnZhioXw\n\tItm60c6kD+mOXm5DRCVixEp+eYvi8iv1MH4gjTq+BAJEqmUPdDG2UKk5Fnm3ikpgoVeq\n\t8s5z63hprqFVc8PsWJPT6iB31BH9ictt8xGAffPjk+Zwujnh5GTRl5zZvBBfFPRo4OC4\n\t80IQ==","X-Gm-Message-State":"AHPjjUiulyT0e+dwMUFf7FtAG7WW8Op/3MqvbHTThTdFKtDiZU6CFkvX\n\tXO9YG/byx2BhtoMNNjAo+DZU3FINN51UEYo=","X-Google-Smtp-Source":"ADKCNb4CuX9xjCyTeNhqKti+GqSKzyNd1MWAroc2ByiN8wG2DTsliarpCCKzXjDQ47Bt31cV6kgiC7FpY6AxHNgJECY=","X-Received":"by 10.36.66.148 with SMTP id i142mr301794itb.106.1504176372748; \n\tThu, 31 Aug 2017 03:46:12 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<7hr2vsdgj4.fsf@baylibre.com>","References":"<20170828142915.27020-1-jbrunet@baylibre.com>\n\t<CAPDyKFogy=L=BDKtMhKCN0nMLi-Yy5SHgAhB_=jog+CX91breA@mail.gmail.com>\n\t<7hr2vsdgj4.fsf@baylibre.com>","From":"Ulf Hansson <ulf.hansson@linaro.org>","Date":"Thu, 31 Aug 2017 12:46:12 +0200","Message-ID":"<CAPDyKFpTkqH4qvLAraWxfxm=LOuWZv7P9nrznJ+o81y+9hDjjA@mail.gmail.com>","Subject":"Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades","To":"Kevin Hilman <khilman@baylibre.com>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170831_034633_783564_BF0CA94A ","X-CRM114-Status":"GOOD (  14.66  )","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:231 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-mmc@vger.kernel.org\" <linux-mmc@vger.kernel.org>,\n\t\"linux-kernel@vger.kernel.org\" <linux-kernel@vger.kernel.org>,\n\tCarlo Caione <carlo@caione.org>,\n\t\"open list:ARM/Amlogic Meson...\" <linux-amlogic@lists.infradead.org>, \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t<linux-arm-kernel@lists.infradead.org>, \n\tJerome Brunet <jbrunet@baylibre.com>","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"}}]