Re: [clk:clk-versatile 1/1] clk-versatile.c:undefined reference to `of_iomap'

[Lee Jones <lee.jones@linaro.org>]

On Thu, 03 Jun 2021, Stephen Boyd wrote:
65;6200;1c
> Quoting kernel test robot (2021-06-03 07:58:13)
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-versatile
> > head:   419b3ab6987f2a108cc4710030184365b17467ac
> > commit: 419b3ab6987f2a108cc4710030184365b17467ac [1/1] clk: versatile: remove dependency on ARCH_*
> > config: s390-allmodconfig (attached as .config)
> > compiler: s390-linux-gcc (GCC) 9.3.0
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git/commit/?id=419b3ab6987f2a108cc4710030184365b17467ac
> >         git remote add clk https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git
> >         git fetch --no-tags clk clk-versatile
> >         git checkout 419b3ab6987f2a108cc4710030184365b17467ac
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 
> > 
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> 
> Probably ought to depend on HAS_IOMEM. Linus or Lee?

That would be my best guess also.

> ----8<----
> diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig
> index 792315d893db..481de5657d85 100644
> --- a/drivers/clk/versatile/Kconfig
> +++ b/drivers/clk/versatile/Kconfig
> @@ -1,6 +1,7 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  
>  menu "Clock driver for ARM Reference designs"
> +	depends on HAS_IOMEM
>  
>  config ICST
>  	bool "Clock driver for ARM Reference designs ICST"
> 
> > 
> > All errors (new ones prefixed by >>):
> > 
> >    s390-linux-ld: kernel/dma/coherent.o: in function `dma_init_coherent_memory':
> >    coherent.c:(.text+0x3bc): undefined reference to `memremap'
> >    s390-linux-ld: coherent.c:(.text+0x500): undefined reference to `memunmap'
> >    s390-linux-ld: kernel/dma/coherent.o: in function `dma_declare_coherent_memory':
> >    coherent.c:(.text+0xae6): undefined reference to `memunmap'
> >    s390-linux-ld: drivers/irqchip/irq-al-fic.o: in function `al_fic_init_dt':
> >    irq-al-fic.c:(.init.text+0x98): undefined reference to `of_iomap'
> >    s390-linux-ld: irq-al-fic.c:(.init.text+0x596): undefined reference to `iounmap'
> >    s390-linux-ld: drivers/clk/clk-fixed-mmio.o: in function `fixed_mmio_clk_setup':
> >    clk-fixed-mmio.c:(.text+0x9a): undefined reference to `of_iomap'
> >    s390-linux-ld: clk-fixed-mmio.c:(.text+0xe6): undefined reference to `iounmap'
> >    s390-linux-ld: drivers/clk/versatile/clk-versatile.o: in function `cm_osc_setup':
> > >> clk-versatile.c:(.init.text+0xbc): undefined reference to `of_iomap'
> >    s390-linux-ld: drivers/clk/versatile/clk-sp810.o: in function `clk_sp810_of_setup':
> > >> clk-sp810.c:(.init.text+0x174): undefined reference to `of_iomap'
> >    s390-linux-ld: drivers/clocksource/timer-of.o: in function `timer_of_init':
> >    timer-of.c:(.init.text+0xcc): undefined reference to `of_iomap'
> >    s390-linux-ld: timer-of.c:(.init.text+0x8ee): undefined reference to `iounmap'
> >    s390-linux-ld: drivers/clocksource/timer-of.o: in function `timer_of_cleanup':
> >    timer-of.c:(.init.text+0xb9a): undefined reference to `iounmap'
> >    s390-linux-ld: drivers/clocksource/timer-microchip-pit64b.o: in function `mchp_pit64b_dt_init_timer':
> >    timer-microchip-pit64b.c:(.init.text+0x150): undefined reference to `of_iomap'
> >    s390-linux-ld: timer-microchip-pit64b.c:(.init.text+0xc78): undefined reference to `iounmap'
> >

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog