Commit ffcda352 authored by Alex Deucher's avatar Alex Deucher

drm/radeon: set si_notify_smc_display_change properly

This is effectively a revert of 4573388c.

Forcing a display active when there is none causes problems with
dpm on some SI boards which results in improperly initialized
dpm state and boot failures on some boards.  As for the bug commit
4573388c tried to address, one can manually force the state to
high for better performance when using the card as a headless compute
node until a better fix is developed.

bugs:
https://bugs.freedesktop.org/show_bug.cgi?id=73788
https://bugs.freedesktop.org/show_bug.cgi?id=69395Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
cc: stable@vger.kernel.org
parent e9a321c6
...@@ -3590,10 +3590,9 @@ static void si_program_display_gap(struct radeon_device *rdev) ...@@ -3590,10 +3590,9 @@ static void si_program_display_gap(struct radeon_device *rdev)
/* Setting this to false forces the performance state to low if the crtcs are disabled. /* Setting this to false forces the performance state to low if the crtcs are disabled.
* This can be a problem on PowerXpress systems or if you want to use the card * This can be a problem on PowerXpress systems or if you want to use the card
* for offscreen rendering or compute if there are no crtcs enabled. Set it to * for offscreen rendering or compute if there are no crtcs enabled.
* true for now so that performance scales even if the displays are off.
*/ */
si_notify_smc_display_change(rdev, true /*rdev->pm.dpm.new_active_crtc_count > 0*/); si_notify_smc_display_change(rdev, rdev->pm.dpm.new_active_crtc_count > 0);
} }
static void si_enable_spread_spectrum(struct radeon_device *rdev, bool enable) static void si_enable_spread_spectrum(struct radeon_device *rdev, bool enable)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment