On Tue, Sep 03, 2024 at 02:26:41PM +0000, Nechita, Ramona wrote: > From: Jonathan Cameron <jic23@xxxxxxxxxx> > Sent: Saturday, July 27, 2024 6:41 PM > To: Nechita, Ramona <Ramona.Nechita@xxxxxxxxxx> ... > >> + for_each_set_bit(bit, indio_dev->active_scan_mask, AD7779_NUM_CHANNELS - 1) > >> + tmp[k++] = st->spidata_rx[bit]; > > > >Update this to use Nuno's new macros for iterating over the scan mask. > > Does this refer to iio_for_each_active_channel ? I checked and noticed that > the patch containing this macro is not upstream yet, should I wait for it to > be merged before sending out a new patch? It's in the maintainer's tree, which your patch should be based on. So I don't see any issues here. Do you? -- With Best Regards, Andy Shevchenko