[Intel-gfx] [v2 3/7] drm: Add gamma mode property
Sam Ravnborg
sam at ravnborg.org
Mon Apr 1 18:37:26 UTC 2019
Hi Uma.
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -256,6 +256,13 @@ struct drm_crtc_state {
> struct drm_property_blob *gamma_mode_caps;
>
> /**
> + * @gamma_mode:
> + *
> + * FIXME
> + */
> + struct drm_property_blob *gamma_mode;
HEre the name matches, but please add documentation too.
>
> +struct drm_color_mode_lut {
> + /* DRM_MODE_LUT_* */
> + __u32 flags;
> + /* number of points on the curve */
> + __u32 count;
> + /* Name of Gamma Mode */
> + char name[DRM_PROP_NAME_LEN];
> + /* Pointer to Lut elements */
> + __u64 lut;
> +};
>From an alignment point of view the __u64 should come before the char name[].
But the above may be fine depending on DRM_PROP_NAME_LEN
Sam
More information about the Intel-gfx
mailing list