[U-Boot] [PATCH v4 7/7] arm: vf610: Add basic support for Vybrid VF610TWR board

[Wang Huan-B18965]

Hi, Benoit,

> > +
> > +#define CONFIG_CMD_PING
> > +#define CONFIG_CMD_DHCP
> > +#define CONFIG_CMD_MII
> > +#define CONFIG_CMD_NET
> > +#define CONFIG_FEC_MXC
> > +#define CONFIG_MII
> > +#define IMX_FEC_BASE			ENET_BASE_ADDR
> > +#define CONFIG_FEC_XCV_TYPE		RMII
> > +#define CONFIG_FEC_MXC_PHYADDR          0
> 
> Why don't you add support for the 2nd FEC? Do you plan to do it later?
[Alison Wang] In u-boot, one FEC is enough for user. We do not plan to do it later.
> 
> > +#define CONFIG_PHYLIB
> > +#define CONFIG_PHY_MICREL
> > +
> > +#define CONFIG_BOOTDELAY		3
> > +
> > +#define CONFIG_SYS_TEXT_BASE		0x3f008000
> > +
> > +/* Miscellaneous configurable options */
> > +#define CONFIG_SYS_LONGHELP		/* undef to save memory */
> > +#define CONFIG_SYS_HUSH_PARSER		/* use "hush" command parser
> */
> > +#define CONFIG_SYS_PROMPT_HUSH_PS2	"> "
> > +#define CONFIG_SYS_PROMPT		"Vybrid U-Boot > "
> > +#undef CONFIG_AUTO_COMPLETE
> > +#define CONFIG_SYS_CBSIZE		256	/* Console I/O Buffer Size */
> > +#define CONFIG_SYS_PBSIZE		\
> > +			(CONFIG_SYS_CBSIZE + sizeof(CONFIG_SYS_PROMPT) + 16)
> > +#define CONFIG_SYS_MAXARGS		16	/* max number of command args
> */
> > +#define CONFIG_SYS_BARGSIZE		CONFIG_SYS_CBSIZE
> > +
> > +#define CONFIG_CMD_MEMTEST
> > +#define CONFIG_SYS_MEMTEST_START	0x80010000
> > +#define CONFIG_SYS_MEMTEST_END		0x87C00000
> 
> Please make sure to have runtime-tested this address range with the
> mtest command since bad mtest addresses are the reason why
> CONFIG_CMD_MEMTEST has been removed from the default commands.
[Alison Wang] Thanks for your reminder, we have tested.
> 

Best Regards,
Alison Wang