From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Wed, 19 Jun 2013 15:40:19 +0200 Subject: [U-Boot] nand onfi 8bit/16bit bus support In-Reply-To: <1368135452.654.12@snotra> References: <1368135452.654.12@snotra> Message-ID: <51C1B4C3.1060701@monstr.eu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Scott, On 05/09/2013 11:37 PM, Scott Wood wrote: > On 05/06/2013 11:53:52 AM, Michal Simek wrote: >> Hi guys, >> >> Zynq supports 8 and 16bit ONFI nand flashes. >> We can count number of connected pins and from that >> we know if 8bit or 16bit nand flash is used. >> >> I have looked at the code and contains ONFI detection >> with correct detection of bus width + checking later. >> Why also not setup proper chip->options for NAND_BUSWIDTH_16? >> >> Not sure how others but if driver expect just ONFI devices >> and support 8 and 16bit options why not just to setup >> chip options based on information from ONFI or >> not check that (busw != (chip->options & NAND_BUSWIDTH_16)) >> for onfi case. > > This is fixed in current Linux, with NAND_BUSWIDTH_AUTO. There was a patchset to merge in recent Linux MTD code a while back, but an expected respin didn't come. I'll try to fix it up when I process this merge window's patches. Have you done this change? I have grepped the latest version and I can't see it there. Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 263 bytes Desc: OpenPGP digital signature URL: