All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland
@ 2016-04-02 17:59 Khem Raj
  2016-04-02 17:59 ` [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library Khem Raj
  2016-04-13  3:30 ` [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Andrei Gherzan
  0 siblings, 2 replies; 9+ messages in thread
From: Khem Raj @ 2016-04-02 17:59 UTC (permalink / raw)
  To: yocto

Update userland to latest

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 ...Fix-for-framerate-with-nested-composition.patch | 21 ++++++-----
 ...014-Fix-errors-due-to-ignored-return-code.patch | 43 ----------------------
 recipes-graphics/userland/userland_git.bb          |  3 +-
 3 files changed, 12 insertions(+), 55 deletions(-)
 delete mode 100644 recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch

diff --git a/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch b/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
index 2fa9d66..ff9ccce 100644
--- a/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
+++ b/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
@@ -1,17 +1,17 @@
-From e5affc4ff145eedf61e2625a8919622c92d0b8ba Mon Sep 17 00:00:00 2001
+From f8779a124fc447b63c1ba1795330bdc2d765dc7a Mon Sep 17 00:00:00 2001
 From: Khem Raj <raj.khem@gmail.com>
 Date: Tue, 29 Mar 2016 20:38:30 -0700
-Subject: [PATCH 13/14] Fix for framerate with nested composition
+Subject: [PATCH 13/13] Fix for framerate with nested composition
 
 frame rate appears irregular and lower than expected when using nested composition.
 
 Signed-off-by: Khem Raj <raj.khem@gmail.com>
 ---
- interface/khronos/egl/egl_client.c | 9 ++++++++-
- 1 file changed, 8 insertions(+), 1 deletion(-)
+ interface/khronos/egl/egl_client.c | 8 ++++++++
+ 1 file changed, 8 insertions(+)
 
 diff --git a/interface/khronos/egl/egl_client.c b/interface/khronos/egl/egl_client.c
-index 024f3ed..d8a007c 100644
+index 024f3ed..83970ec 100644
 --- a/interface/khronos/egl/egl_client.c
 +++ b/interface/khronos/egl/egl_client.c
 @@ -2342,6 +2342,9 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
@@ -33,7 +33,7 @@ index 024f3ed..d8a007c 100644
                    surface->front_wl_buffer->in_use = 1;
                    wl_surface_attach(wl_egl_window->wl_surface,
                                      surface->front_wl_buffer->wl_buffer,
-@@ -2360,10 +2365,11 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
+@@ -2360,11 +2365,13 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
                    wl_surface_damage(wl_egl_window->wl_surface, 0, 0,
                                      surface->width, surface->height);
                    wl_surface_commit(wl_egl_window->wl_surface);
@@ -41,12 +41,13 @@ index 024f3ed..d8a007c 100644
  
                    while(ret != -1 && surface->back_wl_buffer->in_use)
                       ret = wl_display_dispatch_queue(wl_display, process->wl_queue);
--               } else
-+               } else {
+                } else
  #endif
