Commit ceb75600 authored Aug 25, 2022 by Aurabindo Pillai Committed by Alex Deucher Sep 13, 2022

drm/amd/display: add workaround for subvp cursor corruption for DCN32/321



[Why&How]
Kernel does not have a means to tell the userspace to use software
cursor. Due to lack of this functionality, reducing the max cursor size
is the only way to ensure that power savings of Subview port feature is
utilized for asics that support it. The workaround could be removed
after cursor caching is fixed while a subviewport config is active.

Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Reviewed-by: Alvin Lee <Alvin.Lee2@amd.com>
Acked-by: Pavle Kotarac <Pavle.Kotarac@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

parent 1bb8df66

drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -2040,7 +2040,8 @@ static bool dcn32_resource_construct(
		dc->caps.max_downscale_ratio = 600;
		dc->caps.i2c_speed_in_khz = 100;
		dc->caps.i2c_speed_in_khz_hdcp = 100; /1.4 w/a applied by default/
		dc->caps.max_cursor_size = 256;
		/* TODO: Bring max_cursor_size back to 256 after subvp cursor corruption is fixed*/
		dc->caps.max_cursor_size = 64;
		dc->caps.min_horizontal_blanking_period = 80;
		dc->caps.dmdata_alloc_size = 2048;
		dc->caps.mall_size_per_mem_channel = 0;

drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -1652,7 +1652,8 @@ static bool dcn321_resource_construct(
		dc->caps.max_downscale_ratio = 600;
		dc->caps.i2c_speed_in_khz = 100;
		dc->caps.i2c_speed_in_khz_hdcp = 100; /1.4 w/a applied by default/
		dc->caps.max_cursor_size = 256;
		/* TODO: Bring max cursor size back to 256 after subvp cursor corruption is fixed*/
		dc->caps.max_cursor_size = 64;
		dc->caps.min_horizontal_blanking_period = 80;
		dc->caps.dmdata_alloc_size = 2048;
		dc->caps.mall_size_per_mem_channel = 0;