Re: [PATCH v3 4/8] iio: imu: inv_icm42607: Switch to little endian
From: Kanak Shilledar
Date: Wed Sep 16 2026 - 05:35:43 EST
Hi Jonathan,
On Mon, 2026-09-07 at 02:54 +0100, Jonathan Cameron wrote:
> > The device supports both big and little endian formats for sensor
> > data,
> > this is handled via the SENSOR_DATA_BIT in INTF_CONFIG0 register.
> > The
> > default value is big-endian format.
> >
> > Switch to little-endian as it matches the native byte order on vast
> > majority of systems and avoids unnecessary byte-swapping.
>
> This is an ABI change. Whilst in theory they are self describing and
> no one should be assuming they are stable, in practice people do.
>
> So we need a stronger reason than it saves a few byte swaps.
> If there is nothing else leave the ABI alone.
I will drop this patch in v4.
Thanks and Regards,
Kanak Shilledar
Attachment:
signature.asc
Description: This is a digitally signed message part