From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cTNr0-0006uX-K6 for barebox@lists.infradead.org; Tue, 17 Jan 2017 07:06:52 +0000 Date: Tue, 17 Jan 2017 08:06:28 +0100 From: Sascha Hauer Message-ID: <20170117070628.e2kootn4e4bbbmul@pengutronix.de> References: <1484575497-2883-1-git-send-email-akurz@blala.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1484575497-2883-1-git-send-email-akurz@blala.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] ARM i.MX31 clk: add pll_ref_clk and MCU PLL bypass select To: Alexander Kurz Cc: barebox@lists.infradead.org Hi Alexander, On Mon, Jan 16, 2017 at 03:04:57PM +0100, Alexander Kurz wrote: > The three MX31 PLL may be clocked from either CKIH or a frequency-multiplied > derivate of CKIL generated by the Frequency Pre Multiplier FPM. > > Add the pll_ref_clk selection infrastructure and support for MCU PLL bypass > to support clock switching and boards not clocked CKIH. > > Signed-off-by: Alexander Kurz > --- > arch/arm/mach-imx/clk-imx31.c | 28 +++++++++++++++++++++++----- > 1 file changed, 23 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/mach-imx/clk-imx31.c b/arch/arm/mach-imx/clk-imx31.c The clock support is now under drivers/clk/imx/clk-imx31.c. Could you send an updated patch? Sascha > index 8d135c9..dbb5c15 100644 > --- a/arch/arm/mach-imx/clk-imx31.c > +++ b/arch/arm/mach-imx/clk-imx31.c > @@ -63,14 +63,27 @@ enum mx31_clks { > mstick1_gate, mstick2_gate, csi_gate, rtc_gate, wdog_gate, pwm_gate, > sim_gate, ect_gate, usb_gate, kpp_gate, ipu_gate, uart3_gate, > uart4_gate, uart5_gate, owire_gate, ssi2_gate, cspi1_gate, cspi2_gate, > - gacc_gate, emi_gate, rtic_gate, firi_gate, clk_max > + gacc_gate, emi_gate, rtic_gate, firi_gate, fpm, pll_ref, mpll_byp, > + clk_max > }; > > static struct clk *clks[clk_max]; > > +static const char *pll_ref_sel[] = { > + "dummy", > + "fpm", > + "ckih", > + "dummy", > +}; > + > +static const char *mpll_byp_sel[] = { > + "mpll", > + "pll_ref", > +}; > + > static const char *mcu_main_sel[] = { > "spll", > - "mpll", > + "mpll_byp", > }; > > static const char *per_sel[] = { > @@ -94,9 +107,14 @@ static int imx31_ccm_probe(struct device_d *dev) > > clks[ckih] = clk_fixed("ckih", 26000000); > clks[ckil] = clk_fixed("ckil", 32768); > - clks[mpll] = imx_clk_pllv1("mpll", "ckih", base + CCM_MPCTL); > - clks[spll] = imx_clk_pllv1("spll", "ckih", base + CCM_SRPCTL); > - clks[upll] = imx_clk_pllv1("upll", "ckih", base + CCM_UPCTL); > + clks[fpm] = imx_clk_fixed_factor("fpm", "ckil", 1024, 1); > + clks[pll_ref] = imx_clk_mux("pll_ref", base + CCM_CCMR, 1, 2, > + pll_ref_sel, ARRAY_SIZE(pll_ref_sel)); > + clks[mpll] = imx_clk_pllv1("mpll", "pll_ref", base + CCM_MPCTL); > + clks[spll] = imx_clk_pllv1("spll", "pll_ref", base + CCM_SRPCTL); > + clks[upll] = imx_clk_pllv1("upll", "pll_ref", base + CCM_UPCTL); > + clks[mpll_byp] = imx_clk_mux("mpll_byp", base + CCM_CCMR, 7, 1, > + mpll_byp_sel, ARRAY_SIZE(mpll_byp_sel)); > clks[mcu_main] = imx_clk_mux("mcu_main", base + CCM_PMCR0, 31, 1, > mcu_main_sel, ARRAY_SIZE(mcu_main_sel)); > clks[hsp] = imx_clk_divider("hsp", "mcu_main", base + CCM_PDR0, 11, 3); > -- > 2.1.4 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox