All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/2] weston: fix i.MX support
@ 2020-06-19 12:54 Gary Bisson
  2020-06-19 12:54 ` [Buildroot] [PATCH 1/2] package/weston: add imx-gpu-viv as possible egl provider Gary Bisson
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Gary Bisson @ 2020-06-19 12:54 UTC (permalink / raw)
  To: buildroot

Hi,

This series aims at fixing the weston support for i.MX processors.

Long time ago, weston-imx was added for i.MX8MQ as part of the weston
package. Then it was split from it as the versions differed.

Since then the weston-imx was broken and updating it seems like a bad
idea as it now requires forking several other components:
- libdrm [1]
- waylands-protocols [2]

This series therefore removes weston-imx and adds vivante support to
the regular weston package.

Drawback here is that the HDR capabilities of the i.MX8MQ or the GPU2D
of the i.MX8M Mini won't be used in Weston. I'd rather have those
limitations and use standard weston than forking everything.

Also, note that this series was tested on Nitrogen8M and Nitrogen8M Mini
platforms, making sure the Vivante 3D examples are properly running.
However in order to compile weston properly with Vivante as egl
provider, one patch sent earlier [3] is needed.

Let me know if you have any questions.

Regards,
Gary

[1] https://source.codeaurora.org/external/imx/meta-imx/tree/meta-bsp/recipes-graphics/drm/libdrm_2.4.99.imx.bb?h=zeus-5.4.3-1.0.0
[2] https://source.codeaurora.org/external/imx/meta-imx/tree/meta-bsp/recipes-graphics/wayland/wayland-protocols_1.18.imx.bb?h=zeus-5.4.3-1.0.0
[3] http://patchwork.ozlabs.org/project/buildroot/patch/20200402130842.918696-3-gary.bisson at boundarydevices.com/

Gary Bisson (2):
  package/weston: add imx-gpu-viv as possible egl provider
  package/weston-imx: remove deprecated package

 Config.in.legacy                   |   6 ++
 package/Config.in                  |   1 -
 package/weston-imx/Config.in       | 114 -----------------------
 package/weston-imx/weston-imx.hash |   3 -
 package/weston-imx/weston-imx.mk   | 144 -----------------------------
 package/weston/Config.in           |   8 +-
 6 files changed, 10 insertions(+), 266 deletions(-)
 delete mode 100644 package/weston-imx/Config.in
 delete mode 100644 package/weston-imx/weston-imx.hash
 delete mode 100644 package/weston-imx/weston-imx.mk

-- 
2.26.2

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

* [Buildroot] [PATCH 1/2] package/weston: add imx-gpu-viv as possible egl provider
  2020-06-19 12:54 [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
@ 2020-06-19 12:54 ` Gary Bisson
  2020-06-19 12:55 ` [Buildroot] [PATCH 2/2] package/weston-imx: remove deprecated package Gary Bisson
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Gary Bisson @ 2020-06-19 12:54 UTC (permalink / raw)
  To: buildroot

As the imx-gpu-viv package provides both EGL and gbm libraries.

Signed-off-by: Gary Bisson <gary.bisson@boundarydevices.com>
---
 package/weston/Config.in | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/package/weston/Config.in b/package/weston/Config.in
index bf783b70d4..d07eba4c9d 100644
--- a/package/weston/Config.in
+++ b/package/weston/Config.in
@@ -39,11 +39,11 @@ config BR2_PACKAGE_WESTON_DEFAULT_FBDEV
 
 config BR2_PACKAGE_WESTON_DEFAULT_DRM
 	bool "drm"
-	depends on BR2_PACKAGE_MESA3D_OPENGL_EGL
+	depends on BR2_PACKAGE_MESA3D_OPENGL_EGL || BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
 	select BR2_PACKAGE_WESTON_DRM
 
 comment "drm backend needs mesa3d w/ EGL driver"
-	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL
+	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL && !BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
 
 config BR2_PACKAGE_WESTON_DEFAULT_X11
 	bool "X11"
@@ -63,11 +63,11 @@ config BR2_PACKAGE_WESTON_DEFAULT_COMPOSITOR
 
 config BR2_PACKAGE_WESTON_DRM
 	bool "DRM compositor"
-	depends on BR2_PACKAGE_MESA3D_OPENGL_EGL
+	depends on BR2_PACKAGE_MESA3D_OPENGL_EGL || BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
 
 # Uses libgbm from mesa3d
 comment "DRM compositor needs an OpenGL EGL backend provided by mesa3d"
-	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL
+	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL && !BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
 
 config BR2_PACKAGE_WESTON_FBDEV
 	bool "fbdev compositor"
-- 
2.26.2

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

* [Buildroot] [PATCH 2/2] package/weston-imx: remove deprecated package
  2020-06-19 12:54 [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
  2020-06-19 12:54 ` [Buildroot] [PATCH 1/2] package/weston: add imx-gpu-viv as possible egl provider Gary Bisson
