On Sat, May 1, 2021 at 7:03 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > To make code more readable, use a structure to express the channel > layout and ensure the timestamp is 8 byte aligned. > > Found during an audit of all calls of uses of > iio_push_to_buffers_with_timestamp() > > Fixes: c91746a2361d ("iio: magn: Add support for BMC150 magnetometer") > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > Cc: Stephan Gerhold <stephan@xxxxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> Excellent work Jonathan. Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> I wonder if there is some way for us to abstract this into the core so we can't get it wrong. Yours, Linus Walleij