[meta-freescale] [meta-fsl-arm][PATCH v3 08/13] qemu: upgrade to v2.2.0 plus fsl patches

[Liu Ting]

Hello Daiane,

Thank you very much for the kindly comments. Yes, we have a lot to improve.

Next, we will try to get help from development team to summarize the detailed info (what changed/fixed, etc.). But this may need some time as the pkgs for qoriq are developed by different teams.

The process is that, development team maintain their own git repositories. Once codes are ready for release, Yocto team updates the corresponding branch/commit in recipes. 

It is a bit hard to summarize the development history which lasts about 6 months.

-Ting

> -----Original Message-----
> From: angolini at gmail.com [mailto:angolini at gmail.com] On Behalf Of Daiane
> Angolini
> Sent: Tuesday, July 07, 2015 9:33 PM
> To: Liu Ting-B28495
> Cc: meta-freescale at yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH v3 08/13] qemu: upgrade to
> v2.2.0 plus fsl patches
> 
> On Tue, Jul 7, 2015 at 5:16 AM,  <b28495 at freescale.com> wrote:
> > From: Ting Liu <ting.liu at freescale.com>
> >
> > Also fix indent.
> 
> Maybe:
> 
> "Update the supported QEMU version from v1.7 to v2.2.0 and include the
> Freescale patches on top of this.
> 
> In addition to the Freescale patches from SDK1.7, this bump also include:
> * feature 1
> * bugfix of bugx
> 
> Also fix indent.
> "
> 
> Or, maybe:
> 
> "Update the suported QEMU version from v1.7 to v2.2.0 and include the
> Freescale patches on top of this. The patches from Freescale remains the same
> from past release.
> 
> Also fix indent.
> "
> 
> Commit log is not just a boring text, it is the documentation of the code. And
> more than that, you must include the reason why a user would want to get the
> update. Why should I care about this update?
> 
> One of the most frequent questions on my support job is exactly related with
> CHANGELOG, and most of times I cannot rely on commit log because there is no
> commit log.
> 
> Daiane
> 
> >
> > Signed-off-by: Ting Liu <ting.liu at freescale.com>
> > ---
> >  recipes-devtools/qemu/qemu-fsl_git.bb | 34
> > +++++++++++++++-------------------
> >  1 file changed, 15 insertions(+), 19 deletions(-)
> >
> > diff --git a/recipes-devtools/qemu/qemu-fsl_git.bb
> > b/recipes-devtools/qemu/qemu-fsl_git.bb
> > index ca36c3d..94fb9b1 100644
> > --- a/recipes-devtools/qemu/qemu-fsl_git.bb
> > +++ b/recipes-devtools/qemu/qemu-fsl_git.bb
> > @@ -3,20 +3,17 @@ require recipes-devtools/qemu/qemu.inc
> > LIC_FILES_CHKSUM = "file://COPYING;md5=441c28d2cf86e15a37fa47e15a72fbac \
> >
> file://COPYING.LIB;endline=24;md5=c04def7ae38850e7d3ef548588159913"
> >
> > -# This means QEMU v1.7 with FSL specific patches applied -PV =
> > "1.7+${SRCPV}"
> > +# This means QEMU v2.2.0 with FSL specific patches applied PV =
> > +"2.2.0+${SRCPV}"
> >
> > -# NOTE: these options are note available in qemu 1.7, but qemu.inc
> > assumes -# version 2.0+ where they are available. For now we unset
> > them, but we should -# remove the following lines when upgrading to qemu
> 2.0+:
> > -PACKAGECONFIG[quorum] = ""
> > -PACKAGECONFIG[lzo]    = ""
> > -PACKAGECONFIG[numa]   = ""
> > -PACKAGECONFIG[gtk+]   = ""
> > +# NOTE: this recipe requires poky's qemu.inc which assumes version
> > +2.3 # where glx enable config option changed to --enable-opengl. For
> > +now we # restore it, but we should remove the following lines when
> > +upgrading # to qemu 2.3:
> > +PACKAGECONFIG[glx] = "--enable-glx,--disable-glx,mesa"
> >
> > -SRC_URI = "git://git.freescale.com/ppc/sdk/qemu.git;branch=sdk-v1.7.x
> > \ -"
> > -SRCREV = "6ac4597c059d35e2737b234747243e56d340f4db"
> > +SRC_URI = "git://git.freescale.com/ppc/sdk/qemu.git;branch=master"
> > +SRCREV = "00ac004143e9fe46944a1885b04268fcd3a95a3a"
> >
> >  S = "${WORKDIR}/git"
> >
> > @@ -26,23 +23,22 @@ inherit pkgconfig
> >
> >  # Append build host pkg-config paths for native target since the host
> > may provide sdl
> >  do_configure_prepend() {
> > -        export PKG_CONFIG=${STAGING_DIR_NATIVE}${bindir_native}/pkg-config
> > +    export
> > + PKG_CONFIG=${STAGING_DIR_NATIVE}${bindir_native}/pkg-config
> >  }
> >
> >  do_configure_append () {
> > -   if ! grep 'CONFIG_FDT=y' config-host.mak; then
> > -        echo "CONFIG_RDMA=y" >> config_host_mak
> > -   fi
> > +    if ! grep 'CONFIG_FDT=y' config-host.mak; then
> > +         echo "CONFIG_RDMA=y" >> config_host_mak
> > +    fi
> >  }
> >
> >  # gets around qemu.inc trying to install powerpc_rom.bin
> >  do_install_prepend() {
> > -       touch ${WORKDIR}/powerpc_rom.bin
> > +    touch ${WORKDIR}/powerpc_rom.bin
> >  }
> >
> >  do_install_append() {
> > -       rm ${WORKDIR}/powerpc_rom.bin
> > -
> > +    rm ${WORKDIR}/powerpc_rom.bin
> >      # Prevent QA warnings about installed ${localstatedir}/run
> >      if [ -d ${D}${localstatedir}/run ]; then rmdir
> > ${D}${localstatedir}/run; fi  }
> > --
> > 1.9.1
> >
> > --
> > _______________________________________________
> > meta-freescale mailing list
> > meta-freescale at yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-freescale