[PATCH v4] iio: imu: bmi270: Match PNP ID found on newer GPD firmware

[PATCH v4] iio: imu: bmi270: Match PNP ID found on newer GPD firmware

[Cryolitia PukNgae via B4 Relay]

From: Cryolitia PukNgae <cryolitia@uniontech.com>

GPD devices originally used BMI160 sensors with the "BMI0160" PNP ID.
When they switched to BMI260 sensors in newer hardware, they reused
the existing Windows driver which accepts both "BMI0160" and "BMI0260"
IDs. Consequently, they kept "BMI0160" in DSDT tables for new BMI260
devices, causing driver mismatches in Linux.

1. GPD updated BIOS v0.40+[1] for newer devices to report "BMI0260" for
BMI260 sensors to avoid loading the bmi160 driver on Linux. While this
isn't Bosch's VID;
2. Bosch's official Windows driver uses "BMI0260" as a compatible ID
3. We're seeing real devices shipping with "BMI0260" in DSDT

The DSDT excerpt of GPD G1619-04 with BIOS v0.40:

Scope (_SB.I2CC)
{
    Device (BMA2)
    {
        Name (_ADR, Zero)  // _ADR: Address
        Name (_HID, "BMI0260")  // _HID: Hardware ID
        Name (_CID, "BMI0260")  // _CID: Compatible ID
        Name (_DDN, "Accelerometer")  // _DDN: DOS Device Name
        Name (_UID, One)  // _UID: Unique ID
        Method (_CRS, 0, NotSerialized)  // _CRS: Current Resource Settings
        {
            Name (RBUF, ResourceTemplate ()
            {
                I2cSerialBusV2 (0x0069, ControllerInitiated, 0x00061A80,
                    AddressingMode7Bit, "\\_SB.I2CC",
                    0x00, ResourceConsumer, , Exclusive,
                    )
            })
            Return (RBUF) /* \_SB_.I2CC.BMA2._CRS.RBUF */
        }
        # omit some noise
    }
}

Link: http://download.softwincn.com/WIN%20Max%202024/Max2-7840-BIOS-V0.41.zip #1
Signed-off-by: Cryolitia PukNgae <cryolitia@uniontech.com>
---
Changes in v4:
- Improve commit message
- Link to v3: https://lore.kernel.org/r/20250821-bmi270-gpd-acpi-v3-1-91a873cb87c0@uniontech.com

Changes in v3:
- Improve commit message
- Link to v2: https://lore.kernel.org/r/20250815-bmi270-gpd-acpi-v2-1-8e1db68d36d1@uniontech.com

Changes in v2:
- Improve commit message
- Add DSDT table
- Link to v1: https://lore.kernel.org/r/20250730-bmi270-gpd-acpi-v1-1-1ffc85b17266@uniontech.com
---
 drivers/iio/imu/bmi270/bmi270_i2c.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/iio/imu/bmi270/bmi270_i2c.c b/drivers/iio/imu/bmi270/bmi270_i2c.c
