From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Sun, 19 Apr 2009 10:35:42 +0200
Subject: [U-Boot] [PATCH v2] Marvell MV88F6281GTW_GE Board support
In-Reply-To: <73173D32E9439E4ABB5151606C3E19E201C6B9BA3E@SC-VEXCH1.marvell.com>
References: <20090418073353.GB1413@game.jcrosoft.org>
	<73173D32E9439E4ABB5151606C3E19E201C6B9BA3E@SC-VEXCH1.marvell.com>
Message-ID: <20090419083542.GA19818@game.jcrosoft.org>
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 23:26 Sat 18 Apr     , Prafulla Wadaskar wrote:
> > > > > +
> > > > > +#endif /* CONFIG_MISC_INIT_R */
> > > > > diff --git a/board/Marvell/mv88f6281gtw_ge/u-boot.lds
> > > > > b/board/Marvell/mv88f6281gtw_ge/u-boot.lds
> > > > is it possible to have a shorter name for the board?
> > > No Jean, not possible, kernel patches also represents the 
> > same name and machine is also register with the same name, 
> > pleas bear with this, thanks..
> > ok if possible next time try a shorter name
> Sure... Except one more exception please "mv88f6192gtw_fe"
> This is next board whose kernel support and board name registry is already done in linux,
> this will be the next board support patch from me after getting this stuff in.
> For next upcomming boards I will keep in mind
ok
> 
> > > > > +#define LE			1	/* Specify 
> > LE/BE operation */
> > > > why?
> > > Because SOC can be initialized to work in both the modes.
> > so the binary will be compile as LE or BE so __ARMEL__ or 
> > __ARMEB__ will be defined 
> Okay I will change accordingly
> 
> > > > > +#define CONFIG_SYS_MALLOC_LEN	0x00400000	/* 4M */
> > > > 4M?
> > > What it should be?
> > just ask why do you need 4M of malloc?
> In future to support all SOC drivers, I will keep it 4M if you agree
ok

Best Regards,
J.