From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 22 Jul 2011 13:57:32 +0800 From: Shawn Guo To: Huang Shijie Subject: Re: [PATCH v8 0/3] add the GPMI controller driver for IMX23/IMX28 Message-ID: <20110722055731.GI26837@S2100-06.ap.freescale.net> References: <1311230846-26437-1-git-send-email-b32955@freescale.com> <20110721215020.GA25344@pengutronix.de> <4E28EEE1.2050008@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <4E28EEE1.2050008@freescale.com> Cc: arnd@arndb.de, dedekind1@gmail.com, Wolfram Sang , linux-mtd@lists.infradead.org, shijie8@gmail.com, linux-arm-kernel@lists.infradead.org, LW@KARO-electronics.de List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Jul 22, 2011 at 11:30:41AM +0800, Huang Shijie wrote: > Hi, > >> The general-purpose media interface(GPMI) controller is a flexible interface > >> to up to several NAND flashs. > > ... > >> To Walfram & Artem: > >> About how to disable the JFFS2 to use the OOB: > >> I read the code, and I still have no idea about how to use the ecclayout > >> to do the job. Could you give me some hint? thanks. > > Have you checked mxc_nand.c for example? There is > > > > static struct nand_ecclayout nandv1_hw_eccoob_smallpage = { > > .eccbytes = 5, > > .eccpos = {6, 7, 8, 9, 10}, > > .oobfree = {{0, 5}, {12, 4}, } > > } > > > > defined as one layout. Now, you could define one where oobfree is empty and > > eccbytes as big as the oob-area. > > > thanks. I will check the code. > >> The driver depends on another GPMI-NAND device patch set, you can find them at : > >> [1] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037033.html > >> [2] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037031.html > >> [3] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037032.html > >> [4] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037034.html > >> > >> The driver also depends on another DMA patch by Shawn: > >> [0] http://lists.infradead.org/pipermail/linux-mtd/2011-June/036820.html > > This makes it difficult for testers/reviewers. Please try to get a git-branch > > from Freescale or Linaro. > > > Shawn will merge my patches to his Linaro branch. > git://git.linaro.org/people/shawnguo/linux-2.6.git mxs-gpmi Shijie, please check and test it. I only did a build test. -- Regards, Shawn From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawn.guo@freescale.com (Shawn Guo) Date: Fri, 22 Jul 2011 13:57:32 +0800 Subject: [PATCH v8 0/3] add the GPMI controller driver for IMX23/IMX28 In-Reply-To: <4E28EEE1.2050008@freescale.com> References: <1311230846-26437-1-git-send-email-b32955@freescale.com> <20110721215020.GA25344@pengutronix.de> <4E28EEE1.2050008@freescale.com> Message-ID: <20110722055731.GI26837@S2100-06.ap.freescale.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Jul 22, 2011 at 11:30:41AM +0800, Huang Shijie wrote: > Hi, > >> The general-purpose media interface(GPMI) controller is a flexible interface > >> to up to several NAND flashs. > > ... > >> To Walfram & Artem: > >> About how to disable the JFFS2 to use the OOB: > >> I read the code, and I still have no idea about how to use the ecclayout > >> to do the job. Could you give me some hint? thanks. > > Have you checked mxc_nand.c for example? There is > > > > static struct nand_ecclayout nandv1_hw_eccoob_smallpage = { > > .eccbytes = 5, > > .eccpos = {6, 7, 8, 9, 10}, > > .oobfree = {{0, 5}, {12, 4}, } > > } > > > > defined as one layout. Now, you could define one where oobfree is empty and > > eccbytes as big as the oob-area. > > > thanks. I will check the code. > >> The driver depends on another GPMI-NAND device patch set, you can find them at : > >> [1] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037033.html > >> [2] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037031.html > >> [3] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037032.html > >> [4] http://lists.infradead.org/pipermail/linux-mtd/2011-July/037034.html > >> > >> The driver also depends on another DMA patch by Shawn: > >> [0] http://lists.infradead.org/pipermail/linux-mtd/2011-June/036820.html > > This makes it difficult for testers/reviewers. Please try to get a git-branch > > from Freescale or Linaro. > > > Shawn will merge my patches to his Linaro branch. > git://git.linaro.org/people/shawnguo/linux-2.6.git mxs-gpmi Shijie, please check and test it. I only did a build test. -- Regards, Shawn