On Fri, 11 Nov 2011 18:04:03 +0200 ville.syrjala at linux.intel.com wrote: > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > These are the only indication to user space that the plane was disabled. > > Signed-off-by: Ville Syrj?l? <ville.syrjala at linux.intel.com> > --- > drivers/gpu/drm/drm_crtc.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c > index d3b884e..70f5747 100644 > --- a/drivers/gpu/drm/drm_crtc.c > +++ b/drivers/gpu/drm/drm_crtc.c > @@ -1677,6 +1677,8 @@ int drm_mode_setplane(struct drm_device *dev, void > *data, > /* No fb means shut it down */ > if (!plane_req->fb_id) { > plane->funcs->disable_plane(plane); > + plane->crtc = NULL; > + plane->fb = NULL; > goto out; > } >
A 0 return code should also indicate that the function succeeded, but this doesn't hurt. Acked-by: Jesse Barnes <jbarnes at virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20111111/b67d507d/attachment.pgp>