mbox series

[v2,0/3] mgag200: Improve damage handling

Message ID 20220504134026.283417-1-jfalempe@redhat.com (mailing list archive)
Headers show
Series mgag200: Improve damage handling | expand

Message

Jocelyn Falempe May 4, 2022, 1:40 p.m. UTC 
mgag200: Improve damage handling

This series improves the damage handling on Matrox gpu, and allows
Gnome/Wayland to run much better.
Also include some driver cleanup.

Tested on a Dell T310 with Matrox MGA G200eW WPCM450 (rev 0a)

v2:
 Drop remove mgag200_probe_vram()
 Rewrote the patch to warn if startaddr is not 0, and hw doesn't
  support it. (instead of removing the unused flag).

Thanks,

Comments

Thomas Zimmermann May 4, 2022, 2:59 p.m. UTC | #1 
Hi Jocelyn,

with my comment on 3/3 considered, you can add

Reviewed-by: Thomas Zimemrmann <tzimemrmann@suse.de>

to this patchset.

You should get commit access for drm-misc, so that you can land the 
patchset on your own. Please see

   https://drm.pages.freedesktop.org/maintainer-tools/commit-access.html

for how to do that.

Best regards
Thomas

Am 04.05.22 um 15:40 schrieb Jocelyn Falempe:
> mgag200: Improve damage handling
> 
> This series improves the damage handling on Matrox gpu, and allows
> Gnome/Wayland to run much better.
> Also include some driver cleanup.
> 
> Tested on a Dell T310 with Matrox MGA G200eW WPCM450 (rev 0a)
> 
> v2:
>   Drop remove mgag200_probe_vram()
>   Rewrote the patch to warn if startaddr is not 0, and hw doesn't
>    support it. (instead of removing the unused flag).
> 
> Thanks,
>
Jocelyn Falempe May 4, 2022, 4:09 p.m. UTC | #2 
On 04/05/2022 16:59, Thomas Zimmermann wrote:
> Hi Jocelyn,
> 
> with my comment on 3/3 considered, you can add
> 
> Reviewed-by: Thomas Zimemrmann <tzimemrmann@suse.de>
> 
> to this patchset.
> 
> You should get commit access for drm-misc, so that you can land the 
> patchset on your own. Please see
> 
>    https://drm.pages.freedesktop.org/maintainer-tools/commit-access.html
> 
> for how to do that.

Thanks for your reviews.
I created the issue to get commit access :
https://gitlab.freedesktop.org/freedesktop/freedesktop/-/issues/433

Best regards,
Jocelyn Falempe May 16, 2022, 1:12 p.m. UTC | #3 
On 04/05/2022 16:59, Thomas Zimmermann wrote:
> Hi Jocelyn,
> 
> with my comment on 3/3 considered, you can add
> 
> Reviewed-by: Thomas Zimemrmann <tzimemrmann@suse.de>
> 
> to this patchset.
> 
> You should get commit access for drm-misc, so that you can land the 
> patchset on your own. Please see
> 
>    https://drm.pages.freedesktop.org/maintainer-tools/commit-access.html
> 
> for how to do that.

I pushed the patches with the requested changes to drm-misc-next last 
Thursday.

Thanks a lot for your help.