@ 2020-06-19 12:55 ` Gary Bisson
  2020-07-26 19:55 ` [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
  2020-09-12 12:27 ` Thomas Petazzoni
  3 siblings, 0 replies; 6+ messages in thread
From: Gary Bisson @ 2020-06-19 12:55 UTC (permalink / raw)
  To: buildroot

- the package couldn't be selected with vivante backend anyway since the
split
- the package couldn't be updated to newer versions as it would imply
forking libdrm and wayland-protocols for it to build
- drawback of not using NXP weston is the lack of G2D support, let's
have it as a known issue for Buildroot but use standard weston package

Signed-off-by: Gary Bisson <gary.bisson@boundarydevices.com>
---
 Config.in.legacy                   |   6 ++
 package/Config.in                  |   1 -
 package/weston-imx/Config.in       | 114 -----------------------
 package/weston-imx/weston-imx.hash |   3 -
 package/weston-imx/weston-imx.mk   | 144 -----------------------------
 5 files changed, 6 insertions(+), 262 deletions(-)
 delete mode 100644 package/weston-imx/Config.in
 delete mode 100644 package/weston-imx/weston-imx.hash
 delete mode 100644 package/weston-imx/weston-imx.mk

diff --git a/Config.in.legacy b/Config.in.legacy
index 4074c34c56..f004f42c42 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -152,6 +152,12 @@ config BR2_PACKAGE_KODI_PERIPHERAL_STEAMCONTROLLER
 	help
 	  This package is broken.
 
+config BR2_PACKAGE_WESTON_IMX
+	bool "weston-imx package was removed"
+	select BR2_LEGACY
+	help
+	  This package has been removed, use weston instead.
+
 comment "Legacy options removed in 2020.05"
 
 config BR2_PACKAGE_WIRINGPI
diff --git a/package/Config.in b/package/Config.in
index 7e4476d337..780cca86b1 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -345,7 +345,6 @@ comment "QT libraries and helper libraries"
 endif
 	source "package/tekui/Config.in"
 	source "package/weston/Config.in"
-	source "package/weston-imx/Config.in"
 	source "package/x11r7/Config.in"
 
 comment "X applications"
diff --git a/package/weston-imx/Config.in b/package/weston-imx/Config.in
deleted file mode 100644
index 36ff7ae5ab..0000000000
--- a/package/weston-imx/Config.in
+++ /dev/null
@@ -1,114 +0,0 @@
-comment "weston-imx needs udev and a toolchain w/ locale, threads, dynamic library, headers >= 3.0"
-	depends on !BR2_PACKAGE_HAS_UDEV || !BR2_TOOLCHAIN_HAS_THREADS || \
-		!BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 || BR2_STATIC_LIBS || \
-		!BR2_ENABLE_LOCALE
-	depends on BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
-	depends on !BR2_PACKAGE_WESTON
-
-comment "weston-imx needs imx-gpu-viv with wyalnd output"
-	depends on (BR2_arm && BR2_ARM_EABIHF) || BR2_aarch64 # imx-gpu-viv
-	depends on !BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
-	depends on !BR2_PACKAGE_WESTON
-
-config BR2_PACKAGE_WESTON_IMX
-	bool "weston-imx"
-	depends on !BR2_STATIC_LIBS # wayland
-	depends on BR2_TOOLCHAIN_HAS_THREADS # wayland
-	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0
-	depends on BR2_PACKAGE_HAS_UDEV
-	depends on BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL
-	depends on !BR2_PACKAGE_WESTON
-	select BR2_PACKAGE_WAYLAND
-	select BR2_PACKAGE_WAYLAND_PROTOCOLS
-	select BR2_PACKAGE_LIBXKBCOMMON
-	select BR2_PACKAGE_CAIRO
-	select BR2_PACKAGE_CAIRO_PNG
-	select BR2_PACKAGE_LIBPNG
-	select BR2_PACKAGE_JPEG
-	select BR2_PACKAGE_LIBINPUT
-	select BR2_PACKAGE_LIBDRM
-	# Runtime dependency
-	select BR2_PACKAGE_XKEYBOARD_CONFIG
-	# Make sure at least one compositor is selected.
-	select BR2_PACKAGE_WESTON_IMX_FBDEV if !BR2_PACKAGE_WESTON_IMX_HAS_COMPOSITOR
-	help
-	  Weston is the reference implementation of a Wayland
-	  compositor, and a useful compositor in its own right.
-	  Weston has various backends that lets it run on Linux kernel
-	  modesetting and evdev input as well as under X11.
-
-	  http://wayland.freedesktop.org/
-
-if BR2_PACKAGE_WESTON_IMX
-
-# Helper to make sure at least one compositor is selected.
-config BR2_PACKAGE_WESTON_IMX_HAS_COMPOSITOR
-	bool
-
-config BR2_PACKAGE_WESTON_IMX_DRM
-	bool "DRM compositor"
-	depends on BR2_PACKAGE_MESA3D_OPENGL_EGL
-	select BR2_PACKAGE_WESTON_IMX_HAS_COMPOSITOR
-
-# Uses libgbm from mesa3d
-comment "DRM compositor needs an OpenGL EGL backend provided by mesa3d"
-	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL
-
-config BR2_PACKAGE_WESTON_IMX_FBDEV
-	bool "fbdev compositor"
-
-# FreeRDP needs threads and !static, already the case for weston
-config BR2_PACKAGE_WESTON_IMX_RDP
-	bool "RDP compositor"
-	depends on BR2_USE_MMU # freerdp, libglib2
-	depends on BR2_TOOLCHAIN_HAS_SYNC_4
-	depends on BR2_USE_WCHAR # freerdp
-	depends on BR2_INSTALL_LIBSTDCPP # freerdp
-	select BR2_PACKAGE_FREERDP
-	select BR2_PACKAGE_WESTON_IMX_HAS_COMPOSITOR
-	help
-	  This enables the RDP backend, which allows accessing weston
-	  through the network with any RDP-compliant client.
-
-	  Please note that one must pass those options to weston for RDP
-	  to be functional:
-	    --rdp-tls-cert=/path/to/server.crt
-	    --rdp-tls-key=/path/to/server.key
-
-	  By default, Buildroot installs such files in
-	  /etc/freerdp/server/ so you may want to change them in a
-	  post-build script or a rootfs overlay.
-
-comment "RDP compositor needs a toolchain w/ wchar, C++"
-	depends on BR2_USE_MMU
-	depends on BR2_TOOLCHAIN_HAS_SYNC_4
-	depends on !BR2_USE_WCHAR || !BR2_INSTALL_LIBSTDCPP
-
-config BR2_PACKAGE_WESTON_IMX_X11
-	bool "X11 compositor"
-	depends on BR2_PACKAGE_XORG7
-	select BR2_PACKAGE_LIBXCB
-	select BR2_PACKAGE_XLIB_LIBX11
-	select BR2_PACKAGE_WESTON_IMX_HAS_COMPOSITOR
-
-comment "X11 compositor needs X.org enabled"
-	depends on !BR2_PACKAGE_XORG7
-
-config BR2_PACKAGE_WESTON_IMX_XWAYLAND
-	bool "XWayland support"
-	depends on BR2_PACKAGE_XORG7
-	depends on BR2_PACKAGE_LIBEPOXY
-	select BR2_PACKAGE_CAIRO
-	select BR2_PACKAGE_LIBXCB
-	select BR2_PACKAGE_XLIB_LIBX11
-	select BR2_PACKAGE_XLIB_LIBXCURSOR
-
-comment "XWayland support needs libepoxy and X.org enabled"
-	depends on !BR2_PACKAGE_XORG7 || !BR2_PACKAGE_LIBEPOXY
-
-config BR2_PACKAGE_WESTON_IMX_DEMO_CLIENTS
-	bool "demo clients"
-	help
-	  This enables the installation of Weston's demo clients.
-
-endif
diff --git a/package/weston-imx/weston-imx.hash b/package/weston-imx/weston-imx.hash
deleted file mode 100644
index 56885d71ac..0000000000
--- a/package/weston-imx/weston-imx.hash
+++ /dev/null
@@ -1,3 +0,0 @@
-# locally computed
-sha256  f39f7a24a01b023aa204ad47977a6476c17a307769f29d408621a05b5cbbcfc7  weston-imx-rel_imx_4.9.51_8mq_ga.tar.gz
-sha256  fdb65868f65d0fbdb05c2d3b779e10ce9969fa0c4b9262ba4f260e87086ab860  COPYING
diff --git a/package/weston-imx/weston-imx.mk b/package/weston-imx/weston-imx.mk
deleted file mode 100644
index e803fa3ee2..0000000000
--- a/package/weston-imx/weston-imx.mk
+++ /dev/null
@@ -1,144 +0,0 @@
-################################################################################
-#
-# weston-imx
-#
-################################################################################
-
-WESTON_IMX_VERSION = rel_imx_4.9.51_8mq_ga
-WESTON_IMX_SITE = https://source.codeaurora.org/external/imx/weston-imx
-WESTON_IMX_SITE_METHOD = git
-WESTON_IMX_AUTORECONF = YES
-WESTON_IMX_LICENSE = MIT
-WESTON_IMX_LICENSE_FILES = COPYING
-
-WESTON_IMX_DEPENDENCIES = host-pkgconf wayland wayland-protocols \
-	libxkbcommon pixman libpng jpeg udev cairo libinput libdrm \
-	$(if $(BR2_PACKAGE_WEBP),webp)
-
-WESTON_IMX_CONF_OPTS = \
-	--with-dtddir=$(STAGING_DIR)/usr/share/wayland \
-	--disable-headless-compositor \
-	--disable-colord \
-	--disable-devdocs \
-	--disable-setuid-install
-
-WESTON_IMX_MAKE_OPTS = \
-	WAYLAND_PROTOCOLS_DATADIR=$(STAGING_DIR)/usr/share/wayland-protocols
-
-# Uses VIDIOC_EXPBUF, only available from 3.8+
-ifeq ($(BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_8),)
-WESTON_IMX_CONF_OPTS += --disable-simple-dmabuf-v4l-client
-endif
-
-ifeq ($(BR2_PACKAGE_DBUS),y)
-WESTON_IMX_CONF_OPTS += --enable-dbus
-WESTON_IMX_DEPENDENCIES += dbus
-else
-WESTON_IMX_CONF_OPTS += --disable-dbus
-endif
-
-# weston-launch must be u+s root in order to work properly
-ifeq ($(BR2_PACKAGE_LINUX_PAM),y)
-define WESTON_IMX_PERMISSIONS
-	/usr/bin/weston-launch f 4755 0 0 - - - - -
-endef
-define WESTON_IMX_USERS
-	- - weston-launch -1 - - - - Weston launcher group
-endef
-WESTON_IMX_CONF_OPTS += --enable-weston-launch
-WESTON_IMX_DEPENDENCIES += linux-pam
-else
-WESTON_IMX_CONF_OPTS += --disable-weston-launch
-endif
-
-ifeq ($(BR2_PACKAGE_IMX_GPU_G2D),y)
-WESTON_IMX_DEPENDENCIES += imx-gpu-g2d
-# --enable-imxg2d actually disables it, so no CONF_OPTS
-else
-WESTON_IMX_CONF_OPTS += --disable-imxg2d
-endif
-
-ifeq ($(BR2_PACKAGE_HAS_LIBEGL_WAYLAND)$(BR2_PACKAGE_HAS_LIBGLES),yy)
-WESTON_IMX_CONF_OPTS += --enable-egl
-WESTON_IMX_DEPENDENCIES += libegl libgles
-else
-WESTON_IMX_CONF_OPTS += \
-	--disable-egl \
-	--disable-simple-dmabuf-drm-client \
-	--disable-simple-egl-clients
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_RDP),y)
-WESTON_IMX_DEPENDENCIES += freerdp
-WESTON_IMX_CONF_OPTS += --enable-rdp-compositor
-else
-WESTON_IMX_CONF_OPTS += --disable-rdp-compositor
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_FBDEV),y)
-WESTON_IMX_CONF_OPTS += \
-	--enable-fbdev-compositor \
-	WESTON_IMX_NATIVE_BACKEND=fbdev-backend.so
-else
-WESTON_IMX_CONF_OPTS += --disable-fbdev-compositor
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_DRM),y)
-WESTON_IMX_CONF_OPTS += \
-	--enable-drm-compositor \
-	WESTON_IMX_NATIVE_BACKEND=drm-backend.so
-else
-WESTON_IMX_CONF_OPTS += --disable-drm-compositor
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_X11),y)
-WESTON_IMX_CONF_OPTS += \
-	--enable-x11-compositor \
-	WESTON_IMX_NATIVE_BACKEND=x11-backend.so
-WESTON_IMX_DEPENDENCIES += libxcb xlib_libX11
-else
-WESTON_IMX_CONF_OPTS += --disable-x11-compositor
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_XWAYLAND),y)
-WESTON_IMX_CONF_OPTS += --enable-xwayland
-WESTON_IMX_DEPENDENCIES += cairo libepoxy libxcb xlib_libX11 xlib_libXcursor
-else
-WESTON_IMX_CONF_OPTS += --disable-xwayland
-endif
-
-ifeq ($(BR2_PACKAGE_LIBVA),y)
-WESTON_IMX_CONF_OPTS += --enable-vaapi-recorder
-WESTON_IMX_DEPENDENCIES += libva
-else
-WESTON_IMX_CONF_OPTS += --disable-vaapi-recorder
-endif
-
-ifeq ($(BR2_PACKAGE_LCMS2),y)
-WESTON_IMX_CONF_OPTS += --enable-lcms
-WESTON_IMX_DEPENDENCIES += lcms2
-else
-WESTON_IMX_CONF_OPTS += --disable-lcms
-endif
-
-ifeq ($(BR2_PACKAGE_SYSTEMD),y)
-WESTON_IMX_CONF_OPTS += --enable-systemd-login --enable-systemd-notify
-WESTON_IMX_DEPENDENCIES += systemd
-else
-WESTON_IMX_CONF_OPTS += --disable-systemd-login --disable-systemd-notify
-endif
-
-ifeq ($(BR2_PACKAGE_LIBXML2),y)
-WESTON_IMX_CONF_OPTS += --enable-junit-xml
-WESTON_IMX_DEPENDENCIES += libxml2
-else
-WESTON_IMX_CONF_OPTS += --disable-junit-xml
-endif
-
-ifeq ($(BR2_PACKAGE_WESTON_IMX_DEMO_CLIENTS),y)
-WESTON_IMX_CONF_OPTS += --enable-demo-clients-install
-else
-WESTON_IMX_CONF_OPTS += --disable-demo-clients-install
-endif
-
-$(eval $(autotools-package))
-- 
2.26.2

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

* [Buildroot] [PATCH 0/2] weston: fix i.MX support
  2020-06-19 12:54 [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
  2020-06-19 12:54 ` [Buildroot] [PATCH 1/2] package/weston: add imx-gpu-viv as possible egl provider Gary Bisson
  2020-06-19 12:55 ` [Buildroot] [PATCH 2/2] package/weston-imx: remove deprecated package Gary Bisson
@ 2020-07-26 19:55 ` Gary Bisson
  2020-09-12 12:27 ` Thomas Petazzoni
  3 siblings, 0 replies; 6+ messages in thread
From: Gary Bisson @ 2020-07-26 19:55 UTC (permalink / raw)
  To: buildroot

Hi,

Gentle ping on this series, do you have any questions?

Note that I got i.MX8M VPU to work using regular GStreamer packages +
gtreamer-imx-v2 plugin but is only useful if Weston if fixed (as i.MX8*
graphics only support Wayland backend).

Regards,
Gary

On Fri, Jun 19, 2020 at 02:54:58PM +0200, Gary Bisson wrote:
> Hi,
> 
> This series aims at fixing the weston support for i.MX processors.
> 
> Long time ago, weston-imx was added for i.MX8MQ as part of the weston
> package. Then it was split from it as the versions differed.
> 
> Since then the weston-imx was broken and updating it seems like a bad
> idea as it now requires forking several other components:
> - libdrm [1]
> - waylands-protocols [2]
> 
> This series therefore removes weston-imx and adds vivante support to
> the regular weston package.
> 
> Drawback here is that the HDR capabilities of the i.MX8MQ or the GPU2D
> of the i.MX8M Mini won't be used in Weston. I'd rather have those
> limitations and use standard weston than forking everything.
> 
> Also, note that this series was tested on Nitrogen8M and Nitrogen8M Mini
> platforms, making sure the Vivante 3D examples are properly running.
> However in order to compile weston properly with Vivante as egl
> provider, one patch sent earlier [3] is needed.
> 
> Let me know if you have any questions.
> 
> Regards,
> Gary
> 
> [1] https://source.codeaurora.org/external/imx/meta-imx/tree/meta-bsp/recipes-graphics/drm/libdrm_2.4.99.imx.bb?h=zeus-5.4.3-1.0.0
> [2] https://source.codeaurora.org/external/imx/meta-imx/tree/meta-bsp/recipes-graphics/wayland/wayland-protocols_1.18.imx.bb?h=zeus-5.4.3-1.0.0
> [3] http://patchwork.ozlabs.org/project/buildroot/patch/20200402130842.918696-3-gary.bisson at boundarydevices.com/
> 
> Gary Bisson (2):
>   package/weston: add imx-gpu-viv as possible egl provider
>   package/weston-imx: remove deprecated package
> 
>  Config.in.legacy                   |   6 ++
>  package/Config.in                  |   1 -
>  package/weston-imx/Config.in       | 114 -----------------------
>  package/weston-imx/weston-imx.hash |   3 -
>  package/weston-imx/weston-imx.mk   | 144 -----------------------------
>  package/weston/Config.in           |   8 +-
>  6 files changed, 10 insertions(+), 266 deletions(-)
>  delete mode 100644 package/weston-imx/Config.in
>  delete mode 100644 package/weston-imx/weston-imx.hash
>  delete mode 100644 package/weston-imx/weston-imx.mk
> 
> -- 
> 2.26.2
> 

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

* [Buildroot] [PATCH 0/2] weston: fix i.MX support
  2020-06-19 12:54 [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
                   ` (2 preceding siblings ...)
  2020-07-26 19:55 ` [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
@ 2020-09-12 12:27 ` Thomas Petazzoni
  2020-09-12 16:18   ` Gary Bisson
  3 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni @ 2020-09-12 12:27 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 19 Jun 2020 14:54:58 +0200
Gary Bisson <gary.bisson@boundarydevices.com> wrote:

> Gary Bisson (2):
>   package/weston: add imx-gpu-viv as possible egl provider
>   package/weston-imx: remove deprecated package

Thanks, series applied.

For the first patch, I think we need to add a proper libgbm virtual
package. I had a patch series for that, but it's not trivial as not all
gbm implementations implement the full/recent GBM API. I had some ideas
to address that, but needs some more investigation/time.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 0/2] weston: fix i.MX support
  2020-09-12 12:27 ` Thomas Petazzoni
@ 2020-09-12 16:18   ` Gary Bisson
  0 siblings, 0 replies; 6+ messages in thread
From: Gary Bisson @ 2020-09-12 16:18 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Sat, Sep 12, 2020 at 02:27:13PM +0200, Thomas Petazzoni wrote:
> Hello,
> 
> On Fri, 19 Jun 2020 14:54:58 +0200
> Gary Bisson <gary.bisson@boundarydevices.com> wrote:
> 
> > Gary Bisson (2):
> >   package/weston: add imx-gpu-viv as possible egl provider
> >   package/weston-imx: remove deprecated package
> 
> Thanks, series applied.

Thanks, that is awesome!

> For the first patch, I think we need to add a proper libgbm virtual
> package. I had a patch series for that, but it's not trivial as not all
> gbm implementations implement the full/recent GBM API. I had some ideas
> to address that, but needs some more investigation/time.

Yes that would indeed be a better solution, I've noticed other packages
that have added dependency on Mesa3d for libgbm (kmscube, glmark2
pending patch[1]).

From the cover letter of this series I also mentioned another patch sent
before that is necessary for a proper build of Weston with imx-gpu-viv
[2].
I thought that was that patch that was problematic as, as you pointed
out already, the sed isn't an elegant solution but I still believe it's
the less intrusive one.
Let me know what you think of it.

Regards,
Gary

[1] http://patchwork.ozlabs.org/project/buildroot/patch/20200104193919.91589-2-bernd.kuhls at t-online.de/
[2] http://patchwork.ozlabs.org/project/buildroot/patch/20200402130842.918696-3-gary.bisson at boundarydevices.com/

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

end of thread, other threads:[~2020-09-12 16:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-19 12:54 [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
2020-06-19 12:54 ` [Buildroot] [PATCH 1/2] package/weston: add imx-gpu-viv as possible egl provider Gary Bisson
2020-06-19 12:55 ` [Buildroot] [PATCH 2/2] package/weston-imx: remove deprecated package Gary Bisson
2020-07-26 19:55 ` [Buildroot] [PATCH 0/2] weston: fix i.MX support Gary Bisson
2020-09-12 12:27 ` Thomas Petazzoni
2020-09-12 16:18   ` Gary Bisson

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.