Add extra I2C options for MAX32690 non-TQFN packages

Also reformatted pinmaps for SPI/I2C/UART on MAX32690

Signed-off-by: Brandon Hurst <[email protected]>

Author: Brandon-Hurst

ports/analog/peripherals/max32690/max32_i2c.c

@@ -13,14 +13,44 @@
 #include "py/runtime.h"
 #include "py/mperrno.h"
 
+
+/* Note: The MAX32690 assigns the same alternate function to multiple sets
+     * of pins.  The drivers will enable both sets so that either can be used.
+     * Users should ensure the unused set is left unconnected.
+     *
+     * See MAX32690 Rev A2 Errata #16:
+     * https://www.analog.com/media/en/technical-documentation/data-sheets/max32690_a2_errata_rev2.pdf
+     * 
+     * Additionally, note that the TQFN package does not expose some of the duplicate pins.  For this package,
+     * enabling the un-routed GPIOs has been shown to cause initialization issues with the I2C block.
+     * To work around this, "MAX32690GTK_PACKAGE_TQFN" can be defined by the build system.  The recommend place
+     * to do it is in the "board.mk" file of the BSP.  This will prevent the inaccessible pins from being configured.
+     */
+
 const mxc_gpio_cfg_t i2c_maps[NUM_I2C] = {
+    // I2C0
+    { MXC_GPIO2, (MXC_GPIO_PIN_7 | MXC_GPIO_PIN_8), MXC_GPIO_FUNC_ALT1,
+        MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 };
+    // I2C1
+    { MXC_GPIO0, (MXC_GPIO_PIN_11 | MXC_GPIO_PIN_12), MXC_GPIO_FUNC_ALT1,
+        MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 };
+    // I2C2
+    { MXC_GPIO1, (MXC_GPIO_PIN_7 | MXC_GPIO_PIN_8), MXC_GPIO_FUNC_ALT3,
+        MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIOH, MXC_GPIO_DRVSTR_0 };,
+};
+#ifndef MAX32690GTK_PACKAGE_TQFN
+const mxc_gpio_cfg_t i2c_maps_extra[NUM_I2C] = {
     // I2C0A
     { MXC_GPIO0, (MXC_GPIO_PIN_30 | MXC_GPIO_PIN_31), MXC_GPIO_FUNC_ALT1,
       MXC_GPIO_PAD_PULL_UP, MXC_GPIO_VSSEL_VDDIOH, MXC_GPIO_DRVSTR_0 },
     // I2C1A
     { MXC_GPIO2, (MXC_GPIO_PIN_17 | MXC_GPIO_PIN_18), MXC_GPIO_FUNC_ALT1,
       MXC_GPIO_PAD_PULL_UP, MXC_GPIO_VSSEL_VDDIOH, MXC_GPIO_DRVSTR_0 },
+    // I2C2C
+    { MXC_GPIO0, (MXC_GPIO_PIN_13 | MXC_GPIO_PIN_14), MXC_GPIO_FUNC_ALT3,
+        MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 };,
 };
