Thread (38 messages) 38 messages, 5 authors, 2012-11-08

Re: [PATCH 08/12] OMAPDSS: setup default dss fck

flat view

From: Archit Taneja <hidden>
Date: 2012-10-31 06:43:15
Also in: linux-omap

On Tuesday 30 October 2012 09:40 PM, Tomi Valkeinen wrote:
quoted hunk ↗ jump to hunk
We don't currently set the dss fck when starting up. This is not a
problem, as we setup the fck later when configuring the pixel clocks. Or
this is how it was for omap2, for the rest of the omaps this may not be
so.

For DSI, HDMI and also for DPI when using DSI PLL, we don't need to
change the dss fck, and thus it may be left unconfigured. Usually the
dss fck is already setup fine by default, but we can't trust this.

This patch sets the dss fck to maximum at probe time.

Signed-off-by: Tomi Valkeinen <redacted>
---
  drivers/video/omap2/dss/dss.c |   36 ++++++++++++++++++++++++++++++++++++
  1 file changed, 36 insertions(+)
diff --git a/drivers/video/omap2/dss/dss.c b/drivers/video/omap2/dss/dss.c
index 5affa86..034cc1a 100644
--- a/drivers/video/omap2/dss/dss.c
+++ b/drivers/video/omap2/dss/dss.c
@@ -485,6 +485,36 @@ unsigned long dss_get_dpll4_rate(void)
  		return 0;
  }

+static int dss_setup_default_clock(void)
+{
+	unsigned long max_dss_fck, prate;
+	unsigned fck_div;
+	struct dss_clock_info dss_cinfo = { 0 };
+	int r;
+
+	if (dss.dpll4_m4_ck = NULL)
+		return 0;
+
+	max_dss_fck = dss_feat_get_param_max(FEAT_PARAM_DSS_FCK);
+
+	prate = dss_get_dpll4_rate();
Not related to this patch, but maybe we could change the 
dss_get_dpll4_rate() name and dss.dpll4_m4_clk to something better. 
Maybe something like dss_fck_parent?
quoted hunk ↗ jump to hunk
+
+	fck_div = DIV_ROUND_UP(prate * dss.feat->dss_fck_multiplier,
+			max_dss_fck);
+
+	dss_cinfo.fck_div = fck_div;
+
+	r = dss_calc_clock_rates(&dss_cinfo);
+	if (r)
+		return r;
+
+	r = dss_set_clock_div(&dss_cinfo);
+	if (r)
+		return r;
+
+	return 0;
+}
+
  int dss_calc_clock_div(unsigned long req_pck, struct dss_clock_info *dss_cinfo,
  		struct dispc_clock_info *dispc_cinfo)
  {
@@ -913,6 +943,10 @@ static int __init omap_dsshw_probe(struct platform_device *pdev)
  	dss.lcd_clk_source[0] = OMAP_DSS_CLK_SRC_FCK;
  	dss.lcd_clk_source[1] = OMAP_DSS_CLK_SRC_FCK;

+	r = dss_setup_default_clock();
+	if (r)
+		goto err_setup_clocks;
Maybe it's safer to call this before we do a dss_runtime_get(). On 
OMAP4, DSS_FCLK is needed to access registers also. Changing it's rate 
might not be liked by the DSS HW. Also, it seems more logical to call it 
after dss_get_clocks() in omap_dsshw_probe(), then we sort of group the 
clock related stuff together.

Archit
quoted hunk ↗ jump to hunk
+
  	rev = dss_read_reg(DSS_REVISION);
  	printk(KERN_INFO "OMAP DSS rev %d.%d\n",
  			FLD_GET(rev, 7, 4), FLD_GET(rev, 3, 0));
@@ -923,6 +957,8 @@ static int __init omap_dsshw_probe(struct platform_device *pdev)

  	return 0;

+err_setup_clocks:
+	dss_runtime_put();
  err_runtime_get:
  	pm_runtime_disable(&pdev->dev);
  	dss_put_clocks();
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help