diff mbox

[04/16] drm: Add structure for querying palette color capabilities

Message ID 1436965780-6061-5-git-send-email-Kausal.Malladi@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Kausal Malladi July 15, 2015, 1:09 p.m. UTC
The DRM color management framework is targeting various hardware
platforms and drivers. Different platforms can have different color
correction and enhancement capabilities.

A commom user space application can query these capabilities using the
DRM property interface. Each driver can fill this property with its
platform's palette color capabilities.

This patch adds new structure in DRM layer for querying palette color
capabilities. This structure will be used by all user space
agents to configure appropriate color configurations.

Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>
Signed-off-by: Kausal Malladi <Kausal.Malladi@intel.com>
---
 include/uapi/drm/drm.h | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Bish, Jim July 20, 2015, 10:17 p.m. UTC | #1
On 07/15/2015 06:09 AM, Kausal Malladi wrote:
> The DRM color management framework is targeting various hardware

> platforms and drivers. Different platforms can have different color

> correction and enhancement capabilities.

> 

> A commom user space application can query these capabilities using the

> DRM property interface. Each driver can fill this property with its

> platform's palette color capabilities.

> 

> This patch adds new structure in DRM layer for querying palette color

> capabilities. This structure will be used by all user space

> agents to configure appropriate color configurations.

> 

> Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>

> Signed-off-by: Kausal Malladi <Kausal.Malladi@intel.com>

> ---

>  include/uapi/drm/drm.h | 11 +++++++++++

>  1 file changed, 11 insertions(+)

> 

> diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h

> index 3801584..e3c642f 100644

> --- a/include/uapi/drm/drm.h

> +++ b/include/uapi/drm/drm.h

> @@ -829,6 +829,17 @@ struct drm_event_vblank {

>  	__u32 reserved;

>  };

>  

> +struct drm_palette_caps {

> +	/* Structure version. Should be 1 currently */

> +	__u32 version;

> +	/* For padding and future use */

> +	__u32 reserved;

> +	/* This may be 0 if not supported. e.g. plane palette or VLV pipe */

> +	__u32 num_samples_before_ctm;

> +	/* This will be non-zero for pipe. May be zero for planes on some HW */

> +	__u32 num_samples_after_ctm;

> +};

this structure does not match what is documented in the design document.  are we 
missing updates to the design document?
> +

>  /* typedef area */

>  #ifndef __KERNEL__

>  typedef struct drm_clip_rect drm_clip_rect_t;

>
Sharma, Shashank July 29, 2015, 11:39 a.m. UTC | #2
Regards
Shashank

-----Original Message-----
From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of Bish, Jim

Sent: Tuesday, July 21, 2015 3:48 AM
To: intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH 04/16] drm: Add structure for querying palette color capabilities



On 07/15/2015 06:09 AM, Kausal Malladi wrote:
> The DRM color management framework is targeting various hardware 

> platforms and drivers. Different platforms can have different color 

> correction and enhancement capabilities.

> 

> A commom user space application can query these capabilities using the 

> DRM property interface. Each driver can fill this property with its 

> platform's palette color capabilities.

> 

> This patch adds new structure in DRM layer for querying palette color 

> capabilities. This structure will be used by all user space agents to 

> configure appropriate color configurations.

> 

> Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>

> Signed-off-by: Kausal Malladi <Kausal.Malladi@intel.com>

> ---

>  include/uapi/drm/drm.h | 11 +++++++++++

>  1 file changed, 11 insertions(+)

> 

> diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h index 

> 3801584..e3c642f 100644

> --- a/include/uapi/drm/drm.h

> +++ b/include/uapi/drm/drm.h

> @@ -829,6 +829,17 @@ struct drm_event_vblank {

>  	__u32 reserved;

>  };

>  

> +struct drm_palette_caps {

> +	/* Structure version. Should be 1 currently */

> +	__u32 version;

> +	/* For padding and future use */

> +	__u32 reserved;

> +	/* This may be 0 if not supported. e.g. plane palette or VLV pipe */

> +	__u32 num_samples_before_ctm;

> +	/* This will be non-zero for pipe. May be zero for planes on some HW */

> +	__u32 num_samples_after_ctm;

> +};

this structure does not match what is documented in the design document.  are we missing updates to the design document?
Yes, Susanta will be updating the document in a day or two. Thanks for reminding us. 
> +

>  /* typedef area */

>  #ifndef __KERNEL__

>  typedef struct drm_clip_rect drm_clip_rect_t;

> 

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h
index 3801584..e3c642f 100644
--- a/include/uapi/drm/drm.h
+++ b/include/uapi/drm/drm.h
@@ -829,6 +829,17 @@  struct drm_event_vblank {
 	__u32 reserved;
 };
 
+struct drm_palette_caps {
+	/* Structure version. Should be 1 currently */
+	__u32 version;
+	/* For padding and future use */
+	__u32 reserved;
+	/* This may be 0 if not supported. e.g. plane palette or VLV pipe */
+	__u32 num_samples_before_ctm;
+	/* This will be non-zero for pipe. May be zero for planes on some HW */
+	__u32 num_samples_after_ctm;
+};
+
 /* typedef area */
 #ifndef __KERNEL__
 typedef struct drm_clip_rect drm_clip_rect_t;