On Tue, Jun 13, 2017 at 2:00 AM, Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> wrote: > With the unrelated line removed,
Same here. Reviewed-by: Marek Olšák <marek.ol...@amd.com> Marek > > Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> > > On Tue, Jun 13, 2017 at 1:54 AM, Dave Airlie <airl...@gmail.com> wrote: >> On 13 June 2017 at 09:53, Dave Airlie <airl...@gmail.com> wrote: >>> From: Dave Airlie <airl...@redhat.com> >>> >>> has_hw_decode is assigned twice. >>> >>> Pointed out by coverity. >>> >>> Signed-off-by: Dave Airlie <airl...@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. >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/mesa-dev > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev