From 8530c41d7c2686eb5c509111fc677f13793a6288 Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Mon, 15 Sep 2014 15:48:39 +0300 Subject: [PATCH] OMAPDSS: HDMI: fix setting REFSEL Only OMAP5+ has REFSEL field, but at the moment it's set also on OMAP4. Fix this by adding a "has_refsel" field, and setting the REFSEL based on that. Signed-off-by: Tomi Valkeinen --- drivers/video/fbdev/omap2/dss/hdmi.h | 8 -------- drivers/video/fbdev/omap2/dss/hdmi_pll.c | 8 ++++---- 2 files changed, 4 insertions(+), 12 deletions(-) diff --git a/drivers/video/fbdev/omap2/dss/hdmi.h b/drivers/video/fbdev/omap2/dss/hdmi.h index 262771b9b76b..4bbc9d206f4a 100644 --- a/drivers/video/fbdev/omap2/dss/hdmi.h +++ b/drivers/video/fbdev/omap2/dss/hdmi.h @@ -101,13 +101,6 @@ enum hdmi_core_hdmi_dvi { HDMI_HDMI = 1 }; -enum hdmi_clk_refsel { - HDMI_REFSEL_PCLK = 0, - HDMI_REFSEL_REF1 = 1, - HDMI_REFSEL_REF2 = 2, - HDMI_REFSEL_SYSCLK = 3 -}; - enum hdmi_packing_mode { HDMI_PACK_10b_RGB_YUV444 = 0, HDMI_PACK_24b_RGB_YUV444_YUV422 = 1, @@ -199,7 +192,6 @@ struct hdmi_pll_info { u16 regm2; u16 regsd; u16 dcofreq; - enum hdmi_clk_refsel refsel; }; struct hdmi_audio_format { diff --git a/drivers/video/fbdev/omap2/dss/hdmi_pll.c b/drivers/video/fbdev/omap2/dss/hdmi_pll.c index 6d92bb32fe51..b28d41a08a8f 100644 --- a/drivers/video/fbdev/omap2/dss/hdmi_pll.c +++ b/drivers/video/fbdev/omap2/dss/hdmi_pll.c @@ -24,6 +24,7 @@ #define HDMI_DEFAULT_REGM2 1 struct hdmi_pll_features { + bool has_refsel; bool sys_reset; /* this is a hack, need to replace it with a better computation of M2 */ bool bound_dcofreq; @@ -96,9 +97,6 @@ void hdmi_pll_compute(struct hdmi_pll_data *pll, unsigned long clkin, int phy) pi->dcofreq = phy > 1000 * 100; pi->regsd = ((pi->regm * clkin / 10) / (pi->regn * 250) + 5) / 10; - /* Set the reference clock to sysclk reference */ - pi->refsel = HDMI_REFSEL_SYSCLK; - DSSDBG("M = %d Mf = %d\n", pi->regm, pi->regmf); DSSDBG("range = %d sd = %d\n", pi->dcofreq, pi->regsd); } @@ -122,7 +120,8 @@ static int hdmi_pll_config(struct hdmi_pll_data *pll) r = FLD_MOD(r, 0x0, 12, 12); /* PLL_HIGHFREQ divide by 2 */ r = FLD_MOD(r, 0x1, 13, 13); /* PLL_REFEN */ r = FLD_MOD(r, 0x0, 14, 14); /* PHY_CLKINEN de-assert during locking */ - r = FLD_MOD(r, fmt->refsel, 22, 21); /* REFSEL */ + if (pll_feat->has_refsel) + r = FLD_MOD(r, 0x3, 22, 21); /* REFSEL = SYSCLK */ if (fmt->dcofreq) r = FLD_MOD(r, 0x4, 3, 1); /* 1000MHz and 2000MHz */ @@ -222,6 +221,7 @@ static const struct hdmi_pll_features omap44xx_pll_feats = { }; static const struct hdmi_pll_features omap54xx_pll_feats = { + .has_refsel = true, .sys_reset = true, .bound_dcofreq = true, .fint_min = 620000,