tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git debugfs_cleanup head: d52687b617d2f0d9709ddfe812df75aa25fe202f commit: d52687b617d2f0d9709ddfe812df75aa25fe202f [11/11] driver core: remove devm_device_add_groups() config: arm64-randconfig-004-20240303 (https://download.01.org/0day-ci/archive/20240303/202403032205.vsrv47wd-lkp@xxxxxxxxx/config) compiler: aarch64-linux-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240303/202403032205.vsrv47wd-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202403032205.vsrv47wd-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): drivers/nvmem/core.c: In function 'nvmem_populate_sysfs_cells': >> drivers/nvmem/core.c:481:15: error: implicit declaration of function 'devm_device_add_groups'; did you mean 'devm_device_add_group'? [-Werror=implicit-function-declaration] 481 | ret = devm_device_add_groups(&nvmem->dev, nvmem_cells_groups); | ^~~~~~~~~~~~~~~~~~~~~~ | devm_device_add_group cc1: some warnings being treated as errors vim +481 drivers/nvmem/core.c 84400305271937 Srinivas Kandagatla 2020-03-25 429 0331c611949fff Miquel Raynal 2023-12-15 430 static int nvmem_populate_sysfs_cells(struct nvmem_device *nvmem) 0331c611949fff Miquel Raynal 2023-12-15 431 { 0331c611949fff Miquel Raynal 2023-12-15 432 struct bin_attribute **cells_attrs, *attrs; 0331c611949fff Miquel Raynal 2023-12-15 433 struct nvmem_cell_entry *entry; 0331c611949fff Miquel Raynal 2023-12-15 434 unsigned int ncells = 0, i = 0; 0331c611949fff Miquel Raynal 2023-12-15 435 int ret = 0; 0331c611949fff Miquel Raynal 2023-12-15 436 0331c611949fff Miquel Raynal 2023-12-15 437 mutex_lock(&nvmem_mutex); 0331c611949fff Miquel Raynal 2023-12-15 438 0331c611949fff Miquel Raynal 2023-12-15 439 if (list_empty(&nvmem->cells) || nvmem->sysfs_cells_populated) { 0331c611949fff Miquel Raynal 2023-12-15 440 nvmem_cells_group.bin_attrs = NULL; 0331c611949fff Miquel Raynal 2023-12-15 441 goto unlock_mutex; 0331c611949fff Miquel Raynal 2023-12-15 442 } 0331c611949fff Miquel Raynal 2023-12-15 443 0331c611949fff Miquel Raynal 2023-12-15 444 /* Allocate an array of attributes with a sentinel */ 0331c611949fff Miquel Raynal 2023-12-15 445 ncells = list_count_nodes(&nvmem->cells); 0331c611949fff Miquel Raynal 2023-12-15 446 cells_attrs = devm_kcalloc(&nvmem->dev, ncells + 1, 0331c611949fff Miquel Raynal 2023-12-15 447 sizeof(struct bin_attribute *), GFP_KERNEL); 0331c611949fff Miquel Raynal 2023-12-15 448 if (!cells_attrs) { 0331c611949fff Miquel Raynal 2023-12-15 449 ret = -ENOMEM; 0331c611949fff Miquel Raynal 2023-12-15 450 goto unlock_mutex; 0331c611949fff Miquel Raynal 2023-12-15 451 } 0331c611949fff Miquel Raynal 2023-12-15 452 0331c611949fff Miquel Raynal 2023-12-15 453 attrs = devm_kcalloc(&nvmem->dev, ncells, sizeof(struct bin_attribute), GFP_KERNEL); 0331c611949fff Miquel Raynal 2023-12-15 454 if (!attrs) { 0331c611949fff Miquel Raynal 2023-12-15 455 ret = -ENOMEM; 0331c611949fff Miquel Raynal 2023-12-15 456 goto unlock_mutex; 0331c611949fff Miquel Raynal 2023-12-15 457 } 0331c611949fff Miquel Raynal 2023-12-15 458 0331c611949fff Miquel Raynal 2023-12-15 459 /* Initialize each attribute to take the name and size of the cell */ 0331c611949fff Miquel Raynal 2023-12-15 460 list_for_each_entry(entry, &nvmem->cells, node) { 0331c611949fff Miquel Raynal 2023-12-15 461 sysfs_bin_attr_init(&attrs[i]); 0331c611949fff Miquel Raynal 2023-12-15 462 attrs[i].attr.name = devm_kasprintf(&nvmem->dev, GFP_KERNEL, e20f378d993b10 Arnd Bergmann 2024-02-09 463 "%s@%x,%x", entry->name, e20f378d993b10 Arnd Bergmann 2024-02-09 464 entry->offset, e20f378d993b10 Arnd Bergmann 2024-02-09 465 entry->bit_offset); 0331c611949fff Miquel Raynal 2023-12-15 466 attrs[i].attr.mode = 0444; 0331c611949fff Miquel Raynal 2023-12-15 467 attrs[i].size = entry->bytes; 0331c611949fff Miquel Raynal 2023-12-15 468 attrs[i].read = &nvmem_cell_attr_read; 0331c611949fff Miquel Raynal 2023-12-15 469 attrs[i].private = entry; 0331c611949fff Miquel Raynal 2023-12-15 470 if (!attrs[i].attr.name) { 0331c611949fff Miquel Raynal 2023-12-15 471 ret = -ENOMEM; 0331c611949fff Miquel Raynal 2023-12-15 472 goto unlock_mutex; 0331c611949fff Miquel Raynal 2023-12-15 473 } 0331c611949fff Miquel Raynal 2023-12-15 474 0331c611949fff Miquel Raynal 2023-12-15 475 cells_attrs[i] = &attrs[i]; 0331c611949fff Miquel Raynal 2023-12-15 476 i++; 0331c611949fff Miquel Raynal 2023-12-15 477 } 0331c611949fff Miquel Raynal 2023-12-15 478 0331c611949fff Miquel Raynal 2023-12-15 479 nvmem_cells_group.bin_attrs = cells_attrs; 0331c611949fff Miquel Raynal 2023-12-15 480 0331c611949fff Miquel Raynal 2023-12-15 @481 ret = devm_device_add_groups(&nvmem->dev, nvmem_cells_groups); 0331c611949fff Miquel Raynal 2023-12-15 482 if (ret) 0331c611949fff Miquel Raynal 2023-12-15 483 goto unlock_mutex; 0331c611949fff Miquel Raynal 2023-12-15 484 0331c611949fff Miquel Raynal 2023-12-15 485 nvmem->sysfs_cells_populated = true; 0331c611949fff Miquel Raynal 2023-12-15 486 0331c611949fff Miquel Raynal 2023-12-15 487 unlock_mutex: 0331c611949fff Miquel Raynal 2023-12-15 488 mutex_unlock(&nvmem_mutex); 0331c611949fff Miquel Raynal 2023-12-15 489 0331c611949fff Miquel Raynal 2023-12-15 490 return ret; 0331c611949fff Miquel Raynal 2023-12-15 491 } 0331c611949fff Miquel Raynal 2023-12-15 492 :::::: The code at line 481 was first introduced by commit :::::: 0331c611949fffdf486652450901a4dc52bc5cca nvmem: core: Expose cells through sysfs :::::: TO: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> :::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel