Bruno Jiménez <brunoji...@gmail.com> writes:

> From: Tom Stellard <thomas.stell...@amd.com>
>
> ---
>  src/gallium/include/pipe/p_defines.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/include/pipe/p_defines.h 
> b/src/gallium/include/pipe/p_defines.h
> index a3a1ae1..93f9642 100644
> --- a/src/gallium/include/pipe/p_defines.h
> +++ b/src/gallium/include/pipe/p_defines.h
> @@ -641,7 +641,8 @@ enum pipe_compute_cap
>     PIPE_COMPUTE_CAP_MAX_LOCAL_SIZE,
>     PIPE_COMPUTE_CAP_MAX_PRIVATE_SIZE,
>     PIPE_COMPUTE_CAP_MAX_INPUT_SIZE,
> -   PIPE_COMPUTE_CAP_MAX_MEM_ALLOC_SIZE
> +   PIPE_COMPUTE_CAP_MAX_MEM_ALLOC_SIZE,
> +   PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY,

You should also update the docs (src/gallium/docs/source/screen.rst)
with a short description of the new compute cap -- specifically, in
which unit is the clock frequency expressed.  Wouldn't it make more
sense to express it in MHz as it was in the original patch series so it
matches what OpenCL expects?

>  };
>  
>  /**
> -- 
> 1.9.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Attachment: pgpjf0PJa2kSr.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to