Re: [PATCH] drm: Do not call drm_dp_cec_set_edid() while registering DP connectors

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

 



On 11/10/18 02:44, José Roberto de Souza wrote:
> drm_dp_cec_register_connector() is called when registering each DP
> connector in DRM, while sounds a good idea register CEC adapters as
> earlier as possible, it causes some driver initialization delay
> trying to do DPCD transactions in disconnected connectors.
> 
> This change will cause no regressions as drm_dp_cec_set_edid() will
> still be called in further detection of connected connectors with a
> valid edid parameter.
> 
> This change reduced the module load of i915 by average 0.5sec in a
> machine with just one DP port disconnected while reducing more than
> 3sec in a machine with 4 DP ports disconnected.
> 
> Cc: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> Signed-off-by: José Roberto de Souza <jose.souza@xxxxxxxxx>

I acked this on Sep 25th, but it seems you missed that.

Anyway:

Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>

Regards,

	Hans

> ---
>  drivers/gpu/drm/drm_dp_cec.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_dp_cec.c b/drivers/gpu/drm/drm_dp_cec.c
> index 8a718f85079a..b15cee85b702 100644
> --- a/drivers/gpu/drm/drm_dp_cec.c
> +++ b/drivers/gpu/drm/drm_dp_cec.c
> @@ -424,8 +424,6 @@ void drm_dp_cec_register_connector(struct drm_dp_aux *aux, const char *name,
>  	aux->cec.parent = parent;
>  	INIT_DELAYED_WORK(&aux->cec.unregister_work,
>  			  drm_dp_cec_unregister_work);
> -
> -	drm_dp_cec_set_edid(aux, NULL);
>  }
>  EXPORT_SYMBOL(drm_dp_cec_register_connector);
>  
> 

_______________________________________________
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