Re: [Intel-gfx] [PATCH 05/10] drm/i915: Nuke some dead code

["Navare, Manasi" <manasi.d.navare@intel.com>]

On Thu, Feb 03, 2022 at 08:38:18PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> Remove all the dead code from icl_ddi_bigjoiner_pre_enable().
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Yup good catch here and thank you for cleaning up the dead code

Reviewed-by: Manasi Navare <manasi.d.navare@intel.com>

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
>