Re:Re: [PATCH 0/4] delete useless function return values & remove meaningless if(r) check code
Bernard
bernard at vivo.com
Mon Jun 21 01:33:41 UTC 2021
From: "Christian König" <christian.koenig at amd.com>
Date: 2021-06-21 00:59:27
To: Bernard Zhao <bernard at vivo.com>,Alex Deucher <alexander.deucher at amd.com>,David Airlie <airlied at linux.ie>,Daniel Vetter <daniel at ffwll.ch>,amd-gfx at lists.freedesktop.org,dri-devel at lists.freedesktop.org,linux-kernel at vger.kernel.org
Subject: Re: [PATCH 0/4] delete useless function return values & remove meaningless if(r) check code>Am 19.06.21 um 08:43 schrieb Bernard Zhao:
>> Function radeon_fence_driver_init always returns success,
>> the function type maybe coule be changed to void.
>> This patch series will first delete the check of the return
>> value of the function call radeon_fence_driver_init, then,
>> optimise the function declaration and function to void type.
>
>Please merge all of this into a single patch, I don't see any point
>separating this.
>
>Christian.
Hi
Sure, i will resubmit one new ptach, thanks!
BR//Bernard
>>
>> Signed-off-by: Bernard Zhao <bernard at vivo.com>
>>
>> Bernard Zhao (4):
>> drm/radeon: remove meaningless if(r) check code
>> drm/radeon: remove meaningless if(r) check code
>> drm/radeon: remove meaningless if(r) check code
>> drm/radeon: delete useless return values
>>
>> drivers/gpu/drm/radeon/cik.c | 4 +---
>> drivers/gpu/drm/radeon/evergreen.c | 4 +---
>> drivers/gpu/drm/radeon/ni.c | 4 +---
>> drivers/gpu/drm/radeon/r100.c | 4 +---
>> drivers/gpu/drm/radeon/r300.c | 4 +---
>> drivers/gpu/drm/radeon/r420.c | 5 +----
>> drivers/gpu/drm/radeon/r520.c | 4 +---
>> drivers/gpu/drm/radeon/r600.c | 4 +---
>> drivers/gpu/drm/radeon/radeon.h | 2 +-
>> drivers/gpu/drm/radeon/radeon_fence.c | 5 +----
>> drivers/gpu/drm/radeon/rs400.c | 4 +---
>> drivers/gpu/drm/radeon/rs600.c | 4 +---
>> drivers/gpu/drm/radeon/rs690.c | 4 +---
>> drivers/gpu/drm/radeon/rv515.c | 4 +---
>> drivers/gpu/drm/radeon/rv770.c | 4 +---
>> drivers/gpu/drm/radeon/si.c | 4 +---
>> 16 files changed, 16 insertions(+), 48 deletions(-)
>>
>
More information about the amd-gfx
mailing list