Am 07.09.2016 um 19:59 schrieb VDR User :
>> It is broken (see below). Have you ever used dvbv5 tools with vdr format
>> output or did you know a "VDR user" who is using dvbv5-scan and not wscan?
>
> In my experience the v4l scanner, wscan, and VDR's internal scanner
> has
On 07/09/16 23:25, Laurent Pinchart wrote:
> Hello,
Thanks for all of your efforts here!
> Here's the third version of the Renesas R-Car FDP1 driver.
>
> The FDP1 (Fine Display Processor) is a hardware memory-to-memory de-interlacer
> device, with capability to convert from various YCbCr/YUV
The driver used integers for what boolean would have been a better fit.
Use boolean instead.
Signed-off-by: Sakari Ailus
---
drivers/media/i2c/ad5820.c | 19 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git
Hi Hans,
On Thu, 2016-09-08 at 09:21 +0200, Hans Verkuil wrote:
> Hi Tiffany,
>
> On 09/07/2016 08:56 AM, Tiffany Lin wrote:
> > This patch series add Mediatek compressed block format V4L2_PIX_FMT_MT21C,
> > the
> > decoder driver will decoded bitstream to V4L2_PIX_FMT_MT21C format.
> >
> >
The ov5645 sensor from Omnivision supports up to 2592x1944
and CSI2 interface.
The driver adds support for the following modes:
- 1280x960
- 1920x1080
- 2592x1944
Output format is packed 8bit UYVY.
Signed-off-by: Todor Tomov
---
drivers/media/i2c/Kconfig | 12 +
Add the document for ov5645 device tree binding.
Signed-off-by: Todor Tomov
---
.../devicetree/bindings/media/i2c/ov5645.txt | 52 ++
1 file changed, 52 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5645.txt
This is the sixth version of the OV5645 camera sensor driver patchset.
Changes since version 5 include:
- external clock frequency set in DT;
- added v4l2_subdev_pad_ops.init_cfg function to initialize formats;
- current sensor mode not updated if set_fmt is TRY (not ACTIVE);
- other small
On 07/09/16 23:25, Laurent Pinchart wrote:
> The menu control selects the operation mode of a video deinterlacer. The
> menu entries are driver specific.
Excellent. This makes a lot more sense than adding a custom control in
the driver :D
> Signed-off-by: Laurent Pinchart
On 07/09/16 23:25, Laurent Pinchart wrote:
> From: Geert Uytterhoeven
>
> drivers/media/platform/rcar_fdp1.c:1972:2: warning: initialization from
> incompatible pointer type
> .queue_setup = fdp1_queue_setup,
> ^
>
On 09/08/16 11:11, Tiffany Lin wrote:
Hi Hans,
On Thu, 2016-09-08 at 09:21 +0200, Hans Verkuil wrote:
Hi Tiffany,
On 09/07/2016 08:56 AM, Tiffany Lin wrote:
This patch series add Mediatek compressed block format V4L2_PIX_FMT_MT21C, the
decoder driver will decoded bitstream to
FYI: I'm missing a MAINTAINERS entry for this driver.
Please add one when you post v6.
Thanks!
Hans
On 08/30/2016 02:25 PM, Minghsiu Tsai wrote:
> Changes in v5:
> - Add ack in the comment of dts patch
> - Fix s/g_selection()
> - Separate format V4L2_PIX_FMT_MT21C into new patch
>
Hi Tiffany,
On 09/07/2016 08:56 AM, Tiffany Lin wrote:
> This patch series add Mediatek compressed block format V4L2_PIX_FMT_MT21C, the
> decoder driver will decoded bitstream to V4L2_PIX_FMT_MT21C format.
>
> User space applications could use MT8173 MDP driver to convert
> V4L2_PIX_FMT_MT21C
Hi Hans,
On Wed, 2016-09-07 at 11:23 +0200, Hans Verkuil wrote:
> On 09/07/16 08:56, Tiffany Lin wrote:
> > Add V4L2_PIX_FMT_MT21C documentation
> >
> > Signed-off-by: Tiffany Lin
> > ---
> > Documentation/media/uapi/v4l/pixfmt-reserved.rst |6 ++
> > 1 file
On Thu, Sep 08, 2016 at 01:25:01AM +0300, Laurent Pinchart wrote:
> The S_FMT and TRY_FMT handlers in multiplane mode attempt at clearing
> the reserved fields of the v4l2_format structure after the pix_mp
> member. However, the reserved fields are inside pix_mp, not after it.
>
> Signed-off-by:
On Wed, 2016-09-07 at 18:10 +0100, Colin King wrote:
> From: Colin Ian King
>
> pdev is being null checked, however, prior to that it is being
> dereferenced by platform_get_drvdata. Move the assignments of
> vpu and run to after the pdev null check to avoid a
On 09/07/2016 10:38 PM, Linus Walleij wrote:
> On Thu, Sep 1, 2016 at 1:47 PM, Wolfram Sang wrote:
>> On Thu, Sep 01, 2016 at 01:39:16PM +0200, Sylwester Nawrocki wrote:
>
>>> Since commit 04f59143b571161d25315dd52d7a2ecc022cb71a
>>> ("i2c: let I2C masters ignore their
Hi Jon/Markus,
Em Mon, 29 Aug 2016 16:12:39 +0200
Markus Heiser escreveu:
> Am 29.08.2016 um 15:13 schrieb Mauro Carvalho Chehab
> :
>
> > A macro definition is mapped via .. c:function:: at the
> > ReST markup when using the following
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
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: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-get-slot-info.rst | 84 ++-
1 file changed,
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: Mauro Carvalho Chehab
---
Documentation/media/kapi/mc-core.rst | 5 +++--
1 file
One markup tag is wrong here.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-subdev.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
index 6e1d044e3ee8..2c1e328ccb1d 100644
---
One reference there is still using :ref:. Fix it, to solve this
warning:
Documentation/media/uapi/mediactl/media-ioc-g-topology.rst:236: WARNING:
undefined label: media-v2-intf-devnode (if the link has no caption the label
must precede a section header)
Signed-off-by: Mauro Carvalho Chehab
Fix some CEC cross references that are broken.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/cec.h.rst.exceptions | 6 --
Documentation/media/uapi/cec/cec-ioc-adap-g-log-addrs.rst | 10 +-
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: Mauro Carvalho Chehab
c:type is not the right tag for function definitions.
Replace them by .. c:function::
Signed-off-by: Mauro Carvalho Chehab
---
.../media/uapi/v4l/libv4l-introduction.rst | 62 --
1 file changed, 35 insertions(+), 27 deletions(-)
diff --git
Several structs were renamed or removed during V4L2 development.
Don't try to cross-reference the legacy ones.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/hist-v4l2.rst | 24
1 file changed, 12 insertions(+), 12
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:
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: Mauro Carvalho Chehab
---
include/media/v4l2-ctrls.h | 2 +-
1 file changed, 1 insertion(+), 1
The kernel-doc parser didn't handle well the private:
tag. Rewrite it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/demux.h | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/media/dvb-core/demux.h
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
Several links are broken, as they were using the typedef
name, instead of using the corresponding structs. Fix them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-get-cap.rst| 4 ++--
Documentation/media/uapi/dvb/ca-get-descr-info.rst
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: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-get-msg.rst | 33
There are several undocumented functions here; document them.
While here, make checkpatch.pl happy.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/conf_nitpick.py | 1 +
include/media/v4l2-ioctl.h | 502 +---
2
Add a flat-table describing struct ca_caps, as found at
the source file.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-get-cap.rst | 29 -
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git
Now that we moved away from the :ref: type of references,
we need to update the exceptions lists.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/audio.h.rst.exceptions | 6 +-
Documentation/media/ca.h.rst.exceptions| 32
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: Mauro Carvalho Chehab
---
Documentation/media/conf_nitpick.py | 13 +
1 file changed, 9 insertions(+), 4
The :c:type: references point to the structure name, and not to
struct foo.
Fixed via this shell script:
for i in `find Documentation/media -type f`; do perl -ne 'if
(s/\:c\:type\:\`struct\s*(\S+)\`/struct :c:type:`$1`/) {
s/struct\s+struct/struct/;
Before this patch series, there is ~400 warnings when building docs in nitpick
mode.
That means ~400 broken links.
This patch series move the C references to use Sphinx C domain, instead of
using :ref:, add more documentation to some V4L2 headers and fix several broken
links. After this series,
The documentation follows what's there at the ca.h header.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-get-descr-info.rst | 23 +-
1 file changed, 22 insertions(+), 1 deletion(-)
diff --git
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.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 113
Add a parser for the --debug option, in order to allow
seeing what the parser is doing.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git
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:
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: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_ringbuffer.h
There are several broken references there, due to the conversion to
C domain. Fix them using this shell script and manually adjust what's
broken:
# funcs is a file with the broken functions/references
for i in $(cat funcs|sort|uniq|perl -ne 'print "$1\n" if (m/(\S+)$/)');
do
Fix some minor issues at the documentation tags on this file,
adding cross-references where needed, and fixing some broken
ones.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-device.h | 54 ++---
1 file changed, 27
The "struct" were inside the reference, causing it to break.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/kapi/v4l2-dev.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/media/kapi/v4l2-dev.rst
Some structs are pointed via the typedef. As we replaced
those references, fix them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/dmx-get-caps.rst | 5 ++---
Documentation/media/uapi/dvb/dmx-set-source.rst | 2 +-
2 files changed, 3
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: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/diff-v4l.rst | 38
A macro definition is mapped via .. c:function:: at the
ReST markup when using the following kernel-doc tag:
/**
* DMX_FE_ENTRY - Casts elements in the list of registered
* front-ends from the generic type struct list_head
* to the
Change the parse-headers.pl and the corresponding files to use
the C domain for enum references.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/hist-v4l2.rst | 2 +-
Documentation/media/uapi/v4l/subdev-formats.rst | 2 +-
Instead of using c:type:`struct foo `, use:
struct c:type:`foo`
This patch was generated via this shell script:
for i in `find Documentation/media -type f`; do perl -ne 'if
(m/\:c\:type\:\`struct\s+(\S+)\s*\<(\S+)\>\`/) { $s=$1; $r=$2; if ($s eq $r) {
Now that scripts/kernel-doc was fixed to parse the typedef
argument used here, let it produce documentation.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-dv-timings.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Fix those warnings:
Documentation/media/uapi/cec/cec-ioc-dqevent.rst:124: WARNING: c:func
reference target not found: clock_gettime(2)
By replacing it with the right function name, using this shell script:
for i in `find Documentation/media -type f`; do sed
The typedefs and a macro are not defined. While here, improve a
few bits on the documentation.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-ctrls.h | 51 +++---
1 file changed, 44 insertions(+), 7 deletions(-)
Add a table describing the fields on this struct, based
on ca.h header.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/dvb/ca-set-pid.rst | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git
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:
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
The struct v4l2_mpeg_vbi_ITV0 is identical to struct v4l2_mpeg_vbi_itv0,
except by its size, and it is documented at the same place at the
book.
Fix cross reference for it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/dev-sliced-vbi.rst | 6
Use C cross-references to mention the V4L2 API calls on all
places it occurs inside this file.
While here, also mark constants as such.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/uapi/v4l/libv4l-introduction.rst | 33 +-
1 file
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: Mauro Carvalho Chehab
---
Documentation/media/kapi/dtv-core.rst | 38 +++--
On Mon, 2016-09-05 at 12:17 +0200, Hans Verkuil wrote:
> On 08/30/2016 02:25 PM, Minghsiu Tsai wrote:
> > Add MDP driver for MT8173
> >
> > Signed-off-by: Minghsiu Tsai
> > ---
> > drivers/media/platform/Kconfig| 17 +
> >
Hi Mauro,
Thank you for the patch.
On Thursday 08 Sep 2016 09:03:44 Mauro Carvalho Chehab wrote:
> The "struct" were inside the reference, causing it to break.
>
> Signed-off-by: Mauro Carvalho Chehab
Acked-by: Laurent Pinchart
>
Hi Todor,
Thank you for the patch.
On Thursday 08 Sep 2016 12:13:55 Todor Tomov wrote:
> The ov5645 sensor from Omnivision supports up to 2592x1944
> and CSI2 interface.
>
> The driver adds support for the following modes:
> - 1280x960
> - 1920x1080
> - 2592x1944
>
> Output format is packed
Hi Todor,
Thank you for the patch.
On Thursday 08 Sep 2016 12:13:54 Todor Tomov wrote:
> Add the document for ov5645 device tree binding.
>
> Signed-off-by: Todor Tomov
> ---
> .../devicetree/bindings/media/i2c/ov5645.txt | 52 +++
> 1 file
On Thu, 8 Sep 2016 09:03:22 -0300
Mauro Carvalho Chehab wrote:
> Please notice that patches 2 and 3 touches at
> Documentation/sphinx/parse-headers.pl.
>
> If it is ok for you, I intend to merge those patches (except for patch 1)
> on my tree, including the two
There are several functions there there weren't properly
documented. Add documentation for them.
While here, make checkpatch.pl happier.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_ringbuffer.h | 179
1 file
There's a typo there, causing 4 warnings:
Documentation/media/uapi/rc/lirc-read.rst:26: WARNING: c:type reference
target not found: name
Documentation/media/uapi/rc/lirc-read.rst:26: WARNING: c:type reference
target not found: lirc
Documentation/media/uapi/v4l/func-poll.rst:25: WARNING:
./include/media/rc-map.h:121: WARNING: Inline emphasis start-string
without end-string.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/rc-map.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/media/rc-map.h
Now that the uAPI is using c:type, let's use it here too.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-ctrls.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h
index
Hi linux-firmware maintainers,
The following changes since commit c883a6b6186bb2415761d287cbac773062911212:
rtl_bt: Add firmware and config file for RTL8822BE (2016-08-31 02:38:31 +0100)
are available in the git repository at:
https://github.com/andrewct-chen/linux_fw_vpu_v1.0.2.git
Add a DT binding documentation of MDP for the MT8173 SoC
from Mediatek
Signed-off-by: Minghsiu Tsai
Acked-by: Rob Herring
---
.../devicetree/bindings/media/mediatek-mdp.txt | 109
1 file changed, 109 insertions(+)
create
VPU driver add mdp support
Signed-off-by: Minghsiu Tsai
---
drivers/media/platform/mtk-vpu/mtk_vpu.h |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/media/platform/mtk-vpu/mtk_vpu.h
b/drivers/media/platform/mtk-vpu/mtk_vpu.h
index
Am 08.09.2016 um 13:40 schrieb Mauro Carvalho Chehab :
> Hi Jon/Markus,
>
> Em Mon, 29 Aug 2016 16:12:39 +0200
> Markus Heiser escreveu:
>
>> Am 29.08.2016 um 15:13 schrieb Mauro Carvalho Chehab
>> :
>>
>>> A
Allow the sub-device to be probed asynchronously so a bridge driver that's
waiting for the device can be notified and its .bound callback executed.
Signed-off-by: Javier Martinez Canillas
---
drivers/media/i2c/ov9650.c | 7 ++-
1 file changed, 6 insertions(+), 1
Add MDP driver for MT8173
Signed-off-by: Minghsiu Tsai
---
drivers/media/platform/Kconfig| 17 +
drivers/media/platform/Makefile |2 +
drivers/media/platform/mtk-mdp/Makefile |9 +
Add V4L2_PIX_FMT_MT21C in format list.
Signed-off-by: Minghsiu Tsai
---
drivers/media/platform/mtk-mdp/mtk_mdp_m2m.c |8
drivers/media/platform/mtk-mdp/mtk_mdp_regs.c |4
2 files changed, 12 insertions(+)
diff --git
Changes in v6:
- s_selection() can't set the _DEFAULT and _BOUNDS targets
- Add Maintainers entry
Changes in v5:
- Add ack in the comment of dts patch
- Fix s/g_selection()
- Separate format V4L2_PIX_FMT_MT21C into new patch
Changes in v4:
- Add "depends on HAS_DMA" in Kconfig.
- Fix
Add MDP node for MT8173
Signed-off-by: Minghsiu Tsai
---
arch/arm64/boot/dts/mediatek/mt8173.dtsi | 84 ++
1 file changed, 84 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
Add Minghsiu Tsai, Houlong Wei and Andrew-CT Chen as
maintainers for Mediatek MDP driver
Signed-off-by: Minghsiu Tsai
Signed-off-by: Houlong Wei
Signed-off-by: Andrew-CT Chen
---
MAINTAINERS |9 +
1
Hi
BTW: porting parse-headers to python / get rid of media/Makefile
* https://www.mail-archive.com/linux-media@vger.kernel.org/msg101261.html
I think last relevant comment was from you Mauro:
* https://www.mail-archive.com/linux-media@vger.kernel.org/msg101426.html
should we continue this
MEDIA_CEC is no longer a tristate option, so the user can't actually
choose M. Whether the code is built-in or built as a module is
decided somewhere else.
Signed-off-by: Jean Delvare
Fixes: 5bb2399a4fe4 ("[media] cec: fix Kconfig dependency problems")
Cc: Hans Verkuil
of_node_put() has already been called inside of_graph_get_next_endpoint().
Otherwise we may get warnings like
[ 10.118286] omap3isp 480bc000.isp: parsing endpoint
/ocp/isp@480bc000/ports/port@0/endpoint, interface 0
[ 10.118499] ERROR: Bad of_node_put() on
>> There is already a publicly available VDR repository offering the
>> current stable & developer versions, along with all previous versions:
>> http://www.tvdr.de/download.htm
>
> ?? these are tarballs, where is the version control system?
That would be a question for Klaus, the author of VDR.
> Am 08.09.2016 um 15:05 schrieb Javier Martinez Canillas
> :
>
> Allow the sub-device to be probed asynchronously so a bridge driver that's
> waiting for the device can be notified and its .bound callback executed.
>
> Signed-off-by: Javier Martinez Canillas
Em Wed, 7 Sep 2016 10:59:32 -0700
VDR User escreveu:
> I use nscan, which has easily been the
> most successful of the scanners. An additional benefit to nscan is you
> only supply a single transponder on the command line and it will
> populate a channel list for the entire
There are lots of troubles with the function arguments on this
file. Fix them.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-mem2mem.h | 93 +---
1 file changed, 80 insertions(+), 13 deletions(-)
diff --git
After this series, there are only 3 warnings left:
./include/media/media-entity.h:1053: warning: No description found for
parameter '...'
./include/media/v4l2-mem2mem.h:339: WARNING: c:type reference target not found:
queue_init
./include/media/v4l2-subdev.h:424: WARNING: c:type reference
Check for vb2_ops structures that are only stored in the ops field of a
vb2_queue structure. That field is declared const, so vb2_ops structures
that have this property can be declared as const also.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@r
Check for vb2_ops structures that are only stored in the ops field of a
vb2_queue structure. That field is declared const, so vb2_ops structures
that have this property can be declared as const also.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@r
Fix this warning:
Documentation/media/kapi/mc-core.rst:97: WARNING: c:func reference
target not found: media_devnode_release
The media_device_release() is a function internal to media-devnode.c,
and not exported elsewhere. So, we can't cross-reference it here.
Make it explicit at the
The kernel-doc C parser doesn't like opaque structures. So,
document it on another way.
This should get rid of this warning:
./include/media/v4l2-mem2mem.h:62: error: Cannot parse struct or union!
Signed-off-by: Mauro Carvalho Chehab
---
Check for vb2_ops structures that are only stored in the ops field of a
vb2_queue structure. That field is declared const, so vb2_ops structures
that have this property can be declared as const also.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@r
Document vb2_ops_wait_prepare() and vb2_ops_wait_finish(),
in order to fix those two warnings:
Documentation/media/kapi/v4l2-dev.rst:166: WARNING: c:func reference
target not found: vb2_ops_wait_prepare
Documentation/media/kapi/v4l2-dev.rst:166: WARNING: c:func reference
target
This enum was not documented. Document it.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/videobuf2-core.h | 14 ++
1 file changed, 14 insertions(+)
diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h
index
There are several small issues with the documentation. Fix them,
in order to avoid producing warnings.
While here, also make checkpatch.pl happy.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/videobuf2-core.h | 143 -
1
There are a few issues at the documentation: fields not documented,
bad cross refrences, etc.
Fix them.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/videobuf2-v4l2.h | 52 +++---
1 file changed, 33 insertions(+), 19
Fix this warning:
./include/media/v4l2-flash-led-class.h:103: WARNING: c:type reference
target not found: v4l2_flash_ops
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-flash-led-class.h | 15 +++
1 file changed, 11 insertions(+), 4
There are several functions that were documented at the .c
file. As we only include the headers, we need to move them to
there, in order to have documentation.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/v4l2-core/videobuf2-core.c | 254
There are several functions documented at the C file. Move
them to the header, as this is the one used to build the
media books.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/v4l2-core/videobuf2-v4l2.c | 142 -
Several routines are somewhat documented at v4l2-mem2mem.c
file. Move what's there to the header file.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/v4l2-core/v4l2-mem2mem.c | 128 +---
include/media/v4l2-mem2mem.h | 132
1 - 100 of 106 matches
Mail list logo