From: José Roberto de Souza <jose.souza@xxxxxxxxx> Tiger Lake has eDP-capable transcoders rather than a transcoder dedicated to eDP. Transcoder A is the one where we have PSR2. Actually transcoder B also supports PSR2 but only with software tracking that is not implemented. Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx> Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> Signed-off-by: José Roberto de Souza <jose.souza@xxxxxxxxx> Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> --- drivers/gpu/drm/i915/display/intel_psr.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c index 01070eb67571..1d36d7be015d 100644 --- a/drivers/gpu/drm/i915/display/intel_psr.c +++ b/drivers/gpu/drm/i915/display/intel_psr.c @@ -511,12 +511,19 @@ static bool intel_psr2_config_valid(struct intel_dp *intel_dp, int crtc_hdisplay = crtc_state->base.adjusted_mode.crtc_hdisplay; int crtc_vdisplay = crtc_state->base.adjusted_mode.crtc_vdisplay; int psr_max_h = 0, psr_max_v = 0; + enum transcoder supported; if (!dev_priv->psr.sink_psr2_support) return false; - if (crtc_state->cpu_transcoder != TRANSCODER_EDP) { - DRM_DEBUG_KMS("PSR2 is only supported in EDP transcoder\n"); + /* + * TODO: PSR2 is also supported in TRANSCODER_B on TGL+ but it requires + * software tracking + */ + supported = INTEL_GEN(dev_priv) >= 12 ? TRANSCODER_A : TRANSCODER_EDP; + if (crtc_state->cpu_transcoder != supported) { + DRM_DEBUG_KMS("PSR2 not supported in transcoder %s\n", + transcoder_name(supported)); return false; } -- 2.21.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx