Commit 278a7be9 authored by Jouni Högander's avatar Jouni Högander Committed by Animesh Manna
Browse files

drm/i915/alpm: use variable from intel_crtc_state instead of intel_psr 



Currently code is making assumption that PSR is enabled when
intel_alpm_configure is called. This doesn't work if alpm is configured
before PSR is enabled.

Signed-off-by: default avatarJouni Högander <jouni.hogander@intel.com>
Reviewed-by: default avatarAnimesh Manna <animesh.manna@intel.com>
Signed-off-by: default avatarAnimesh Manna <animesh.manna@intel.com>
Link: https://lore.kernel.org/r/20250423092334.2294483-2-animesh.manna@intel.com
parent d56c95d4

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

+2 −2
Original line number Diff line number Diff line
@@ -317,14 +317,14 @@ static void lnl_alpm_configure(struct intel_dp *intel_dp, 
	u32 alpm_ctl;



	if (DISPLAY_VER(display) < 20 ||

	    (!intel_dp->psr.sel_update_enabled && !intel_dp_is_edp(intel_dp)))

	    (!crtc_state->has_sel_update && !intel_dp_is_edp(intel_dp)))

		return;



	/*

	 * Panel Replay on eDP is always using ALPM aux less. I.e. no need to

	 * check panel support at this point.

	 */

	if ((intel_dp->psr.panel_replay_enabled && intel_dp_is_edp(intel_dp)) ||

	if ((crtc_state->has_panel_replay && intel_dp_is_edp(intel_dp)) ||

	    (crtc_state->has_lobf && intel_alpm_aux_less_wake_supported(intel_dp))) {

		alpm_ctl = ALPM_CTL_ALPM_ENABLE |

			ALPM_CTL_ALPM_AUX_LESS_ENABLE |