[PATCH 3/7] drm: plane: Clear plane.crtc and plane.fb after disable_plane()

Jesse Barnes jbarnes at virtuousgeek.org
Fri Nov 11 08:56:19 PST 2011


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>


More information about the dri-devel mailing list