[meta-fsl-arm][PATCH 1/4] amd-gpu-mx51.inc: Add the amd-gpu-mx51 provider

[meta-fsl-arm][PATCH 1/4] amd-gpu-mx51.inc: Add the amd-gpu-mx51 provider

[Otavio Salvador]

This allow for recipes to depend on the AMD GPU without enforcing the
backend to use.

Change-Id: I681603c76c95d647944cbe9115dadfbd508cd63d
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 recipes-graphics/amd-gpu-x11-bin/amd-gpu-mx51.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-graphics/amd-gpu-x11-bin/amd-gpu-mx51.inc b/recipes-graphics/amd-gpu-x11-bin/amd-gpu-mx51.inc
index 743892a..76c7bc4 100644
--- a/recipes-graphics/amd-gpu-x11-bin/amd-gpu-mx51.inc
+++ b/recipes-graphics/amd-gpu-x11-bin/amd-gpu-mx51.inc
@@ -8,7 +8,7 @@ SECTION = "libs"
 # FIXME: Replace for correct AMD license
 LIC_FILES_CHKSUM = "file://usr/include/VG/openvg.h;endline=30;md5=b0109611dd76961057d4c45ae6519802"
 
-PROVIDES = "virtual/egl virtual/libgles1 virtual/libgles2"
+PROVIDES = "amd-gpu-mx51 virtual/egl virtual/libgles1 virtual/libgles2"
 
 SRC_URI = "${FSL_MIRROR}/${PN}-${PV}.bin;fsl-eula=true \
            file://egl.pc \
-- 
2.1.4

[meta-fsl-arm][PATCH 2/4] libz160: Add a build-dependency on AMD GPU

[Otavio Salvador]

This fixes the following warning:

