On Sun, Mar 20, 2022 at 8:14 PM Marek Vasut <marex@xxxxxxx> wrote: > > Instead of storing only data_rate in private data, store pointer to the > whole chip data and use the data_rate from chip data throughout the driver. > No functional change. This is done in preparation for switch to read_avail(). switching ... > if (period <= ads1015_comp_queue[i] * > - USEC_PER_SEC / data->data_rate[dr]) > + USEC_PER_SEC / data_rate[dr]) I would put these two to one line. -- With Best Regards, Andy Shevchenko