Re: [PATCH v2] iio: accel: bmc150: use IIO_DECLARE_BUFFER_WITH_TS
Jonathan Cameron <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <20260812055202.1fcc271e@jic23-huawei> |
On Mon, 10 Aug 2026 12:00:05 +0300 Andy Shevchenko <[email protected]> wrote: > On Mon, Aug 10, 2026 at 12:18:52AM +0100, Jonathan Cameron wrote: > > On Sun, 9 Aug 2026 23:59:34 +0100 > > Jonathan Cameron <[email protected]> wrote: > > > On Sun, 9 Aug 2026 11:24:42 +0530 > > > Yash Suthar <[email protected]> wrote: > > ... > > > Actually no. I've backed that out. > > > > Given the data alignment is fixed, if we were going to do this it would > > be clearer as > > struct { > > s16 chans[3]; //see later, I believe this should be __le16 > > aligned_s64 timestamp; > > }; > > > > But that is very similar to the structure that follows immediately after > > this, but that has __le16 chans[] > > > > So what is going on here? > > AI slop? It's the original code I'm complaining about. Been around a while! So for once, nope (I think!) Jonathan >