On Sat, May 1, 2021 at 7:55 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > I finally got around to do a manual audit of all the calls to > iio_push_to_buffers_with_timestamp() which has the somewhat odd requirements > of: > 1. 8 byte alignment of the provided buffer. > 2. space for an 8 byte naturally aligned timestamp to be inserted at the > end. Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > As discussed previous in > https://lore.kernel.org/linux-iio/20200920112742.170751-1-jic23@xxxxxxxxxx/ > it is not easy to fix the alignment issue without requiring a bounce buffer > (see part 4 of the alignment fixes for a proposal for that where it is > absolutely necessary). > > In these cases the buffer is neither big enough, nor correctly aligned > so fix both issues in one go. > > Cc: Daniel Junho <djunho@xxxxxxxxx> > > Jonathan Cameron (2): > iio: adc: ad7768-1: Fix too small buffer passed to > iio_push_to_buffers_with_timestamp() > iio: adc: ad7923: Fix undersized rx buffer. > > drivers/iio/adc/ad7768-1.c | 8 ++++++-- > drivers/iio/adc/ad7923.c | 4 +++- > 2 files changed, 9 insertions(+), 3 deletions(-) > > -- > 2.31.1 > -- With Best Regards, Andy Shevchenko