Re: [PATCH] clk: versatile: undo some dependency changes

[Arnd Bergmann]

On Thu, May 28, 2020 at 2:34 AM Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
>
> Quoting Arnd Bergmann (2020-05-27 06:40:33)
> > SP810 and ICST are selected by a couple of platforms, most but
> > not all in the versatile family:
> >
> > WARNING: unmet direct dependencies detected for CLK_SP810
> >   Depends on [n]: COMMON_CLK [=y] && COMMON_CLK_VERSATILE [=n]
> >   Selected by [y]:
> >   - ARCH_REALVIEW [=y] && (ARCH_MULTI_V5 [=n] || ARCH_MULTI_V6 [=n] ||
> > ARCH_MULTI_V7 [=y])
> >
> > WARNING: unmet direct dependencies detected for ICST
> >   Depends on [n]: COMMON_CLK [=y] && COMMON_CLK_VERSATILE [=n]
> >   Selected by [y]:
> >   - ARCH_REALVIEW [=y] && (ARCH_MULTI_V5 [=n] || ARCH_MULTI_V6 [=n] || ARCH_MULTI_V7 [=y])
> >   - ARCH_VEXPRESS [=y] && ARCH_MULTI_V7 [=y]
> >   - ARCH_ZYNQ [=y] && ARCH_MULTI_V7 [=y]
> >
> > Change back the Kconfig logic to allow these to be selected
> > without the main option.
> >
> > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> > ---
>
> Is this similar to
> https://lore.kernel.org/r/20200527181307.2482167-1-robh@xxxxxxxxxx
> ?

It's similar, but that version still breaks ZYNQ when CONFIG_COMPILE_TEST
is disabled.

      Arnd