Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2abff001 authored by Adrian Salido-Moreno's avatar Adrian Salido-Moreno Committed by Chandan Uddaraju
Browse files

msm: mdss: dsi: fix configuration issues during resolution switch 



The below configuration settings are needed for panel
resolution switch to work properly:
* Use DSI FIFO for DCS switch commands.
* Reconfigure DSI stream ctrl with new resolution
* Use the correct structure for tearcheck configuration.

Change-Id: I65ca10c2adad21637f8414784986aa15026fd774
Signed-off-by: default avatarAdrian Salido-Moreno <adrianm@codeaurora.org>
Signed-off-by: default avatarChandan Uddaraju <chandanu@codeaurora.org>
parent 6a8f089c

drivers/video/msm/mdss/mdss_dsi.c

+1 −0
Original line number Diff line number Diff line
@@ -630,6 +630,7 @@ int mdss_dsi_switch_mode(struct mdss_panel_data *pdata, int mode) 
	}



	mdss_dsi_clk_ctrl(ctrl_pdata, DSI_ALL_CLKS, 1);

	mdss_dsi_ctrl_setup(ctrl_pdata);

	ctrl_pdata->switch_mode(pdata, mode);

	mdss_dsi_clk_ctrl(ctrl_pdata, DSI_ALL_CLKS, 0);

drivers/video/msm/mdss/mdss_dsi_panel.c

