On Sat, Feb 10, 2024 at 10:57 PM Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote: > > From: Nuno Sa <nuno.sa@xxxxxxxxxx> > > Move to the IIO backend framework. Devices supported by adi-axi-adc now > register themselves as backend devices. ... > struct adi_axi_adc_state { > struct regmap *regmap; > struct device *dev; > }; So, not sure you need these two as one can be derived from the other one (IIUC). In any case, it may be amended in a follow up. ... > + fsleep(10000); As I pointed out, the comment is good to have. Maybe you can add it when reducing the timeout in a follow up patch... -- With Best Regards, Andy Shevchenko