[10:35] *** ChanServ sets mode: +v hverkuil [10:53] *** ChanServ sets mode: +v hverkuil [11:01] *** ChanServ sets mode: +v hverkuil [11:36] *** ChanServ sets mode: +v hverkuil [12:00] <mchehab> hi all [12:02] <mchehab> I've been very busy this week, and got a cold... didn't have any time yet to handle pull requests. I'll try to do it tomorrow [12:02] <mchehab> so, if you have anything pending, please let me know [12:06] <syoung> hello [12:07] <pinchartl> mchehab: I hope you'll recover soon [12:08] <mchehab> thanks! I'm almost recovered today. Had to change the medicine yesterday. After that, I'm feeling a lot better [12:09] <mchehab> as I warned last week, I'll be travelling again next week, where I'll be staying behind a very restricted firewall most of the time [12:10] <mchehab> probably won't be able to apply patches upstream [12:10] <syoung> I would like some advice please. Is there anything I can do (or do differently) to get my patch for man-pages accepted? See https://www.spinics.net/lists/linux-media/msg134601.html [12:10] <mchehab> that's why I expect to be able to do it tomorrow, but I need first to finish some presentations for the event [12:11] <mchehab> syoung: good question... I never sent any patches to Alec. My contact with him was during a Kernel Summit, 2 years ago [12:11] <mchehab> during a BoF about documentation [12:12] <mchehab> Maybe man pages are maintained by someone else those days [12:12] <mchehab> you could try to e-mail to people at linux-doc@vger.kernel.org (and Jonathan) [12:13] <mchehab> they may eventually be able to help you to get it merged at manpages upstream [12:14] <mchehab> https://lwn.net/Articles/680111/ [12:14] <mchehab> it seems that, up to 2016, Michael was doing the man page releases [12:15] <mchehab> hmm... [12:15] <mchehab> https://www.spinics.net/lists/linux-man/msg13017.html [12:16] <mchehab> he forwarded your patch to Alec. I would try to ping them both about that [12:16] <syoung> unfortunately, Alec replied months later and only to me, not to the list [12:16] <mchehab> what did he replied? [12:17] <syoung> see: https://www.spinics.net/lists/linux-man/msg13116.html [12:18] <syoung> maybe I should have phrased that better [12:20] <mchehab> sorry, I guess I missed something... [12:21] <mchehab> did Alec reply to you in priv or not? If he did, what did he say? [12:22] <syoung> Alec replied in private and said: [12:22] <syoung> > Yes, sorry, must have missed your message. These ioctl are used to [12:22] <syoung> > exist, but are no more and should be removed. [12:22] <syoung> I replied to that and cc the list again [12:23] <syoung> the bpf man pages are really out of date, looks like they're having trouble getting things merged too: https://www.spinics.net/lists/linux-man/msg13169.html [12:27] <mchehab> hmm.. LIRC_CAN_REC_RAW still exists [12:27] * mchehab is looking at the patch [12:27] <mchehab> also LIRC_CAN_REC_PULSE [12:28] <mchehab> LIRC_CAN_SET_REC_FILTER too [12:28] <mchehab> and LIRC_CAN_SEND_MODE2 [12:28] <mchehab> why are you removing them from the man-pages? [12:28] <mchehab> the remaining of the patch looks OK on my eyes [12:31] <mchehab> syoung: anyway, I'm answering the e-mail [12:31] <mchehab> with the above notes. It is worth to have your comments about that inside the thread [12:33] <mchehab> sent [12:34] <mchehab> anything else? [12:40] <syoung> mchehab: thank you. LIRC_CAN_REC_RAW etc do still exist in the header but have no implementation; they just exist to not break builds. [12:44] <syoung> anyway, I'll reply so it's in the thread. Thank you again. [12:49] <mchehab> anytime [16:34] *** ChanServ sets mode: +v mchehab`