Re: [PATCH 07/11] clk: imx6ul: fix pll clock parents

From: Dong Aisheng
Date: Sun Jun 12 2016 - 08:25:55 EST


On Sun, Jun 12, 2016 at 07:52:04PM +0800, Dong Aisheng wrote:
> On Sun, Jun 12, 2016 at 07:43:53PM +0800, Shawn Guo wrote:
> > On Wed, Jun 08, 2016 at 10:33:36PM +0800, Dong Aisheng wrote:
> > > pllx_bypass_src mux shouldn't be the parent of pllx clock
> > > since it's only valid when when pllx BYPASS bit is set.
> > > Thus it is actually one parent of pllx_bypass only.
> > >
> > > Instead, pllx parent should be fixed to osc according to
> > > reference manual.
> > > Other plls have the same issue.
> > >
> > > e.g. before fix, the pll tree is:
> > > osc 6 6 24000000 0 0
> > > pll1_bypass_src 0 0 24000000 0 0
> > > pll1 0 0 792000000 0 0
> > > pll1_bypass 0 0 792000000 0 0
> > > pll1_sys 0 0 792000000 0 0
> > >
> > > After the fix, it's:
> > > osc 6 6 24000000 0 0
> > > pll1 0 0 792000000 0 0
> > > pll1_bypass 0 0 792000000 0 0
> > > pll1_sys 0 0 792000000 0 0
> > >
> > > Signed-off-by: Dong Aisheng <aisheng.dong@xxxxxxx>
> >
> > I squashed 7 ~ 11 into one patch and applied it, thanks.
> >
>
> I'm fine.
> Thanks
>

You probably may need to change the patch title after merge.
clk: imx: fix pll clock parents

Regards
Dong Aisheng

> Regards
> Dong Aisheng
>
> > Shawn
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-clk" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html