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.