[Intel-gfx] [PATCH 04/16] drm: Add structure for querying palette color capabilities

[Sharma, Shashank]

Regards
Shashank

-----Original Message-----
From: Intel-gfx [mailto:intel-gfx-bounces at lists.freedesktop.org] On Behalf Of Bish, Jim
Sent: Tuesday, July 21, 2015 3:48 AM
To: intel-gfx at 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 at intel.com>
> Signed-off-by: Kausal Malladi <Kausal.Malladi at 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 at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx