Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

[kernel test robot]

Hi Biju,

kernel test robot noticed the following build errors:

[auto build test ERROR on linus/master]
[also build test ERROR on ulf-hansson-mmc-mirror/next v6.16 next-20250729]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Biju/mmc-tmio-Add-64-bit-read-write-support-for-SD_BUF0-in-polling-mode/20250728-001022
base:   linus/master
patch link:    https://lore.kernel.org/r/20250727160731.106312-2-biju.das.jz%40bp.renesas.com
patch subject: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode
config: powerpc64-randconfig-001-20250730 (https://download.01.org/0day-ci/archive/20250730/202507301421.AmWhOZBk-lkp@intel.com/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 78c460bbe8f1fc17e2e66b37edf419ccecbfecba)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250730/202507301421.AmWhOZBk-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202507301421.AmWhOZBk-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from drivers/mmc/host/uniphier-sd.c:21:
>> drivers/mmc/host/tmio_mmc.h:249:2: error: call to undeclared function 'ioread64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     249 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
         |         ^
>> drivers/mmc/host/tmio_mmc.h:255:2: error: call to undeclared function 'iowrite64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     255 |         iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
         |         ^
   2 errors generated.


vim +/ioread64_rep +249 drivers/mmc/host/tmio_mmc.h

   244	
   245	#ifdef CONFIG_64BIT
   246	static inline void sd_ctrl_read64_rep(struct tmio_mmc_host *host, int addr,
   247					      u64 *buf, int count)
   248	{
 > 249		ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
   250	}
   251	
   252	static inline void sd_ctrl_write64_rep(struct tmio_mmc_host *host, int addr,
   253					       const u64 *buf, int count)
   254	{
 > 255		iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
   256	}
   257	#endif
   258	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 715 bytes --]


>    In file included from drivers/mmc/host/uniphier-sd.c:21:
> >> drivers/mmc/host/tmio_mmc.h:249:2: error: call to undeclared function 'ioread64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
>      249 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
>          |         ^
> >> drivers/mmc/host/tmio_mmc.h:255:2: error: call to undeclared function 'iowrite64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
>      255 |         iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
>          |         ^

Sigh, then the guard seems to be ARM64 after all :(


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

[Geert Uytterhoeven]

Hi Wolfram,

On Wed, 30 Jul 2025 at 11:17, Wolfram Sang <wsa-dev@sang-engineering.com> wrote:
> >    In file included from drivers/mmc/host/uniphier-sd.c:21:
> > >> drivers/mmc/host/tmio_mmc.h:249:2: error: call to undeclared function 'ioread64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
> >      249 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> >          |         ^
> > >> drivers/mmc/host/tmio_mmc.h:255:2: error: call to undeclared function 'iowrite64_rep'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
> >      255 |         iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> >          |         ^
>
> Sigh, then the guard seems to be ARM64 after all :(

ioread64_rep() is defined in include/asm-generic/io.h, and powerpc does
include that.

Perhaps drivers/mmc/host/tmio_mmc.h should just include <linux/io.h>?

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

RE: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

[Biju Das]

Hi Geert/Wolfram,

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 30 July 2025 14:57
> Subject: Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode
> 
> Hi Wolfram,
> 
> On Wed, 30 Jul 2025 at 11:17, Wolfram Sang <wsa-dev@sang-engineering.com> wrote:
> > >    In file included from drivers/mmc/host/uniphier-sd.c:21:
> > > >> drivers/mmc/host/tmio_mmc.h:249:2: error: call to undeclared
> > > >> function 'ioread64_rep'; ISO C99 and later do not support
> > > >> implicit function declarations [-Wimplicit-function-declaration]
> > >      249 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> > >          |         ^
> > > >> drivers/mmc/host/tmio_mmc.h:255:2: error: call to undeclared
> > > >> function 'iowrite64_rep'; ISO C99 and later do not support
> > > >> implicit function declarations [-Wimplicit-function-declaration]
> > >      255 |         iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> > >          |         ^
> >
> > Sigh, then the guard seems to be ARM64 after all :(
> 
> ioread64_rep() is defined in include/asm-generic/io.h, and powerpc does include that.
> 
> Perhaps drivers/mmc/host/tmio_mmc.h should just include <linux/io.h>?

This does not fix the issue.


biju@biju:~/lkp-tests$ COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang-21 ~/lkp-tests/kbuild/make.cross W=1 O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/mmc/
Compiler will be installed in /home/biju/0day
PATH=/home/biju/0day/llvm-21.1.0-rc2-x86_64/bin:/home/biju/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
make --keep-going CONFIG_OF_ALL_DTBS=y CONFIG_DTC=y LLVM=1 CROSS_COMPILE=powerpc64-linux- --jobs=64 KCFLAGS= -Wno-error=return-type -Wreturn-type -funsigned-char -Wundef W=1 O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/mmc/
make[1]: Entering directory '/home/biju/lkp-tests/build_dir'
  GEN     Makefile
  CALL    ../scripts/checksyscalls.sh
  CC [M]  drivers/mmc/host/tmio_mmc_core.o
  CC [M]  drivers/mmc/host/uniphier-sd.o
In file included from ../drivers/mmc/host/uniphier-sd.c:21:
../drivers/mmc/host/tmio_mmc.h:250:2: error: call to undeclared function 'ioread64_rep'; ISO C99 and later do not support implicit function declarations
      [-Wimplicit-function-declaration]
  250 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);

Cheers,
Biju

RE: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode

[Biju Das]

Hi Geert,

> -----Original Message-----
> From: Biju Das
> Sent: 30 July 2025 15:12
> Subject: RE: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support for SD_BUF0 in polling mode
> 
> Hi Geert/Wolfram,
> 
> > -----Original Message-----
> > From: Geert Uytterhoeven <geert@linux-m68k.org>
> > Sent: 30 July 2025 14:57
> > Subject: Re: [PATCH v2 1/2] mmc: tmio: Add 64-bit read/write support
> > for SD_BUF0 in polling mode
> >
> > Hi Wolfram,
> >
> > On Wed, 30 Jul 2025 at 11:17, Wolfram Sang <wsa-dev@sang-engineering.com> wrote:
> > > >    In file included from drivers/mmc/host/uniphier-sd.c:21:
> > > > >> drivers/mmc/host/tmio_mmc.h:249:2: error: call to undeclared
> > > > >> function 'ioread64_rep'; ISO C99 and later do not support
> > > > >> implicit function declarations
> > > > >> [-Wimplicit-function-declaration]
> > > >      249 |         ioread64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> > > >          |         ^
> > > > >> drivers/mmc/host/tmio_mmc.h:255:2: error: call to undeclared
> > > > >> function 'iowrite64_rep'; ISO C99 and later do not support
> > > > >> implicit function declarations
> > > > >> [-Wimplicit-function-declaration]
> > > >      255 |         iowrite64_rep(host->ctl + (addr << host->bus_shift), buf, count);
> > > >          |         ^
> > >
> > > Sigh, then the guard seems to be ARM64 after all :(
> >
> > ioread64_rep() is defined in include/asm-generic/io.h, and powerpc does include that.
> >
> > Perhaps drivers/mmc/host/tmio_mmc.h should just include <linux/io.h>?
> 
> This does not fix the issue.

Reason is CONFIG_GENERIC_IOMAP=y

So, ioread64_rep() is excluded.

Cheers,
Biju