syoung: will you apply https://patchwork.linuxtv.org/patch/43010/ or shall I?
hverkuil: I don't have commit rights on v4l-utils
I'm happy to for you apply it, although I was going to create a request pull for v4l-utils in next few days, and I could include it in that
I'll commit it.
done
sailus: Hi
I'm preparing the fourth version of the QC CAMSS driver. Would you like to review again the s_fmt handling before I add your Acked-by.
mchehab: posted v2 for the smpte-2084 transfer function patch.
Also apologies for missing that there was no MAINTAINERS patch for the meson cec driver. I should have seen that.
np
should I make a pull request for the smpte v2 patch, or will you just take the patch?
no need
I'm about to pick it
OK.
ah, btw, I'm seeing a lot of these on your patches:
WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
I've started paying more attention to this. Hopefully you shouldn't see this with newer patches from me.
ok, thanks
Personally I think these column limitations are nuts, but so be it.
well, at least for me, it is needed...
why? Curious to know.
I have a script that opens a window with 2 columnns...
on the first column, the result of checkpatch
on the second one, the patch itself
both have 80 columns
in order to fit on my screen
something bigger than 80 cols cause line wraps
with takes  a few additional milisseconds for my brain to parse :-)
bbiab
back
I probably could increase it to ~100cols on my desktop, but, on my 13' notebook, more than 80 columns would make it really hard to read
on a somewhat related note, Gerrit restricts the maximum size of patch subject
I had some issues when I backported media patches to Tizen a few years ago due to that...
e-mail sujects needed to be truncated to 72 columns
with is the maximum allowed on Tizen's gerrit instance
(not sure if other instances of Gerrit have the same issue or not)
so, in general, I like the idea of having a maximum columns limit - yet, if it is 80 columns, 132 columns or whatever is a matter of just defining what works best for everyone
pinchartl: thanks, I'll handle those uvc patches on Friday.
hverkuil: thank you
hverkuil: All my virtual terminal windows have 80 columns.
It's now they're meant to be. :-)
s/now/how/
ttomov: Sure.
mchehab: Gerrit is evil.
sailus: gerrit is useful. every time someone tells me about one of gerrit's features, I make sure to do the exact opposite
it's a marvelous counter-example
pinchartl: Still using it can easily cause depression.
That's not good.
mchehab: Any new info on the Media summit schedule in Prague?
I've been offline for some time as my ISP broke my IPv6 connectivity. :-I
hverkuil: mchehab: for ao-cec PR I'm on vacation, can I send a fix in 2 weeks ?
narmstrong: if I make a patch, are you able to sign off on it?
hverkuil: yes
OK, I'll make a patch. No need to wait 2 weeks just for this.
posted, cc-ed to you.
hverkuil: thx