Patchwork meta-fsl-arm: master-next: fsl-image-gui build error

login
register
mail settings
Submitter Gonzalez, Alex
Date Nov. 6, 2013, 9:13 a.m.
Message ID <42B840CE3606D94AAA0E228998676BD802028042871F@dor-sms-exch01.digi.com>
Download mbox | patch
Permalink /patch/61189/
State Not Applicable
Delegated to: Otavio Salvador
Headers show

Comments

Gonzalez, Alex - Nov. 6, 2013, 9:13 a.m.
Hi,

I am testing on a imx6qsabresd with the following setup:

Build Configuration:
BB_VERSION        = "1.21.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "Ubuntu-12.04"
TARGET_SYS        = "arm-poky-linux-gnueabi"
MACHINE           = "imx6qsabresd"
DISTRO            = "poky"
DISTRO_VERSION    = "1.5+snapshot-20131105"
TUNE_FEATURES     = "armv7a vfp neon callconvention-hard cortexa9"
TARGET_FPU        = "vfp-neon"
meta              
meta-yocto        
meta-yocto-bsp    = "master:57498cede4696526c121e0112b5dea3e6e8365ac"
meta-oe           = "master:3cf6fd9961b290f6346be94a98b28a44508176e4"
meta-fsl-arm      = "master-next:96ab7a12bf9e10ba1a66bc46d9c0a3205093368f"
meta-fsl-demos    = "master:3aa554d5aacaab946013527e45ddbc3af8a25bba"

My aim is to test the accelerated graphics and video on a 3.10 kernel with the newest user land components available.

My first question is whether that's a supported build configuration or it's expected to work with the poky master-next branch too. There isn't a master-next branch for meta-oe, so I stuck with master in both.

I needed to make a couple of patches to make fsl-image-gui build. Not sure if anyone is working with the above setup, but here they go just in case. The first one just updates the bbappend recipe name for mesa, and the second one removes a patch for gst-fsl-plugin which apart from not applying does not seem to be needed any more as the package builds fine (apart from tons of warnings).

Is it worth to report problems with this setup or is it too bleeding edge to be worked on right now?

Regards,
Alex

recipes-graphics/mesa/mesa_9.1.6.bbappend          |   40 --------------------
 recipes-graphics/mesa/mesa_9.2.2.bbappend          |   40 ++++++++++++++++++++
 .../remove-GST_INFO-in-mfw_gst_utils.h.patch       |   40 --------------------
 .../gst-plugins/gst-fsl-plugin_3.0.9.bb            |    3 +-
 4 files changed, 41 insertions(+), 82 deletions(-)

From e72b0075d46d591864d9e340f50ebb8ebbe1d33d Mon Sep 17 00:00:00 2001
From: Alex Gonzalez <alex.gonzalez@digi.com>
Date: Tue, 5 Nov 2013 12:16:43 +0100
Subject: [PATCH 1/2] mesa: Update to 9.2.2

Signed-off-by: Alex Gonzalez <alex.gonzalez@digi.com>
---
 recipes-graphics/mesa/mesa_9.1.6.bbappend |   40 -----------------------------
 recipes-graphics/mesa/mesa_9.2.2.bbappend |   40 +++++++++++++++++++++++++++++
 2 files changed, 40 insertions(+), 40 deletions(-)
 delete mode 100644 recipes-graphics/mesa/mesa_9.1.6.bbappend
 create mode 100644 recipes-graphics/mesa/mesa_9.2.2.bbappend
Daiane Angolini - Nov. 6, 2013, 10:33 a.m.
On 06-11-2013 07:13, Gonzalez, Alex wrote:
> Hi,
>
> I am testing on a imx6qsabresd with the following setup:
>
> Build Configuration:
> BB_VERSION        = "1.21.0"
> BUILD_SYS         = "x86_64-linux"
> NATIVELSBSTRING   = "Ubuntu-12.04"
> TARGET_SYS        = "arm-poky-linux-gnueabi"
> MACHINE           = "imx6qsabresd"
> DISTRO            = "poky"
> DISTRO_VERSION    = "1.5+snapshot-20131105"
> TUNE_FEATURES     = "armv7a vfp neon callconvention-hard cortexa9"
> TARGET_FPU        = "vfp-neon"
> meta
> meta-yocto
> meta-yocto-bsp    = "master:57498cede4696526c121e0112b5dea3e6e8365ac"
> meta-oe           = "master:3cf6fd9961b290f6346be94a98b28a44508176e4"
> meta-fsl-arm      = "master-next:96ab7a12bf9e10ba1a66bc46d9c0a3205093368f"
> meta-fsl-demos    = "master:3aa554d5aacaab946013527e45ddbc3af8a25bba"
>
> My aim is to test the accelerated graphics and video on a 3.10 kernel with the newest user land components available.
>
> My first question is whether that's a supported build configuration or it's expected to work with the poky master-next branch too. There isn't a master-next branch for meta-oe, so I stuck with master in both.

