Re: [PATCH v1 13/14] drm/xe/uapi: Move CPU_CACHING defines before doc

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Thu, Dec 07, 2023 at 01:50:08PM +0000, Francois Dugast wrote:
> Move those defines to align on the rule used elsewhere in the file which
> was introduced by commit ("drm/xe: Move defines before relevant fields").
> 
> Signed-off-by: Francois Dugast <francois.dugast@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  include/uapi/drm/xe_drm.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> index 6811a1c96078..7733c5a3659b 100644
> --- a/include/uapi/drm/xe_drm.h
> +++ b/include/uapi/drm/xe_drm.h
> @@ -741,12 +741,12 @@ struct drm_xe_gem_create {
>  	 */
>  	__u32 handle;
>  
> +#define DRM_XE_GEM_CPU_CACHING_WB                      1
> +#define DRM_XE_GEM_CPU_CACHING_WC                      2
>  	/**
>  	 * @cpu_caching: The CPU caching mode to select for this object. If
>  	 * mmaping the object the mode selected here will also be used.
>  	 */
> -#define DRM_XE_GEM_CPU_CACHING_WB                      1
> -#define DRM_XE_GEM_CPU_CACHING_WC                      2
>  	__u16 cpu_caching;
>  	/** @pad: MBZ */
>  	__u16 pad[3];
> -- 
> 2.34.1
>