drm/amd/amdgpu: Cleanup gmc_v9_0_suspend()
authorTom St Denis <tom.stdenis@amd.com>
Fri, 1 Sep 2017 13:55:04 +0000 (09:55 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 1 Sep 2017 16:51:24 +0000 (12:51 -0400)
Even though fini returns 0 always it could theoretically
fail in the future.  Might as well return it instead of 0.

Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c

index 0766dad1fafeef8b9f6caf7703597db5ed2f54ef..7ca9cbec3004f646e66cc2f6d957c96731bc57c6 100644 (file)
@@ -795,9 +795,7 @@ static int gmc_v9_0_suspend(void *handle)
 {
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
-       gmc_v9_0_hw_fini(adev);
-
-       return 0;
+       return gmc_v9_0_hw_fini(adev);
 }
 
 static int gmc_v9_0_resume(void *handle)