index c77839b03a969f6f149c025a0305c4b9b8ac6571..b909a421ad0176ee414f2f96ff09db2297586ded 100644
--- a/drivers/iio/imu/bmi270/bmi270_i2c.c
+++ b/drivers/iio/imu/bmi270/bmi270_i2c.c
@@ -41,6 +41,8 @@ static const struct i2c_device_id bmi270_i2c_id[] = {
 static const struct acpi_device_id bmi270_acpi_match[] = {
 	/* GPD Win Mini, Aya Neo AIR Pro, OXP Mini Pro, etc. */
 	{ "BMI0160",  (kernel_ulong_t)&bmi260_chip_info },
+	/* GPD Win Max 2 2023(sincice BIOS v0.40), etc. */
+	{ "BMI0260",  (kernel_ulong_t)&bmi260_chip_info },
 	{ }
 };
 

---
base-commit: 24ea63ea387714634813359e2c8e0e6c36952f73
change-id: 20250206-bmi270-gpd-acpi-de4d12bce567

Best regards,
-- 
Cryolitia PukNgae <cryolitia@uniontech.com>

Re: [PATCH v4] iio: imu: bmi270: Match PNP ID found on newer GPD firmware

[Andy Shevchenko]

On Thu, Aug 21, 2025 at 10:52 AM Cryolitia PukNgae via B4 Relay
<devnull+cryolitia.uniontech.com@kernel.org> wrote:
>
> From: Cryolitia PukNgae <cryolitia@uniontech.com>
>
> GPD devices originally used BMI160 sensors with the "BMI0160" PNP ID.
> When they switched to BMI260 sensors in newer hardware, they reused
> the existing Windows driver which accepts both "BMI0160" and "BMI0260"
> IDs. Consequently, they kept "BMI0160" in DSDT tables for new BMI260
> devices, causing driver mismatches in Linux.
>
> 1. GPD updated BIOS v0.40+[1] for newer devices to report "BMI0260" for
> BMI260 sensors to avoid loading the bmi160 driver on Linux. While this
> isn't Bosch's VID;
> 2. Bosch's official Windows driver uses "BMI0260" as a compatible ID
> 3. We're seeing real devices shipping with "BMI0260" in DSDT
>
> The DSDT excerpt of GPD G1619-04 with BIOS v0.40:
>
> Scope (_SB.I2CC)
> {
>     Device (BMA2)
>     {
>         Name (_ADR, Zero)  // _ADR: Address
>         Name (_HID, "BMI0260")  // _HID: Hardware ID
>         Name (_CID, "BMI0260")  // _CID: Compatible ID
>         Name (_DDN, "Accelerometer")  // _DDN: DOS Device Name
>         Name (_UID, One)  // _UID: Unique ID
>         Method (_CRS, 0, NotSerialized)  // _CRS: Current Resource Settings
>         {
>             Name (RBUF, ResourceTemplate ()
>             {
>                 I2cSerialBusV2 (0x0069, ControllerInitiated, 0x00061A80,
>                     AddressingMode7Bit, "\\_SB.I2CC",
>                     0x00, ResourceConsumer, , Exclusive,
>                     )
>             })
>             Return (RBUF) /* \_SB_.I2CC.BMA2._CRS.RBUF */
>         }
>         # omit some noise
>     }
> }

Reviewed-by: Andy Shevchenko <andy@kernel.org>

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v4] iio: imu: bmi270: Match PNP ID found on newer GPD firmware

[Jonathan Cameron]

On Thu, 21 Aug 2025 11:10:15 +0300
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:

> On Thu, Aug 21, 2025 at 10:52 AM Cryolitia PukNgae via B4 Relay
> <devnull+cryolitia.uniontech.com@kernel.org> wrote:
> >
> > From: Cryolitia PukNgae <cryolitia@uniontech.com>
> >
> > GPD devices originally used BMI160 sensors with the "BMI0160" PNP ID.
> > When they switched to BMI260 sensors in newer hardware, they reused
> > the existing Windows driver which accepts both "BMI0160" and "BMI0260"
> > IDs. Consequently, they kept "BMI0160" in DSDT tables for new BMI260
> > devices, causing driver mismatches in Linux.
> >
> > 1. GPD updated BIOS v0.40+[1] for newer devices to report "BMI0260" for
> > BMI260 sensors to avoid loading the bmi160 driver on Linux. While this
> > isn't Bosch's VID;
> > 2. Bosch's official Windows driver uses "BMI0260" as a compatible ID
> > 3. We're seeing real devices shipping with "BMI0260" in DSDT
> >
> > The DSDT excerpt of GPD G1619-04 with BIOS v0.40:
> >
> > Scope (_SB.I2CC)
> > {
> >     Device (BMA2)
> >     {
> >         Name (_ADR, Zero)  // _ADR: Address
> >         Name (_HID, "BMI0260")  // _HID: Hardware ID
> >         Name (_CID, "BMI0260")  // _CID: Compatible ID
> >         Name (_DDN, "Accelerometer")  // _DDN: DOS Device Name
> >         Name (_UID, One)  // _UID: Unique ID
> >         Method (_CRS, 0, NotSerialized)  // _CRS: Current Resource Settings
> >         {
> >             Name (RBUF, ResourceTemplate ()
> >             {
> >                 I2cSerialBusV2 (0x0069, ControllerInitiated, 0x00061A80,
> >                     AddressingMode7Bit, "\\_SB.I2CC",
> >                     0x00, ResourceConsumer, , Exclusive,
> >                     )
> >             })
> >             Return (RBUF) /* \_SB_.I2CC.BMA2._CRS.RBUF */
> >         }
> >         # omit some noise
> >     }
> > }  
> 
> Reviewed-by: Andy Shevchenko <andy@kernel.org>
> 

Applied.

Thanks,