Skip to contentSkip to conversation

libv4l: update v4l-utils help text (add more listed tools)

PATCH Superseded Peter Seiderer 3 participants 7 messages Oct 6, 2015 – Oct 7, 2015

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

Peter Seiderer

author
#0 lore raw

PATCH 1/3 libv4l: update v4l-utils help text (add more listed tools)

Signed-off-by Peter Seiderer <[email protected]>
---
 package/libv4l/Config.in | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)
package/libv4l/Config.in +10 -2
index d137613..5a55b2b 100644
@@ -26,8 +26,16 @@ config BR2_PACKAGE_LIBV4L_UTILS
 	help
 	  v4l-utils is a collection of various video4linux and DVB utilities.
 	  Enable this if you want to build the following tools:
-	  cx18-ctl, ir-keytable, ivtv-ctl, v4l2-ctl, v4l2-sysfs-path and
-	  others.
+	  - cx18-ctl
+	  - dvb-fe-tool, dvbv5-zap, dvbv5-scan, dvb-format-convert
+	  - decode_tm6000
+	  - ir-keytable
+	  - media-ctl
+	  - v4l2-compliance
+	  - v4l2-ctl, cx18-ctl, ivtv-ctl
+	  - v4l2-sysfs-path
+	  - rds-ctl
+	  - qv4l2 (if Qt is enabled)
 
 endif
Signature · 1 line
2.1.4
author
#1 lore raw

PATCH 2/3 media-ctl: remove package

Package media-ctl is out-dated, source (and developement) have been moved to v4l-utils since June 2014. Up-to-date version is available in the libv4l package (BR2_PACKAGE_LIBV4L/BR2_PACKAGE_LIBV4L_UTILS).

Signed-off-by Peter Seiderer <[email protected]>
---
 Config.in.legacy                                   |   7 +
 package/Config.in                                  |   1 -
 ...ers-for-compatibility-with-old-toolchains.patch | 303 ---------------------
 package/media-ctl/Config.in                        |   9 -
 package/media-ctl/media-ctl.mk                     |  17 --
 5 files changed, 7 insertions(+), 330 deletions(-)
 delete mode 100644 package/media-ctl/0001-add-kernel-headers-for-compatibility-with-old-toolchains.patch
 delete mode 100644 package/media-ctl/Config.in
 delete mode 100644 package/media-ctl/media-ctl.mk
Config.in.legacy +7 -0
index 3c7fe36..4b7e201 100644
@@ -107,6 +107,13 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2015.11"
 
+config BR2_PACKAGE_MEDIA_CTL
+	bool "media-ctl package has been removed"
+	help
+	  media-ctl source and developement have been moved to
+	  v4l-utils since June 2014. For an up-to-date media-ctl
+	  version select BR2_PACKAGE_LIBV4L and BR2_PACKAGE_LIBV4L_UTILS.
+
 config BR2_PACKAGE_SCHIFRA
 	bool "schifra package has been removed"
 	help
package/Config.in +0 -1
index 3794f44..74f1ffc 100644
@@ -376,7 +376,6 @@ endif
 	source "package/lsuio/Config.in"
 	source "package/lvm2/Config.in"
 	source "package/mdadm/Config.in"
-	source "package/media-ctl/Config.in"
 	source "package/memtest86/Config.in"
 	source "package/memtester/Config.in"
 	source "package/minicom/Config.in"
package/media-ctl/0001-add-kernel-headers-for-compatibility-with-old-toolchains.patch deleted +0 -303

303 lines not on this page · raw message

package/media-ctl/Config.in deleted +0 -9
deleted file mode 100644
index 8d1beda..0000000
@@ -1,9 +0,0 @@
-config BR2_PACKAGE_MEDIA_CTL
-	bool "media-ctl"
-	help
-	  Media controller control application
-
-	  Useful mostly with ARM OMAP processors to configure and
-	  control the image subsystem
-
-	  http://git.ideasonboard.org/media-ctl.git
package/media-ctl/media-ctl.mk deleted +0 -17
deleted file mode 100644
index ae0d470..0000000
@@ -1,17 +0,0 @@
-################################################################################
-#
-# media-ctl
-#
-################################################################################
-
-MEDIA_CTL_VERSION = 87f58c2ed885d636229f7209fde883058433a1ce
-MEDIA_CTL_SITE = git://git.ideasonboard.org/media-ctl.git
-MEDIA_CTL_INSTALL_STAGING = YES
-MEDIA_CTL_LICENSE = LGPLv2.1+
-MEDIA_CTL_LICENSE_FILES = COPYING.LIB
-
-MEDIA_CTL_AUTORECONF = YES
-MEDIA_CTL_CONF_OPTS = --with-kernel-headers=$(STAGING_DIR)/usr/include
-MEDIA_CTL_DEPENDENCIES = host-pkgconf
-
-$(eval $(autotools-package))
Signature · 1 line
2.1.4
#2 lore raw

