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>
Hans Verkuil [Wed, 9 Apr 2014 08:03:30 +0000 (10:03 +0200)]
v4l2-compliance: don't do the global format test when using libv4l2.
This test will fail when using the wrapper library as the wrapper
library does not see when another filehandle changes the format.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Wed, 9 Apr 2014 07:50:10 +0000 (09:50 +0200)]
libv4lconvert: fix a v4l2-compliance error
memset frmsize->reserved to 0.
fail: v4l2-test-formats.cpp(150): frmsize.reserved not zeroed
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:45 +0000 (17:51 +0200)]
INSTALL: add cross compile and install procedures for Android
This patch adds some words in the INSTALL file about how to cross
compile some utilities (compliance, ctl, and dbg) for Android and
install the on a device running Android.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:44 +0000 (17:51 +0200)]
v4l2-dbg: add Android.mk makefile
This Android.mk makefile is used by the Android build system to compile
and link the v4l2-dbg utility.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:43 +0000 (17:51 +0200)]
v4l2-ctl: add Android.mk makefile
This Android.mk makefile is used by the Android build system to compile
and link the v4l2-ctl utility.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:42 +0000 (17:51 +0200)]
v4l2-compliance: add Android.mk makefile
This Android.mk makefile is used by the Android build system to compile
and link the v4l2-compliance utility.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Fri, 11 Apr 2014 14:39:24 +0000 (16:39 +0200)]
v4l2-ctl/dbg: include android-config.h header file if ANDROID is set
If the ANDROID symbol is defined, done by Android toolchain by default,
include the android-config.h header file instead of the autogenerated
config.h header file.
Android build system is completely different from autoconf and make, and
is not capable, without some hacks, to generate the config.h header
file.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
[hans.verkuil@cisco.com: moved config(-android).h include to v4l2-ctl.h]
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:40 +0000 (17:51 +0200)]
Add static configuration file for Android builds
This patch adds an android-config.h configuration file that matches the
Android toolchain configuration.
It also includes an implementation of getsubopt, with the strchrnul
dependency, since the Android toolchain does not provide this
functionality.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
[hans.verkuil@cisco.com: made functions inline]
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Fri, 11 Apr 2014 14:26:26 +0000 (16:26 +0200)]
v4l2-compliance/ctl: drop libv4l2 dependency if NO_LIBV4L2 is defined
This patch introduces a check for a NO_LIBV4L2 symbol in v4l2-compliance
and v4l2-ctl, which, when defined, will let the utility be built without
linking with libv4l2.
The wrapper functionality libv4l2 provides is obviously lost when
building with NO_LIBV4L2 defined.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
[hans.verkuil@cisco.com: modified to reduce number of #ifdefs]
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans-Christian Egtvedt [Thu, 10 Apr 2014 15:51:38 +0000 (17:51 +0200)]
v4l2-ctl: remove unnecessary inclusion of libv4l2.h
These files do not utilize anything from libv4l2.h, hence the include is
not needed.
Signed-off-by: Hans-Christian Egtvedt <hegtvedt@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 7 Apr 2014 10:07:30 +0000 (12:07 +0200)]
qv4l2: fix modulator frequency bugs
When setting up the modulator frequency field the tuner capabilities were used
instead of the modulator capabilities. The field label also still contained the
frequency unit, which is now part of the input field, so that could be simplified.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 7 Apr 2014 09:43:32 +0000 (11:43 +0200)]
qv4l2: only set m_isRadio if m_isSDR is true.
The m_isRadio bool was reset if m_isSDR was false, that's not what should
happen. Only if m_isSDR is true should the m_isRadio value be changed.
This bug caused the frame width/height etc. fields to appear for a radio
transmitter.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Tue, 1 Apr 2014 15:30:22 +0000 (17:30 +0200)]
v4l2-compliance: fix 32-bit compilation for v4l_fd_init()
Since mmap and v4l2_mmap have different prototypes (why?!) we have
to make a small wrapper to ensure that it compiles for both 32 and 64
bit systems.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Tue, 1 Apr 2014 06:34:15 +0000 (08:34 +0200)]
v4l2-compliance: add missing 'class' keyword after 'friend'
This led to this compile error for some compilers:
In file included from v4l2-compliance.h:31,
from v4l2-compliance.cpp:37:
./cv4l-helpers.h:9: error: a class-key must be used when declaring a friend
./cv4l-helpers.h:9: error: friend declaration does not name a class or function
I'm not sure why this doesn't fail for all compilers, weird.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Reported-by: Prabhakar Lad <prabhakar.csengg@gmail.com>
André Roth [Sun, 30 Mar 2014 16:21:18 +0000 (13:21 -0300)]
libdvbv5: add support for tables with multiple ts_id and section gaps
this restores the logic for reading tables like EIT, where the
ts_id indicates the service and the sections are incremented by 8
for example. an application might wish to read a complete EIT
containing all services. in this case the function dvb_read_sections
can be used with allow_section_gaps set to 1 in the dvb_table_filter
struct.
- make struct dvb_table_filter public
- use more generic void * for tables
- make dvb_read_sections public
- fix continuous parsing when table_length > 0
- fix logging
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:17 +0000 (13:21 -0300)]
libdvbv5: fix lost ressource in atsc_eit
if the title of the EIT event is longer than the available data,
make sure the allocated buffer is not lost
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:16 +0000 (13:21 -0300)]
libdvbv5: remove unneeded includes
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:15 +0000 (13:21 -0300)]
libdvbv5: descriptor parser return int
If parsing a descriptor fails, let the outside world know.
Returning 0 means success, negative values indicate a parsing
error.
Free allocated memory on error.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:14 +0000 (13:21 -0300)]
libdvbv5: allow table parsers to get specific pointer to table struct
this will allow simplifying the parser functions and
to return the number of bytes read or an error code.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:13 +0000 (13:21 -0300)]
libdvbv5: make dvb_desc_default_init and dvb_desc_default_print private
dvb_desc_default_init and dvb_desc_default_print are used
internaly only, remove them from the header file.
add extern "C" to the parser functions, so they can be used
from C++ directly.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:12 +0000 (13:21 -0300)]
libdvbv5: use DVB_DESC_HEADER macro in all descriptors
the first fields of a descriptor need to be the
type, length and the next pointer. in order to
prevent changing these fields by accident,
provide them in a macro.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Sun, 30 Mar 2014 16:21:11 +0000 (13:21 -0300)]
libdvbv5: cleanup printing tables and descriptors
- log hex values where appropriate
- cleanup indents
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Hans Verkuil [Mon, 31 Mar 2014 15:59:13 +0000 (17:59 +0200)]
v4l2-compliance: fix function pointer prototype.
There was a conflict between the mmap function pointer prototype of
struct v4l_fd and the actual function used. Make sure it is in sync
with the prototype of v4l2_mmap.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 31 Mar 2014 08:44:59 +0000 (10:44 +0200)]
qv4l2: add support for V4L2_PIX_FMT_NV16M/61M
Add support for the NV16M and NV61M planar YUV 4:2:2 formats.
This is not the prettiest code. Once the (c)v4l-helpers.h are more
mature (particularly w.r.t. the v4l2-format handling) the qv4l2 code
will switch to that and that should greatly improve the code.
For now though it is good to have support for at least one multiplanar
format in qv4l2 to help testing the multiplanar API.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 31 Mar 2014 08:37:58 +0000 (10:37 +0200)]
v4l2-compliance: test control/format/codec ioctls for all inputs/outputs
Different inputs or outputs can support different controls, formats and codec
ioctls. So iterate over all inputs/outputs to test this.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 31 Mar 2014 08:33:47 +0000 (10:33 +0200)]
v4l2-compliance: m2m devices should have only one input and output
It makes no sense to have multiple inputs and outputs for a m2m device.
Check for this.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 31 Mar 2014 08:31:50 +0000 (10:31 +0200)]
v4l2-compliance: use the new cv4l-helpers.h header.
This hides most of the single/multi planar differences, thus making it
easier to write test-cases.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Mon, 31 Mar 2014 08:22:01 +0000 (10:22 +0200)]
v4l2-compliance: add helper headers
In order to simplify working with single and multiple planar formats
these two helper headers were created: v4l-helpers.h is the C version
and cv4l-helpers.h is a C++ version on top of v4l-helpers.h.
For now this will only be used in v4l2-compliance, but the intention
is to use it elsewhere as well eventually.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
André Roth [Tue, 25 Mar 2014 18:20:01 +0000 (15:20 -0300)]
libdvbv5: fix PMT parser
- check for correct table ID
- parse all descriptos
- improve table printing
[m.chehab@samsung.com: fix the copyright line change]
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:20:00 +0000 (15:20 -0300)]
libdvbv5: add parser for ca and ca_identifier descriptors
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:59 +0000 (15:19 -0300)]
libdvbv5: add parser for CAT
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:58 +0000 (15:19 -0300)]
libdvbv5: add attribute packed to structs and unions
__attribute__((packed)) is needed for nested structs and
unions to get the correct size.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:57 +0000 (15:19 -0300)]
libdvbv5: fix dvb_parse_descriptors and make dvb_desc_init private
- set list pointer to NULL in case of an error
- improve size checking
- dvb_desc_init is used only internal, remove from header
[m.chehab@samsung.com: revert the change from calloc() to malloc()]
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:56 +0000 (15:19 -0300)]
libdvbv5: remove header files from SOURCES in Makefile.am
The header files are not needed in the Makefile.am since
automake finds the dependencies automatically. This is not
needed for installing header files either.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Fri, 28 Mar 2014 16:42:49 +0000 (17:42 +0100)]
libdvbv5: shared lib and installing headers
- move headers to subdirectory for correct install location
- fix includes in dvb table and descriptor parsers
- fix compile warnings
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:54 +0000 (15:19 -0300)]
libdvbv5: fix EIT parsing
the dvb_table_eit_event now contains the service_id,
indicating where the events belong to.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:53 +0000 (15:19 -0300)]
libdvbv5: mpeg elementary stream parsers
Implement parsers for mpeg TS, PES and ES
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:52 +0000 (15:19 -0300)]
libdvbv5: fix asprintf compile warnings
- allow logging in dvb_vchannel
- get rid of compile warings about unused asprintf return value
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
André Roth [Tue, 25 Mar 2014 18:19:51 +0000 (15:19 -0300)]
libdvbv5: support info info log via dvb_loginfo
dvb_loginfo log support without setting output color.
Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Sun, 16 Mar 2014 11:40:00 +0000 (08:40 -0300)]
libdvbv5: Frequency is unsigned. use %u to print it
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Mauro Carvalho Chehab [Sat, 15 Mar 2014 12:08:09 +0000 (09:08 -0300)]
dvbv5-zap: only start audio filtering if audio PID > 0
When used with -P, audio filter PID is zero. Yet, it is
currently starting an audio PID filter. This could cause
troubles with some hardware filters.
As PID is always bigger than zero, this patch should fix
the issue.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Hans Verkuil [Mon, 3 Mar 2014 23:05:23 +0000 (00:05 +0100)]
libv4lconvert: remove broken ALTERNATE handling
The V4L2 specification used to say that if field == V4L2_FIELD_ALTERNATE, the
height would have to be divided by two. This is incorrect, the height is that of
a single field. This has been corrected in the spec, now this code in libv4lconvert
needs to be removed as well.
Tested with both bttv and saa7146, the only two drivers supporting FIELD_ALTERNATE
today.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Fri, 14 Mar 2014 14:56:05 +0000 (15:56 +0100)]
v4l2-compliance: add G/S_EDID tests.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Hans Verkuil [Fri, 14 Mar 2014 14:12:42 +0000 (15:12 +0100)]
v4l2-compliance: add test setting all enumerated timings.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>