Commit d65359d5 authored by Tony Cheng's avatar Tony Cheng Committed by Alex Deucher
Browse files

drm/amd/display: revert order change of HUBP and MPC disable



- root cause was we disable opp clk in MPC disconnect
- hubp_blank is not double buffered, so we can't blank until MPC disconnect or we have risk of underflow

Signed-off-by: default avatarTony Cheng <tony.cheng@amd.com>
Reviewed-by: default avatarEric Yang <eric.yang2@amd.com>
Acked-by: default avatarHarry Wentland <Harry.Wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 15e17335
Loading
Loading
Loading
Loading
+0 −2
Original line number Original line Diff line number Diff line
@@ -288,7 +288,6 @@ struct dce_hwseq_registers {
	HWS_SF1(OTG0_, PHYPLL_PIXEL_RATE_CNTL, PHYPLL_PIXEL_RATE_SOURCE, mask_sh), \
	HWS_SF1(OTG0_, PHYPLL_PIXEL_RATE_CNTL, PHYPLL_PIXEL_RATE_SOURCE, mask_sh), \
	HWS_SF(OTG0_, OTG_GLOBAL_SYNC_STATUS, VUPDATE_NO_LOCK_EVENT_CLEAR, mask_sh), \
	HWS_SF(OTG0_, OTG_GLOBAL_SYNC_STATUS, VUPDATE_NO_LOCK_EVENT_CLEAR, mask_sh), \
	HWS_SF(OTG0_, OTG_GLOBAL_SYNC_STATUS, VUPDATE_NO_LOCK_EVENT_OCCURRED, mask_sh), \
	HWS_SF(OTG0_, OTG_GLOBAL_SYNC_STATUS, VUPDATE_NO_LOCK_EVENT_OCCURRED, mask_sh), \
	HWS_SF(HUBP0_, DCHUBP_CNTL, HUBP_NO_OUTSTANDING_REQ, mask_sh), \
	HWS_SF(HUBP0_, DCHUBP_CNTL, HUBP_VTG_SEL, mask_sh), \
	HWS_SF(HUBP0_, DCHUBP_CNTL, HUBP_VTG_SEL, mask_sh), \
	HWS_SF(HUBP0_, HUBP_CLK_CNTL, HUBP_CLOCK_ENABLE, mask_sh), \
	HWS_SF(HUBP0_, HUBP_CLK_CNTL, HUBP_CLOCK_ENABLE, mask_sh), \
	HWS_SF(DPP_TOP0_, DPP_CONTROL, DPP_CLOCK_ENABLE, mask_sh), \
	HWS_SF(DPP_TOP0_, DPP_CONTROL, DPP_CLOCK_ENABLE, mask_sh), \
@@ -351,7 +350,6 @@ struct dce_hwseq_registers {
#define HWSEQ_DCN_REG_FIELD_LIST(type) \
#define HWSEQ_DCN_REG_FIELD_LIST(type) \
	type VUPDATE_NO_LOCK_EVENT_CLEAR; \
	type VUPDATE_NO_LOCK_EVENT_CLEAR; \
	type VUPDATE_NO_LOCK_EVENT_OCCURRED; \
	type VUPDATE_NO_LOCK_EVENT_OCCURRED; \
	type HUBP_NO_OUTSTANDING_REQ; \
	type HUBP_VTG_SEL; \
	type HUBP_VTG_SEL; \
	type HUBP_CLOCK_ENABLE; \
	type HUBP_CLOCK_ENABLE; \
	type DPP_CLOCK_ENABLE; \
	type DPP_CLOCK_ENABLE; \
+25 −16
Original line number Original line Diff line number Diff line
@@ -436,29 +436,17 @@ static void reset_back_end_for_pipe(
					pipe_ctx->pipe_idx, pipe_ctx->tg->inst);
					pipe_ctx->pipe_idx, pipe_ctx->tg->inst);
}
}


