From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============0200212476369164171==" MIME-Version: 1.0 From: Helge Deller To: kbuild-all@lists.01.org Subject: Re: [linux-next:master 3272/11956] undefined reference to `ioread64_hi_lo' Date: Wed, 30 Sep 2020 17:29:05 +0200 Message-ID: <9bfdfbf3-1eea-f5bb-317f-a90de131529e@gmx.de> In-Reply-To: List-Id: --===============0200212476369164171== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On 9/30/20 4:38 PM, Helge Deller wrote: > On 9/30/20 3:44 AM, kernel test robot wrote: >> Hi Helge, >> >> FYI, the error/warning still remains. > > I answered on 8/29/20, 10:47 PM and didn't got any reply.... > Here is what I wrote: > > In drivers/firmware/arm_scmi/perf.c iowrite64_hi_lo() is called directly. > So, I could add this function to arch/parisc/lib/iomap.c to prevent the l= inker error. > But this file is compiled on parisc just because CONFIG_COMPILE_TEST is s= et, while > in production builds it would never be compiled. > > So, what's the usual take in such a scenario? > Shall I ignore this linker error and 0-day warning, or shall I add a work= around? I fixed it by adding the functions in my for-next tree: https://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git/com= mit/?h=3Dfor-next Thanks! Helge >> >> tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.= git master >> head: 49e7e3e905e437a02782019570f70997e2da9101 >> commit: ad697ae387f9f707c4065ab26d583c007bec76e4 [3272/11956] parisc: Ad= d qemu fw_cfg interface >> config: parisc-allyesconfig (attached as .config) >> compiler: hppa-linux-gcc (GCC) 9.3.0 >> reproduce (this is a W=3D1 build): >> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sb= in/make.cross -O ~/bin/make.cross >> chmod +x ~/bin/make.cross >> # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-nex= t.git/commit/?id=3Dad697ae387f9f707c4065ab26d583c007bec76e4 >> git remote add linux-next https://git.kernel.org/pub/scm/linux/k= ernel/git/next/linux-next.git >> git fetch --no-tags linux-next master >> git checkout ad697ae387f9f707c4065ab26d583c007bec76e4 >> # save the attached .config to linux build tree >> COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dgcc-9.3.0 make.cro= ss ARCH=3Dparisc >> >> If you fix the issue, kindly add following tag as appropriate >> Reported-by: kernel test robot >> >> All errors (new ones prefixed by >>): >> >> hppa-linux-ld: drivers/firmware/arm_scmi/perf.o: in function `scmi_pe= rf_fc_ring_db': >>>> (.text+0x610): undefined reference to `ioread64_hi_lo' >>>> hppa-linux-ld: (.text+0x63c): undefined reference to `iowrite64_hi_lo' >> >> --- >> 0-DAY CI Kernel Test Service, Intel Corporation >> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org >> > --===============0200212476369164171==--