[U-Boot] [PATCH v2] Marvell MV88F6281GTW_GE Board support

[Jean-Christophe PLAGNIOL-VILLARD]

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.