Thanks for the comments, Thierry. We can surely do this. Regards Shashank -----Original Message----- From: Thierry Reding [mailto:thierry.reding@xxxxxxxxx] 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@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx; hverkuil@xxxxxxxxx; daniel@xxxxxxxxxxxxx; 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@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
