Re: [PATCH v3 2/2] iio: imu: st_lsm6dsx: Add ACPI ID SMOCF05 for LSM6DS3TR-C

[Andy Shevchenko <andriy.shevchenko@intel.com>]

On Sun, Sep 27, 2026 at 05:05:59PM -0500, Jared Wuerzburger wrote:

> The Minisforum V3 and V3 SE tablets declare their ST LSM6DS3TR-C IMU in
> the DSDT with the hardware ID SMOCF05 instead of SMO8B30, so the driver
> never binds and the accelerometer and gyroscope are missing. The
> Windows driver (version 1.0.4.3) binds the LSM6DS3TR-C to both IDs.
> 
> Relevant part of the DSDT, identical on both models (full excerpt in
> the cover letter):

Don't refer to the cover letter in the commit message. Most will not understand
this pointer.

>   Scope (_SB.I2CD) { Device (STS) {
>     Name (_HID, EisaId ("SMOCF05"))
>     Name (_CID, EisaId ("SMOCF05"))
>     _CRS: I2cSerialBusV2 (0x006A, ..., "\\_SB.I2CD")  vendor data "SLA0"
>           I2cSerialBusV2 (0x006A, ..., "\\_SB.I2CD")  vendor data "SLG0"
>           GpioInt (Edge, ActiveHigh, ..., "\\_SB.GPIO") { 9 }  "SLA0"
>     Method (SLA0): "-1 0 0", "0 -1 0", "0 0 -1"
>     Method (SLG0): "1 0 0", "0 1 0", "0 0 1"
>   } }

Just ask maintainer to add a Link tag to the series instead.

> Both I2C resources describe the same address, and the driver uses the
> first one. The vendor-specific SLA0/SLG0 mount matrices are not parsed
> here; the accelerometer orientation is handled in userspace through
> the systemd hwdb, as for SMOCF00.

Hmm... But is that wrong? I mean does ACPI provide the correct data in mount
matrix?

> Tested on:
>  - V3 SE (Ryzen 7 7735U, BIOS 1.03) with this change.
>  - V3 (Ryzen 7 8840U, BIOS 1.06), kernel 7.2.7, with this change and
>    the stock firmware tables. The driver binds to i2c-SMOCF05:00, the
>    WHO_AM_I check passes (0x6a), and both lsm6ds3tr-c_accel and
>    lsm6ds3tr-c_gyro are registered. Screen auto-rotation works with
>    iio-sensor-proxy (checked with the device renamed to SMO8B30 via a
>    DSDT override, i.e. the same ST_LSM6DS3TRC_ID driver data).
> 
> Link: https://github.com/mudkipme/awesome-minisforum-v3/issues/2

Closes?

> Signed-off-by: Samuel Dionne-Riel <samuel@dionne-riel.com>
> [Jared Wuerzburger: rewrote the changelog with the DSDT excerpt and test
>  results, dropped the mount matrix patch from the series]
> Assisted-by: Claude:claude-opus-5-5

Assisted-by: LLM

> Signed-off-by: Jared Wuerzburger <jwuerz@gmail.com>

-- 
With Best Regards,
Andy Shevchenko