On Sun, Feb 28, 2021 at 10:08 AM Lars-Peter Clausen [off-list ref] wrote:

On 2/15/21 11:40 AM, Alexandru Ardelean wrote:

quoted

+int devm_iio_kfifo_buffer_setup(struct device *dev,
+                             struct iio_dev *indio_dev,
+                             int mode_flags,
+                             const struct iio_buffer_setup_ops *setup_ops)
+{
+     struct iio_buffer *buffer;
+
+     if (mode_flags)
+             mode_flags &= kfifo_access_funcs.modes;

The if seems unnecessary. If it is 0 and we AND it with something it is
still 0.

Makes sense.
Will send an update for this.