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 Thu, Oct 11, 2018 at 08:24:55AM +0200, Hans Verkuil wrote:
> 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.

For next time around: Just go ahead and apply to drm-misc, since that
probably means the author doesn't have commit rights :-)

Anyway applied now, thanks for the patch.
-Daniel

> 
> 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

-- 
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