From: Peter Tyser <ptyser@xes-inc.com>
Date: Mon, 01 Dec 2008 14:31:56 -0600
Subject: [U-Boot] [PATCH v3 2/2] XPedite5200 board support
In-Reply-To: <1228162644.24649.15.camel@ld0161-tx32>
References: <cover.1228160377.git.ptyser@xes-inc.com>
	<4490c75bb635d31ec187bc3b45d041a4bed49375.1228160377.git.ptyser@xes-inc.com>
	<7994b5f3c1c0438228e922a1620da8ae8ff7c29e.1228160377.git.ptyser@xes-inc.com>
	<1228162644.24649.15.camel@ld0161-tx32>
Message-ID: <1228163516.6206.42.camel@localhost.localdomain>
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 Mon, 2008-12-01 at 14:17 -0600, Jon Loeliger wrote:
> On Mon, 2008-12-01 at 13:47 -0600, Peter Tyser wrote:
> > Initial support for Extreme Engineering Solutions XPedite5200 -
> > a MPC8548-based PMC single board computer.
> > 
> > Signed-off-by: Peter Tyser <ptyser@xes-inc.com>
> > ---
> 
> > +/*
> > + * Use L1 as initial stack
> > + */
> > +#define CONFIG_SYS_INIT_RAM_LOCK	1
> 
> *sigh*
> 
> Shouldn't introduce this here either...
> > +#define CONFIG_SYS_INIT_RAM_ADDR	0xe0000000
> > +#define CONFIG_SYS_INIT_RAM_END		0x4000
> > +
> 
> Thanks,
> jdl

Which define are you referring to?  A quick glance looks like these 3
defines are used legitimately.  Am I missing something?

If a define should be removed, would it be possible to accept these
patches?  After they are accepted I'd be happy to submit another patch
to clean up the defines for all boards.  The boards I've used as
references have the similar defines which should also be cleaned up.

Thanks for the feedback,
Peter