Re: [Intel-gfx] [PATCH] drm: Describe the Rotation property bits.

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Mon, Nov 02, 2015 at 03:24:15PM +0100, Robert Fekete wrote:
>     Adds clarification of the rotation property bits. I.e. rotation is
>     Counter clockwise and that reflects are applied before any rotation.
> 
> Signed-off-by: Robert Fekete <robert.fekete@linux.intel.com>
> ---
>  include/drm/drm_crtc.h | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> index 3f0c6909dda1..45334fa0cb10 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -85,7 +85,11 @@ static inline uint64_t I642U64(int64_t val)
>  	return (uint64_t)*((uint64_t *)&val);
>  }
>  
> -/* rotation property bits */
> +/*
> + * Rotation property bits. Rotate-(degrees)

the "rotate-" and "reflect-" things were supposed to be the names of the
property bits. I guess some quotes around them would have been good.

In any case, I tink here we should refer to the define names. eg
"DRM_ROTATE_<degrees> rotates the image by the ..."

> rotates the image by the specified
> + * amount in degrees in counter clockwise direction. reflect-x and reflect-y

same here with
"DRM_REFLECT_X and DRM_REFLECT_Y ..."

> + * reflects the image along the specified axis prior to rotation
> + */
>  #define DRM_ROTATE_MASK 0x0f
>  #define DRM_ROTATE_0	0
>  #define DRM_ROTATE_90	1
> -- 
> 1.9.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel