On Sat, 1 May 2021 22:14:16 +0300 Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > 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> Thanks! Applied to the fixes-togreg branch of iio.git and marked for stable. For these two I decided to go the quick way as they are broken even on systems where the alignment issue doesn't matter. Thanks, Jonathan > > > 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 > > > >
