[03:47] *** ChanServ sets mode: +v mchehab`
[05:38] <syoung> mchehab: ok, I'll change that in future
[07:47] <hverkuil> pinchartl: can you review https://patchwork.linuxtv.org/patch/57336/ ? Or delegate it to yourself if you prefer. The patch looks good to me, but I prefer to have your Acked/Reviewed-by as well.
[08:24] <hverkuil> mchehab: can we add a Duplicate state to patchwork to mark duplicate patches?
[10:10] <mchehab> hverkuil: we can, but IMO, it is overkill... just mark as superseded
[10:15] <hverkuil> But superseded suggests that there is a newer version of the patch. In this case it is a duplicate patch. It's useful feedback to the submitter IMHO.
[12:08] <mchehab> hverkuil: if the submitter sent the same patch two times, he'll know
[12:09] <mchehab> however, it takes us time to identify if the two submitted patches are identical or not
[12:09] <mchehab> with would be better spent reviewing another patch :-)
[12:10] <mchehab> I mean: two patches with the same diff may have different comments - you're proposing that, if both diffstat and comments are equal, we mark as "duplicated", if not, we mark as "superseed"
[12:11] <mchehab> that sounds overkill and uneeded extra work :-)
[12:11] <mchehab> s/diffstat/diff
[16:27] <pinchartl> hverkuil: I'll try to review it ASAP. I'm on vacation in Japan this week but I'll try to address it before flying back