drm/i915/tv: extract set_color_conversion
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 24 Apr 2014 21:54:40 +0000 (23:54 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 5 May 2014 08:56:57 +0000 (10:56 +0200)
intel_tv_mode_set is still too bug.

Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_tv.c

index 04bf8caaac0c19d399de83a2d604086ee03dfac4..a6acaeec7d69e250241346d7695881fb55b0fa76 100644 (file)
@@ -993,6 +993,26 @@ set_tv_mode_timings(struct drm_i915_private *dev_priv,
        I915_WRITE(TV_V_CTL_7, vctl7);
 }
 
+static void set_color_conversion(struct drm_i915_private *dev_priv,
+                                const struct color_conversion *color_conversion)
+{
+       if (!color_conversion)
+               return;
+
+       I915_WRITE(TV_CSC_Y, (color_conversion->ry << 16) |
+                  color_conversion->gy);
+       I915_WRITE(TV_CSC_Y2, (color_conversion->by << 16) |
+                  color_conversion->ay);
+       I915_WRITE(TV_CSC_U, (color_conversion->ru << 16) |
+                  color_conversion->gu);
+       I915_WRITE(TV_CSC_U2, (color_conversion->bu << 16) |
+                  color_conversion->au);
+       I915_WRITE(TV_CSC_V, (color_conversion->rv << 16) |
+                  color_conversion->gv);
+       I915_WRITE(TV_CSC_V2, (color_conversion->bv << 16) |
+                  color_conversion->av);
+}
+
 static void intel_tv_mode_set(struct intel_encoder *encoder)
 {
        struct drm_device *dev = encoder->base.dev;
@@ -1079,20 +1099,7 @@ static void intel_tv_mode_set(struct intel_encoder *encoder)
        I915_WRITE(TV_SC_CTL_2, scctl2);
        I915_WRITE(TV_SC_CTL_3, scctl3);
 
-       if (color_conversion) {
-               I915_WRITE(TV_CSC_Y, (color_conversion->ry << 16) |
-                          color_conversion->gy);
-               I915_WRITE(TV_CSC_Y2, (color_conversion->by << 16) |
-                          color_conversion->ay);
-               I915_WRITE(TV_CSC_U, (color_conversion->ru << 16) |
-                          color_conversion->gu);
-               I915_WRITE(TV_CSC_U2, (color_conversion->bu << 16) |
-                          color_conversion->au);
-               I915_WRITE(TV_CSC_V, (color_conversion->rv << 16) |
-                          color_conversion->gv);
-               I915_WRITE(TV_CSC_V2, (color_conversion->bv << 16) |
-                          color_conversion->av);
-       }
+       set_color_conversion(dev_priv, color_conversion);
 
        if (INTEL_INFO(dev)->gen >= 4)
                I915_WRITE(TV_CLR_KNOBS, 0x00404000);