| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When | |
|---|---|---|---|
| *** | ChanServ sets mode: +v snawrocki | [09:27] | |
| ............................... (idle for 2h33mn) | |||
| sailus | Hello! | [12:00] | |
| *** | ChanServ sets mode: +v mchehab | [12:01] | |
| mchehab | hi all | [12:01] | |
| hverkuil | Good afternoon!
It's probably just us since Laurent is in the US. | [12:01] | |
| mchehab | and syoung is in vacations
from my side, I don't have anything to discuss I'm waiting for the close of the merge window to start handling patches | [12:02] | |
| hverkuil | mchehab: I have several pull requests pending for you to take once 4.14-rc1 is out. There will probably be one or two more tomorrow. | [12:04] | |
| mchehab | ok | [12:04] | |
| hverkuil | one fixes pull request for 4.14, there is a good chance I'll add another patch to that tomorrow. | [12:04] | |
| sailus | I'll send v13 of the big fwnode lens / flash patchset I've been posting likely tomorrow. The changes will be relatively minor.
hverkuil: Thank you for reviewing it. | [12:05] | |
| mchehab | hverkuil, that fix is a regression against 4.13? | [12:06] | |
| hverkuil | the pull request for fixes that I posted are for 4.14. | [12:06] | |
| mchehab | ok, so no hurry
I'll pick it after -rc1, then anything else? | [12:07] | |
| sailus | Yes.
On a completely different matter --- I remember we discussed that we should document who's mainly responsible for which part of the media subsystem. | [12:08] | |
| mchehab | yes | [12:09] | |
| sailus | I don't think we have this anywhere yet.
I'm not sure if it matters that much to developers though, the patches go through patchwork. But the developers don't know who applied the patch or where did it go to, until it reaches mainline. | [12:09] | |
| mchehab | let's finish the discussions with regards to v4l-utils
then we can place it everywhere | [12:10] | |
| sailus | Ok. | [12:11] | |
| mchehab | btw, someone complained me in priv that he wasn't aware to where to direct e-mails related to v4l-utils
I didn't check, but maybe it is missed on its README | [12:11] | |
| hverkuil | Checked: it's not there. | [12:12] | |
| mchehab | a simple patch should fix it | [12:13] | |
| hverkuil | I have time tomorrow for that. | [12:13] | |
| mchehab | good
with regards to documenting the maintainers, I guess placing it somewhere at wiki would be enough at the developers section | [12:14] | |
| hverkuil | ack | [12:14] | |
| mchehab | hmm... I remember mkrufky added somewhere a page describing the media developers
too informal for me, as it describes hobbies, and stuff like that... but maybe we could change it in order to add what's really relevant for developers e.g. who is responsible for what | [12:15] | |
| hverkuil | For the v4l-utils README: "Any questions/remarks/patches can be sent to the linux-media mailinglist. See https://linuxtv.org/lists.php for more information." | [12:16] | |
| mchehab | Ok. I would also mention v4l-utils wiki pages | [12:17] | |
| hverkuil | more information about the mailinglist
we have a v4l-utils wiki page? I'm not aware of anything v4l-utils specific. | [12:17] | |
| mchehab | yes, we do
I remember I updated a lot of pages there related to libdvbv5 | [12:19] | |
| hverkuil | sailus: does this patch belong to your async patch series? https://patchwork.linuxtv.org/patch/42843/ | [12:20] | |
| mchehab | https://linuxtv.org/wiki/index.php/V4l-utils | [12:20] | |
| hverkuil | And patches 42840-42842 are all obsolete, right? | [12:20] | |
| sailus | hverkuil: It's not there right now.
I can add it there. | [12:21] | |
| hverkuil | sailus: those patches are delegated to me, perhaps it is best if I delegate those 4 to you and you can decide what to do with them. | [12:21] | |
| sailus | I'd say so, yes. | [12:21] | |
| hverkuil | Done. | [12:22] | |
| sailus | hverkuil: Works for me. | [12:22] | |
| hverkuil | mchehab: amazing, I remembered my wiki password :-)
Hmm, hasn't been touched for over 2 years. | [12:23] | |
| sailus | I don't have a wiki account. :-) | [12:24] | |
| mchehab | sailus: easy to create | [12:24] | |
| sailus | I'll request one. | [12:24] | |
| mchehab | account creation are currently moderated, to avoid SPAM | [12:24] | |
| sailus | Makes sense. | [12:25] | |
| mchehab | so, please write a good description about why you need and mention that you're sub-maintaining it
for Johannes to recognize you :-) | [12:25] | |
| sailus | :-) | [12:25] | |
| hverkuil | Proposed README text for wiki: | [12:25] | |
| mchehab | (he is the one that usually handle wiki stuff - I can do it myself, although I never did) | [12:25] | |
| hverkuil | There is also a wiki page for the v4l-utils (although it is a bit out of date):
       https://linuxtv.org/wiki/index.php/V4l-utils | [12:25] | |
