>-----Original Message-----
>From: Ville Syrjala <ville.syrj...@linux.intel.com>
>Sent: Thursday, November 28, 2019 11:54 PM
>To: intel-gfx@lists.freedesktop.org
>Cc: Shankar, Uma <uma.shan...@intel.com>
>Subject: [PATCH] drm/i915: Use the correct PCH transcoder for LPT/WPT in
>intel_sanitize_frame_start_delay()
>
>From: Ville Syrjälä <ville.syrj...@linux.intel.com>
>
>LPT/WPT only have PCH transcoder A. Make sure we poke at its chicken register
>instead of some non-existent register when FDI is being driven by pipe B or C.

Change looks good to me.
Reviewed-by: Uma Shankar <uma.shan...@intel.com>

>Cc: Uma Shankar <uma.shan...@intel.com>
>Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
>---
> drivers/gpu/drm/i915/display/intel_display.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>b/drivers/gpu/drm/i915/display/intel_display.c
>index 53dc310a5f6d..f99dbc3d9696 100644
>--- a/drivers/gpu/drm/i915/display/intel_display.c
>+++ b/drivers/gpu/drm/i915/display/intel_display.c
>@@ -17272,7 +17272,8 @@ static void intel_sanitize_frame_start_delay(const 
>struct
>intel_crtc_state *crtc
>               val |= TRANS_FRAME_START_DELAY(0);
>               I915_WRITE(reg, val);
>       } else {
>-              i915_reg_t reg = TRANS_CHICKEN2(crtc->pipe);
>+              enum pipe pch_transcoder = intel_crtc_pch_transcoder(crtc);
>+              i915_reg_t reg = TRANS_CHICKEN2(pch_transcoder);
>               u32 val;
>
>               val = I915_READ(reg);
>--
>2.23.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to