On 10-06-19, 08:08, Stephen Boyd wrote: > Quoting Vinod Koul (2019-06-08 02:15:37) > > On 07-06-19, 10:43, Stephen Boyd wrote: > > > Quoting Vinod Koul (2019-06-07 03:12:34) > > > > diff --git a/drivers/clk/qcom/gcc-sm8150.c b/drivers/clk/qcom/gcc-sm8150.c > > > > new file mode 100644 > > > > index 000000000000..1cbc884444c9 > > > > --- /dev/null > > > > +++ b/drivers/clk/qcom/gcc-sm8150.c > > > > +static const struct parent_map gcc_parent_map_0[] = { > > > > + { P_BI_TCXO, 0 }, > > > > + { P_GPLL0_OUT_MAIN, 1 }, > > > > + { P_GPLL0_OUT_EVEN, 6 }, > > > > + { P_CORE_BI_PLL_TEST_SE, 7 }, > > > > +}; > > > > + > > > > +static const char * const gcc_parent_names_0[] = { > > > > > > We have a new way of specifying clk parents now. Can you use that > > > instead of using strings everywhere? > > > > Okay I will update, any pointers on new implementation I can look up? > > > > Not really yet. Chen-Yu is working on the Allwinner driver > (drivers/clk/sunxi-ng/) and some work from Jeff for MSM8998 is on the > list that you can look at. Thanks for the pointer, I have done the conversion and will post the update in v2 -- ~Vinod