| mchehab | hverkuil, I wouldn't mention that it is a bit out of date
instead, I would update the wiki :-) | [12:26] | |
| hverkuil | Ah! You're volunteering! | [12:26] | |
| mchehab | hehehe
well, I can do that, if you point me what's outdated there on your view libdvbv5 stuff is pretty updated | [12:27] | |
| hverkuil | cec info is missing, but I'll add that. | [12:28] | |
| mchehab | I did a somewhat recent change related to it | [12:28] | |
| hverkuil | no ir-ctl (something for you)
media-control should be media-ctl, I'll fix that. v4l-keytable should be ir-keytable, right? | [12:28] | |
| sailus | On topics to the Media summit --- I'll try to reply tomorrow on that. | [12:33] | |
| hverkuil | OK, added cec info, fixed media-control -> media-ctl and push the README patch.
that's it from me. | [12:33] | |
| sailus | On the information in wiki on who's mainly responsible for what --- would it be a best option to create a new page?
Or use some existing page? (I have to admit I have read the wiki very seldom.) | [12:36] | |
| mchehab | bbiab | [12:39] | |
| hverkuil | I thought there was a page somewhere with some info about developers. I just can't remember where that was. It would be the logical place for something like that.
https://www.linuxtv.org/wiki/index.php/People_behind_V4L-DVB last touched 2012. | [12:40] | |
| mchehab | back
yeah, that's old hverkuil, I'll document the IR stuff sailus: wiki is a "collaboration" tool... | [12:42] | |
| hverkuil | I think it is better to add a page to the "Developer section" in the wiki with the core maintainers and areas of responsibility. | [12:43] | |
| mchehab | most of the pages there were added by some random user | [12:43] | |
| hverkuil | I'm inclined to remove the "People behind" page. | [12:43] | |
| mchehab | there's no problem on having it linked on multiple parts of the Wiki page
yeah, that's too outdated mchehab didn't look on it for years I suspect that Antti is keeping his entry updated there | [12:44] | |
| sailus | hverkuil: Both suggestions sounds good to me.
Antti is probably the only one. :-) | [12:46] | |
| mchehab | :-) | [12:46] | |
| sailus | No, one of the few.
I'll let you know if I end up adding something. | [12:46] | |
| mchehab | This page has been accessed 55,966 times.
it seems that people look on it I have a different suggestion: | [12:47] | |
| sailus | Does that include bots? | [12:47] | |
| mchehab | let's add a "maintainers" page
and place a link there to it | [12:47] | |
| sailus | What we could do as well is to add a link forwarding to the new page here. | [12:47] | |
| mchehab | at the beginning | [12:47] | |
| sailus | Indeed. | [12:48] | |
| mchehab | I don't feel comfortable removing people from there that used to contribute to the subsystem in the past
sailus: I don't think that the counts include robots... just clicked at "random page" and got another page with 12K access
(with is older, I guess) | [12:48] | |
| https://www.linuxtv.org/wiki/index.php/People_behind_V4L-DVB
what about that? (it is just a boilerplate right now) | [12:55] | ||
| sailus | Works for me. | [12:56] | |
| mchehab | the same page should also be pointed at Developers section | [12:56] | |
| hverkuil | Sounds good. | [12:56] | |
| sailus | I need to leave the office.
Thanks, and have a nice evening! | [13:02] | |
| hverkuil | same to you! | [13:02] | |
| mchehab | https://www.linuxtv.org/wiki/index.php/Media_Maintainers
would that work? I already added a link to the Developer's section to it too (just updated, adding a Scope) | [13:04] | |
| hverkuil | I maintain libv4l2rds as well. Looks good otherwise, although the second columns is not nicely aligned (some names appear to have an extra tab in front of them)
utils/rds should really be moved to contrib or something and renamed to rds-saa6588. I'm not even sure if it still works. does decode_tm6000 belong in utils? | [13:13] | |
| mchehab | https://www.linuxtv.org/wiki/index.php/Media_Maintainers#Linux_Kernel
feel free to fix rds references there decode_tm6000 should likely be moved to contrib it is just an ancillary tool the names there at v4l utils are the ones that currently have access to it I kept hdegoede for the libv4l stuff for now | [13:23] | |
| hverkuil | xc3028-firmware is still needed, right? I.e. it is not an ancillary tool? | [13:28] | |
| ............... (idle for 1h12mn) | |||
| mchehab | it is an ancillary tool
btw, the V4L-Utils page is in sync with README so, if something changes on one place, it should likely be reflected at the other one I don't use xc3028-firmware for years, but debugging firmware issues for xc3028 is not easy again, that's something that can go to contrib I'm moving them right now | [14:40] | |
| ...... (idle for 29mn) | |||
| hverkuil: I moved utils/rds ->contrib/rds-6588 as per your request and did the same changes for decode_tm6000 and xc3028_firmware. I also updated them at the Media Maintainers wiki page | [15:13] | ||
| ...... (idle for 29mn) | |||
| hverkuil | Nice! Thanks. | [15:42] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |