hverkuil: Hello Hans, did you have a look to pending dcmi commits (lock scheme and jpeg are pending)
hfr: I plan to look at those tomorrow.
hverkuil: ok thks Hans, feel free to ping me tomorrow if needed
hfr: can you check that your patches still apply? A lot of commits were merged recently. I don't think they conflict, but it is easy enough to check.
hverkuil: of course, I will prepare commits based on latest master in case of
hverkuil: patches applies on master silently with git am, here is the order I used to apply patches:
$ git format-patch -4
0001-media-stm32-dcmi-fix-lock-scheme.patch
0002-media-stm32-dcmi-rework-overrun-error-case.patch
0003-media-stm32-dcmi-fix-unnecessary-parentheses.patch
0004-media-stm32-dcmi-add-JPEG-support.patch
hfr: thanks!
hverkuil: Please note that I've just sent a v2 for "rework-overrun-error-case": removed a blank line
I noticed.
gnurou, thanks for the replies!
gnurou, just a quick word: I already have a patch for the planes copy thing
paulk-gagarine: thanks for the feedback! :)
I'll send this later today
oh, perfect then
if you don't mind I will squash it into the proper patch of the series
sure thing
go for it
sailus: how is the progress with what you wanted to do btw? If you are too busy, I think we can still move forward with codecs based on what we have now, and rework the internals later
the UABI would not change, and that's what is important
hverkuil: same thing about the control work - with a bit of guidance from you I can probably take this over too
mchehab: hverkuil: hi, following an email that I've sent to the list a few days ago, what's the recommended process when patches don't get reviewed for many months by the respective maintainer?
pinchartl: what's up with this uvc patch series?
lyakh: ping the maintainer
mchehab: sure, and if that doesn't help over > 1.5 years?
what's the patch in question
mchehab: my 2 UVC patches, let me fetch a link...
https://patchwork.linuxtv.org/patch/45931/ https://patchwork.linuxtv.org/patch/45932/
gnurou: I'll push the patches to my linuxtv.org tree today / tomorrow.
They're still untested though.
I can't see anything really odd there, except for the names of the vars: "error, tmp"
but I'm not an expert on uvc protocol
pinchartl: what's the problem with the above patch, if any?
mchehab: hverkuil: could you maybe reply to https://www.mail-archive.com/linux-media@vger.kernel.org/msg126922.html to not depend on whether pinchartl notices these messages here?
lyakh: maybe the best would be if you resend a [PATCH RESEND] with the contents of the proposed patch
that makes easier to reply on it
at least here, I do annual archives
I probably don't have your original patch email anymore (except on my backups)
maybe the others have the same issue
mchehab: those are 2 patches, ok, can resend, sure, thanks
thanks!
mchehab: I do the same :-)
resent