On Sun, Feb 28, 2021 at 10:08 AM Lars-Peter Clausen <lars@xxxxxxxxxx> wrote: > > On 2/15/21 11:40 AM, Alexandru Ardelean wrote: > > +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. >