Re: [PATCH v3] iio: imu: bno055: add explicit scan buf layout

From: Andy Shevchenko

Date: Mon Mar 16 2026 - 11:08:32 EST


On Sat, Mar 14, 2026 at 04:38:25PM -0500, David Lechner wrote:
> Move the scan buf.chans array into a union along with a struct that
> gives the layout of the buffer with all channels enabled.
>
> Although not technically required in this case, if there had been a
> different number of items before the quaternion, there could have been
> a subtle bug with the special alignment needed for the quaternion
> channel data and the array would have been too small.

This looks good to me,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>

--
With Best Regards,
Andy Shevchenko