+38 −28
Original line number Diff line number Diff line
@@ -528,6 +528,7 @@ static void mdss_dsi_panel_switch_mode(struct mdss_panel_data *pdata, 
	struct mdss_dsi_ctrl_pdata *ctrl_pdata = NULL;

	struct mipi_panel_info *mipi;

	struct dsi_panel_cmds *pcmds;

	u32 flags = 0;



	if (pdata == NULL) {

		pr_err("%s: Invalid input data\n", __func__);
@@ -558,12 +559,13 @@ static void mdss_dsi_panel_switch_mode(struct mdss_panel_data *pdata, 


		pr_debug("%s: sending switch commands\n", __func__);

		pcmds = &pt->switch_cmds;

		flags |= CMD_REQ_DMA_TPG;

	} else {

		pr_warn("%s: Invalid mode switch attempted\n", __func__);

		return;

	}



	mdss_dsi_panel_cmds_send(ctrl_pdata, pcmds, 0);

	mdss_dsi_panel_cmds_send(ctrl_pdata, pcmds, flags);

}



static void mdss_dsi_panel_bl_ctrl(struct mdss_panel_data *pdata,
@@ -987,9 +989,9 @@ static int mdss_dsi_parse_fbc_params(struct device_node *np, 
}



static void mdss_panel_parse_te_params(struct device_node *np,

				       struct mdss_panel_info *panel_info)

				       struct mdss_panel_timing *timing)

{



	struct mdss_mdp_pp_tear_check *te = &timing->te;

	u32 tmp;

	int rc = 0;

	/*
@@ -998,27 +1000,27 @@ static void mdss_panel_parse_te_params(struct device_node *np, 
	 * vclk_line base on 60 fps; write is faster than read;

	 * init == start == rdptr;

	 */

	panel_info->te.tear_check_en =

	te->tear_check_en =

		!of_property_read_bool(np, "qcom,mdss-tear-check-disable");

	rc = of_property_read_u32

		(np, "qcom,mdss-tear-check-sync-cfg-height", &tmp);

	panel_info->te.sync_cfg_height = (!rc ? tmp : 0xfff0);

	te->sync_cfg_height = (!rc ? tmp : 0xfff0);

	rc = of_property_read_u32

		(np, "qcom,mdss-tear-check-sync-init-val", &tmp);

	panel_info->te.vsync_init_val = (!rc ? tmp : panel_info->yres);

	te->vsync_init_val = (!rc ? tmp : timing->yres);

	rc = of_property_read_u32

		(np, "qcom,mdss-tear-check-sync-threshold-start", &tmp);

	panel_info->te.sync_threshold_start = (!rc ? tmp : 4);

	te->sync_threshold_start = (!rc ? tmp : 4);

	rc = of_property_read_u32

		(np, "qcom,mdss-tear-check-sync-threshold-continue", &tmp);

	panel_info->te.sync_threshold_continue = (!rc ? tmp : 4);

	te->sync_threshold_continue = (!rc ? tmp : 4);

	rc = of_property_read_u32(np, "qcom,mdss-tear-check-start-pos", &tmp);

	panel_info->te.start_pos = (!rc ? tmp : panel_info->yres);

	te->start_pos = (!rc ? tmp : te->vsync_init_val);

	rc = of_property_read_u32

		(np, "qcom,mdss-tear-check-rd-ptr-trigger-intr", &tmp);

	panel_info->te.rd_ptr_irq = (!rc ? tmp : panel_info->yres + 1);

	te->rd_ptr_irq = (!rc ? tmp : te->vsync_init_val + 1);

	rc = of_property_read_u32(np, "qcom,mdss-tear-check-frame-rate", &tmp);

	panel_info->te.refx100 = (!rc ? tmp : 6000);

	te->refx100 = (!rc ? tmp : 6000);

}
@@ -1507,7 +1509,7 @@ int mdss_dsi_panel_timing_switch(struct mdss_dsi_ctrl_pdata *ctrl, 
	return 0;

}



static int __mdss_dsi_timing_from_dt(struct device_node *np,

static int mdss_dsi_panel_timing_from_dt(struct device_node *np,

	struct dsi_panel_timing *pt)

{

	u32 tmp;
@@ -1573,14 +1575,6 @@ static int __mdss_dsi_timing_from_dt(struct device_node *np, 
	rc = of_property_read_u32(np, "qcom,mdss-dsi-t-clk-post", &tmp);

	pt->t_clk_post = (!rc ? tmp : 0x03);



	mdss_dsi_parse_dcs_cmds(np, &pt->on_cmds,

		"qcom,mdss-dsi-on-command", "qcom,mdss-dsi-on-command-state");



	mdss_dsi_parse_dcs_cmds(np, &pt->switch_cmds,

		"qcom,mdss-dsi-timing-switch-command",

		"qcom,mdss-dsi-timing-switch-command-state");



	mdss_dsi_parse_fbc_params(np, &pt->timing.fbc);

	if (np->name) {

		pt->timing.name = kstrdup(np->name, GFP_KERNEL);

		pr_info("%s: found new timing \"%s\" (%p)\n", __func__,
@@ -1590,7 +1584,20 @@ static int __mdss_dsi_timing_from_dt(struct device_node *np, 
	return 0;

}



static int __mdss_panel_parse_display_timings(struct device_node *np,

static void  mdss_dsi_panel_config_res_properties(struct device_node *np,

		struct dsi_panel_timing *pt)

{

	mdss_dsi_parse_dcs_cmds(np, &pt->on_cmds,

			"qcom,mdss-dsi-on-command",

			"qcom,mdss-dsi-on-command-state");

	mdss_dsi_parse_dcs_cmds(np, &pt->switch_cmds,

			"qcom,mdss-dsi-timing-switch-command",

			"qcom,mdss-dsi-timing-switch-command-state");

	mdss_dsi_parse_fbc_params(np, &pt->timing.fbc);

	mdss_panel_parse_te_params(np, &pt->timing);

}



static int mdss_dsi_panel_parse_display_timings(struct device_node *np,

		struct mdss_panel_data *panel_data)

{

	struct mdss_dsi_ctrl_pdata *ctrl;
@@ -1613,10 +1620,11 @@ static int __mdss_panel_parse_display_timings(struct device_node *np, 
		 * timings directly from root node instead

		 */

		pr_debug("reading display-timings from panel node\n");

		rc = __mdss_dsi_timing_from_dt(np, &pt);

		if (!rc)

		rc = mdss_dsi_panel_timing_from_dt(np, &pt);

		if (!rc) {

			mdss_dsi_panel_config_res_properties(np, &pt);

			rc = mdss_dsi_panel_timing_switch(ctrl, &pt.timing);



		}

		return rc;

	}
@@ -1635,11 +1643,14 @@ static int __mdss_panel_parse_display_timings(struct device_node *np, 
	}



	for_each_child_of_node(timings_np, entry) {

		rc = __mdss_dsi_timing_from_dt(entry, modedb + i);

		rc = mdss_dsi_panel_timing_from_dt(entry, modedb + i);

		if (rc) {

			kfree(modedb);

			goto exit;

		}



		mdss_dsi_panel_config_res_properties(entry, (modedb + i));



		/* if default is set, use it otherwise use first as default */

		if (of_property_read_bool(entry,

				"qcom,mdss-dsi-timing-default"))
@@ -1670,7 +1681,8 @@ static int mdss_panel_parse_dt(struct device_node *np, 
	static const char *pdest;

	struct mdss_panel_info *pinfo = &(ctrl_pdata->panel_data.panel_info);



	rc = __mdss_panel_parse_display_timings(np, &ctrl_pdata->panel_data);

	rc = mdss_dsi_panel_parse_display_timings(np,

					&ctrl_pdata->panel_data);

	if (rc)

		return rc;

	rc = of_property_read_u32(np,
@@ -1915,8 +1927,6 @@ static int mdss_panel_parse_dt(struct device_node *np, 


	mdss_dsi_parse_lane_swap(np, &(pinfo->mipi.dlane_swap));



	mdss_panel_parse_te_params(np, pinfo);



	mdss_dsi_parse_reset_seq(np, pinfo->rst_seq, &(pinfo->rst_seq_len),

		"qcom,mdss-dsi-reset-sequence");