Re: [PATCH v2] clk: imx: Refactor entire sccg pll clk

[Abel Vesa]

On 19-02-26 11:18:08, Aisheng Dong wrote:
> > From: Stephen Boyd [mailto:sboyd@xxxxxxxxxx]
> > Sent: Tuesday, February 26, 2019 1:23 AM> 
> > Quoting Abel Vesa (2019-02-23 02:58:14)
> > > On 19-02-22 09:46:23, Stephen Boyd wrote:
> > > > Quoting Abel Vesa (2019-02-22 09:07:32)
> > > > > Make the entire combination of plls to be one single clock. The
> > > > > parents used for bypasses are specified each as an index in the parents
> > list.
> > > > > The determine_rate does a lookup throughout all the possible
> > > > > combinations for all the divs and returns the best possible
> > > > > 'setup' which in turn is used by set_rate later to set up all the divs and
> > bypasses.
> > > > >
> > > > > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxx>
> 
> This significantly simply the clock tree output.
> Thanks for the work.
> 
> > > > > Tested-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>
> > > > > Acked-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>
> > > >
> > > > I suspect these tested by and acked tags should have been dropped,
> > > > unless you discussed and tested off-list?
> > > >
> > >
> > > Oups, I forgot to drop them.
> > 
> > Ok. Can Lucas re-test?
> 
> Tested on clk-next by cherry-pick a few defconfig patches from imx tree.
> So:
> Tested-by: Dong Aisheng <aisheng.dong@xxxxxxx>
> 

Thanks for helping with the testing.

I don't think there should be any defconfig changes required by this patch.

Can you please list here the CONFIGs added?

Just wanna make sure I'm not missing anything.

> Regards
> Dong Aisheng