[Mesa-dev] [PATCH 1/6] gallium\auxiliary\vl: Move dirty define to header file

[Christian König]

Am 01.02.19 um 17:28 schrieb Zhu, James:
> Move dirty define to header file to share with compute shader.
>
> Signed-off-by: James Zhu <James.Zhu at amd.com>
> ---
>   src/gallium/auxiliary/vl/vl_compositor.c | 3 ---
>   src/gallium/auxiliary/vl/vl_compositor.h | 2 ++
>   2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/auxiliary/vl/vl_compositor.c b/src/gallium/auxiliary/vl/vl_compositor.c
> index 159a295..2c6d585 100644
> --- a/src/gallium/auxiliary/vl/vl_compositor.c
> +++ b/src/gallium/auxiliary/vl/vl_compositor.c
> @@ -42,9 +42,6 @@
>   #include "vl_types.h"
>   #include "vl_compositor.h"
>   
> -#define MIN_DIRTY (0)
> -#define MAX_DIRTY (1 << 15)
> -
>   enum VS_OUTPUT
>   {
>      VS_O_VPOS = 0,
> diff --git a/src/gallium/auxiliary/vl/vl_compositor.h b/src/gallium/auxiliary/vl/vl_compositor.h
> index 8819176..d51b5f5 100644
> --- a/src/gallium/auxiliary/vl/vl_compositor.h
> +++ b/src/gallium/auxiliary/vl/vl_compositor.h
> @@ -44,6 +44,8 @@ struct pipe_context;
>    */
>   
>   #define VL_COMPOSITOR_MAX_LAYERS 16
> +#define MIN_DIRTY (0)
> +#define MAX_DIRTY (1 << 15)

That needs a proper prefix.

E.g. put VL_COMPOSITOR_ in front of the name and rename all usages.

Christian.

>   
>   /* deinterlace allgorithem */
>   enum vl_compositor_deinterlace