Improve Fruit Jam

* Add bus, port_numbers and speed to USB devices. Allows for
  differentiating identical devices.
* Autodetect display and adjust if widescreen
* Enable I2S DAC via its reset pin
* Show keyboard emoji in status bar. (Only works over serial connection
  now. Built in font doesn't have the emoji.)

Author: tannewt

.gitmodules

@@ -345,7 +345,8 @@
 	url = https://github.com/adafruit/Adafruit_CircuitPython_Wave.git
 [submodule "ports/raspberrypi/lib/Pico-PIO-USB"]
 	path = ports/raspberrypi/lib/Pico-PIO-USB
-	url = https://github.com/adafruit/Pico-PIO-USB.git
+	url = https://github.com/tannewt/Pico-PIO-USB.git
+	branch = better_timeouts
 [submodule "lib/micropython-lib"]
 	path = lib/micropython-lib
 	url = https://github.com/micropython/micropython-lib.git

lib/tinyusb

@@ -9,6 +9,8 @@
 #include "shared-bindings/usb_host/Port.h"
 #include "supervisor/board.h"
 
+#include "common-hal/picodvi/__init__.h"
+
 // Use the MP_WEAK supervisor/shared/board.c versions of routines not defined here.
 
 
@@ -25,12 +27,22 @@ bool board_reset_pin_number(uint8_t pin_number) {
 
         return true;
     }
+    // Set I2S out of reset.
+    if (pin_number == 22) {
+        gpio_put(pin_number, 1);
+        gpio_set_dir(pin_number, GPIO_OUT);
+        gpio_set_function(pin_number, GPIO_FUNC_SIO);
+
+        return true;
+    }
     return false;
 }
 #endif
 
-#if defined(DEFAULT_USB_HOST_DATA_PLUS)
 void board_init(void) {
+    #if defined(DEFAULT_USB_HOST_DATA_PLUS)
     common_hal_usb_host_port_construct(DEFAULT_USB_HOST_DATA_PLUS, DEFAULT_USB_HOST_DATA_MINUS);
+    #endif
+
+    picodvi_autoconstruct();
 }
-#endif

ports/raspberrypi/boards/adafruit_fruit_jam/board.c

@@ -21,4 +21,18 @@
 #define DEFAULT_USB_HOST_DATA_MINUS (&pin_GPIO2)
 #define DEFAULT_USB_HOST_5V_POWER (&pin_GPIO11)
 
+#define DEFAULT_DVI_BUS_CLK_DN (&pin_GPIO12)
+#define DEFAULT_DVI_BUS_CLK_DP (&pin_GPIO13)
+#define DEFAULT_DVI_BUS_RED_DN (&pin_GPIO14)
+#define DEFAULT_DVI_BUS_RED_DP (&pin_GPIO15)
+#define DEFAULT_DVI_BUS_GREEN_DN (&pin_GPIO16)
+#define DEFAULT_DVI_BUS_GREEN_DP (&pin_GPIO17)
+#define DEFAULT_DVI_BUS_BLUE_DN (&pin_GPIO18)
+#define DEFAULT_DVI_BUS_BLUE_DP (&pin_GPIO19)
+
 #define CIRCUITPY_PSRAM_CHIP_SELECT (&pin_GPIO47)
+
+// #define CIRCUITPY_CONSOLE_UART_TX (&pin_GPIO44)
+// #define CIRCUITPY_CONSOLE_UART_RX (&pin_GPIO45)
+
+// #define CIRCUITPY_DEBUG_TINYUSB 0

ports/raspberrypi/boards/adafruit_fruit_jam/mpconfigboard.h

@@ -8,3 +8,6 @@ CHIP_PACKAGE = B
 CHIP_FAMILY = rp2
 
 EXTERNAL_FLASH_DEVICES = "W25Q128JVxQ"
+
+# CIRCUITPY_DISPLAY_FONT = $(TOP)/tools/fonts/unifont-16.0.02-all.bdf
+# CIRCUITPY_FONT_EXTRA_CHARACTERS = "🖮🖱️"

ports/raspberrypi/boards/adafruit_fruit_jam/mpconfigboard.mk

@@ -15,8 +15,6 @@ static const mp_rom_map_elem_t board_module_globals_table[] = {
     { MP_OBJ_NEW_QSTR(MP_QSTR_A4), MP_ROM_PTR(&pin_GPIO44) },
     { MP_OBJ_NEW_QSTR(MP_QSTR_A5), MP_ROM_PTR(&pin_GPIO45) },
 
-    // On-board switch reverses D0 and D1 connections to RX and TX.
-
     { MP_OBJ_NEW_QSTR(MP_QSTR_D6), MP_ROM_PTR(&pin_GPIO6) },
     { MP_OBJ_NEW_QSTR(MP_QSTR_D7), MP_ROM_PTR(&pin_GPIO7) },
     { MP_OBJ_NEW_QSTR(MP_QSTR_D8), MP_ROM_PTR(&pin_GPIO8) },

ports/raspberrypi/boards/adafruit_fruit_jam/pins.c

@@ -271,6 +271,11 @@ void common_hal_picodvi_framebuffer_construct(picodvi_framebuffer_obj_t *self,
     // streaming support.
     self->framebuffer = (uint32_t *)port_malloc(framebuffer_size * sizeof(uint32_t), true);
     if (self->framebuffer == NULL || ((size_t)self->framebuffer & 0xf0000000) == 0x10000000) {
+        if (self->framebuffer != NULL) {
+            // Return the memory in PSRAM.
+            port_free(self->framebuffer);
+            self->framebuffer = NULL;
+        }
         m_malloc_fail(framebuffer_size * sizeof(uint32_t));
         return;
     }

ports/raspberrypi/common-hal/picodvi/Framebuffer_RP2350.c

@@ -17,7 +17,7 @@
 #include "supervisor/port_heap.h"
 
 #if defined(DEFAULT_DVI_BUS_CLK_DP)
-static bool picodvi_autoconstruct_enabled(void) {
+static bool picodvi_autoconstruct_enabled(mp_int_t *default_width, mp_int_t *default_height) {
     char buf[sizeof("detect")];
     buf[0] = 0;
 
@@ -42,6 +42,48 @@ static bool picodvi_autoconstruct_enabled(void) {
         return false;
     }
     bool probed = common_hal_busio_i2c_probe(i2c, 0x50);
+    if (probed) {
+        uint8_t edid[128];
+        uint8_t out[1] = {0};
+        common_hal_busio_i2c_write_read(i2c, 0x50, out, 1, edid, sizeof(edid));
+        bool edid_ok = true;
+        if (edid[0] != 0x00 || edid[1] != 0xFF || edid[2] != 0xFF || edid[3] != 0xFF || edid[4] != 0xFF || edid[5] != 0xFF || edid[6] != 0xFF || edid[7] != 0x00) {
+            edid_ok = false;
+        }
+        uint8_t checksum = 0;
+        for (size_t i = 0; i < sizeof(edid); i++) {
+            checksum += edid[i];
+        }
+        if (checksum != 0) {
+            edid_ok = false;
+        }
+
+        if (edid_ok) {
+            uint8_t established_timings = edid[35];
+            if ((established_timings & 0xa0) == 0) {
+                // Check that 720x400@70Hz or 640x480@60Hz is supported. If not
+                // and we read EDID ok, then don't autostart.
+                probed = false;
+            } else {
+                size_t offset = 54;
+                uint16_t preferred_pixel_clock = edid[offset] | (edid[offset + 1] << 8);
+                if (preferred_pixel_clock != 0) {
+                    size_t preferred_width = ((edid[offset + 4] & 0xf0) << 4) | edid[offset + 2];
+                    size_t preferred_height = ((edid[offset + 7] & 0xf0) << 4) | edid[offset + 5];
+                    // Use 720x400 on 1080p, 4k and 8k displays.
+                    if ((established_timings & 0x80) != 0 &&
+                        preferred_width % 1920 == 0 &&
+                        preferred_height % 1080 == 0) {
+                        *default_width = 720 / 2;
+                        *default_height = 400 / 2;
+                    } else {
+                        *default_width = 640 / 2;
+                        *default_height = 480 / 2;
+                    }
+                }
+            }
+        }
+    }
     common_hal_busio_i2c_unlock(i2c);
     return probed;
 }
@@ -53,11 +95,13 @@ void picodvi_autoconstruct(void) {
         return;
     }
 
-    if (!picodvi_autoconstruct_enabled()) {
+    mp_int_t default_width = 320;
+    mp_int_t default_height = 240;
+    if (!picodvi_autoconstruct_enabled(&default_width, &default_height)) {
         return;
     }
 
-    mp_int_t width = 320;
+    mp_int_t width = default_width;
     mp_int_t height = 0;
     mp_int_t color_depth = 16;
     mp_int_t rotation = 0;
@@ -75,6 +119,9 @@ void picodvi_autoconstruct(void) {
             case 320:
                 height = 240;
                 break;
+            case 360:
+                height = 200;
+                break;
         }
     }
 
@@ -85,8 +132,8 @@ void picodvi_autoconstruct(void) {
 
     if (!common_hal_picodvi_framebuffer_preflight(width, height, color_depth)) {
         // invalid configuration, set back to default
-        width = 320;
-        height = 240;
+        width = default_width;
+        height = default_height;
         color_depth = 16;
     }
 

ports/raspberrypi/common-hal/picodvi/__init__.c

@@ -739,6 +739,7 @@ SRC_SHARED_MODULE_ALL = \
 	usb/__init__.c \
 	usb/core/__init__.c \
 	usb/core/Device.c \
+	usb/util/__init__.c \
 	ustack/__init__.c \
 	vectorio/Circle.c \
 	vectorio/Polygon.c \