fri3d_2026: remove CH32 workaround for version read, fix version read after install

ThomasFarstrike · ThomasFarstrike · commit 50232801b695 · 2026-04-30T10:39:27.000+02:00
diff --git a/internal_filesystem/lib/drivers/fri3d/expander.py b/internal_filesystem/lib/drivers/fri3d/expander.py
@@ -137,12 +137,6 @@ def install_firmware_if_needed(
         except Exception as e:
             print("Could not check CH32 firmware version, assuming 0.0.0")
             current_version = (0, 0, 0)
-        if current_version == (0, 255, 15):
-            warning_cb(msg="CH32 firmware version (0,255,15) bug, trying re-read...")
-            current_version = self.version # re-read seems to work (often?)
-            if current_version == (0, 255, 15):
-                warning_cb(msg="CH32 firmware version (0,255,15) bug after re-read, skipping firmware install/update", sleep_ms=4000, r=96, g=21, b=21)
-                return False
         if latest_version <= current_version:
             print(f"CH32 firmware {latest_version} <= {current_version} so not updating it")
             return False
@@ -152,11 +146,4 @@ def install_firmware_if_needed(
             success_cb()
         except Exception as e:
             failure_cb(e)
-            return True # re-initialize expander_i2c, even if the install failed, just in case
-        print("Firmware installed, checking new version")
-        try:
-            current_version = self.version
-            print(f"After install, new current_version of CH32 firmware: {current_version}")
-        except Exception as e:
-            print("Could not check CH32 firmware version after install, many things, including LCD RESET, won't work!")
-        return True
+        return True # re-initialize expander_i2c, even if the install failed, just in case
diff --git a/internal_filesystem/lib/mpos/board/fri3d_2026.py b/internal_filesystem/lib/mpos/board/fri3d_2026.py
@@ -121,6 +121,10 @@ def failure(e):
     print("Re-initializing expander_i2c")
     expander_i2c = I2C(1, sda=Pin(39), scl=Pin(42), freq=400000)
     expander = Expander(i2c_bus=expander_i2c)
+    try:
+        print(f"CH32 coprocessor firmware version is now: {expander.version}")
+    except Exception as e:
+        print("Could not re-check CH32 firmware version. Many things, including LCD RESET, might not work!")
 
 # Make expander accessible later
 import mpos