static void reset_front_end(
static void plane_atomic_stop(
		struct core_dc *dc,
		struct core_dc *dc,
		int fe_idx)
		int fe_idx)
{
{
	struct dce_hwseq *hws = dc->hwseq;
	struct mpcc_cfg mpcc_cfg;
	struct mpcc_cfg mpcc_cfg;
	struct mem_input *mi = dc->res_pool->mis[fe_idx];
	struct mem_input *mi = dc->res_pool->mis[fe_idx];
	struct transform *xfm = dc->res_pool->transforms[fe_idx];
	struct transform *xfm = dc->res_pool->transforms[fe_idx];
	struct mpcc *mpcc = dc->res_pool->mpcc[fe_idx];
	struct mpcc *mpcc = dc->res_pool->mpcc[fe_idx];
	struct timing_generator *tg = dc->res_pool->timing_generators[mpcc->opp_id];
	struct timing_generator *tg = dc->res_pool->timing_generators[mpcc->opp_id];
	unsigned int opp_id = mpcc->opp_id;

	/*Already reset*/
	if (opp_id == 0xf)
		return;

	tg->funcs->lock(tg);


	mi->funcs->dcc_control(mi, false, false);
	mi->funcs->dcc_control(mi, false, false);
	mi->funcs->set_blank(mi, true);
	REG_WAIT(DCHUBP_CNTL[fe_idx],
			HUBP_NO_OUTSTANDING_REQ, 1,
			1, 200);


	mpcc_cfg.opp_id = 0xf;
	mpcc_cfg.opp_id = 0xf;
	mpcc_cfg.top_dpp_id = 0xf;
	mpcc_cfg.top_dpp_id = 0xf;
@@ -466,23 +454,44 @@ static void reset_front_end(
	mpcc_cfg.top_of_tree = tg->inst == mpcc->inst;
	mpcc_cfg.top_of_tree = tg->inst == mpcc->inst;
	mpcc->funcs->set(mpcc, &mpcc_cfg);
	mpcc->funcs->set(mpcc, &mpcc_cfg);


	xfm->funcs->transform_reset(xfm);
}

static void reset_front_end(
		struct core_dc *dc,
		int fe_idx)
{
	struct dce_hwseq *hws = dc->hwseq;
	struct mem_input *mi = dc->res_pool->mis[fe_idx];
	struct mpcc *mpcc = dc->res_pool->mpcc[fe_idx];
	struct timing_generator *tg = dc->res_pool->timing_generators[mpcc->opp_id];
	unsigned int opp_id = mpcc->opp_id;

	/*Already reset*/
	if (opp_id == 0xf)
		return;

	tg->funcs->lock(tg);

	plane_atomic_stop(dc, fe_idx);

	REG_UPDATE(OTG_GLOBAL_SYNC_STATUS[tg->inst], VUPDATE_NO_LOCK_EVENT_CLEAR, 1);
	REG_UPDATE(OTG_GLOBAL_SYNC_STATUS[tg->inst], VUPDATE_NO_LOCK_EVENT_CLEAR, 1);
	tg->funcs->unlock(tg);
	tg->funcs->unlock(tg);
	REG_WAIT(OTG_GLOBAL_SYNC_STATUS[tg->inst], VUPDATE_NO_LOCK_EVENT_OCCURRED, 1, 20000, 200000);
	REG_WAIT(OTG_GLOBAL_SYNC_STATUS[tg->inst], VUPDATE_NO_LOCK_EVENT_OCCURRED, 1, 20000, 200000);


	mpcc->funcs->wait_for_idle(mpcc);
	mpcc->funcs->wait_for_idle(mpcc);


	mi->funcs->set_blank(mi, true);

	REG_UPDATE(HUBP_CLK_CNTL[fe_idx],
	REG_UPDATE(HUBP_CLK_CNTL[fe_idx],
			HUBP_CLOCK_ENABLE, 0);
			HUBP_CLOCK_ENABLE, 0);
	REG_UPDATE(DPP_CONTROL[fe_idx],
	REG_UPDATE(DPP_CONTROL[fe_idx],
			DPP_CLOCK_ENABLE, 0);
			DPP_CLOCK_ENABLE, 0);


	if (mpcc_cfg.top_of_tree)
	if (tg->inst == mpcc->inst)
		REG_UPDATE(OPP_PIPE_CONTROL[opp_id],
		REG_UPDATE(OPP_PIPE_CONTROL[opp_id],
				OPP_PIPE_CLOCK_EN, 0);
				OPP_PIPE_CLOCK_EN, 0);


	xfm->funcs->transform_reset(xfm);

	dm_logger_write(dc->ctx->logger, LOG_DC,
	dm_logger_write(dc->ctx->logger, LOG_DC,
					"Reset front end %d\n",
					"Reset front end %d\n",
					fe_idx);
					fe_idx);
+5 −0
Original line number Original line Diff line number Diff line
@@ -46,6 +46,11 @@ static void min10_set_blank(struct mem_input *mem_input, bool blank)
	REG_UPDATE_2(DCHUBP_CNTL,
	REG_UPDATE_2(DCHUBP_CNTL,
			HUBP_BLANK_EN, blank_en,
			HUBP_BLANK_EN, blank_en,
			HUBP_TTU_DISABLE, blank_en);
			HUBP_TTU_DISABLE, blank_en);

	if (blank)
		REG_WAIT(DCHUBP_CNTL,
				HUBP_NO_OUTSTANDING_REQ, 1,
				1, 200);
}
}


static void min10_vready_workaround(struct mem_input *mem_input,
static void min10_vready_workaround(struct mem_input *mem_input,
+2 −0
Original line number Original line Diff line number Diff line
@@ -304,6 +304,7 @@ struct dcn_mi_registers {
#define MI_MASK_SH_LIST_DCN(mask_sh)\
#define MI_MASK_SH_LIST_DCN(mask_sh)\
	MI_SF(HUBP0_DCHUBP_CNTL, HUBP_BLANK_EN, mask_sh),\
	MI_SF(HUBP0_DCHUBP_CNTL, HUBP_BLANK_EN, mask_sh),\
	MI_SF(HUBP0_DCHUBP_CNTL, HUBP_TTU_DISABLE, mask_sh),\
	MI_SF(HUBP0_DCHUBP_CNTL, HUBP_TTU_DISABLE, mask_sh),\
	MI_SF(HUBP0_DCHUBP_CNTL, HUBP_NO_OUTSTANDING_REQ, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, NUM_PIPES, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, NUM_PIPES, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, NUM_BANKS, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, NUM_BANKS, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, PIPE_INTERLEAVE, mask_sh),\
	MI_SF(HUBP0_DCSURF_ADDR_CONFIG, PIPE_INTERLEAVE, mask_sh),\
@@ -463,6 +464,7 @@ struct dcn_mi_registers {
#define DCN_MI_REG_FIELD_LIST(type) \
#define DCN_MI_REG_FIELD_LIST(type) \
	type HUBP_BLANK_EN;\
	type HUBP_BLANK_EN;\
	type HUBP_TTU_DISABLE;\
	type HUBP_TTU_DISABLE;\
	type HUBP_NO_OUTSTANDING_REQ;\
	type NUM_PIPES;\
	type NUM_PIPES;\
	type NUM_BANKS;\
	type NUM_BANKS;\
	type PIPE_INTERLEAVE;\
	type PIPE_INTERLEAVE;\