[wsa:renesas/sdhi-uniphier 2/22] drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw'

[wsa:renesas/sdhi-uniphier 2/22] drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw'

[kbuild test robot]

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/sdhi-uniphier
head:   f870c81320d3fd97c0a49eb2b304a100a09cf2bc
commit: de4f5406eab6319fdafc5fa9da840824baef7f8d [2/22] mmc: renesas_sdhi: remove wrong depends on to enable compile test
config: m32r-allyesconfig (attached as .config)
compiler: m32r-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout de4f5406eab6319fdafc5fa9da840824baef7f8d
        # save the attached .config to linux build tree
        make.cross ARCH=m32r 

All errors (new ones prefixed by >>):

   In file included from arch/m32r/include/uapi/asm/byteorder.h:8:0,
                    from arch/m32r/include/asm/bitops.h:22,
                    from include/linux/bitops.h:38,
                    from include/linux/kernel.h:11,
                    from include/linux/delay.h:22,
                    from drivers/mmc/host/tmio_mmc_core.c:30:
   include/linux/byteorder/big_endian.h:8:2: warning: #warning inconsistent configuration, needs CONFIG_CPU_BIG_ENDIAN [-Wcpp]
    #warning inconsistent configuration, needs CONFIG_CPU_BIG_ENDIAN
     ^~~~~~~
   In file included from drivers/mmc/host/tmio_mmc_core.c:53:0:
   drivers/mmc/host/tmio_mmc.h: In function 'sd_ctrl_read16_rep':
>> drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw' [-Werror=implicit-function-declaration]
     readsw(host->ctl + (addr << host->bus_shift), buf, count);
     ^~~~~~
   drivers/mmc/host/tmio_mmc.h: In function 'sd_ctrl_read32_rep':
>> drivers/mmc/host/tmio_mmc.h:262:2: error: implicit declaration of function 'readsl' [-Werror=implicit-function-declaration]
     readsl(host->ctl + (addr << host->bus_shift), buf, count);
     ^~~~~~
   drivers/mmc/host/tmio_mmc.h: In function 'sd_ctrl_write16_rep':
>> drivers/mmc/host/tmio_mmc.h:279:2: error: implicit declaration of function 'writesw' [-Werror=implicit-function-declaration]
     writesw(host->ctl + (addr << host->bus_shift), buf, count);
     ^~~~~~~
   drivers/mmc/host/tmio_mmc.h: In function 'sd_ctrl_write32_rep':
>> drivers/mmc/host/tmio_mmc.h:292:2: error: implicit declaration of function 'writesl' [-Werror=implicit-function-declaration]
     writesl(host->ctl + (addr << host->bus_shift), buf, count);
     ^~~~~~~
   cc1: some warnings being treated as errors

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

