From mboxrd@z Thu Jan  1 00:00:00 1970
From: Gabriel Huau <contact@huau-gabriel.fr>
Date: Sun, 29 Apr 2012 23:39:00 +0200
Subject: [U-Boot] [PATCH 0/2] Add support for MINI2440 (s3c2440)
In-Reply-To: <20120429213227.7A1F3202B38@gemini.denx.de>
References: <1335734845-27396-1-git-send-email-contact@huau-gabriel.fr>
	<20120429213227.7A1F3202B38@gemini.denx.de>
Message-ID: <20120429213859.GA20804@debian>
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 Sun, Apr 29, 2012 at 11:32:27PM +0200, Wolfgang Denk wrote:
> Dear Gabriel Huau,
> 
> In message <1335734845-27396-1-git-send-email-contact@huau-gabriel.fr> you wrote:
> >
> > >> +/*
> > >> + * When booting from NAND, it is impossible to access the lowest addresses
> > >> + * due to the SteppingStone being in the way. Luckily the NOR doesn't really
> > >> + * care about the highest 16 bits of address, so we set the controlers
> > >> + * registers to go and poke over there, instead.
> > >> + */
> > >> +#define PHYS_FLASH_1                 0x0
> > >> +#define CONFIG_SYS_FLASH_BASE        0x0
> > >
> > >Urghh... this sounds very much like a serious design issue?
> > 
> > About this point, I ported it from the old version uboot as well. It may need
> > some investigation, but I remember it was a big problem with this board. In the
> > case of a NAND boot, we don't have access to NOR because the SteppingStone
> > (SRAM) is mapped at the same range.
> 
> Should this not only affect the SPL part of the code, while you could
> use normal mappings for the real U-Boot.
> 
> Umm... but I don't even see any reference to SPL files here.  Is this
> current code supposed to support NAND booting?  and if so, why doesn;t
> it use SPL?
> 

Nope, laters. Currently, the code is only supporting boot from NOR.
Should I remove these defines ?

> Best regards,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> Genitiv ins Wasser, weil's Dativ ist!