The color correction blob values are loaded during set_property
calls. This patch adds a function to find the blob and apply the
correction values to the display registers, during the atomic
commit call.

Signed-off-by: Shashank Sharma <shashank.sharma at intel.com>
Signed-off-by: Kausal Malladi <kausalmalladi at gmail.com>
---
 drivers/gpu/drm/i915/intel_color_manager.c | 44 ++++++++++++++++++++++++++++++
 drivers/gpu/drm/i915/intel_display.c       |  2 ++
 drivers/gpu/drm/i915/intel_drv.h           |  2 ++
 3 files changed, 48 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_color_manager.c 
b/drivers/gpu/drm/i915/intel_color_manager.c
index 6661c53..6b59531 100644
--- a/drivers/gpu/drm/i915/intel_color_manager.c
+++ b/drivers/gpu/drm/i915/intel_color_manager.c
@@ -291,6 +291,50 @@ static int chv_set_gamma(struct drm_device *dev, struct 
drm_property_blob *blob,
        }
 }

+void intel_color_manager_crtc_commit(struct drm_device *dev,
+               struct drm_crtc_state *crtc_state)
+{
+       struct drm_property_blob *blob;
+       struct drm_crtc *crtc = crtc_state->crtc;
+       int ret = -EINVAL;
+
+       blob = crtc_state->palette_after_ctm_blob;
+       if (blob) {
+               /* Gamma correction is platform specific */
+               if (IS_CHERRYVIEW(dev))
+                       ret = chv_set_gamma(dev, blob, crtc);
+
+               if (ret)
+                       DRM_ERROR("set Gamma correction failed\n");
+               else
+                       DRM_DEBUG_DRIVER("Gamma correction success\n");
+       }
+
+       blob = crtc_state->palette_before_ctm_blob;
+       if (blob) {
+               /* Degamma correction */
+               if (IS_CHERRYVIEW(dev))
+                       ret = chv_set_degamma(dev, blob, crtc);
+
+               if (ret)
+                       DRM_ERROR("set degamma correction failed\n");
+               else
+                       DRM_DEBUG_DRIVER("degamma correction success\n");
+       }
+
+       blob = crtc_state->ctm_blob;
+       if (blob) {
+               /* CSC correction */
+               if (IS_CHERRYVIEW(dev))
+                       ret = chv_set_csc(dev, blob, crtc);
+
+               if (ret)
+                       DRM_ERROR("set CSC correction failed\n");
+               else
+                       DRM_DEBUG_DRIVER("CSC correction success\n");
+       }
+}
+
 void intel_attach_color_properties_to_crtc(struct drm_device *dev,
                struct drm_crtc *crtc)
 {
diff --git a/drivers/gpu/drm/i915/intel_display.c 
b/drivers/gpu/drm/i915/intel_display.c
index 7cad341..f94ed6d 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -13530,6 +13530,8 @@ static void intel_begin_crtc_commit(struct drm_crtc 
*crtc,
                intel_update_pipe_config(intel_crtc, old_intel_state);
        else if (INTEL_INFO(dev)->gen >= 9)
                skl_detach_scalers(intel_crtc);
+
+       intel_color_manager_crtc_commit(dev, crtc->state);
 }

 static void intel_finish_crtc_commit(struct drm_crtc *crtc,
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 2e4e97d..4b24496 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -1480,4 +1480,6 @@ extern const struct drm_plane_helper_funcs 
intel_plane_helper_funcs;
 /* intel_color_manager.c */
 void intel_attach_color_properties_to_crtc(struct drm_device *dev,
        struct drm_crtc *crtc);
+void intel_color_manager_crtc_commit(struct drm_device *dev,
+       struct drm_crtc_state *crtc_state);
 #endif /* __INTEL_DRV_H__ */
-- 
1.9.1

Reply via email to