PATCH 2/3 media-ctl: remove package

Quoted 21 lines · Peter Seiderer

Package media-ctl is out-dated, source (and developement) have been moved to v4l-utils since June 2014. Up-to-date version is available in the libv4l package (BR2_PACKAGE_LIBV4L/BR2_PACKAGE_LIBV4L_UTILS).

Signed-off-by Peter Seiderer <[email protected]>
---
 Config.in.legacy                                   |   7 +
 package/Config.in                                  |   1 -
 ...ers-for-compatibility-with-old-toolchains.patch | 303 ---------------------
 package/media-ctl/Config.in                        |   9 -
 package/media-ctl/media-ctl.mk                     |  17 --
 5 files changed, 7 insertions(+), 330 deletions(-)
 delete mode 100644 package/media-ctl/0001-add-kernel-headers-for-compatibility-with-old-toolchains.patch
 delete mode 100644 package/media-ctl/Config.in
 delete mode 100644 package/media-ctl/media-ctl.mk
Config.in.legacy +0 -0
index 3c7fe36..4b7e201 100644

Config.in.legacy endif · line 115

Quoted 9 lines
Config.in.legacy +6 -0
@@ -107,6 +107,13 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2015.11"
 
+config BR2_PACKAGE_MEDIA_CTL
+	bool "media-ctl package has been removed"
+	help
+	  media-ctl source and developement have been moved to
+	  v4l-utils since June 2014. For an up-to-date media-ctl
+	  version select BR2_PACKAGE_LIBV4L and BR2_PACKAGE_LIBV4L_UTILS.

These two options should be selected automatically. See e.g.
BR2_PACKAGE_DIRECTFB_EXAMPLES_ANDI

You should also select BR2_LEGACY.

Config.in.legacy endif · line 118

Quoted 3 lines
Config.in.legacy +1 -0
+
 config BR2_PACKAGE_SCHIFRA
 	bool "schifra package has been removed"

Oops, here BR2_LEGACY is not selected... And in a couple of other places as well...

Regards,
Arnout

Quoted 1 line
Config.in.legacy +0 -0
 	help
Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
author
#3 lore raw

PATCH 2/3 media-ctl: remove package

Hello Arnout,

Quoted 35 lines · Arnout Vandecappelle
Quoted 21 lines · Peter Seiderer

Package media-ctl is out-dated, source (and developement) have been moved to v4l-utils since June 2014. Up-to-date version is available in the libv4l package (BR2_PACKAGE_LIBV4L/BR2_PACKAGE_LIBV4L_UTILS).

Signed-off-by Peter Seiderer <[email protected]>
---
 Config.in.legacy                                   |   7 +
 package/Config.in                                  |   1 -
 ...ers-for-compatibility-with-old-toolchains.patch | 303 ---------------------
 package/media-ctl/Config.in                        |   9 -
 package/media-ctl/media-ctl.mk                     |  17 --
 5 files changed, 7 insertions(+), 330 deletions(-)
 delete mode 100644 package/media-ctl/0001-add-kernel-headers-for-compatibility-with-old-toolchains.patch
 delete mode 100644 package/media-ctl/Config.in
 delete mode 100644 package/media-ctl/media-ctl.mk
Config.in.legacy +0 -0
index 3c7fe36..4b7e201 100644

Config.in.legacy endif · line 115

Quoted 9 lines
Config.in.legacy +6 -0
@@ -107,6 +107,13 @@ endif
 ###############################################################################
 comment "Legacy options removed in 2015.11"
 
+config BR2_PACKAGE_MEDIA_CTL
+	bool "media-ctl package has been removed"
+	help
+	  media-ctl source and developement have been moved to
+	  v4l-utils since June 2014. For an up-to-date media-ctl
+	  version select BR2_PACKAGE_LIBV4L and BR2_PACKAGE_LIBV4L_UTILS.

These two options should be selected automatically. See e.g.
BR2_PACKAGE_DIRECTFB_EXAMPLES_ANDI

Quoted 2 lines

You should also select BR2_LEGACY.

O.k., will fix it...

Quoted 5 lines

Config.in.legacy endif · line 118

Quoted 3 lines
Config.in.legacy +1 -0
+
 config BR2_PACKAGE_SCHIFRA
 	bool "schifra package has been removed"

Oops, here BR2_LEGACY is not selected... And in a couple of other places as well...

O.k., will take a look at it too...

Regards,
Peter

Quoted 6 lines

Regards,
Arnout

Quoted 1 line
Config.in.legacy +0 -0
 	help
author
#4 lore raw

PATCH 3/3 libv4l: bump version to 1.8.0

In reply to Peter Seiderer

And add proper qt5base/qt depndency (needed for qv4l2, qt5 support added since 1.8.0).

Signed-off-by Peter Seiderer <[email protected]>
---
 package/libv4l/libv4l.hash | 2 +-
 package/libv4l/libv4l.mk   | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)
