Quoting Daniel Palmer (2020-12-19 22:42:40) > > > + > > > +static int msc313_mpll_remove(struct platform_device *pdev) > > > +{ > > > + struct msc313_mpll *mpll = platform_get_drvdata(pdev); > > > + int i; > > > + > > > + for (i = 0; i < ARRAY_SIZE(output_dividers); i++) > > > + clk_hw_unregister_fixed_factor(to_divider_hw(mpll, i)); > > > > Maybe add a devm_ for this if it doesn't exist. > > I did think about adding this. Would I need to do that in a separate > series or would it be ok to roll it into this one? > Can be part of the same series.