[U-Boot] [PATCH] Davinci: Configurable NAND chip selects

[Nick Thompson]

On 16/11/09 18:02, Paulraj, Sandeep wrote:
> Hi Nick,
> 
>> -----Original Message-----
>> From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
>> On Behalf Of Nick Thompson
>> Sent: Monday, November 16, 2009 12:50 PM
>> To: u-boot at 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.thompson at 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.