<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 13 Nov 2020, 21:31 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, Nov 13, 2020 at 9:53 PM Lee Jones <<a href="mailto:lee.jones@linaro.org" target="_blank" rel="noreferrer">lee.jones@linaro.org</a>> wrote:<br>
><br>
><br>
><br>
> On Fri, 13 Nov 2020, 20:50 Daniel Vetter, <<a href="mailto:daniel@ffwll.ch" target="_blank" rel="noreferrer">daniel@ffwll.ch</a>> wrote:<br>
>><br>
>> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote:<br>
>> > Hi Lee,<br>
>> ><br>
>> > On Thu, Nov 12, 2020 at 07:00:11PM +0000, Lee Jones wrote:<br>
>> > > The precedent has already been set by other macros in the same file.<br>
>> > ><br>
>> > > Fixes the following W=1 kernel build warning(s):<br>
>> > ><br>
>> > >  drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but not used [-Wunused-but-set-variable]<br>
>> > >  55 | struct drm_crtc *crtc;<br>
>> > >  | ^~~~<br>
>> > ><br>
>> > > Cc: Maarten Lankhorst <<a href="mailto:maarten.lankhorst@linux.intel.com" target="_blank" rel="noreferrer">maarten.lankhorst@linux.intel.com</a>><br>
>> > > Cc: Maxime Ripard <<a href="mailto:mripard@kernel.org" target="_blank" rel="noreferrer">mripard@kernel.org</a>><br>
>> > > Cc: Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de" target="_blank" rel="noreferrer">tzimmermann@suse.de</a>><br>
>> > > Cc: David Airlie <<a href="mailto:airlied@linux.ie" target="_blank" rel="noreferrer">airlied@linux.ie</a>><br>
>> > > Cc: Daniel Vetter <<a href="mailto:daniel@ffwll.ch" target="_blank" rel="noreferrer">daniel@ffwll.ch</a>><br>
>> > > Cc: Rob Clark <<a href="mailto:robdclark@gmail.com" target="_blank" rel="noreferrer">robdclark@gmail.com</a>><br>
>> > > Cc: <a href="mailto:dri-devel@lists.freedesktop.org" target="_blank" rel="noreferrer">dri-devel@lists.freedesktop.org</a><br>
>> > > Signed-off-by: Lee Jones <<a href="mailto:lee.jones@linaro.org" target="_blank" rel="noreferrer">lee.jones@linaro.org</a>><br>
>> ><br>
>> > Also applied to drm-misc-next.<br>
>> > This was the last patch from this batch I will process.<br>
>> > The others are left for the maintainers to pick up.<br>
>><br>
>> btw for patches that maintainers don't pick up, the usual process is that<br>
>> we give them 2 weeks, then just mass apply. Now you're producing a lot of<br>
>> patches, too much for me to keep track, so please just ping me with a<br>
>> resend for those that expired and I'll go through and pick them all up.<br>
><br>
><br>
> That's great Daniel. Thanks for your support.<br>
><br>
> I can do one better than that.<br>
><br>
> Would a pull-request suit you?<br>
<br>
We have a few trees going on, and your patches are landing through all<br>
kinds of them. So this might be more coordination pain. If you can<br>
exclude patches for the separately and usually fairly well maintained<br>
drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915,<br>
drm/nouveau, drm/msm and drm/omapdrm probably the usual ones).<br>
<br>
Or you just send the next pile and we'll see.<br>
<br>
Also I guess I can't really interest you in commit rights so this<br>
patch bombs get off my back again? :-)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">What does that mean? Merge my own patches? </div><div dir="auto"><br></div><div dir="auto">Not sure how that works with your group maintenance setup. </div><div dir="auto"><br></div><div dir="auto">Is it just a `git push`? </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">
</blockquote></div></div></div>