++               {
                 RPC_CALL6(eglIntSwapBuffers_impl,
                       thread,
-@@ -2376,6 +2382,7 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
+                      EGLINTSWAPBUFFERS_ID,
+@@ -2376,6 +2383,7 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
                       RPC_UINT(khrn_platform_get_window_position(surface->win)));
  
                 RPC_FLUSH(thread);
@@ -55,5 +56,5 @@ index 024f3ed..d8a007c 100644
  #ifdef ANDROID
                 CLIENT_UNLOCK();
 -- 
-2.7.4
+2.8.0
 
diff --git a/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch b/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch
deleted file mode 100644
index 2d11e9b..0000000
--- a/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8ac57765d8b971aaa5ee5c610d8df25fa2ea36da Mon Sep 17 00:00:00 2001
-From: Khem Raj <raj.khem@gmail.com>
-Date: Thu, 24 Mar 2016 15:03:50 -0700
-Subject: [PATCH 14/14] Fix errors due to ignored return code
-
-Clang warns/errors about not checking the return
-values from chdir() and fread(), here we check
-for chdir return code and report back using error()
-API, and second fix to fread is to let compiler
-ignore the value via assigning it to an unused variable
-
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- host_applications/linux/apps/dtoverlay/dtoverlay_main.c | 6 ++++--
- 1 file changed, 4 insertions(+), 2 deletions(-)
-
-diff --git a/host_applications/linux/apps/dtoverlay/dtoverlay_main.c b/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
-index 4b33809..fa8324f 100755
---- a/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
-+++ b/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
-@@ -353,7 +353,8 @@ static int dtoverlay_remove(STATE_T *state, const char *overlay)
-     int rmpos;
-     int i;
- 
--    chdir(work_dir);
-+    if (chdir(work_dir) != 0)
-+	return error("chdir to %s failed", work_dir);
- 
-     overlay_dir = sprintf_dup("%s/%s", dt_overlays_dir, overlay);
-     if (!dir_exists(overlay_dir))
-@@ -633,7 +634,8 @@ static int overlay_applied(const char *overlay_dir)
-     FILE *fp = fopen(status_path, "r");
-     if (fp)
-     {
--	fread(status, sizeof(status), 1, fp);
-+	int bytes __attribute__((unused));
-+	bytes = fread(status, sizeof(status), 1, fp);
- 	fclose(fp);
-     }
-     free_string(status_path);
--- 
-2.7.4
-
diff --git a/recipes-graphics/userland/userland_git.bb b/recipes-graphics/userland/userland_git.bb
index ca9123e..efa2128 100644
--- a/recipes-graphics/userland/userland_git.bb
+++ b/recipes-graphics/userland/userland_git.bb
@@ -16,7 +16,7 @@ COMPATIBLE_MACHINE = "raspberrypi"
 
 SRCBRANCH = "master"
 SRCFORK = "raspberrypi"
-SRCREV = "2f56a2943a9eb8420df52ccf91f5a1c5a70e8713"
+SRCREV = "703a2c4b35e23ee44ad84db6b9c3f89c0a627143"
 
 SRC_URI = "\
     git://github.com/${SRCFORK}/userland.git;protocol=git;branch=${SRCBRANCH} \
@@ -33,7 +33,6 @@ SRC_URI = "\
     file://0011-fix-cmake-dependency-race.patch \
     file://0012-Fix-enum-conversion-warnings.patch \
     file://0013-Fix-for-framerate-with-nested-composition.patch \
-    file://0014-Fix-errors-due-to-ignored-return-code.patch \
     file://0015-wl-dispmanx-buffer-wrapping.patch \
 "
 S = "${WORKDIR}/git"
-- 
2.8.0



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

* [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-02 17:59 [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Khem Raj
@ 2016-04-02 17:59 ` Khem Raj
  2016-04-13 10:05   ` Jonathan Liu
  2016-04-13  3:30 ` [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Andrei Gherzan
  1 sibling, 1 reply; 9+ messages in thread
From: Khem Raj @ 2016-04-02 17:59 UTC (permalink / raw)
  To: yocto

This helps in using this library when building
with PIE binaries

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../0014-build-shared-library-for-vchostif.patch   | 28 ++++++++++++++++++++++
 recipes-graphics/userland/userland_git.bb          |  1 +
 2 files changed, 29 insertions(+)
 create mode 100644 recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch

diff --git a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
new file mode 100644
index 0000000..ca7cb66
--- /dev/null
+++ b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
@@ -0,0 +1,28 @@
+From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Sat, 2 Apr 2016 10:37:24 -0700
+Subject: [PATCH] build shared library for vchostif
+
+Fixes #149
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ interface/vmcs_host/CMakeLists.txt | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/interface/vmcs_host/CMakeLists.txt b/interface/vmcs_host/CMakeLists.txt
+index 22949a8..f8c2f2f 100755
+--- a/interface/vmcs_host/CMakeLists.txt
++++ b/interface/vmcs_host/CMakeLists.txt
+@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
+ #            ${VMCS_TARGET}/vmcs_main.c
+ #  vc_vchi_haud.c
+ 
+-add_library(vchostif ${VCHOSTIF_SOURCE})
++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
+ 
+ #add_library(bufman            vc_vchi_bufman.c            )
+ 
+-- 
+2.8.0
+
diff --git a/recipes-graphics/userland/userland_git.bb b/recipes-graphics/userland/userland_git.bb
index efa2128..6ed8fcf 100644
--- a/recipes-graphics/userland/userland_git.bb
+++ b/recipes-graphics/userland/userland_git.bb
@@ -33,6 +33,7 @@ SRC_URI = "\
     file://0011-fix-cmake-dependency-race.patch \
     file://0012-Fix-enum-conversion-warnings.patch \
     file://0013-Fix-for-framerate-with-nested-composition.patch \
+    file://0014-build-shared-library-for-vchostif.patch \
     file://0015-wl-dispmanx-buffer-wrapping.patch \
 "
 S = "${WORKDIR}/git"
-- 
2.8.0



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

* Re: [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland
  2016-04-02 17:59 [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Khem Raj
  2016-04-02 17:59 ` [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library Khem Raj
@ 2016-04-13  3:30 ` Andrei Gherzan
  1 sibling, 0 replies; 9+ messages in thread
From: Andrei Gherzan @ 2016-04-13  3:30 UTC (permalink / raw)
  To: Khem Raj; +Cc: yocto

[-- Attachment #1: Type: text/plain, Size: 6925 bytes --]

On Sat, Apr 02, 2016 at 10:59:11AM -0700, Khem Raj wrote:
> Update userland to latest
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  ...Fix-for-framerate-with-nested-composition.patch | 21 ++++++-----
>  ...014-Fix-errors-due-to-ignored-return-code.patch | 43 ----------------------
>  recipes-graphics/userland/userland_git.bb          |  3 +-
>  3 files changed, 12 insertions(+), 55 deletions(-)
>  delete mode 100644 recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch
>
> diff --git a/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch b/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
> index 2fa9d66..ff9ccce 100644
> --- a/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
> +++ b/recipes-graphics/userland/userland/0013-Fix-for-framerate-with-nested-composition.patch
> @@ -1,17 +1,17 @@
> -From e5affc4ff145eedf61e2625a8919622c92d0b8ba Mon Sep 17 00:00:00 2001
> +From f8779a124fc447b63c1ba1795330bdc2d765dc7a Mon Sep 17 00:00:00 2001
>  From: Khem Raj <raj.khem@gmail.com>
>  Date: Tue, 29 Mar 2016 20:38:30 -0700
> -Subject: [PATCH 13/14] Fix for framerate with nested composition
> +Subject: [PATCH 13/13] Fix for framerate with nested composition
>
>  frame rate appears irregular and lower than expected when using nested composition.
>
>  Signed-off-by: Khem Raj <raj.khem@gmail.com>
>  ---
> - interface/khronos/egl/egl_client.c | 9 ++++++++-
> - 1 file changed, 8 insertions(+), 1 deletion(-)
> + interface/khronos/egl/egl_client.c | 8 ++++++++
> + 1 file changed, 8 insertions(+)
>
>  diff --git a/interface/khronos/egl/egl_client.c b/interface/khronos/egl/egl_client.c
> -index 024f3ed..d8a007c 100644
> +index 024f3ed..83970ec 100644
>  --- a/interface/khronos/egl/egl_client.c
>  +++ b/interface/khronos/egl/egl_client.c
>  @@ -2342,6 +2342,9 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
> @@ -33,7 +33,7 @@ index 024f3ed..d8a007c 100644
>                     surface->front_wl_buffer->in_use = 1;
>                     wl_surface_attach(wl_egl_window->wl_surface,
>                                       surface->front_wl_buffer->wl_buffer,
> -@@ -2360,10 +2365,11 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
> +@@ -2360,11 +2365,13 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
>                     wl_surface_damage(wl_egl_window->wl_surface, 0, 0,
>                                       surface->width, surface->height);
>                     wl_surface_commit(wl_egl_window->wl_surface);
> @@ -41,12 +41,13 @@ index 024f3ed..d8a007c 100644
>
>                     while(ret != -1 && surface->back_wl_buffer->in_use)
>                        ret = wl_display_dispatch_queue(wl_display, process->wl_queue);
> --               } else
> -+               } else {
> +                } else
>   #endif
> ++               {
>                  RPC_CALL6(eglIntSwapBuffers_impl,
>                        thread,
> -@@ -2376,6 +2382,7 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
> +                      EGLINTSWAPBUFFERS_ID,
> +@@ -2376,6 +2383,7 @@ EGLAPI EGLBoolean EGLAPIENTRY eglSwapBuffers(EGLDisplay dpy, EGLSurface surf)
>                        RPC_UINT(khrn_platform_get_window_position(surface->win)));
>
>                  RPC_FLUSH(thread);
> @@ -55,5 +56,5 @@ index 024f3ed..d8a007c 100644
>   #ifdef ANDROID
>                  CLIENT_UNLOCK();
>  --
> -2.7.4
> +2.8.0
>
> diff --git a/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch b/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch
> deleted file mode 100644
> index 2d11e9b..0000000
> --- a/recipes-graphics/userland/userland/0014-Fix-errors-due-to-ignored-return-code.patch
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -From 8ac57765d8b971aaa5ee5c610d8df25fa2ea36da Mon Sep 17 00:00:00 2001
> -From: Khem Raj <raj.khem@gmail.com>
> -Date: Thu, 24 Mar 2016 15:03:50 -0700
> -Subject: [PATCH 14/14] Fix errors due to ignored return code
> -
> -Clang warns/errors about not checking the return
> -values from chdir() and fread(), here we check
> -for chdir return code and report back using error()
> -API, and second fix to fread is to let compiler
> -ignore the value via assigning it to an unused variable
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ----
> - host_applications/linux/apps/dtoverlay/dtoverlay_main.c | 6 ++++--
> - 1 file changed, 4 insertions(+), 2 deletions(-)
> -
> -diff --git a/host_applications/linux/apps/dtoverlay/dtoverlay_main.c b/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
> -index 4b33809..fa8324f 100755
> ---- a/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
> -+++ b/host_applications/linux/apps/dtoverlay/dtoverlay_main.c
> -@@ -353,7 +353,8 @@ static int dtoverlay_remove(STATE_T *state, const char *overlay)
> -     int rmpos;
> -     int i;
> -
> --    chdir(work_dir);
> -+    if (chdir(work_dir) != 0)
> -+	return error("chdir to %s failed", work_dir);
> -
> -     overlay_dir = sprintf_dup("%s/%s", dt_overlays_dir, overlay);
> -     if (!dir_exists(overlay_dir))
> -@@ -633,7 +634,8 @@ static int overlay_applied(const char *overlay_dir)
> -     FILE *fp = fopen(status_path, "r");
> -     if (fp)
> -     {
> --	fread(status, sizeof(status), 1, fp);
> -+	int bytes __attribute__((unused));
> -+	bytes = fread(status, sizeof(status), 1, fp);
> - 	fclose(fp);
> -     }
> -     free_string(status_path);
> ---
> -2.7.4
> -
> diff --git a/recipes-graphics/userland/userland_git.bb b/recipes-graphics/userland/userland_git.bb
> index ca9123e..efa2128 100644
> --- a/recipes-graphics/userland/userland_git.bb
> +++ b/recipes-graphics/userland/userland_git.bb
> @@ -16,7 +16,7 @@ COMPATIBLE_MACHINE = "raspberrypi"
>
>  SRCBRANCH = "master"
>  SRCFORK = "raspberrypi"
> -SRCREV = "2f56a2943a9eb8420df52ccf91f5a1c5a70e8713"
> +SRCREV = "703a2c4b35e23ee44ad84db6b9c3f89c0a627143"
>
>  SRC_URI = "\
>      git://github.com/${SRCFORK}/userland.git;protocol=git;branch=${SRCBRANCH} \
> @@ -33,7 +33,6 @@ SRC_URI = "\
>      file://0011-fix-cmake-dependency-race.patch \
>      file://0012-Fix-enum-conversion-warnings.patch \
>      file://0013-Fix-for-framerate-with-nested-composition.patch \
> -    file://0014-Fix-errors-due-to-ignored-return-code.patch \
>      file://0015-wl-dispmanx-buffer-wrapping.patch \
>  "
>  S = "${WORKDIR}/git"
> --
> 2.8.0
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto

Merged both patches. Thanks.

--
Andrei Gherzan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-02 17:59 ` [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library Khem Raj
@ 2016-04-13 10:05   ` Jonathan Liu
  2016-04-13 14:12     ` Khem Raj
  0 siblings, 1 reply; 9+ messages in thread
From: Jonathan Liu @ 2016-04-13 10:05 UTC (permalink / raw)
  To: Khem Raj; +Cc: Yocto Project

Hi Khem,

On 3 April 2016 at 03:59, Khem Raj <raj.khem@gmail.com> wrote:
> This helps in using this library when building
> with PIE binaries
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  .../0014-build-shared-library-for-vchostif.patch   | 28 ++++++++++++++++++++++
>  recipes-graphics/userland/userland_git.bb          |  1 +
>  2 files changed, 29 insertions(+)
>  create mode 100644 recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>
> diff --git a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> new file mode 100644
> index 0000000..ca7cb66
> --- /dev/null
> +++ b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> @@ -0,0 +1,28 @@
> +From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
> +From: Khem Raj <raj.khem@gmail.com>
> +Date: Sat, 2 Apr 2016 10:37:24 -0700
> +Subject: [PATCH] build shared library for vchostif
> +
> +Fixes #149
> +
> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> +---
> + interface/vmcs_host/CMakeLists.txt | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/interface/vmcs_host/CMakeLists.txt b/interface/vmcs_host/CMakeLists.txt
> +index 22949a8..f8c2f2f 100755
> +--- a/interface/vmcs_host/CMakeLists.txt
> ++++ b/interface/vmcs_host/CMakeLists.txt
> +@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
> + #            ${VMCS_TARGET}/vmcs_main.c
> + #  vc_vchi_haud.c
> +
> +-add_library(vchostif ${VCHOSTIF_SOURCE})
> ++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
> +
> + #add_library(bufman            vc_vchi_bufman.c            )
> +
> +--
> +2.8.0
> +
> diff --git a/recipes-graphics/userland/userland_git.bb b/recipes-graphics/userland/userland_git.bb
> index efa2128..6ed8fcf 100644
> --- a/recipes-graphics/userland/userland_git.bb
> +++ b/recipes-graphics/userland/userland_git.bb
> @@ -33,6 +33,7 @@ SRC_URI = "\
>      file://0011-fix-cmake-dependency-race.patch \
>      file://0012-Fix-enum-conversion-warnings.patch \
>      file://0013-Fix-for-framerate-with-nested-composition.patch \
> +    file://0014-build-shared-library-for-vchostif.patch \
>      file://0015-wl-dispmanx-buffer-wrapping.patch \
>  "
>  S = "${WORKDIR}/git"
> --
> 2.8.0
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto

This breaks omxplayer compilation when linking omxplayer.bin:
[...]/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
omxplayer.o: undefined reference to symbol
'vc_dispmanx_resource_write_data'
[...]/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding
symbols: DSO missing from command line

Regards,
Jonathan


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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-13 10:05   ` Jonathan Liu
@ 2016-04-13 14:12     ` Khem Raj
  2016-04-13 14:21       ` Jonathan Liu
  0 siblings, 1 reply; 9+ messages in thread
From: Khem Raj @ 2016-04-13 14:12 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: Yocto Project

On Wed, Apr 13, 2016 at 3:05 AM, Jonathan Liu <net147@gmail.com> wrote:
> Hi Khem,
>
> On 3 April 2016 at 03:59, Khem Raj <raj.khem@gmail.com> wrote:
>> This helps in using this library when building
>> with PIE binaries
>>
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>  .../0014-build-shared-library-for-vchostif.patch   | 28 ++++++++++++++++++++++
>>  recipes-graphics/userland/userland_git.bb          |  1 +
>>  2 files changed, 29 insertions(+)
>>  create mode 100644 recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>>
>> diff --git a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> new file mode 100644
>> index 0000000..ca7cb66
>> --- /dev/null
>> +++ b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> @@ -0,0 +1,28 @@
>> +From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
>> +From: Khem Raj <raj.khem@gmail.com>
>> +Date: Sat, 2 Apr 2016 10:37:24 -0700
>> +Subject: [PATCH] build shared library for vchostif
>> +
>> +Fixes #149
>> +
>> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> +---
>> + interface/vmcs_host/CMakeLists.txt | 2 +-
>> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> +
>> +diff --git a/interface/vmcs_host/CMakeLists.txt b/interface/vmcs_host/CMakeLists.txt
>> +index 22949a8..f8c2f2f 100755
>> +--- a/interface/vmcs_host/CMakeLists.txt
>> ++++ b/interface/vmcs_host/CMakeLists.txt
>> +@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
>> + #            ${VMCS_TARGET}/vmcs_main.c
>> + #  vc_vchi_haud.c
>> +
>> +-add_library(vchostif ${VCHOSTIF_SOURCE})
>> ++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
>> +
>> + #add_library(bufman            vc_vchi_bufman.c            )
>> +
>> +--
>> +2.8.0
>> +
>> diff --git a/recipes-graphics/userland/userland_git.bb b/recipes-graphics/userland/userland_git.bb
>> index efa2128..6ed8fcf 100644
>> --- a/recipes-graphics/userland/userland_git.bb
>> +++ b/recipes-graphics/userland/userland_git.bb
>> @@ -33,6 +33,7 @@ SRC_URI = "\
>>      file://0011-fix-cmake-dependency-race.patch \
>>      file://0012-Fix-enum-conversion-warnings.patch \
>>      file://0013-Fix-for-framerate-with-nested-composition.patch \
>> +    file://0014-build-shared-library-for-vchostif.patch \
>>      file://0015-wl-dispmanx-buffer-wrapping.patch \
>>  "
>>  S = "${WORKDIR}/git"
>> --
>> 2.8.0
>>
>> --
>> _______________________________________________
>> yocto mailing list
>> yocto@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/yocto
>
> This breaks omxplayer compilation when linking omxplayer.bin:
> [...]/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
> omxplayer.o: undefined reference to symbol
> 'vc_dispmanx_resource_write_data'
> [...]/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding
> symbols: DSO missing from command line

Interestingly i dont see this error. Are you using gold ?
see if this library is on linker cmdline or not

>
> Regards,
> Jonathan


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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-13 14:12     ` Khem Raj
@ 2016-04-13 14:21       ` Jonathan Liu
  2016-04-13 16:36         ` Khem Raj
  0 siblings, 1 reply; 9+ messages in thread
From: Jonathan Liu @ 2016-04-13 14:21 UTC (permalink / raw)
  To: Khem Raj; +Cc: Yocto Project

[-- Attachment #1: Type: text/plain, Size: 5195 bytes --]

On Thursday, 14 April 2016, Khem Raj <raj.khem@gmail.com> wrote:

> On Wed, Apr 13, 2016 at 3:05 AM, Jonathan Liu <net147@gmail.com
> <javascript:;>> wrote:
> > Hi Khem,
> >
> > On 3 April 2016 at 03:59, Khem Raj <raj.khem@gmail.com <javascript:;>>
> wrote:
> >> This helps in using this library when building
> >> with PIE binaries
> >>
> >> Signed-off-by: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> ---
> >>  .../0014-build-shared-library-for-vchostif.patch   | 28
> ++++++++++++++++++++++
> >>  recipes-graphics/userland/userland_git.bb          |  1 +
> >>  2 files changed, 29 insertions(+)
> >>  create mode 100644
> recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >>
> >> diff --git
> a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >> new file mode 100644
> >> index 0000000..ca7cb66
> >> --- /dev/null
> >> +++
> b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >> @@ -0,0 +1,28 @@
> >> +From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
> >> +From: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> +Date: Sat, 2 Apr 2016 10:37:24 -0700
> >> +Subject: [PATCH] build shared library for vchostif
> >> +
> >> +Fixes #149
> >> +
> >> +Signed-off-by: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> +---
> >> + interface/vmcs_host/CMakeLists.txt | 2 +-
> >> + 1 file changed, 1 insertion(+), 1 deletion(-)
> >> +
> >> +diff --git a/interface/vmcs_host/CMakeLists.txt
> b/interface/vmcs_host/CMakeLists.txt
> >> +index 22949a8..f8c2f2f 100755
> >> +--- a/interface/vmcs_host/CMakeLists.txt
> >> ++++ b/interface/vmcs_host/CMakeLists.txt
> >> +@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
> >> + #            ${VMCS_TARGET}/vmcs_main.c
> >> + #  vc_vchi_haud.c
> >> +
> >> +-add_library(vchostif ${VCHOSTIF_SOURCE})
> >> ++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
> >> +
> >> + #add_library(bufman            vc_vchi_bufman.c            )
> >> +
> >> +--
> >> +2.8.0
> >> +
> >> diff --git a/recipes-graphics/userland/userland_git.bb
> b/recipes-graphics/userland/userland_git.bb
> >> index efa2128..6ed8fcf 100644
> >> --- a/recipes-graphics/userland/userland_git.bb
> >> +++ b/recipes-graphics/userland/userland_git.bb
> >> @@ -33,6 +33,7 @@ SRC_URI = "\
> >>      file://0011-fix-cmake-dependency-race.patch \
> >>      file://0012-Fix-enum-conversion-warnings.patch \
> >>      file://0013-Fix-for-framerate-with-nested-composition.patch \
> >> +    file://0014-build-shared-library-for-vchostif.patch \
> >>      file://0015-wl-dispmanx-buffer-wrapping.patch \
> >>  "
> >>  S = "${WORKDIR}/git"
> >> --
> >> 2.8.0
> >>
> >> --
> >> _______________________________________________
> >> yocto mailing list
> >> yocto@yoctoproject.org <javascript:;>
> >> https://lists.yoctoproject.org/listinfo/yocto
> >
> > This breaks omxplayer compilation when linking omxplayer.bin:
> >
> [...]/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
> > omxplayer.o: undefined reference to symbol
> > 'vc_dispmanx_resource_write_data'
> > [...]/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding
> > symbols: DSO missing from command line
>
> Interestingly i dont see this error. Are you using gold ?
> see if this library is on linker cmdline or not

Using whatever is the default for OE-core in master branch.

 | arm-custom-linux-gnueabi-g++  -march=armv7ve -marm
 -mfpu=neon-vfpv4  -mfloat-abi=hard -mcpu=cortex-a7
--sysroot=/home/build/git/pi2/build/tmp/sysroots/ra
spberrypi2 -L/home/build/git/pi2/build/tmp/work/cor
texa7hf-neon-vfpv4-custom-linux-gnueabi/omxplayer/gi
t-r4/git/ffmpeg_compiled/usr/lib                   -
L/home/build/git/pi2/build/tmp/sysroots/raspberrypi
2/lib                   -L/home/build/git/pi2/build
/tmp/sysroots/raspberrypi2/usr/lib
 -L./ -Lffmpeg_compiled/usr/lib/ -lc -lWFC -lGLESv2
-lEGL -lbcm_host -lopenmaxil -lfreetype -lz -o omxpl
ayer.bin linux/XMemUtils.o utils/log.o DynamicDll.o
utils/PCMRemap.o utils/RegExp.o OMXSubtitleTagSami.o
 OMXOverlayCodecText.o BitstreamConverter.o linux/RB
P.o OMXThread.o OMXReader.o OMXStreamInfo.o OMXAudio
CodecOMX.o OMXCore.o OMXVideo.o OMXAudio.o OMXClock.
o File.o OMXPlayerVideo.o OMXPlayerAudio.o OMXPlayer
Subtitles.o SubtitleRenderer.o Unicode.o Srt.o KeyCo
nfig.o OMXControl.o Keyboard.o omxplayer.o -lvchiq_a
rm -lvcos -ldbus-1 -lrt -lpthread -lavutil -lavcodec
 -lavformat -lswscale -lswresample -lpcre
|
/home/build/git/pi2/build/tmp/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
omxplayer.o: undefined reference to symbol 'vc_dispmanx_resource_write_data'
|
/home/build/git/pi2/build/tmp/sysroots/raspberrypi2/usr/lib/libvchostif.so:
error adding symbols: DSO
 missing from command line
| collect2: error: ld returned 1 exit status
| Makefile:53: recipe for target 'omxplayer.bin' failed
| make: *** [omxplayer.bin] Error 1

>
> >
> > Regards,
> > Jonathan
>

[-- Attachment #2: Type: text/html, Size: 7285 bytes --]

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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-13 14:21       ` Jonathan Liu
@ 2016-04-13 16:36         ` Khem Raj
  2016-04-16  1:24           ` Jonathan Liu
  0 siblings, 1 reply; 9+ messages in thread
From: Khem Raj @ 2016-04-13 16:36 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: Yocto Project


[-- Attachment #1.1: Type: text/plain, Size: 5546 bytes --]

Thats interesting, I will try to reproduce it

> On Apr 13, 2016, at 7:21 AM, Jonathan Liu <net147@gmail.com> wrote:
> 
> On Thursday, 14 April 2016, Khem Raj <raj.khem@gmail.com <mailto:raj.khem@gmail.com>> wrote:
> On Wed, Apr 13, 2016 at 3:05 AM, Jonathan Liu <net147@gmail.com <javascript:;>> wrote:
> > Hi Khem,
> >
> > On 3 April 2016 at 03:59, Khem Raj <raj.khem@gmail.com <javascript:;>> wrote:
> >> This helps in using this library when building
> >> with PIE binaries
> >>
> >> Signed-off-by: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> ---
> >>  .../0014-build-shared-library-for-vchostif.patch   | 28 ++++++++++++++++++++++
> >>  recipes-graphics/userland/userland_git.bb <http://userland_git.bb/>          |  1 +
> >>  2 files changed, 29 insertions(+)
> >>  create mode 100644 recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >>
> >> diff --git a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >> new file mode 100644
> >> index 0000000..ca7cb66
> >> --- /dev/null
> >> +++ b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
> >> @@ -0,0 +1,28 @@
> >> +From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
> >> +From: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> +Date: Sat, 2 Apr 2016 10:37:24 -0700
> >> +Subject: [PATCH] build shared library for vchostif
> >> +
> >> +Fixes #149
> >> +
> >> +Signed-off-by: Khem Raj <raj.khem@gmail.com <javascript:;>>
> >> +---
> >> + interface/vmcs_host/CMakeLists.txt | 2 +-
> >> + 1 file changed, 1 insertion(+), 1 deletion(-)
> >> +
> >> +diff --git a/interface/vmcs_host/CMakeLists.txt b/interface/vmcs_host/CMakeLists.txt
> >> +index 22949a8..f8c2f2f 100755
> >> +--- a/interface/vmcs_host/CMakeLists.txt
> >> ++++ b/interface/vmcs_host/CMakeLists.txt
> >> +@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
> >> + #            ${VMCS_TARGET}/vmcs_main.c
> >> + #  vc_vchi_haud.c
> >> +
> >> +-add_library(vchostif ${VCHOSTIF_SOURCE})
> >> ++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
> >> +
> >> + #add_library(bufman            vc_vchi_bufman.c            )
> >> +
> >> +--
> >> +2.8.0
> >> +
> >> diff --git a/recipes-graphics/userland/userland_git.bb <http://userland_git.bb/> b/recipes-graphics/userland/userland_git.bb <http://userland_git.bb/>
> >> index efa2128..6ed8fcf 100644
> >> --- a/recipes-graphics/userland/userland_git.bb <http://userland_git.bb/>
> >> +++ b/recipes-graphics/userland/userland_git.bb <http://userland_git.bb/>
> >> @@ -33,6 +33,7 @@ SRC_URI = "\
> >>      file://0011-fix-cmake-dependency-race.patch \
> >>      file://0012-Fix-enum-conversion-warnings.patch \
> >>      file://0013-Fix-for-framerate-with-nested-composition.patch \
> >> +    file://0014-build-shared-library-for-vchostif.patch \
> >>      file://0015-wl-dispmanx-buffer-wrapping.patch \
> >>  "
> >>  S = "${WORKDIR}/git"
> >> --
> >> 2.8.0
> >>
> >> --
> >> _______________________________________________
> >> yocto mailing list
> >> yocto@yoctoproject.org <javascript:;>
> >> https://lists.yoctoproject.org/listinfo/yocto <https://lists.yoctoproject.org/listinfo/yocto>
> >
> > This breaks omxplayer compilation when linking omxplayer.bin:
> > [...]/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
> > omxplayer.o: undefined reference to symbol
> > 'vc_dispmanx_resource_write_data'
> > [...]/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding
> > symbols: DSO missing from command line
> 
> Interestingly i dont see this error. Are you using gold ?
> see if this library is on linker cmdline or not
> Using whatever is the default for OE-core in master branch.
> 
>  | arm-custom-linux-gnueabi-g++  -march=armv7ve -marm
>  -mfpu=neon-vfpv4  -mfloat-abi=hard -mcpu=cortex-a7
> --sysroot=/home/build/git/pi2/build/tmp/sysroots/ra
> spberrypi2 -L/home/build/git/pi2/build/tmp/work/cor
> texa7hf-neon-vfpv4-custom-linux-gnueabi/omxplayer/gi
> t-r4/git/ffmpeg_compiled/usr/lib                   -
> L/home/build/git/pi2/build/tmp/sysroots/raspberrypi
> 2/lib                   -L/home/build/git/pi2/build
> /tmp/sysroots/raspberrypi2/usr/lib
>  -L./ -Lffmpeg_compiled/usr/lib/ -lc -lWFC -lGLESv2
> -lEGL -lbcm_host -lopenmaxil -lfreetype -lz -o omxpl
> ayer.bin linux/XMemUtils.o utils/log.o DynamicDll.o
> utils/PCMRemap.o utils/RegExp.o OMXSubtitleTagSami.o
>  OMXOverlayCodecText.o BitstreamConverter.o linux/RB
> P.o OMXThread.o OMXReader.o OMXStreamInfo.o OMXAudio
> CodecOMX.o OMXCore.o OMXVideo.o OMXAudio.o OMXClock.
> o File.o OMXPlayerVideo.o OMXPlayerAudio.o OMXPlayer
> Subtitles.o SubtitleRenderer.o Unicode.o Srt.o KeyCo
> nfig.o OMXControl.o Keyboard.o omxplayer.o -lvchiq_a
> rm -lvcos -ldbus-1 -lrt -lpthread -lavutil -lavcodec
>  -lavformat -lswscale -lswresample -lpcre
> | /home/build/git/pi2/build/tmp/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld: omxplayer.o: undefined reference to symbol 'vc_dispmanx_resource_write_data'
> | /home/build/git/pi2/build/tmp/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding symbols: DSO
>  missing from command line
> | collect2: error: ld returned 1 exit status
> | Makefile:53: recipe for target 'omxplayer.bin' failed
> | make: *** [omxplayer.bin] Error 1
> 
> >
> > Regards,
> > Jonathan


[-- Attachment #1.2: Type: text/html, Size: 9551 bytes --]

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-13 16:36         ` Khem Raj
@ 2016-04-16  1:24           ` Jonathan Liu
  2016-04-21 17:12             ` Andrei Gherzan
  0 siblings, 1 reply; 9+ messages in thread
From: Jonathan Liu @ 2016-04-16  1:24 UTC (permalink / raw)
  To: Khem Raj; +Cc: Yocto Project

Hi Khem,

On 14 April 2016 at 02:36, Khem Raj <raj.khem@gmail.com> wrote:
> Thats interesting, I will try to reproduce it

I have submitted a patch to the mailing list to bump the omxplayer
revision which fixes the issue.

>
> On Apr 13, 2016, at 7:21 AM, Jonathan Liu <net147@gmail.com> wrote:
>
> On Thursday, 14 April 2016, Khem Raj <raj.khem@gmail.com> wrote:
>>
>> On Wed, Apr 13, 2016 at 3:05 AM, Jonathan Liu <net147@gmail.com> wrote:
>> > Hi Khem,
>> >
>> > On 3 April 2016 at 03:59, Khem Raj <raj.khem@gmail.com> wrote:
>> >> This helps in using this library when building
>> >> with PIE binaries
>> >>
>> >> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> >> ---
>> >>  .../0014-build-shared-library-for-vchostif.patch   | 28
>> >> ++++++++++++++++++++++
>> >>  recipes-graphics/userland/userland_git.bb          |  1 +
>> >>  2 files changed, 29 insertions(+)
>> >>  create mode 100644
>> >> recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> >>
>> >> diff --git
>> >> a/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> >> b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> >> new file mode 100644
>> >> index 0000000..ca7cb66
>> >> --- /dev/null
>> >> +++
>> >> b/recipes-graphics/userland/userland/0014-build-shared-library-for-vchostif.patch
>> >> @@ -0,0 +1,28 @@
>> >> +From 4e9aa3fffb458ca4fafd2241c18e9c8008b66ad8 Mon Sep 17 00:00:00 2001
>> >> +From: Khem Raj <raj.khem@gmail.com>
>> >> +Date: Sat, 2 Apr 2016 10:37:24 -0700
>> >> +Subject: [PATCH] build shared library for vchostif
>> >> +
>> >> +Fixes #149
>> >> +
>> >> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> >> +---
>> >> + interface/vmcs_host/CMakeLists.txt | 2 +-
>> >> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> >> +
>> >> +diff --git a/interface/vmcs_host/CMakeLists.txt
>> >> b/interface/vmcs_host/CMakeLists.txt
>> >> +index 22949a8..f8c2f2f 100755
>> >> +--- a/interface/vmcs_host/CMakeLists.txt
>> >> ++++ b/interface/vmcs_host/CMakeLists.txt
>> >> +@@ -17,7 +17,7 @@ set(VCHOSTIF_SOURCE
>> >> + #            ${VMCS_TARGET}/vmcs_main.c
>> >> + #  vc_vchi_haud.c
>> >> +
>> >> +-add_library(vchostif ${VCHOSTIF_SOURCE})
>> >> ++add_library(vchostif SHARED ${VCHOSTIF_SOURCE})
>> >> +
>> >> + #add_library(bufman            vc_vchi_bufman.c            )
>> >> +
>> >> +--
>> >> +2.8.0
>> >> +
>> >> diff --git a/recipes-graphics/userland/userland_git.bb
>> >> b/recipes-graphics/userland/userland_git.bb
>> >> index efa2128..6ed8fcf 100644
>> >> --- a/recipes-graphics/userland/userland_git.bb
>> >> +++ b/recipes-graphics/userland/userland_git.bb
>> >> @@ -33,6 +33,7 @@ SRC_URI = "\
>> >>      file://0011-fix-cmake-dependency-race.patch \
>> >>      file://0012-Fix-enum-conversion-warnings.patch \
>> >>      file://0013-Fix-for-framerate-with-nested-composition.patch \
>> >> +    file://0014-build-shared-library-for-vchostif.patch \
>> >>      file://0015-wl-dispmanx-buffer-wrapping.patch \
>> >>  "
>> >>  S = "${WORKDIR}/git"
>> >> --
>> >> 2.8.0
>> >>
>> >> --
>> >> _______________________________________________
>> >> yocto mailing list
>> >> yocto@yoctoproject.org
>> >> https://lists.yoctoproject.org/listinfo/yocto
>> >
>> > This breaks omxplayer compilation when linking omxplayer.bin:
>> >
>> > [...]/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
>> > omxplayer.o: undefined reference to symbol
>> > 'vc_dispmanx_resource_write_data'
>> > [...]/sysroots/raspberrypi2/usr/lib/libvchostif.so: error adding
>> > symbols: DSO missing from command line
>>
>> Interestingly i dont see this error. Are you using gold ?
>> see if this library is on linker cmdline or not
>
> Using whatever is the default for OE-core in master branch.
>
>  | arm-custom-linux-gnueabi-g++  -march=armv7ve -marm
>  -mfpu=neon-vfpv4  -mfloat-abi=hard -mcpu=cortex-a7
> --sysroot=/home/build/git/pi2/build/tmp/sysroots/ra
> spberrypi2 -L/home/build/git/pi2/build/tmp/work/cor
> texa7hf-neon-vfpv4-custom-linux-gnueabi/omxplayer/gi
> t-r4/git/ffmpeg_compiled/usr/lib                   -
> L/home/build/git/pi2/build/tmp/sysroots/raspberrypi
> 2/lib                   -L/home/build/git/pi2/build
> /tmp/sysroots/raspberrypi2/usr/lib
>  -L./ -Lffmpeg_compiled/usr/lib/ -lc -lWFC -lGLESv2
> -lEGL -lbcm_host -lopenmaxil -lfreetype -lz -o omxpl
> ayer.bin linux/XMemUtils.o utils/log.o DynamicDll.o
> utils/PCMRemap.o utils/RegExp.o OMXSubtitleTagSami.o
>  OMXOverlayCodecText.o BitstreamConverter.o linux/RB
> P.o OMXThread.o OMXReader.o OMXStreamInfo.o OMXAudio
> CodecOMX.o OMXCore.o OMXVideo.o OMXAudio.o OMXClock.
> o File.o OMXPlayerVideo.o OMXPlayerAudio.o OMXPlayer
> Subtitles.o SubtitleRenderer.o Unicode.o Srt.o KeyCo
> nfig.o OMXControl.o Keyboard.o omxplayer.o -lvchiq_a
> rm -lvcos -ldbus-1 -lrt -lpthread -lavutil -lavcodec
>  -lavformat -lswscale -lswresample -lpcre
> |
> /home/build/git/pi2/build/tmp/sysroots/x86_64-linux/usr/libexec/arm-custom-linux-gnueabi/gcc/arm-custom-linux-gnueabi/5.3.0/ld:
> omxplayer.o: undefined reference to symbol 'vc_dispmanx_resource_write_data'
> |
> /home/build/git/pi2/build/tmp/sysroots/raspberrypi2/usr/lib/libvchostif.so:
> error adding symbols: DSO
>  missing from command line
> | collect2: error: ld returned 1 exit status
> | Makefile:53: recipe for target 'omxplayer.bin' failed
> | make: *** [omxplayer.bin] Error 1

Regards,
Jonathan


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

* Re: [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library
  2016-04-16  1:24           ` Jonathan Liu
@ 2016-04-21 17:12             ` Andrei Gherzan
  0 siblings, 0 replies; 9+ messages in thread
From: Andrei Gherzan @ 2016-04-21 17:12 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: Yocto Project

[-- Attachment #1: Type: text/plain, Size: 385 bytes --]

On Sat, Apr 16, 2016 at 11:24:01AM +1000, Jonathan Liu wrote:
> Hi Khem,
>
> On 14 April 2016 at 02:36, Khem Raj <raj.khem@gmail.com> wrote:
> > Thats interesting, I will try to reproduce it
>
> I have submitted a patch to the mailing list to bump the omxplayer
> revision which fixes the issue.
>

Cool. I'll take a look on it, test it and merge today.
--
Andrei Gherzan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

end of thread, other threads:[~2016-04-21 17:12 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-02 17:59 [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Khem Raj
2016-04-02 17:59 ` [meta-raspberrypi][PATCH 2/2] userland: Make vchostif as shared library Khem Raj
2016-04-13 10:05   ` Jonathan Liu
2016-04-13 14:12     ` Khem Raj
2016-04-13 14:21       ` Jonathan Liu
2016-04-13 16:36         ` Khem Raj
2016-04-16  1:24           ` Jonathan Liu
2016-04-21 17:12             ` Andrei Gherzan
2016-04-13  3:30 ` [meta-raspberrypi][PATCH 1/2] userland: Fix a build regression in builds not using wayland Andrei Gherzan

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.