All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-fsl-arm][PATCH 1/4] weston: Remove "Desktop-shell-Don-t-assume-there-is-a-pointer.patch"
@ 2014-11-24 20:39 Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 2/4] gstreamer1.0-plugins-bad: Remove useless overrides Otavio Salvador
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Otavio Salvador @ 2014-11-24 20:39 UTC (permalink / raw)
  To: meta-freescale Mailing List; +Cc: Otavio Salvador

This patch is applied upstream and can be removed now.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 ...top-shell-Don-t-assume-there-is-a-pointer.patch | 36 ----------------------
 recipes-graphics/wayland/weston_%.bbappend         |  1 -
 2 files changed, 37 deletions(-)
 delete mode 100644 recipes-graphics/wayland/weston/0004-Desktop-shell-Don-t-assume-there-is-a-pointer.patch

diff --git a/recipes-graphics/wayland/weston/0004-Desktop-shell-Don-t-assume-there-is-a-pointer.patch b/recipes-graphics/wayland/weston/0004-Desktop-shell-Don-t-assume-there-is-a-pointer.patch
deleted file mode 100644
index f6ef253..0000000
--- a/recipes-graphics/wayland/weston/0004-Desktop-shell-Don-t-assume-there-is-a-pointer.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-From 1194ccebc0db14ba681bd8272e01715b3e7008cc Mon Sep 17 00:00:00 2001
-From: Yong Gan <b45748@freescale.com>
-Date: Fri, 27 Jun 2014 11:33:53 +0800
-Subject: [PATCH] ENGR00320189 desktop-shell: Don't assume there is a pointer
- when resizing
-
-Fixes a crash on touch devices without a pointer, when touching
-the window frame of a client.
-cherry-pick from 5872b6821be7c9c0a17a0de962e3f4a04f5d0c6f
-
-Date: Jun 27, 2014
-
-Upstream Status: N/A
-
-Signed-off-by Yong Gan <B45748@freescale.com>
----
- desktop-shell/shell.c |    3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c
-index 84f5c83..d965618 100644
---- a/desktop-shell/shell.c
-+++ b/desktop-shell/shell.c
-@@ -1784,7 +1784,8 @@ common_surface_resize(struct wl_resource *resource,
- 	struct shell_surface *shsurf = wl_resource_get_user_data(resource);
- 	struct weston_surface *surface;
- 
--	if (seat->pointer->button_count == 0 ||
-+	if (seat->pointer == NULL ||
-+	    seat->pointer->button_count == 0 ||
- 	    seat->pointer->grab_serial != serial ||
- 	    seat->pointer->focus == NULL)
- 		return;
--- 
-1.7.9.5
-
diff --git a/recipes-graphics/wayland/weston_%.bbappend b/recipes-graphics/wayland/weston_%.bbappend
index 7b50283..01773d2 100644
--- a/recipes-graphics/wayland/weston_%.bbappend
+++ b/recipes-graphics/wayland/weston_%.bbappend
@@ -3,7 +3,6 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 SRC_URI_append_mx6 = " file://0001-ENGR00314805-1-Add-Vivante-EGL-support.patch \
              file://0002-ENGR00314805-2-Add-Vivante-GAL2D-support.patch \
              file://0003-Distorted-line-and-shadow-if-use-2d-com.patch \
-             file://0004-Desktop-shell-Don-t-assume-there-is-a-pointer.patch \
              file://0005-Enable-GAL2D-compositor-in-SoloLite.patch \
              file://0006-Change-GAL2D-compositor-to-be-default-i.patch \
              "
-- 
2.1.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [meta-fsl-arm][PATCH 2/4] gstreamer1.0-plugins-bad: Remove useless overrides
  2014-11-24 20:39 [meta-fsl-arm][PATCH 1/4] weston: Remove "Desktop-shell-Don-t-assume-there-is-a-pointer.patch" Otavio Salvador
@ 2014-11-24 20:39 ` Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl Otavio Salvador
  2 siblings, 0 replies; 9+ messages in thread
From: Otavio Salvador @ 2014-11-24 20:39 UTC (permalink / raw)
  To: meta-freescale Mailing List; +Cc: Otavio Salvador

The PACKAGECONFIG_GL has the same value as the provided
by the recipe, by default. This removes this duplication.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
index a3f1f19..6d62b7e 100644
--- a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
+++ b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
@@ -6,9 +6,6 @@ CFLAGS_append_mx6 = " -DLINUX \
                          base_contains('DISTRO_FEATURES', 'directfb', '-DEGL_API_DFB', \
                          '-DEGL_API_FB', d),d),d)}"
 
-PACKAGECONFIG_GL_mx6q = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'gles2', '', d)}"
-PACKAGECONFIG_GL_mx6dl = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'gles2', '', d)}"
-PACKAGECONFIG_GL_mx6sx = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'gles2', '', d)}"
 PACKAGECONFIG_GL_mx6sl = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', \
                            base_contains('DISTRO_FEATURES', 'x11', \
                                     'opengl', '', d), '', d)}"
-- 
2.1.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL
  2014-11-24 20:39 [meta-fsl-arm][PATCH 1/4] weston: Remove "Desktop-shell-Don-t-assume-there-is-a-pointer.patch" Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 2/4] gstreamer1.0-plugins-bad: Remove useless overrides Otavio Salvador
@ 2014-11-24 20:39 ` Otavio Salvador
  2014-11-25 11:36   ` Daiane Angolini
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl Otavio Salvador
  2 siblings, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2014-11-24 20:39 UTC (permalink / raw)
  To: meta-freescale Mailing List; +Cc: Otavio Salvador

libfslvpuwrap is only available for i.MX6Q and i.MX6DL. This makes
gstreamer1.0-plugins-imx unusable on other i.MX6 SoC familes and this
commit updates the default setting accordingly.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 conf/machine/include/imx-base.inc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/conf/machine/include/imx-base.inc b/conf/machine/include/imx-base.inc
index 362d3cb..83476bc 100644
--- a/conf/machine/include/imx-base.inc
+++ b/conf/machine/include/imx-base.inc
@@ -81,7 +81,8 @@ MACHINE_GSTREAMER_PLUGIN_vf60 ?= ""
 
 # GStreamer 1.0 plugins
 MACHINE_GSTREAMER_1_0_PLUGIN ?= ""
-MACHINE_GSTREAMER_1_0_PLUGIN_mx6 ?= "gstreamer1.0-plugins-imx-meta"
+MACHINE_GSTREAMER_1_0_PLUGIN_mx6q ?= "gstreamer1.0-plugins-imx-meta"
+MACHINE_GSTREAMER_1_0_PLUGIN_mx6dl ?= "gstreamer1.0-plugins-imx-meta"
 
 # mx5 GPU libraries
 GPU_FLAVOUR = "${@base_contains('DISTRO_FEATURES', 'x11', 'x11-', '', d)}"
-- 
2.1.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl
  2014-11-24 20:39 [meta-fsl-arm][PATCH 1/4] weston: Remove "Desktop-shell-Don-t-assume-there-is-a-pointer.patch" Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 2/4] gstreamer1.0-plugins-bad: Remove useless overrides Otavio Salvador
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL Otavio Salvador
@ 2014-11-24 20:39 ` Otavio Salvador
  2014-11-25 11:23   ` Daiane Angolini
  2 siblings, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2014-11-24 20:39 UTC (permalink / raw)
  To: meta-freescale Mailing List; +Cc: Otavio Salvador

The build system can also use egl when GL is not avail so we ought to
add this to the dependency chain as gpu-viv-bin-mx6q would not be
included otherwise, for i.MX6SL.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 recipes-graphics/piglit/piglit_git.bbappend | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/recipes-graphics/piglit/piglit_git.bbappend b/recipes-graphics/piglit/piglit_git.bbappend
index f5e16b3..316bec9 100644
--- a/recipes-graphics/piglit/piglit_git.bbappend
+++ b/recipes-graphics/piglit/piglit_git.bbappend
@@ -1,3 +1,5 @@
+DEPENDS_append_mx6 = " virtual/egl"
+
 # Provide the need flags and backend settings to work with Vivante GPU
 CFLAGS_append_mx6 = " -DLINUX \
                       ${@base_contains('DISTRO_FEATURES', 'x11', '-DEGL_API_FB', \
-- 
2.1.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl Otavio Salvador
@ 2014-11-25 11:23   ` Daiane Angolini
  2014-11-25 11:37     ` Otavio Salvador
  0 siblings, 1 reply; 9+ messages in thread
From: Daiane Angolini @ 2014-11-25 11:23 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale Mailing List

On Mon, Nov 24, 2014 at 6:39 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The build system can also use egl when GL is not avail so we ought to
> add this to the dependency chain as gpu-viv-bin-mx6q would not be
> included otherwise, for i.MX6SL.

In this case, would it be applicable only for imx6SL instead of all imx6?


Daiane
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  recipes-graphics/piglit/piglit_git.bbappend | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/recipes-graphics/piglit/piglit_git.bbappend b/recipes-graphics/piglit/piglit_git.bbappend
> index f5e16b3..316bec9 100644
> --- a/recipes-graphics/piglit/piglit_git.bbappend
> +++ b/recipes-graphics/piglit/piglit_git.bbappend
> @@ -1,3 +1,5 @@
> +DEPENDS_append_mx6 = " virtual/egl"
> +
>  # Provide the need flags and backend settings to work with Vivante GPU
>  CFLAGS_append_mx6 = " -DLINUX \
>                        ${@base_contains('DISTRO_FEATURES', 'x11', '-DEGL_API_FB', \
> --
> 2.1.1
>
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL
  2014-11-24 20:39 ` [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL Otavio Salvador
@ 2014-11-25 11:36   ` Daiane Angolini
  2014-11-25 11:40     ` Otavio Salvador
  0 siblings, 1 reply; 9+ messages in thread
From: Daiane Angolini @ 2014-11-25 11:36 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale Mailing List

On Mon, Nov 24, 2014 at 6:39 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> libfslvpuwrap is only available for i.MX6Q and i.MX6DL. This makes
> gstreamer1.0-plugins-imx unusable on other i.MX6 SoC familes and this
> commit updates the default setting accordingly.

What causes the need of segmentation for imx6Q and imx6DL is not the
"libfslvpuwrap".

In fact, the gstreamer1.0-plugins-imx is not possible to be used with
imx6SX and imx6SL becase it does not support a non vpu chip.

So I propose a new commit log:


"Gstreamer1.0-plugins-imx can only be used with iMX6Q and iMX6DL
(instead of any i.MX6) and this commit updates the default setting
accordingly."

Daiane


>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  conf/machine/include/imx-base.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/conf/machine/include/imx-base.inc b/conf/machine/include/imx-base.inc
> index 362d3cb..83476bc 100644
> --- a/conf/machine/include/imx-base.inc
> +++ b/conf/machine/include/imx-base.inc
> @@ -81,7 +81,8 @@ MACHINE_GSTREAMER_PLUGIN_vf60 ?= ""
>
>  # GStreamer 1.0 plugins
>  MACHINE_GSTREAMER_1_0_PLUGIN ?= ""
> -MACHINE_GSTREAMER_1_0_PLUGIN_mx6 ?= "gstreamer1.0-plugins-imx-meta"
> +MACHINE_GSTREAMER_1_0_PLUGIN_mx6q ?= "gstreamer1.0-plugins-imx-meta"
> +MACHINE_GSTREAMER_1_0_PLUGIN_mx6dl ?= "gstreamer1.0-plugins-imx-meta"
>
>  # mx5 GPU libraries
>  GPU_FLAVOUR = "${@base_contains('DISTRO_FEATURES', 'x11', 'x11-', '', d)}"
> --
> 2.1.1
>
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl
  2014-11-25 11:23   ` Daiane Angolini
@ 2014-11-25 11:37     ` Otavio Salvador
  2014-11-25 12:21       ` Daiane Angolini
  0 siblings, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2014-11-25 11:37 UTC (permalink / raw)
  To: Daiane Angolini; +Cc: meta-freescale Mailing List

On Tue, Nov 25, 2014 at 9:23 AM, Daiane Angolini <daiane.list@gmail.com> wrote:
> On Mon, Nov 24, 2014 at 6:39 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The build system can also use egl when GL is not avail so we ought to
>> add this to the dependency chain as gpu-viv-bin-mx6q would not be
>> included otherwise, for i.MX6SL.
>
> In this case, would it be applicable only for imx6SL instead of all imx6?

Not necessary; the other cases are catch due virtual/libgl which SL
does not has.

-- 
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


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL
  2014-11-25 11:36   ` Daiane Angolini
@ 2014-11-25 11:40     ` Otavio Salvador
  0 siblings, 0 replies; 9+ messages in thread
From: Otavio Salvador @ 2014-11-25 11:40 UTC (permalink / raw)
  To: Daiane Angolini; +Cc: meta-freescale Mailing List

On Tue, Nov 25, 2014 at 9:36 AM, Daiane Angolini <daiane.list@gmail.com> wrote:
> On Mon, Nov 24, 2014 at 6:39 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> libfslvpuwrap is only available for i.MX6Q and i.MX6DL. This makes
>> gstreamer1.0-plugins-imx unusable on other i.MX6 SoC familes and this
>> commit updates the default setting accordingly.
>
> What causes the need of segmentation for imx6Q and imx6DL is not the
> "libfslvpuwrap".
>
> In fact, the gstreamer1.0-plugins-imx is not possible to be used with
> imx6SX and imx6SL becase it does not support a non vpu chip.
>
> So I propose a new commit log:
>
>
> "Gstreamer1.0-plugins-imx can only be used with iMX6Q and iMX6DL
> (instead of any i.MX6) and this commit updates the default setting
> accordingly."

I changed it locally to:

---
imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL

gstreamer1.0-plugins-imx can only be used with i.MX6Q and i.MX6DL
(instead of any i.MX6) as the others does not provide a VPU. This
commit updates the default setting accordingly.

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


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl
  2014-11-25 11:37     ` Otavio Salvador
@ 2014-11-25 12:21       ` Daiane Angolini
  0 siblings, 0 replies; 9+ messages in thread
From: Daiane Angolini @ 2014-11-25 12:21 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale Mailing List

On Tue, Nov 25, 2014 at 9:37 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Tue, Nov 25, 2014 at 9:23 AM, Daiane Angolini <daiane.list@gmail.com> wrote:
>> On Mon, Nov 24, 2014 at 6:39 PM, Otavio Salvador
>> <otavio@ossystems.com.br> wrote:
>>> The build system can also use egl when GL is not avail so we ought to
>>> add this to the dependency chain as gpu-viv-bin-mx6q would not be
>>> included otherwise, for i.MX6SL.
>>
>> In this case, would it be applicable only for imx6SL instead of all imx6?
>
> Not necessary; the other cases are catch due virtual/libgl which SL
> does not has.

Ok, thanks

Daiane


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2014-11-25 12:21 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-24 20:39 [meta-fsl-arm][PATCH 1/4] weston: Remove "Desktop-shell-Don-t-assume-there-is-a-pointer.patch" Otavio Salvador
2014-11-24 20:39 ` [meta-fsl-arm][PATCH 2/4] gstreamer1.0-plugins-bad: Remove useless overrides Otavio Salvador
2014-11-24 20:39 ` [meta-fsl-arm][PATCH 3/4] imx-base: Include gstreamer1.0-plugins-imx for i.MX6Q and i.MX6DL Otavio Salvador
2014-11-25 11:36   ` Daiane Angolini
2014-11-25 11:40     ` Otavio Salvador
2014-11-24 20:39 ` [meta-fsl-arm][PATCH 4/4] piglit: Add a dependency over virtual/egl Otavio Salvador
2014-11-25 11:23   ` Daiane Angolini
2014-11-25 11:37     ` Otavio Salvador
2014-11-25 12:21       ` Daiane Angolini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.