Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available
@ 2020-06-08  0:00 Ezequiel Garcia
  2020-07-02 20:54 ` Ezequiel Garcia
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Ezequiel Garcia @ 2020-06-08  0:00 UTC (permalink / raw)
  To: buildroot

From: Nicolas Dufresne <nicolas.dufresne@collabora.com>

Video4Linux2 plugins can udev for device probing and monitor.
This greatly improves load time and monitoring performance.

It also enables hotplug monitoring for cameras.

Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
---
v3:
* Apply the dependency only if V4L2 plugins are enabled
v2:
* Enable/disable v4l2-gudev explicitly

 package/gstreamer1/gst1-plugins-good/Config.in            | 1 +
 package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk | 7 +++++++
 2 files changed, 8 insertions(+)

diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
index 20aebc473e89..be77f5a82e3d 100644
--- a/package/gstreamer1/gst1-plugins-good/Config.in
+++ b/package/gstreamer1/gst1-plugins-good/Config.in
@@ -295,6 +295,7 @@ config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
 config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
 	bool "v4l2"
 	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
+	select BR2_PACKAGE_LIBGUDEV if BR2_PACKAGE_HAS_UDEV
 	help
 	  elements for Video 4 Linux
 
diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
index 223af9850897..7bb2fa5f3720 100644
--- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
+++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
@@ -355,6 +355,13 @@ endif
 
 ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
 GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=enabled
+# Enable use of gudev if available, for device probing and monitoring.
+ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
+GST1_PLUGINS_GOOD_DEPENDENCIES += libgudev
+GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=enabled
+else
+GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=disabled
+endif
 else
 GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=disabled
 endif
-- 
2.26.0.rc2

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

* [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available
  2020-06-08  0:00 [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available Ezequiel Garcia
@ 2020-07-02 20:54 ` Ezequiel Garcia
  2020-07-02 21:51 ` Yann E. MORIN
  2020-07-20  6:17 ` Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Ezequiel Garcia @ 2020-07-02 20:54 UTC (permalink / raw)
  To: buildroot

Gentle ping :-)

On Sun, 7 Jun 2020 at 21:01, Ezequiel Garcia
<ezequiel@vanguardiasur.com.ar> wrote:
>
> From: Nicolas Dufresne <nicolas.dufresne@collabora.com>
>
> Video4Linux2 plugins can udev for device probing and monitor.
> This greatly improves load time and monitoring performance.
>
> It also enables hotplug monitoring for cameras.
>
> Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
> Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
> ---
> v3:
> * Apply the dependency only if V4L2 plugins are enabled
> v2:
> * Enable/disable v4l2-gudev explicitly
>
>  package/gstreamer1/gst1-plugins-good/Config.in            | 1 +
>  package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk | 7 +++++++
>  2 files changed, 8 insertions(+)
>
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index 20aebc473e89..be77f5a82e3d 100644
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -295,6 +295,7 @@ config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
>         bool "v4l2"
>         depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> +       select BR2_PACKAGE_LIBGUDEV if BR2_PACKAGE_HAS_UDEV
>         help
>           elements for Video 4 Linux
>
> diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> index 223af9850897..7bb2fa5f3720 100644
> --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> @@ -355,6 +355,13 @@ endif
>
>  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
>  GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=enabled
> +# Enable use of gudev if available, for device probing and monitoring.
> +ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> +GST1_PLUGINS_GOOD_DEPENDENCIES += libgudev
> +GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=enabled
> +else
> +GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=disabled
> +endif
>  else
>  GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=disabled
>  endif
> --
> 2.26.0.rc2
>

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

* [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available
  2020-06-08  0:00 [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available Ezequiel Garcia
  2020-07-02 20:54 ` Ezequiel Garcia
@ 2020-07-02 21:51 ` Yann E. MORIN
  2020-07-20  6:17 ` Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2020-07-02 21:51 UTC (permalink / raw)
  To: buildroot

Nicolas, Ezequiel, All,

On 2020-06-07 21:00 -0300, Ezequiel Garcia spake thusly:
> From: Nicolas Dufresne <nicolas.dufresne@collabora.com>
> 
> Video4Linux2 plugins can udev for device probing and monitor.
> This greatly improves load time and monitoring performance.
> 
> It also enables hotplug monitoring for cameras.

Here, I've added a blurb explaining why we do not propagate the
dependencies from libgudev eventhough we select it.

I am always a bit conflicted that we don't do that explicitly, even if
superfluous.

But heck, the libglib2 world is not ready to drop the existing
dependencies (MMU, wchar, and threads) anytime soon, so this ought
to be enough for everybody (famous last words).

> Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
> Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
> v3:
> * Apply the dependency only if V4L2 plugins are enabled
> v2:
> * Enable/disable v4l2-gudev explicitly
> 
>  package/gstreamer1/gst1-plugins-good/Config.in            | 1 +
>  package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk | 7 +++++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/package/gstreamer1/gst1-plugins-good/Config.in b/package/gstreamer1/gst1-plugins-good/Config.in
> index 20aebc473e89..be77f5a82e3d 100644
> --- a/package/gstreamer1/gst1-plugins-good/Config.in
> +++ b/package/gstreamer1/gst1-plugins-good/Config.in
> @@ -295,6 +295,7 @@ config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_SHOUT2
>  config BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2
>  	bool "v4l2"
>  	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 # multi planar api
> +	select BR2_PACKAGE_LIBGUDEV if BR2_PACKAGE_HAS_UDEV
>  	help
>  	  elements for Video 4 Linux
>  
> diff --git a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> index 223af9850897..7bb2fa5f3720 100644
> --- a/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> +++ b/package/gstreamer1/gst1-plugins-good/gst1-plugins-good.mk
> @@ -355,6 +355,13 @@ endif
>  
>  ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_PLUGIN_V4L2),y)
>  GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=enabled
> +# Enable use of gudev if available, for device probing and monitoring.
> +ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> +GST1_PLUGINS_GOOD_DEPENDENCIES += libgudev
> +GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=enabled
> +else
> +GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2-gudev=disabled
> +endif
>  else
>  GST1_PLUGINS_GOOD_CONF_OPTS += -Dv4l2=disabled
>  endif
> -- 
> 2.26.0.rc2
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available
  2020-06-08  0:00 [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available Ezequiel Garcia
  2020-07-02 20:54 ` Ezequiel Garcia
  2020-07-02 21:51 ` Yann E. MORIN
@ 2020-07-20  6:17 ` Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2020-07-20  6:17 UTC (permalink / raw)
  To: buildroot

>>>>> "Ezequiel" == Ezequiel Garcia <ezequiel@vanguardiasur.com.ar> writes:

 > From: Nicolas Dufresne <nicolas.dufresne@collabora.com>
 > Video4Linux2 plugins can udev for device probing and monitor.
 > This greatly improves load time and monitoring performance.

 > It also enables hotplug monitoring for cameras.

 > Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
 > Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
 > ---
 > v3:
 > * Apply the dependency only if V4L2 plugins are enabled
 > v2:
 > * Enable/disable v4l2-gudev explicitly

Committed to 2020.02.x and 2020.05.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2020-07-20  6:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-08  0:00 [Buildroot] [PATCH v3] package/gstreamer1/gst1-plugins-good: Enable GUDEV for V4L2 if available Ezequiel Garcia
2020-07-02 20:54 ` Ezequiel Garcia
2020-07-02 21:51 ` Yann E. MORIN
2020-07-20  6:17 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox