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

2017-06-13 Thread Marek Olšák
On Tue, Jun 13, 2017 at 2:00 AM, Bas Nieuwenhuizen wrote: > With the unrelated line removed, Same here. Reviewed-by: Marek Olšák Marek > > Reviewed-by: Bas Nieuwenhuizen > > On Tue, Jun 13, 2017 at 1:54 AM, Dave Airlie wrote: >> On 13 June 2017 at 09:53, Dave Airlie wrote: >>> From: Dave

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

2017-06-12 Thread Bas Nieuwenhuizen
With the unrelated line removed, Reviewed-by: Bas Nieuwenhuizen On Tue, Jun 13, 2017 at 1:54 AM, Dave Airlie wrote: > On 13 June 2017 at 09:53, Dave Airlie wrote: >> From: Dave Airlie >> >> has_hw_decode is assigned twice. >> >> Pointed out by coverity. >> >> Signed-off-by: Dave Airlie >> --

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

2017-06-12 Thread Dave Airlie
On 13 June 2017 at 09:53, Dave Airlie wrote: > From: Dave Airlie > > has_hw_decode is assigned twice. > > Pointed out by coverity. > > Signed-off-by: Dave Airlie > --- > src/amd/common/ac_gpu_info.c | 1 - > src/amd/vulkan/radv_cmd_buffer.c | 2 +- > 2 files changed, 1 insertion(+), 2 delet

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

2017-06-12 Thread Ilia Mirkin
On Mon, Jun 12, 2017 at 7:53 PM, Dave Airlie wrote: > From: Dave Airlie > > has_hw_decode is assigned twice. > > Pointed out by coverity. > > Signed-off-by: Dave Airlie > --- > src/amd/common/ac_gpu_info.c | 1 - > src/amd/vulkan/radv_cmd_buffer.c | 2 +- > 2 files changed, 1 insertion(+),

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

2017-06-12 Thread Dave Airlie
From: Dave Airlie has_hw_decode is assigned twice. Pointed out by coverity. Signed-off-by: Dave Airlie --- 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/comm