On 07/25, Quentin Schulz wrote: > Hi Stephen, > > I forgot to answer one of your questions: > > On 22/07/2017 00:20, Stephen Boyd wrote: > > On 07/13, Quentin Schulz wrote: > >> diff --git a/drivers/clk/at91/clk-audio-pll-pad.c b/drivers/clk/at91/clk-audio-pll-pad.c > >> new file mode 100644 > >> index 000000000000..10dd6d625696 > >> --- /dev/null > >> +++ b/drivers/clk/at91/clk-audio-pll-pad.c > [...] > >> +} > >> + > >> +CLK_OF_DECLARE(of_sama5d2_clk_audio_pll_pad_setup, > >> + "atmel,sama5d2-clk-audio-pll-pad", > >> + of_sama5d2_clk_audio_pll_pad_setup); > > > > We can't have a device driver for this? > > > > I tried to work on that but the device does not get probed because the > parent node (pmc) does not register its children. We would need to > modify the DT to add such a support and break DT compat. > > As we are speaking about getting rid of clk subnodes (c.f. Rob and > Alexandre answers on the first version of this patch series: > https://patchwork.kernel.org/patch/9805991/) in favor of a clock > controller node that would break the DT compat as well, IMHO it would be > best to break the DT compat as few times as possible and thus, rework > this driver once we switch to a clock controller node. > Ok. That's fine. Is there work in progress to make a new binding here? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
