diff options
| author | Mario Limonciello <mario.limonciello@amd.com> | 2023-10-06 13:50:26 -0500 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2023-10-13 11:01:30 -0400 |
| commit | 10406abe036b97a9c3549779a41c93ca488a0b7b (patch) | |
| tree | 73efa5108865b537e22fb6b839c15d50e4f6919e /drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | |
| parent | 06ad7e16425619a4a77154c2e85bededb3e04a4f (diff) | |
drm/amd/display: make dc_set_power_state() return type `void` again
As dc_set_power_state() no longer allocates memory, it's not necessary
to have return types and check return code as it can't fail anymore.
Change it back to `void`.
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c')
| -rw-r--r-- | drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 17 |
1 files changed, 5 insertions, 12 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 37d8c46ddfea..9121482d059e 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c @@ -2667,11 +2667,6 @@ static void hpd_rx_irq_work_suspend(struct amdgpu_display_manager *dm) } } -static int dm_set_power_state(struct dc *dc, enum dc_acpi_cm_power_state power_state) -{ - return dc_set_power_state(dc, power_state) ? 0 : -ENOMEM; -} - static int dm_suspend(void *handle) { struct amdgpu_device *adev = handle; @@ -2707,7 +2702,9 @@ static int dm_suspend(void *handle) hpd_rx_irq_work_suspend(dm); - return dm_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D3); + dc_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D3); + + return 0; } struct drm_connector * @@ -2901,9 +2898,7 @@ static int dm_resume(void *handle) if (r) DRM_ERROR("DMUB interface failed to initialize: status=%d\n", r); - r = dm_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D0); - if (r) - return r; + dc_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D0); dc_resume(dm->dc); @@ -2953,9 +2948,7 @@ static int dm_resume(void *handle) } /* power on hardware */ - r = dm_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D0); - if (r) - return r; + dc_set_power_state(dm->dc, DC_ACPI_CM_POWER_STATE_D0); /* program HPD filter */ dc_resume(dm->dc); |