[{"id":1776416,"web_url":"http://patchwork.ozlabs.org/comment/1776416/","msgid":"<20170927170811.57830c1f@bbrezillon>","list_archive_url":null,"date":"2017-09-27T15:08:11","subject":"Re: [PATCH v3 4/8] mtd: nand: atmel: Avoid ECC errors when leaving\n\tbackup mode","submitter":{"id":63120,"url":"http://patchwork.ozlabs.org/api/people/63120/","name":"Boris Brezillon","email":"boris.brezillon@free-electrons.com"},"content":"On Wed, 27 Sep 2017 10:35:51 +0200\nRomain Izard <romain.izard.pro@gmail.com> wrote:\n\n> During backup mode, the contents of all registers will be cleared as the\n> SoC will be completely powered down. For a product that boots on NAND\n> Flash memory, the bootloader will obviously use the related controller\n> to read the Flash and correct any detected error in the memory, before\n> handling back control to the kernel's resuming entry point.\n> \n> But it does not clean the NAND controller registers after use and on its\n> side the kernel driver expects the error locator to be powered down and\n> in a clean state. Add a resume hook for the PMECC error locator, and\n> reset its registers.\n> \n> Signed-off-by: Romain Izard <romain.izard.pro@gmail.com>\n> ---\n> Change in v3:\n> * keep the PMECC disabled when not in use, and use atmel_pmecc_resume to\n>   reset the controller after the bootloader has left it enabled.\n> \n>  drivers/mtd/nand/atmel/nand-controller.c |  3 +++\n>  drivers/mtd/nand/atmel/pmecc.c           | 22 ++++++++++++++--------\n>  drivers/mtd/nand/atmel/pmecc.h           |  1 +\n>  3 files changed, 18 insertions(+), 8 deletions(-)\n> \n> diff --git a/drivers/mtd/nand/atmel/nand-controller.c b/drivers/mtd/nand/atmel/nand-controller.c\n> index f25eca79f4e5..86c2199380c2 100644\n> --- a/drivers/mtd/nand/atmel/nand-controller.c\n> +++ b/drivers/mtd/nand/atmel/nand-controller.c\n> @@ -2530,6 +2530,9 @@ static __maybe_unused int atmel_nand_controller_resume(struct device *dev)\n>  \tstruct atmel_nand_controller *nc = dev_get_drvdata(dev);\n>  \tstruct atmel_nand *nand;\n>  \n> +\tif (nand->pmecc)\n> +\t\tatmel_pmecc_resume(nand->pmecc);\n> +\n\nnand is used uninitialized here, and atmel_pmecc_resume() should be\npassed a atmel_pmecc object not a atmel_pmecc_user.\n\n\tif (nc->pmecc)\n\t\tatmel_pmecc_resume(nc->pmecc);\n\n>  \tlist_for_each_entry(nand, &nc->chips, node) {\n>  \t\tint i;\n>  \n> diff --git a/drivers/mtd/nand/atmel/pmecc.c b/drivers/mtd/nand/atmel/pmecc.c\n> index 146af8218314..ff09c0f25dd4 100644\n> --- a/drivers/mtd/nand/atmel/pmecc.c\n> +++ b/drivers/mtd/nand/atmel/pmecc.c\n> @@ -765,6 +765,12 @@ void atmel_pmecc_get_generated_eccbytes(struct atmel_pmecc_user *user,\n>  }\n>  EXPORT_SYMBOL_GPL(atmel_pmecc_get_generated_eccbytes);\n>  \n> +void atmel_pmecc_reset(struct atmel_pmecc *pmecc)\n> +{\n> +\twritel(PMECC_CTRL_RST, pmecc->regs.base + ATMEL_PMECC_CTRL);\n> +\twritel(PMECC_CTRL_DISABLE, pmecc->regs.base + ATMEL_PMECC_CTRL);\n> +}\n\nIt's not used outside of this file, so it should have a static\nspecifier. Anyway, I wonder why you don't expose atmel_pmecc_reset()\ndirectly instead of creating this atmel_pmecc_resume() wrapper.","headers":{"Return-Path":"<linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org; spf=none (mailfrom)\n\tsmtp.mailfrom=lists.infradead.org (client-ip=65.50.211.133;\n\thelo=bombadil.infradead.org;\n\tenvelope-from=linux-mtd-bounces+incoming=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=\"GPBnSPUA\"; \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 3y2Lnb4TzSz9tXs\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 28 Sep 2017 01:09:07 +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 1dxDx3-0005cn-5u; Wed, 27 Sep 2017 15:08:41 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxDwv-0005YV-P6\n\tfor linux-mtd@lists.infradead.org; Wed, 27 Sep 2017 15:08:39 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 5C475208CD; Wed, 27 Sep 2017 17:08:11 +0200 (CEST)","from bbrezillon (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id C6A8120884;\n\tWed, 27 Sep 2017 17:08:10 +0200 (CEST)"],"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:References:In-Reply-To:\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=8rM7DCEHCDvyZt7C+RXGM7+IvwaCtbeIazk/vUtKqoc=;\n\tb=GPBnSPUARJYpTc\n\tByumm4KGArTinQopfjnzhSL7G3Ow0e19QJJz15Xf64R0p6x9Y3wMIbiIKrpe4Xsg+mvrIpKJ9jBbB\n\tDK4HaK3aFy90viixHauNaVvhorYre/EGkLYXA5mXzmMboYYDvVzkt1rhb/NFY2yXL6XbjB5LHdIMd\n\t2XkxX0Wc7NioETPvELDCg1ECXBcrhPLBpDoiYMMvlaC98q83PZ1sfQTg3aW5AbeOaSEfPznjYLBwx\n\t9u7bHJ4leGoEzd0nW4YgQUOmjrf980RxqlB9nmGVbr8/RuUIpL6qTTWl15RQkB7DRIVGYeKDa0Zww\n\tIlTSr/4Hwxpe8fM+XDtg==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT\n\tshortcircuit=ham autolearn=disabled version=3.4.0","Date":"Wed, 27 Sep 2017 17:08:11 +0200","From":"Boris Brezillon <boris.brezillon@free-electrons.com>","To":"Romain Izard <romain.izard.pro@gmail.com>","Subject":"Re: [PATCH v3 4/8] mtd: nand: atmel: Avoid ECC errors when leaving\n\tbackup mode","Message-ID":"<20170927170811.57830c1f@bbrezillon>","In-Reply-To":"<20170927083555.16580-5-romain.izard.pro@gmail.com>","References":"<20170927083555.16580-1-romain.izard.pro@gmail.com>\n\t<20170927083555.16580-5-romain.izard.pro@gmail.com>","X-Mailer":"Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_080834_118228_EFBAB539 ","X-CRM114-Status":"GOOD (  19.51  )","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-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list <linux-mtd.lists.infradead.org>","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-mtd>,\n\t<mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-mtd/>","List-Post":"<mailto:linux-mtd@lists.infradead.org>","List-Help":"<mailto:linux-mtd-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-mtd>,\n\t<mailto:linux-mtd-request@lists.infradead.org?subject=subscribe>","Cc":"Michael Turquette <mturquette@baylibre.com>, linux-kernel@vger.kernel.org,\n\tThierry Reding <thierry.reding@gmail.com>,\n\tAlexandre Belloni <alexandre.belloni@free-electrons.com>,\n\tLee Jones <lee.jones@linaro.org>, linux-clk@vger.kernel.org,\n\tRichard Weinberger <richard@nod.at>,\n\tMarek Vasut <marek.vasut@gmail.com>, \n\tLudovic Desroches <ludovic.desroches@microchip.com>,\n\tAlan Stern <stern@rowland.harvard.edu>, linux-serial@vger.kernel.org, \n\tJiri Slaby <jslaby@suse.com>, Josh Wu <rainyfeeling@outlook.com>,\n\tlinux-pwm@vger.kernel.org, Richard Genoud <richard.genoud@gmail.com>, \n\tGreg Kroah-Hartman <gregkh@linuxfoundation.org>,\n\tlinux-usb@vger.kernel.org, Stephen Boyd <sboyd@codeaurora.org>,\n\tNicolas Ferre <nicolas.ferre@microchip.com>,\n\tWenyou Yang <wenyou.yang@atmel.com>,\n\tCyrille Pitchen <cyrille.pitchen@wedev4u.fr>,\n\tlinux-mtd@lists.infradead.org, \n\tBrian Norris <computersforpeace@gmail.com>,\n\tDavid Woodhouse <dwmw2@infradead.org>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-mtd\" <linux-mtd-bounces@lists.infradead.org>","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1776427,"web_url":"http://patchwork.ozlabs.org/comment/1776427/","msgid":"<CAGkQfmOYg6Usyd++KuGqMjDnaWaP77U5jfOL++3nN7PTcAhh9A@mail.gmail.com>","list_archive_url":null,"date":"2017-09-27T15:32:10","subject":"Re: [PATCH v3 4/8] mtd: nand: atmel: Avoid ECC errors when leaving\n\tbackup mode","submitter":{"id":8236,"url":"http://patchwork.ozlabs.org/api/people/8236/","name":"Romain Izard","email":"romain.izard.pro@gmail.com"},"content":"2017-09-27 17:08 GMT+02:00 Boris Brezillon <boris.brezillon@free-electrons.com>:\n> On Wed, 27 Sep 2017 10:35:51 +0200\n> Romain Izard <romain.izard.pro@gmail.com> wrote:\n>\n>> During backup mode, the contents of all registers will be cleared as the\n>> SoC will be completely powered down. For a product that boots on NAND\n>> Flash memory, the bootloader will obviously use the related controller\n>> to read the Flash and correct any detected error in the memory, before\n>> handling back control to the kernel's resuming entry point.\n>>\n>> But it does not clean the NAND controller registers after use and on its\n>> side the kernel driver expects the error locator to be powered down and\n>> in a clean state. Add a resume hook for the PMECC error locator, and\n>> reset its registers.\n>>\n>> Signed-off-by: Romain Izard <romain.izard.pro@gmail.com>\n>> ---\n>> Change in v3:\n>> * keep the PMECC disabled when not in use, and use atmel_pmecc_resume to\n>>   reset the controller after the bootloader has left it enabled.\n>>\n>>  drivers/mtd/nand/atmel/nand-controller.c |  3 +++\n>>  drivers/mtd/nand/atmel/pmecc.c           | 22 ++++++++++++++--------\n>>  drivers/mtd/nand/atmel/pmecc.h           |  1 +\n>>  3 files changed, 18 insertions(+), 8 deletions(-)\n>>\n>> diff --git a/drivers/mtd/nand/atmel/nand-controller.c b/drivers/mtd/nand/atmel/nand-controller.c\n>> index f25eca79f4e5..86c2199380c2 100644\n>> --- a/drivers/mtd/nand/atmel/nand-controller.c\n>> +++ b/drivers/mtd/nand/atmel/nand-controller.c\n>> @@ -2530,6 +2530,9 @@ static __maybe_unused int atmel_nand_controller_resume(struct device *dev)\n>>       struct atmel_nand_controller *nc = dev_get_drvdata(dev);\n>>       struct atmel_nand *nand;\n>>\n>> +     if (nand->pmecc)\n>> +             atmel_pmecc_resume(nand->pmecc);\n>> +\n>\n> nand is used uninitialized here, and atmel_pmecc_resume() should be\n> passed a atmel_pmecc object not a atmel_pmecc_user.\n>\n>         if (nc->pmecc)\n>                 atmel_pmecc_resume(nc->pmecc);\n>\nAnd yet I thought I correctly tested this code...\n\n:(\n\n>>       list_for_each_entry(nand, &nc->chips, node) {\n>>               int i;\n>>\n>> diff --git a/drivers/mtd/nand/atmel/pmecc.c b/drivers/mtd/nand/atmel/pmecc.c\n>> index 146af8218314..ff09c0f25dd4 100644\n>> --- a/drivers/mtd/nand/atmel/pmecc.c\n>> +++ b/drivers/mtd/nand/atmel/pmecc.c\n>> @@ -765,6 +765,12 @@ void atmel_pmecc_get_generated_eccbytes(struct atmel_pmecc_user *user,\n>>  }\n>>  EXPORT_SYMBOL_GPL(atmel_pmecc_get_generated_eccbytes);\n>>\n>> +void atmel_pmecc_reset(struct atmel_pmecc *pmecc)\n>> +{\n>> +     writel(PMECC_CTRL_RST, pmecc->regs.base + ATMEL_PMECC_CTRL);\n>> +     writel(PMECC_CTRL_DISABLE, pmecc->regs.base + ATMEL_PMECC_CTRL);\n>> +}\n>\n> It's not used outside of this file, so it should have a static\n> specifier. Anyway, I wonder why you don't expose atmel_pmecc_reset()\n> directly instead of creating this atmel_pmecc_resume() wrapper.\n\nI will fix this...","headers":{"Return-Path":"<linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org; spf=none (mailfrom)\n\tsmtp.mailfrom=lists.infradead.org (client-ip=65.50.211.133;\n\thelo=bombadil.infradead.org;\n\tenvelope-from=linux-mtd-bounces+incoming=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=\"d+lSD4OZ\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tsecure) header.d=mobile-devices.fr header.i=@mobile-devices.fr\n\theader.b=\"PqMvXj7D\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"IHEf9Po7\"; 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 3y2MKD3Gxtz9t3m\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 28 Sep 2017 01:33:04 +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 1dxEKX-00079y-8a; Wed, 27 Sep 2017 15:32:57 +0000","from mail-qk0-x231.google.com ([2607:f8b0:400d:c09::231])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxEKS-00073O-NR\n\tfor linux-mtd@lists.infradead.org; Wed, 27 Sep 2017 15:32:55 +0000","by mail-qk0-x231.google.com with SMTP id b82so13671671qkc.4\n\tfor <linux-mtd@lists.infradead.org>;\n\tWed, 27 Sep 2017 08:32:31 -0700 (PDT)","by 10.200.48.120 with HTTP; Wed, 27 Sep 2017 08:32:10 -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=VtUrN1Ah2PffnoRyGZi/JMYuxIUOUC/+0YrLyIwC7CU=;\n\tb=d+lSD4OZhaU2KM\n\tEMh8eIb13j8VnODEa1Vnt2ObCvN0ji7nBTfUtxtplOo1QC2W2HuiA0Pu+1TIEO6p5ceWmPqzKWDzQ\n\tifiOjS6sewFmU8O0eMwLf9yY3GfmUXAHmSy9bJAWnaIMjGNtYo/6mZLyvEGs3Zko/to4QibNIaXpR\n\tY/I52bBLbUXXiJpE5T/URQQXfB9ePOLFLIo0p9MyiQsaO1nKN2xUgBpm6pfc62a98J9rvcPsd4p5Q\n\t6gcRwrs3tDxtJcb1c/DUnzPOcmm8iCS02kOFh1Zp7M0k2Sw8akZFscsRMfi1l2YhIhrxPZ0XA4Cch\n\tfmeezsnyjN2eUeIktVyw==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=mobile-devices.fr; s=google;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=Sm17Y/k078HA50+jIyUFtsFrVDAi0q8yANlgCdc982w=;\n\tb=PqMvXj7D3h+padP5OR4S94QUx6uvg/RzI8sY/k8HH2jXoa6aWveL2ixkCGfYbSNlHU\n\t2z83lb3puDGjT/RUeIOrz5rJ/+vHkSg1UxhPhNVN99Y0bkzu6hBXVj1+pc+S4UQCKrLP\n\t10T/YR112ZlRLg2vlU22XgoGtKQqKIBhSKPfo=","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=Sm17Y/k078HA50+jIyUFtsFrVDAi0q8yANlgCdc982w=;\n\tb=IHEf9Po7aFkuiVfmft5oV1twEhDvz5kS1OSN3cYKO8mbUnq21juO0/R4nQbPVef+4W\n\tomgbvvpeHKTxOz97TcMc+kWQnDcBV1DJHdD4XwwpP2/cR5MRXiTaz3fFjp4vwsaiTV3L\n\tvcYCyBzdsPPpyJkrR8kvmmD1R8RCM5HNC4tM09o5i4G5ZSAGlKB75zsGMSiKfZXOzAWw\n\tzKeCgk39m7QonfRr0EQ01zYck76P/QTcbfT/o55yxqrHZHNxfqeshRDZ1y6P/5UTFMU3\n\tSF1G0KE2V7VK5dbfxY+ic4apnx0Pjsq22sFEryzJ/AMqtUwdWZsmzhpRDVvkdWlDo5H1\n\tCRhg=="],"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:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=Sm17Y/k078HA50+jIyUFtsFrVDAi0q8yANlgCdc982w=;\n\tb=f90uWrswGhMfkRUpQo6MhQRHCbOHg6X53ejxh3HXJqV638GPHRz0UQ+UYiCOG8H8ab\n\t2JUhXzAkR97axNLgZyHGjeOMtphhxoQEhg4rJxDmqyqGKKim4g3J3YM4/oR53MSK5zf5\n\twLrF5ccAjoTyZ0x20ZpyZrDQkTk/Z74zu27mIoz7xBS9yK09NSPi0SmP7L8osC/VNpCH\n\tkh1wdSu7SAMUqp2x9ih5itrBKTmw9ArRYcWMa8s1ZQ4GMyd1B1mJDLYHOObgc6RmQa+m\n\t51jthpmDpaZZfAJ+rDrh07Jl8Cfz0Gcc3zRjEA/ZXgYwy2blpoCtbG/IaPUWNi9QgCA2\n\tpLaQ==","X-Gm-Message-State":"AHPjjUgFtRT+cH9jOvaYULqkolTSuXl7meM6KYsClxsC/T39/3gG504i\n\tJneJ4CQQQQ62fEdqOuHh1GfqVS3OtebqJY6FlsUGraWx","X-Google-Smtp-Source":"AOwi7QCU750XPNP+UztucKF0g0HTw/NbXD4TdDdONOCDkAUwN+9uZjC6Sfg1u5o67Or/b4mxssYnoONMTKy0dowA6gU=","X-Received":"by 10.55.209.16 with SMTP id s16mr2835038qki.46.1506526350963;\n\tWed, 27 Sep 2017 08:32:30 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170927170811.57830c1f@bbrezillon>","References":"<20170927083555.16580-1-romain.izard.pro@gmail.com>\n\t<20170927083555.16580-5-romain.izard.pro@gmail.com>\n\t<20170927170811.57830c1f@bbrezillon>","From":"Romain Izard <romain.izard.pro@gmail.com>","Date":"Wed, 27 Sep 2017 17:32:10 +0200","X-Google-Sender-Auth":"pReTrqq9fvUFtxS8Rke_Vhngehg","Message-ID":"<CAGkQfmOYg6Usyd++KuGqMjDnaWaP77U5jfOL++3nN7PTcAhh9A@mail.gmail.com>","Subject":"Re: [PATCH v3 4/8] mtd: nand: atmel: Avoid ECC errors when leaving\n\tbackup mode","To":"Boris Brezillon <boris.brezillon@free-electrons.com>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_083253_101757_CC79E759 ","X-CRM114-Status":"GOOD (  18.99  )","X-Spam-Score":"-2.5 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.5 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:400d:c09:0:0:0:231 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (romain.izard.pro[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\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\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list <linux-mtd.lists.infradead.org>","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-mtd>,\n\t<mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-mtd/>","List-Post":"<mailto:linux-mtd@lists.infradead.org>","List-Help":"<mailto:linux-mtd-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-mtd>,\n\t<mailto:linux-mtd-request@lists.infradead.org?subject=subscribe>","Cc":"Michael Turquette <mturquette@baylibre.com>,\n\tLKML <linux-kernel@vger.kernel.org>,\n\tThierry Reding <thierry.reding@gmail.com>, \n\tAlexandre Belloni <alexandre.belloni@free-electrons.com>,\n\tLee Jones <lee.jones@linaro.org>, linux-clk@vger.kernel.org,\n\tRichard Weinberger <richard@nod.at>,\n\tMarek Vasut <marek.vasut@gmail.com>, \n\tLudovic Desroches <ludovic.desroches@microchip.com>,\n\tAlan Stern <stern@rowland.harvard.edu>, linux-serial@vger.kernel.org, \n\tJiri Slaby <jslaby@suse.com>, Josh Wu <rainyfeeling@outlook.com>,\n\tlinux-pwm@vger.kernel.org, Richard Genoud <richard.genoud@gmail.com>, \n\tGreg Kroah-Hartman <gregkh@linuxfoundation.org>,\n\tlinux-usb@vger.kernel.org, Stephen Boyd <sboyd@codeaurora.org>,\n\tNicolas Ferre <nicolas.ferre@microchip.com>,\n\tWenyou Yang <wenyou.yang@atmel.com>,\n\tCyrille Pitchen <cyrille.pitchen@wedev4u.fr>,\n\tlinux-mtd <linux-mtd@lists.infradead.org>,\n\tBrian Norris <computersforpeace@gmail.com>,\n\tDavid Woodhouse <dwmw2@infradead.org>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-mtd\" <linux-mtd-bounces@lists.infradead.org>","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}}]