On 10/05/2018 10:46 PM, Daniel Vetter wrote: > 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 Thanks, Thomas _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel