[{"id":1780349,"web_url":"http://patchwork.ozlabs.org/comment/1780349/","msgid":"<20171005094138.11b73a0e@xps13>","list_archive_url":null,"date":"2017-10-05T07:41:38","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","submitter":{"id":71917,"url":"http://patchwork.ozlabs.org/api/people/71917/","name":"Miquel Raynal","email":"miquel.raynal@free-electrons.com"},"content":"Hello Kalyan,\n\nOn Thu, 28 Sep 2017 13:57:56 +1300\nKalyan Kinthada wrote:\n\n> When the arbitration between NOR and NAND flash is enabled\n> the field bit[21] in the Data Flash Control Register\n> needs to be set to 1 according to guidleine GL-5830741 mentioned\n\nTypo: guideline ^\n\n> in Marvell Errata document MV-S501377-00, Rev. D.\n\nThanks for that, I checked it.\n\n> \n> This commit sets the FORCE_CSX bit to 1 for all\n> ARMADA370 variants as the arbitration is always enabled by default.\n> This change does not apply for pxa3xx variants because the FORCE_CSX\n> bit does not exist/reserved on the NFCv1.\n\nSorry to bother you again but I am reworking the pxa3xx_nand driver and\nso I would like to deeply understand why this is needed because I will\nhave to integrate it in my work too.\n\nSo please tell me what is your setup, do you really use NAND/NOR\narbitration? Do you have not-Don't Care CS NAND chips? I have some\ndoubts because even if the spec precises in the NAND controller chapter\nthat arbitration is always enabled by default, the bit 27 (NfArbiterEn)\nin the SoC Device Multiplex Register at offset 0x00018208 is actually\nat 0 by default (disabled). Did you enable this bit manually ? I\nchecked with devmem on my setup and this bit was unset.\n\nThank you for your time,\nMiquèl\n\n> \n> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is unused\n> so remove it along with NDCR_NAND_MODE.\n> \n> Signed-off-by: Kalyan Kinthada \n> ---\n\n\n\n> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\n> 1 file changed, 7 insertions(+), 2 deletions(-)\n> \n> diff --git a/drivers/mtd/nand/pxa3xx_nand.c\n> b/drivers/mtd/nand/pxa3xx_nand.c index 85cff68643e0..a6a7a5af7bed\n> 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c\n> +++ b/drivers/mtd/nand/pxa3xx_nand.c\n> @@ -67,8 +67,7 @@\n> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\n> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\n> #define NDCR_NCSX\t\t(0x1 << 23)\n> -#define NDCR_ND_MODE\t\t(0x3 << 21)\n> -#define NDCR_NAND_MODE \t(0x0)\n> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\n> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\n> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\n> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\n> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct\n> pxa3xx_nand_info *info) info->chunk_size = PAGE_CHUNK_SIZE;\n> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\n> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN :\n> 0;\n> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n> GL-5830741 */\n> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\n> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\n> \n> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct\n> pxa3xx_nand_info *info) info->reg_ndcr = ndcr &\n> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN |\n> NFCV1_NDCR_ARB_CNTL); info->reg_ndcr |= (pdata->enable_arbiter) ?\n> NDCR_ND_ARB_EN : 0;\n> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n> GL-5830741 */\n> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\n> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\n> }","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"uOfNr5sJ\"; \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 3y74VG1s03z9t3R\n\tfor ;\n\tThu, 5 Oct 2017 18:42:14 +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 1e00nH-0003To-Ey; Thu, 05 Oct 2017 07:42:07 +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 1e00nC-0003Ow-1r\n\tfor linux-mtd@lists.infradead.org; Thu, 05 Oct 2017 07:42:04 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 8D4772094A; Thu, 5 Oct 2017 09:41:38 +0200 (CEST)","from xps13 (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 2BD2820939;\n\tThu, 5 Oct 2017 09:41:38 +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=MVetXtcqeU1WVJPbm2R82HjWJ2SF9MmOuhe+xd+NtKQ=;\n\tb=uOfNr5sJhiDlDP\n\tXZU6Yd59tgr/JNwHMHY+IpKccFq5USX4+98jxluxr4xbzd4k/Vi7zu1g4yQRA4bVNalQrsN3A6qSe\n\t+EbWkK4Qr8B1YyfqAdnmaVhLUE+J2+6qL+JD8jwuGzfCUnIkXsc+GmOYzgaHud90FT54hjgYk0FW4\n\tNZBvDtaIU05kTndPFpkZaXlr/u4Ou3HBJ5hygD6o4sl5UMqpoUdPelmrpGzmI/5vvWRbzrUcJuq8u\n\tIeF+Ozv2iWCec99lJqjzSGZAHd0SzPndRYRipSOR9npAgojFiU0lACTc/EJsHHxHM3sTNArGIqRui\n\twAIwVVLySmiL1WeNaZFw==;","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\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Date":"Thu, 5 Oct 2017 09:41:38 +0200","From":"Miquel RAYNAL ","To":"Kalyan Kinthada ","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Message-ID":"<20171005094138.11b73a0e@xps13>","In-Reply-To":"<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>","Organization":"Free Electrons","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-20171005_004202_423465_6D7DB479 ","X-CRM114-Status":"GOOD ( 20.35 )","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 ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, boris.brezillon@free-electrons.com,\n\tdevicetree@vger.kernel.org, richard@nod.at,\n\tchris.packham@alliedtelesis.co.nz, \n\tlinux-kernel@vger.kernel.org, marek.vasut@gmail.com, robh+dt@kernel.org, \n\tlinux-mtd@lists.infradead.org, ezequiel.garcia@free-electrons.com,\n\tcyrille.pitchen@wedev4u.fr, computersforpeace@gmail.com,\n\tdwmw2@infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1782387,"web_url":"http://patchwork.ozlabs.org/comment/1782387/","msgid":"","list_archive_url":null,"date":"2017-10-09T02:31:30","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","submitter":{"id":72417,"url":"http://patchwork.ozlabs.org/api/people/72417/","name":"Kalyan Kinthada","email":"kalyan.kinthada@alliedtelesis.co.nz"},"content":"On 05/10/17 20:41, Miquel RAYNAL wrote:\r\n> Hello Kalyan,\r\n>\r\n> On Thu, 28 Sep 2017 13:57:56 +1300\r\n> Kalyan Kinthada wrote:\r\n>\r\n>> When the arbitration between NOR and NAND flash is enabled\r\n>> the field bit[21] in the Data Flash Control Register\r\n>> needs to be set to 1 according to guidleine GL-5830741 mentioned\r\n> Typo: guideline ^\r\nI will correct this typo in the next patch version.\r\n>> in Marvell Errata document MV-S501377-00, Rev. D.\r\n> Thanks for that, I checked it.\r\n>\r\n>> This commit sets the FORCE_CSX bit to 1 for all\r\n>> ARMADA370 variants as the arbitration is always enabled by default.\r\n>> This change does not apply for pxa3xx variants because the FORCE_CSX\r\n>> bit does not exist/reserved on the NFCv1.\r\n> Sorry to bother you again but I am reworking the pxa3xx_nand driver and\r\n> so I would like to deeply understand why this is needed because I will\r\n> have to integrate it in my work too.\r\n>\r\n> So please tell me what is your setup, do you really use NAND/NOR\r\n> arbitration? Do you have not-Don't Care CS NAND chips? I have some\r\n> doubts because even if the spec precises in the NAND controller chapter\r\n> that arbitration is always enabled by default, the bit 27 (NfArbiterEn)\r\n> in the SoC Device Multiplex Register at offset 0x00018208 is actually\r\n> at 0 by default (disabled). Did you enable this bit manually ? I\r\n> checked with devmem on my setup and this bit was unset.\r\nYes, my setup use NAND/NOR arbitration and use a Don't\r\ncare CS Nand Chip. The bit 27 at offset 0x00018208 is set to 1\r\nby default in my case. I did not manually set the bit 27 to 1.\r\n\r\nPlease let me now if you have any other question or do you want me\r\nto send the updated patch version with the typo fixed.\r\n\r\nThank You,\r\nKalyan\r\n\r\n>\r\n> Thank you for your time,\r\n> Miquèl\r\n>\r\n>> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is unused\r\n>> so remove it along with NDCR_NAND_MODE.\r\n>>\r\n>> Signed-off-by: Kalyan Kinthada \r\n>> ---\r\n>\r\n>\r\n>> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\r\n>> 1 file changed, 7 insertions(+), 2 deletions(-)\r\n>>\r\n>> diff --git a/drivers/mtd/nand/pxa3xx_nand.c\r\n>> b/drivers/mtd/nand/pxa3xx_nand.c index 85cff68643e0..a6a7a5af7bed\r\n>> 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c\r\n>> +++ b/drivers/mtd/nand/pxa3xx_nand.c\r\n>> @@ -67,8 +67,7 @@\r\n>> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\r\n>> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\r\n>> #define NDCR_NCSX\t\t(0x1 << 23)\r\n>> -#define NDCR_ND_MODE\t\t(0x3 << 21)\r\n>> -#define NDCR_NAND_MODE \t(0x0)\r\n>> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\r\n>> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\r\n>> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\r\n>> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\r\n>> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct\r\n>> pxa3xx_nand_info *info) info->chunk_size = PAGE_CHUNK_SIZE;\r\n>> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\r\n>> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN :\r\n>> 0;\r\n>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\r\n>> GL-5830741 */\r\n>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\r\n>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\r\n>> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\r\n>> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\r\n>> \r\n>> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct\r\n>> pxa3xx_nand_info *info) info->reg_ndcr = ndcr &\r\n>> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN |\r\n>> NFCV1_NDCR_ARB_CNTL); info->reg_ndcr |= (pdata->enable_arbiter) ?\r\n>> NDCR_ND_ARB_EN : 0;\r\n>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\r\n>> GL-5830741 */\r\n>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\r\n>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\r\n>> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\r\n>> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\r\n>> }\r\n>\r\n>","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"S5/OEHwy\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=alliedtelesis.co.nz\n\theader.i=@alliedtelesis.co.nz header.b=\"dKwxuL1H\"; \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 3y9PR40qkbz9t3B\n\tfor ;\n\tMon, 9 Oct 2017 13:32:32 +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 1e1Nrd-000477-Fj; Mon, 09 Oct 2017 02:32:17 +0000","from gate2.alliedtelesis.co.nz ([202.36.163.20])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1NrY-00040W-HN\n\tfor linux-mtd@lists.infradead.org; Mon, 09 Oct 2017 02:32:15 +0000","from mmarshal3.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (Client did not present a certificate)\n\tby gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id 81D53800E6;\n\tMon, 9 Oct 2017 15:31:31 +1300 (NZDT)","from svr-chch-ex1.atlnz.lc (Not Verified[10.32.16.77]) by\n\tmmarshal3.atlnz.lc with Trustwave SEG (v7, 5, 8, 10121)\n\tid ; Mon, 09 Oct 2017 15:31:31 +1300","from svr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) by\n\tsvr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) with Microsoft SMTP\n\tServer (TLS) id 15.0.1156.6; Mon, 9 Oct 2017 15:31:31 +1300","from svr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8]) by\n\tsvr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8%12]) with mapi id\n\t15.00.1156.000; Mon, 9 Oct 2017 15:31:31 +1300"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Ql+n8jOKFiv4bSVkKIZOlAdJjXyZlX5ZtMD4LFHZ+GE=;\n\tb=S5/OEHwy1qJitY\n\tFrzXrr3chn/aqFzOQCepcl0/jBTkNsQzabER+Irbs7k2M3NNjrsz7fo2J836840/WLR0ytTdQZbO9\n\t7Y6gQM4lVmxaUOUGnVSg1HQQn7JO35tYeYu8srXdPM82tF62Vq7ugMXswIQwHdWXhpicrPn+AOnTm\n\t/MGSVhbFkHdIFFv+T+JMZK7+yEZ8uvQkaWc2iFPochVwwVOTgluEDir/7jeh7LulE0YCapvPVve/R\n\t24Eh+m92Ki2AnKx3otJaJI193b2fvugPX+WEkSO5DfHmLks7+nxgGSFuTEcy3RSz0SXgugLg8YCXZ\n\t8eh0vLfHkypAFF0YUf3A==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; \n\ts=mail; t=1507516291;\n\tbh=4PvFuoNz7ESISVjnHmEUmb/R9hdOBTbHA1P9vMw6EwA=;\n\th=From:To:CC:Subject:Date:References:In-Reply-To;\n\tb=dKwxuL1HsNi8/HGNlhDMlrY/RFOBBS0uIoeYbPi9InvxApLAytSOwcdlAT4DN18kB\n\tvH9AwDaEGt+iC1hFeDNUEscWvRNALKjNXg0HYBpBkWpAKCK6L4ZkoZBJbOLe2tXIhp\n\tsIYD996byMRsrRhqUKlQAe/GKxYAodw2ZrK0XlKE="],"From":"Kalyan Kinthada ","To":"Miquel RAYNAL ","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","Thread-Topic":"[PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Thread-Index":"AQHTN/TV84NhrKrRv0aGyA/oHIy1raLUEOkAgAXypQA=","Date":"Mon, 9 Oct 2017 02:31:30 +0000","Message-ID":"","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20171005094138.11b73a0e@xps13>","In-Reply-To":"<20171005094138.11b73a0e@xps13>","Accept-Language":"en-NZ, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-ms-exchange-messagesentrepresentingtype":"1","x-ms-exchange-transport-fromentityheader":"Hosted","x-originating-ip":"[10.32.16.78]","Content-ID":"","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171008_193213_139477_4865549A ","X-CRM114-Status":"GOOD ( 22.28 )","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-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO 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-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"mark.rutland@arm.com\" ,\n\t\"boris.brezillon@free-electrons.com\"\n\t, \n\t\"devicetree@vger.kernel.org\" ,\n\t\"richard@nod.at\" ,\n\tChris Packham ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"marek.vasut@gmail.com\" ,\n\t\"robh+dt@kernel.org\" ,\n\t\"linux-mtd@lists.infradead.org\" ,\n\t\"ezequiel.garcia@free-electrons.com\"\n\t, \n\t\"cyrille.pitchen@wedev4u.fr\" ,\n\t\"computersforpeace@gmail.com\" ,\n\t\"dwmw2@infradead.org\" ","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1782527,"web_url":"http://patchwork.ozlabs.org/comment/1782527/","msgid":"<20171009081848.49681975@xps13>","list_archive_url":null,"date":"2017-10-09T06:18:48","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","submitter":{"id":71917,"url":"http://patchwork.ozlabs.org/api/people/71917/","name":"Miquel Raynal","email":"miquel.raynal@free-electrons.com"},"content":"Hello Kalyan,\n\nOn Mon, 9 Oct 2017 02:31:30 +0000\nKalyan Kinthada wrote:\n\n> On 05/10/17 20:41, Miquel RAYNAL wrote:\n> > Hello Kalyan,\n> >\n> > On Thu, 28 Sep 2017 13:57:56 +1300\n> > Kalyan Kinthada wrote:\n> > \n> >> When the arbitration between NOR and NAND flash is enabled\n> >> the field bit[21] in the Data Flash Control Register\n> >> needs to be set to 1 according to guidleine GL-5830741 mentioned \n> > Typo: guideline ^ \n> I will correct this typo in the next patch version.\n> >> in Marvell Errata document MV-S501377-00, Rev. D. \n> > Thanks for that, I checked it.\n> > \n> >> This commit sets the FORCE_CSX bit to 1 for all\n> >> ARMADA370 variants as the arbitration is always enabled by default.\n> >> This change does not apply for pxa3xx variants because the\n> >> FORCE_CSX bit does not exist/reserved on the NFCv1. \n> > Sorry to bother you again but I am reworking the pxa3xx_nand driver\n> > and so I would like to deeply understand why this is needed because\n> > I will have to integrate it in my work too.\n> >\n> > So please tell me what is your setup, do you really use NAND/NOR\n> > arbitration? Do you have not-Don't Care CS NAND chips? I have some\n> > doubts because even if the spec precises in the NAND controller\n> > chapter that arbitration is always enabled by default, the bit 27\n> > (NfArbiterEn) in the SoC Device Multiplex Register at offset\n> > 0x00018208 is actually at 0 by default (disabled). Did you enable\n> > this bit manually ? I checked with devmem on my setup and this bit\n> > was unset. \n> Yes, my setup use NAND/NOR arbitration and use a Don't\n> care CS Nand Chip. The bit 27 at offset 0x00018208 is set to 1\n> by default in my case. I did not manually set the bit 27 to 1.\n\nActually if you use Don't Care CS NAND chips you should not need the\nforce CS bit, as it is mentioned in the guidelines you pointed, this\nbit is only useful for *not* don't care CS NAND chips (and the name\n\"force CS\" is pretty straight forward too!).\n\nOtherwise what bootloader and kernel do you use to have this\nbit set by default? You may also want to check if this bit is set by\nyour bootloader by stopping it before it loads Linux and check manually\nthe value of this bit with 'md' (on U-Boot).\n\nThank you,\nMiquèl\n\n> \n> Please let me now if you have any other question or do you want me\n> to send the updated patch version with the typo fixed.\n> \n> Thank You,\n> Kalyan\n> \n> >\n> > Thank you for your time,\n> > Miquèl\n> > \n> >> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is\n> >> unused so remove it along with NDCR_NAND_MODE.\n> >>\n> >> Signed-off-by: Kalyan Kinthada\n> >> --- \n> >\n> > \n> >> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\n> >> 1 file changed, 7 insertions(+), 2 deletions(-)\n> >>\n> >> diff --git a/drivers/mtd/nand/pxa3xx_nand.c\n> >> b/drivers/mtd/nand/pxa3xx_nand.c index 85cff68643e0..a6a7a5af7bed\n> >> 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c\n> >> +++ b/drivers/mtd/nand/pxa3xx_nand.c\n> >> @@ -67,8 +67,7 @@\n> >> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\n> >> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\n> >> #define NDCR_NCSX\t\t(0x1 << 23)\n> >> -#define NDCR_ND_MODE\t\t(0x3 << 21)\n> >> -#define NDCR_NAND_MODE \t(0x0)\n> >> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\n> >> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\n> >> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\n> >> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\n> >> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct\n> >> pxa3xx_nand_info *info) info->chunk_size = PAGE_CHUNK_SIZE;\n> >> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\n> >> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ?\n> >> NDCR_ND_ARB_EN : 0;\n> >> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n> >> GL-5830741 */\n> >> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> >> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> >> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\n> >> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\n> >> \n> >> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct\n> >> pxa3xx_nand_info *info) info->reg_ndcr = ndcr &\n> >> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN |\n> >> NFCV1_NDCR_ARB_CNTL); info->reg_ndcr |= (pdata->enable_arbiter) ?\n> >> NDCR_ND_ARB_EN : 0;\n> >> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n> >> GL-5830741 */\n> >> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> >> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> >> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\n> >> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\n> >> } \n> >\n> >","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"p7KHxWMc\"; \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 3y9VT42TzRz9tXv\n\tfor ;\n\tMon, 9 Oct 2017 17:19:32 +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 1e1RPJ-0003cU-Tt; Mon, 09 Oct 2017 06:19:17 +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 1e1RPF-0003Vx-RP\n\tfor linux-mtd@lists.infradead.org; Mon, 09 Oct 2017 06:19:16 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 57363208B9; Mon, 9 Oct 2017 08:18:49 +0200 (CEST)","from xps13 (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 EF87A208A9;\n\tMon, 9 Oct 2017 08:18:48 +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=0IIemBhNYMmDRMzU2Eq9RB6h/6VwgrKsrKWzF6Fp9Ng=;\n\tb=p7KHxWMchpoTbN\n\tY9CrLUlMUSgQhfhIV5pnHKTBVkg9H01UTeZuAZL3dGhB5DmWKg/FfHHX6wKUGbOUiwJ+Y68QzerwA\n\trbHiIKiJadGerqELXQR1Kfs2zrPK5VCoo/AqpZXdjI8+LHv3jA7cmgmyJ545a0GzWKxv4JFh7xRRy\n\tqe9kZnwNoEJFGbmFxX9gUzjPaMqIRFYqmBf0qNEqpW+R37JWYuhIAkZ2jZJZA3IqJElSaA3nBKw+3\n\thmSfSrFHJZ1hCPEri6oW+jIUashj2KDRHC4YkCXs43yK2KafcnJEUQE6IbVgsyYjX62nflBbyv9uk\n\tI1OOBI+MePOQBPGoscKA==;","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":"Mon, 9 Oct 2017 08:18:48 +0200","From":"Miquel RAYNAL ","To":"Kalyan Kinthada ","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Message-ID":"<20171009081848.49681975@xps13>","In-Reply-To":"","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20171005094138.11b73a0e@xps13>\n\t","Organization":"Free Electrons","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-20171008_231914_201749_314B8F4E ","X-CRM114-Status":"GOOD ( 30.79 )","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 ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"mark.rutland@arm.com\" ,\n\t\"boris.brezillon@free-electrons.com\"\n\t, \n\t\"devicetree@vger.kernel.org\" ,\n\t\"richard@nod.at\" ,\n\tChris Packham ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"marek.vasut@gmail.com\" ,\n\t\"robh+dt@kernel.org\" ,\n\t\"linux-mtd@lists.infradead.org\" ,\n\t\"ezequiel.garcia@free-electrons.com\"\n\t, \n\t\"cyrille.pitchen@wedev4u.fr\" ,\n\t\"computersforpeace@gmail.com\" ,\n\t\"dwmw2@infradead.org\" ","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1784113,"web_url":"http://patchwork.ozlabs.org/comment/1784113/","msgid":"<0a9cf087-63de-19db-f695-ca8f3796fad0@alliedtelesis.co.nz>","list_archive_url":null,"date":"2017-10-10T20:26:07","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","submitter":{"id":72417,"url":"http://patchwork.ozlabs.org/api/people/72417/","name":"Kalyan Kinthada","email":"kalyan.kinthada@alliedtelesis.co.nz"},"content":"Hello Miquel,\r\nOn 09/10/17 19:18, Miquel RAYNAL wrote:\r\n> Hello Kalyan,\r\n>\r\n> On Mon, 9 Oct 2017 02:31:30 +0000\r\n> Kalyan Kinthada wrote:\r\n>\r\n>> On 05/10/17 20:41, Miquel RAYNAL wrote:\r\n>>> Hello Kalyan,\r\n>>>\r\n>>> On Thu, 28 Sep 2017 13:57:56 +1300\r\n>>> Kalyan Kinthada wrote:\r\n>>> \r\n>>>> When the arbitration between NOR and NAND flash is enabled\r\n>>>> the field bit[21] in the Data Flash Control Register\r\n>>>> needs to be set to 1 according to guidleine GL-5830741 mentioned\r\n>>> Typo: guideline ^\r\n>> I will correct this typo in the next patch version.\r\n>>>> in Marvell Errata document MV-S501377-00, Rev. D.\r\n>>> Thanks for that, I checked it.\r\n>>> \r\n>>>> This commit sets the FORCE_CSX bit to 1 for all\r\n>>>> ARMADA370 variants as the arbitration is always enabled by default.\r\n>>>> This change does not apply for pxa3xx variants because the\r\n>>>> FORCE_CSX bit does not exist/reserved on the NFCv1.\r\n>>> Sorry to bother you again but I am reworking the pxa3xx_nand driver\r\n>>> and so I would like to deeply understand why this is needed because\r\n>>> I will have to integrate it in my work too.\r\n>>>\r\n>>> So please tell me what is your setup, do you really use NAND/NOR\r\n>>> arbitration? Do you have not-Don't Care CS NAND chips? I have some\r\n>>> doubts because even if the spec precises in the NAND controller\r\n>>> chapter that arbitration is always enabled by default, the bit 27\r\n>>> (NfArbiterEn) in the SoC Device Multiplex Register at offset\r\n>>> 0x00018208 is actually at 0 by default (disabled). Did you enable\r\n>>> this bit manually ? I checked with devmem on my setup and this bit\r\n>>> was unset.\r\n>> Yes, my setup use NAND/NOR arbitration and use a Don't\r\n>> care CS Nand Chip. The bit 27 at offset 0x00018208 is set to 1\r\n>> by default in my case. I did not manually set the bit 27 to 1.\r\n> Actually if you use Don't Care CS NAND chips you should not need the\r\n> force CS bit, as it is mentioned in the guidelines you pointed, this\r\n> bit is only useful for *not* don't care CS NAND chips (and the name\r\n> \"force CS\" is pretty straight forward too!).\r\nOk. I understand now. Thank you.\r\n>\r\n> Otherwise what bootloader and kernel do you use to have this\r\n> bit set by default? You may also want to check if this bit is set by\r\n> your bootloader by stopping it before it loads Linux and check manually\r\n> the value of this bit with 'md' (on U-Boot).\r\nThe bit27 at offset 0x00018208 is set in bootloader.\r\nUboot version 2016.11\r\nLinux Kernel 4.4.6\r\n\r\nThanks,\r\nKalyan\r\n>\r\n> Thank you,\r\n> Miquèl\r\n>\r\n>> Please let me now if you have any other question or do you want me\r\n>> to send the updated patch version with the typo fixed.\r\n>>\r\n>> Thank You,\r\n>> Kalyan\r\n>>\r\n>>> Thank you for your time,\r\n>>> Miquèl\r\n>>> \r\n>>>> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is\r\n>>>> unused so remove it along with NDCR_NAND_MODE.\r\n>>>>\r\n>>>> Signed-off-by: Kalyan Kinthada\r\n>>>> ---\r\n>>> \r\n>>>> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\r\n>>>> 1 file changed, 7 insertions(+), 2 deletions(-)\r\n>>>>\r\n>>>> diff --git a/drivers/mtd/nand/pxa3xx_nand.c\r\n>>>> b/drivers/mtd/nand/pxa3xx_nand.c index 85cff68643e0..a6a7a5af7bed\r\n>>>> 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c\r\n>>>> +++ b/drivers/mtd/nand/pxa3xx_nand.c\r\n>>>> @@ -67,8 +67,7 @@\r\n>>>> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\r\n>>>> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\r\n>>>> #define NDCR_NCSX\t\t(0x1 << 23)\r\n>>>> -#define NDCR_ND_MODE\t\t(0x3 << 21)\r\n>>>> -#define NDCR_NAND_MODE \t(0x0)\r\n>>>> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\r\n>>>> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\r\n>>>> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\r\n>>>> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\r\n>>>> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct\r\n>>>> pxa3xx_nand_info *info) info->chunk_size = PAGE_CHUNK_SIZE;\r\n>>>> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\r\n>>>> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ?\r\n>>>> NDCR_ND_ARB_EN : 0;\r\n>>>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\r\n>>>> GL-5830741 */\r\n>>>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\r\n>>>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\r\n>>>> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\r\n>>>> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\r\n>>>> \r\n>>>> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct\r\n>>>> pxa3xx_nand_info *info) info->reg_ndcr = ndcr &\r\n>>>> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN |\r\n>>>> NFCV1_NDCR_ARB_CNTL); info->reg_ndcr |= (pdata->enable_arbiter) ?\r\n>>>> NDCR_ND_ARB_EN : 0;\r\n>>>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\r\n>>>> GL-5830741 */\r\n>>>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\r\n>>>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\r\n>>>> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\r\n>>>> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\r\n>>>> }\r\n>>> \r\n>\r\n>","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"scCINolZ\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=alliedtelesis.co.nz\n\theader.i=@alliedtelesis.co.nz header.b=\"LsIhWrAr\"; \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 3yBTDF5wNJz9t69\n\tfor ;\n\tWed, 11 Oct 2017 07:26:53 +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 1e2171-0004c2-3Q; Tue, 10 Oct 2017 20:26:47 +0000","from gate2.alliedtelesis.co.nz ([2001:df5:b000:5::4])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e216w-0004Z0-Lu\n\tfor linux-mtd@lists.infradead.org; Tue, 10 Oct 2017 20:26:45 +0000","from mmarshal3.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (Client did not present a certificate)\n\tby gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id DB711800E6;\n\tWed, 11 Oct 2017 09:26:11 +1300 (NZDT)","from svr-chch-ex1.atlnz.lc (Not Verified[10.32.16.77]) by\n\tmmarshal3.atlnz.lc with Trustwave SEG (v7, 5, 8, 10121)\n\tid ; Wed, 11 Oct 2017 09:26:12 +1300","from svr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) by\n\tsvr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) with Microsoft SMTP\n\tServer (TLS) id 15.0.1156.6; Wed, 11 Oct 2017 09:26:08 +1300","from svr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8]) by\n\tsvr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8%12]) with mapi id\n\t15.00.1156.000; Wed, 11 Oct 2017 09:26:08 +1300"],"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:Content-ID:In-Reply-To:\n\tReferences:Message-ID:Date:Subject:To:From:Reply-To:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=W/ngybChPBKkfTLUoWz8CSd843WNzygfUmg15NoI4lg=;\n\tb=scCINolZNxgUlI\n\tRKUM6+5Xi+TzDeouD6BQlWqpSmYMWXi1iiaurXtabOrq4/kcvdMuYFQoH/jPhbu2cQJV4NI6b9Cyw\n\tMetYjJbLeBh8q0wt0AD+FIHRBTNWQqj4+CubGemIvFIYjaZ1LRpekjOODqL/Q53AQvJhyBwlEEkm9\n\tEXYapTGNODXcM3YTVvnGNF0HDaspatFIhwWqfK1vwXSxeaazQlVcrgHPPh//7hucocoYmfZUnSHpv\n\twZtjTwBEULSqCzmyjxpgspolXE//YdJR3jRY5hlL9o/Fw8k1ZLuWmVF03esb+j5b4ZrKATJ4L/GHX\n\toQalCPUCEHhTWPVrxvcw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; \n\ts=mail; t=1507667171;\n\tbh=QMIRSWe/UJgq3EKKsIdNXc7B7ZgetQCSbiKtzHK9bQo=;\n\th=From:To:CC:Subject:Date:References:In-Reply-To;\n\tb=LsIhWrAr6fvHITgTjgSU/QA8dZAjSFSGw/nEEC79xUKyAGz/79d5qgRxtUcEr1Xf6\n\tZpFQUZK6uupoblE413PgIOu0Gtg9Q5qvRTeHE6BjYf76jOomaFYRAEmeFETcsBqQ9j\n\tJR0EP3J5iympwwYoO/3Y2OF3J+SMOTDL/hVhW4l0="],"From":"Kalyan Kinthada ","To":"Miquel RAYNAL ","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","Thread-Topic":"[PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Thread-Index":"AQHTN/TV84NhrKrRv0aGyA/oHIy1raLUEOkAgAXypQCAAD+JAIACfw2A","Date":"Tue, 10 Oct 2017 20:26:07 +0000","Message-ID":"<0a9cf087-63de-19db-f695-ca8f3796fad0@alliedtelesis.co.nz>","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20171005094138.11b73a0e@xps13>\n\t\n\t<20171009081848.49681975@xps13>","In-Reply-To":"<20171009081848.49681975@xps13>","Accept-Language":"en-NZ, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-ms-exchange-messagesentrepresentingtype":"1","x-ms-exchange-transport-fromentityheader":"Hosted","x-originating-ip":"[10.32.16.78]","Content-ID":"<312455A69AFC9D49B538D78BE1DD05BD@atlnz.lc>","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171010_132643_097426_014454F1 ","X-CRM114-Status":"GOOD ( 22.05 )","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-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO 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-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"mark.rutland@arm.com\" ,\n\t\"boris.brezillon@free-electrons.com\"\n\t, \n\t\"devicetree@vger.kernel.org\" ,\n\t\"richard@nod.at\" ,\n\tChris Packham ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"marek.vasut@gmail.com\" ,\n\t\"robh+dt@kernel.org\" ,\n\t\"linux-mtd@lists.infradead.org\" ,\n\t\"ezequiel.garcia@free-electrons.com\"\n\t, \n\t\"cyrille.pitchen@wedev4u.fr\" ,\n\t\"computersforpeace@gmail.com\" ,\n\t\"dwmw2@infradead.org\" ","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1786832,"web_url":"http://patchwork.ozlabs.org/comment/1786832/","msgid":"<20171014151233.4e9ff319@bbrezillon>","list_archive_url":null,"date":"2017-10-14T13:12:33","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","submitter":{"id":63120,"url":"http://patchwork.ozlabs.org/api/people/63120/","name":"Boris Brezillon","email":"boris.brezillon@free-electrons.com"},"content":"Hi Kalyan,\n\nOn Thu, 28 Sep 2017 13:57:56 +1300\nKalyan Kinthada wrote:\n\n> When the arbitration between NOR and NAND flash is enabled\n> the field bit[21] in the Data Flash Control Register\n> needs to be set to 1 according to guidleine GL-5830741 mentioned\n> in Marvell Errata document MV-S501377-00, Rev. D.\n\nThe real question is, is this patch fixing a real bug or are you just\nsetting this bit because a guideline says it should be set? As far as I\nunderstand, noone fully understands what this bit does and why it needs\nto be set, so if you're not fixing a real bug, I'd prefer keeping the\ncode unchanged code until someone complains for a good reason.\n\nRegards,\n\nBoris\n\n> \n> This commit sets the FORCE_CSX bit to 1 for all\n> ARMADA370 variants as the arbitration is always enabled by default.\n> This change does not apply for pxa3xx variants because the FORCE_CSX\n> bit does not exist/reserved on the NFCv1.\n> \n> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is unused so\n> remove it along with NDCR_NAND_MODE.\n> \n> Signed-off-by: Kalyan Kinthada \n> ---\n> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\n> 1 file changed, 7 insertions(+), 2 deletions(-)\n> \n> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c\n> index 85cff68643e0..a6a7a5af7bed 100644\n> --- a/drivers/mtd/nand/pxa3xx_nand.c\n> +++ b/drivers/mtd/nand/pxa3xx_nand.c\n> @@ -67,8 +67,7 @@\n> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\n> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\n> #define NDCR_NCSX\t\t(0x1 << 23)\n> -#define NDCR_ND_MODE\t\t(0x3 << 21)\n> -#define NDCR_NAND_MODE \t(0x0)\n> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\n> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\n> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\n> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\n> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct pxa3xx_nand_info *info)\n> \tinfo->chunk_size = PAGE_CHUNK_SIZE;\n> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\n> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0;\n> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#: GL-5830741 */\n> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\n> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\n> \n> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct pxa3xx_nand_info *info)\n> \tinfo->reg_ndcr = ndcr &\n> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN | NFCV1_NDCR_ARB_CNTL);\n> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0;\n> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#: GL-5830741 */\n> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\n> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\n> }","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"Q99PqXq7\"; \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 3yDlR16C47z9t38\n\tfor ;\n\tSun, 15 Oct 2017 00:14:04 +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 1e3MG0-0001Y8-1B; Sat, 14 Oct 2017 13:13:36 +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 1e3MFZ-0001QI-OV\n\tfor linux-mtd@lists.infradead.org; Sat, 14 Oct 2017 13:13:12 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 3A0E820417; Sat, 14 Oct 2017 15:12:46 +0200 (CEST)","from bbrezillon (91-160-177-164.subs.proxad.net [91.160.177.164])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id DFF1B203B3;\n\tSat, 14 Oct 2017 15:12:35 +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=nVgWNkVPfJTDsbDdmTeIW8ICqRIm3AMiuK9Sw8SAwQU=;\n\tb=Q99PqXq7aGeC1g\n\tz4ldAUctY/kp9d6UJjzF8oAMa10v2wXiKKiRPPjAEehpmZmbc4jzHbIprTqPtt0bvaMueXfKXHsvh\n\ti2I7qdBHyOLZ6D+SL6vVqGOAXoJoVwCyzYCiR+OgjbpHuxud8cBFJfrW6EDQopj3m1QouaFTohK2e\n\tNpVwt0RbXjO2NZHb8yFddV30omvLwzIC1TN6ZO1akL/DRUD8qgGWgu8Rz6zoi19ht6ogcteBYVjSA\n\tb419rK7Mf29GQ128rolP4+QrBKJvAQS/IGAgyXxo6xOf6NazOB2zM3ljGaOP3s1cMe+0jRiVazMDW\n\tCjjStd45OekOHl2GuqrQ==;","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":"Sat, 14 Oct 2017 15:12:33 +0200","From":"Boris Brezillon ","To":"Kalyan Kinthada ","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Message-ID":"<20171014151233.4e9ff319@bbrezillon>","In-Reply-To":"<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>","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-20171014_061310_177391_361C9602 ","X-CRM114-Status":"GOOD ( 19.44 )","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 ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, richard@nod.at,\n\tmiquel.raynal@free-electrons.com, chris.packham@alliedtelesis.co.nz, \n\tlinux-kernel@vger.kernel.org, marek.vasut@gmail.com, robh+dt@kernel.org, \n\tlinux-mtd@lists.infradead.org, ezequiel.garcia@free-electrons.com,\n\tcyrille.pitchen@wedev4u.fr, computersforpeace@gmail.com,\n\tdwmw2@infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1787077,"web_url":"http://patchwork.ozlabs.org/comment/1787077/","msgid":"<5d708701612f48cf958da5feeb68365f@svr-chch-ex1.atlnz.lc>","list_archive_url":null,"date":"2017-10-15T20:55:43","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","submitter":{"id":27499,"url":"http://patchwork.ozlabs.org/api/people/27499/","name":"Chris Packham","email":"chris.packham@alliedtelesis.co.nz"},"content":"Hi Boris,\n\nOn 15/10/17 02:13, Boris Brezillon wrote:\n> Hi Kalyan,\n> \n> On Thu, 28 Sep 2017 13:57:56 +1300\n> Kalyan Kinthada wrote:\n> \n>> When the arbitration between NOR and NAND flash is enabled\n>> the field bit[21] in the Data Flash Control Register\n>> needs to be set to 1 according to guidleine GL-5830741 mentioned\n>> in Marvell Errata document MV-S501377-00, Rev. D.\n> \n> The real question is, is this patch fixing a real bug or are you just\n> setting this bit because a guideline says it should be set?\n\nYes that's a fair summary. It is in a document called \"Functional \nErrata\" so there is some feeling here that we (allied telesis) should \nfollow up things in these kinds of documents.\n\n> As far as I\n> understand, noone fully understands what this bit does and why it needs\n> to be set, so if you're not fixing a real bug, I'd prefer keeping the\n> code unchanged code until someone complains for a good reason.\n\nAgreed. We've certainly not noticed a change in behaviour on our boards. \nI've requested more information from Marvell through their support \nchannels. I'll share what I can if/when they come back with something \nuseful.\n\n> \n> Regards,\n> \n> Boris\n> \n>>\n>> This commit sets the FORCE_CSX bit to 1 for all\n>> ARMADA370 variants as the arbitration is always enabled by default.\n>> This change does not apply for pxa3xx variants because the FORCE_CSX\n>> bit does not exist/reserved on the NFCv1.\n>>\n>> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is unused so\n>> remove it along with NDCR_NAND_MODE.\n>>\n>> Signed-off-by: Kalyan Kinthada \n>> ---\n>> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\n>> 1 file changed, 7 insertions(+), 2 deletions(-)\n>>\n>> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c\n>> index 85cff68643e0..a6a7a5af7bed 100644\n>> --- a/drivers/mtd/nand/pxa3xx_nand.c\n>> +++ b/drivers/mtd/nand/pxa3xx_nand.c\n>> @@ -67,8 +67,7 @@\n>> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\n>> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\n>> #define NDCR_NCSX\t\t(0x1 << 23)\n>> -#define NDCR_ND_MODE\t\t(0x3 << 21)\n>> -#define NDCR_NAND_MODE \t(0x0)\n>> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\n>> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\n>> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\n>> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\n>> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct pxa3xx_nand_info *info)\n>> \tinfo->chunk_size = PAGE_CHUNK_SIZE;\n>> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\n>> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0;\n>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#: GL-5830741 */\n>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n>> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\n>> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\n>> \n>> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct pxa3xx_nand_info *info)\n>> \tinfo->reg_ndcr = ndcr &\n>> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN | NFCV1_NDCR_ARB_CNTL);\n>> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0;\n>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#: GL-5830741 */\n>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n>> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\n>> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\n>> }\n> \n>","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"uIPTfyAP\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=alliedtelesis.co.nz\n\theader.i=@alliedtelesis.co.nz header.b=\"aVXIYL99\"; \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 3yFYg00GJfz9sRm\n\tfor ;\n\tMon, 16 Oct 2017 07:57:16 +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 1e3pxm-0000Pt-Dc; Sun, 15 Oct 2017 20:56:46 +0000","from gate2.alliedtelesis.co.nz ([2001:df5:b000:5::4])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e3pxJ-0000HM-Hh\n\tfor linux-mtd@lists.infradead.org; Sun, 15 Oct 2017 20:56:23 +0000","from mmarshal3.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (Client did not present a certificate)\n\tby gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id 14BF18066C;\n\tMon, 16 Oct 2017 09:55:44 +1300 (NZDT)","from svr-chch-ex1.atlnz.lc (Not Verified[10.32.16.77]) by\n\tmmarshal3.atlnz.lc with Trustwave SEG (v7, 5, 8, 10121)\n\tid ; Mon, 16 Oct 2017 09:55:42 +1300","from svr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) by\n\tsvr-chch-ex1.atlnz.lc (2001:df5:b000:bc8::77) with Microsoft SMTP\n\tServer (TLS) id 15.0.1156.6; Mon, 16 Oct 2017 09:55:43 +1300","from svr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8]) by\n\tsvr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8%12]) with mapi id\n\t15.00.1156.000; Mon, 16 Oct 2017 09:55:43 +1300"],"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:Message-ID:Date\n\t:Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:\n\tList-Owner; bh=pYrtxe7iSrlsvntrEK2qXds/nryuBdH1Lb2bTpmyLAE=;\n\tb=uIPTfyAPyjkSsT\n\tf6cm34cAYThl5K4hLALDd3bHvIq4l8Jzd18mNXG573t54tS6ur0QD7xpjE17GT5ivLtTxhQm/mSQi\n\tnbs5bwH9zVxOt8ihi+sqcJjl9YL8vy+eOOLnQmoqw+M/lLqwBbmZz2EmUA2P1h7+rXnbFIM1ZDvVw\n\tgFJO/SF1QOceCBqqpoVGWjCswP+0ZGLqPlCP5pU3PvRwavOl8u0HORqMMIi7+xa/FxJlXeIK0zc5M\n\t81Ma/fKrHYB0ZDreBIxto18zQLPgnEgRcmS4HN7+tGzLWgstOBXczErknrxTAjeUJFTbeCcDb0H3K\n\tA7nnRGJlW2CL1KJh5a0Q==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; \n\ts=mail; t=1508100944;\n\tbh=YRnkvgBZ3iIPFeeqhotInaFFdrTmaigAeZMMDjz+8u4=;\n\th=From:To:CC:Subject:Date:References;\n\tb=aVXIYL99R5Pl23TQtK2yT+s/jSZT0MTnCkg5D800F7dvE8tsnynHn2mqJtPvP7769\n\teVdcSIl4BnJhZidEAmZE8TG51VJKoGzzvPB7WQv6AyDm6QDSQedlMp5PqfuU2nZtD5\n\trK8nKgLacSLqVH8Fyoah90PsOMEfdLorb3nXialg="],"From":"Chris Packham ","To":"Boris Brezillon , Kalyan Kinthada\n\t","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","Thread-Topic":"[PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Thread-Index":"AQHTN/TV5c3BTkMJ8EyjaCc6GEyjZw==","Date":"Sun, 15 Oct 2017 20:55:43 +0000","Message-ID":"<5d708701612f48cf958da5feeb68365f@svr-chch-ex1.atlnz.lc>","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20171014151233.4e9ff319@bbrezillon>","Accept-Language":"en-NZ, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-ms-exchange-transport-fromentityheader":"Hosted","x-originating-ip":"[2001:df5:b000:22:3a2c:4aff:fe70:2b02]","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171015_135618_041401_923A0618 ","X-CRM114-Status":"GOOD ( 15.26 )","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-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO 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-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"mark.rutland@arm.com\" ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"richard@nod.at\" ,\n\t\"miquel.raynal@free-electrons.com\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"marek.vasut@gmail.com\" ,\n\t\"robh+dt@kernel.org\" ,\n\t\"linux-mtd@lists.infradead.org\" ,\n\t\"ezequiel.garcia@free-electrons.com\"\n\t, \n\t\"cyrille.pitchen@wedev4u.fr\" ,\n\t\"computersforpeace@gmail.com\" ,\n\t\"dwmw2@infradead.org\" ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}},{"id":1796383,"web_url":"http://patchwork.ozlabs.org/comment/1796383/","msgid":"","list_archive_url":null,"date":"2017-10-31T03:15:01","subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","submitter":{"id":27499,"url":"http://patchwork.ozlabs.org/api/people/27499/","name":"Chris Packham","email":"chris.packham@alliedtelesis.co.nz"},"content":"Hi Miquel,\n\nOn 09/10/17 19:19, Miquel RAYNAL wrote:\n> Hello Kalyan,\n> \n> On Mon, 9 Oct 2017 02:31:30 +0000\n> Kalyan Kinthada wrote:\n> \n>> On 05/10/17 20:41, Miquel RAYNAL wrote:\n>>> Hello Kalyan,\n>>>\n>>> On Thu, 28 Sep 2017 13:57:56 +1300\n>>> Kalyan Kinthada wrote:\n>>> \n>>>> When the arbitration between NOR and NAND flash is enabled\n>>>> the field bit[21] in the Data Flash Control Register\n>>>> needs to be set to 1 according to guidleine GL-5830741 mentioned\n>>> Typo: guideline ^\n>> I will correct this typo in the next patch version.\n>>>> in Marvell Errata document MV-S501377-00, Rev. D.\n>>> Thanks for that, I checked it.\n>>> \n>>>> This commit sets the FORCE_CSX bit to 1 for all\n>>>> ARMADA370 variants as the arbitration is always enabled by default.\n>>>> This change does not apply for pxa3xx variants because the\n>>>> FORCE_CSX bit does not exist/reserved on the NFCv1.\n>>> Sorry to bother you again but I am reworking the pxa3xx_nand driver\n>>> and so I would like to deeply understand why this is needed because\n>>> I will have to integrate it in my work too.\n>>>\n>>> So please tell me what is your setup, do you really use NAND/NOR\n>>> arbitration? Do you have not-Don't Care CS NAND chips? I have some\n>>> doubts because even if the spec precises in the NAND controller\n>>> chapter that arbitration is always enabled by default, the bit 27\n>>> (NfArbiterEn) in the SoC Device Multiplex Register at offset\n>>> 0x00018208 is actually at 0 by default (disabled). Did you enable\n>>> this bit manually ? I checked with devmem on my setup and this bit\n>>> was unset.\n>> Yes, my setup use NAND/NOR arbitration and use a Don't\n>> care CS Nand Chip. The bit 27 at offset 0x00018208 is set to 1\n>> by default in my case. I did not manually set the bit 27 to 1.\n> \n> Actually if you use Don't Care CS NAND chips you should not need the\n> force CS bit, as it is mentioned in the guidelines you pointed, this\n> bit is only useful for *not* don't care CS NAND chips (and the name\n> \"force CS\" is pretty straight forward too!).\n\nIt's less straight forward than you think. It's FORCE_CSX (the X is \nimportant) which is defined as \"force chip select false on busy\". What \nit means is that the NF_CSn is de-asserted if the interface is \narbitrated away from the NAND flash controller. I think that means you \nwon't end up with the chip-select for the dev-bus/NOR firing immediately \nwhen that device is arbitrated.\n\n> Otherwise what bootloader and kernel do you use to have this\n> bit set by default? You may also want to check if this bit is set by\n> your bootloader by stopping it before it loads Linux and check manually\n> the value of this bit with 'md' (on U-Boot).\n> \n> Thank you,\n> Miquèl\n> \n>>\n>> Please let me now if you have any other question or do you want me\n>> to send the updated patch version with the typo fixed.\n>>\n>> Thank You,\n>> Kalyan\n>>\n>>>\n>>> Thank you for your time,\n>>> Miquèl\n>>> \n>>>> The NDCR_ND_MODE conflicts with the new NDCR_FORCE_CSX but is\n>>>> unused so remove it along with NDCR_NAND_MODE.\n>>>>\n>>>> Signed-off-by: Kalyan Kinthada\n>>>> ---\n>>>\n>>> \n>>>> drivers/mtd/nand/pxa3xx_nand.c | 9 +++++++--\n>>>> 1 file changed, 7 insertions(+), 2 deletions(-)\n>>>>\n>>>> diff --git a/drivers/mtd/nand/pxa3xx_nand.c\n>>>> b/drivers/mtd/nand/pxa3xx_nand.c index 85cff68643e0..a6a7a5af7bed\n>>>> 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c\n>>>> +++ b/drivers/mtd/nand/pxa3xx_nand.c\n>>>> @@ -67,8 +67,7 @@\n>>>> #define NDCR_DWIDTH_M\t\t(0x1 << 26)\n>>>> #define NDCR_PAGE_SZ\t\t(0x1 << 24)\n>>>> #define NDCR_NCSX\t\t(0x1 << 23)\n>>>> -#define NDCR_ND_MODE\t\t(0x3 << 21)\n>>>> -#define NDCR_NAND_MODE \t(0x0)\n>>>> +#define NDCR_FORCE_CSX\t\t(0x1 << 21)\n>>>> #define NDCR_CLR_PG_CNT\t\t(0x1 << 20)\n>>>> #define NFCV1_NDCR_ARB_CNTL\t(0x1 << 19)\n>>>> #define NFCV2_NDCR_STOP_ON_UNCOR\t(0x1 << 19)\n>>>> @@ -1464,6 +1463,9 @@ static int pxa3xx_nand_config_ident(struct\n>>>> pxa3xx_nand_info *info) info->chunk_size = PAGE_CHUNK_SIZE;\n>>>> \tinfo->reg_ndcr = 0x0; /* enable all interrupts */\n>>>> \tinfo->reg_ndcr |= (pdata->enable_arbiter) ?\n>>>> NDCR_ND_ARB_EN : 0;\n>>>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n>>>> GL-5830741 */\n>>>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n>>>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n>>>> \tinfo->reg_ndcr |= NDCR_RD_ID_CNT(READ_ID_BYTES);\n>>>> \tinfo->reg_ndcr |= NDCR_SPARE_EN;\n>>>> \n>>>> @@ -1498,6 +1500,9 @@ static void pxa3xx_nand_detect_config(struct\n>>>> pxa3xx_nand_info *info) info->reg_ndcr = ndcr &\n>>>> \t\t~(NDCR_INT_MASK | NDCR_ND_ARB_EN |\n>>>> NFCV1_NDCR_ARB_CNTL); info->reg_ndcr |= (pdata->enable_arbiter) ?\n>>>> NDCR_ND_ARB_EN : 0;\n>>>> +\t/* Set FORCE_CSX bit for all ARMADA370 Variants. Ref#:\n>>>> GL-5830741 */\n>>>> +\tif (info->variant == PXA3XX_NAND_VARIANT_ARMADA370)\n>>>> +\t\tinfo->reg_ndcr |= NDCR_FORCE_CSX;\n>>>> \tinfo->ndtr0cs0 = nand_readl(info, NDTR0CS0);\n>>>> \tinfo->ndtr1cs0 = nand_readl(info, NDTR1CS0);\n>>>> }\n>>>\n>>> \n> \n> \n>","headers":{"Return-Path":"","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=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"NAgTV0+0\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=alliedtelesis.co.nz\n\theader.i=@alliedtelesis.co.nz header.b=\"b7Ga01Ho\"; \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 3yQxM84mNzz9t2c\n\tfor ;\n\tTue, 31 Oct 2017 14:16:04 +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 1e9N1v-0003bc-Bl; Tue, 31 Oct 2017 03:15:55 +0000","from gate2.alliedtelesis.co.nz ([2001:df5:b000:5::4])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e9N1r-0002k5-Bh\n\tfor linux-mtd@lists.infradead.org; Tue, 31 Oct 2017 03:15:53 +0000","from mmarshal3.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (Client did not present a certificate)\n\tby gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id 953D58365F;\n\tTue, 31 Oct 2017 16:15:06 +1300 (NZDT)","from svr-chch-ex1.atlnz.lc (Not Verified[10.32.16.77]) by\n\tmmarshal3.atlnz.lc with Trustwave SEG (v7, 5, 8, 10121)\n\tid ; Tue, 31 Oct 2017 16:15:06 +1300","from svr-chch-ex1.atlnz.lc (2001:df5:b000:bc8:409d:36f5:8899:92e8)\n\tby svr-chch-ex1.atlnz.lc (2001:df5:b000:bc8:409d:36f5:8899:92e8)\n\twith Microsoft SMTP Server (TLS) id 15.0.1156.6;\n\tTue, 31 Oct 2017 16:15:01 +1300","from svr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8]) by\n\tsvr-chch-ex1.atlnz.lc ([fe80::409d:36f5:8899:92e8%12]) with mapi id\n\t15.00.1156.000; Tue, 31 Oct 2017 16:15:01 +1300"],"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:Message-ID:Date\n\t:Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:\n\tList-Owner; bh=iGwXEJWoOkqPeWZZ46B8ndetpqkOvseTLVeyC72qmWc=;\n\tb=NAgTV0+0BnYopP\n\tLBhVMRDqMEM1wJEeuLG4gx+IJvDeAYppvnsRvMGICYFtTsxEAK1RKSYBLKfqBGM+quXB19sD+KDSp\n\tmP+Pr4CQgapC0rZgzYwsRIIh4BQmLQGvSWIlydqjLHbdvdokkVQyTRL8WxUjgE1DALzsnYd1WlRI3\n\tApNhAn2aRW9LQeHk2FgRZCXDVFjcNmicp9SRjvn7tqR3rKOmQ/IRgiYyaoVhhMDfFCEyoyBdSaxWL\n\t/Ei/eKOgaN3XUcLqqqsgfxSsQSUCG3N5GIiX7Gl0Dgp3tJ5lIlEOx4KPxqRrZSeGV1Fu3XiX5d1TH\n\t11EZbH5tsfzRS+e/CiUw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; \n\ts=mail; t=1509419706;\n\tbh=lqYCFKb5U1q+gynoEJ5wRCl8n6nYTc++g7pt0AruuR8=;\n\th=From:To:CC:Subject:Date:References;\n\tb=b7Ga01HoLM1FA4+t2TPqWIevNCAavRRRlPdzhNSpV+Nil3UjQc44V9d2BYUkiMmg9\n\ta1hq/XSXNzYNr5zmSCUQX4xye0iAwDApuUXIlrAKLKt1Qlz/roxvyWD0GyKl0a+Sm+\n\t3YMVztgCS5RZwv7wv2qf8rEUYgtQ81qgsbwQTskk="],"From":"Chris Packham ","To":"Miquel RAYNAL , Kalyan Kinthada\n\t","Subject":"Re: [PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to ARMADA370\n\tvariants.","Thread-Topic":"[PATCH v3 1/1] mtd: nand: pxa3xx: Set FORCE_CSX bit to\n\tARMADA370 variants.","Thread-Index":"AQHTN/TV5c3BTkMJ8EyjaCc6GEyjZw==","Date":"Tue, 31 Oct 2017 03:15:01 +0000","Message-ID":"","References":"<20170928005756.3938-1-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20170928005756.3938-2-kalyan.kinthada@alliedtelesis.co.nz>\n\t<20171005094138.11b73a0e@xps13>\n\t\n\t<20171009081848.49681975@xps13>","Accept-Language":"en-NZ, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-ms-exchange-transport-fromentityheader":"Hosted","x-originating-ip":"[2001:df5:b000:22:3a2c:4aff:fe70:2b02]","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171030_201551_761651_A20BD122 ","X-CRM114-Status":"GOOD ( 20.31 )","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-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO 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-mtd@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"Linux MTD discussion mailing list ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"mark.rutland@arm.com\" ,\n\t\"boris.brezillon@free-electrons.com\"\n\t, \n\t\"devicetree@vger.kernel.org\" ,\n\t\"richard@nod.at\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"marek.vasut@gmail.com\" ,\n\t\"robh+dt@kernel.org\" ,\n\t\"linux-mtd@lists.infradead.org\" ,\n\t\"ezequiel.garcia@free-electrons.com\"\n\t, \n\t\"cyrille.pitchen@wedev4u.fr\" ,\n\t\"computersforpeace@gmail.com\" ,\n\t\"dwmw2@infradead.org\" ","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","Sender":"\"linux-mtd\" ","Errors-To":"linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org"}}]