Better handling of LEDs on Pi 5.

Author: Gadgetoid

inventorhatmini/__init__.py

@@ -1,10 +1,12 @@
 #!/usr/bin/env python3
 
 import time
+import warnings
 
 import gpiod
 import gpiodevice
 from gpiod.line import Bias, Direction, Value
+from gpiodevice import platform
 from ioexpander import ADC, SuperIOE
 from ioexpander.common import NORMAL_DIR
 from ioexpander.encoder import MMME_CPR, ROTARY_CPR, Encoder
@@ -111,13 +113,18 @@ def __init__(self, address=IOE_ADDRESS, motor_gear_ratio=50, init_motors=True, i
         self.__init_servos = init_servos
         self.reinit()
 
-        if init_leds:
+        is_pi5 = platform.get_name().starts_with("Raspberry Pi 5")
+
+        if init_leds and not is_pi5:
             # Setup the PixelStrip object to use with Inventor's LEDs, wrapped in a Plasma class
             self.leds = Plasma(NUM_LEDS, self.PI_LED_DATA_PIN)
         else:
             # Setup a dummy Plasma class, so examples don't need to check LED presence
             self.leds = DummyPlasma()
 
+        if is_pi5:
+            warnings.warn("LEDs are not yet supported on Pi 5.")
+
     def _write_pin(self, pin, state):
         lines, offset = pin
         lines.set_value(offset, Value.ACTIVE if state else Value.INACTIVE)

inventorhatmini/plasma.py

@@ -1,5 +1,7 @@
 from colorsys import hsv_to_rgb
 
+from rpi_ws281x import PixelStrip
+
 from inventorhatmini.errors import LED_INIT_FAILED
 
 
@@ -28,12 +30,6 @@ class Plasma():
         222, 224, 227, 229, 231, 233, 235, 237, 239, 241, 244, 246, 248, 250, 252, 255]
 
     def __init__(self, num_leds, pin):
-        try:
-            from rpi_ws281x import PixelStrip
-        except ImportError:
-            self = DummyPlasma()
-            return
-
         # Setup the PixelStrip object to use with Inventor's LEDs
         self.leds = PixelStrip(num_leds, pin, self.LED_FREQ_HZ, self.LED_DMA, self.LED_INVERT, self.LED_BRIGHTNESS, self.LED_CHANNEL, self.LED_GAMMA)
         try:

pyproject.toml

@@ -37,11 +37,7 @@ classifiers = [
 ]
 dependencies = [
     "pimoroni-ioexpander>=1.0.0",
-    "smbus2"
-]
-
-[project.optional-dependencies]
-leds = [
+    "smbus2",
     "rpi_ws281x"
 ]