[Bug] Fix pmw33xx sensor corruption on get-cpi call (#23116)

qmk · Feb 20, 2024 · 6678078 · 6678078
1 parent 6efb71d
commit 6678078
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/drivers/sensors/pmw3360.c b/drivers/sensors/pmw3360.c
@@ -15,6 +15,9 @@ uint16_t pmw33xx_get_cpi(uint8_t sensor) {
     }
 
     uint8_t cpival = pmw33xx_read(sensor, REG_Config1);
+    // In some cases (100, 900, 1700, 2500), reading the CPI corrupts the firmware and the sensor stops responding.
+    // To avoid this, we write the value back to the sensor, which seems to prevent the corruption.
+    pmw33xx_write(sensor, REG_Config1, cpival);
     return (uint16_t)((cpival + 1) & 0xFF) * PMW33XX_CPI_STEP;
 }