On Wed, Feb 14, 2018 at 09:23:22PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> Include color_enconding and color_range in the plane state dump.
> 
> Cc: Harry Wentland <harry.wentl...@amd.com>
> Cc: Daniel Vetter <dan...@ffwll.ch>
> Cc: Daniel Stone <dan...@fooishbar.org>
> Cc: Russell King - ARM Linux <li...@armlinux.org.uk>
> Cc: Ilia Mirkin <imir...@alum.mit.edu>
> Cc: Hans Verkuil <hverk...@xs4all.nl>
> Cc: Uma Shankar <uma.shan...@intel.com>
> Cc: Shashank Sharma <shashank.sha...@intel.com>
> Cc: Jyri Sarha <jsa...@ti.com>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> ---
>  drivers/gpu/drm/drm_atomic.c        |  4 ++++
>  drivers/gpu/drm/drm_color_mgmt.c    | 16 ++++++++++++++++
>  drivers/gpu/drm/drm_crtc_internal.h |  2 ++
>  3 files changed, 22 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index 452a0b0bafbc..9552052ed31a 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -952,6 +952,10 @@ static void drm_atomic_plane_print_state(struct 
> drm_printer *p,
>       drm_printf(p, "\tcrtc-pos=" DRM_RECT_FMT "\n", DRM_RECT_ARG(&dest));
>       drm_printf(p, "\tsrc-pos=" DRM_RECT_FP_FMT "\n", DRM_RECT_FP_ARG(&src));
>       drm_printf(p, "\trotation=%x\n", state->rotation);
> +     drm_printf(p, "\tcolor-encoding=%s\n",
> +                drm_get_color_encoding_name(state->color_encoding));
> +     drm_printf(p, "\tcolor-range=%s\n",
> +                drm_get_color_range_name(state->color_range));
>  
>       if (plane->funcs->atomic_print_state)
>               plane->funcs->atomic_print_state(p, state);
> diff --git a/drivers/gpu/drm/drm_color_mgmt.c 
> b/drivers/gpu/drm/drm_color_mgmt.c
> index 061d342f9d96..06851d575f14 100644
> --- a/drivers/gpu/drm/drm_color_mgmt.c
> +++ b/drivers/gpu/drm/drm_color_mgmt.c
> @@ -365,6 +365,22 @@ static const char * const color_range_name[] = {
>       [DRM_COLOR_YCBCR_LIMITED_RANGE] = "YCbCr limited range",
>  };
>  

kerneldoc would be neat here ...
-Daniel

> +const char *drm_get_color_encoding_name(enum drm_color_encoding encoding)
> +{
> +     if (WARN_ON(encoding >= ARRAY_SIZE(color_encoding_name)))
> +             return "unknown";
> +
> +     return color_encoding_name[encoding];
> +}
> +
> +const char *drm_get_color_range_name(enum drm_color_range range)
> +{
> +     if (WARN_ON(range >= ARRAY_SIZE(color_range_name)))
> +             return "unknown";
> +
> +     return color_range_name[range];
> +}
> +
>  /**
>   * drm_plane_create_color_properties - color encoding related plane 
> properties
>   * @supported_encodings: bitfield indicating supported color encodings
> diff --git a/drivers/gpu/drm/drm_crtc_internal.h 
> b/drivers/gpu/drm/drm_crtc_internal.h
> index af00f42ba269..8ca2ffef6231 100644
> --- a/drivers/gpu/drm/drm_crtc_internal.h
> +++ b/drivers/gpu/drm/drm_crtc_internal.h
> @@ -71,6 +71,8 @@ int drm_mode_destroy_dumb_ioctl(struct drm_device *dev,
>                               void *data, struct drm_file *file_priv);
>  
>  /* drm_color_mgmt.c */
> +const char *drm_get_color_encoding_name(enum drm_color_encoding encoding);
> +const char *drm_get_color_range_name(enum drm_color_range range);
>  
>  /* IOCTLs */
>  int drm_mode_gamma_get_ioctl(struct drm_device *dev,
> -- 
> 2.13.6
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to