> -----Original Message----- > From: YueHaibing <[email protected]> > Sent: Saturday, December 08, 2018 11:01 PM > To: Deucher, Alexander <[email protected]>; Koenig, Christian > <[email protected]>; Zhou, David(ChunMing) > <[email protected]>; [email protected]; Liu, Leo <[email protected]>; > Gao, Likun <[email protected]>; Panariti, David > <[email protected]>; S, Shirish <[email protected]>; Zhu, Rex > <[email protected]>; Grodzovsky, Andrey <[email protected]> > Cc: YueHaibing <[email protected]>; [email protected]; > [email protected]; [email protected]; kernel- > [email protected] > Subject: [PATCH -next] drm/amdgpu: remove set but not used variable > 'grbm_soft_reset' > > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c: In function > 'gfx_v8_0_pre_soft_reset': > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:4950:27: warning: > variable 'srbm_soft_reset' set but not used [-Wunused-but-set-variable] > > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c: In function > 'gfx_v8_0_post_soft_reset': > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:5054:27: warning: > variable 'srbm_soft_reset' set but not used [-Wunused-but-set-variable] > > It never used since introduction in commit d31a501ead7f ("drm/amdgpu: add > pre_soft_reset ip func") and e4ae0fc33631 ("drm/amdgpu: implement > gfx8 post_soft_reset") > > Signed-off-by: YueHaibing <[email protected]>
Reviewed-by: Chunming Zhou <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > index 1454fc3..8c1ba79 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > @@ -4947,14 +4947,13 @@ static bool gfx_v8_0_check_soft_reset(void > *handle) static int gfx_v8_0_pre_soft_reset(void *handle) { > struct amdgpu_device *adev = (struct amdgpu_device *)handle; > - u32 grbm_soft_reset = 0, srbm_soft_reset = 0; > + u32 grbm_soft_reset = 0; > > if ((!adev->gfx.grbm_soft_reset) && > (!adev->gfx.srbm_soft_reset)) > return 0; > > grbm_soft_reset = adev->gfx.grbm_soft_reset; > - srbm_soft_reset = adev->gfx.srbm_soft_reset; > > /* stop the rlc */ > adev->gfx.rlc.funcs->stop(adev); > @@ -5051,14 +5050,13 @@ static int gfx_v8_0_soft_reset(void *handle) > static int gfx_v8_0_post_soft_reset(void *handle) { > struct amdgpu_device *adev = (struct amdgpu_device *)handle; > - u32 grbm_soft_reset = 0, srbm_soft_reset = 0; > + u32 grbm_soft_reset = 0; > > if ((!adev->gfx.grbm_soft_reset) && > (!adev->gfx.srbm_soft_reset)) > return 0; > > grbm_soft_reset = adev->gfx.grbm_soft_reset; > - srbm_soft_reset = adev->gfx.srbm_soft_reset; > > if (REG_GET_FIELD(grbm_soft_reset, GRBM_SOFT_RESET, > SOFT_RESET_CP) || > REG_GET_FIELD(grbm_soft_reset, GRBM_SOFT_RESET, > SOFT_RESET_CPF) || > > _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
