| Message ID | 20210520161702.3746174-1-lee.jones@linaro.org (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [RESEND,1/1] clk: versatile: remove dependency on ARCH_* | expand |
On Thu, May 20, 2021 at 6:17 PM Lee Jones <lee.jones@linaro.org> wrote: > From: Peter Collingbourne <pcc@google.com> > > It is now possible to build a modular kernel for vexpress by > not setting CONFIG_ARCH_VEXPRESS=y and instead setting =m on the > drivers that it normally implies. This is with the exception of > CLK_VEXPRESS_OSC which is currently hidden behind a dependency on > one of several ARCH_* variables. Remove that dependency so that > CLK_VEXPRESS_OSC may be enabled without it. > > Signed-off-by: Peter Collingbourne <pcc@google.com> > Link: https://linux-review.googlesource.com/id/I435a21e2e5f6187db54f4ef2079b60028ab2ea69 > Signed-off-by: Lee Jones <lee.jones@linaro.org> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
On Fri, 21 May 2021, Linus Walleij wrote: > On Thu, May 20, 2021 at 6:17 PM Lee Jones <lee.jones@linaro.org> wrote: > > > From: Peter Collingbourne <pcc@google.com> > > > > It is now possible to build a modular kernel for vexpress by > > not setting CONFIG_ARCH_VEXPRESS=y and instead setting =m on the > > drivers that it normally implies. This is with the exception of > > CLK_VEXPRESS_OSC which is currently hidden behind a dependency on > > one of several ARCH_* variables. Remove that dependency so that > > CLK_VEXPRESS_OSC may be enabled without it. > > > > Signed-off-by: Peter Collingbourne <pcc@google.com> > > Link: https://linux-review.googlesource.com/id/I435a21e2e5f6187db54f4ef2079b60028ab2ea69 > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Still not in -next. Who will take this please?
Quoting Lee Jones (2021-05-20 09:17:02) > From: Peter Collingbourne <pcc@google.com> > > It is now possible to build a modular kernel for vexpress by > not setting CONFIG_ARCH_VEXPRESS=y and instead setting =m on the > drivers that it normally implies. This is with the exception of > CLK_VEXPRESS_OSC which is currently hidden behind a dependency on > one of several ARCH_* variables. Remove that dependency so that > CLK_VEXPRESS_OSC may be enabled without it. > > Signed-off-by: Peter Collingbourne <pcc@google.com> > Link: https://linux-review.googlesource.com/id/I435a21e2e5f6187db54f4ef2079b60028ab2ea69 > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- Applied to clk-next
diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig index 91f0ff54237d2..792315d893db3 100644 --- a/drivers/clk/versatile/Kconfig +++ b/drivers/clk/versatile/Kconfig @@ -1,8 +1,6 @@ # SPDX-License-Identifier: GPL-2.0-only menu "Clock driver for ARM Reference designs" - depends on ARCH_INTEGRATOR || ARCH_REALVIEW || \ - ARCH_VERSATILE || ARCH_VEXPRESS || COMPILE_TEST config ICST bool "Clock driver for ARM Reference designs ICST"