Skip to content

Commit acce1eb

Browse files
Daniel Saalexdeucher
Daniel Sa
authored andcommitted
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: Nevenko Stupar <[email protected]> Signed-off-by: Jerry Zuo <[email protected]> Signed-off-by: Daniel Sa <[email protected]> Tested-by: Daniel Wheeler <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent be7a6a5 commit acce1eb

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

drivers/gpu/drm/amd/display/dc/dpp/dcn401/dcn401_dpp_cm.c

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -246,16 +246,6 @@ void dpp401_set_cursor_matrix(
246246
enum dc_color_space color_space,
247247
struct dc_csc_transform cursor_csc_color_matrix)
248248
{
249-
struct dpp_input_csc_matrix cursor_tbl_entry;
250-
unsigned int i;
251-
252-
if (cursor_csc_color_matrix.enable_adjustment == true) {
253-
for (i = 0; i < 12; i++)
254-
cursor_tbl_entry.regval[i] = cursor_csc_color_matrix.matrix[i];
255-
256-
cursor_tbl_entry.color_space = color_space;
257-
dpp401_program_cursor_csc(dpp_base, color_space, &cursor_tbl_entry);
258-
} else {
259-
dpp401_program_cursor_csc(dpp_base, color_space, NULL);
260-
}
249+
//Since we don't have cursor matrix information, force bypass mode by passing in unknown color space
250+
dpp401_program_cursor_csc(dpp_base, COLOR_SPACE_UNKNOWN, NULL);
261251
}

0 commit comments

Comments
 (0)