diff options
| author |   Srinivasan Shanmugam <srinivasan.shanmugam@amd.com> | 2023-06-30 21:02:50 +0530 |
|---|---|---|
| committer |   Alex Deucher <alexander.deucher@amd.com> | 2023-07-10 09:02:37 -0400 |
| commit | b8f68f1da50e1e117dff704fa55602f999539598 (patch) | |
| tree | 6c8702264b063740e7530bcd3e1c96c8b568e4a0 | |
| parent | drm/amdgpu: Fix warnings in gfxhub_v2_1.c (diff) | |
| download | linux-rng-b8f68f1da50e1e117dff704fa55602f999539598.tar.xz linux-rng-b8f68f1da50e1e117dff704fa55602f999539598.zip | |
drm/amdgpu: Remove else after return statement in 'gmc_v8_0_check_soft_reset'
Fix below checkpatch warnings:
WARNING: else is not generally useful after a break or return
+ return true;
+ } else {
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c index 581ed922dbe3..21cb97bb6776 100644 --- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c @@ -1319,13 +1319,15 @@ static bool gmc_v8_0_check_soft_reset(void *handle) srbm_soft_reset = REG_SET_FIELD(srbm_soft_reset, SRBM_SOFT_RESET, SOFT_RESET_MC, 1); } + if (srbm_soft_reset) { adev->gmc.srbm_soft_reset = srbm_soft_reset; return true; - } else { - adev->gmc.srbm_soft_reset = 0; - return false; } + + adev->gmc.srbm_soft_reset = 0; + + return false; } static int gmc_v8_0_pre_soft_reset(void *handle) |