Re: [PATCH V4 2/7] mmc: sdhci: Change SDMA address register for v4 mode

From: kbuild test robot
Date: Mon Jul 23 2018 - 18:27:23 EST


Hi Chunyan,

I love your patch! Perhaps something to improve:

[auto build test WARNING on ulf.hansson-mmc/next]
[also build test WARNING on v4.18-rc6 next-20180723]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Chunyan-Zhang/mmc-add-support-for-sdhci-4-0/20180724-045328
base: git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git next
config: arm-exynos_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=7.2.0 make.cross ARCH=arm

All warnings (new ones prefixed by >>):

In file included from include/linux/kernel.h:14:0,
from include/linux/delay.h:22,
from drivers/mmc/host/sdhci.c:16:
drivers/mmc/host/sdhci.c: In function 'sdhci_data_irq':
>> drivers/mmc/host/sdhci.c:43:11: warning: format '%p' expects argument of type 'void *', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
pr_debug("%s: " DRIVER_NAME ": " f, mmc_hostname(host->mmc), ## x)
^
include/linux/printk.h:288:21: note: in definition of macro 'pr_fmt'
#define pr_fmt(fmt) fmt
^~~
include/linux/printk.h:336:2: note: in expansion of macro 'dynamic_pr_debug'
dynamic_pr_debug(fmt, ##__VA_ARGS__)
^~~~~~~~~~~~~~~~
drivers/mmc/host/sdhci.c:43:2: note: in expansion of macro 'pr_debug'
pr_debug("%s: " DRIVER_NAME ": " f, mmc_hostname(host->mmc), ## x)
^~~~~~~~
drivers/mmc/host/sdhci.c:2849:4: note: in expansion of macro 'DBG'
DBG("DMA base %pad, transferred 0x%06x bytes, next %pad\n",
^~~
drivers/mmc/host/sdhci.c:2849:19: note: format string is defined here
DBG("DMA base %pad, transferred 0x%06x bytes, next %pad\n",
~^
%d
In file included from include/linux/kernel.h:14:0,
from include/linux/delay.h:22,
from drivers/mmc/host/sdhci.c:16:
drivers/mmc/host/sdhci.c:43:11: warning: format '%p' expects argument of type 'void *', but argument 6 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
pr_debug("%s: " DRIVER_NAME ": " f, mmc_hostname(host->mmc), ## x)
^
include/linux/printk.h:288:21: note: in definition of macro 'pr_fmt'
#define pr_fmt(fmt) fmt
^~~
include/linux/printk.h:336:2: note: in expansion of macro 'dynamic_pr_debug'
dynamic_pr_debug(fmt, ##__VA_ARGS__)
^~~~~~~~~~~~~~~~
drivers/mmc/host/sdhci.c:43:2: note: in expansion of macro 'pr_debug'
pr_debug("%s: " DRIVER_NAME ": " f, mmc_hostname(host->mmc), ## x)
^~~~~~~~
drivers/mmc/host/sdhci.c:2849:4: note: in expansion of macro 'DBG'
DBG("DMA base %pad, transferred 0x%06x bytes, next %pad\n",
^~~
drivers/mmc/host/sdhci.c:2849:56: note: format string is defined here
DBG("DMA base %pad, transferred 0x%06x bytes, next %pad\n",
~^
%d

vim +43 drivers/mmc/host/sdhci.c

d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 @16 #include <linux/delay.h>
5a436cc0a drivers/mmc/host/sdhci.c Adrian Hunter 2017-03-20 17 #include <linux/ktime.h>
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 18 #include <linux/highmem.h>
b8c86fc5d drivers/mmc/host/sdhci.c Pierre Ossman 2008-03-18 19 #include <linux/io.h>
88b476797 drivers/mmc/host/sdhci.c Paul Gortmaker 2011-07-03 20 #include <linux/module.h>
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 21 #include <linux/dma-mapping.h>
5a0e3ad6a drivers/mmc/host/sdhci.c Tejun Heo 2010-03-24 22 #include <linux/slab.h>
117636092 drivers/mmc/host/sdhci.c Ralf Baechle 2007-10-23 23 #include <linux/scatterlist.h>
bd9b90279 drivers/mmc/host/sdhci.c Linus Walleij 2018-01-29 24 #include <linux/sizes.h>
250dcd114 drivers/mmc/host/sdhci.c Ulf Hansson 2017-11-27 25 #include <linux/swiotlb.h>
9bea3c850 drivers/mmc/host/sdhci.c Marek Szyprowski 2010-08-10 26 #include <linux/regulator/consumer.h>
66fd8ad51 drivers/mmc/host/sdhci.c Adrian Hunter 2011-10-03 27 #include <linux/pm_runtime.h>
92e0c44b9 drivers/mmc/host/sdhci.c Zach Brown 2016-11-02 28 #include <linux/of.h>
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 29
2f730fec8 drivers/mmc/host/sdhci.c Pierre Ossman 2008-03-17 30 #include <linux/leds.h>
2f730fec8 drivers/mmc/host/sdhci.c Pierre Ossman 2008-03-17 31
22113efd0 drivers/mmc/host/sdhci.c Aries Lee 2010-12-15 32 #include <linux/mmc/mmc.h>
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 33 #include <linux/mmc/host.h>
473b095a7 drivers/mmc/host/sdhci.c Aaron Lu 2012-07-03 34 #include <linux/mmc/card.h>
85cc1c331 drivers/mmc/host/sdhci.c Corneliu Doban 2015-02-09 35 #include <linux/mmc/sdio.h>
bec9d4e59 drivers/mmc/host/sdhci.c Guennadi Liakhovetski 2012-09-17 36 #include <linux/mmc/slot-gpio.h>
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 37
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 38 #include "sdhci.h"
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 39
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 40 #define DRIVER_NAME "sdhci"
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 41
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 42 #define DBG(f, x...) \
f421865d5 drivers/mmc/host/sdhci.c Adrian Hunter 2017-03-20 @43 pr_debug("%s: " DRIVER_NAME ": " f, mmc_hostname(host->mmc), ## x)
d129bceb1 drivers/mmc/sdhci.c Pierre Ossman 2006-03-24 44

:::::: The code at line 43 was first introduced by commit
:::::: f421865d5b4ce57013040fb1700edceb43a14b42 mmc: sdhci: Improve debug print format

:::::: TO: Adrian Hunter <adrian.hunter@xxxxxxxxx>
:::::: CC: Ulf Hansson <ulf.hansson@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip