[07:52] <ribalda> hverkuil: Hi! Why this patch is susperseeded ? https://patchwork.linuxtv.org/patch/35566/ [07:53] <hverkuil> no idea [07:53] <hverkuil> put back to NEW. [07:53] <ribalda> ok. Thanks! [07:57] <hverkuil> mchehab: can you pull from https://patchwork.linuxtv.org/patch/35614/ and apply https://patchwork.linuxtv.org/patch/35615/? [07:58] <hverkuil> That should take care of a bunch of CEC bugs and minor API changes that I found while creating API compliance tests for CEC. [08:16] <Renault> hverkuil: it's possible to test a fix for my gs1662 driver (kconfig dependency) with the build system before to send to the ML? [08:32] <hverkuil> You're missing a dependency on VIDEO_V4L2, that's all. [08:33] <Renault> and VIDEO_V4L2_SUBDEV_API [08:33] <hverkuil> Take a look at drivers/media/i2c/Kconfig: everything is under if VIDEO_V4L2. You can do the same for spi/Kconfig [08:34] <hverkuil> BTW, I plan to review your patch later this week. [08:35] <Renault> ok, I'll wait your review ;) [08:35] <Renault> thanks [09:27] <hverkuil> mchehab: I've updated my CEC doc-rst patch to v3: https://patchwork.linuxtv.org/patch/35618/ [14:30] *** Peetz0r has quit IRC (Max SendQ exceeded) [14:31] *** darvon has quit IRC (Max SendQ exceeded) [15:17] *** benjiG has left [16:20] <mchehab> hmm... 35615 is marked as superseded [16:20] <mchehab> hverkuil: did you meant to say 35618? [16:23] <hverkuil> it's superseded by 35618, yes. [16:30] <mchehab> hverkuil: please try to keep the 80 cols at the rst books. That makes easier to review [16:31] <mchehab> another thing: [16:31] <mchehab> :c:func:`clock_gettime(2)` [16:31] <mchehab> perhaps we'll need to change it to :cpp:func: [16:31] <mchehab> when the documentation for this function arrives to Sphinx [16:34] <mchehab> anyway, patches merged [20:00] *** awalls1 has left