drm/amdgpu/gfx10: drop unused variable

alexdeucher · alexdeucher · commit d78e816a3d61 · 2023-06-09T09:28:50.000-04:00
Just check the return value directly.

Reviewed-by: Christian König &lt;christian.koenig@amd.com&gt;
Signed-off-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
@@ -7159,15 +7159,13 @@ static int gfx_v10_0_hw_init(void *handle)
 static int gfx_v10_0_hw_fini(void *handle)
 {
 	struct amdgpu_device *adev = (struct amdgpu_device *)handle;
-	int r;
 
 	amdgpu_irq_put(adev, &adev->gfx.priv_reg_irq, 0);
 	amdgpu_irq_put(adev, &adev->gfx.priv_inst_irq, 0);
 
 	if (!adev->no_hw_access) {
 		if (amdgpu_async_gfx_ring) {
-			r = amdgpu_gfx_disable_kgq(adev, 0);
-			if (r)
+			if (amdgpu_gfx_disable_kgq(adev, 0))
 				DRM_ERROR("KGQ disable failed\n");
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -7159,15 +7159,13 @@ static int gfx_v10_0_hw_init(void *handle)`
`7159`	`7159`	`static int gfx_v10_0_hw_fini(void *handle)`
`7160`	`7160`	`{`
`7161`	`7161`	`struct amdgpu_device adev = (struct amdgpu_device )handle;`
`7162`		`- int r;`
`7163`	`7162`
`7164`	`7163`	`amdgpu_irq_put(adev, &adev->gfx.priv_reg_irq, 0);`
`7165`	`7164`	`amdgpu_irq_put(adev, &adev->gfx.priv_inst_irq, 0);`
`7166`	`7165`
`7167`	`7166`	`if (!adev->no_hw_access) {`
`7168`	`7167`	`if (amdgpu_async_gfx_ring) {`
`7169`		`- r = amdgpu_gfx_disable_kgq(adev, 0);`
`7170`		`- if (r)`
	`7168`	`+ if (amdgpu_gfx_disable_kgq(adev, 0))`
`7171`	`7169`	`DRM_ERROR("KGQ disable failed\n");`
`7172`	`7170`	`}`
`7173`	`7171`