Hi Fred,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on vfio/next]
[also build test ERROR on v5.8-rc4 next-20200709]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Fred-Gao/vfio-pci-Refine-Intel-I...
base:
https://github.com/awilliam/linux-vfio.git next
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-14) 9.3.0
reproduce (this is a W=1 build):
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>):
> drivers/vfio/pci/vfio_pci.c:67:2: error: implicit declaration of
function 'INTEL_DG1_IDS'; did you mean 'INTEL_G33_IDS'?
[-Werror=implicit-function-declaration]
67 | INTEL_DG1_IDS(0),
| ^~~~~~~~~~~~~
| INTEL_G33_IDS
> drivers/vfio/pci/vfio_pci.c:67:2: error: initializer element is
not constant
drivers/vfio/pci/vfio_pci.c:67:2: note: (near initialization for
'intel_dgfx_pciids[0].vendor')
> drivers/vfio/pci/vfio_pci.c:68:2: warning: braces around scalar
initializer
68 | { }
| ^
drivers/vfio/pci/vfio_pci.c:68:2: note: (near initialization for
'intel_dgfx_pciids[0].device')
cc1: some warnings being treated as errors
vim +67 drivers/vfio/pci/vfio_pci.c
64
65 /* Intel's dgfx is not IGD, so don't handle them the same way */
66 static const struct pci_device_id intel_dgfx_pciids[] = {
67 INTEL_DG1_IDS(0),
68 { }
69 };
70
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org