[PATCH 00/11] Enable Atomic Mode Set on vmwgfx

Daniel Vetter daniel at ffwll.ch
Tue Mar 28 07:54:30 UTC 2017


On Mon, Mar 27, 2017 at 03:00:53PM -0700, Sinclair Yeh wrote:
> This series enables atomic mode set on vmwgfx.  Developed in
> collaboration with Thomas Hellstrom and the VMWare Graphics
> Team.

Yay!

I've given it a quick read-thru and sprinkled some comments over a bunch
of places. One top-level suggestion would be to review all the hooks you
have and vmw-specific callbacks and make sure you really need them all. A
bunch can definitely be removed after the transition, and in general we've
tried really hard to make the atomic helpers not get in the way so much
and require pointless boilerplate.

I haven't seen them, but I assume you have suspend/resume and stuff like
that too. Might be good to look into the suspend/resume/shutdown helpers
we now have in drm-misc.

On the series, with my comments addressed or ignored to your judgment:

Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list