| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | mchehab has quit IRC (Read error: Connection reset by peer)
ChanServ sets mode: +v mchehab | [00:05] |
| ................................................................................ (idle for 6h35mn) | ||
| ChanServ sets mode: +v mchehab | [06:43] | |
| ............................................................... (idle for 5h13mn) | ||
| mchehab | hi all | [11:56] |
| syoung | Hello | [11:57] |
| hverkuil | hi | [11:58] |
| sailus | EHLO | [11:59] |
| mchehab | didn't send pull request yet, because we had a merge from docs tree
As Linus pulled from it yesterday: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=255442c93843f52b6891b21d0b485bf2c97f93c3 my plan is to send the pull requests today - or on Monday | [12:00] |
| syoung | great, I've been looking forward to my rc-core changes making it upstream :) | [12:02] |
| mchehab | :-) | [12:03] |
| hverkuil | I'll wait until it has landed in mainline, then I'll start making backports for the compat patch series to the LTS kernels. | [12:05] |
| mchehab | ok | [12:07] |
| hverkuil | Are there any patch series that I should review? I haven't had much time for that in the past week. | [12:09] |
| mchehab | I didn't have time to review things either\
being busy with some bugs with ISOC on RPi3 | [12:10] |
| sailus | I've been mostly reviewing and collecting patches recently. Not much to report really.
Most importantly the request API now I guess. | [12:12] |
| hverkuil | I plan on trying his RFCv2 series and get a bit of a feeling for it. Also useful to improve the control framework. | [12:14] |
| mchehab | what's the status of the explicit fences patchset?
didn't review yet its last version | [12:15] |
| hverkuil | waiting for a v8. | [12:17] |
| mchehab | just text fixes at docs or anything relevant?
on a quick look on the comments people posted, it seems mostly just editoral revision at the texts | [12:17] |
| hverkuil | It needs better testing, also the code is still confusing in several places.
It definitely not ready to merge yet, although it is getting close. | [12:18] |
| mchehab | I'm not seeing any comments at v7 about "the code is still confusing in several places"... | [12:20] |
| hverkuil | sailus: can you look at this? https://www.spinics.net/lists/linux-media/msg128061.html | [12:20] |
| mchehab | if you're seeing problems at the code, better to comment it :-) | [12:20] |
| sailus | hverkuil: Yes. | [12:21] |
| hverkuil | From my review of patch 5/6 of the fence patch series: "As mentioned above, it is very confusing that the same vb2_core_qbuf 'fence' argument is used for both in and out fences."
Mixing in and out fences is weird and I couldn't be certain it was correct. | [12:23] |
| mchehab | OK
somehow, I missed it | [12:24] |
| hverkuil | And definitely more testing with ADV_DEBUG on and making sure all the ops are balanced.
Also some corner case in an error path that needed testing since I suspected that code was wrong. Anything else? | [12:24] |
| mchehab | not from my side | [12:28] |
| ........................................................................................................................ (idle for 9h57mn) | ||
| *** | ChanServ sets mode: +v mchehab | [22:25] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |