Re: [PATCH v2 00/10] Color Manager Implementation

Jindal, Sonika
2015-06-05T04:01:03Z

HI Kausal,

You don't need to send the entire series again .
Just send the updated patch --in-reply-to to the last message.
Otherwise the thread gets lost.

You can send the entire series once the review is complete and you feel that the patches are too nested inside.
Please keep sending the patches using --in-reply-to tag.

Regards,
Sonika

-----Original Message-----
From: Malladi, Kausal
Sent: Thursday, June 4, 2015 7:13 PM
To: Roper, Matthew D; Barnes, Jesse; Lespiau, Damien; Jindal, Sonika; R, Durgadoss; Purushothaman, Vijay A; intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org
Cc: Vetter, Daniel; Sharma, Shashank; Kamath, Sunil; Mukherjee, Indranil; Matheson, Annie J; R, Dhanya p; Palleti, Avinash Reddy; Malladi, Kausal
Subject: [PATCH v2 00/10] Color Manager Implementation

From: Kausal Malladi <Kausal.Malladi@intel.com>

This patch set adds color manager implementation in drm/i915 layer.
Color Manager is an extension in i915 driver to support color correction/enhancement. Various Intel platforms support several color correction capabilities. Color Manager provides abstraction of these properties and allows a user space UI agent to correct/enhance the display.

The first three patches add code for the framework, which will be common across all the Intel platforms.
drm/i915: Initialize Color Manager
drm/i915: Attach color properties to CRTC
drm/i915: Add Set property interface for CRTC

In the next patches, we are adding support for Gamma and CSC color properties. Please note that:
1. The current implementation is only limited for CHV/BSW platforms, and the support for
other platforms will be following up soon.
2. The current patch set implements only the "set" part of the properties, "get" part will
be following up soon.

The design of color management on linux is done by:
Jesse Barnes
Sirisha Muppavarapu
Shashank Sharma
Susanta Bhattacharjee

The complete design is explained in the design document, which is available at https://docs.google.com/document/d/1jyfNSAStKHEpmIUZd_1Gd68cPuyiyr8wmJXThSDb_2w/edit#

v2: Addressed review comments from Sonika and Daniel Stone.

Kausal Malladi (10):
drm/i915: Initialize Color Manager
drm/i915: Attach color properties to CRTC
drm/i915: Add atomic set property interface for CRTC
drm: Add Gamma correction structure
drm: Add a new function for updating color blob
drm: Avoid atomic commit path for CRTC property (Gamma)
drm/i915: Add pipe level Gamma correction for CHV/BSW
drm: Add CSC correction structure
drm: Avoid atomic commit path for CSC property on CRTC
drm/i915: Add CSC support for CHV/BSW

drivers/gpu/drm/drm_atomic_helper.c | 18 +-
drivers/gpu/drm/drm_crtc.c | 15 ++
drivers/gpu/drm/i915/Makefile | 3 +
drivers/gpu/drm/i915/intel_atomic.c | 19 +-
drivers/gpu/drm/i915/intel_color_manager.c | 348 +++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_color_manager.h | 122 ++++++++++
drivers/gpu/drm/i915/intel_display.c | 8 +
drivers/gpu/drm/i915/intel_drv.h | 4 +
include/drm/drm_crtc.h | 12 +
include/uapi/drm/drm.h | 18 ++
10 files changed, 563 insertions(+), 4 deletions(-) create mode 100644 drivers/gpu/drm/i915/intel_color_manager.c
create mode 100644 drivers/gpu/drm/i915/intel_color_manager.h

--
2.4.2

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org