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.verk...@cisco.com>
> Signed-off-by: José Roberto de Souza <jose.so...@intel.com>
I acked this on Sep 25th, but it seems you missed that.
Anyway:
Acked-by: Hans Verkuil <hans.verk...@cisco.com>
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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
