Re: [PATCH 21/29] drm/omap: Remove src field from omap_dss_device structure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Wed, Dec 05, 2018 at 05:00:14PM +0200, Laurent Pinchart wrote:
> The field is only used to check whether the device is connected, and we
> can do so by checking the dss field instead. Remove the src field.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---

Reviewed-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>

-- Sebastian

>  drivers/gpu/drm/omapdrm/dss/base.c    | 14 +-------------
>  drivers/gpu/drm/omapdrm/dss/omapdss.h |  1 -
>  2 files changed, 1 insertion(+), 14 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/dss/base.c b/drivers/gpu/drm/omapdrm/dss/base.c
> index 8454e505b282..36f2d41c9935 100644
> --- a/drivers/gpu/drm/omapdrm/dss/base.c
> +++ b/drivers/gpu/drm/omapdrm/dss/base.c
> @@ -187,7 +187,7 @@ EXPORT_SYMBOL(omapdss_device_get_next);
>  
>  static bool omapdss_device_is_connected(struct omap_dss_device *dssdev)
>  {
> -	return dssdev->src;
> +	return dssdev->dss;
>  }
>  
>  int omapdss_device_connect(struct dss_device *dss,
> @@ -209,11 +209,6 @@ int omapdss_device_connect(struct dss_device *dss,
>  		return ret;
>  	}
>  
> -	if (src) {
> -		WARN_ON(dst->src);
> -		dst->src = src;
> -	}
> -
>  	return 0;
>  }
>  EXPORT_SYMBOL_GPL(omapdss_device_connect);
> @@ -228,13 +223,6 @@ void omapdss_device_disconnect(struct omap_dss_device *src,
>  		return;
>  	}
>  
> -	if (src) {
> -		if (WARN_ON(dst->src != src))
> -			return;
> -
> -		dst->src = NULL;
> -	}
> -
>  	WARN_ON(dst->state != OMAP_DSS_DISPLAY_DISABLED);
>  
>  	dst->ops->disconnect(src, dst);
> diff --git a/drivers/gpu/drm/omapdrm/dss/omapdss.h b/drivers/gpu/drm/omapdrm/dss/omapdss.h
> index 7c9f8a5ceb37..80db32bc896f 100644
> --- a/drivers/gpu/drm/omapdrm/dss/omapdss.h
> +++ b/drivers/gpu/drm/omapdrm/dss/omapdss.h
> @@ -414,7 +414,6 @@ struct omap_dss_device {
>  	struct module *owner;
>  
>  	struct dss_device *dss;
> -	struct omap_dss_device *src;
>  	struct omap_dss_device *next;
>  
>  	struct list_head list;
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux