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>

Reply via email to