From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <4FB0E480.1070609@atmel.com> Date: Mon, 14 May 2012 18:54:56 +0800 From: Josh Wu MIME-Version: 1.0 To: dedekind1@gmail.com Subject: Re: [PATCH v6] MTD: atmel_nand: Update driver to support Programmable Multibit ECC controller References: <1336561255-30905-1-git-send-email-josh.wu@atmel.com> <1336738300.2625.57.camel@sauron.fi.intel.com> In-Reply-To: <1336738300.2625.57.camel@sauron.fi.intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: hongxu.cn@gmail.com, nicolas.ferre@atmel.com, linux-mtd@lists.infradead.org, ivan.djelic@parrot.com, plagnioj@jcrosoft.com, linux-arm-kernel@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi, Artem On 5/11/2012 8:11 PM, Artem Bityutskiy wrote: > On Wed, 2012-05-09 at 19:00 +0800, Josh Wu wrote: >> The Programmable Multibit ECC (PMECC) controller is a programmable binary >> BCH(Bose, Chaudhuri and Hocquenghem) encoder and decoder. This controller >> can be used to support both SLC and MLC NAND Flash devices. It supports to >> generate ECC to correct 2, 4, 8, 12 or 24 bits of error per sector of data. >> >> To use this driver, the user needs to pass in the correction capability and >> the sector size. >> >> This driver has been tested on AT91SAM9X5-EK and AT91SAM9N12-EK with JFFS2, >> YAFFS2, UBIFS and mtd-utils. >> >> Signed-off-by: Hong Xu >> Signed-off-by: Josh Wu > The patch is really big - please, split it on more smaller patches. OK, I can spit to more patches. > Besides, it does not compile for me: > > ERROR: "of_flat_dt_is_compatible" [drivers/mtd/nand/atmel_nand.ko] undefined! > ERROR: "of_get_flat_dt_root" [drivers/mtd/nand/atmel_nand.ko] undefined! > make[2]: *** [__modpost] Error 1 This function will depend on CONFIG_OF_FLATTREE and CONFIG_OF_EARLY_FLATTREE. Would it be possible that CONFIG_OF is defined, but above two config options are not defined? Can I see you .config file go get more details? Best Regards, Josh Wu From mboxrd@z Thu Jan 1 00:00:00 1970 From: josh.wu@atmel.com (Josh Wu) Date: Mon, 14 May 2012 18:54:56 +0800 Subject: [PATCH v6] MTD: atmel_nand: Update driver to support Programmable Multibit ECC controller In-Reply-To: <1336738300.2625.57.camel@sauron.fi.intel.com> References: <1336561255-30905-1-git-send-email-josh.wu@atmel.com> <1336738300.2625.57.camel@sauron.fi.intel.com> Message-ID: <4FB0E480.1070609@atmel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Artem On 5/11/2012 8:11 PM, Artem Bityutskiy wrote: > On Wed, 2012-05-09 at 19:00 +0800, Josh Wu wrote: >> The Programmable Multibit ECC (PMECC) controller is a programmable binary >> BCH(Bose, Chaudhuri and Hocquenghem) encoder and decoder. This controller >> can be used to support both SLC and MLC NAND Flash devices. It supports to >> generate ECC to correct 2, 4, 8, 12 or 24 bits of error per sector of data. >> >> To use this driver, the user needs to pass in the correction capability and >> the sector size. >> >> This driver has been tested on AT91SAM9X5-EK and AT91SAM9N12-EK with JFFS2, >> YAFFS2, UBIFS and mtd-utils. >> >> Signed-off-by: Hong Xu >> Signed-off-by: Josh Wu > The patch is really big - please, split it on more smaller patches. OK, I can spit to more patches. > Besides, it does not compile for me: > > ERROR: "of_flat_dt_is_compatible" [drivers/mtd/nand/atmel_nand.ko] undefined! > ERROR: "of_get_flat_dt_root" [drivers/mtd/nand/atmel_nand.ko] undefined! > make[2]: *** [__modpost] Error 1 This function will depend on CONFIG_OF_FLATTREE and CONFIG_OF_EARLY_FLATTREE. Would it be possible that CONFIG_OF is defined, but above two config options are not defined? Can I see you .config file go get more details? Best Regards, Josh Wu