device: limited support for extended adjustable dpi

lib/logitech_receiver/settings_templates.py

@@ -756,7 +756,7 @@ def build(cls, setting_class, device):
 
 class DpiSlidingXY(_RawXYProcessing):
     def activate_action(self):
-        self.dpiSetting = next(filter(lambda s: s.name == "dpi", self.device.settings), None)
+        self.dpiSetting = next(filter(lambda s: s.name == "dpi" or s.name == "dpi_extended", self.device.settings), None)
         self.dpiChoices = list(self.dpiSetting.choices)
         self.otherDpiIdx = self.device.persister.get("_dpi-sliding", -1) if self.device.persister else -1
         if not isinstance(self.otherDpiIdx, int) or self.otherDpiIdx < 0 or self.otherDpiIdx >= len(self.dpiChoices):
@@ -819,7 +819,7 @@ def move_action(self, dx, dy):
 
 class MouseGesturesXY(_RawXYProcessing):
     def activate_action(self):
-        self.dpiSetting = next(filter(lambda s: s.name == "dpi", self.device.settings), None)
+        self.dpiSetting = next(filter(lambda s: s.name == "dpi" or s.name == "dpi_extended", self.device.settings), None)
         self.fsmState = "idle"
         self.initialize_data()
 
@@ -962,27 +962,52 @@ class AdjustableDpi(_Setting):
     feature = _F.ADJUSTABLE_DPI
     rw_options = {"read_fnid": 0x20, "write_fnid": 0x30}
     choices_universe = _NamedInts.range(200, 4000, str, 50)
+    sensor_list_function = 0x10
+    sensor_list_bytes_ignore = 1
 
     class validator_class(_ChoicesV):
         @classmethod
         def build(cls, setting_class, device):
-            # [1] getSensorDpiList(sensorIdx)
-            reply = device.feature_request(_F.ADJUSTABLE_DPI, 0x10)
+            y = False
+            if setting_class.feature == _F.EXTENDED_ADJUSTABLE_DPI:
+                reply = device.feature_request(setting_class.feature, 0x10, 0x00)
+                y = reply[2] & 0x01
+            reply = device.feature_request(setting_class.feature, setting_class.sensor_list_function, 0x00, 0x00, 0x00)
             assert reply, "Oops, DPI list cannot be retrieved!"
+            dpi_bytes = reply[setting_class.sensor_list_bytes_ignore :]
+            i = 1
+            while _bytes2int(dpi_bytes[-2:]) != 0:
+                reply = device.feature_request(setting_class.feature, setting_class.sensor_list_function, 0x00, 0x00, i)
+                assert reply, "Oops, DPI list cannot be retrieved!"
+                dpi_bytes += reply[setting_class.sensor_list_bytes_ignore :]
+                i += 1
             dpi_list = []
-            step = None
-            for val in _unpack("!7H", reply[1 : 1 + 14]):
+            i = 0
+            while i < len(dpi_bytes):
+                val = _bytes2int(dpi_bytes[i : i + 2])
                 if val == 0:
                     break
                 if val >> 13 == 0b111:
-                    assert step is None and len(dpi_list) == 1, f"Invalid DPI list item: {val!r}"
                     step = val & 0x1FFF
+                    last = _bytes2int(dpi_bytes[i + 2 : i + 4])
+                    assert len(dpi_list) > 0 and last > dpi_list[-1], f"Invalid DPI list item: {val!r}"
+                    dpi_list += range(dpi_list[-1] + step, last + 1, step)
+                    i += 4
                 else:
                     dpi_list.append(val)
-            if step:
-                assert len(dpi_list) == 2, f"Invalid DPI list range: {dpi_list!r}"
-                dpi_list = range(dpi_list[0], dpi_list[1] + 1, step)
-            return cls(choices=_NamedInts.list(dpi_list), byte_count=3) if dpi_list else None
+                    i += 2
+            setting = cls(choices=_NamedInts.list(dpi_list), byte_count=2, write_prefix_bytes=b"\x00") if dpi_list else None
+            setting.y = y
+            return setting
+
+        def prepare_write(self, new_value, current_value=None):
+            data_bytes = super().prepare_write(new_value, current_value)
+            print("PW", data_bytes)
+            if self.y:
+                bytes = data_bytes[len(self._write_prefix_bytes) :]
+                data_bytes = self._write_prefix_bytes + bytes + bytes
+                print("PW", len(self._write_prefix_bytes), bytes, data_bytes)
+            return data_bytes
 
         def validate_read(self, reply_bytes):  # special validator to use default DPI if needed
             reply_value = _bytes2int(reply_bytes[1:3])
@@ -993,6 +1018,18 @@ def validate_read(self, reply_bytes):  # special validator to use default DPI if
             return valid_value
 
 
+class ExtendedAdjustableDpi(AdjustableDpi):
+    # the extended version allows for two dimensions, longer dpi descriptions
+    # still assume only one sensor (and X only?)
+    # [5] getSensorDpiParameters(sensorIdx) → sensorIdx, dpiX, defaultDpiX, dpiY, defaultDpiY, lod
+    # [6] setSensorDpiParameters(sensorIdx, dpiX, dpiY, lod) → sensorIdx, dpiX, dpiY, lod
+    name = "dpi_extended"
+    feature = _F.EXTENDED_ADJUSTABLE_DPI
+    rw_options = {"read_fnid": 0x50, "write_fnid": 0x60}
+    sensor_list_function = 0x20
+    sensor_list_bytes_ignore = 3
+
+
 class SpeedChange(_Setting):
     """Implements the ability to switch Sensitivity by clicking on the DPI_Change button."""
 
@@ -1594,6 +1631,7 @@ def build(cls, device):
     ExtendedReportRate,
     PointerSpeed,  # simple
     AdjustableDpi,  # working
+    ExtendedAdjustableDpi,
     SpeedChange,
     #    Backlight,  # not working - disabled temporarily
     Backlight2,  # working