Hans Verkuil [Thu, 3 Jul 2014 05:52:54 +0000 (07:52 +0200)]
qv4l2: GL_RED is not supported everywhere, use GL_LUMINANCE.
While GL_RED works fine on my nvidia card, it fails when used under
vmware. Replace by GL_LUMINANCE which is accepted by both.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Wed, 2 Jul 2014 20:30:18 +0000 (22:30 +0200)]
qv4l2: colorspace & opengl improvements
Handle unknown colorspaces gracefully and refactor the default
pixelformat handling in the opengl renderer.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Wed, 2 Jul 2014 16:52:02 +0000 (18:52 +0200)]
v4l2-compliance: improve colorspace checks.
Check that V4L2_COLORSPACE_BT878 is never used and that V4L2_COLORSPACE_JPEG
is only used in combination with V4L2_PIX_FMT_JPEG.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Wed, 2 Jul 2014 14:40:43 +0000 (16:40 +0200)]
qv4l2: add extensive colorspace support
Support for colorspace handling has been added to the qv4l2 tool.
Two new comboboxes have been added: one to select the colorspace
of the captured video (by default it will use the colorspace as
returned by the driver), and one to select the colorspace of the
display which is sRGB by default.
Special care has been taken to do the correct Y'CbCr to R'G'B'
conversion, the R'G'B' to RGB conversion, the actual colorspace
conversion and the conversion to the display colorspace.
Note that colorspace handling is only supported with the openGL
acceleration. Doing this in the software implementation would be
much too slow.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Mauro Carvalho Chehab [Tue, 1 Jul 2014 00:55:26 +0000 (21:55 -0300)]
dvbv5-zap: Fix LNBf selection
There were a number of issues with LNBf selection:
1) the test if lnb parameter was specified was wrong;
2) if LNB is provided on both channel file and command line, use
the latest.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Gregor Jasny [Sun, 29 Jun 2014 10:30:41 +0000 (12:30 +0200)]
Fix kFreeBSD build
* fixed BSDfication of V4L related headers
* Disabled libdvbv5 and v4l-utils build
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 17:39:14 +0000 (19:39 +0200)]
qv4l2: Fix typo
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:48:56 +0000 (18:48 +0200)]
Revert "libdvbv5: Do not install libdvbv5 by default"
This reverts commit
4fe1165ab5239bb2fadeed60f9cc8f0f1d6253f5.
Gregor Jasny [Sat, 28 Jun 2014 16:48:47 +0000 (18:48 +0200)]
Start v4l-utils 1.3.x development
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 15:59:10 +0000 (17:59 +0200)]
Prepare for 1.2.0 release
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:24:22 +0000 (18:24 +0200)]
libdvbv5: Do not install libdvbv5 by default
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:21:14 +0000 (18:21 +0200)]
qv4l2: Distribute manpage
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:16:58 +0000 (18:16 +0200)]
v4l2-compliance: Distribute internal header, too
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:16:30 +0000 (18:16 +0200)]
buildsystem: Distribute Android.mk files
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:12:44 +0000 (18:12 +0200)]
mediactl: Distribute internal header, too
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Sat, 28 Jun 2014 16:08:04 +0000 (18:08 +0200)]
libdvbv5: Distribute internal header, too
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Mauro Carvalho Chehab [Sat, 28 Jun 2014 12:45:06 +0000 (09:45 -0300)]
parse_dib0700.pl: parse REQUEST_GET_VERSION
Identify the firmware version on the parsed logs.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Sat, 28 Jun 2014 12:25:22 +0000 (09:25 -0300)]
parse_dib0700.pl: Parse REQUEST_SET_I2C_PARAM
Parses I2C speed changes.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Hans Verkuil [Fri, 27 Jun 2014 09:03:51 +0000 (11:03 +0200)]
v4l2-compliance: M2M cap + input or output caps should be an error.
Currently this is a warning, but this is now an obsolete combination
and should never be used.
In the very beginning when m2m devices were introduced they were marked
as capture+output devices, but some applications scan video devices for
those that have the CAPTURE cap set (quite reasonable), and they would
also match such m2m devices. Quite soon we realized that this was a
problem and we introduced the m2m cap.
Because of this the combination of M2M + CAPTURE or OUTPUT caps is
illegal and should not be used anymore.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Laurent Pinchart [Wed, 25 Jun 2014 09:37:24 +0000 (11:37 +0200)]
Revert "wip"
The commit was pushed by mistake, revert it.
This reverts commit
83d0df45745873f856c8a63bf4a86fec70559fca.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 21:47:07 +0000 (18:47 -0300)]
parse_dib0700.pl: autoflush writes
Autoflushing avoids loosing message when CTRL-C is pressed.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 21:42:59 +0000 (18:42 -0300)]
parse_dib0700.pl: Make the delays multiple of 10ms
That makes it more real to what the Kernel would provide.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 20:31:21 +0000 (17:31 -0300)]
parse_dib0700.pl: add support to identify delays
This could be relevant for GPIO settings and similar stuff.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 19:34:26 +0000 (16:34 -0300)]
parse_dib0700.pl: improve parser
When using ./parse_tcpdump_log.pl|parse_dib0700.pl, it were not
parsing properly.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 17:28:10 +0000 (14:28 -0300)]
parse_dib0700.pl: parse the remaining messages on dib0700
Parse the remaining messages on dib0700, except for the newer
I2C read/write formats.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Tue, 24 Jun 2014 16:29:20 +0000 (13:29 -0300)]
contrib: add a parser for dib0700
Add an initial parser for dib0700. While here, change the file
permisions of contrib scripts to be executable.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Hans Verkuil [Mon, 23 Jun 2014 14:14:54 +0000 (16:14 +0200)]
qv4l2: fix opengl support for RGB32/BGR32.
For whatever reason the GL_RGBA4 internal format was used instead of
GL_RGB8, so each color component was just 4 bits instead of 8.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 23 Jun 2014 07:33:53 +0000 (09:33 +0200)]
qv4l2-qt3: remove utility
qv4l2-qt3 has been superseded by qv4l2. Nobody uses Qt3 anymore and this
qt3 version of qv4l2 hasn't been maintained for a long time. Time to
retire it.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 23 Jun 2014 07:32:06 +0000 (09:32 +0200)]
v4lgrab/vbi-test: remove these utilities
v4lgrab doesn't work anymore since the v4l1 compat layer has been
removed a long time ago.
vbi-test has been replaced by v4l2-ctl --get-fmt-vbi and serves no good
purpose anymore.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Laurent Pinchart [Sat, 21 Jun 2014 08:26:47 +0000 (10:26 +0200)]
media-ctl: Pass a positive error value to strerror()
Negative error values are passed to the strerror() function which takes
positive error value arguments. Fix it.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Wed, 31 Jul 2013 22:58:52 +0000 (00:58 +0200)]
wip
Laurent Pinchart [Tue, 3 Jun 2014 10:10:52 +0000 (12:10 +0200)]
Update sync-with-kernel to use installed kernel headers
Kernel headers exported to userspace can contain kernel-specific
statements (such as __user annotations) that are removed when installing
the headers with 'make headers_install' in the kernel sources. Only
those headers must be used by userspace, raw headers are private to the
kernel.
Update the sync-with-kernel make target to use the installed headers.
The user must install the kernel headers by running
make headers_install
in KERNEL_DIR prior to run sync-with-kernel.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Fri, 31 Jan 2014 13:21:57 +0000 (14:21 +0100)]
media-ctl: Add DV timings support
Support printing (with -p) and setting (with --set-dv) DV timings at the
pad level.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Tue, 4 Feb 2014 10:22:15 +0000 (11:22 +0100)]
media-ctl: Move flags printing code to a new print_flags function
This will allow reusing the flag printing code for the DV timings flags.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Mon, 2 Jun 2014 14:49:15 +0000 (16:49 +0200)]
media-ctl: libv4l2subdev: Add DV timings support
Expose the pad-level get caps, query, get and set DV timings ioctls.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Antonio Ospite [Mon, 16 Jun 2014 15:00:41 +0000 (17:00 +0200)]
libv4lconvert: Fix a regression when converting from Y10B
Fix a regression introduced in commit
efc29f1764a30808ebf7b3e1d9bfa27b909bf641 (libv4lconvert: Reject too
short source buffer before accessing it).
The old code:
case V4L2_PIX_FMT_Y10BPACK:
...
if (result == 0 && src_size < (width * height * 10 / 8)) {
V4LCONVERT_ERR("short y10b data frame\n");
errno = EPIPE;
result = -1;
}
...
meant to say "If the conversion was *successful* _but_ the frame size
was invalid, then take the error path", but in
efc29f1764a30808ebf7b3e1d9bfa27b909bf641 this (maybe weird) logic was
misunderstood and v4lconvert_convert_pixfmt() was made to return an
error even in the case of a successful conversion from Y10B.
Drop the no longer needed second check to fix this.
Signed-off-by: Antonio Ospite <ao2@ao2.it>
Cc: Gregor Jasny <gjasny@googlemail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hans Verkuil [Tue, 17 Jun 2014 11:09:33 +0000 (13:09 +0200)]
v4l2-ctl: add support to try/set raw VBI formats
This was the last bit of missing functionality from v4l2-ctl, now that
this is in the full V4L2 API functionality is available through v4l2-ctl.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Thiago Santos [Mon, 9 Jun 2014 13:51:56 +0000 (10:51 -0300)]
libv4l2: release the lock before doing a DQBUF
In blocking mode, if there are no buffers available the DQBUF will block
waiting for a QBUF to be called but it will block holding the streaming
lock which will prevent any QBUF from happening, causing a deadlock.
Signed-off-by: Thiago Santos <ts.santos@sisa.samsung.com>
Acked-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Fri, 13 Jun 2014 14:07:35 +0000 (16:07 +0200)]
v4l2-compliance: don't warn about missing frameintervals
Only warn about missing ENUM_FRAMEINTERVALS support for discrete
framesizes, not for stepwise or continuous. Those framesize types
can be used as well for video receivers where ENUM_FRAMEINTERVALS
may not make sense.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Fri, 13 Jun 2014 13:14:51 +0000 (15:14 +0200)]
qv4l2: add support for the V4L2_EVENT_SOURCE_CHANGE event.
Subscribe to the V4L2_EVENT_SOURCE_CHANGE event. When such an event
arrives for the current input the application will call QUERYSTD or
QUERY_DV_TIMINGS to update the standard/timings.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Thu, 12 Jun 2014 13:58:23 +0000 (15:58 +0200)]
v4l2-compliance: fix broken querymenu check.
QUERYMENU can also return -ENOTTY if it isn't implemented at all.
Allow for that.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans de Goede [Thu, 8 May 2014 19:28:27 +0000 (21:28 +0200)]
rc_keymaps: Add keymap for the remote shipped with the Wobo i5 tv box
The Wobo i5 is an allwinner based android tv box, with its own custom remote,
this adds a keymap for this remote (like all allwinner devices, the wobo i5
can run plain Linux).
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hans de Goede [Thu, 8 May 2014 19:22:46 +0000 (21:22 +0200)]
rc_keymaps: Add a keymap for the remote shipped with allwinner i12-a20 tv boxes
This is an unbranded remote found with unbranded allwinner a20 based android
tv boxes, which are referred to as i12 tv boxes since the pcb is marked i12
(the i12 name is also used in u-boot and for the devicetree files).
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hans de Goede [Thu, 8 May 2014 19:20:35 +0000 (21:20 +0200)]
rc_keymaps: Add a keymap for the remote shipped with allwinner ba10 tv boxes
This is an unbranded remote found with unbranded allwinner a10 based android
tv boxes, which are referred to as ba10 tv boxes since the pcb is marked ba10
(the ba10 name is also used in u-boot and for the devicetree files).
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Rob Barker [Thu, 29 May 2014 15:20:53 +0000 (12:20 -0300)]
libdvbv5: Find other TS in NIT pointer fix
dvb_scan_add_entry() is modified to return the pointer to the newly
created entry (the last in the list), so the add_update_nit_ functions
now modify the correct element, instead of the penultimate one. This
fixes the finding of other TS in NIT scan feature.
Signed-off-by: Rob Barker <robert.barker@redembedded.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Wed, 4 Jun 2014 17:14:55 +0000 (14:14 -0300)]
dvb-file: Fix bandwidth handling at online write function
The conversion from bandwidth in Hz to bandwidth with the legacy
oneline files is wrong. It was always returning bandwidth auto,
instead of writing the proper values on it. Fix it.
Reported-by: Reynaldo H. Verdejo Pinochet <r.verdejo@sisa.samsung.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Laurent Pinchart [Tue, 3 Jun 2014 09:55:42 +0000 (11:55 +0200)]
media-ctl: Add fall through comment to fix Coverity warning
The -f case falls through the -V case during option parsing. This is
desired but result in a missing break statement warning from Coverity.
Fix it by adding a fall through comment.
Reported-by: Coverity Scan
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Tue, 3 Jun 2014 09:54:58 +0000 (11:54 +0200)]
media-ctl: Fix media device version printing
Extract the version major, minor and patch level version components by
shifting right, not left.
Reported-by: Coverity Scan
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Tue, 3 Jun 2014 09:53:23 +0000 (11:53 +0200)]
media-ctl: Fix off-by-one buffer overflow with readlink
readlink() returns the number of bytes written to the buffer, which can
be up to the passed buffer size, without including the terminating '\0'.
This causes an off-by-one overflow as we pass the total buffer length to
the function and then try to append a terminating '\0'.
Fix it by passing the buffer size minus one to readlink().
Reported-by: Coverity Scan
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Hans Verkuil [Tue, 3 Jun 2014 09:04:16 +0000 (11:04 +0200)]
qv4l2: fix segmentation fault when compiled without ALSA support.
If opening the video device fails, then qv4l2 attempts to disable
two alsa widgets. If qv4l2 was compiled without alsa support, then
those widget pointers are NULL and you get a segfault.
Add the proper #ifdef so this is only done if alsa support was
compiled in.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Tue, 3 Jun 2014 08:19:56 +0000 (10:19 +0200)]
Synced with latest kernel
This adds the missing v4l2-mediabus.h header.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Tue, 3 Jun 2014 08:15:44 +0000 (10:15 +0200)]
Add missing v4l2-mediabus.h
This header should also be copied when syncing with the kernel.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Laurent Pinchart [Mon, 2 Jun 2014 14:28:12 +0000 (16:28 +0200)]
media-ctl: Update copyright years
We're well into 2014 and there no going back.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Sat, 31 May 2014 11:12:18 +0000 (13:12 +0200)]
media-ctl: Don't install libmediactl and libv4l2subdev
The libraries APIs are not deemed to be stable yet, don't install the
libraries. We must thus compile the two libraries statically to link
media-ctl accordingly.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Sat, 31 May 2014 11:08:14 +0000 (13:08 +0200)]
media-ctl: Always print the full help message
The short help text isn't descriptive enough to be useful.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Fri, 30 May 2014 23:00:34 +0000 (01:00 +0200)]
media-ctl: Document the --links option argument
The --links option take a mandatory argument. Document it.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Fri, 30 May 2014 22:59:17 +0000 (00:59 +0200)]
media-ctl: Remove the obsolete device argument from the help text
The standalone device argument isn't supported anymore, remove it from
the documentation.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Fri, 30 May 2014 22:51:06 +0000 (00:51 +0200)]
media-ctl: Rename source files
Prefix the library files with lib and rename main.c to media-ctl.c.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Laurent Pinchart [Fri, 30 May 2014 15:18:36 +0000 (17:18 +0200)]
Import the media-ctl utility and libraries
This corresponds to commit
114c1b274edc40e07e9b99a435d26438f5b99943
("Fix uninitialized pointer dereference in media_device_add_entity") in
the git://git.ideasonboard.org/media-ctl.git repository.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Hans Verkuil [Mon, 26 May 2014 06:42:24 +0000 (08:42 +0200)]
Synced with latest kernel.
This syncs changes to VIDIOC_DV_TIMINGS_CAP and VIDIOC_DV_ENUM_TIMINGS.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 26 May 2014 06:39:35 +0000 (08:39 +0200)]
libdvbv5: generate the correct include in dvb-v5.c
When dvb-v5.c is generated the #include line is incorrect: instead
of including <libdvbv5/dvb-v5.h> it should include "dvb-v5.h".
Fixed.
An earlier patch patches the dvb-v5.c source, but since that is
regenerated by 'make sync-with-kernel' I had to change it in the
code that generates the source instead.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Sun, 25 May 2014 14:06:31 +0000 (16:06 +0200)]
v4l2-ctl: add support V4L2_EVENT_SOURCE_CHANGE
This new event was added, add support for it to v4l2-ctl.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Sun, 25 May 2014 14:04:55 +0000 (16:04 +0200)]
libdvbv5: fix compile error
Fix the following compile error:
dvb-v5.c:6:29: fatal error: libdvbv5/dvb-v5.h: No such file or directory
by using #include "dvb-v5.h", just like the other sources that include
this header.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Sun, 25 May 2014 13:42:27 +0000 (15:42 +0200)]
sync-with-kernel: update to latest kernel headers.
This syncs with the latest kernel code. In particular it adds support
for V4L2_EVENT_SOURCE_CHANGE.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Mauro Carvalho Chehab [Fri, 2 May 2014 13:52:29 +0000 (06:52 -0700)]
libdvbv5: Fix the speedup scan condition
The DVB table read routine has 3 operational modes: it can seek for an
specific table section, it can expect sequential table sections, or it
can wait non-sequencial sections, stopping when a section is duplicated.
The second mode (sequencial table sections) finishes the reading after
receiving all table sections from 0 to the max table section, speeding
up its reception, while being more reliant to section errors.
However, the tests for the stop condition for this mode were wrong.
Fix it.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Wed, 23 Apr 2014 19:00:58 +0000 (16:00 -0300)]
libdvbv5: don't override a parameter when setting lna
We should not fill the parameters before setting LNA, as, otherwise,
the first property (typically frequency) would be overriden.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Gregor Jasny [Wed, 23 Apr 2014 08:12:33 +0000 (10:12 +0200)]
libv4lconvert: Gracefully handle unknown SE401 destination formats
Detected by Coverity.
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
CC: Hans de Goede <hdegoede@redhat.com>
Gregor Jasny [Wed, 23 Apr 2014 08:07:17 +0000 (10:07 +0200)]
libv4lconvert: Error out if source or destination pointers are null
Detected by Coverity.
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Wed, 23 Apr 2014 07:52:19 +0000 (09:52 +0200)]
dvbv5-zap: Always initialize status variable
In case of an early timeout it would be used uninitialized.
Detected by Coverity.
CC: Mauro Carvalho Chehab <m.chehab@samsung.com>
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Wed, 23 Apr 2014 07:48:20 +0000 (09:48 +0200)]
dvbv5-zap: Close open file descriptor in error path
Dectected by Coverity.
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
CC: Mauro Carvalho Chehab <m.chehab@samsung.com>
Gregor Jasny [Wed, 23 Apr 2014 07:28:46 +0000 (09:28 +0200)]
v4l2-ctl: Do not leak framebuffer handle in error paths
Detected by Coverity
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Gregor Jasny [Wed, 23 Apr 2014 06:46:41 +0000 (08:46 +0200)]
libdvbv5; Fix typo in MPEG TS print function
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
Gregor Jasny [Wed, 23 Apr 2014 06:43:09 +0000 (08:43 +0200)]
libdvbv5: FIx check for available adaptation field
The field is only a single bit wide, so applying a the
0x2 mask will never be true.
Detected by Coverity.
Signed-off-by: Gregor Jasny <gjasny@googlemail.com>
CC: Mauro Carvalho Chehab <m.chehab@samsung.com>
Hans de Goede [Tue, 22 Apr 2014 09:38:39 +0000 (11:38 +0200)]
libv4l2: decomress-helper: close fds then wait for helper to exit
Killing the helper does not always work (as it inherits its parent sigmask),
instead of meddling with the sigmask simply close the fds and wait for the
helper to cleanly exit.
Reported-by: Волков Андрей <volkov.am@ekb-info.ru>
Tested-by: Волков Андрей <volkov.am@ekb-info.ru>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hans de Goede [Sun, 2 Mar 2014 13:57:51 +0000 (14:57 +0100)]
libv4lcontrol: Add Asus F3Sc with 04f2:b012 cam as upside down false positive
Also see:
https://bugzilla.redhat.com/show_bug.cgi?id=844999
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hans de Goede [Sun, 2 Mar 2014 13:55:36 +0000 (14:55 +0100)]
libv4lcontrol: Check control_flags before doing wildcard maching on upside_down
Check the more specific control_flags table before doing wildcard maching on
entries in the upside_down table, and if there is a match in the control_flags
table skip the upside_down table.
This way the control flags table can be used to override false positives caused
by the upside_down table wildcard matching.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Mauro Carvalho Chehab [Thu, 17 Apr 2014 20:59:39 +0000 (17:59 -0300)]
Revert "libdvbv5: Change license to LGPL v2.1"
This patch was merged by mistake. The plan is to move the license
to LGPL, but I'm still waiting for some contributors to send their
ack.
So, for now, revert this change.
This reverts commit
beb426496ca0e8fbed1b67eea56686e1b60381d7.
Mauro Carvalho Chehab [Thu, 17 Apr 2014 20:07:29 +0000 (17:07 -0300)]
libdvbv5: Remove two warnings on 32 bits compilation
tables/mpeg_pes.c: In function 'dvb_mpeg_pes_print':
tables/mpeg_pes.c:129:4: warning: format '%lx' expects argument of type 'long unsigned int', but argument 3 has type 'uint64_t' [-Wformat=]
dvb_loginfo(" - pts %lx (%fs)", pes->optional->pts, (float) pes->optional->pts / 90000.0);
^
tables/mpeg_pes.c:131:4: warning: format '%lx' expects argument of type 'long unsigned int', but argument 3 has type 'uint64_t' [-Wformat=]
dvb_loginfo(" - dts %lx (%fs)", pes->optional->dts, (float) pes->optional->dts/ 90000.0);
^
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 16 Apr 2014 21:22:10 +0000 (18:22 -0300)]
libdvbv5: improve DVB header handling
the table parsers now initialize the complete table,
and do no longer rely on an already initialized table
header in the supplied buffer.
adds section length checking in PAT as well.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Wed, 16 Apr 2014 13:06:41 +0000 (10:06 -0300)]
libdvbv5: Change license to LGPL v2.1
Licensing it under LGPL allows its usage on more projects. This license
is required by some other Open Source projects, like Gstreamer.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 15 Apr 2014 20:33:01 +0000 (17:33 -0300)]
libdvbv5: improve CRC size handling
- provide buffer without CRC to the table parsers
- remove unneeded defines
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 15 Apr 2014 18:39:33 +0000 (15:39 -0300)]
libdvbv5: build dynamic libdvbv5 by default
- renamed ./configure option --enable-libdvbv5 to --disable-libdvbv5
- thus libdvbv5 will be build shared by default
- cleanups in configure.ac
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 15 Apr 2014 18:39:32 +0000 (15:39 -0300)]
libdvbv5: short API description
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 15 Apr 2014 18:39:31 +0000 (15:39 -0300)]
libdvbv5: cleanup parser API
- fix initialization of dvb_table_initializers
- check return value of dvb_desc_parse
- allocate table inside the parser
- remove unneeded table_length argument
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 15 Apr 2014 18:39:30 +0000 (20:39 +0200)]
libdvbv5: move table parsers to separate directory
move the dvb table parsers from the descriptors/ directory
to tables/ to have a nice and clear separation.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 21:35:16 +0000 (18:35 -0300)]
dvb-zap: fix --search argument
The search argument is used to filter the monitored traffic.
Also, it requires a string as argument.
Fix it.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 21:27:48 +0000 (18:27 -0300)]
dvbv5-zap: only open the DVR interface if outputing
As other processes will be reading the dvr interface, we should
not open it, just announce that the dvr interface is ready.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 21:17:05 +0000 (18:17 -0300)]
dvbv5-zap: better implement the --record flag
Instead of outputing to STDOUT, just keep the dvr file opened,
in order to let other applications to use it.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 20:39:48 +0000 (17:39 -0300)]
dvbv5-zap: Make dvbv5 as the default format
While doc says so, dvbv5-zap was always requiring the usage of
the -I argument. Fix it.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 20:19:31 +0000 (17:19 -0300)]
dvbv5-zap: fix usage of --record without --output
If --output is not specified, output data to STDOUT.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 18:49:05 +0000 (15:49 -0300)]
libdvbv5: fix the area code size on ISDB-T
The area code has 12 bits, and not 6, accordig with the
ABNT NBR 15603-2:2007 version3 of 2009.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 18:43:50 +0000 (15:43 -0300)]
libdvbv5: move ISDB-T frq conversion to the descriptors code
Instead of having this done on several places, just do it once,
at the code that parses the ISDB-T descriptor.
This is the same as the other descriptors do.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Fri, 11 Apr 2014 17:27:35 +0000 (10:27 -0700)]
libdvbv5: fix ISDB-T frequency calculus on 32 bits machines
On 32 bits, the frequency got truncated and become a negative
number. Fix it.
Signed-off-by: Mauro Carvalho Chehab <mchehab@brisa.lan>
André Roth [Wed, 9 Apr 2014 22:27:00 +0000 (19:27 -0300)]
libdvbv5: rename descriptor functions
prefix the descriptor functions with dvb_desc
for a nice API and code completion support
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:59 +0000 (19:26 -0300)]
libdvbv5: cleanup printing of tables and descriptors
- use log info for table and descpritor printing
- cleanup 'out of memory' logs
- fix error condition in desc_ts_info
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:58 +0000 (19:26 -0300)]
libdvbv5: cleanup table parsers
- make the code look similar
- check for correct table ID
- ignore null packets with ID 0x1fff
- return table length, or:
- return error code on error
- update / fix Copyrights
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:57 +0000 (19:26 -0300)]
libdvbv5: make dvb_table_filter_free public
make dvb_table_filter_free public so it can be used by
applications.
fix potential double free.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:56 +0000 (19:26 -0300)]
libdvbv5: make crc32 public
applications reading dvb files might want to
check crc, therefore provide the crc32 function
in the header files.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:55 +0000 (19:26 -0300)]
libdvbv5: fix asprintf compile warning in dvbv5-scan
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Wed, 9 Apr 2014 22:26:54 +0000 (19:26 -0300)]
libdvbv5: reunite atsc_table_header and dvb_table_header
this solves the ambiguity when parsing the dvb/atsc header
in dvb-scan.c
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>