Commit acce1eb8 authored by Daniel Sa's avatar Daniel Sa Committed by Alex Deucher

drm/amd/display: Set Cursor Matrix to bypass instead of Input Plane

why:
When the cursor disappears/reappears on fullscreen video, there is a
short transitional period where the cursor's color matrix is using the
same format as the video plane. This sets the cursor to the wrong color
momentarily before the UI plane appears, correcting the color.

how:
Instead of defaulting to using the color space from the input plane,
default to bypass mode.
Reviewed-by: default avatarNevenko Stupar <nevenko.stupar@amd.com>
Signed-off-by: default avatarJerry Zuo <jerry.zuo@amd.com>
Signed-off-by: default avatarDaniel Sa <daniel.sa@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent be7a6a51
...@@ -246,16 +246,6 @@ void dpp401_set_cursor_matrix( ...@@ -246,16 +246,6 @@ void dpp401_set_cursor_matrix(
enum dc_color_space color_space, enum dc_color_space color_space,
struct dc_csc_transform cursor_csc_color_matrix) struct dc_csc_transform cursor_csc_color_matrix)
{ {
struct dpp_input_csc_matrix cursor_tbl_entry; //Since we don't have cursor matrix information, force bypass mode by passing in unknown color space
unsigned int i; dpp401_program_cursor_csc(dpp_base, COLOR_SPACE_UNKNOWN, NULL);
if (cursor_csc_color_matrix.enable_adjustment == true) {
for (i = 0; i < 12; i++)
cursor_tbl_entry.regval[i] = cursor_csc_color_matrix.matrix[i];
cursor_tbl_entry.color_space = color_space;
dpp401_program_cursor_csc(dpp_base, color_space, &cursor_tbl_entry);
} else {
dpp401_program_cursor_csc(dpp_base, color_space, NULL);
}
} }
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