Skip to content

Commit 8e7cedc

Browse files
committed
drm/amdgpu/pm: adjust display configuration after powerstate
set_power_state defaults to no displays, so we need to update the display configuration after setting up the powerstate on the first call. In most cases this is not an issue since ends up getting called multiple times at any given modeset and the proper order is achieved in the display changed handling at the top of the function. Reviewed-by: Christian König <christian.koenig@amd.com> Acked-by: Jordan Lazare <Jordan.Lazare@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Cc: stable@vger.kernel.org
1 parent 0c67df4 commit 8e7cedc

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -648,8 +648,6 @@ static void amdgpu_dpm_change_power_state_locked(struct amdgpu_device *adev)
648648

649649
/* update display watermarks based on new power state */
650650
amdgpu_display_bandwidth_update(adev);
651-
/* update displays */
652-
amdgpu_dpm_display_configuration_changed(adev);
653651

654652
adev->pm.dpm.current_active_crtcs = adev->pm.dpm.new_active_crtcs;
655653
adev->pm.dpm.current_active_crtc_count = adev->pm.dpm.new_active_crtc_count;
@@ -669,6 +667,9 @@ static void amdgpu_dpm_change_power_state_locked(struct amdgpu_device *adev)
669667

670668
amdgpu_dpm_post_set_power_state(adev);
671669

670+
/* update displays */
671+
amdgpu_dpm_display_configuration_changed(adev);
672+
672673
if (adev->pm.funcs->force_performance_level) {
673674
if (adev->pm.dpm.thermal_active) {
674675
enum amdgpu_dpm_forced_level level = adev->pm.dpm.forced_level;

0 commit comments

Comments
 (0)