qv4l2: fix enabling/disable frame width/height/size

The logic that dealt with when the frame width, height and size
fields had to be enabled and disabled was flawed. Starting and
stopping streaming would just enable all those field unconditionally,
and updating the format state could enable them while streaming.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: zero initial buffers

If the initial compose rectangle doesn't cover the full buffer,
then there will be a lot of garbage in the buffer if it isn't
zeroed.

Changing the compose rectangle while streaming will correctly
clear buffers as well, it was just this initial setup that caused
problems.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: fix incorrect disabling of the blending mode

When the blending mode was disabled, the blending function
wasn't turned off. Reworked the code to fix this and to make
the glBlendFunc call sequence easier to follow.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

libv4l-mplane: fix compile warning

libv4l-mplane.c:356:24: warning: unused variable 'plugin' [-Wunused-variable]

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

libv4l-mplane: v4l2-compliance fixes

Fixed a variety of issues with this plugin. Found by running
v4l2-compliance -w with my multiplanar-enabled vivi driver.

- v4l2_pix_format's priv field wasn't zeroed.
- attempts to use the multiplanar buf types should return an error
  since they are disabled in the querycap capabilities.
- implement read() and write(): let the driver decide what to do with
  them.
- don't convert the buf type for the cropping ioctls. Just like the
  selection ioctls these shouldn't be converted.
- the memory field of v4l2_buffer should be copied back.
- errno was set to a negative value in at least one place.
- the create_bufs_ioctl() implementation was completely wrong.
- the querycap code didn't convert the device_caps field.

There are still a few issues left, but those come from the libv4l2
library, not from this plugin.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

libv4l-mplane: copy back v4l2_buffer.m, fix type problems

The ioctls operating on v4l2_buffer structs should copy back the
v4l2_plane.m to v4l2_buffer.m. Applications (like qv4l2) rely on
VIDIOC_QUERYBUF to return the correct m.offset, etc.

The format ioctls and create_bufs would call the ioctl directly
if the buffer type wasn't one of VIDEO_CAPTURE or VIDEO_OUTPUT.
However, it used the local zeroed format/v4l2_create_buffers
struct instead of the real struct. This is now fixed.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: make the destination sizeimage large enough

Normally v4lconvert_try_format() will make sure that sizeimage is
large enough by calling try_fmt(). But if the multiplanar plugin is
in use, then v4lconvert_try_format() bypasses that plugin and try_fmt()
will fail since the driver expects the _MPLANE buffer types.

Normal programs won't use this function, so it won't be a problem there.
But in our case the solution is fairly simple: just calculate sizeimage
ourselves.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: remove two forgotten debug printfs

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: implement SEQ_TB/BT support for the openGL renderer

Add openGL support for frames captured using the V4L2_FIELD_SEQ_TB/BT
field setting. This was the only field type that wasn't yet supported.

The software renderer still doesn't support it, only openGL does.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: add support for crop and compose

Implement GUI support to set the crop and compose rectangles if that
is supported by the driver.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: fix pixel aspect handling and set compose rectangle

The pixel aspect ratio handling did not take scaling into account.

When changing the format the compose rectangle should be set to the
same size as well.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: fix control range handling and broken slider disabling.

The introduction of slider + spinbox widgets for simple integer controls
missed that when such a control is disabled both widgets have to be
disabled, instead only the spinbox was disabled.

Changing the control range on the fly was never properly implemented, this
is now fixed.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: fix compilation error if HAVE_QTGL is undefined

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: revert most of the previous patch

The driver I was testing this swapped the numerator and denominator,
causing me to think that the qv4l2 code was wrong. However, it is
correct except for a status message that swapped it as well.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

dvb-file: better store channels without SDT

There's  a risk of two channels with different frequencies to have
the same name, as they could have the same service ID.

So, add the frequency to the channel name, to avoid such risk.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

qv4l2: fix pixel aspect ratio calculation

The autodetect of the aspect ratio failed (swapped numerator and
denominator) due to the confusing calculation.

Ensure that the numerator always applies to the height of the pixel
and denominator to the width of the pixel (as returned by cropcap).

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

dvb-file: store services even when SDT is not found

Sometimes, both PAT and PMT tables are properly parsed, but SDT
table is missing. Without SDT, we don't know the program names,
but it is still possible to store the services. So, do it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Improve debug logs while scanning PMT

Better display what's happening during PMT program scan.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

qv4l2: added user controls

Autogenerated integer controls where the range is less than or equal to
0xFFFF now have both a Slider and a SpinBox to change the value.
Using unsigned to get relevant result in range comparison.

Signed-off-by: Anton Arbring <aarbring@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: correctly disable two menu items

The "Save Raw Frames" menu item should be disabled if no video device is open,
the "Use OpenGL Rendering" menu item should be disabled if streaming is in
progress.

Signed-off-by: Anton Arbring <aarbring@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: changed and removed icons

The new start icon is more intuitive than the old record button.
Some color added to the snapshot icon to make it more visible when enabled.
The close window and show frames icons are removed from the toolbar as they
are not used very frequently.

Signed-off-by: Anton Arbring <aarbring@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: clarified two strings

Signed-off-by: Anton Arbring <aarbring@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

qv4l2: add a 'blending' mode to test images with an alpha channel.

When enabled the tool will draw a black square on a white background
and use the alpha channel to make the image transparent so the square
becomes visible.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

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>

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>

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>

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>

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>

Fix kFreeBSD build
* fixed BSDfication of V4L related headers
* Disabled libdvbv5 and v4l-utils build

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

qv4l2: Fix typo

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

Revert "libdvbv5: Do not install libdvbv5 by default"

This reverts commit 4fe1165ab5239bb2fadeed60f9cc8f0f1d6253f5.

Start v4l-utils 1.3.x development

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

Prepare for 1.2.0 release

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

libdvbv5: Do not install libdvbv5 by default

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

qv4l2: Distribute manpage

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

v4l2-compliance: Distribute internal header, too

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

buildsystem: Distribute Android.mk files

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

mediactl: Distribute internal header, too

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

libdvbv5: Distribute internal header, too

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

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>

parse_dib0700.pl: Parse REQUEST_SET_I2C_PARAM

Parses I2C speed changes.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

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>

Revert "wip"

The commit was pushed by mistake, revert it.

This reverts commit 83d0df45745873f856c8a63bf4a86fec70559fca.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

parse_dib0700.pl: autoflush writes

Autoflushing avoids loosing message when CTRL-C is pressed.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

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>

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>

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>

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>

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>

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>

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>

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>

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>

wip

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

Synced with latest kernel

This adds the missing v4l2-mediabus.h header.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

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>

media-ctl: Update copyright years

We're well into 2014 and there no going back.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

libv4lconvert: Error out if source or destination pointers are null

Detected by Coverity.

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

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>

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>

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>

libdvbv5; Fix typo in MPEG TS print function

Signed-off-by: Gregor Jasny <gjasny@googlemail.com>

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>

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>

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>

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>

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.