On Wed, Jul 01, 2015 at 09:18:17PM +0530, Kausal Malladi wrote:

From: Kausal Malladi Kausal.Malladi@intel.com

This patch adds new structures in DRM layer for Palette color correction. These structures will be used by user space agents to configure appropriate number of samples and Palette LUT for a platform.

Signed-off-by: Shashank Sharma shashank.sharma@intel.com Signed-off-by: Kausal Malladi Kausal.Malladi@intel.com

---

include/uapi/drm/drm.h | 12 ++++++++++++ 1 file changed, 12 insertions(+)

diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h index d9562a2..04a8f2a 100644 --- a/include/uapi/drm/drm.h +++ b/include/uapi/drm/drm.h @@ -863,6 +863,18 @@ struct drm_color_caps { struct drm_cge_caps cge_caps; };

+struct drm_r32g32b32 {

• __u32 r32;
• __u32 g32;
• __u32 b32;

+};

r32 means red is an u32 to me. Is that the right encoding? I thought we need values > 1.0 for our hw?

+struct drm_palette {

• __u32 version;
• __u32 palette_num_samples;

No need to prefix the fields with the structure name.

• struct drm_r32g32b32 palette_lut[0];

+};

/* typedef area */ #ifndef __KERNEL__ typedef struct drm_clip_rect drm_clip_rect_t; -- 2.4.5