Re: [PATCH 3/4] iommu: Remove pci.h include from trace/events/iommu.h

From: kbuild test robot
Date: Thu Apr 27 2017 - 10:17:15 EST


Hi Joerg,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.11-rc8 next-20170426]
[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/Joerg-Roedel/iommu-Header-file-cleanups/20170427-160734
config: tile-allmodconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 4.6.2
reproduce:
wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=tile

All warnings (new ones prefixed by >>):

In file included from drivers/infiniband//hw/qedr/main.c:39:0:
>> include/linux/qed/qede_roce.h:57:12: warning: 'struct pci_dev' declared inside parameter list [enabled by default]
>> include/linux/qed/qede_roce.h:57:12: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]
>> drivers/infiniband//hw/qedr/main.c:905:2: warning: initialization from incompatible pointer type [enabled by default]
drivers/infiniband//hw/qedr/main.c:905:2: warning: (near initialization for 'qedr_drv.add') [enabled by default]
--
In file included from drivers/infiniband/hw/qedr/main.c:39:0:
>> include/linux/qed/qede_roce.h:57:12: warning: 'struct pci_dev' declared inside parameter list [enabled by default]
>> include/linux/qed/qede_roce.h:57:12: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]
drivers/infiniband/hw/qedr/main.c:905:2: warning: initialization from incompatible pointer type [enabled by default]
drivers/infiniband/hw/qedr/main.c:905:2: warning: (near initialization for 'qedr_drv.add') [enabled by default]

vim +57 include/linux/qed/qede_roce.h

cee9fbd8 Ram Amrani 2016-10-01 41 QEDE_DOWN,
cee9fbd8 Ram Amrani 2016-10-01 42 QEDE_CHANGE_ADDR,
cee9fbd8 Ram Amrani 2016-10-01 43 QEDE_CLOSE
cee9fbd8 Ram Amrani 2016-10-01 44 };
cee9fbd8 Ram Amrani 2016-10-01 45
cee9fbd8 Ram Amrani 2016-10-01 46 struct qede_roce_event_work {
cee9fbd8 Ram Amrani 2016-10-01 47 struct list_head list;
cee9fbd8 Ram Amrani 2016-10-01 48 struct work_struct work;
cee9fbd8 Ram Amrani 2016-10-01 49 void *ptr;
cee9fbd8 Ram Amrani 2016-10-01 50 enum qede_roce_event event;
cee9fbd8 Ram Amrani 2016-10-01 51 };
cee9fbd8 Ram Amrani 2016-10-01 52
cee9fbd8 Ram Amrani 2016-10-01 53 struct qedr_driver {
cee9fbd8 Ram Amrani 2016-10-01 54 unsigned char name[32];
cee9fbd8 Ram Amrani 2016-10-01 55
cee9fbd8 Ram Amrani 2016-10-01 56 struct qedr_dev* (*add)(struct qed_dev *, struct pci_dev *,
cee9fbd8 Ram Amrani 2016-10-01 @57 struct net_device *);
cee9fbd8 Ram Amrani 2016-10-01 58
cee9fbd8 Ram Amrani 2016-10-01 59 void (*remove)(struct qedr_dev *);
cee9fbd8 Ram Amrani 2016-10-01 60 void (*notify)(struct qedr_dev *, enum qede_roce_event);
cee9fbd8 Ram Amrani 2016-10-01 61 };
cee9fbd8 Ram Amrani 2016-10-01 62
cee9fbd8 Ram Amrani 2016-10-01 63 /* APIs for RoCE driver to register callback handlers,
cee9fbd8 Ram Amrani 2016-10-01 64 * which will be invoked when device is added, removed, ifup, ifdown
cee9fbd8 Ram Amrani 2016-10-01 65 */

:::::: The code at line 57 was first introduced by commit
:::::: cee9fbd8e2e9e713cd8bf227c6492fd8854de74b qede: Add qedr framework

:::::: TO: Ram Amrani <Ram.Amrani@xxxxxxxxxxxxxxxxxx>
:::::: CC: David S. Miller <davem@xxxxxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip