denko 0.13.2 → 0.13.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1cefcc0106edf784dea04c7217bf8475887402da89aacf0c9b578185ee4c152
-  data.tar.gz: '08b1bcaac7973a3eb5b2fbbae7b7227cfbdec9250dffe9d41a95a1fe0a90565b'
+  metadata.gz: '082654ef6ccaec931c9cba643db7c528cc8b20d33e969af6a101a735078b7c02'
+  data.tar.gz: 780397a81acfa1a44591deb0c1beedb6f39071b606fb108914c79af334e47847
 SHA512:
-  metadata.gz: a0344465f58fe0e23fb579d7e903a93ecaf49a923270991c471caf674a34bf5fece80e56598878a2e690966461d2f08db15b49eccd58c09de7d2140803f2f77e
-  data.tar.gz: 7395727636ea674940fd84a7fdf55b61a32dde2329cf2f561aea7e73ea177098e8346ea7fd2a5ea0b4d6f035a0b88d2e4386ba33a3356966b9afe4322a53912c
+  metadata.gz: 58e7ec2753368aa202a63792c6f142bd150a386c95d37605ee4c1985022d4c06e552108f504675c599effcde0e0438b7d1029d0c700ec7fee40fffdffc60d1f4
+  data.tar.gz: fec058bef713c8f5c7e84eed3dd87a84db93267e921abb27d3c4966b40bf66a223a3a76442559742bec04a58665e18d3d8d30cd3954fdb846148e5d1fb3b0cd2

data/.github/workflows/build_megaavr.yml

@@ -36,7 +36,7 @@ jobs:
               - name: IRremote
                 version: 4.1.2
               - name: Ethernet
-              - name: WiFi
+              - name: WiFiNINA
               - name: Adafruit NeoPixel
                 
     steps:

data/CHANGELOG.md

@@ -1,5 +1,45 @@
 # Changelog
 
+## 0.13.3
+
+### Board Updates / Fixes
+
+- Arduino UNO R4 Minima & Wi-Fi (`--target ra4m1`):
+  - RA4M1 fully tested
+  - Only IR-remote and WS2812 libraries not working. Disabled temporarily.
+
+- Arduino Nano Every
+  - ATmega4809 fully tested and issues fixed
+  - Works similarly to the regular AVR chips
+
+- SAMD21
+  - Changed serial acknowledgement threshold to 128 instead of 64. More stable, but might be slightly slower performing.
+
+### Network Chip Updates / Fixes
+
+- WINC1500
+  - Added support for WiFi101 library
+  - Tested on Arduino MKR 1000
+
+- NINA-W102
+  - Added conditions to select the WiFiNINA library on appropriate hardware.
+  - Untested in hardware
+
+### API Changes
+
+- Default aux message size is now  784 bytes (768 + 16) instead of (512 + 16)
+
+### Component Changes
+
+- WS2812
+  - Larger aux message size now allows up to 256 pixels (3 bytes per pixel) on a strip
+  
+### Bug Fixes
+
+- Fixed a bug with WS2812 strips where it would try to memcpy 3x the number of necessary bytes. Fixed this by just sending the total number of bytes to write to the strip, rather than number of pixels * bytes per pxiel.
+- The `val` variable in Arduino is now 16-bit instead of 8-bit.
+- Fixed instances where `Minitest` was referred to as `MiniTest`, causing tests to fail.
+
 ## 0.13.2
 
 ### New Boards

data/DEPS_CLI.md

@@ -21,6 +21,8 @@ arduino-cli core install rp2040:rp2040
 arduino-cli lib install Servo
 arduino-cli lib install Ethernet
 arduino-cli lib install WiFi
+arduino-cli lib install WiFi101
+arduino-cli lib install WiFiNINA
 arduino-cli lib install IRremote@4.1.2
 arduino-cli lib install IRremoteESP8266@2.8.4
 arduino-cli lib install ESP32Servo
@@ -34,6 +36,7 @@ arduino-cli core install arduino:megaavr
 arduino-cli lib install Servo
 arduino-cli lib install Ethernet
 arduino-cli lib install WiFi
+arduino-cli lib install WiFiNINA
 arduino-cli lib install IRremote@4.1.2
 arduino-cli lib install "Adafruit NeoPixel"
 ````
@@ -47,6 +50,8 @@ arduino-cli core install arduino:renesas_uno
 arduino-cli lib install Servo
 arduino-cli lib install Ethernet
 arduino-cli lib install WiFi
+arduino-cli lib install WiFi101
+arduino-cli lib install WiFiNINA
 arduino-cli lib install IRremote@4.1.2
 arduino-cli lib install "Adafruit NeoPixel"
 ````

data/DEPS_IDE.md

@@ -40,6 +40,8 @@ All platforms will require libraries to be installed. To install a library do th
     Servo                      by Michael Margolis, Arduino
     Ethernet                   by Various
     WiFi                       by Arduino
+    WiFi101                    by Arduino
+    WiFiNINA                   by Arduino
     IRremote         @ v4.1.2  by shirriff, z3to, ArminJo
     IRremoteESP82666 @ v2.8.5  by David Conran, Sebastien Warin
     ESP32Servo                 by Kevin Harrington, John K. Bennett
@@ -56,6 +58,7 @@ All platforms will require libraries to be installed. To install a library do th
     Servo                      by Michael Margolis, Arduino
     Ethernet                   by Various
     WiFi                       by Arduino
+    WiFiNINA                   by Arduino
     IRremote         @ v4.1.2  by shirriff, z3to, ArminJo
     Adafruit NeoPixel          by Adafruit
     ````
@@ -72,6 +75,8 @@ All platforms will require libraries to be installed. To install a library do th
     Servo                      by Michael Margolis, Arduino
     Ethernet                   by Various
     WiFi                       by Arduino
+    WiFi101                    by Arduino
+    WiFiNINA                   by Arduino
     IRremote         @ v4.1.2  by shirriff, z3to, ArminJo
     Adafruit NeoPixel          by Adafruit
     ````

data/HARDWARE.md

@@ -2,7 +2,7 @@
 
 :green_heart: Full support :yellow_heart: Partial support :heart: Planned. No support yet :question: Works in theory. Untested in real hardware.
 
