From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chin Liang See Date: Thu, 27 Feb 2014 15:02:33 -0600 Subject: [U-Boot] [PATCH v2 1/2] nand/denali: Adding Denali NAND driver support In-Reply-To: <20140227233551.77C1.AA925319@jp.panasonic.com> References: <1393015864-7957-1-git-send-email-clsee@altera.com> <20140227233551.77C1.AA925319@jp.panasonic.com> Message-ID: <1393534953.1993.6.camel@clsee-VirtualBox.altera.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Masahiro, On Thu, 2014-02-27 at 23:35 +0900, Masahiro Yamada wrote: > Hello Chin, > > > > + > > + nand->ecc.mode = NAND_ECC_HW; > > + nand->ecc.size = CONFIG_NAND_DENALI_ECC_SIZE; > > + nand->ecc.read_oob = denali_read_oob; > > + nand->ecc.write_oob = denali_write_oob; > > + nand->ecc.read_page = denali_read_page; > > + nand->ecc.read_page_raw = denali_read_page_raw; > > + nand->ecc.write_page = denali_write_page; > > + nand->ecc.write_page_raw = denali_write_page_raw; > > +#ifdef CONFIG_SYS_NAND_15BIT_HW_ECC_OOBFIRST > > + /* 15bit ECC */ > > + nand->ecc.bytes = 26; > > + nand->ecc.layout = &nand_15bit_oob; > > +#else /* 8bit ECC */ > > + nand->ecc.bytes = 14; > > + nand->ecc.layout = &nand_8bit_oob; > > +#endif > > + nand->ecc.calculate = denali_ecc_calculate; > > + nand->ecc.correct = denali_ecc_correct; > > + nand->ecc.hwctl = denali_ecc_hwctl; > > You set nand->ecc.mode = NAND_ECC_HW, > but it looks like you don't set nand->ecc.strength. > > So, I think initialization will fail in nand_scan_tail() function. > > Here, > > if (mtd->writesize >= chip->ecc.size) { > if (!chip->ecc.strength) { > pr_warn("Driver must set ecc.strength when using hardware ECC\n"); > BUG(); > } > break; > > > > Where do you set nand->ecc.strength? I believe this is only applicable for NAND_ECC_HW_SYNDROME mode. We are using the NAND_ECC_HW (without the syndrome). Wonder you hit error during run? Chin Liang > > > > > Best Regards > Masahiro Yamada > >