[PATCH] runqemu: Add palmetto target

[Joel Stanley]

On Thu, Mar 17, 2016 at 3:22 PM, Andrew Jeffery <andrew at aj.id.au> wrote:
> The patch does a few things, but is more for inciting feedback than for
> merging:
>
> * Adds qemu_git.bb in favour of qemu_2.4.0.bb
> * Configured to build from master, at the commit merging the AST2400 support
> * Adds a runqemu target 'palmetto'

This is a good idea. Fix up the commit message so we can merge it :)

>
> diff --git a/yocto-poky/meta/recipes-devtools/qemu/qemu.inc b/yocto-poky/meta/recipes-devtools/qemu/qemu.inc
> index f6c0ae3..80ba2ca 100644
> --- a/yocto-poky/meta/recipes-devtools/qemu/qemu.inc
> +++ b/yocto-poky/meta/recipes-devtools/qemu/qemu.inc

yocto-poky is supposed to be the upstream repository, and shouldn't be
patched by us.

I don't know how poky lets us override built in recipies with those
from other meta- packages. Can we carry your qemu-git.bb in
meta-whatever instead?

Patrick, do you have any suggestions here?


> diff --git a/yocto-poky/scripts/runqemu b/yocto-poky/scripts/runqemu
> index 23cf5be..f09da50 100755
> --- a/yocto-poky/scripts/runqemu
> +++ b/yocto-poky/scripts/runqemu

The changes to this script look like a good start.

> @@ -110,7 +110,7 @@ while true; do
>      arg=${1}
>      case "$arg" in
>          "qemux86" | "qemux86-64" | "qemuarm" | "qemuarm64" | "qemumips" | "qemumipsel" | \
> -        "qemumips64" | "qemush4"  | "qemuppc" | "qemumicroblaze" | "qemuzynq")
> +        "qemumips64" | "qemush4"  | "qemuppc" | "qemumicroblaze" | "qemuzynq" | "palmetto")

Perhaps call it qemupalmetto? qemupalmbmc? to match the existing ones.

>              [ -z "$MACHINE" ] && MACHINE=$arg || \
>                  error "conflicting MACHINE types [$MACHINE] and [$arg]"
>              ;;
> @@ -236,13 +236,13 @@ fi
>