| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | Anderlok has left | [00:19] |
| ................... (idle for 1h33mn) | ||
| Anderlok has left | [01:52] | |
| ................ (idle for 1h16mn) | ||
| drekary has left | [03:08] | |
| ....... (idle for 31mn) | ||
| Bildrel has left | [03:39] | |
| .................................................................................................... (idle for 8h18mn) | ||
| hverkuil | hi all
mchehab: sailus: pinchartl: syoung: ping | [11:57] |
| mchehab | hi all | [12:01] |
| hverkuil | ah, I'm no longer alone :-) | [12:02] |
| syoung | hi all | [12:03] |
| hverkuil | just us three? | [12:05] |
| mchehab | it sounds so
from my side, I didn't have time yet to handle patches this week will probably do patch handling today or tomorrow | [12:06] |
| hverkuil | OK, I'll try to make another pull request today with some smaller stuff that's pending.
I would appreciate it if I can get some feedback for this RFC patch: https://patchwork.linuxtv.org/patch/54304/ And also for the epoll() RFC patch series I just posted. Esp. the dvb-core patch. | [12:07] |
| mchehab | will try to do it this week | [12:10] |
| hverkuil | syoung: I checked the poll_wait handling in the rc subsystem and that looks good w.r.t. epoll()
Nothing else from my side. | [12:11] |
| syoung | hverkuil: I've looked at epoll before (exactly for rc) so I'll have a look at the dvb patches when I get some time | [12:14] |
| hverkuil | thanks!
v4l2-ctl now has a --epoll-for-event option and v4l2-compliance has epoll tests as well, although not yet for control events. It is also disabled at the moment since it fails without these patches. | [12:14] |
| syoung | hverkuil: how was fosdem? | [12:16] |
| hverkuil | Very good. My CEC and V4L2 Status Update videos are already uploaded.
Had a nice talk with Randy Li about rockchip stateless codecs and with Guillaume Tucker about integrating the new regression test scripts into kernelci. | [12:16] |
| sailus | Hello! | [12:18] |
| syoung | Great, I'll look at the cec & v4l2 status updates videos
I still need to find some time to add rc to vivid. Not enough time, so annoying | [12:19] |
| sailus | mchehab, hverkuil: Could you check these two:
https://patchwork.linuxtv.org/patch/54292/ https://patchwork.linuxtv.org/patch/54293/ If they're fine, I'll submit some more to move the SoC camera framework to staging. | [12:28] |
| hverkuil | done. I like removing drivers :-) | [12:29] |
| mchehab | me too
provided that it won't remove the functionality from the Kernel both looks ok on my eyes s/looks/look/ mchehab needs coffeee | [12:30] |
| sailus | mchehab, hverkuil: Thanks! | [12:39] |
| mchehab | anything else? | [12:42] |
| hverkuil | not from me. | [12:47] |
| .... (idle for 16mn) | ||
| sailus | Nor me. | [13:03] |
| ..... (idle for 23mn) | ||
| hverkuil | sailus: can you review https://patchwork.linuxtv.org/patch/54357/ and the epoll RFC series I posted earlier this afternoon?
The pxa_camera patch is something I'd like to get an ack for asap so I can add it to my pull request. | [13:26] |
| ......... (idle for 42mn) | ||
| sailus | hverkuil: On the 5th patch --- there are a few functions alike that which aren't made easier to read by such changes.
It'd be nice to fix sparse instead but I presume that'd be quite a bit more work. I expect similar false positive warnings elsewhere, too. | [14:09] |
| hverkuil | sailus: actually, almost everywhere else it is first assigned to a v4l2_subdev pointer variable.
drivers/staging/media/omap4iss/iss.c is the only one that doesn't do that. Ah, I'm not building smatch for staging, that's why I never saw warnings about omap4. | [14:13] |
| sailus | Mostly, I agree.
But the question is: how much we change the code to make slightly dumb static analysers happy? It'd be nice to be able to mark things as false positives and be done with it btw. | [14:17] |
| hverkuil | To be honest, I find the code a bit more readable with this patch :-)
Hmm, I get a bunch of ipu3 warnings now. | [14:20] |
| sailus | Yeah, there are bound to be some in the imgu driver.
hverkuil: There will be more once the vc branch is merged. There are functions dealing with a number of entities (and sub-devices). The pattern suggested by the 5th patch increases the number of local variables. Anyway, I'm fine with the 5th patch, too. | [14:21] |
| hverkuil | Yeah, I now get the same sparse warning for omap4. Next daily build will include staging/media results.
sailus: mailed you the list of sparse/smatch warnings I get for ipu3. | [14:24] |
| sailus | hverkuil: Thanks! | [14:28] |
| ....................................... (idle for 3h11mn) | ||
| mchehab | pull requests handled except for i.MX7
will likely look on it tomorrow | [17:39] |
| .......................................................... (idle for 4h45mn) | ||
| sailus | \o/ | [22:25] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |