Make the media_device structure private

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>

Split media_device creation and opening

Make the media_device refcounted to manage its life time.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>

Read errno before media_dbg calls

The media_dbg() in error paths can modify the errno value, leading to
the wrong error (or no error at all) being returned. Fix this by reading
the errno value before calling media_dbg().

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

Print v4l parser position on error

Most parser functions take a **endp argument to indicate the caller
where parsing has stopped. This is currently only used after parsing
something successfully. This patch sets **endp to the erroneous
position in the error case and prints its position after an error
has occured.

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

Print more detailed v4l parse error messages

The following errors usually resulted in the same 'Unable to parse
format' message:

- syntax error in format description
- the requested pixel code isn't supported

Add more detailed error messages to give the user a clue what is going
wrong.

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

Support both " and ' as string delimiters when parsing the command line

Entity names are enclosed in quotes. Support both double-quoted and
single-quoted strings.

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

options: don't imply -v when doing a -p.

This avoids us getting (unnecessary) debug messages during -p.

Signed-off-by: Alain Volmat <alain.volmat@st.com>
Signed-off-by: André Draszik <andre.draszik@linaro.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Print parser position on error

Most parser functions take a **endp argument to indicate the caller
where parsing has stopped. This is currently only used after parsing
something successfully. This patch sets **endp to the erroneous
position in the error case and prints its position after an error
has occured.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
[Remove endp NULL checks spread around the code]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Print more detailed parse error messages

The following errors usually resulted in the same 'Unable to parse link'
message:

- one of the given entities does not exist
- one of the pads of a given entity does not exist
- No link exists between given pads
- syntax error in link description

Add more detailed error messages to give the user a clue what is going wrong.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Fix linking of shared libraries

When using libudev, it is actually libmediactl that uses it and not the
media-ctl executable. libv4l2subdev uses functions from libmediactl and
therefore needs to be linked against it.

Signed-off-by: Andreas Bombe <aeb@debian.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Add missing stdlib.h and ctype.h includes

src/mediactl.c needs ctype.h for its use of isspace().

src/v4l2subdev.c needs stdlib.h for strtoul() and ctype.h for isspace()
and isupper().

Signed-off-by: Andreas Bombe <aeb@debian.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

configure.ac: Respect CPPFLAGS from environment

Signed-off-by: Andreas Bombe <aeb@debian.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Sort headers alphabetically

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

Print media entity flags when printing device topology

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

Add documentation generation through Doxygen

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

media: Fix valgrind warnings

valgrind has no way to know that the kernel will fill links and pads
arrays passed to an ioctl through __user pointers. Allocate the arrays
with calloc() to make sure they get initialized. This also fixes the
potential integer overflow.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>

media-ctl: Fix build error with newer autotools

 Rename configure.in to be configure.ac - required for newer
 versions of autotools (older versions silently handled
 this, now it's an error)

Signed-off-by: Gary Thomas <gary@mlbassoc.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

fix for automake-1.12

This patch fixes following kind of issues with automake 1.12
| automake: warnings are treated as errors
...
| warning: 'libmediactl.la': linking libtool libraries using a non-POSIX
| ltlibrary.am: archiver requires 'AM_PROG_AR' in 'configure.in'
| warning: 'libv4l2subdev.la': linking libtool libraries using a non-POSIX
| ltlibrary.am: archiver requires 'AM_PROG_AR' in 'configure.in'
...
| src/Makefile.am:10: warning: compiling 'main.c' with per-target flags requires 'AM_PROG_CC_C_O' in 'configure.in'

Signed-off-by: Stefan Herbrechtsmeier <sherbrec@cit-ec.uni-bielefeld.de>

Add a copy of V4L2 kernel headers

To make compilation possible on older (< v3.6) kernels, include a
private copy of the V4L2 headers. API availability is checked and
handled at runtime.

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

Replace V4L2 subdev selection targets with the V4L2 ones

V4L2 selection targets will replace V4L2 subdev selection targets in the
near future. As the targets are guaranteed to be the same and the chance is
anticipated very soon, replace the subdev targets with more future-proof
V4L2 targets.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Drop _ACTUAL from selection target names

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

media-ctl: Compose print fixes

The compose rectangles were printed incorrectly in my recent patch "Compose
rectangle support for libv4l2subdev" without parenthesis. Fix this.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Compose rectangle support for libv4l2subdev

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

New, more flexible syntax for format

More flexible and extensible syntax for format which allows better usage
of the selection API.

Continue supporting the old syntax but remove the documentation for it. It
was not supported in an official release and its use is thus deprecated.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

options: Remove unnecessary commas from EBNF syntax

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

libmediactl: Implement MEDIA_ENT_ID_FLAG_NEXT in media_get_entity_by_id()

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
[laurent.pinchart@ideasonboard Clarify documentation]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Support selections API for crop

Support the new selections API for crop. Fall back to use the old crop API
in case the selection API isn't available.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Compare name length to avoid false positives in media_get_entity_by_name

If two subdevice have names that only differ by a suffix (such as "OMAP4
ISS ISP IPIPE" and "OMAP4 ISS ISP IPIPEIF") the media_get_entity_by_name
function might return a pointer to the entity with the longest name when
called with the shortest name. Fix this by verifying that the candidate
entity name length is equal to the requested name length.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Add support for YUV420 formats

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Several printout fixes.

- There are sink and source pads, not input and output.
- Print also DYNAMIC flag.
- Don't print "pad" before pad number in some cases. The strings are more
  usable for link parsing now.
- Don't print extra commas afterlink flags.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Print media device information when called with -p/--print

In addition to printing pipeline information, print the media device
information.

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

Switch to the LGPLv2.1+ license

Mail sent to all contributors on 2011-09-30:

Hi everybody,

Sakari made me realize today that the media-ctl project is still licensed
under the GPL v2+ license, while I would have sworn that we had switched to
LGPL a long time ago.

I would like to relicense the media-ctl project under the LGPL v2.1+, to allow
LGPL-compatible applications and libraries (such as libv4l for instance, or
its plugins) to link to libmediactl and libv4l2subdev.

As you all have submitted patches that are now included in the media-ctl
project, I'm requesting your permission to proceed with the license change.

--------------------------------------------------------------------------------
From: Todor Tomov <ttomov@mm-sol.com>
  To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2011-09-30 12:06:17

Hi Laurent,

No problems from my side.

--
Best regards,
Todor Tomov

--------------------------------------------------------------------------------
From: "Aguirre, Sergio" <saaguirre@ti.com>
  To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2011-09-30 15:41:45

Hi Laurent,

No problem for me. You can go ahead, if everyone else agrees of course.

Regards,
Sergio

--------------------------------------------------------------------------------
From: Sakari Ailus <sakari.ailus@iki.fi>
  To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2011-09-30 16:17:18

I'm in favour of the change, actually I think it's next to mandatory for the
library to be useful in a way it was intended.

Cheers,

--
Sakari Ailus

--------------------------------------------------------------------------------
From: Michael Jones <michael.jones@matrix-vision.de>
  To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2011-10-04 08:33:17

Changing the license is fine with me.

-Michael

--------------------------------------------------------------------------------
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> (Intel Finland Oy)
  To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2011-10-10 14:26:00

I have no objections.
But I didn't remember libudev exact license... Ah, okay, it's licensed
under LGPLv2.1+.

--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

--------------------------------------------------------------------------------

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

Additional formats used by BT-656 sensors

Signed-off-by: Gary Thomas <gary@mlbassoc.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Remove extra verbosity

Remove extra verbosity by default; "-v" option brings back what used to be
there. The error messages are now being printed by main.c with the possibly
helpful error code attached.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Add debugging handler

Add debugging handler to media_device that may be used to redirect all debug
formatting to user-supplied function. fprintf will do, and that's what
media-ctl test program will use.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Add link to media_device from the media_entity

This makes it possible to obtain the media device an entity belongs to.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

libv4l2subdev and libmediactl are not test programs

Call the libraries libraries rather than test programs.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Move V4L2 subdev format parsing from main.c to subdev.c

This makes format parsing a part of the libv4l2subdev and thus available on
all who use the library.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Move link parsing from main.c to media.c, making it part of libmediactl

This makes it possible to benefit from the link parsing code anywhere the
library is being used.

dprintf macro will later be replaced with proper debug support.

Also fix a case where -1 was returned on error and the user was expected to
check the value of errno. Negative error codes are now returned
consistently.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Rename files to match the names of the libraries

Rename media.* to mediactl.* and subdev.* v4l2subdev.*.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

libmediactl: get rid of memset via using calloc

The code snippet
x = malloc(sizeof(*x));
memset(x, 0, sizeof(*x));
could be easily changed to
x = calloc(1, sizeof(*x));

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

libmediactl: get the device name via udev

If configured with --with-libudev, the libmediactl is built with libudev
support. It allows to get the device name in right way in the modern linux
systems.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
[laurent.pinchart@ideasonboard.com: Remove media_private structure]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

libmediactl: split media_get_devname_sysfs from media_enum_entities

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

libmediactl: restruct error path

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
[laurent.pinchart@ideasonboard.com: Fix return value at end of
enumeration]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Don't perform AC_FUNC_MALLOC and AC_FUNC_REALLOC when cross-compiling

Those autoconf tests assume that the target libc doesn't have
GNU-compatible malloc and realloc implementations when cross-compiling,
which breaks compilation.

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

Allow using autoconf 2.61+

Autoconf v2.61 seems to work just fine. Allow it.

Signed-off-by: Michael Jones <michael.jones@matrix-vision.de>

Add Y10, Y12 formats

Signed-off-by: Michael Jones <michael.jones@matrix-vision.de>

Add missing 8-, 10- and 12-bit Bayer formats

Thanks for Javier Martin for pointing out they were missing.

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

Reset file descriptor value after closing a subdev

After a subdev is closed set entity->fd to -1.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Update stored link flags correctly in media_setup_link()

The flags stored in media_link structure are not updated correctly if
media_setup_link() is called on an immutable link - the immutable flag
is cleared. Fix this.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Update to the latest media kernel API

The media kernel API has seen many changes during the review process,
update the media-ctl application accordingly.

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

Print all links in media_print_topology_text()

Add printing of inbound links in media_print_topology_text().

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Update to the latest media kernel API

The media kernel API has seen many changes during the review process,
update the media-ctl application accordingly.

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

Rename libraries to descriptive names

Rename libraries to more descriptive names:
libmedia  -> libmediactl
libsubdev -> libv4l2subdev

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Install header files to subdir

Install media.h and subdev.h header files to mediactl/ subdirectory.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Update flags in twin link in media_setup_link()

Fix media_setup_link() to update flags in twin link too.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Use local variable to access link's flags in media_enum_links()

Local variable link is defined in media_enum_links() for more simple
access to link's data. Use it when accessing link's flags.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Update .gitignore

Add autotools temporary files to .gitignore.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Use pkg-config

Install libmedia.pc and libsubdev.pc files for use from pkg-config.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Create debian packages

Create debian packages for media-ctl application, media controller
and v4l2 subdevice libraries.
Set initial version 0.0.1-1.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Build libmedia and libsubdev

Build media controller and v4l2 subdevice shared libraries.
Use libtool for building.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Use autotools

Use autoconf and automake for building.
Quick instructions are added in INSTALL.
Application source files are moved to src subdirectory.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add v4l2_subdev_get_frame_interval()

Add function to get frame interval.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add doxygen comments

Add doxygen format comments to header files.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Shorten media structures names

Shorten structures names:
struct media_entity_link => struct media_link
struct media_entity_pad => struct media_pad

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add padding to media structures

Add padding to
struct media_entity_link
struct media_entity_pad
struct media_entity
struct media_device

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add v4l2_subdev_open() and v4l2_subdev_close() to subdev.h

Add v4l2_subdev_close() function and declare both in subdev.h.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add inbound links support

Store all inbound and outbound links in media_entity.links.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Modify media_entity_remote_pad() to return source pad only

Rename media_entity_remote_pad() to media_entity_remote_source().
When it is called on a sink pad, return the linked source pad.
When it is called on a source pad, return NULL.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Move printing functions to main.c

Move printing and helper functions to media-ctl application source.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Add media_entity_type() to media.h and make it inline.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Fix printf typos

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

media-ctl: subdev: add Y8 format.

main.c: allow spaces in format definition

Allow spaces after '[' in format definition.
Example:
media-ctl -f '"OMAP3 ISP resizer":1[ UYVY 864x480]'

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

media.c: fix NULL pointer dereference in media_enum_links()

Do not dereference source and sink in media_enum_links() if one
of them is NULL.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

media.c: fix array overrun in media_entity_subtype_to_string()

Fix overrun of static arrays node_types and subdev_types in
media_entity_subtype_to_string().

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Use installed kernel headers instead of in-tree headers

To install kernel headers to usr/include inside a kernel tree, just run
make headers_install. media-ctl can then be compiled by setting KDIR to
the patch to the kernel tree.

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

Update to the latest media kernel API

The media kernel API has seen many changes during the review process,
update the media-ctl application accordingly.

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

Print subdev device node names in dot diagrams

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

main.c: Fix crop rectangle printing format

Fix crop rectangle printing format after L,T/WxH to (L,T)/WxH change.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Use sysfs to locate devices nodes

Changes made during the v4l2_subdev API review renamed the /dev/subdev*
device nodes to /dev/v4l-subdev*. Instead of hardcoding the names in the
media-ctl application, use sysfs to retrieve the kernel device name and
assume the device node name is identical.

This isn't a perfect solution, we should really use libudev, but it will
do for now.

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

Make format setting optional for -f option

This allows to setup crop rectangle or frame interval alone - without format.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Change crop rectangle format from L,T/WxH to (L,T)/WxH

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Add a --get-format option

The option takes a pad name as argument and returns the active format
(and optional crop settings if available) for that pad.

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

Configure cropping on output pads before setting the format

Image size on output pads depend on the cropping parameters. Cropping
must be configured before setting the image size on those pads.

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

Be able to add more CFLAGS

This allows the gcc compilation to build with extra parameters.

For example, if we want to build with -static, we just do:

make CFLAGS=-static

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>

Create initial .gitignore file

Ignore build generated files as well as the "standard" patches subfolder
used by quilt.

Signed-off-by: Sergio Aguirre <saaguirre@ti.com>

Print video device node names in dot diagrams

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

main.c: fix spaces check in parse_pad()

Check for spaces before colon also when entity is numeric.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>

Print crop rectangle when printing the topology

For subdev pads that support cropping, print the active crop rectangle
in media_print_topology_text().

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

V4L2 subdev frame interval support

Extend the V4L2 media bus format syntax to support an optional frame interval
value. If specified, the frame interval is set on the subdev.

Signed-off-by: Todor Tomov <ttomov@mm-sol.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

V4L2 subdev crop support

Extend the V4L2 media bus format syntax to support an optional crop
rectangle. If specified, the crop rectangle is set on the subdev pad.

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

Enumerate entities using MEDIA_ENTITY_ID_FLAG_NEXT

Entity IDs don't have to be contiguous. Use the new flag-based
enumeration system instead of looping over consecutive IDs.

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

strerror takes a non-negative integer

Negative error codes were passed untouched to strerror which then
reported an unknown error. Use the positive error code instead.

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

Don't exit with an error when called with -h

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

Print active formats when printing device topology

When run with the -p argument, print the active format on all subdev
pads.

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

Add --print-dot option to print topology as a dot graph

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

Don't set formats on remote pads if the remote entity is not a subdev

When setting format on an output pad, the same format is automatically
set on the remote pads, if any. As the format setting API is
subdev-specific, skip remote pads that belong to a non-subdev entity.

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

Zero structures before filling them

The parse_format function fills a v4l2_mbus_framefmt structure but
doesn't zero it beforehand. This results in some fields being
uninitialized. Fix this.

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

Return -errno in functions returning a negative error code

errno is a positive value, fix functions that return errno when they are
expected to return a negative error code.

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