On Thu, Oct 11, 2018 at 6:28 PM Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
> Quoting Tero Kristo (2018-10-03 06:12:39)
> > Here are the late changes for TI clock driver for 4.20 merge window.
> > Sorry about the late pull request, but this was waiting for resolution
> > on one of the dependencies (patch "clk: ti: fix OF child-node
> > lookup..."). Thanks for setting up the immutable branch for it.
> >
> > Please especially note the core clock changes in this series for adding
> > support of clk save/restore mechanism (two patches from Russ Dill.) I
> > decided to go ahead by adding them to this pull as there has been no
> > comments on the patches and they have been pending on the lists already
> > for a while. They aren't touching anything existing either, so they
> > should be safe to apply also.
>
> Thanks for calling it out and pushing me to react. I have been avoiding
> the suspend/resume patches unsuccessfully and now after looking at them
> I see a few nitpicks that I can resolve after merging this PR. It's also
> a sort of one-off solution for TI where all clks need to run through
> save/restore in a depth-first tree order and just blast in
> enable/disable bits, so we'll need to improve on it in the future. But
> I'm fine to go with it for now and do _something_ instead of just sit
> around and do nothing.
For the record, drivers/clk/renesas/renesas-cpg-mssr.c uses a notifier chain
for save/restore, where individual clock type drivers can register with.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
