ports: analog: Update SPI peripheral helpers

- Fix pinsToSpi function signature to correctly matchup pins
- Pin changes for MAX32665 to match package on common Eval boards
- Use '&' instead of '==' for pinmasks to allow for easier pin usage in C code
(This may make it simpler to copy GPIO structs from MSDK down the line)

Signed-off-by: Brandon-Hurst <brandon.hurst97@gmail.com>

Author: Brandon-Hurst

ports/analog/peripherals/max32650/max32_spi.c

@@ -29,11 +29,11 @@ const mxc_gpio_cfg_t spi_maps[NUM_SPI] = {
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
 };
 
-int pinsToSpi(const mcu_pin_obj_t *mosi, const mcu_pin_obj_t *miso,
-    const mcu_pin_obj_t *sck) {
+int pinsToSpi(const mcu_pin_obj_t *sck, const mcu_pin_obj_t *mosi,
+    const mcu_pin_obj_t *miso) {
     for (int i = 0; i < NUM_SPI; i++) {
         if ((spi_maps[i].port == (MXC_GPIO_GET_GPIO(mosi->port)))
-            && (spi_maps[i].mask == ((mosi->mask) | (miso->mask) | (sck->mask)))) {
+            && (spi_maps[i].mask & ((mosi->mask) | (miso->mask) | (sck->mask)))) {
             return i;
         }
     }

ports/analog/peripherals/max32665/max32_spi.c

@@ -19,25 +19,22 @@
 
 const mxc_gpio_cfg_t spi_maps[NUM_SPI] = {
     // SPI0A
-    { MXC_GPIO1,
-      (MXC_GPIO_PIN_9 | MXC_GPIO_PIN_10 | MXC_GPIO_PIN_11 | MXC_GPIO_PIN_12 | MXC_GPIO_PIN_13),
+    { MXC_GPIO1, (MXC_GPIO_PIN_9 | MXC_GPIO_PIN_10 | MXC_GPIO_PIN_11),
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
     // SPI1
-    { MXC_GPIO0,
-      (MXC_GPIO_PIN_17 | MXC_GPIO_PIN_18 | MXC_GPIO_PIN_19 | MXC_GPIO_PIN_20 | MXC_GPIO_PIN_21),
+    { MXC_GPIO0, (MXC_GPIO_PIN_17 | MXC_GPIO_PIN_18 | MXC_GPIO_PIN_19),
       MXC_GPIO_FUNC_ALT2, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
     // SPI2
-    { MXC_GPIO0,
-      (MXC_GPIO_PIN_25 | MXC_GPIO_PIN_26 | MXC_GPIO_PIN_27 | MXC_GPIO_PIN_28 | MXC_GPIO_PIN_29),
+    { MXC_GPIO0, (MXC_GPIO_PIN_25 | MXC_GPIO_PIN_26 | MXC_GPIO_PIN_27),
       MXC_GPIO_FUNC_ALT2, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
 };
 
 
-int pinsToSpi(const mcu_pin_obj_t *mosi, const mcu_pin_obj_t *miso,
-    const mcu_pin_obj_t *sck) {
+int pinsToSpi(const mcu_pin_obj_t *sck, const mcu_pin_obj_t *mosi,
+    const mcu_pin_obj_t *miso) {
     for (int i = 0; i < NUM_SPI; i++) {
         if ((spi_maps[i].port == (MXC_GPIO_GET_GPIO(mosi->port)))
-            && (spi_maps[i].mask == ((mosi->mask) | (miso->mask) | (sck->mask)))) {
+            && (spi_maps[i].mask & ((mosi->mask) | (miso->mask) | (sck->mask)))) {
             return i;
         }
     }

ports/analog/peripherals/max32690/max32_spi.c

@@ -31,11 +31,11 @@ const mxc_gpio_cfg_t spi_maps[NUM_SPI] = {
       MXC_GPIO_FUNC_ALT1, MXC_GPIO_PAD_NONE, MXC_GPIO_VSSEL_VDDIO, MXC_GPIO_DRVSTR_0 },
 };
 
-int pinsToSpi(const mcu_pin_obj_t *mosi, const mcu_pin_obj_t *miso,
-    const mcu_pin_obj_t *sck) {
+int pinsToSpi(const mcu_pin_obj_t *sck, const mcu_pin_obj_t *mosi,
+    const mcu_pin_obj_t *miso) {
     for (int i = 0; i < NUM_SPI; i++) {
         if ((spi_maps[i].port == (MXC_GPIO_GET_GPIO(mosi->port)))
-            && (spi_maps[i].mask == ((mosi->mask) | (miso->mask) | (sck->mask)))) {
+            && (spi_maps[i].mask & ((mosi->mask) | (miso->mask) | (sck->mask)))) {
             return i;
         }
     }