[PATCH] drm/amdgpu/gfx8: fix bugon.cocci warnings

Alex Deucher alexdeucher at gmail.com
Thu Jan 12 14:39:57 UTC 2017


On Tue, Jan 10, 2017 at 4:38 AM, Yu, Xiangliang <Xiangliang.Yu at amd.com> wrote:
>>  Use BUG_ON instead of a if condition followed by BUG.
>>
>> Generated by: scripts/coccinelle/misc/bugon.cocci
>>
>> CC: Xiangliang Yu <Xiangliang.Yu at amd.com>
>> Signed-off-by: Julia Lawall <julia.lawall at lip6.fr>
>> Signed-off-by: Fengguang Wu <fengguang.wu at intel.com>

Applied.  thanks!

Alex

>> ---
>>
>>  gfx_v8_0.c |    3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>> @@ -6623,8 +6623,7 @@ static void gfx_v8_0_ring_emit_fence_kiq
>>                                        u64 seq, unsigned int flags)
>>  {
>>       /* we only allocate 32bit for each seq wb address */
>> -     if (flags & AMDGPU_FENCE_FLAG_64BIT)
>> -             BUG();
>> +     BUG_ON(flags & AMDGPU_FENCE_FLAG_64BIT);
>>
>>       /* write fence seq to the "addr" */
>>       amdgpu_ring_write(ring, PACKET3(PACKET3_WRITE_DATA, 3));
>
> Reviewd-by: Xiangliang.Yu <Xiangliang.Yu at amd.com>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list