>-----Original Message-----
>From: Kulkarni, Vandita
>Sent: Tuesday, July 2, 2019 9:49 AM
>To: intel-gfx@lists.freedesktop.org
>Cc: ville.syrj...@linux.intel.com; Nikula, Jani <jani.nik...@intel.com>; 
>Shankar, Uma
><uma.shan...@intel.com>; Kulkarni, Vandita <vandita.kulka...@intel.com>
>Subject: [PATCH 1/4] drm/i915/tgl/dsi: Program TRANS_VBLANK register
>
>Program vblank register for mipi dsi in video mode on TGL.
>
>Signed-off-by: Vandita Kulkarni <vandita.kulka...@intel.com>
>---
> drivers/gpu/drm/i915/display/icl_dsi.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
>diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c
>b/drivers/gpu/drm/i915/display/icl_dsi.c
>index b8673debf932..556eba2636fe 100644
>--- a/drivers/gpu/drm/i915/display/icl_dsi.c
>+++ b/drivers/gpu/drm/i915/display/icl_dsi.c
>@@ -866,6 +866,15 @@ gen11_dsi_set_transcoder_timings(struct intel_encoder
>*encoder,
>               dsi_trans = dsi_port_to_transcoder(port);
>               I915_WRITE(VSYNCSHIFT(dsi_trans), vsync_shift);
>       }
>+
>+      /* program TRANS_VBLANK register, should be same as vtotal progammed */

Typo here in programmed.

>+      if (INTEL_GEN(dev_priv) >= 12) {
>+              for_each_dsi_port(port, intel_dsi->ports) {
>+                      dsi_trans = dsi_port_to_transcoder(port);
>+                      I915_WRITE(VBLANK(dsi_trans),
>+                                 (vactive - 1) | ((vtotal - 1) << 16));

We can put this line along with VTOTAL and get rid of this extra for loop.

>+              }
>+      }
> }
>
> static void gen11_dsi_enable_transcoder(struct intel_encoder *encoder)
>--
>2.21.0.5.gaeb582a

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

Reply via email to