iio: buffer: initialize masklength accumulator to 0
authorDavid Lechner <dlechner@baylibre.com>
Thu, 25 Apr 2024 15:03:29 +0000 (10:03 -0500)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 29 Apr 2024 19:53:24 +0000 (20:53 +0100)
Since masklength is marked as [INTERN], no drivers should assign it and
the value will always be 0. Therefore, the local ml accumulator variable
in iio_buffers_alloc_sysfs_and_mask() will always start out as 0.

This changes the code to explicitly set ml to 0 to make it clear that
drivers should not be trying to override the masklength field.

Signed-off-by: David Lechner <dlechner@baylibre.com>
Link: https://lore.kernel.org/r/20240425-b4-iio-masklength-cleanup-v1-3-d3d16318274d@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/industrialio-buffer.c

index 1d950a3e153b8a97603c883dca1773125a4eba5d..cec58a604d73a93b46dd4e2c5a64c92d1d4f31ab 100644 (file)
@@ -1744,7 +1744,7 @@ int iio_buffers_alloc_sysfs_and_mask(struct iio_dev *indio_dev)
 
        channels = indio_dev->channels;
        if (channels) {
-               int ml = indio_dev->masklength;
+               int ml = 0;
 
                for (i = 0; i < indio_dev->num_channels; i++)
                        ml = max(ml, channels[i].scan_index + 1);