With the unrelated line removed, Reviewed-by: Bas Nieuwenhuizen <[email protected]>
On Tue, Jun 13, 2017 at 1:54 AM, Dave Airlie <[email protected]> wrote: > On 13 June 2017 at 09:53, Dave Airlie <[email protected]> wrote: >> From: Dave Airlie <[email protected]> >> >> has_hw_decode is assigned twice. >> >> Pointed out by coverity. >> >> Signed-off-by: Dave Airlie <[email protected]> >> --- >> 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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
