Thanks for the comments, Thierry. We can surely do this.
Regards Shashank -----Original Message----- From: Thierry Reding [mailto:thierry.reding@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-devel@lists.freedesktop.org; hverkuil@xs4all.nl; daniel@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