Commit 0b02e1fd authored by Joshua Aberback's avatar Joshua Aberback Committed by Alex Deucher

drm/amd/display: Compare mpcc_inst to mpcc_count instead of a constant

[Why]
This assert triggers a false negative because there are more than 4 MPCCs
on many asics.

[How]
 - change assert comparisson
 - remove unused variable
Signed-off-by: default avatarJoshua Aberback <joshua.aberback@amd.com>
Acked-by: default avatarAurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 958000cb
...@@ -220,15 +220,13 @@ static void dcn30_set_writeback( ...@@ -220,15 +220,13 @@ static void dcn30_set_writeback(
struct dc_writeback_info *wb_info, struct dc_writeback_info *wb_info,
struct dc_state *context) struct dc_state *context)
{ {
struct dwbc *dwb;
struct mcif_wb *mcif_wb; struct mcif_wb *mcif_wb;
struct mcif_buf_params *mcif_buf_params; struct mcif_buf_params *mcif_buf_params;
ASSERT(wb_info->dwb_pipe_inst < MAX_DWB_PIPES); ASSERT(wb_info->dwb_pipe_inst < MAX_DWB_PIPES);
ASSERT(wb_info->wb_enabled); ASSERT(wb_info->wb_enabled);
ASSERT(wb_info->mpcc_inst >= 0); ASSERT(wb_info->mpcc_inst >= 0);
ASSERT(wb_info->mpcc_inst < 4); ASSERT(wb_info->mpcc_inst < dc->res_pool->mpcc_count);
dwb = dc->res_pool->dwbc[wb_info->dwb_pipe_inst];
mcif_wb = dc->res_pool->mcif_wb[wb_info->dwb_pipe_inst]; mcif_wb = dc->res_pool->mcif_wb[wb_info->dwb_pipe_inst];
mcif_buf_params = &wb_info->mcif_buf_params; mcif_buf_params = &wb_info->mcif_buf_params;
......
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