From: Mario Limonciello <mario.limoncie...@amd.com>

The `ret` variable in dm_suspend() doesn't get set and is just used
to return 0.  Drop the needless declaration.

Signed-off-by: Mario Limonciello <mario.limoncie...@amd.com>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index b26ae1dd1fd78..0d78cabeece70 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -3087,7 +3087,6 @@ static int dm_suspend(struct amdgpu_ip_block *ip_block)
 {
        struct amdgpu_device *adev = ip_block->adev;
        struct amdgpu_display_manager *dm = &adev->dm;
-       int ret = 0;
 
        if (amdgpu_in_reset(adev)) {
                mutex_lock(&dm->dc_lock);
@@ -3105,7 +3104,7 @@ static int dm_suspend(struct amdgpu_ip_block *ip_block)
 
                hpd_rx_irq_work_suspend(dm);
 
-               return ret;
+               return 0;
        }
 
        WARN_ON(adev->dm.cached_state);
-- 
2.43.0

Reply via email to