On 16/11/09 18:02, Paulraj, Sandeep wrote:
> Hi Nick,
> 
>> -----Original Message-----
>> From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de]
>> On Behalf Of Nick Thompson
>> Sent: Monday, November 16, 2009 12:50 PM
>> To: u-boot@lists.denx.de
>> Subject: [U-Boot] [PATCH] Davinci: Configurable NAND chip selects
>>
>> Davinci: Configurable NAND chip selects
>>
>> Add a CONFIG_SYS_NAND_CS setting to all davinci configs and
>> use it to setup the NAND controller in the davinci_nand
>> mtd driver.
>>
>> Signed-off-by: Nick Thompson <nick.thomp...@gefanuc.com>
>> ---
>> This config item is already used on many other platforms
>> so is not a new config item as such.
>>
>> This change should not result in any functional changes to
>> current davinci boards. They all seem to use CS2 as this
>> was the only chip select supported by the driver. da830evm
>> however uses CS3, so a configurable driver is required.
> 
> My testing branch has all the NAND updates. I'll add this and test it out. I 
> don't expect to see any issues though.

That's great, thank you.

>>
>> Applies to: u-boot-ti
>>
>>  drivers/mtd/nand/davinci_nand.c          |   16 ++++++++++------
>>  include/asm-arm/arch-davinci/emif_defs.h |    9 +++++++++
>>  include/configs/davinci_dm355evm.h       |    1 +
>>  include/configs/davinci_dm355leopard.h   |    1 +
>>  include/configs/davinci_dm365evm.h       |    1 +
>>  include/configs/davinci_dm6467evm.h      |    1 +
>>  include/configs/davinci_dvevm.h          |    1 +
>>  include/configs/davinci_schmoogie.h      |    1 +
>>  include/configs/davinci_sffsdr.h         |    1 +
>>  include/configs/davinci_sonata.h         |    1 +
>>  10 files changed, 27 insertions(+), 6 deletions(-)
> OK but I don't see a change to the DA830 config which was the reason for the 
> patch in the first place.

This config entry is already in da830evm. It was inherited from the
1.3.3 port of U-Boot to da830.

There will in any case, be a separate patch for NAND on da830, since
there are several defines that need to be added for the current code
base.

Thanks,
Nick.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to