drm/i915: Drop redundant shared_dpll=NULL assignments
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 6 Mar 2025 16:34:03 +0000 (18:34 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 7 Mar 2025 17:02:13 +0000 (19:02 +0200)
The crtc state is expected to be fully cleared before readout,
so there is no need to clear the shared_dpll pointers by hand.

Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250306163420.3961-2-ville.syrjala@linux.intel.com
drivers/gpu/drm/i915/display/intel_display.c

index 51c011d..5c1c2cb 100644 (file)
@@ -2997,7 +2997,6 @@ static bool i9xx_get_pipe_config(struct intel_crtc *crtc,
        pipe_config->output_format = INTEL_OUTPUT_FORMAT_RGB;
        pipe_config->sink_format = pipe_config->output_format;
        pipe_config->cpu_transcoder = (enum transcoder) crtc->pipe;
-       pipe_config->shared_dpll = NULL;
 
        ret = false;
 
@@ -3339,7 +3338,6 @@ static bool ilk_get_pipe_config(struct intel_crtc *crtc,
                return false;
 
        pipe_config->cpu_transcoder = (enum transcoder) crtc->pipe;
-       pipe_config->shared_dpll = NULL;
 
        ret = false;
        tmp = intel_de_read(display,
@@ -3906,8 +3904,6 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
                                                       POWER_DOMAIN_PIPE(crtc->pipe)))
                return false;
 
-       pipe_config->shared_dpll = NULL;
-
        active = hsw_get_transcoder_state(crtc, pipe_config, &crtc->hw_readout_power_domains);
 
        if ((display->platform.geminilake || display->platform.broxton) &&