Commit cf388c0d authored by Arindam Nath's avatar Arindam Nath Committed by Alex Deucher

drm/amd/display: fix cursor disappearing after resume

Since during suspend, the cursor registers are cleared,
once the system resumes back, the cursor remains disabled.
cursor_set_attributes() only sets the cursor attributes
along with cursor size and surface address, but does not
enable the cursor back on.

We need to save the current cursor location so that
we can resume back to the same location. This is done
in dm_crtc_cursor_move(), where we save the current
cursor location into cursor_x and cursor_y. Later during
resume we use these same values to set the cursor
position along with cursor attributes.
Signed-off-by: default avatarArindam Nath <arindam.nath@amd.com>
Reviewed-by: default avatarHarry Wentland <harry.wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 2555039d
...@@ -78,6 +78,11 @@ static void dm_set_cursor( ...@@ -78,6 +78,11 @@ static void dm_set_cursor(
uint32_t height) uint32_t height)
{ {
struct dc_cursor_attributes attributes; struct dc_cursor_attributes attributes;
struct dc_cursor_position position;
struct drm_crtc *crtc = &amdgpu_crtc->base;
int x, y;
int xorigin = 0, yorigin = 0;
amdgpu_crtc->cursor_width = width; amdgpu_crtc->cursor_width = width;
amdgpu_crtc->cursor_height = height; amdgpu_crtc->cursor_height = height;
...@@ -91,11 +96,41 @@ static void dm_set_cursor( ...@@ -91,11 +96,41 @@ static void dm_set_cursor(
attributes.rotation_angle = 0; attributes.rotation_angle = 0;
attributes.attribute_flags.value = 0; attributes.attribute_flags.value = 0;
x = amdgpu_crtc->cursor_x;
y = amdgpu_crtc->cursor_y;
/* avivo cursor are offset into the total surface */
x += crtc->primary->state->src_x >> 16;
y += crtc->primary->state->src_y >> 16;
if (x < 0) {
xorigin = min(-x, amdgpu_crtc->max_cursor_width - 1);
x = 0;
}
if (y < 0) {
yorigin = min(-y, amdgpu_crtc->max_cursor_height - 1);
y = 0;
}
position.enable = true;
position.x = x;
position.y = y;
position.hot_spot_enable = true;
position.x_hotspot = xorigin;
position.y_hotspot = yorigin;
if (!dc_target_set_cursor_attributes( if (!dc_target_set_cursor_attributes(
amdgpu_crtc->target, amdgpu_crtc->target,
&attributes)) { &attributes)) {
DRM_ERROR("DC failed to set cursor attributes\n"); DRM_ERROR("DC failed to set cursor attributes\n");
} }
if (!dc_target_set_cursor_position(
amdgpu_crtc->target,
&position)) {
DRM_ERROR("DC failed to set cursor position\n");
}
} }
static int dm_crtc_unpin_cursor_bo_old( static int dm_crtc_unpin_cursor_bo_old(
...@@ -274,6 +309,9 @@ static int dm_crtc_cursor_move(struct drm_crtc *crtc, ...@@ -274,6 +309,9 @@ static int dm_crtc_cursor_move(struct drm_crtc *crtc,
int xorigin = 0, yorigin = 0; int xorigin = 0, yorigin = 0;
struct dc_cursor_position position; struct dc_cursor_position position;
amdgpu_crtc->cursor_x = x;
amdgpu_crtc->cursor_y = y;
/* avivo cursor are offset into the total surface */ /* avivo cursor are offset into the total surface */
x += crtc->primary->state->src_x >> 16; x += crtc->primary->state->src_x >> 16;
y += crtc->primary->state->src_y >> 16; y += crtc->primary->state->src_y >> 16;
......
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