Thread (9 messages) 9 messages, 5 authors, 2021-05-28

Re: [PATCH v1] kbuild: Disable compile testing if HAVE_LEGACY_CLK enabled

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-05-24 08:55:20
Also in: linux-arm-kernel, linux-m68k, linux-mips, linux-omap, linux-tegra, lkml

Hi Dmitry,

On Mon, May 24, 2021 at 1:26 AM Dmitry Osipenko [off-list ref] wrote:
There are couple older platforms that can't be compile-tested because they
partially implement CLK API. It causes build failure of kernel drivers due
to the missing symbols of the unimplemented part of CLK API.

These platforms are: ARM EP93XX, ARM OMAP1, m68k ColdFire, MIPS AR7,
                     MIPS Ralink.

Disable compile-testing for HAVE_LEGACY_CLK=y.

Reported-by: kernel test robot <redacted>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Thanks for your patch!
quoted hunk ↗ jump to hunk
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -131,7 +131,7 @@ config INIT_ENV_ARG_LIMIT

 config COMPILE_TEST
        bool "Compile also drivers which will not load"
-       depends on HAS_IOMEM
+       depends on HAS_IOMEM && !HAVE_LEGACY_CLK
That sounds a bit drastic to me.  Usually we just try to implement the
missing functionality, or provide stubs.
Which functions are missing?
        help
          Some drivers can be compiled on a different platform than they are
          intended to be run on. Despite they cannot be loaded there (or even
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help