[Mesa-dev] [PATCH 2/2] ac/gpu: drop duplicated code line.

Dave Airlie airlied at gmail.com
Mon Jun 12 23:54:40 UTC 2017


On 13 June 2017 at 09:53, Dave Airlie <airlied at gmail.com> wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> has_hw_decode is assigned twice.
>
> Pointed out by coverity.
>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
>  src/amd/common/ac_gpu_info.c     | 1 -
>  src/amd/vulkan/radv_cmd_buffer.c | 2 +-
>  2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c
> index dc3ae25..7d6f6e7 100644
> --- a/src/amd/common/ac_gpu_info.c
> +++ b/src/amd/common/ac_gpu_info.c
> @@ -249,7 +249,6 @@ bool ac_query_gpu_info(int fd, amdgpu_device_handle dev,
>         info->max_shader_clock = amdinfo->max_engine_clk / 1000;
>         info->max_se = amdinfo->num_shader_engines;
>         info->max_sh_per_se = amdinfo->num_shader_arrays_per_engine;
> -       info->has_hw_decode = uvd.available_rings != 0;
>         info->has_hw_decode =
>                 (uvd.available_rings != 0) || (vcn_dec.available_rings != 0);
>         info->uvd_fw_version =


Ooops ignore hunk below.

Dave.


More information about the mesa-dev mailing list