Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread David Brownell
Hi Sandeep, On Tuesday 28 April 2009, Paulraj, Sandeep wrote: > > > > I had been thinking that the davinci_nand driver should > > probably change to let board_nand_init() really become a > > board-specific function... > > > > It would call a driver routine to init the callback functions > > and

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread Paulraj, Sandeep
Dave, Please see inline. > -Original Message- > From: David Brownell [mailto:davi...@pacbell.net] > Sent: Tuesday, April 28, 2009 5:51 PM > To: Paulraj, Sandeep; u-boot@lists.denx.de > Cc: davinci-linux-open-sou...@linux.davincidsp.com > Subject: Re: [PATCH] ARM DaVinci: Adding CONFIG opt

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread Paulraj, Sandeep
Subject: Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE On Tue, Apr 28, 2009 at 04:33:14PM -0400, s-paul...@ti.com wrote: > diff --git a/include/configs/davinci_dvevm.h b/include/configs/davinci_dvevm.h > index fae430b..1fe3e19 100644 > --- a/includ

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread David Brownell
On Tuesday 28 April 2009, s-paul...@ti.com wrote: > The CLE and ALE for DaVinci DM644x is not the same as DM646x. This patch > adds 2 CONFIG_ options to the config.h header files to all the DM6446 based > boards. These values are then used by the driver. I had been thinking that the davinci_nand d

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread Steve Chen
On Tue, 2009-04-28 at 16:33 -0400, s-paul...@ti.com wrote: > The CLE and ALE for DaVinci DM644x is not the same as DM646x. This patch > adds 2 CONFIG_ options to the config.h header files to all the DM6446 based > boards. These values are then used by the driver. This has been tested on the > DM644

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread Scott Wood
On Tue, Apr 28, 2009 at 04:33:14PM -0400, s-paul...@ti.com wrote: > diff --git a/include/configs/davinci_dvevm.h b/include/configs/davinci_dvevm.h > index fae430b..1fe3e19 100644 > --- a/include/configs/davinci_dvevm.h > +++ b/include/configs/davinci_dvevm.h > @@ -128,6 +128,8 @@ > #define CONFIG_

Re: [U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread Paulraj, Sandeep
Steve, My intention was to change that when I add the DM6467 Support which I am working on even as we speak. I am aware of it. Thanks, Sandeep > -Original Message- > From: Steve Chen [mailto:sc...@mvista.com] > Sent: Tuesday, April 28, 2009 4:54 PM > To: Paulraj, Sandeep > Cc: u-b

[U-Boot] [PATCH] ARM DaVinci: Adding CONFIG options for NAND ALE and CLE

2009-04-28 Thread s-paulraj
The CLE and ALE for DaVinci DM644x is not the same as DM646x. This patch adds 2 CONFIG_ options to the config.h header files to all the DM6446 based boards. These values are then used by the driver. This has been tested on the DM644x, DM355, DM357 and DM365. Support for the latter 3 will be added s