drm/i915/psr: Enable psr2 early transport as possible
Check source and sink support for psr2 early transport and enable it if not disabled by debug flag. Bspec: 68934 Signed-off-by: Jouni Högander <jouni.hogander@intel.com> Reviewed-by: Mika Kahola <mika.kahola@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20231218175004.52875-7-jouni.hogander@intel.com
This commit is contained in:
@@ -1213,6 +1213,7 @@ struct intel_crtc_state {
|
|||||||
bool has_psr;
|
bool has_psr;
|
||||||
bool has_psr2;
|
bool has_psr2;
|
||||||
bool enable_psr2_sel_fetch;
|
bool enable_psr2_sel_fetch;
|
||||||
|
bool enable_psr2_su_region_et;
|
||||||
bool req_psr2_sdp_prior_scanline;
|
bool req_psr2_sdp_prior_scanline;
|
||||||
bool has_panel_replay;
|
bool has_panel_replay;
|
||||||
bool wm_level_disabled;
|
bool wm_level_disabled;
|
||||||
@@ -1690,6 +1691,7 @@ struct intel_psr {
|
|||||||
#define I915_PSR_DEBUG_FORCE_PSR1 0x03
|
#define I915_PSR_DEBUG_FORCE_PSR1 0x03
|
||||||
#define I915_PSR_DEBUG_ENABLE_SEL_FETCH 0x4
|
#define I915_PSR_DEBUG_ENABLE_SEL_FETCH 0x4
|
||||||
#define I915_PSR_DEBUG_IRQ 0x10
|
#define I915_PSR_DEBUG_IRQ 0x10
|
||||||
|
#define I915_PSR_DEBUG_SU_REGION_ET_DISABLE 0x20
|
||||||
|
|
||||||
u32 debug;
|
u32 debug;
|
||||||
bool sink_support;
|
bool sink_support;
|
||||||
|
|||||||
@@ -528,7 +528,7 @@ static void _psr_init_dpcd(struct intel_dp *intel_dp)
|
|||||||
intel_dp_get_sink_sync_latency(intel_dp);
|
intel_dp_get_sink_sync_latency(intel_dp);
|
||||||
|
|
||||||
if (DISPLAY_VER(i915) >= 9 &&
|
if (DISPLAY_VER(i915) >= 9 &&
|
||||||
intel_dp->psr_dpcd[0] == DP_PSR2_WITH_Y_COORD_IS_SUPPORTED) {
|
intel_dp->psr_dpcd[0] >= DP_PSR2_WITH_Y_COORD_IS_SUPPORTED) {
|
||||||
bool y_req = intel_dp->psr_dpcd[1] &
|
bool y_req = intel_dp->psr_dpcd[1] &
|
||||||
DP_PSR2_SU_Y_COORDINATE_REQUIRED;
|
DP_PSR2_SU_Y_COORDINATE_REQUIRED;
|
||||||
bool alpm = intel_dp_get_alpm_status(intel_dp);
|
bool alpm = intel_dp_get_alpm_status(intel_dp);
|
||||||
@@ -601,6 +601,18 @@ static void hsw_psr_setup_aux(struct intel_dp *intel_dp)
|
|||||||
aux_ctl);
|
aux_ctl);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static bool psr2_su_region_et_valid(struct intel_dp *intel_dp)
|
||||||
|
{
|
||||||
|
struct drm_i915_private *i915 = dp_to_i915(intel_dp);
|
||||||
|
|
||||||
|
if (DISPLAY_VER(i915) >= 20 &&
|
||||||
|
intel_dp->psr_dpcd[0] == DP_PSR2_WITH_Y_COORD_ET_SUPPORTED &&
|
||||||
|
!(intel_dp->psr.debug & I915_PSR_DEBUG_SU_REGION_ET_DISABLE))
|
||||||
|
return true;
|
||||||
|
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
static void intel_psr_enable_sink(struct intel_dp *intel_dp)
|
static void intel_psr_enable_sink(struct intel_dp *intel_dp)
|
||||||
{
|
{
|
||||||
struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
|
struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
|
||||||
@@ -616,6 +628,8 @@ static void intel_psr_enable_sink(struct intel_dp *intel_dp)
|
|||||||
DP_ALPM_LOCK_ERROR_IRQ_HPD_ENABLE);
|
DP_ALPM_LOCK_ERROR_IRQ_HPD_ENABLE);
|
||||||
|
|
||||||
dpcd_val |= DP_PSR_ENABLE_PSR2 | DP_PSR_IRQ_HPD_WITH_CRC_ERRORS;
|
dpcd_val |= DP_PSR_ENABLE_PSR2 | DP_PSR_IRQ_HPD_WITH_CRC_ERRORS;
|
||||||
|
if (psr2_su_region_et_valid(intel_dp))
|
||||||
|
dpcd_val |= DP_PSR_ENABLE_SU_REGION_ET;
|
||||||
} else {
|
} else {
|
||||||
if (intel_dp->psr.link_standby)
|
if (intel_dp->psr.link_standby)
|
||||||
dpcd_val |= DP_PSR_MAIN_LINK_ACTIVE;
|
dpcd_val |= DP_PSR_MAIN_LINK_ACTIVE;
|
||||||
@@ -866,6 +880,9 @@ static void hsw_activate_psr2(struct intel_dp *intel_dp)
|
|||||||
intel_de_write(dev_priv, PSR2_MAN_TRK_CTL(cpu_transcoder), 0);
|
intel_de_write(dev_priv, PSR2_MAN_TRK_CTL(cpu_transcoder), 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (psr2_su_region_et_valid(intel_dp))
|
||||||
|
val |= LNL_EDP_PSR2_SU_REGION_ET_ENABLE;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* PSR2 HW is incorrectly using EDP_PSR_TP1_TP3_SEL and BSpec is
|
* PSR2 HW is incorrectly using EDP_PSR_TP1_TP3_SEL and BSpec is
|
||||||
* recommending keep this bit unset while PSR2 is enabled.
|
* recommending keep this bit unset while PSR2 is enabled.
|
||||||
@@ -1028,6 +1045,9 @@ static bool intel_psr2_sel_fetch_config_valid(struct intel_dp *intel_dp,
|
|||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (psr2_su_region_et_valid(intel_dp))
|
||||||
|
crtc_state->enable_psr2_su_region_et = true;
|
||||||
|
|
||||||
return crtc_state->enable_psr2_sel_fetch = true;
|
return crtc_state->enable_psr2_sel_fetch = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -159,6 +159,7 @@
|
|||||||
#define TGL_EDP_PSR2_BLOCK_COUNT_MASK REG_BIT(28)
|
#define TGL_EDP_PSR2_BLOCK_COUNT_MASK REG_BIT(28)
|
||||||
#define TGL_EDP_PSR2_BLOCK_COUNT_NUM_2 REG_FIELD_PREP(TGL_EDP_PSR2_BLOCK_COUNT_MASK, 0)
|
#define TGL_EDP_PSR2_BLOCK_COUNT_NUM_2 REG_FIELD_PREP(TGL_EDP_PSR2_BLOCK_COUNT_MASK, 0)
|
||||||
#define TGL_EDP_PSR2_BLOCK_COUNT_NUM_3 REG_FIELD_PREP(TGL_EDP_PSR2_BLOCK_COUNT_MASK, 1)
|
#define TGL_EDP_PSR2_BLOCK_COUNT_NUM_3 REG_FIELD_PREP(TGL_EDP_PSR2_BLOCK_COUNT_MASK, 1)
|
||||||
|
#define LNL_EDP_PSR2_SU_REGION_ET_ENABLE REG_BIT(27)
|
||||||
#define EDP_Y_COORDINATE_ENABLE REG_BIT(25) /* display 10, 11 and 12 */
|
#define EDP_Y_COORDINATE_ENABLE REG_BIT(25) /* display 10, 11 and 12 */
|
||||||
#define EDP_PSR2_SU_SDP_SCANLINE REG_BIT(25) /* display 13+ */
|
#define EDP_PSR2_SU_SDP_SCANLINE REG_BIT(25) /* display 13+ */
|
||||||
#define EDP_MAX_SU_DISABLE_TIME_MASK REG_GENMASK(24, 20)
|
#define EDP_MAX_SU_DISABLE_TIME_MASK REG_GENMASK(24, 20)
|
||||||
|
|||||||
Reference in New Issue
Block a user