+#endif
 
 int pinsToI2c(const mcu_pin_obj_t *sda, const mcu_pin_obj_t *scl) {
     for (int i = 0; i < NUM_I2C; i++) {
@@ -29,6 +59,17 @@ int pinsToI2c(const mcu_pin_obj_t *sda, const mcu_pin_obj_t *scl) {
             return i;
         }
     }
+
+    // Additional for loop to cover alternate potential I2C maps
+    #ifndef MAX32690GTK_PACKAGE_TQFN
+    for (int i = 0; i < NUM_I2C; i++) {
+        if ((i2c_maps_extra[i].port == (MXC_GPIO_GET_GPIO(sda->port)))
+            && (i2c_maps_extra[i].mask == ((sda->mask) | (scl->mask)))) {
+            return i;
+        }
+    }
+    #endif
+
     mp_raise_RuntimeError_varg(MP_ERROR_TEXT("ERR: Unable to find an I2C matching pins...\nSCL: port %d mask %d\nSDA: port %d mask %d\n"),
         sda->port, sda->mask, scl->port, scl->mask);
     return -1;

ports/analog/peripherals/max32690/max32_spi.c

@@ -13,21 +13,20 @@
 #include "py/runtime.h"
 #include "py/mperrno.h"
 
-// TODO Decouple from APARD board
 const mxc_gpio_cfg_t spi_maps[NUM_SPI] = {
-    // DUMMY entry for SPI0 (not on APARD board)
-    { MXC_GPIO0, 0xFFFFFFFF, 0, 0, 0, 0},
-
-    // SPI1A
+    // SPI0
+    { MXC_GPIO2, (MXC_GPIO_PIN_27 | MXC_GPIO_PIN_28 | MXC_GPIO_PIN_29),
+    MXC_GPIO_FUNC_ALT2, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 };
+    // SPI1
     { MXC_GPIO1, (MXC_GPIO_PIN_26 | MXC_GPIO_PIN_28 | MXC_GPIO_PIN_29),
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
-    // SPI2A
+    // SPI2
     { MXC_GPIO2, (MXC_GPIO_PIN_2 | MXC_GPIO_PIN_3 | MXC_GPIO_PIN_4),
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
-    // SPI3A
+    // SPI3
     { MXC_GPIO0, (MXC_GPIO_PIN_16 | MXC_GPIO_PIN_20 | MXC_GPIO_PIN_21),
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
-    // SPI4A
+    // SPI4
     { MXC_GPIO1, (MXC_GPIO_PIN_1 | MXC_GPIO_PIN_2 | MXC_GPIO_PIN_3),
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
 };

ports/analog/peripherals/max32690/max32_uart.c

@@ -13,29 +13,19 @@
 #include "py/runtime.h"
 #include "py/mperrno.h"
 
-// FIXME: Remove upon test!
-// mxc_uart_regs_t max32_uarts[NUM_UARTS] = {
-//     MXC_UART0,
-//     MXC_UART1,
-//     MXC_UART2,
-//     MXC_UART3,
-// };
-
 const mxc_gpio_cfg_t uart_maps[NUM_UARTS] = {
     { MXC_GPIO2, (MXC_GPIO_PIN_11 | MXC_GPIO_PIN_12), MXC_GPIO_FUNC_ALT1,
       MXC_GPIO_PAD_WEAK_PULL_UP, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
     { MXC_GPIO2, (MXC_GPIO_PIN_14 | MXC_GPIO_PIN_16), MXC_GPIO_FUNC_ALT1,
       MXC_GPIO_PAD_WEAK_PULL_UP, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
-    { MXC_GPIO1, (MXC_GPIO_PIN_9 | MXC_GPIO_PIN_10),
-      MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_WEAK_PULL_UP,
-      MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
-    { MXC_GPIO3, (MXC_GPIO_PIN_0 | MXC_GPIO_PIN_1),
-      MXC_GPIO_FUNC_ALT2, MXC_GPIO_PAD_WEAK_PULL_UP,
-      MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 }
+    { MXC_GPIO1, (MXC_GPIO_PIN_9 | MXC_GPIO_PIN_10), MXC_GPIO_FUNC_ALT1, 
+      MXC_GPIO_PAD_WEAK_PULL_UP, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
+    { MXC_GPIO3, (MXC_GPIO_PIN_0 | MXC_GPIO_PIN_1), MXC_GPIO_FUNC_ALT2, 
+      MXC_GPIO_PAD_WEAK_PULL_UP, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 }
 };
 
 int pinsToUart(const mcu_pin_obj_t *rx, const mcu_pin_obj_t *tx) {
-    for (int i = 0; i < NUM_UARTS; i++) {
+    for (int i = 0; i < uart_entries; i++) {
         if ((uart_maps[i].port == (MXC_GPIO_GET_GPIO(tx->port)))
             && (uart_maps[i].mask == ((tx->mask) | (rx->mask)))) {
             return i;