On Fri, Oct 05, 2018 at 04:21:43PM +0000, Thomas Hellstrom wrote: > On 10/05/2018 06:06 PM, Daniel Vetter wrote: > > On Thu, Oct 04, 2018 at 10:24:41PM +0200, Daniel Vetter wrote: > >> From: Thomas Hellstrom <thellstrom@xxxxxxxxxx> > >> > >> Use the correct helper and also return early on helper > >> success rather than on helper failure. > >> > >> Also explicitly return 0 in the case of no fb. > >> > >> v2: Check for !fb after updating state->visible (Ville). > >> > >> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > >> Signed-off-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx> (v1) > >> Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > >> Reported-by: Daniel Vetter <daniel@xxxxxxxx> > >> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > >> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > >> Cc: VMware Graphics <linux-graphics-maintainer@xxxxxxxxxx> > >> Cc: Sinclair Yeh <syeh@xxxxxxxxxx> > >> Cc: Thomas Hellstrom <thellstrom@xxxxxxxxxx> > > Ack for pulling this one in through drm-misc? It blocks the next patch. > > All the other vmwgfx patches can be merged when/however, except this one > > here. > > > > Thanks, Daniel > > Sure, Thanks, applied to drm-misc-next, toghether with the remaining patches. I'll leave all the other vmwgfx patches up to you. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