When I test meta-fsl-arm master-next, I use poky/meta-oe on master.

>
> I needed to make a couple of patches to make fsl-image-gui build. Not sure if anyone is working with the above setup, but here they go just in case. The first one just updates the bbappend recipe name for mesa, and the second one removes a patch for gst-fsl-plugin which apart from not applying does not seem to be needed any more as the package builds fine (apart from tons of warnings).

Please, send your patches properly for review. (please, let me know if 
you don't know what is "properly")

>
> Is it worth to report problems with this setup or is it too bleeding edge to be worked on right now?

I'm not sure if I understood. But, please, master-next is supposed to be 
tested and any issue reported.

Because of that, it's not supposed to be used on your 
product/production, so, it's "too blending edge to be worked on".

Don't base your work on master-next, however, please, work with 
master-next in order to be sure it's good enough to be merged on master. =D


Daiane

Patch

diff --git a/recipes-graphics/mesa/mesa_9.1.6.bbappend b/recipes-graphics/mesa/mesa_9.1.6.bbappend
deleted file mode 100644
index 9975abc..0000000
--- a/recipes-graphics/mesa/mesa_9.1.6.bbappend
+++ /dev/null
@@ -1,40 +0,0 @@ 
-# FIXME: We may need to disable EGL, GL ES1 and GL ES2
-python __anonymous () {
-    import re
-
-    # SoC families to work on
-    families = ['mx5', 'mx6']
-    cur_soc_families = d.getVar('SOC_FAMILY', True)
-    if cur_soc_families and \
-        any(map(lambda x: x in cur_soc_families.split(':'), families)):
-        # Remove egl gles1 and gles2 configure options
-        extra_oeconf = d.getVar('EXTRA_OECONF', True).split()
-        take_out = ['--enable-egl', '--enable-gles1', '--enable-gles2']
-        put_in = ['--disable-egl', '--disable-gles1', '--disable-gles2']
-        pattern = re.compile("--with-egl-platforms")
-        new_extra_oeconf = []
-        for i in extra_oeconf:
-            if i not in take_out and not pattern.match(i):
-                new_extra_oeconf.append(i)
-        for i in put_in:
-            new_extra_oeconf.append(i)
-
-        d.setVar('EXTRA_OECONF', ' '.join(new_extra_oeconf))
-
-        # Remove itens from provides
-        provides = d.getVar('PROVIDES', True).split()
-        take_out = ['virtual/libgles1', 'virtual/libgles2', 'virtual/egl']
-        if 'mx6' in cur_soc_families.split(':'):
-            take_out.append('virtual/libgl')
-        new_provides = []
-        for i in provides:
-            if i not in take_out:
-                new_provides.append(i)
-
-        d.setVar('PROVIDES', ' '.join(new_provides))
-}
-
-# FIXME: Dirty hack to allow use of Vivante GPU libGL binary
-do_install_append_mx6 () {
-    rm -f ${D}${libdir}/libGL.*
-}
diff --git a/recipes-graphics/mesa/mesa_9.2.2.bbappend b/recipes-graphics/mesa/mesa_9.2.2.bbappend
new file mode 100644
index 0000000..9975abc
--- /dev/null

From 33e426a1ce27b8dd3481ca2e3c25a1d9ea4f13e9 Mon Sep 17 00:00:00 2001
From: Alex Gonzalez <alex.gonzalez@digi.com>
Date: Tue, 5 Nov 2013 12:17:26 +0100
Subject: [PATCH 2/2] gst-fsl-plugin: Remove GST_INFO patch.

Signed-off-by: Alex Gonzalez <alex.gonzalez@digi.com>
---
 .../remove-GST_INFO-in-mfw_gst_utils.h.patch       |   40 --------------------
 .../gst-plugins/gst-fsl-plugin_3.0.9.bb            |    3 +-
 2 files changed, 1 insertion(+), 42 deletions(-)
 delete mode 100644 recipes-multimedia/gst-plugins/gst-fsl-plugin/remove-GST_INFO-in-mfw_gst_utils.h.patch

diff --git a/recipes-multimedia/gst-plugins/gst-fsl-plugin/remove-GST_INFO-in-mfw_gst_utils.h.patch b/recipes-multimedia/gst-plugins/gst-fsl-plugin/remove-GST_INFO-in-mfw_gst_utils.h.patch
deleted file mode 100644
index c4dac2b..0000000
--- a/recipes-multimedia/gst-plugins/gst-fsl-plugin/remove-GST_INFO-in-mfw_gst_utils.h.patch
+++ /dev/null
@@ -1,40 +0,0 @@ 
-From 28ad25933dc51d8af13927a9943a353677c31a3a Mon Sep 17 00:00:00 2001
-From: Lyon Wang <lyon.wang@freescale.com>
-Date: Tue, 22 Oct 2013 16:15:43 +0800
-Subject: [PATCH] remove GST_INFO in mfw_gst_utils.h
-
-GST_INFO() should not be used in the global header
-or it will cause some macro definiation error.
-
-so comment the GST_INFO() in the mfw_gst_utils.h
-
-Upstream-Status: Backport [GIT]
----
- fsl_mm_linux/Multimedia/inc/plugin/mfw_gst_utils.h |    4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
-
-diff --git a/fsl_mm_linux/Multimedia/inc/plugin/mfw_gst_utils.h b/fsl_mm_linux/Multimedia/inc/plugin/mfw_gst_utils.h
-index 366d50e..c1b6f90 100755
---- a/fsl_mm_linux/Multimedia/inc/plugin/mfw_gst_utils.h
-+++ b/fsl_mm_linux/Multimedia/inc/plugin/mfw_gst_utils.h
-@@ -876,7 +876,7 @@ getChipCodeFromSocid (void)
-   }
-   fclose(fp);
- 
--  GST_INFO("SOC is %s\n", soc_name);
-+  //GST_INFO("SOC is %s\n", soc_name);
- 
-   int num = sizeof(soc_info) / sizeof(SOC_INFO);
-   int i;
-@@ -913,7 +913,7 @@ getChipCode (void)
-   }
- 
-   kv = ((kv_major << 16) + (kv_minor << 8) + kv_rel);
--  GST_INFO("kernel:%s, %d.%d.%d\n", sys_name.release, kv_major, kv_minor, kv_rel);
-+  //GST_INFO("kernel:%s, %d.%d.%d\n", sys_name.release, kv_major, kv_minor, kv_rel);
- 
-   if (kv < KERN_VER(3, 10, 0))
-     return getChipCodeFromCpuinfo();
--- 
-1.7.9.5
-
diff --git a/recipes-multimedia/gst-plugins/gst-fsl-plugin_3.0.9.bb b/recipes-multimedia/gst-plugins/gst-fsl-plugin_3.0.9.bb
index 4e26719..b95f576 100644
--- a/recipes-multimedia/gst-plugins/gst-fsl-plugin_3.0.9.bb
+++ b/recipes-multimedia/gst-plugins/gst-fsl-plugin_3.0.9.bb
@@ -5,8 +5,7 @@  require gst-fsl-plugin.inc
 
 SRC_URI[md5sum] = "a338a5c25225c765ae5f44af82fe7413"
 SRC_URI[sha256sum] = "e809daa0cd60381f05a79f34a53305a5ee53bfed22192a1cda6d998acc84a6fd"
-SRC_URI = "${FSL_MIRROR}/gst-fsl-plugins-3.10.9-1.0.0.tar.gz \
-           file://remove-GST_INFO-in-mfw_gst_utils.h.patch"
+SRC_URI = "${FSL_MIRROR}/gst-fsl-plugins-3.10.9-1.0.0.tar.gz"
 
 S = "${WORKDIR}/gst-fsl-plugins-3.10.9-1.0.0"