On Thu, Feb 03, 2022 at 08:38:18PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <[email protected]> > > Remove all the dead code from icl_ddi_bigjoiner_pre_enable(). > > Signed-off-by: Ville Syrjälä <[email protected]>
Yup good catch here and thank you for cleaning up the dead code Reviewed-by: Manasi Navare <[email protected]> Manasi > --- > drivers/gpu/drm/i915/display/intel_display.c | 18 +----------------- > 1 file changed, 1 insertion(+), 17 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > b/drivers/gpu/drm/i915/display/intel_display.c > index 48869478efc2..d5dc2c25c1f6 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -1974,23 +1974,7 @@ static void hsw_set_frame_start_delay(const struct > intel_crtc_state *crtc_state) > static void icl_ddi_bigjoiner_pre_enable(struct intel_atomic_state *state, > const struct intel_crtc_state > *crtc_state) > { > - struct intel_crtc_state *master_crtc_state; > - struct intel_crtc *master_crtc; > - struct drm_connector_state *conn_state; > - struct drm_connector *conn; > - struct intel_encoder *encoder = NULL; > - int i; > - > - master_crtc = intel_master_crtc(crtc_state); > - master_crtc_state = intel_atomic_get_new_crtc_state(state, master_crtc); > - > - for_each_new_connector_in_state(&state->base, conn, conn_state, i) { > - if (conn_state->crtc != &master_crtc->base) > - continue; > - > - encoder = to_intel_encoder(conn_state->best_encoder); > - break; > - } > + struct intel_crtc *master_crtc = intel_master_crtc(crtc_state); > > /* > * Enable sequence steps 1-7 on bigjoiner master > -- > 2.34.1 >