-### AVR/MegaAVR Based in Arduino Products (and Clones)
+### AVR/MegaAVR Based Arduino Products (and Clones)
 [![AVR Build Status](https://github.com/denko-rb/denko/actions/workflows/build_avr.yml/badge.svg)](https://github.com/denko-rb/denko/actions/workflows/build_avr.yml)
 [![MegaAVR Build Status](https://github.com/denko-rb/denko/actions/workflows/build_megaavr.yml/badge.svg)](https://github.com/denko-rb/denko/actions/workflows/build_megaavr.yml)
 
@@ -13,7 +13,7 @@
 | ATmega32u4     | :green_heart:   | Leonardo, Micro, Leonardo ETH, Esplora, LilyPad USB |
 | ATmega1280     | :green_heart:   | Mega |
 | ATmega2560     | :green_heart:   | Mega2560, Arduino Mega ADK |
-| ATmega4809     | :question:      | Nano Every, Uno WiFi Rev2 | No hardware to test, but should work
+| ATmega4809     | :green_heart:   | Nano Every, Uno WiFi Rev2 |
 
 **Note:** Only USB boards listed. Any board with a supported chip should work, once you can flash it and connect to serial.
 
@@ -25,16 +25,17 @@
 |    Chip        | Status          | Products         | Notes |
 | :--------      | :------:        | :--------------- |------ |
 | ATSAM3X8E      | :yellow_heart:  | Due | Native USB port. Tone, and IR Out don't work.
-| ATSAMD21       | :green_heart:   | Zero, M0, M0 Pro | Native USB port
-| RA4M1          | :yellow_heart:  | Uno R4 Minima, Uno R4 WiFi | IR Out and WS2812 unsupported. UART & Wi-Fi untested
+| ATSAMD21       | :green_heart:   | Zero, M0 Series, Nano 33 IOT, MKR WiFi 1010 | Native USB
+| RA4M1          | :yellow_heart:  | Uno R4 Minima, Uno R4 WiFi | IR and WS2812 libraries don't support this yet
 
-### Arduino Accessories
+### Arduino Networking
 
-|    Chip               | Status          | Product          | Notes |
+|    Chip               | Status          | Products         | Notes |
 | :--------             | :------:        | :--------------- |------ |
 | Wiznet W5100/5500     | :green_heart:   | Ethernet Shield  | Wired Ethernet for Uno/Mega pin-compatibles
-| HDG204 + AT32UC3      | :question:      | WiFi Shield      | WiFi for Uno. No hardware to test, but compiles
-| ATWINC1500            | :question:      | WiFi Shield 101  | Same as above, high memory use, Mega only
+| HDG204 + AT32UC3      | :question:      | WiFi Shield      | Compiles, but no hardware
+| ATWINC1500            | :green_heart:   | MKR1000, WiFi Shield 101 | #define WIFI_101 for shield. Automatic for MKR1000
+| u-blox NINA-W102      | :question:      | Uno WiFi Rev 2, MKR WiFi 1010, Nano 33 IOT | Should work. No hardware
 
 ### Espressif Chips with Built-In Wi-Fi
 [![ESP8266 Build Status](https://github.com/denko-rb/denko/actions/workflows/build_esp8266.yml/badge.svg)](https://github.com/denko-rb/denko/actions/workflows/build_esp8266.yml)
@@ -90,12 +91,12 @@
 | I2C Bit Bang          | :heart:         | S     | `I2C::BitBang`           | Any pins
 | SPI                   | :green_heart:   | H     | `SPI::Bus`               | Predetermined pins from IDE
 | SPI Bit Bang          | :green_heart:   | S     | `SPI::BitBang`           | Any pins
-| UART                  | :green_heart:   | H     | `UART::Hardware`         | Except Atmega328, ATmega168, RA4M1
+| UART                  | :green_heart:   | H     | `UART::Hardware`         | Except Atmega328, ATmega168
 | UART Bit Bang         | :green_heart:   | S     | `UART::BitBang`          | Only ATmega328, ATmega168
 | Maxim OneWire         | :green_heart:   | S     | `OneWire::Bus`           | No overdrive
-| Infrared Emitter      | :green_heart:   | S     | `PulseIO::IRTransmitter` | Except RA4M1
+| Infrared Emitter      | :green_heart:   | S     | `PulseIO::IRTransmitter` | Except SAM3X, RA4M1
 | Infrared Receiver     | :heart:         | S     | `PulseIO::IRReceiver`    | Doable with existing library
-| WS2812                | :green_heart:   | S     | See LED table            | Except RP2040
+| WS2812                | :green_heart:   | S     | See LED table            | Except RP2040, RA4M1
 | ESP32-PCNT            | :heart:         | H     | -                        | Only ESP32. Pulse counter (for encoders)
 | ESP32-MCPWM           | :heart:         | H     | -                        | Only ESP32. Motor control PWM
 
@@ -141,9 +142,10 @@ Polling and reading follow a call and response pattern.
 | :---------------     | :------:       | :--------      | :---------------   |------ |
 | Generic Hobby Servo  | :green_heart:  | Servo/ESC PWM  | `Motor::Servo`     | Max depends on PWM channel count
 | Generic ESC          | :yellow_heart: | Servo/ESC PWM  | `Motor::Servo`     | Works. Needs its own class.
+| PCA9685              | :heart:        | I2C            | `PulseIO::PCA9685` | 16ch 12-bit PWM for servo or LED
 | L298N                | :green_heart:  | Digi + PWM Out | `Motor::L298`      | H-Bridge DC motor driver
 | A3967                | :green_heart:  | Digital Out    | `Motor::Stepper`   | 1ch microstepper (EasyDriver)
-| PCA9685              | :heart:        | I2C            | `PulseIO::PCA9685` | 16ch 12-bit PWM for servo or LED
+| TMC2209              | :heart:        | -              | -                  | 1ch silent stepper driver
 
 ### I/O Expansion
 
@@ -210,6 +212,12 @@ Polling and reading follow a call and response pattern.
 | DS1302           | :heart:        | I2C       | `RTC::DS1302`     |
 | DS3231           | :green_heart:  | I2C       | `RTC::DS3231`     | Alarms not implemented
 
+### GPS
+
+| Name             | Status         | Interface | Component Class   | Notes |
+| :--------------- | :------:       | :-------- | :---------------  |------ |
+| GT-U7            | :heart:        | UART      | -                 |
+
 ### Miscellaneous
 
 | Name             | Status         | Interface  | Component Class      | Notes |

data/README.md

@@ -1,4 +1,4 @@
-# Denko 0.13.1 [![Test Status](https://github.com/denko-rb/denko/actions/workflows/ruby.yml/badge.svg)](https://github.com/denko-rb/denko/actions/workflows/ruby.yml)
+# Denko 0.13.3 [![Test Status](https://github.com/denko-rb/denko/actions/workflows/ruby.yml/badge.svg)](https://github.com/denko-rb/denko/actions/workflows/ruby.yml)
 ### Ruby Meets Microcontrollers
 Denko gives you a high-level Ruby interface to low-level hardware, without writing microcontroller code. Use LEDs, buttons, sensors and more, just as easily as any Ruby object:
 

data/benchmarks/i2c_ssd1306_refresh.rb

@@ -22,8 +22,13 @@
 #
 # July 15 2023 | i7 8700k CPU | CRuby 3.2.1 | 100 frames | 1 Mhz I2C frequency | Average of 3 runs
 #
-#   Arduino Uno R4      :  7.5 fps (native USB @ 115200, 32-byte I2C limit, no I2C level shifter)
-#   Arduino Uno R4      :  7.8 fps (native USB @ 230400, 32-byte I2C limit, no I2C level shifter)
+#   Arduino Uno R4 WiFi :  7.5 fps (USB through ESP32-S3 @ 115200, 32-byte I2C limit, no I2C level shifter)
+#   Arduino Uno R4 WiFi :  7.8 fps (USB through ESP32-S3 @ 230400, 32-byte I2C limit, no I2C level shifter)
+#
+# August 13 2023 | i7 8700k CPU | CRuby 3.2.1 | 100 frames | 1 Mhz I2C frequency | Average of 3 runs
+#
+#   Arduino Nano Every  :  9.2 fps (USB through ATSAMD11 @ 115200, 128-byte I2C limit, no I2C level shifter)
+#   Arduino Nano Every  : 13.0 fps (USB through ATSAMD11 @ 230400, 128-byte I2C limit, no I2C level shifter)
 #
 require 'bundler/setup'
 require 'denko'

data/examples/pulse_io/buzzer.rb

@@ -7,7 +7,6 @@ require 'denko'
 board = Denko::Board.new(Denko::Connection::Serial.new)
 buzzer = Denko::PulseIO::Buzzer.new(board: board, pin: 9)
 
-=begin
 C4 = 262
 D4 = 294
 E4 = 330
@@ -29,42 +28,3 @@ end
 
 buzzer.stop
 board.finish_write
-=end
-
-
-
-AS3 = 233
-BN3 = 257
-DS4 = 311
-FN4 = 349
-FS4 = 370
-GS4 = 415
-
-notes = [
-  [AS3, 1],       [AS3, 1],             [AS3, 1], [AS3, 1],
-  [AS3, 0.75], [BN3, 0.75], [DS4, 0.5], [AS3, 1], [AS3, 0.5], [nil, 0.5],
-  [AS3, 0.75], [BN3, 0.75], [DS4, 0.5], [AS3, 1], [AS3, 0.5], [nil, 0.5],
-  [AS3, 0.75], [BN3, 0.75], [DS4, 0.5], [FN4, 1], [FN4, 0.5], [nil, 0.5],
-  [GS4, 0.75], [FS4, 0.75], [FN4, 0.5], [DS4, 1], [DS4, 0.5], [nil, 0.5],
-  [GS4, 0.75], [FS4, 0.75], [FN4, 0.5], [DS4, 1], [DS4, 0.5], [nil, 0.5],
-  [AS3, 0.75], [BN3, 0.75], [DS4, 0.5], [AS3, 1], [AS3, 0.5], [nil, 0.5],
-  [AS3, 0.75], [BN3, 0.75], [DS4, 0.5], [AS3, 1], [AS3, 0.5], [nil, 0.5],
-]
-  
-bpm = 135
-beat_time = 60.to_f / bpm
-
-notes.each do |note|
-  if note[0]
-    buzzer.tone(note[0])
-    sleep(note[1] * beat_time)
-  else
-    buzzer.stop
-    sleep((note[1] * beat_time))
-  end
-  buzzer.stop
-  buzzer.micro_delay(5)
-end
-
-buzzer.stop
-board.finish_write

data/lib/denko/board/led_array.rb

@@ -1,17 +1,14 @@
 module Denko
   class Board
     def show_ws2812(pin, pixel_buffer)
-      length = pixel_buffer.length
-
       # Settings are blank for now.
       settings = pack :uint8, [0, 0, 0, 0]
       
-      # Limit to 100 pixels (3-bytes each) for now
-      packed_pixels = pack :uint8, pixel_buffer, max: 300
-
+      packed_pixels = pack :uint8, pixel_buffer
+      
       write_and_halt Message.encode command: 19,
                                     pin: pin,
-                                    value: length,
+                                    value: pixel_buffer.length,
                                     aux_message: settings + packed_pixels
     end
   end

data/lib/denko/message.rb

@@ -11,7 +11,7 @@ module Denko
         aux_message = aux_message.to_s
 
         # Validate aux_message before escaping characters.
-        raise ArgumentError, 'aux_message is limited to 528 characters' if aux_message.length > 528
+        raise ArgumentError, 'aux_message is limited to 528 characters' if aux_message.length > 784
 
         # Escape \ and \n.
         aux_message = aux_message.gsub("\\","\\\\\\\\").gsub("\n", "\\\n")

data/lib/denko/version.rb

@@ -1,3 +1,3 @@
 module Denko
-  VERSION = "0.13.2"
+  VERSION = "0.13.3"
 end

data/src/denko_wifi.ino

@@ -1,3 +1,25 @@
+// NOTE!!!
+// For some boards the standard <WiFi.h> header file does not work.
+//
+// Make sure to define:
+// WIFI_101 if using the WiFi Shield 101, or any board that uses the ATWINC1500 for Wi-Fi.
+// WIFI_NINA if using any board with the NINA-W102 WiFi chip.
+//
+// Some known boards are handled automatically.
+//
+// #define WIFI_101
+// #define WIFI_NINA
+
+// WiFi101 Boards
+#ifdef ARDUINO_SAMD_MKR1000
+  #define WIFI_101
+#endif
+
+// Nina WiFi Boards
+#if defined(ARDUINO_SAMD_MKRWIFI1010) || defined(ARDUINO_AVR_UNO_WIFI_REV2) || defined(ARDUINO_SAMD_NANO_33_IOT)
+  #define WIFI_NINA
+#endif
+
 #include "Denko.h"
 #if defined(ESP8266)
   #include <ESP8266WiFi.h>
@@ -12,9 +34,17 @@
   // #include <ArduinoOTA.h>
   #define WIFI_STATUS_LED 2
 #else
-  #include <SPI.h>
-  #include <WiFi.h>
   #define WIFI_STATUS_LED 13
+  #if defined(WIFI_101)
+    #include <WiFi101.h>
+  #elif defined(WIFI_NINA)
+    #include <WiFiNINA.h>
+  #elif defined(ARDUINO_UNOWIFIR4)
+    #include <WiFiS3.h>
+  #else
+    #include <SPI.h>
+    #include <WiFi.h>
+  #endif
 #endif
 
 // Configure your WiFi options here. IP address is not configurable. Uses DHCP.
@@ -45,7 +75,12 @@ void printWifiStatus() {
   DENKO_SERIAL_IF.print(WiFi.RSSI());
   DENKO_SERIAL_IF.println(" dBm");
   DENKO_SERIAL_IF.print("IP Address: ");
-  DENKO_SERIAL_IF.println(WiFi.localIP());
+  #ifdef WIFI_101
+    IPAddress ip = WiFi.localIP();
+    DENKO_SERIAL_IF.println(ip);
+  #else
+    DENKO_SERIAL_IF.println(WiFi.localIP());
+  #endif
   DENKO_SERIAL_IF.print("Denko TCP Port: ");
   DENKO_SERIAL_IF.println(port);
   indicateWiFi(true);

data/src/lib/Denko.cpp

@@ -198,7 +198,6 @@ void Denko::process() {
     case 90: handshake                ();  break;
     case 91: resetState               ();  break;
     case 92: sendReady                ();  break;
-    case 93: enumerate                ();  break;
     case 95: setRegisterDivider       ();  break;
     case 96: setAnalogWriteResolution ();  break;
     case 97: setAnalogReadResolution  ();  break;
@@ -306,14 +305,6 @@ void Denko::resetState() {
   lastTime = micros();
 }
 
-// CMD = 93
-void Denko::enumerate() {
-  stream->print("ENUMERATION:");
-  #ifdef A0
-    stream->println(A0);
-  #endif
-}
-
 // CMD = 95
 // Set the register read divider. Powers of 2 up to 128 are valid.
 void Denko::setRegisterDivider() {

data/src/lib/Denko.h

@@ -177,7 +177,6 @@ class Denko {
     //
     void handshake                      ();  //cmd = 90
     void resetState                     ();  //cmd = 91
-    void enumerate                      ();  //cmd = 92
     void setRegisterDivider             ();  //cmd = 97
     void setAnalogWriteResolution       ();  //cmd = 96
     void setAnalogReadResolution        ();  //cmd = 97
@@ -202,7 +201,7 @@ class Denko {
     // Parsed message storage.
     byte cmdStr[4]; byte cmd;
     byte pinStr[4]; byte pin;
-    byte valStr[4]; byte val;
+    byte valStr[4]; uint16_t val;
     byte auxMsg[AUX_SIZE];
 
     //

data/src/lib/DenkoDefines.h

@@ -65,7 +65,7 @@
 #ifdef DENKO_UART
   // Look for TX pin definitions on RP2040.
   #if defined(RP2040)
-    #if   defined(PIN_SERIAL2_TX)
+    #if defined(PIN_SERIAL2_TX)
       #define DENKO_UARTS 2
     #elif defined(PIN_SERIAL_1_TX)
       #define DENKO_UARTS 1
@@ -83,7 +83,15 @@
   #elif defined(ESP8266) && defined(SERIAL_PORT_HARDWARE_OPEN)
     #define DENKO_UARTS 1
 
-  // This works for all the Atmel cores except RA4M1.
+  // Define 1 UART for UNO R4 boards. Always use Serial1.
+  #elif defined(_RENESAS_RA_)
+    #define DENKO_UARTS 1
+
+  // Define 1 UART for ATmega4809.
+  #elif defined(__AVR_ATmega4809__) && defined(SERIAL_PORT_HARDWARE)
+    #define DENKO_UARTS 1
+
+  // This works for all the Atmel cores exept ATmega4809.
   #else
     #if defined(SERIAL_PORT_HARDWARE3)
       #define DENKO_UARTS 3
@@ -102,9 +110,9 @@
 // If no high usage features (core sketch), 32 + 16.
 #if !defined(DENKO_SHIFT) && !defined (DENKO_I2C) && !defined(DENKO_SPI) && !defined(DENKO_UARTS) && !defined(DENKO_UART_BB) && !defined(DENKO_IR_OUT) && !defined(DENKO_LED_ARRAY)
   #define AUX_SIZE 48
-// If using IR_OUT or LED_ARRAY, and not on the ATmega168, 512 + 16.
+// If using IR_OUT or LED_ARRAY, and not on the ATmega168, 768 + 16.
 #elif (defined(DENKO_IR_OUT) || defined(DENKO_LED_ARRAY)) && !defined(__AVR_ATmega168__)
-  #define AUX_SIZE 528
+  #define AUX_SIZE 784
 // Default aux message size to 256 + 16 bytes.
 #else
   #define AUX_SIZE 272
@@ -133,10 +141,10 @@
 #elif defined(_RENESAS_RA_)
   #define DENKO_SERIAL_BUFFER_SIZE 512
   #define DENKO_RX_ACK_INTERVAL 64
-// SAMD21 is 256/64 in native USB mode ONLY. Must use defaults on programming port to avoid data loss.
+// SAMD21 is 256/128 in native USB mode ONLY. Must use defaults on programming port to avoid data loss.
 #elif defined(__SAMD21G18A__) && defined(DENKO_USB_CDC)
   #define DENKO_SERIAL_BUFFER_SIZE 256
-  #define DENKO_RX_ACK_INTERVAL 64
+  #define DENKO_RX_ACK_INTERVAL 128
 // 32u4 is odd. Size is 63 instead of 64. Interval must be 31. 32 doesn't work at all. Off by 1 errors?
 #elif defined(__AVR_ATmega32U4__)
   #define DENKO_SERIAL_BUFFER_SIZE 63

data/src/lib/DenkoLEDArray.cpp

@@ -17,7 +17,7 @@
 // Write data to a WS2812 LED array. Will generalize to other types later.
 //
 // pin          = Microcontroller pin connected to Data In pin of the LED array.
-// val          = Number of RGB or RGBW LEDS (aka pixels) in the array.
+// val          = Number of raw pixel data bytes to expect. Max 9999.
 // auxMsg[0..3] = Reserved for future settings.
 // auxMsg[4+]   = Raw pixel data, already in correct byte order (GRB, RGB, etc.).
 //
@@ -27,18 +27,12 @@ void Denko::showLEDArray() {
   ledArray.begin();
 
   // Copy LED data into the pixel buffer.
-  // NOTE: val * 3 will have to change to val * 4 for RGBW leds.
-  memcpy(ledArray.getPixels(), &auxMsg[4], val * 3);
+  memcpy(ledArray.getPixels(), &auxMsg[4], val);
 
-  // I have NO idea what causes the first green LED to stay stuck on.
-  // A small delay here seems like a consistent fix. ESP32 doesn't need it.
-  // AVR can get away with 32us, but others need 64.
-  #if defined(ESP8266) || defined(__SAMD21G18A__) || defined(__SAM3X8E__)
+  // ATmega4809 still needs this delay to avoid corrupt data. Not sure why.
+  #if defined(__AVR_ATmega4809__)
     delayMicroseconds(64);
   #endif
-  #ifdef __AVR__
-    delayMicroseconds(32);
-  #endif
 
   // Write the pixel buffer to the array.
   ledArray.show();

data/target.yml

@@ -1,37 +1,3 @@
-# name: rp2040
-# fqbn: rp2040:rp2040:rpipicow
-# port: /dev/cu.usbmodem14B01
-
-# name: mega
-# fqbn: arduino:avr:uno
-# ort: /dev/cu.usbmodem14B01
-
-# name: mega
-# fqbn: arduino:avr:mega
-# port: /dev/cu.usbmodem14C01
-
 name: mega
-fqbn: arduino:avr:leonardo
+fqbn: arduino:avr:uno
 port: /dev/cu.usbmodem14B01
-
-# name: esp32
-# fqbn: esp32:esp32:esp32doit-devkit-v1
-# port: /dev/cu.SLAB_USBtoUART
-
-# name: esp8266
-# fqbn: esp8266:esp8266:nodemcuv2
-# port: /dev/cu.SLAB_USBtoUART
-
-# name: sam3x
-# fqbn: arduino:sam:arduino_due_x
-# port: /dev/cu.usbmodem14C0
-
-# name: samd
-# fqbn: arduino:samd:arduino_zero_native
-# port: /dev/cu.usbmodem14C01
-
-# Native USB ESP32 shouldn't be used since this doesn't enable USB CDC on boot.
-
-# name: esp32
-# fqbn: esp32:esp32:lolin_s3
-# port: /dev/cu.usbmodem14C01

data/test/analog_io/input_test.rb

@@ -10,7 +10,7 @@ class AnalogIOInputTest < Minitest::Test
   end
 
   def test__read
-    mock = MiniTest::Mock.new.expect :call, nil, [14, nil, nil, nil]
+    mock = Minitest::Mock.new.expect :call, nil, [14, nil, nil, nil]
     board.stub(:analog_read, mock) do
       part._read
     end
@@ -18,7 +18,7 @@ class AnalogIOInputTest < Minitest::Test
   end
 
   def test__listen
-    mock = MiniTest::Mock.new
+    mock = Minitest::Mock.new
     mock.expect :call, nil, [14, 16]
     mock.expect :call, nil, [14, 32]
     board.stub(:analog_listen, mock) do
@@ -29,7 +29,7 @@ class AnalogIOInputTest < Minitest::Test
   end
 
   def test__stop_listen
-    mock = MiniTest::Mock.new.expect :call, nil, [14]
+    mock = Minitest::Mock.new.expect :call, nil, [14]
     board.stub(:stop_listener, mock) do
       part._stop_listener
     end

data/test/analog_io/output_test.rb

@@ -14,7 +14,7 @@ class AnalogIOOutputTest < Minitest::Test
   end
   
   def test_dac_write
-    mock = MiniTest::Mock.new.expect :call, nil, [14, 128]
+    mock = Minitest::Mock.new.expect :call, nil, [14, 128]
 
     board.stub(:dac_write, mock) do
       part.write 128

data/test/analog_io/potentiometer_test.rb

@@ -1,6 +1,6 @@
 require_relative '../test_helper'
 
-class AnalogIOPotentiometerTest < MiniTest::Test
+class AnalogIOPotentiometerTest < Minitest::Test
   def board
     @board ||= BoardMock.new
   end
@@ -10,7 +10,7 @@ class AnalogIOPotentiometerTest < MiniTest::Test
   end
 
   def test_setup
-    mock = MiniTest::Mock.new.expect(:call, nil, [14,8])
+    mock = Minitest::Mock.new.expect(:call, nil, [14,8])
     board.stub(:analog_listen, mock) do
       part
     end

data/test/behaviors/bus_peripheral_addressed.rb

@@ -33,7 +33,7 @@ class BusPeripheralAddressedTest < Minitest::Test
   end
   
   def test_can_use_bus_atomically   
-    mock = MiniTest::Mock.new
+    mock = Minitest::Mock.new
     1.times {mock.expect(:call, nil)}
     
     bus.mutex.stub(:synchronize, mock) do

data/test/behaviors/bus_peripheral_test.rb

@@ -28,7 +28,7 @@ class BusPeripheralTest < Minitest::Test
   end
 
   def test_can_use_bus_atomically   
-    mock = MiniTest::Mock.new
+    mock = Minitest::Mock.new
     1.times {mock.expect(:call, nil)}
     
     bus.mutex.stub(:synchronize, mock) do

data/test/behaviors/callbacks_test.rb

@@ -29,7 +29,7 @@ class CallbacksTest < Minitest::Test
 
   def test_callback_mutex
     callback = Proc.new{}
-    mock = MiniTest::Mock.new
+    mock = Minitest::Mock.new
     3.times {mock.expect(:call, nil)}
     
     part.callback_mutex.stub(:synchronize, mock) do
@@ -73,8 +73,8 @@ class CallbacksTest < Minitest::Test
   end
 
   def test_update_runs_callbacks_and_removes_read_callbacks
-    cb1 = MiniTest::Mock.new.expect :call, nil
-    cb2 = MiniTest::Mock.new.expect :call, nil
+    cb1 = Minitest::Mock.new.expect :call, nil
+    cb2 = Minitest::Mock.new.expect :call, nil
     part.add_callback        { cb1.call }
     part.add_callback(:read) { cb2.call }
     part.update("data")
@@ -84,7 +84,7 @@ class CallbacksTest < Minitest::Test
   end
 
   def test_pre_callback_filter_modifies_data
-    cb = MiniTest::Mock.new.expect :call, nil, ["denko: value"]
+    cb = Minitest::Mock.new.expect :call, nil, ["denko: value"]
     part.add_callback { |x| cb.call(x) }
     part.update("value")
     cb.verify

data/test/behaviors/component_test.rb

@@ -36,7 +36,7 @@ class ComponentTest < Minitest::Test
   end
   
   def test_state_through_mutex
-    mock = MiniTest::Mock.new
+    mock = Minitest::Mock.new
     2.times {mock.expect(:call, nil)}
     
     part.instance_variable_get(:@state_mutex).stub(:synchronize, mock) do
@@ -47,7 +47,7 @@ class ComponentTest < Minitest::Test
   end
   
   def test_micro_delay
-    mock = MiniTest::Mock.new.expect :call, nil, [1000]
+    mock = Minitest::Mock.new.expect :call, nil, [1000]
     
     board.stub(:micro_delay, mock) do
       part.micro_delay(1000)