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

Malladi, Kausal
2015-06-05T09:28:13Z

Hi Sonika,

Thanks for your suggestion. Will follow that from next time.

Regards
Kausal

On Friday 05 June 2015 09:31 AM, Jindal, Sonika wrote:
> 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