[PATCH AUTOSEL 6.19-6.18] iio: bmi270_i2c: Add MODULE_DEVICE_TABLE for BMI260/270
From: Sasha Levin
Date: Wed Feb 18 2026 - 21:10:07 EST
From: "Derek J. Clark" <derekjohn.clark@xxxxxxxxx>
[ Upstream commit f69b5ac682dbc61e6aca806c22ce2ae74d598e45 ]
Currently BMI260 & BMI270 devices do not automatically load this
driver. To fix this, add missing MODULE_DEVICE_TABLE for the i2c,
acpi, and of device tables so the driver will load when the hardware
is detected.
Tested on my OneXPlayer F1 Pro.
Signed-off-by: Derek J. Clark <derekjohn.clark@xxxxxxxxx>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
---
LLM Generated explanations, may be completely bogus:
So:
- v6.13: Driver introduced with BMI260 support (no MODULE_DEVICE_TABLE)
- v6.14-v6.16: No changes to this file
- v6.17: Added suspend/resume pm_ops (affects the `.pm = pm_ptr(...)`
line in the driver struct)
- v6.18: Added BMI0260 ACPI ID
The MODULE_DEVICE_TABLE additions are simple line additions between
existing code, so they should apply cleanly to v6.13+ (possibly with
minor context adjustments for v6.17+ changes and v6.18+ ACPI table
differences). A minor backport adaptation may be needed for older stable
trees that don't have the BMI0260 ACPI entry or the pm_ptr line, but
these are trivial context issues.
### Summary
**What it fixes**: BMI260/BMI270 I2C driver doesn't autoload when
hardware is detected because `MODULE_DEVICE_TABLE` macros were never
added. Users must manually load the module.
**Why it matters**: This is a device enablement bug affecting real
consumer hardware (handheld gaming PCs like GPD Win Mini, Aya Neo, OXP
Mini Pro, OneXPlayer F1 Pro). The driver exists but doesn't work without
manual intervention.
**Risk**: Essentially zero. `MODULE_DEVICE_TABLE` is a compile-time-only
construct that generates module alias information. It cannot cause
runtime regressions.
**Scope**: 3 lines added in 1 file. Minimal, well-understood change.
**Stable criteria met**:
- Obviously correct (trivially so)
- Fixes a real bug (driver doesn't autoload)
- Small and contained
- No new features
- Tested on real hardware
- Reviewed by respected kernel developer
### Verification
- `git log --diff-filter=A -- drivers/iio/imu/bmi270/bmi270_i2c.c`
confirmed driver was introduced in commit `3ea51548d6b25` (v6.13)
- `git show 3ea51548d6b25` confirmed the original driver never had
`MODULE_DEVICE_TABLE` — this has been a bug since day one
- `git show f35f3c832eb58` confirmed BMI260 support (with ACPI and OF
tables) was added in v6.13
- `git tag --contains 3ea51548d6b25` confirmed the driver is in v6.13+
- `git tag --contains dc757dc1572d5` confirmed the BMI0260 ACPI ID is
only in v6.18+, so the backport may need minor context adjustment for
older trees
- `git log --oneline v6.16..v6.17 --
drivers/iio/imu/bmi270/bmi270_i2c.c` confirmed suspend/resume was
added in v6.17, which affects context near the driver struct
- The change is purely additive (3 macro invocations), verified by
reading the diff
**YES**
drivers/iio/imu/bmi270/bmi270_i2c.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/iio/imu/bmi270/bmi270_i2c.c b/drivers/iio/imu/bmi270/bmi270_i2c.c
index b909a421ad017..b92da4e0776fa 100644
--- a/drivers/iio/imu/bmi270/bmi270_i2c.c
+++ b/drivers/iio/imu/bmi270/bmi270_i2c.c
@@ -37,6 +37,7 @@ static const struct i2c_device_id bmi270_i2c_id[] = {
{ "bmi270", (kernel_ulong_t)&bmi270_chip_info },
{ }
};
+MODULE_DEVICE_TABLE(i2c, bmi270_i2c_id);
static const struct acpi_device_id bmi270_acpi_match[] = {
/* GPD Win Mini, Aya Neo AIR Pro, OXP Mini Pro, etc. */
@@ -45,12 +46,14 @@ static const struct acpi_device_id bmi270_acpi_match[] = {
{ "BMI0260", (kernel_ulong_t)&bmi260_chip_info },
{ }
};
+MODULE_DEVICE_TABLE(acpi, bmi270_acpi_match);
static const struct of_device_id bmi270_of_match[] = {
{ .compatible = "bosch,bmi260", .data = &bmi260_chip_info },
{ .compatible = "bosch,bmi270", .data = &bmi270_chip_info },
{ }
};
+MODULE_DEVICE_TABLE(of, bmi270_of_match);
static struct i2c_driver bmi270_i2c_driver = {
.driver = {
--
2.51.0