Hi Dave, I love your patch! Perhaps something to improve: [auto build test WARNING on vkoul-dmaengine/next] [also build test WARNING on v5.14-rc1 next-20210714] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Dave-Jiang/Fix-idxd-sub-drivers-setup/20210715-072229 base: https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next config: x86_64-randconfig-a006-20210714 (attached as .config) compiler: gcc-10 (Debian 10.2.1-6) 10.2.1 20210110 reproduce (this is a W=1 build): # https://github.com/0day-ci/linux/commit/b3be8eaa8a16899b6c4084b14d5a2173b43299e5 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Dave-Jiang/Fix-idxd-sub-drivers-setup/20210715-072229 git checkout b3be8eaa8a16899b6c4084b14d5a2173b43299e5 # save the attached .config to linux build tree make W=1 ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): drivers/dma/idxd/init.c: In function 'idxd_alloc': >> drivers/dma/idxd/init.c:498:47: warning: implicit conversion from 'enum idxd_type' to 'enum idxd_dev_type' [-Wenum-conversion] 498 | idxd_dev_set_type(&idxd->idxd_dev, idxd->data->type); | ~~~~~~~~~~^~~~~~ vim +498 drivers/dma/idxd/init.c 483 484 static struct idxd_device *idxd_alloc(struct pci_dev *pdev, struct idxd_driver_data *data) 485 { 486 struct device *dev = &pdev->dev; 487 struct device *conf_dev; 488 struct idxd_device *idxd; 489 int rc; 490 491 idxd = kzalloc_node(sizeof(*idxd), GFP_KERNEL, dev_to_node(dev)); 492 if (!idxd) 493 return NULL; 494 495 conf_dev = idxd_confdev(idxd); 496 idxd->pdev = pdev; 497 idxd->data = data; > 498 idxd_dev_set_type(&idxd->idxd_dev, idxd->data->type); 499 idxd->id = ida_alloc(&idxd_ida, GFP_KERNEL); 500 if (idxd->id < 0) 501 return NULL; 502 503 device_initialize(conf_dev); 504 conf_dev->parent = dev; 505 conf_dev->bus = &dsa_bus_type; 506 conf_dev->type = idxd->data->dev_type; 507 rc = dev_set_name(conf_dev, "%s%d", idxd->data->name_prefix, idxd->id); 508 if (rc < 0) { 509 put_device(conf_dev); 510 return NULL; 511 } 512 513 spin_lock_init(&idxd->dev_lock); 514 spin_lock_init(&idxd->cmd_lock); 515 516 return idxd; 517 } 518 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org