On Wed, 08 Jan 2020, Manasi Navare <[email protected]> wrote: > Link has not been trained during connector initialization > hence reset the variable intel_dp->link_trained. This > avoids accidently calling link training routine before the pipe > is set up. > > Issue: https://gitlab.freedesktop.org/drm/intel/issues/585 > Signed-off-by: Manasi Navare <[email protected]> > Cc: Ville Syrjälä <[email protected]> > Cc: Jani Nikula <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > b/drivers/gpu/drm/i915/display/intel_dp.c > index d299cee175e7..f45533af7524 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -7471,6 +7471,7 @@ intel_dp_init_connector(struct intel_digital_port > *intel_dig_port, > intel_dp_set_source_rates(intel_dp); > > intel_dp->reset_link_params = true; > + intel_dp->link_trained = false;
intel_dig_port and thus intel_dp are kzalloc'ed, making ->link_trained false to begin with. I can't see any way ->link_training could be true at this point. BR, Jani. > intel_dp->pps_pipe = INVALID_PIPE; > intel_dp->active_pipe = INVALID_PIPE; -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
