| Message ID | 1336971887-24616-1-git-send-email-richard.zhao@freescale.com |
|---|---|
| State | New |
| Headers | show |
On Mon, May 14, 2012 at 01:04:47PM +0800, Richard Zhao wrote: > This also makes any clk be possible init on. > > Signed-off-by: Richard Zhao <richard.zhao@freescale.com> This is something I forgot to fix, thanks for doing this. > > +static enum mx6q_clks const clks_init_on[] __initconst = { > + mmdc_ch0_axi, mmdc_ch1_axi, usboh3, > +}; BTW, why usboh3? Do we need this? Right now we do not have USB support on i.MX6. Once we have it the clock should be turned on in the USB driver, right? Sascha
On Mon, May 14, 2012 at 09:29:34AM +0200, Buddhi wrote: > On Mon, May 14, 2012 at 01:04:47PM +0800, Richard Zhao wrote: > > This also makes any clk be possible init on. > > > > Signed-off-by: Richard Zhao <richard.zhao@freescale.com> > > This is something I forgot to fix, thanks for doing this. > > > > > +static enum mx6q_clks const clks_init_on[] __initconst = { > > + mmdc_ch0_axi, mmdc_ch1_axi, usboh3, > > +}; > > BTW, why usboh3? Do we need this? Right now we do not have USB support > on i.MX6. Once we have it the clock should be turned on in the USB > driver, right? I don't know why either. I just convert it. Thanks Richard > > Sascha > > -- > 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 | >
On Mon, May 14, 2012 at 04:18:26PM +0800, Shawn Guo wrote: > Sascha, > > On Mon, May 14, 2012 at 09:29:34AM +0200, Buddhi wrote: > > > +static enum mx6q_clks const clks_init_on[] __initconst = { > > > + mmdc_ch0_axi, mmdc_ch1_axi, usboh3, > > > +}; > > > > BTW, why usboh3? Do we need this? Right now we do not have USB support > > on i.MX6. Once we have it the clock should be turned on in the USB > > driver, right? > > > I remember that I have no usboh3 in clks_init_on when I handed the patch > over to you. Oops, then this probably leaked in because we have USB support hacked into an internal branch. I'll prepare a patch to remove it. Sascha
Sascha, On Mon, May 14, 2012 at 09:29:34AM +0200, Buddhi wrote: > > +static enum mx6q_clks const clks_init_on[] __initconst = { > > + mmdc_ch0_axi, mmdc_ch1_axi, usboh3, > > +}; > > BTW, why usboh3? Do we need this? Right now we do not have USB support > on i.MX6. Once we have it the clock should be turned on in the USB > driver, right? > I remember that I have no usboh3 in clks_init_on when I handed the patch over to you.
diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c index 1d83d03..f7eb965 100644 --- a/arch/arm/mach-imx/clk-imx6q.c +++ b/arch/arm/mach-imx/clk-imx6q.c @@ -122,10 +122,6 @@ static const char *cko1_sels[] = { "pll3_usb_otg", "pll2_bus", "pll1_sys", "pll5 "dummy", "axi", "enfc", "ipu1_di0", "ipu1_di1", "ipu2_di0", "ipu2_di1", "ahb", "ipg", "ipg_per", "ckil", "pll4_audio", }; -static const char * const clks_init_on[] __initconst = { - "mmdc_ch0_axi", "mmdc_ch1_axi", "usboh3", -}; - enum mx6q_clks { dummy, ckil, ckih, osc, pll2_pfd0_352m, pll2_pfd1_594m, pll2_pfd2_396m, pll3_pfd0_720m, pll3_pfd1_540m, pll3_pfd2_508m, pll3_pfd3_454m, @@ -160,11 +156,14 @@ enum mx6q_clks { static struct clk *clk[clk_max]; +static enum mx6q_clks const clks_init_on[] __initconst = { + mmdc_ch0_axi, mmdc_ch1_axi, usboh3, +}; + int __init mx6q_clocks_init(void) { struct device_node *np; void __iomem *base; - struct clk *c; int i, irq; clk[dummy] = imx_clk_fixed("dummy", 0); @@ -424,15 +423,8 @@ int __init mx6q_clocks_init(void) clk_register_clkdev(clk[dummy], NULL, "20bc000.wdog"); clk_register_clkdev(clk[dummy], NULL, "20c0000.wdog"); - for (i = 0; i < ARRAY_SIZE(clks_init_on); i++) { - c = clk_get_sys(clks_init_on[i], NULL); - if (IS_ERR(c)) { - pr_err("%s: failed to get clk %s", __func__, - clks_init_on[i]); - return PTR_ERR(c); - } - clk_prepare_enable(c); - } + for (i = 0; i < ARRAY_SIZE(clks_init_on); i++) + clk_prepare_enable(clk[clks_init_on[i]]); np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-gpt"); base = of_iomap(np, 0);
This also makes any clk be possible init on. Signed-off-by: Richard Zhao <richard.zhao@freescale.com> --- arch/arm/mach-imx/clk-imx6q.c | 20 ++++++-------------- 1 files changed, 6 insertions(+), 14 deletions(-)