[04:13] *** RedMarsBlueMoon has left [08:49] <hverkuil> arnd: ping [08:49] <arnd> hi hverkuil [08:49] <arnd> hverkuil: I'm sure there was a patch I was meant to follow up on, let me check [08:49] <hverkuil> arnd: good morning! [08:49] <hverkuil> https://patchwork.linuxtv.org/project/linux-media/patch/20210318134334.2933141-1-arnd@kernel.org/ [08:50] <hverkuil> that's the one :-) [08:50] <arnd> right. I had applied it on my randconfig series, which has a couple of other stuff applied, including an older global cleanup for time32 that I never sent, I'll rebase and resend the fix [08:51] <hverkuil> Thanks! [09:42] <hverkuil> ezequielg: ping [10:03] <sinajit> Is it possible for a capability retrieved by VIDIOC_QUERYCAP to be supported only for some image formats? [10:06] <sinajit> For some reason VIDIOC_STREAMON fails on my device for RX24 but it works for RGB3/BGR3 [10:13] <hverkuil> sinajit: what driver are you using? [10:13] <sinajit> Don't know, how would I check that? [10:14] <hverkuil> v4l2-ctl -D [10:14] <hverkuil> (QUERYCAP) [10:14] <sinajit> bm2835 mmal [10:15] <sinajit> Ah nevermind, it appears it was a memory issue. [10:15] <hverkuil> That's one reason why it can fail :-) [12:47] <ezequielg> hverkuil: hey Hans. [12:58] <ezequielg> hverkuil: thanks for reviewing the mpeg-2 series. [13:08] <hverkuil> ezequielg: no problem. [13:09] <hverkuil> ezequielg: I replied to "NULL pointer dereference at v4l2_ctrl_request_complete" a few days ago. Were you able to test with the provided patch so I can get a bit more information about what is going on? [14:15] <ezequielg> hverkuil: Not yet. I wasn't at the office these day. It's a high priority issue for me, so I'll run tests as soon as I get back to work. [14:15] <hverkuil> ezequielg: ah, good. I'll wait for feedback then. [14:15] <ezequielg> (BTW, I had the same printks, keep in mind it gets super spammy, because it takes time to reproduce). [14:16] <ezequielg> I have the feeling it happens with those controls that aren't explicitly set with S_EXT_CTRL and a request submit. For instance, MPEG2_SEQUENCE, or H264 decoding mode control. [14:18] <hverkuil> how long does it typically take to reproduce? [14:22] <ezequielg> a few seconds, with 3 pipelines. [14:22] <ezequielg> but enough to spam the logs. [14:25] <hverkuil> ezequielg: when will you resume work on this? I'll make sure I'm available at the time so we can dig into this together. [14:25] * ezequielg how about tomorrow 8am argentina? [14:26] <ezequielg> hverkuil: ^ [14:27] <hverkuil> sounds good. [15:15] <hverkuil> mchehab: this PR https://patchwork.linuxtv.org/project/linux-media/patch/b2a45eb6-df13-05e3-c24b-6025bb41fd04@xs4all.nl/ is marked as Accepted today, but I don't see the patches in our master branch. Did something go wrong? [15:26] <Phill93> Hello, I have a Startech Framegrabber (SVID2USB232) this has the chip EM28281 installed. The em28xx driver does not recognize this. Can someone help me to get it running? [17:24] <mchehab> hverkuil: I remember merging sometthing like this today: vivid: update EDID [17:25] <hverkuil> But none of the others arrived. [17:25] <mchehab> commit 443ec4bbc6116f6f492a7a1282bfd8422c862158 [17:25] <mchehab> Author: Hans Verkuil <hverkuil-cisco@xs4all.nl> [17:25] <mchehab> AuthorDate: Thu Mar 25 08:48:21 2021 +0100 [17:25] <mchehab> Commit: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> [17:25] <mchehab> CommitDate: Tue Apr 6 10:37:18 2021 +0200 [17:25] <mchehab> media: vivid: update EDID [17:26] <mchehab> funny [17:31] <mchehab> hverkuil: yeah, I suspect it was marked as accepted by mistake - not sure if it was due to script or not... [17:31] <mchehab> Tue Apr 6 10:38:20 2021: pwclient update -s 'Accepted' 72624 # patches/lmml_72624_git_pull_for_v5_13_various_fixes.patch [17:31] <mchehab> Tue Apr 6 10:38:20 2021: pwclient update -s 'Accepted' 72675 # patches/lmml_72675_git_pull_for_v5_13_various_fixes.patch [17:31] <mchehab> there were two PRs with the same name, one after the other [17:33] <mchehab> I'm applying right now the second PR [17:35] <mchehab> next time, it could be worth numerating e-mails with the same subject - or use a different name for each [17:37] <mchehab> anyway, problem solved. patches applied [17:38] <hverkuil> much appreciated! [17:38] <mchehab> you're welcome [17:38] <mchehab> I've been more aggressive those days with a new cleanup script set [17:39] <mchehab> that tries harder to identify duplicated patches [17:40] <mchehab> maybe the script is the one to blame, as both PRs were "accepted" at the same second [17:41] <hverkuil> Ah, that same patch was actually in two PRs somehow. [17:42] <hverkuil> I'm not sure how that happened, but that clearly was my mistake.