From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= Date: Wed, 29 May 2013 16:49:15 +0200 (CEST) Subject: [U-Boot] [PATCH v4 7/7] arm: vf610: Add basic support for Vybrid VF610TWR board In-Reply-To: <81BA6E5E0BC2344391CABCEE22D1B6D83400A9@039-SN1MPN1-003.039d.mgd.msft.net> References: <1369731347-9994-1-git-send-email-b18965@freescale.com> <1369731347-9994-8-git-send-email-b18965@freescale.com> <1122476427.1272253.1369771054188.JavaMail.root@advansee.com> <81BA6E5E0BC2344391CABCEE22D1B6D83400A9@039-SN1MPN1-003.039d.mgd.msft.net> Message-ID: <1753274237.1323710.1369838955219.JavaMail.root@advansee.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Alison, On Wednesday, May 29, 2013 7:52:33 AM, Wang Huan-B18965 wrote: > 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. OK, then, for this patch: Reviewed-by: Beno?t Th?baudeau Best regards, Beno?t