CC: [email protected] In-Reply-To: <[email protected]> References: <[email protected]> TO: Alexandru Ardelean <[email protected]> TO: [email protected] TO: [email protected] CC: [email protected] CC: [email protected] CC: [email protected] CC: [email protected] CC: [email protected] CC: Alexandru Ardelean <[email protected]>
Hi Alexandru, I love your patch! Perhaps something to improve: [auto build test WARNING on iio/togreg] [also build test WARNING on linux/master linus/master v5.11-rc5 next-20210122] [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/Alexandru-Ardelean/iio-core-buffer-add-support-for-multiple-IIO-buffers-per-IIO-device/20210125-053419 base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg :::::: branch date: 10 hours ago :::::: commit date: 10 hours ago config: i386-randconfig-m021-20210125 (attached as .config) compiler: gcc-9 (Debian 9.3.0-20) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> smatch warnings: drivers/iio/industrialio-buffer.c:1606 iio_buffer_alloc_sysfs_and_mask() error: uninitialized symbol 'cnt'. vim +/cnt +1606 drivers/iio/industrialio-buffer.c e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1550 ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1551 /** ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1552 * iio_buffer_alloc_sysfs_and_mask() - Allocate sysfs attributes to attached buffers ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1553 * @indio_dev: the iio device for which to create the buffer sysfs attributes ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1554 * ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1555 * Return 0, or negative for error. ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1556 * ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1557 * If the IIO device has no buffer attached, no sysfs attributes will be created. ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1558 * This function must be called after the IIO device object has been created and ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1559 * registered with device_add(). See __iio_buffer_alloc_sysfs_and_mask() for more ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1560 * details. ac4ba6772ea01a56 Alexandru Ardelean 2021-01-22 1561 */ e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1562 int iio_buffer_alloc_sysfs_and_mask(struct iio_dev *indio_dev) d967cb6bd4e79c0c Lars-Peter Clausen 2014-11-26 1563 { 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1564 struct iio_dev_opaque *iio_dev_opaque = to_iio_dev_opaque(indio_dev); ff3f7e049aef9202 Alexandru Ardelean 2020-04-24 1565 struct iio_buffer *buffer = indio_dev->buffer; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1566 const struct iio_chan_spec *channels; 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1567 int i, cnt, ret; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1568 e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1569 channels = indio_dev->channels; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1570 if (channels) { e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1571 int ml = indio_dev->masklength; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1572 e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1573 for (i = 0; i < indio_dev->num_channels; i++) e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1574 ml = max(ml, channels[i].scan_index + 1); e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1575 indio_dev->masklength = ml; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1576 } ff3f7e049aef9202 Alexandru Ardelean 2020-04-24 1577 ff3f7e049aef9202 Alexandru Ardelean 2020-04-24 1578 if (!buffer) e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1579 return 0; e16e0a778fec8ac1 Alexandru Ardelean 2020-09-17 1580 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1581 for (i = 0; i < iio_dev_opaque->attached_buffers_cnt; i++) { 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1582 buffer = iio_dev_opaque->attached_buffers[i]; 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1583 ret = __iio_buffer_alloc_sysfs_and_mask(buffer, indio_dev, i); e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1584 if (ret) 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1585 goto error_unwind_sysfs_and_mask; 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1586 } d967cb6bd4e79c0c Lars-Peter Clausen 2014-11-26 1587 e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1588 ret = sysfs_create_link(&indio_dev->dev.kobj, e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1589 &indio_dev->buffer->buffer_dir, e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1590 "buffer"); e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1591 if (ret) 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1592 goto error_unwind_sysfs_and_mask; e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1593 e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1594 ret = sysfs_create_link(&indio_dev->dev.kobj, e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1595 &indio_dev->buffer->scan_el_dir, e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1596 "scan_elements"); e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1597 if (ret) e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1598 goto error_remove_buffer_dir_link; e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1599 e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1600 return 0; e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1601 e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1602 error_remove_buffer_dir_link: e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1603 sysfs_remove_link(&indio_dev->dev.kobj, "buffer"); 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1604 cnt = iio_dev_opaque->attached_buffers_cnt - 1; 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1605 error_unwind_sysfs_and_mask: 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 @1606 for (; cnt >= 0; cnt--) { 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1607 buffer = iio_dev_opaque->attached_buffers[cnt]; e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1608 __iio_buffer_free_sysfs_and_mask(buffer); 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1609 } 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1610 kfree(iio_dev_opaque->attached_buffers); 8b70c93a9d04de1e Alexandru Ardelean 2021-01-22 1611 iio_dev_opaque->attached_buffers = NULL; e2eb176de4748db8 Alexandru Ardelean 2021-01-22 1612 return ret; d967cb6bd4e79c0c Lars-Peter Clausen 2014-11-26 1613 } d967cb6bd4e79c0c Lars-Peter Clausen 2014-11-26 1614 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/[email protected]
.config.gz
Description: application/gzip
_______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
