From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Brownell <david-b@pacbell.net>
Date: Wed, 3 Jun 2009 14:12:29 -0700
Subject: [U-Boot] [PATCH] ARM DaVinci: EMIF settings
In-Reply-To: <20090531120347.GA10249@game.jcrosoft.org>
References: <49FDD85D.1020604@corelatus.se>
	<20090531120347.GA10249@game.jcrosoft.org>
Message-ID: <200906031412.29237.david-b@pacbell.net>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sunday 31 May 2009, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 19:46 Sun 03 May     , Thomas Lange wrote:
> > NAND module should not modify EMIF registers unrelated to CS2
> > that is used for NAND, i.e. do not modify EWAIT config register
> > or registers for other Chip Selects.
> > 
> > Without this patch, EMIF configurations made in board_init()
> > will be invalidated.
> > 
> > Signed-off-by: Thomas Lange <thomas@corelatus.se>
> > ---
> David, Sandeep
> is one of you can test it on some boards too?

I'd just merge it as-is (it builds, yes?).

That code is *extremely* dubious already, and the boards
using that driver only use NAND on CS2.  Code reconfiguring
other chipselects is broken.

That code is already dm6446-specific.  I have one dm6446
board, which only uses CS2 ... so the only error I could
turn up would be a build error.