From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from 2.mo6.mail-out.ovh.net ([46.105.76.65] helo=mo6.mail-out.ovh.net) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WKgee-0006iY-Qs for barebox@lists.infradead.org; Tue, 04 Mar 2014 04:08:33 +0000 Received: from mail95.ha.ovh.net (gw6.ovh.net [213.251.189.206]) by mo6.mail-out.ovh.net (Postfix) with SMTP id 57F18FF9604 for ; Tue, 4 Mar 2014 05:01:05 +0100 (CET) Date: Tue, 4 Mar 2014 05:02:06 +0100 From: Jean-Christophe PLAGNIOL-VILLARD Message-ID: <20140304040206.GO9671@ns203013.ovh.net> References: <5310B108.8040800@celad.com> <5313FACC.8030202@atmel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <5313FACC.8030202@atmel.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: nand pmecc crash detected To: Josh Wu Cc: Herve Codina , barebox@lists.infradead.org, linux-arm-kernel@lists.infradead.org On 11:45 Mon 03 Mar , Josh Wu wrote: > Dear Herve > > On 2/28/2014 11:53 PM, Herve Codina wrote: > >Hi, > > > >I am using an sam5d35 and its pmecc nand ecc controller. > > > >In barebox, using similar atmel_nand.c the system crashes. > > > >The problem was a call to chip->ecc.hwctl from nand_write_subpage_hwecc > >(nand_base.c) when we write a sub page. > > > >chip->ecc.hwctl function is not set when we are using PMECC controller. > > > >As a workaround, i set NAND_NO_SUBPAGE_WRITE for PMECC controller in > >order to disable sub page access in nand_write_page. > > > >I think we can have the same problem in Linux. I do not performed > >subpage nand writes from Linux but i made the following patch according > >to my Barebox defect. > > Thank you for catching this crash bug. The fix is fine for me. But I > think it's better to send out the Linux kernel patch alone. > > You can rebase your patch on the top of Linux-mtd git > repo(git://git.infradead.org/l2-mtd.git) instead of your local > Linux. > Then use 'git send-email' send to linux-mtd@lists.infradead.org and > linux-arm-kernel@lists.infradead.org. I will ack your patch. > > If the patch is merged in Linux-mtd, it should be easier port to > barebox then. no need for that notion of priority send it to both Best Regards, J. > > > > >Here after are my patches (same modification) for 1st my local Barebox > >and 2nd my local Linux : > > > >---8<------------------- > > From 757e8dbf141619a55b7a5b1ca11780b2589eacc1 Mon Sep 17 00:00:00 2001 > >From: Herve Codina > >Date: Fri, 28 Feb 2014 08:39:08 +0100 > >Subject: [PATCH 5/5] Disable Subpage nand write when using Atmel PMECC. > > > > > >Signed-off-by: Herve Codina > >--- > > drivers/mtd/nand/atmel_nand.c | 1 + > > 1 file changed, 1 insertion(+) > > > >diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c > >index 2ff7427..cdbe685 100644 > >--- a/drivers/mtd/nand/atmel_nand.c > >+++ b/drivers/mtd/nand/atmel_nand.c > >@@ -837,6 +837,7 @@ static int __init > >atmel_pmecc_nand_init_params(struct device_d *dev, > > return err_no; > > } > > > >+ nand_chip->options |= NAND_NO_SUBPAGE_WRITE; > > nand_chip->ecc.read_page = atmel_nand_pmecc_read_page; > > nand_chip->ecc.write_page = atmel_nand_pmecc_write_page; > > > > Best Regards, > Josh Wu > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox