Some structs are pointed via the typedef. As we replaced
those references, fix them.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Trivially fix those broken references, by copying the structs
fron the header, just like other API documentation at the
DVB side.

This doesn't have the level of quality used at the V4L2 side
of the API, but, as this documents a deprecated API, used
only by av7110 driver, it doesn't make much sense to invest
time making it better.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Several links are broken, as they were using the typedef
name, instead of using the corresponding structs. Fix them.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Add a table describing the fields on this struct, based
on ca.h header.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Add documentation for struct ca_slot_info and for the two
sets of define used by it, according with what's there at the
ca.h header.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are no descriptions at ca.h header for this struct.
Yet, as we want to get rid of the warnings, let's add a
boilerplate, with just the struct types and fields.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The documentation follows what's there at the ca.h header.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Add a flat-table describing struct ca_caps, as found at
the source file.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Now that we moved away from the :ref: type of references,
we need to update the exceptions lists.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

One markup tag is wrong here.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are two warnings that are due to functions that has long
gone:

	Documentation/media/kapi/v4l2-subdev.rst:417: WARNING: c:func reference target not found: v4l2_i2c_new_subdev_cfg
	Documentation/media/kapi/v4l2-subdev.rst:436: WARNING: c:func reference target not found: v4l2_i2c_new_probed_subdev

Update the documentation to remove those.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Now that scripts/kernel-doc was fixed to parse the typedef
argument used here, let it produce documentation.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Fix some minor issues at the documentation tags on this file,
adding cross-references where needed, and fixing some broken
ones.

While here, fix a few spaces before tabs to make Checkpatch happier.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The "struct" were inside the reference, causing it to break.
Acked-by: NLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are several undocumented functions here; document them.

While here, make checkpatch.pl happy.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_state_change
Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_state_change
Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_lost_msgs
Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_lost_msgs
Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_state_change
Documentation/output/cec.h.rst:6: WARNING: c:type reference target not found: cec_event_lost_msgs
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Now that the uAPI is using c:type, let's use it here too.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Change the parse-headers.pl and the corresponding files to use
the C domain for enum references.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The prepare_ext_ctrls() function is actually internal to the
v4l2-ctrls.c implementation, so it doesn't have a declaration
for the kAPI header to reference it.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The V4L version 1 structures had long gone from the Linux Kernel.
It doesn't make sense to use cross-references for them, as they
won't be found.

So, get rid of them.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

instead of declaring the uAPI structs using usual refs, e. g.:
	.. _foo-struct:

Use the C domain way:
	.. c:type:: foo_struct

This way, the kAPI documentation can use cross-references to
point to the uAPI symbols.

That solves about ~100 undefined warnings like:
	WARNING: c:type reference target not found: foo_struct
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The typedefs and a macro are not defined. While here, improve a
few bits on the documentation.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are some structs/enums that aren't documented via
kernel-doc markup. Add documentation for them.

Fix those warnings:
./include/media/rc-map.h:103: WARNING: c:type reference target not found: rc_map_list
./include/media/rc-map.h:110: WARNING: c:type reference target not found: rc_map_list
./include/media/rc-map.h:117: WARNING: c:type reference target not found: rc_map
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are some functions/macros used by the mediactl documentation
that are alien to the media subsystem. Ignore them.

After this patch, the media core will only complain about this
static function:
	Documentation/media/kapi/mc-core.rst:97: WARNING: c:func reference target not found: media_devnode_release
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are several issues on the documentation:
  - the media.h header were not properly referenced;
  - verbatim expressions were not properly marked as such;
  - struct member references were wrong;
  - some notes were not using the right markup;
  - a comment that were moved to the kernel-doc markup were
    duplicated as a comment inside the struct media_entity;
  - some args were not pointing to the struct they're using;
  - macros weren't documented.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The kernel-doc parser didn't handle well the private:
tag. Rewrite it.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

The cross-reference to "struct media_pad" was pointing to
a place that doesn't exist. Fix it, and adjust the second
reference on the same paragraph to use the same text.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

demux.h was lacking documentation for the DMX_FE_ENTRY macro:
	./drivers/media/dvb-core/demux.h:511: WARNING: c:func reference target not found: DMX_FE_ENTRY

While here, get rid of unused parameters and fix a few minor issues
at the header file.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are a few define macros not documented, because the ReST
output was causing more warnings.

Now that this got fixed, document them. While here, fix the
remaining coding style issues.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Instead of keeping those notes at the file on a non-structured
way, move them to dtv-core.rst, using the proper ReST tags.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There are several functions there there weren't properly
documented. Add documentation for them.

While here, make checkpatch.pl happier.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

There a some other types and functions that aren't declared
inside the media document but are elsewhere. Add them to the
ignore list.
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Instead of keep using the normal reference, move to the C
domain ones. Using C domains everywhere will allow
cross-references between kAPI and uAPI docs.
Acked-by: NJonathan Corbet <corbet@lwn.net>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Add a parser for the --debug option, in order to allow
seeing what the parser is doing.
Acked-by: NJonathan Corbet <corbet@lwn.net>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: NYannick Fertre <yannick.fertre@st.com>
Signed-off-by: NJean-Christophe Trotin <jean-christophe.trotin@st.com>
Acked-by: NPeter Griffin <peter.griffin@linaro.org>
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Update the cobalt EDID, fixing various incorrect values (wrong name,
product code, various video capabilities).
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Update the vivid EDID, fixing various incorrect values (wrong name,
product code, various video capabilities).
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

As warned by smatch:
	drivers/media/pci/tw5864/tw5864-core.c:160 tw5864_h264_isr() error: double lock 'irqsave:flags'
	drivers/media/pci/tw5864/tw5864-core.c:174 tw5864_h264_isr() error: double unlock 'irqsave:flags'

Two different spinlocks are obtained, so having two calls is correct,
but second irqsave is superfluous, and using same "flags" variable is
just wrong.
Reported-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: NAndrey Utkin <andrey.utkin@corp.bluecherry.net>
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

This reverts commit 617e901a ("[media] tw5864: remove double irq lock code").

That commit was rejected by Andrey Utkin <andrey_utkin@fastmail.com>,
but it had already been merged. Revert and apply Andrey's corrected
patch next.
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>

Fix some small things:
    - clean up setup function
    - use MSGEND instead of 0xfe
    - don't assign "return value" from cec_phys_addr to err,
      it has return type void.
Signed-off-by: NJohan Fjeldtvedt <jaffe1@gmail.com>
Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>