a11862d3 Simon Horman      2011-06-21  245  
a11862d3 Simon Horman      2011-06-21  246  static inline void sd_ctrl_read16_rep(struct tmio_mmc_host *host, int addr,
a11862d3 Simon Horman      2011-06-21  247  				      u16 *buf, int count)
a11862d3 Simon Horman      2011-06-21  248  {
7445bf9e Kuninori Morimoto 2015-01-13 @249  	readsw(host->ctl + (addr << host->bus_shift), buf, count);
a11862d3 Simon Horman      2011-06-21  250  }
a11862d3 Simon Horman      2011-06-21  251  
f2218db8 Simon Horman      2017-06-16  252  static inline u32 sd_ctrl_read16_and_16_as_32(struct tmio_mmc_host *host,
f2218db8 Simon Horman      2017-06-16  253  					      int addr)
a11862d3 Simon Horman      2011-06-21  254  {
7445bf9e Kuninori Morimoto 2015-01-13  255  	return readw(host->ctl + (addr << host->bus_shift)) |
7445bf9e Kuninori Morimoto 2015-01-13  256  	       readw(host->ctl + ((addr + 2) << host->bus_shift)) << 16;
a11862d3 Simon Horman      2011-06-21  257  }
a11862d3 Simon Horman      2011-06-21  258  
8185e51f Chris Brandt      2016-09-12  259  static inline void sd_ctrl_read32_rep(struct tmio_mmc_host *host, int addr,
8185e51f Chris Brandt      2016-09-12  260  				      u32 *buf, int count)
8185e51f Chris Brandt      2016-09-12  261  {
8185e51f Chris Brandt      2016-09-12 @262  	readsl(host->ctl + (addr << host->bus_shift), buf, count);
8185e51f Chris Brandt      2016-09-12  263  }
8185e51f Chris Brandt      2016-09-12  264  
f2218db8 Simon Horman      2017-06-16  265  static inline void sd_ctrl_write16(struct tmio_mmc_host *host, int addr,
f2218db8 Simon Horman      2017-06-16  266  				   u16 val)
a11862d3 Simon Horman      2011-06-21  267  {
973ed3af Simon Horman      2011-06-21  268  	/* If there is a hook and it returns non-zero then there
973ed3af Simon Horman      2011-06-21  269  	 * is an error and the write should be skipped
973ed3af Simon Horman      2011-06-21  270  	 */
dfe9a229 Kuninori Morimoto 2015-01-13  271  	if (host->write16_hook && host->write16_hook(host, addr))
973ed3af Simon Horman      2011-06-21  272  		return;
7445bf9e Kuninori Morimoto 2015-01-13  273  	writew(val, host->ctl + (addr << host->bus_shift));
a11862d3 Simon Horman      2011-06-21  274  }
a11862d3 Simon Horman      2011-06-21  275  
a11862d3 Simon Horman      2011-06-21  276  static inline void sd_ctrl_write16_rep(struct tmio_mmc_host *host, int addr,
a11862d3 Simon Horman      2011-06-21  277  				       u16 *buf, int count)
a11862d3 Simon Horman      2011-06-21  278  {
7445bf9e Kuninori Morimoto 2015-01-13 @279  	writesw(host->ctl + (addr << host->bus_shift), buf, count);
a11862d3 Simon Horman      2011-06-21  280  }
a11862d3 Simon Horman      2011-06-21  281  
f2218db8 Simon Horman      2017-06-16  282  static inline void sd_ctrl_write32_as_16_and_16(struct tmio_mmc_host *host,
f2218db8 Simon Horman      2017-06-16  283  						int addr, u32 val)
a11862d3 Simon Horman      2011-06-21  284  {
7c42dbf3 Wolfram Sang      2016-05-27  285  	writew(val & 0xffff, host->ctl + (addr << host->bus_shift));
7445bf9e Kuninori Morimoto 2015-01-13  286  	writew(val >> 16, host->ctl + ((addr + 2) << host->bus_shift));
a11862d3 Simon Horman      2011-06-21  287  }
a11862d3 Simon Horman      2011-06-21  288  
8185e51f Chris Brandt      2016-09-12  289  static inline void sd_ctrl_write32_rep(struct tmio_mmc_host *host, int addr,
8185e51f Chris Brandt      2016-09-12  290  				       const u32 *buf, int count)
8185e51f Chris Brandt      2016-09-12  291  {
8185e51f Chris Brandt      2016-09-12 @292  	writesl(host->ctl + (addr << host->bus_shift), buf, count);
8185e51f Chris Brandt      2016-09-12  293  }
8185e51f Chris Brandt      2016-09-12  294  

:::::: The code at line 249 was first introduced by commit
:::::: 7445bf9e6f4e5d7755e22c7c9b06f4ae0d6160c6 mmc: tmio: tmio_mmc_host has .bus_shift

:::::: TO: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
:::::: CC: Ulf Hansson <ulf.hansson@linaro.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 44608 bytes --]

Re: [wsa:renesas/sdhi-uniphier 2/22] drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw'

[Wolfram Sang]

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

Hey guys,

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/sdhi-uniphier

buildbot is a really great and helpful service, so a lot of thanks for
that.

But please, again, drop the i2c mailing list for my personal development
branches. *Only* include the i2c list, if there is an "i2c/" prefix in
the branch name. This branch, for example, is MMC related and has
nothing to do with I2C! Or, if this is not possible, drop the I2C list
from all my branches and just mail me and the author.

The current state creates too much noise and it starts to get a little
annoying. Please let's find some fix for that. buildbot is so useful
otherwise.

Kind regards,

   Wolfram


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

Re: [wsa:renesas/sdhi-uniphier 2/22] drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw'

[Fengguang Wu]

Hi Wolfram,

On Mon, Dec 04, 2017 at 05:00:27PM +0100, Wolfram Sang wrote:
>Hey guys,
>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/sdhi-uniphier
>
>buildbot is a really great and helpful service, so a lot of thanks for
>that.
>
>But please, again, drop the i2c mailing list for my personal development
>branches. *Only* include the i2c list, if there is an "i2c/" prefix in
>the branch name. This branch, for example, is MMC related and has
>nothing to do with I2C! Or, if this is not possible, drop the I2C list
>from all my branches and just mail me and the author.
>
>The current state creates too much noise and it starts to get a little
>annoying. Please let's find some fix for that. buildbot is so useful
>otherwise.

Sorry for the noise! We've added your tree to private reporting,
however it looks the mail_cc option has higher priority. So I'll
just remove the mail_cc. We don't support selectively CC mailing lists
based on branch name, which looks a bit overkill for most people.

Thanks,
Fengguang

Re: [wsa:renesas/sdhi-uniphier 2/22] drivers/mmc/host/tmio_mmc.h:249:2: error: implicit declaration of function 'readsw'

[Wolfram Sang]

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


> Sorry for the noise! We've added your tree to private reporting,
> however it looks the mail_cc option has higher priority. So I'll
> just remove the mail_cc. We don't support selectively CC mailing lists
> based on branch name, which looks a bit overkill for most people.

I think private reporting is good enough for me. Thank you!


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