Re: [PATCH v5 0/3] iio: imu: inv_icm42600: add support for icm42630

Jonathan Cameron <[email protected]>
Newsgroups org.kernel.vger.linux-iio,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID <20260823184211.1bc951b1@jic23-huawei>
On Sun, 23 Aug 2026 02:33:17 +0200
Jorijn van der Graaf <[email protected]> wrote:

> The TDK InvenSense ICM-42630 is a 6-axis IMU of the ICM-42600 family,
> used in the Fairphone 6. It needs no part-specific driver code: TDK
> confirmed it can be supported by inv_icm42600 without modification.
> 
> The binding documents the new compatible with an invensense,icm42631
> fallback, and a WHOAMI mismatch no longer fails probe, so that a
> fallback-matched part can probe; one informational line records the
> mismatch, and a whoami of 0x00/0xff (no response) still fails. No
> driver id entries are added: the part binds through the fallback.
> 
> Tested on the Fairphone 6 with the fallback pair in the devicetree,
> via a backport to its 7.1-based kernel: matched as icm42631 through
> the fallback, the part probes with the one informational line, and
> accelerometer, gyroscope and temperature reads work.
> 
> Changes in v5:
> - patch 2: the no-response guard moves before the whoami-mismatch
>   comparison as its own check (Andy); it does not depend on
>   hw->whoami, since no supported part uses either value
> - patch 2: the four Link trailers are dropped (Andy, Jonathan); the
>   Suggested-by tags carry the attribution
> 
> v4: https://lore.kernel.org/all/[email protected]/
> v3: https://lore.kernel.org/all/[email protected]/
> v2: https://lore.kernel.org/all/[email protected]/
> v1: https://lore.kernel.org/all/[email protected]/
> 
> Jorijn van der Graaf (3):
>   iio: imu: inv_icm42600: sort device id tables numerically
>   iio: imu: inv_icm42600: log whoami mismatch instead of failing probe
>   dt-bindings: iio: imu: icm42600: add icm42630

Applied to the testing branch of iio.git.  Note I'll be rebasing on rc1 once
available (in about 1 week) so until then I'm fine adding tags or with changes
on top if any review feedback comes in.

Thanks,

Jonathan

> 
>  .../bindings/iio/imu/invensense,icm42600.yaml | 24 +++++++++++--------
>  .../iio/imu/inv_icm42600/inv_icm42600_core.c  | 17 ++++++++++---
>  .../iio/imu/inv_icm42600/inv_icm42600_i2c.c   | 16 ++++++-------
>  .../iio/imu/inv_icm42600/inv_icm42600_spi.c   | 16 ++++++-------
>  4 files changed, 44 insertions(+), 29 deletions(-)
> 
> 
> base-commit: 350d1fb9204b13c5f95e511e98b8bcb47574d425
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.