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