Messages in this thread | | | From | Geert Uytterhoeven <> | Date | Mon, 11 Jun 2018 11:13:30 +0200 | Subject | Re: [PATCH 3/3 RFC] Revert "net: stmmac: fix build failure due to missing COMMON_CLK dependency" |
| |
Hi Arnd,
On Mon, Jun 11, 2018 at 10:59 AM Arnd Bergmann <arnd@arndb.de> wrote: > On Mon, Jun 11, 2018 at 10:44 AM, Geert Uytterhoeven > <geert@linux-m68k.org> 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@linux-m68k.org> > > --- > > 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@linux-m68k.org
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
| |