From: Ville Syrjälä <ville.syrj...@linux.intel.com>

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.govindapil...@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c 
b/drivers/gpu/drm/i915/display/intel_display.c
index 51c011de4a16..5c1c2cb83d51 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -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) &&
-- 
2.45.3

Reply via email to