Skip to content

Commit 72eea91

Browse files
committed
drm/i915: Drop redundant shared_dpll=NULL assignments
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 <[email protected]> Signed-off-by: Ville Syrjälä <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent 68b97b1 commit 72eea91

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

drivers/gpu/drm/i915/display/intel_display.c

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2997,7 +2997,6 @@ static bool i9xx_get_pipe_config(struct intel_crtc *crtc,
29972997
pipe_config->output_format = INTEL_OUTPUT_FORMAT_RGB;
29982998
pipe_config->sink_format = pipe_config->output_format;
29992999
pipe_config->cpu_transcoder = (enum transcoder) crtc->pipe;
3000-
pipe_config->shared_dpll = NULL;
30013000

30023001
ret = false;
30033002

@@ -3339,7 +3338,6 @@ static bool ilk_get_pipe_config(struct intel_crtc *crtc,
33393338
return false;
33403339

33413340
pipe_config->cpu_transcoder = (enum transcoder) crtc->pipe;
3342-
pipe_config->shared_dpll = NULL;
33433341

33443342
ret = false;
33453343
tmp = intel_de_read(display,
@@ -3906,8 +3904,6 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
39063904
POWER_DOMAIN_PIPE(crtc->pipe)))
39073905
return false;
39083906

3909-
pipe_config->shared_dpll = NULL;
3910-
39113907
active = hsw_get_transcoder_state(crtc, pipe_config, &crtc->hw_readout_power_domains);
39123908

39133909
if ((display->platform.geminilake || display->platform.broxton) &&

0 commit comments

Comments
 (0)