,----
| WARNING: QA Issue: libz160 requires libgsl-fsl.so.1, but no providers
|     in its RDEPENDS [file-rdeps]
`----

Change-Id: I920eb5f31f94118ed46105545b9d0bd175151690
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 recipes-graphics/amd-gpu-x11-bin/libz160_11.09.01.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/recipes-graphics/amd-gpu-x11-bin/libz160_11.09.01.bb b/recipes-graphics/amd-gpu-x11-bin/libz160_11.09.01.bb
index 1ff762a..43b9f7f 100644
--- a/recipes-graphics/amd-gpu-x11-bin/libz160_11.09.01.bb
+++ b/recipes-graphics/amd-gpu-x11-bin/libz160_11.09.01.bb
@@ -1,9 +1,12 @@
+# Copyright (C) 2011-2015 O.S. Systems Software LTDA.
 # Copyright (C) 2011, 2012 Freescale
 # Released under the MIT license (see COPYING.MIT for the terms)
 
 DESCRIPTION = "AMD libz160 gpu driver"
 LICENSE = "Proprietary"
 SECTION = "libs"
+DEPENDS = "amd-gpu-mx51"
+
 PR = "r2"
 
 #todo: Replace for correct AMD license
-- 
2.1.4

[meta-fsl-arm][PATCH 3/4] u-boot-fslc: Update to revision 6e66aed

[Otavio Salvador]

This includes the following backports from the 2015.04 upcoming release:

6e66aed imx:mx6 set normal APS and standby PFM mode
7a5cac0 pmic: fix missing SWITCH_SIZE
f98e73a pmic:pfuze implement pmic_mode_init
d79d747 mx53loco: Fix boot hang during reboot stress test
a5fb3c4 arm: mxs: Add 'Wait for JTAG user' if booted in JTAG mode
685ec23 arm: mxs: Enable booting of mx28 without battery
19161e5 arm: mxs: Add debug outputs and comments to mxs SPL source files
062e666 imx: mx6: Fixed AIPS3 base address issue
bc83af2 imx: imximage: Add QuadSPI boot support
364658f imx: mx6qsabreauto: Change to use common GPMI IO clock function
8c23f6f pmic:pfuz100 add switch mode and more registers
8385185 pmic:pfuze add macro for setting voltage
d8cd731 imx:mx6 update fuse_bank0_regs
d41380a fsl/ls1021qds: Add deep sleep support
8fe3f3b arm: ls102xa: Add LPUART support for LS1021ATWR board
3408c48 arm: ls102xa: Add LPUART support for LS1021AQDS board
e70b26f ls102xa: fdt: Disable QSPI and DSPI in NOR/NAND/SD boot
809e163 arm: ls102xa: Update snoop settings for CCI-400
2650c5d ARM: HYP/non-sec: Make variable gic_dist_addr as a local one
b2c8ac3 ls102xa: dcu: Add platform support for DCU on LS1021AQDS board
7f0f245 crypto/fsl: Add fixup for crypto node
8023778 arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL
79dcb3a ls102xa: etsec: Select ge0_clk125 for eTSEC clock muxing as default
636e863 arm/ls1021a: Add workaround for DDR erratum A008378

Change-Id: I91b013eb0d4a7741aa360f701e86f3dfdfccb6c8
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 recipes-bsp/u-boot/u-boot-fslc-mxsboot_2015.01.bb | 2 +-
 recipes-bsp/u-boot/u-boot-fslc_2015.01.bb         | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2015.01.bb b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2015.01.bb
index 4ffcb2a..4281cc2 100644
--- a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2015.01.bb
+++ b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2015.01.bb
@@ -7,7 +7,7 @@ PROVIDES = "u-boot-mxsboot"
 
 PV = "v2015.01+git${SRCPV}"
 
-SRCREV = "4dab988b9ca2344eb187e2e093c37203e1dfcb3d"
+SRCREV = "6e66aed373b038820664ca4630d3f5adc813d3e7"
 SRCBRANCH = "patches-2015.01"
 SRC_URI = "git://github.com/Freescale/u-boot-fslc.git;branch=${SRCBRANCH}"
 
diff --git a/recipes-bsp/u-boot/u-boot-fslc_2015.01.bb b/recipes-bsp/u-boot/u-boot-fslc_2015.01.bb
index 6c70348..010ab10 100644
--- a/recipes-bsp/u-boot/u-boot-fslc_2015.01.bb
+++ b/recipes-bsp/u-boot/u-boot-fslc_2015.01.bb
@@ -2,5 +2,5 @@ include u-boot-fslc.inc
 
 PV = "v2015.01+git${SRCPV}"
 
-SRCREV = "4dab988b9ca2344eb187e2e093c37203e1dfcb3d"
+SRCREV = "6e66aed373b038820664ca4630d3f5adc813d3e7"
 SRCBRANCH = "patches-2015.01"
-- 
2.1.4

[meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Otavio Salvador]

The use of TSLib, ICU and availability of examples is a distribution
configuration and not a BSP thing. Drop this settings from the
bbappend.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 qt5-layer/recipes-qt/qt5/qtbase_%.bbappend | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
index 64966c2..3152956 100644
--- a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
+++ b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
@@ -1,3 +1,4 @@
+# Copyright (C) 2013-2015 O.S. Systems Software LTDA.
 # Copyright (C) 2013 Eric Bénard - Eukréa Electromatique
 
 FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
@@ -10,17 +11,15 @@ IS_MX6SL = "0"
 IS_MX6SL_mx6sl = "1"
 
 PACKAGECONFIG_GL_mx5 = "gles2"
-PACKAGECONFIG_append_mx5 = " tslib icu examples"
 
 PACKAGECONFIG_GL_mx6q = "gles2"
 PACKAGECONFIG_GL_mx6dl = "gles2"
 PACKAGECONFIG_GL_mx6sl = "${@base_contains('DISTRO_FEATURES', 'x11', \
                             ' gl', '', d)}"
-PACKAGECONFIG_append_mx6 = " tslib icu examples"
 QT_CONFIG_FLAGS_append_mx6q = "${@base_contains('DISTRO_FEATURES', 'x11', ' -no-eglfs', ' -eglfs', d)}"
 QT_CONFIG_FLAGS_append_mx6dl = "${@base_contains('DISTRO_FEATURES', 'x11', ' -no-eglfs', ' -eglfs', d)}"
 QT_CONFIG_FLAGS_append_mx6sl = "${@base_contains('DISTRO_FEATURES', 'x11', \
-                            ' -no-eglfs', ' -no-opengl -linuxfb -no-icu -no-eglfs', d)}"
+                            ' -no-eglfs', ' -no-opengl -linuxfb -no-eglfs', d)}"
 
 do_configure_prepend_mx5() {
     sed -i 's!load(qt_config)!!' ${S}/mkspecs/linux-oe-g++/qmake.conf
-- 
2.1.4

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Eric Bénard]

Hi Otavio,

Le Tue, 17 Feb 2015 10:49:24 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :
> diff --git a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> index 64966c2..3152956 100644
> --- a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> +++ b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> @@ -1,3 +1,4 @@
> +# Copyright (C) 2013-2015 O.S. Systems Software LTDA.
>  # Copyright (C) 2013 Eric Bénard - Eukréa Electromatique
>  
A detail : in most projects, removing or adding a few lines to a file
is not usually considered as a way to add a copyright to this file else
you should also add the list of all the contributors in the header ;-)

Acked-by: Eric Bénard <eric@eukrea.com>

Best regards
Eric

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Otavio Salvador]

On Thu, Feb 19, 2015 at 6:32 AM, Eric Bénard <eric@eukrea.com> wrote:
> Le Tue, 17 Feb 2015 10:49:24 -0200,
> Otavio Salvador <otavio@ossystems.com.br> a écrit :
>> diff --git a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
>> index 64966c2..3152956 100644
>> --- a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
>> +++ b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
>> @@ -1,3 +1,4 @@
>> +# Copyright (C) 2013-2015 O.S. Systems Software LTDA.
>>  # Copyright (C) 2013 Eric Bénard - Eukréa Electromatique
>>
> A detail : in most projects, removing or adding a few lines to a file
> is not usually considered as a way to add a copyright to this file else
> you should also add the list of all the contributors in the header ;-)

I drop it when commiting. No worries.

> Acked-by: Eric Bénard <eric@eukrea.com>
>
> Best regards
> Eric



-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Alexandre Belloni]

Hi,

I just want to mention that removing icu will break any application
using qtwebkit (but not qtwebkit itself) when building with a distro
that does not add icu to the qtbase PACKAGECONFIG (like poky).

I'm not saying this shouldn't be done, I'm just warning that we may see
some reports of broken applications soon ;)


On 17/02/2015 at 10:49:24 -0200, Otavio Salvador wrote :
> The use of TSLib, ICU and availability of examples is a distribution
> configuration and not a BSP thing. Drop this settings from the
> bbappend.
> 
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  qt5-layer/recipes-qt/qt5/qtbase_%.bbappend | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> index 64966c2..3152956 100644
> --- a/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> +++ b/qt5-layer/recipes-qt/qt5/qtbase_%.bbappend
> @@ -1,3 +1,4 @@
> +# Copyright (C) 2013-2015 O.S. Systems Software LTDA.
>  # Copyright (C) 2013 Eric Bénard - Eukréa Electromatique
>  
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> @@ -10,17 +11,15 @@ IS_MX6SL = "0"
>  IS_MX6SL_mx6sl = "1"
>  
>  PACKAGECONFIG_GL_mx5 = "gles2"
> -PACKAGECONFIG_append_mx5 = " tslib icu examples"
>  
>  PACKAGECONFIG_GL_mx6q = "gles2"
>  PACKAGECONFIG_GL_mx6dl = "gles2"
>  PACKAGECONFIG_GL_mx6sl = "${@base_contains('DISTRO_FEATURES', 'x11', \
>                              ' gl', '', d)}"
> -PACKAGECONFIG_append_mx6 = " tslib icu examples"
>  QT_CONFIG_FLAGS_append_mx6q = "${@base_contains('DISTRO_FEATURES', 'x11', ' -no-eglfs', ' -eglfs', d)}"
>  QT_CONFIG_FLAGS_append_mx6dl = "${@base_contains('DISTRO_FEATURES', 'x11', ' -no-eglfs', ' -eglfs', d)}"
>  QT_CONFIG_FLAGS_append_mx6sl = "${@base_contains('DISTRO_FEATURES', 'x11', \
> -                            ' -no-eglfs', ' -no-opengl -linuxfb -no-icu -no-eglfs', d)}"
> +                            ' -no-eglfs', ' -no-opengl -linuxfb -no-eglfs', d)}"
>  
>  do_configure_prepend_mx5() {
>      sed -i 's!load(qt_config)!!' ${S}/mkspecs/linux-oe-g++/qmake.conf
> -- 
> 2.1.4
> 
> -- 
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Eric Bénard]

Hi Alexandre,

Le Thu, 19 Feb 2015 11:41:42 +0100,
Alexandre Belloni <alexandre.belloni@free-electrons.com> a écrit :
> I just want to mention that removing icu will break any application
> using qtwebkit (but not qtwebkit itself) when building with a distro
> that does not add icu to the qtbase PACKAGECONFIG (like poky).
> 
> I'm not saying this shouldn't be done, I'm just warning that we may see
> some reports of broken applications soon ;)
> 
sure, these options where here to facilitate general use case of qt
with i.MX : I agree with Otavio that's cleaner to let the choice to the
distro but of course that will break existing configurations which were
working thanks to these 2 lines (other example: those needing tslib
plugin) .

Best regards,
Eric

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Andreas Müller]

On Thu, Feb 19, 2015 at 11:41 AM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> wrote:
> Hi,
>
> I just want to mention that removing icu will break any application
> using qtwebkit (but not qtwebkit itself) when building with a distro
> that does not add icu to the qtbase PACKAGECONFIG (like poky).
>
> I'm not saying this shouldn't be done, I'm just warning that we may see
> some reports of broken applications soon ;)
>
I think we share same opinion here:

* This should not be the job of a BSP
* qtdeclarative fails to configure without icu and qtwebkit depends on
it. qtdeclarative requiring icu is mentioned in qtbase.inc

Andreas

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Otavio Salvador]

On Thu, Feb 19, 2015 at 9:03 AM, Andreas Müller
<schnitzeltony@googlemail.com> wrote:
> On Thu, Feb 19, 2015 at 11:41 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
>> I just want to mention that removing icu will break any application
>> using qtwebkit (but not qtwebkit itself) when building with a distro
>> that does not add icu to the qtbase PACKAGECONFIG (like poky).
>>
>> I'm not saying this shouldn't be done, I'm just warning that we may see
>> some reports of broken applications soon ;)
>>
> I think we share same opinion here:
>
> * This should not be the job of a BSP
> * qtdeclarative fails to configure without icu and qtwebkit depends on
> it. qtdeclarative requiring icu is mentioned in qtbase.inc

Agreed. So I think I will amend the commit log as follows:

qtbase: Remove distribution-related settings

The use of TSLib, ICU and availability of examples is a distribution
configuration and not a BSP thing. Drop this settings from the
bbappend.

WARNING
-------

   This breaks QtDeclarative and QtWebKit based application,
   as OE-Core and Poky distros does not add 'icu' to the
   DISTRO_FEATURES list.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>


-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Alexandre Belloni]

On 19/02/2015 at 12:03:16 +0100, Andreas Müller wrote :
> * qtdeclarative fails to configure without icu and qtwebkit depends on
> it. qtdeclarative requiring icu is mentioned in qtbase.inc

qtdeclarative doesn't actually need icu, that comment is wrong and I
sent a patch to correct it.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Otavio Salvador]

On Thu, Feb 19, 2015 at 9:54 AM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> wrote:
> On 19/02/2015 at 12:03:16 +0100, Andreas Müller wrote :
>> * qtdeclarative fails to configure without icu and qtwebkit depends on
>> it. qtdeclarative requiring icu is mentioned in qtbase.inc
>
> qtdeclarative doesn't actually need icu, that comment is wrong and I
> sent a patch to correct it.

So now it looks like:

qtbase: Remove distribution-related settings

The use of TSLib, ICU and availability of examples is a distribution
configuration and not a BSP thing. Drop this settings from the
bbappend.

WARNING
-------

   This breaks QtWebKit based applications, as OE-Core and Poky
   distros does not add 'icu' to the DISTRO_FEATURES list.

Change-Id: Ief633eab9473c3fddd359132ce7196e30b3ea293
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Eric Bénard]

Le Thu, 19 Feb 2015 14:01:09 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :

> On Thu, Feb 19, 2015 at 9:54 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
> > On 19/02/2015 at 12:03:16 +0100, Andreas Müller wrote :
> >> * qtdeclarative fails to configure without icu and qtwebkit depends on
> >> it. qtdeclarative requiring icu is mentioned in qtbase.inc
> >
> > qtdeclarative doesn't actually need icu, that comment is wrong and I
> > sent a patch to correct it.
> 
> So now it looks like:
> 
> qtbase: Remove distribution-related settings
> 
> The use of TSLib, ICU and availability of examples is a distribution
> configuration and not a BSP thing. Drop this settings from the
> bbappend.
> 
> WARNING
> -------
> 
>    This breaks QtWebKit based applications, as OE-Core and Poky
>    distros does not add 'icu' to the DISTRO_FEATURES list.
> 
> Change-Id: Ief633eab9473c3fddd359132ce7196e30b3ea293
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> 

Acked-by: Eric Bénard <eric@eukrea.com>

Re: [meta-fsl-arm][PATCH 4/4] qtbase: Remove distribution-related settings

[Alexandre Belloni]

On 19/02/2015 at 14:01:09 -0200, Otavio Salvador wrote :
> On Thu, Feb 19, 2015 at 9:54 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
> > On 19/02/2015 at 12:03:16 +0100, Andreas Müller wrote :
> >> * qtdeclarative fails to configure without icu and qtwebkit depends on
> >> it. qtdeclarative requiring icu is mentioned in qtbase.inc
> >
> > qtdeclarative doesn't actually need icu, that comment is wrong and I
> > sent a patch to correct it.
> 
> So now it looks like:
> 
> qtbase: Remove distribution-related settings
> 
> The use of TSLib, ICU and availability of examples is a distribution
> configuration and not a BSP thing. Drop this settings from the
> bbappend.
> 
> WARNING
> -------
> 
>    This breaks QtWebKit based applications, as OE-Core and Poky
>    distros does not add 'icu' to the DISTRO_FEATURES list.
> 
> Change-Id: Ief633eab9473c3fddd359132ce7196e30b3ea293
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> 

That seems ok to me


-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com