Re: [PATCH v2 2/4] clk: qcom: Add SDX55 GCC support

From: Stephen Boyd
Date: Wed Nov 18 2020 - 22:23:57 EST


Quoting Manivannan Sadhasivam (2020-11-18 06:44:01)
> On Sat, Nov 14, 2020 at 12:35:27PM -0800, Stephen Boyd wrote:
> > Quoting Vinod Koul (2020-11-06 01:38:19)
> > > On 05-11-20, 16:18, Manivannan Sadhasivam wrote:
> > > > +#include "reset.h"
> > > > +
> > > > +enum {
> > > > + P_BI_TCXO,
> > > > + P_CORE_BI_PLL_TEST_SE,
> > >
> > > This is for test and we removed this for upstream, so can you do that as
> > > well (not parent will decrease for clks below)
> > >
> > > With that updated:
> > >
> > > Reviewed-by: Vinod Koul <vkoul@xxxxxxxxxx>
> > >
> >
> > I half understood that. Just an FYI that I'm waiting for this update.
>
> Looks like this change is not needed. Can you please consider merging
> this series?
>

No. Please document core_bi_pll_test_se in the binding.