On Mon, Oct 9, 2017 at 10:02 AM, Geert Uytterhoeven
<ge...@linux-m68k.org> wrote:
> On Thu, Oct 5, 2017 at 6:52 PM, kbuild test robot
> <fengguang...@intel.com> wrote:
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
>> topic/rcar-gen3-z-clk-v3
>> head: 4b41ea6e92cdc7fce15ba99b02c40938a134cff7
>> commit: fda8dbe29dd1f04b61c443147a106962cf5d3f14 [2/6] clk: renesas:
>> rcar-gen3: Add Z2 clock divider support
>> config: i386-allmodconfig (attached as .config)
>> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
>> reproduce:
>> git checkout fda8dbe29dd1f04b61c443147a106962cf5d3f14
>> # save the attached .config to linux build tree
>> make ARCH=i386
>>
>> All errors (new ones prefixed by >>):
>>
>> drivers/clk/renesas/rcar-gen3-cpg.o: In function `cpg_z_clk_set_rate':
>>>> rcar-gen3-cpg.c:(.text+0x289): undefined reference to `__ffsdi2'
>
> This is caused by the use of __builtin_ffsll() in <linux/bitfield.h>.
> On several architectures, gcc generates a call to __ffsdi2().
> This is the case on (non-exhaustive list):
> alpha
> arm32
> avr32
> bfin
> cris
> frv
> h8300
> ia32
> m32r
> m68k
> microblaze
> mn10300
> mips32
> mips64
> openrisc
> parisc
> powerpc32
> s390x
> sh4
> sparc32
> sparc64
> unicore
> xtensa
>
> The following are OK:
> amd64
> arm64
> ia64
> powerpc64
> tilegx (but tile is the only one exporting the symbol to modules?)
(now I had my morning coffee)
Ah, <linux/bitfield.h> could just use __ffs64() + 1 instead.
Will send a patch...
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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
