With the unrelated line removed, 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