Re: [PATCH 1/2] drm/sti: set initial connectors status at binding

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

 



On Wed, Jan 04, 2017 at 10:15:45AM +0100, Vincent Abriou wrote:
> This avoid to have an "unknown" connector status after the sti driver is
> registered.
> 
> Signed-off-by: Vincent Abriou <vincent.abriou@xxxxxx>

Why is this needed? The fb helpers will probe on their on, your driver
should never need to do this. It also slows down initial boot when you
don't need fbdev ...
-Daniel

> ---
>  drivers/gpu/drm/sti/sti_dvo.c  | 3 +++
>  drivers/gpu/drm/sti/sti_hdmi.c | 3 +++
>  2 files changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/sti/sti_dvo.c b/drivers/gpu/drm/sti/sti_dvo.c
> index e8c1ed0..0e73541 100644
> --- a/drivers/gpu/drm/sti/sti_dvo.c
> +++ b/drivers/gpu/drm/sti/sti_dvo.c
> @@ -498,6 +498,9 @@ static int sti_dvo_bind(struct device *dev, struct device *master, void *data)
>  	drm_connector_helper_add(drm_connector,
>  				 &sti_dvo_connector_helper_funcs);
>  
> +	/* set initial status */
> +	drm_connector->status = sti_dvo_connector_detect(drm_connector, false);
> +
>  	err = drm_mode_connector_attach_encoder(drm_connector, encoder);
>  	if (err) {
>  		DRM_ERROR("Failed to attach a connector to a encoder\n");
> diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
> index 9c0025e..2c949bc 100644
> --- a/drivers/gpu/drm/sti/sti_hdmi.c
> +++ b/drivers/gpu/drm/sti/sti_hdmi.c
> @@ -1322,6 +1322,9 @@ static int sti_hdmi_bind(struct device *dev, struct device *master, void *data)
>  	/* initialise property */
>  	sti_hdmi_connector_init_property(drm_dev, drm_connector);
>  
> +	/* set initial status */
> +	drm_connector->status = sti_hdmi_connector_detect(drm_connector, false);
> +
>  	hdmi->drm_connector = drm_connector;
>  
>  	err = drm_mode_connector_attach_encoder(drm_connector, encoder);
> -- 
> 2.7.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
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