[U-Boot] [PATCH 2/2] vexpress: Add vexpress qemu variant

[Ian Campbell]

On Mon, 2013-12-16 at 17:08 +0100, Albert ARIBAUD wrote:
> Hi Ian,
> 
> On Sun, 17 Nov 2013 15:17:43 +0000, Ian Campbell <ijc at hellion.org.uk>
> wrote:
> 
> > The main purpose of this is to disable the use of the virtualisation extensions
> > which Qemu does not emulate.
> > 
> > Also enable a few additional commands and the u-boot API.
> > 
> > This works for me with:
> > $ dd if=/dev/zero of=pflash0.img bs=1M count=64
> > $ dd if=/dev/zero of=pflash1.img bs=1M count=64
> > $ qemu-system-arm -M vexpress-a15 -cpu cortex-a15 -kernel u-boot -m 1024m -sd mmc.img -nographic -pflash pflash0.img -pflash pflash1.img
> > 
> > (for some reason both pflash images are needed for saveenv to work)
> 
> Please split the above as follows:
> 
> - a short problem and solution description in the commit message,
>   above the commit message delimiter '---';
> 
> - anything else (examples, comments, etc) below the commit delimiter.

I was intentionally including the qemu command line in the main body of
the commit message since I was adding a target specifically for use with
qemu. But this has now become a bit moot (see below).

> > +#define CONFIG_CMD_ECHO
> 
> Here and below, you are adding commands generally, i.e. not specifically
> for the new boards you're creating. I'd rather have this in a separate
> patch.

Sorry, I should have split this out in the first place.

But actually in the meantime I've discovered that the vexpress-a9
version of uboot works just fine on qemu -M vexpress-a9 (I guess I'm
just used to working with a15's so it didn't initially occur to me!)

With that in mind here is the patch which just adds the extra commands.

8<--------------