| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | headless has quit IRC (Quit: headless) | [22:44] |
| pinchartl | mchehab: ping | [01:10] |
| ...... (idle for 25mn) | ||
| mchehab: I've reviewed v6
I haven't commented on 4/8 and 5/8, but the comments I made on patches 1/8 to 4/8 apply there | [01:35] | |
| mchehab | pinchartl: thanks for the review. I'll take a look on them tomorrow | [01:42] |
| ....................................................................................................................... (idle for 9h54mn) | ||
| pinchartl: FYI, we can't use ftrace due to things like: http://pastebin.com/3v2T2Wt0 | [11:36] | |
| ribalda | hverkuil: Hello, thanks for your review!
one fast question. Can I modify saa7164 so it accespts CLASS_MPEG and V4L2_CTRL_WHICH_CUR_VAL ? | [11:43] |
| hverkuil | ribalda: I wouldn't spend time on it. It should be converted to the control framework instead. | [11:46] |
| ribalda | it is not a big deal. Takes two second to change it | [11:48] |
| hverkuil | don't bother. Leaving this out is just another incentive to finally convert this driver. | [11:53] |
| ............ (idle for 57mn) | ||
| ribalda: I'm working on a quick saa7164 conversion to the ctrl framework.
couldn't stand it anymore :-) | [12:50] | |
| ribalda | hverkuil: could you take a fast look to this? https://github.com/ribalda/linux/commit/41e45f8c4ae61e26f72e94d6bb92566790e1b95a
is this what you mean on your first mail? | [12:55] |
| hverkuil | ribalda: yes, but you forgot to do the same update in Documentation/
Just do a git grep on the full kernel so you're certain you got it all. Note: the only place where ctrl_class should still appear is in the VIDIOC_G_EXT_CTRLS DocBook documentation. | [12:57] |
| ribalda | hverkuil: Better now? https://github.com/ribalda/linux/commit/e801d93820d31ce7aafc610a620e7610f860f0a8 | [13:00] |
| hverkuil | yes | [13:02] |
| ribalda | ok, I pack it and ship it as v2. Thanks for the hyperfast review :). | [13:03] |
| hverkuil | oh, one more:
in v4l2-ioctl.c, v4l_print_ext_controls, the pr_cont call uses "class=0x%x", that should become "which=0x%x" | [13:03] |
| ribalda | goot catch, sorry about that | [13:04] |
| .......................................... (idle for 3h27mn) | ||
| pinchartl | mchehab: I'm not sure to see why that prevents using ftrace :-) | [16:31] |
| mchehab | did you see my email?
need to get this at serial console, before Kernel hangs due to some OOPS and printed debug messages should be displayed at the order they occur | [16:32] |
| pinchartl | no I haven't checked the e-mails yet | [16:33] |
| mchehab | pinchartl: btw, you mentioned about DocBook compilation on one of the emails... where are the media controller DocBook header files located after calling make htmldocs? | [16:37] |
| pinchartl | mchehab: good question :-)
let me check (I'm in a meeting so I'll be a bit slow) | [16:38] |
| mchehab | I tried here:
ack -a media_entity_operations Documentation/DocBook/ but it didn't find anything | [16:43] |
| pinchartl | it might not be there | [16:43] |
| mchehab | and media_entity_operations has the tags | [16:43] |
| pinchartl | by the way make htmldocs fails on the media docbook on master :-/ | [16:43] |
| mchehab | just fixed that | [16:43] |
| pinchartl | ok :-)
let me pull | [16:43] |
| mchehab | a </para> missing
I sent the patch not merged upstream yet | [16:44] |
| pinchartl | ok
let me apply it then :-) | [16:44] |
| mchehab | ok | [16:44] |
| pinchartl | make htmldocs still fails :-/ | [16:55] |
| mchehab | well, make htmldocs currently causes *LOTS* of warnings
due to a bug at htmlto s/htmlto/xmlto/ what I do to generate the main xml is: make DOCBOOKS=media_api.xml htmldocs 2>&1 | grep -v "element.*: validity error : ID .* already defined" (media stuff) | [16:55] |
| pinchartl | warning: failed to load external entity "/home/laurent/src/kernel/media.git/Documentation/DocBook/pixfmt-y16-be.xml"
/home/laurent/src/kernel/media.git/Documentation/DocBook/pixfmt.xml:1513: parser error : Failure to process entity sub-y16-be &sub-y16-be; ^ /home/laurent/src/kernel/media.git/Documentation/DocBook/pixfmt.xml:1513: parser error : Entity 'sub-y16-be' not defined &sub-y16-be; | [16:57] |
| mchehab | $ make htmldocs
GEN Documentation/DocBook//v4l2.xml rm -rf Documentation/DocBook/index.html; echo '<h1>Linux Kernel HTML Documentation</h1>' >> Documentation/DocBook/index.html && echo '<h2>Kernel Version: 4.2.0-rc2</h2>' >> Documentation/DocBook/index.html && cat Documentation/DocBook/80211.html Documentation/DocBook/alsa-driver-api.html Documentation/DocBook/crypto-API.html Documentation/DocBook/debugobjects.html Documentation/DocBook/device-drivers.html Documentation/DocBook/deviceiobook.html Documentation/DocBook/drm.html Documentation/DocBook/filesystems.html Documentation/DocBook/gadget.html Documentation/DocBook/genericirq.html Documentation/DocBook/kernel-api.html Documentation/DocBook/kernel-hacking.html Documentation/DocBook/kernel-locking.html Documentation/DocBook/kgdb.html Documentation/DocBook/libata.html Documentation/DocBook/librs.html Documentation/DocBook/lsm.html Documentation/DocBook/media_api.html Documentation/DocBook/mtdnand.html Documentation/DocBook/networking.html Documentation/DocBook/rapidio.html Documentation/DocBook/regulator.html Documentation/DocBook/s390-drivers.html Documentation/DocBook/scsi.html Documentation/DocBook/sh.html Documentation/DocBook/tracepoint.html Documentation/DocBook/uio-howto.html Documentation/DocBook/usb.html Documentation/DocBook/w1.html Documentation/DocBook/writing-an-alsa-driver.html Documentation/DocBook/writing_musb_glue_layer.html Documentation/DocBook/writing_usb_driver.html Documentation/DocBook/z8530book.html >> Documentation/DocBook/index.html mchehab is using the upstream tree on the above plus the patch I just submitted --- a/Documentation/DocBook/media/dvb/intro.xml +++ b/Documentation/DocBook/media/dvb/intro.xml @@ -164,7 +164,7 @@ are called:</para> from 0, and M enumerates the devices of each type within each adapter, starting from 0, too. We will omit the “ <constant>/dev/dvb/adapterN/</constant>” in the further discussion -of these devices. +of these devices.</para> diff --git a/Documentation/DocBook/media/dvb/intro.xml b/Documentation/DocBook/media/dvb/intro.xml index d30751338493..51db15648099 100644 --- a/Documentation/DocBook/media/dvb/intro.xml +++ b/Documentation/DocBook/media/dvb/intro.xml @@ -167 +167 @@ adapter, starting from 0, too. We will omit the “ -of these devices. +of these devices.</para> | [16:57] |
| pinchartl | weird
unable to parse /home/laurent/src/kernel/media.git/Documentation/DocBook/media_api.xml | [16:58] |
| mchehab | are you using it with O= parameter? | [16:58] |
| pinchartl | no | [16:59] |
| mchehab | weird | [16:59] |
| pinchartl | just make htmldocs | [16:59] |
| mchehab | perhaps some leftovers?
try make cleandocs | [16:59] |
| pinchartl | https://paste.kde.org/pp9jqq12c | [17:00] |
| mchehab | really weird
perhaps a broken xmlto toolchain | [17:00] |
| pinchartl | let me cleandocs
it helped, thanks looks like we're not processing any header neither MC nor V4L2 | [17:01] |
| mchehab | yep | [17:04] |
| pinchartl | it shouldn't be difficult to fix
it's not a prerequisite for your patch series, but it would be nice to be able to test kerneldoc in the not too distant future I'll try to reply to your replies later today | [17:04] |
| headless | reply to the replies and answer the answers :-) | [17:08] |
| hverkuil | intro.xml is broken, there is a missing </para>. We never changed that, so it comes from upstream.
noticed the same problem yesterday never mind. I see the patch. | [17:08] |
| pinchartl | commit ed099f66dadd8bac93571fb28b05bdae066b39a2
Author: Masanari Iida <standby24x7@gmail.com> Date: Mon Jul 13 20:36:50 2015 -0300 [media] DocBook: Fix typo in intro.xml it has your SoB ;-) | [17:09] |
| mchehab | pinchartl: adding things there are easy...
but one should do the hardwork of fixing the already broken things http://pastebin.com/JcXsqcVH just with core stuff | [17:14] |
| pinchartl | that really shows we have never compiled it :-) | [17:19] |
| mchehab | yep
patch enabling it sent to ML won't apply until someone have time to fix things Actually, just two errors: Error(.//drivers/media/dvb-core/dvb_math.h:35): duplicate section name 'example' Error(.//drivers/media/dvb-core/dvb_math.h:39): duplicate section name 'example' easily fixable, it seems still lots of: .//include/media/videobuf2-memops.h - Document generation inconsistency at the generated docbook | [17:19] |
| pinchartl | as long as compilation doesn't break we can fix that as we go | [17:28] |
| mchehab | int - callback function for use with vb2_thread
Synopsis typedef int ( * vb2_thread_fnc); Arguments vb2_thread_fnc -- undescribed -- Description This is called whenever a buffer is dequeued in the thread. that's funny... we just invented a function called "int" :) somewhere in VB2 pinchartl: I guess the best is to add for now just the stuff that minimally produces something useable... fix then and move to the other stuff that are just plain broken (even if compilation won't cause errors) like this: http://pastebin.com/XP0s3WJr still, lots of warnings to fix: http://pastebin.com/Ld916dFi pinchartl, hverkuil: sent a version 2 of the patch enabling it | [17:30] |
| pinchartl | mchehab: agreed
it should also be in a clearly separated section of the docbook documentation as it documents the in-kernel API, not the userspace API | [17:44] |
| mchehab | pinchartl: it is
I added it at the device-drivers section, just after Sound drivers | [17:46] |
| ....... (idle for 32mn) | ||
| shuah | mchehab: I haven't had a chance to review all the patches yet. It might be easier to review them with a use-case - is it possible to show how these interfaces re used
btw - we can discuss this later in detail. I am in a meeting and have to head to the airport soon | [18:19] |
| *** | shuah has quit IRC (Quit: shuah) | [18:20] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |