| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| hverkuil | pH5: regarding https://patchwork.linuxtv.org/patch/53185/ and https://patchwork.linuxtv.org/patch/53186/ :
are you waiting for someone to confirm your assumptions? | [09:10] |
| sailus: ping
svarbanov: ping | [09:17] | |
| svarbanov | hverkuil: pong | [09:21] |
| hverkuil | Can you look at https://patchwork.linuxtv.org/patch/52718/ ?
If OK, then I'll pick it up. (I want to process all venus patches delegated to me today) Also, a question that came up yesterday: drivers/media/platform/qcom/venus/venc.c, venc_s_selection(): | [09:22] |
| svarbanov | hverkuil: you can pick up https://patchwork.linuxtv.org/patch/52718/ with Tomasz's ack | [09:23] |
| hverkuil | it doesn't allow you to change the crop rectangle. Is that because cropping is not possible in the HW, or because it has not been implemented yet?
I have 52718 queued up already. | [09:23] |
| svarbanov | hverkuil: the cropping is not implemented yet (will have to investigate how should be implemented) | [09:24] |
| hverkuil | Ah, OK. I was wondering about that.
Perhaps a small patch that adds a TODO comment would be useful. | [09:25] |
| svarbanov | where I can check what you taken for venus? | [09:25] |
| hverkuil | https://patchwork.linuxtv.org/project/linux-media/list/?q=venus&delegate=2704
Two patches from Kelvin, the key frame patch (and can you Ack that as well? Sorry, forgot to ask that) and an old one from Ezequiel. Not sure if that one is still valid, I'll have a look. | [09:26] |
| svarbanov | OK, I'll ack the patches from Kelvin and key frame patches | [09:29] |
| hverkuil | Great, I'll pick those up today for 4.21. After that I won't touch venus patches anymore since you can do the review and make the pull requests. | [09:32] |
| pH5 | hverkuil: svarbanov confirmed for venus
https://lore.kernel.org/linux-media/d6e60d32-d4eb-1d51-1f08-2f3547bb9524@linaro.org/ | [09:36] |
| hverkuil | Ah, good. I'll pick up these two patches. | [09:37] |
| svarbanov | hverkuil: why this https://patchwork.linuxtv.org/patch/53288/ is not delegated to you? | [09:49] |
| hverkuil | Dec 5, I haven't gotten around to that one yet. If you ack it, I can take it now. | [09:50] |
| svarbanov | hverkuil: I'll ack it, but want to make a quick test | [09:59] |
| hverkuil | snawrocki: shall I pick up https://patchwork.linuxtv.org/patch/53352/ ?
sailus: ping | [10:05] |
| snawrocki | hverkuil: sure, go ahead | [10:05] |
| svarbanov | hverkuil: I acked https://patchwork.linuxtv.org/patch/53288/ | [10:09] |
| ......... (idle for 40mn) | ||
| *** | benjiG has left | [10:49] |
| hverkuil | neg: re patch https://patchwork.linuxtv.org/patch/51432/
Should this just be accepted? mchehab: ping This patch series https://patchwork.linuxtv.org/patch/50755/ removing soc_camera support from defconfigs: AFAIK there are effectively no mobile sh maintainers, so if we want this done, we will have to merge this. jmondi: please confirm ^^^ | [10:50] |
| mchehab | hverkuil: pong
maybe arnd could either pick it or ack | [10:57] |
| jmondi | hverkuil: Hi Hans, yes, 50755 has been rotting in patchwork for a while, and the SH guys are mostly silent
hverkuil: should I re-send and update it to v4.20 maybe? hverkuil: (thanks for the SUBDEV_API answer btw) | [11:01] |
| arnd | jmondi: I suppose we should be able to pick this up as part of the new soc group, so please send to:soc@kernel.org cc:linux-sh and maintainers | [11:03] |
| hverkuil | jmondi: there you go! | [11:04] |
| arnd | we're still testing out the exact process for this | [11:04] |
| hverkuil | problem solved :-)
arnd: thanks! | [11:04] |
| arnd | no worries | [11:05] |
| mchehab | arnd: thanks! | [11:06] |
| jmondi | arnd: thank you :)
I'll re-do that for v4.20, and resend including the soc group | [11:10] |
| ......... (idle for 40mn) | ||
| hverkuil | paulk-leonov: ping
Can I merge patches 11-13 of your v2 Cedrus H5 and A64 support with A33 and H3 updates series? Those appear to be the only media related patches in that series | [11:50] |
| .... (idle for 19mn) | ||
| paulk-leonov | hverkuil, yes please!
the other ones have been picked up already | [12:10] |
| hverkuil | paulk-leonov: will pick those three up. | [12:10] |
| paulk-leonov | thanks! | [12:10] |
| ............... (idle for 1h10mn) | ||
| hverkuil | I asked a colleague about = {}. He says it is absolutely part of the standard and he'll dig up the reference. He's heavily involved in C/C++ standardization, so if anyone knows it's him. | [13:20] |
| neg | hverkuil: regarding patch https://patchwork.linuxtv.org/patch/51432/, I'm tempted as I really want to get them in. But not un-tested, only one I know who have access to a V3H baord is Sergei. So he needs to test his patches or I need to find someone else who can test them | [13:31] |
| hverkuil | neg: can it do harm? I accidentally already made this part of my pull request :-)
mchehab: ping | [13:32] |
| neg | hverkuil: it's probobly ok :-) If you already sent the pull request keep it in | [13:37] |
| hverkuil | It's already merged :-)
mchehab: unping neg: it didn't look like it was very likely that someone would actually test it, and the patch looks reasonable. | [13:37] |
| neg | I agree the patch looks good, and it can't create a regression as V3H is not supported upstream today | [13:41] |
| hverkuil | mchehab: OK, I now have 98 patches delegated to me of which 15 are in the TODO state. It's the best I can do today :-)
I might be able to clean up a few more next week. 95 patches (just found 3 more superseded patches!) | [13:46] |
| .... (idle for 16mn) | ||
| mchehab | hverkuil: thanks a lot!
I'll probably delegate a few more to you :-p | [14:04] |
| hverkuil | mchehab: regarding the 'aligned with open parenthesis' vs 'line length too long' checkpatch warning: can we just accepts whatever the author does?
mchehab: you are right regarding '= {}': it's not standard C :-) But apparently such a common extension that it even ended up in books on the language. | [14:14] |
| mchehab | yeah, I wish I was not right about that
it would be a way better if this becomes part of the standard | [14:19] |
| *** | hverkuil has left | [14:20] |
| mchehab | ezequielg: with regards to:
[RFC,2/3] USB: core: Add non-coherent buffer allocation helpers I'll mark it as RFC I guess you're working (or submitted) a new version of it. If not, feel free to do so hmm... this is for USB core so doesn't apply to us anyway (as patch handlers) only patch 3/3 is specific for media. that's the one I'll tag as RFC at patchwork | [14:25] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |