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
