From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sat, 22 Sep 2012 10:51:51 +0200 Subject: [U-Boot] [PATCH] ARM: lacie_kw: add support for WIRELESS_SPACE In-Reply-To: <20120822151621.GP1419@kw.sim.vm.gnt> References: <1345140514-9048-1-git-send-email-albert.u.boot@aribaud.net> <20120817175538.GA3255@bill-the-cat> <20120817232833.1b5da97b@lilith> <20120822151621.GP1419@kw.sim.vm.gnt> Message-ID: <20120922105151.1ec0a062@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Simon, On Wed, 22 Aug 2012 17:16:21 +0200, Simon Guinot wrote: > On Tue, Aug 21, 2012 at 12:16:40AM -0700, Prafulla Wadaskar wrote: > > > > > > > -----Original Message----- > > > From: u-boot-bounces at lists.denx.de [mailto:u-boot- > > > bounces at lists.denx.de] On Behalf Of Albert ARIBAUD > > > Sent: 18 August 2012 02:59 > > > To: Tom Rini > > > Cc: u-boot at lists.denx.de > > > Subject: Re: [U-Boot] [PATCH] ARM: lacie_kw: add support for > > > WIRELESS_SPACE > > > > > > Hi Tom, > > > > > > On Fri, 17 Aug 2012 10:55:38 -0700, Tom Rini wrote: > > > > > > > On Thu, Aug 16, 2012 at 08:08:34PM +0200, Albert ARIBAUD wrote: > > > > > > > > > Signed-off-by: Albert ARIBAUD > > > > [snip] > > > > > @@ -90,8 +100,14 @@ > > > > > #define CONFIG_SYS_IDE_MAXDEVICE 1 > > > > > #if defined(CONFIG_NET2BIG_V2) > > > > > #define CONFIG_SYS_PROMPT "2big2> " > > > > > -#else > > > > > +#elif defined(CONFIG_NETSPACE_V2) > > > > > #define CONFIG_SYS_PROMPT "ns2> " > > > > > +#elif defined(CONFIG_INETSPACE_V2) > > > > > +#define CONFIG_SYS_PROMPT "is2> " > > > > > +#elif defined(CONFIG_WIRELESS_SPACE) > > > > > +#define CONFIG_SYS_PROMPT "ws> " > > > > > +#else > > > > > +#define CONFIG_SYS_PROMPT "lacie-kw> " > > > > > > > > Is this really a good thing? Over here I've been trying / > > > > hoping to get everyone to just use 'U-Boot # ' as the prompt as > > > > this makes automated testing easier and there's other run-time > > > > ways to see what we're on. > > > > I personally vote to have common boot prompt 'U-boot #'. > > > > > > > > I quite like prompts that identify the board I'm on, but it's not > > > a must either. Cc:ing Simon in cas his own patches re lacie_kw > > > would be impacted by your suggestion. > > > > Well, we can expose a command to detect/print the board info, if > > needed. > > The run-time command "version" already returns both the system and the > machine informations. This command can be used for testing purposes. > IMHO the only requirement for an U-Boot prompt is to be short enough. > Everything else is a matter of taste. Mine goes for the machine name. > > Regards, > > Simon Tom, Can automated test cope with a general prompt of the form '.*#' ? Amicalement, -- Albert.