From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eric Millbrandt <emillbrandt@coldhaus.com>
Date: Fri, 28 Aug 2009 07:02:27 -0500
Subject: [U-Boot] [PATCH] Add chip select region to galaxy5200
In-Reply-To: <fa686aa40908271646j1d994919j729336eafa16cd42@mail.gmail.com>
References: <20090827220505.M77733@coldhaus.com>
	<fa686aa40908271646j1d994919j729336eafa16cd42@mail.gmail.com>
Message-ID: <20090828120027.M71105@coldhaus.com>
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 Thu, 27 Aug 2009 17:46:03 -0600, Grant Likely wrote
> On Thu, Aug 27, 2009 at 4:11 PM, Eric
> Millbrandt<emillbrandt@coldhaus.com> wrote:
> > Add chip select region for an Epson S1D15313.
> >
> > Signed-off-by: Eric Millbrandt <emillbrandt@dekaresearch.com>
> > ---
> > ?include/configs/galaxy5200.h | ? ?5 +++++
> > ?1 files changed, 5 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/configs/galaxy5200.h b/include/configs/galaxy5200.h
> > index eb2cae6..54c6b80 100644
> > --- a/include/configs/galaxy5200.h
> > +++ b/include/configs/galaxy5200.h
> > @@ -212,6 +212,11 @@
> > ?#define CONFIG_SYS_CS1_SIZE ? ? ? ? ? ?0x90000
> > ?#define CONFIG_SYS_CS1_CFG ? ? ? ? ? ? 0x0002d900
> >
> > +/* Chip Select configuration for Epson S1D13513 */
> > +#define CONFIG_SYS_CS3_START ? ? ? ? ? 0x10000000
> > +#define CONFIG_SYS_CS3_SIZE ? ? ? ? ? ?0x103fffff
> > +#define CONFIG_SYS_CS3_CFG ? ? ? ? ? ? 0xffff3d10
> 
> This doesn't look right.  Shouldn't CONFIG_SYS_CS3_SIZE be 0x00400000?
> (Compare with SYS_CS1 above)
> 
Yes, you are correct.  I must have been thinking of the STOP location instead
of the size of the region.