lkml.org 
[lkml]   [2020]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/4] SFH: PCI driver to add support of AMD sensor fusion Hub using HID framework
Hi Sandeep,

I love your patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.5-rc5 next-20200110]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url: https://github.com/0day-ci/linux/commits/Sandeep-Singh/SFH-Add-Support-for-AMD-Sensor-Fusion-Hub/20200110-084435
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git b07f636fca1c8fbba124b0082487c0b3890a0e0c
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.5.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.5.0 make.cross ARCH=arm

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

In file included from include/linux/printk.h:331:0,
from include/linux/kernel.h:15,
from include/linux/list.h:9,
from include/linux/pci.h:32,
from drivers/hid/amd-sfh-hid/amd_mp2_pcie.h:12,
from drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:9:
drivers/hid/amd-sfh-hid/amd_mp2_pcie.c: In function 'amd_mp2_pci_init':
>> drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:155:30: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
dev_dbg(ndev_dev(privdata), "Base addr:%llx size:%llx\n", base, size);
^
include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call'
func(&id, ##__VA_ARGS__); \
^~~~~~~~~~~
include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call'
_dynamic_func_call(fmt,__dynamic_dev_dbg, \
^~~~~~~~~~~~~~~~~~
include/linux/device.h:1784:2: note: in expansion of macro 'dynamic_dev_dbg'
dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~~~~~
include/linux/device.h:1784:23: note: in expansion of macro 'dev_fmt'
dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~
>> drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:155:2: note: in expansion of macro 'dev_dbg'
dev_dbg(ndev_dev(privdata), "Base addr:%llx size:%llx\n", base, size);
^~~~~~~
drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:155:30: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 5 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
dev_dbg(ndev_dev(privdata), "Base addr:%llx size:%llx\n", base, size);
^
include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call'
func(&id, ##__VA_ARGS__); \
^~~~~~~~~~~
include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call'
_dynamic_func_call(fmt,__dynamic_dev_dbg, \
^~~~~~~~~~~~~~~~~~
include/linux/device.h:1784:2: note: in expansion of macro 'dynamic_dev_dbg'
dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~~~~~
include/linux/device.h:1784:23: note: in expansion of macro 'dev_fmt'
dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~
>> drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:155:2: note: in expansion of macro 'dev_dbg'
dev_dbg(ndev_dev(privdata), "Base addr:%llx size:%llx\n", base, size);
^~~~~~~

vim +155 drivers/hid/amd-sfh-hid/amd_mp2_pcie.c

118
119 static int amd_mp2_pci_init(struct amd_mp2_dev *privdata, struct pci_dev *pdev)
120 {
121 int rc;
122 int bar_index = 2;
123 resource_size_t size, base;
124
125 pci_set_drvdata(pdev, privdata);
126
127 rc = pci_enable_device(pdev);
128 if (rc)
129 goto err_pci_enable;
130
131 rc = pci_request_regions(pdev, DRIVER_NAME);
132 if (rc)
133 goto err_pci_regions;
134
135 pci_set_master(pdev);
136
137 rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(64));
138 if (rc) {
139 rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));
140 if (rc)
141 goto err_dma_mask;
142 dev_warn(ndev_dev(privdata), "Cannot DMA highmem\n");
143 }
144
145 rc = pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(64));
146 if (rc) {
147 rc = pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(32));
148 if (rc)
149 goto err_dma_mask;
150 dev_warn(ndev_dev(privdata), "Cannot DMA consistent highmem\n");
151 }
152
153 base = pci_resource_start(pdev, bar_index);
154 size = pci_resource_len(pdev, bar_index);
> 155 dev_dbg(ndev_dev(privdata), "Base addr:%llx size:%llx\n", base, size);
156
157 privdata->mmio = ioremap(base, size);
158 if (!privdata->mmio) {
159 rc = -EIO;
160 goto err_dma_mask;
161 }
162
163 return 0;
164
165 err_dma_mask:
166 pci_clear_master(pdev);
167 pci_release_regions(pdev);
168 err_pci_regions:
169 pci_disable_device(pdev);
170 err_pci_enable:
171 pci_set_drvdata(pdev, NULL);
172 return rc;
173 }
174

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2020-01-11 01:39    [W:0.065 / U:9.424 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site