Thanks for the comments, Thierry.
We can surely do this.
Regards
Shashank
-----Original Message-----
From: Thierry Reding [mailto:thierry.red...@gmail.com]
Sent: Wednesday, July 15, 2015 6:55 PM
To: Malladi, Kausal
Cc: Roper, Matthew D; Barnes, Jesse; Lespiau, Damien; Jindal, Sonika; R,
Durgadoss; Purushothaman, Vijay A; intel-gfx@lists.freedesktop.org;
dri-de...@lists.freedesktop.org; hverk...@xs4all.nl; dan...@fooishbar.org;
Matheson, Annie J; Palleti, Avinash Reddy; Mukherjee, Indranil; R, Dhanya p;
Kamath, Sunil; Vetter, Daniel; Bhattacharjee, Susanta; Kumar, Kiran S; Sharma,
Shashank
Subject: Re: [PATCH 02/16] drm: Create Color Management DRM properties
On Wed, Jul 15, 2015 at 06:39:26PM +0530, Kausal Malladi wrote:
[...]
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index
> 57ca8cc..408d39a 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -1178,6 +1178,12 @@ struct drm_mode_config {
> struct drm_property *suggested_x_property;
> struct drm_property *suggested_y_property;
>
> + /* Color Management Properties */
> + struct drm_property *prop_color_capabilities;
> + struct drm_property *prop_palette_before_ctm;
> + struct drm_property *prop_palette_after_ctm;
> + struct drm_property *prop_ctm;
All existing properties are named *_property, and it'd be good to keep things
consistent by following that scheme. Perhaps they could have a common prefix
(cm_?) to group them?
Thierry
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
