<div dir="auto"><div>Hi Christian,<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 26 Jun 2020, 18:10 Daniel Vetter, <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jun 26, 2020 at 9:03 AM Christian König<br>
<<a href="mailto:christian.koenig@amd.com" target="_blank" rel="noreferrer">christian.koenig@amd.com</a>> wrote:<br>
><br>
> Am 26.06.20 um 06:43 schrieb Sumit Semwal:<br>
> > On Fri, 26 Jun 2020 at 01:24, Daniel Vetter <<a href="mailto:daniel@ffwll.ch" target="_blank" rel="noreferrer">daniel@ffwll.ch</a>> wrote:<br>
> >> Ignoring everything else ...<br>
> >><br>
> >> On Thu, Jun 25, 2020 at 9:28 PM Jani Nikula <<a href="mailto:jani.nikula@linux.intel.com" target="_blank" rel="noreferrer">jani.nikula@linux.intel.com</a>> wrote:<br>
> >>> As a side note, there seem to be extra checks in place for acks when<br>
> >>> applying non-i915 patches to drm-intel; there are no such checks for<br>
> >>> drm-misc.<br>
> >> One option to generalize that that I pondered is to consult<br>
> >> <a href="http://get_maintainers.pl" rel="noreferrer noreferrer" target="_blank">get_maintainers.pl</a> asking for git repo link, and if that returns<br>
> >> something else, then insist that there's an ack from a relevant<br>
> >> maintainer. It's a bit of typing, but I think the bigger problem is<br>
> >> that there's a ton of false positives.<br>
> > Right; for the particular patch, I wasn't even in the to: or cc: field<br>
> > and that made it slip from my radar. I would definitely ask any one<br>
> > sending patches for dma-buf directory to follow the <a href="http://get_maintainers.pl" rel="noreferrer noreferrer" target="_blank">get_maintainers.pl</a><br>
> > religiously.<br>
> >> But maybe that's a good thing, would give some motivation to keep<br>
> >> MAINTAINERS updated.<br>
><br>
> Should I maybe add myself as maintainer as well? I've written enough<br>
> stuff in there to know the code quite a bit.<br>
<br>
I think that makes lots of sense, since defacto you already are :-)<br>
<br>
If you feel like bikeshed, <a href="http://get_maintainers.pl" rel="noreferrer noreferrer" target="_blank">get_maintainers.pl</a> also supports R: for<br>
reviewer, but given that you also push patches to drm-misc M: for<br>
maintainer feels more accurate.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think given you've been reviewing and changing most of the code around dma-fences, it should be ok to add you as the maintainer for those bits?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-Daniel<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto">Sumit.</div></div>