On Tue, Jun 13, 2017 at 2:00 AM, Bas Nieuwenhuizen <[email protected]> wrote: > With the unrelated line removed,
Same here. Reviewed-by: Marek Olšák <[email protected]> Marek > > 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 _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
