[04:21] *** Muzer has quit IRC (Ping timeout: 256 seconds) [09:56] *** macmaN has quit IRC (Read error: Connection reset by peer) [10:01] <hfr> hverkuil: Hi Hans, I have a question regarding documentation generation regarding Nicolas' comment in http://www.mail-archive.com/linux-media@vger.kernel.org/msg107955.html [10:02] <hfr> hverkuil: currently, I have copied/pasted manually structures documentation from v4l2-controls.h in Documentation/media/uapi/v4l/extended-controls.rst [10:03] <hfr> hverkuil: I understand from Nicolas that a way should exist to do this documentation directly from code using special syntax in v4l2-controls.h header file [10:04] <hfr> hverkuil: could you point me out an example on how to do this ? [10:30] <hverkuil> hfr: I'll look at that later today. [10:48] <hfr> hverkuil: thanks Hans [11:27] <hverkuil> hfr: see for example struct v4l2_bt_timings in videodev2.h. The comment block starting with '/**' is parsed when building the documentation. [11:27] <hverkuil> See also Documentation/kernel-doc-nano-HOWTO.txt [13:03] <hfr> hverkuil: ok thanks [13:07] *** henr_k is now known as hnrk [13:37] <hverkuil> mchehab: I assume that the window for 4.10 patches has closed? It concerns this patch: https://patchwork.linuxtv.org/patch/39306/ [13:37] <hverkuil> It's OK to do this via stable later. [13:37] *** hfr has left [14:05] <hfr> hverkuil: I have checked v4l2_bt_timings, with /** and find the related hyperlink in Documentation/output/media/uapi/v4l/videodev.html [14:06] <hfr> hverkuil: but I can't find the same for file v4l2-controls.h [14:07] <hfr> hverkuil: the file v4l2-controls.h is not exported in documentation as a browsable html file as it is for videodev2.h in videodev.html [14:08] <hfr> hverkuil: so even if I change my /* to /** inside v4l2-controls.h, this will have not effect in doc [14:09] <hfr> hverkuil: (there are no occurence of "/**" inside v4l2-controls.h) [14:15] <hfr> hverkuil: I would suggest to keep my comment in /**/ as it is done for rest of this file, complete documentation is in extended-controls.rst file [15:02] <mchehab> hverkuil: yes, window is closed [15:04] <mchehab> if that revert patch is for a regression fix, I can pick it after the merge window (or even during it) [16:12] *** prabhakarlad has left [17:50] <faLUCE> hello. When I call ioctl(fd, VIDIOC_DQBUF, &buf) for a PLANAR frame, does it store plane1's data in buf.index == 0, plane2's data in buf.index == 1 etc. ? [17:52] <faLUCE> hello. When I call ioctl(fd, VIDIOC_DQBUF, &buf) for a PLANAR frame, does it store plane1's data FOR buf.index == 0, plane2's data FOR buf.index == 1 etc. ? [19:57] <headless> anyone seen Renesas IMR driver patch? [19:57] <headless> not seeing it in the archives... [19:58] <headless> sent with 'quilt mail'