Hi Arnd, On Mon, Jun 11, 2018 at 10:59 AM Arnd Bergmann <arnd@xxxxxxxx> wrote: > On Mon, Jun 11, 2018 at 10:44 AM, Geert Uytterhoeven > <geert@xxxxxxxxxxxxxx> wrote: > > This reverts commit bde4975310eb1982bd0bbff673989052d92fd481. > > > > All legacy clock implementations now implement clk_set_rate() (Some > > implementations may be dummies, though). > > > > Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > --- > > Marked "RFC", as this depends on "m68k: coldfire: Normalize clk API" and > > "MIPS: AR7: Normalize clk API". > > This seems reasonable. It's possible that it will cause regressions because the > COMMON_CLK dependency hides another dependency on something else > that not everything implements, but we should fix that properly if that happens. Compile-testing was enabled 2 years ago, in commit 2e280c188f06b190 ("stmmac: make platform drivers depend on their associated SoC"), but the dependency on COMMON_CLK was added only recently. That's what triggered me: the drivers were suddenly disabled in m68k allmodconfig, while they built fine for years before. 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