From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Vorontsov Date: Thu, 29 May 2008 15:39:32 +0400 Subject: [U-Boot-Users] [PATCH] TQM85xx: NAND support via local bus UPMB In-Reply-To: <483DA6AD.6030509@grandegger.com> References: <1211998349-2117-5-git-send-email-wg@grandegger.com> <1211998349-2117-6-git-send-email-wg@grandegger.com> <1211998349-2117-7-git-send-email-wg@grandegger.com> <1211998349-2117-8-git-send-email-wg@grandegger.com> <1211998349-2117-9-git-send-email-wg@grandegger.com> <1211998349-2117-10-git-send-email-wg@grandegger.com> <1211998349-2117-11-git-send-email-wg@grandegger.com> <1211998349-2117-12-git-send-email-wg@grandegger.com> <20080528182531.GA3308@loki.buserror.net> <483DA6AD.6030509@grandegger.com> Message-ID: <20080529113932.GA5835@polina.dev.rtsoft.ru> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, May 28, 2008 at 08:38:37PM +0200, Wolfgang Grandegger wrote: > Scott Wood wrote: > > On Wed, May 28, 2008 at 08:12:28PM +0200, Wolfgang Grandegger wrote: > >> This patch adds support for NAND FLASH on the TQM8548. It is disabled by > >> default and can be enabled for the TQM8548 modules. Note that the R/B pin > >> is not supported by that module requiring to use the specified maximum > >> delay time. > >> > >> Note: With NAND support enabled the size of the U-Boot image exceeds > >> 256 KB and TEXT_BASE must therefore be set to 0xfff80000 in config.mk, > >> doubling the image size :-(. > > > > What does this do differently from the code in drivers/mtd/nand/fsl_upm.c? > > Maybe it does not support multi banks on a NAND chip. I have to check. Me thinks that you'll have to call fsl_upm_nand_init() for each chip, and that's all. If not, feel free to patch it as you feel appropriate, I'll able to regress-test this driver on MPC8360E-RDK. > > How much of this is board-specific? > > Well, I already gave drivers/mtd/nand/fsl_upm.c a try but was unable to > get it working on this board. Therefore I decided to keep this known to > work driver which we have already for a while. This isn't really an excuse to duplicate drivers. :-) This driver was tested on MPC8555 and MPC8360 CPUs, so it should work with no drastic changes. Some issues might still be there, and if so, fixes are highly appreciated. > With Linux, I had more success. ..especially if general idea works well, we should use single driver. Thanks, -- Anton Vorontsov email: cbouatmailru at gmail.com irc://irc.freenode.net/bd2