package/libv4l/libv4l.hash +1 -1
index 3371d34..1395eda 100644
@@ -1,2 +1,2 @@
 # Locally calculated
-sha256 164abf5c1befcd27e8e6ef824a82d4015bdfb5d99ae82daa00e77d895ff9864c  v4l-utils-1.6.3.tar.bz2
+sha256 50c7be033636d878e902dad0a207fc9f6d831bec27f1b410f1102acddaa5565e  v4l-utils-1.8.0.tar.bz2
package/libv4l/libv4l.mk +4 -1
index edb2a36..7978341 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LIBV4L_VERSION = 1.6.3
+LIBV4L_VERSION = 1.8.0
 LIBV4L_SOURCE = v4l-utils-$(LIBV4L_VERSION).tar.bz2
 LIBV4L_SITE = http://linuxtv.org/downloads/v4l-utils
 LIBV4L_INSTALL_STAGING = YES

@@ -40,6 +40,9 @@ LIBV4L_CONF_OPTS += --without-libudev
 endif
 
 ifeq ($(BR2_PACKAGE_LIBV4L_UTILS),y)
+ifneq ($(BR2_PACKAGE_QT5BASE)$(BR2_PACKAGE_QT),)
+LIBV4L_DEPENDENCIES += $(if $(BR2_PACKAGE_QT5BASE),qt5base,qt)
+endif
 LIBV4L_CONF_OPTS += --enable-v4l-utils
 # clock_gettime is used, which is provided by librt for glibc < 2.17
 LIBV4L_LIBS += -lrt
Signature · 1 line
2.1.4
#5 lore raw

PATCH 1/3 libv4l: update v4l-utils help text (add more listed tools)

In reply to Peter Seiderer

Dear Peter Seiderer,

Quoted 15 lines · Peter Seiderer
Signed-off-by Peter Seiderer <[email protected]>
---
 package/libv4l/Config.in | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)
package/libv4l/Config.in +0 -2
index d137613..5a55b2b 100644
@@ -26,8 +26,16 @@ config BR2_PACKAGE_LIBV4L_UTILS
 	help
 	  v4l-utils is a collection of various video4linux and DVB utilities.
 	  Enable this if you want to build the following tools:
-	  cx18-ctl, ir-keytable, ivtv-ctl, v4l2-ctl, v4l2-sysfs-path and
-	  others.

package/libv4l/Config.in config BR2_PACKAGE_LIBV4L_UTILS · line 38

Quoted 10 lines
package/libv4l/Config.in +10 -0
+	  - cx18-ctl
+	  - dvb-fe-tool, dvbv5-zap, dvbv5-scan, dvb-format-convert
+	  - decode_tm6000
+	  - ir-keytable
+	  - media-ctl
+	  - v4l2-compliance
+	  - v4l2-ctl, cx18-ctl, ivtv-ctl
+	  - v4l2-sysfs-path
+	  - rds-ctl
+	  - qv4l2 (if Qt is enabled)

To make the things more future-proof, shouldn't we remove the list of tools, and just say "Enable this option to enable various tools to control Video4Linux devices" (or some other better wording) ?

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
author
#6 lore raw

PATCH 1/3 libv4l: update v4l-utils help text (add more listed tools)

Hello Thomas,

Quoted 24 lines · Thomas Petazzoni

Dear Peter Seiderer,

Quoted 21 lines · Peter Seiderer
Signed-off-by Peter Seiderer <[email protected]>
---
 package/libv4l/Config.in | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)
package/libv4l/Config.in +6 -2
index d137613..5a55b2b 100644
@@ -26,8 +26,16 @@ config BR2_PACKAGE_LIBV4L_UTILS
 	help
 	  v4l-utils is a collection of various video4linux and DVB utilities.
 	  Enable this if you want to build the following tools:
-	  cx18-ctl, ir-keytable, ivtv-ctl, v4l2-ctl, v4l2-sysfs-path and
-	  others.
+	  - cx18-ctl
+	  - dvb-fe-tool, dvbv5-zap, dvbv5-scan, dvb-format-convert
+	  - decode_tm6000
+	  - ir-keytable
+	  - media-ctl
+	  - v4l2-compliance
Quoted 8 lines

package/libv4l/Config.in config BR2_PACKAGE_LIBV4L_UTILS · line 38

Quoted 4 lines
package/libv4l/Config.in +4 -0
+	  - v4l2-ctl, cx18-ctl, ivtv-ctl
+	  - v4l2-sysfs-path
+	  - rds-ctl
+	  - qv4l2 (if Qt is enabled)

To make the things more future-proof, shouldn't we remove the list of tools, and just say "Enable this option to enable various tools to control Video4Linux devices" (or some other better wording) ?

This has pro and cons, my reasoning was to mention media-ctl somewhere before removing the superfluously extra media-ctl package (as hint for people who are not that familiar with the v4l-utils package)...

Regards,
Peter

Quoted 2 lines

Thomas