drivers/ninaw10: Fix ESP32 input-only pins.

iabdalkader · dpgeorge · commit 9025671e7216 · 2023-04-04T15:16:44.000+10:00
ESP32 pins 34, 35, 36 and 39 are input only, and should not be configured
as output.
diff --git a/drivers/ninaw10/machine_pin_nina.c b/drivers/ninaw10/machine_pin_nina.c
@@ -43,6 +43,7 @@
 #define NINA_GPIO_MODE          (0x50)
 #define NINA_GPIO_READ          (0x53)
 #define NINA_GPIO_WRITE         (0x51)
+#define NINA_GPIO_IS_INPUT_ONLY(p) ((p >= 34 && p <= 36) || (p == 39))
 
 static uint8_t pin_map[MICROPY_HW_PIN_EXT_COUNT] = {
     27, // LEDR
@@ -82,12 +83,17 @@ void machine_pin_ext_config(machine_pin_obj_t *self, int mode, int value) {
     } else if (mode == MACHINE_PIN_MODE_OUT) {
         mode = NINA_GPIO_OUTPUT;
         self->is_output = true;
-        machine_pin_ext_set(self, value);
     } else {
         mp_raise_ValueError("only Pin.OUT and Pin.IN are supported for this pin");
     }
     if (self->id >= 0 && self->id < MICROPY_HW_PIN_EXT_COUNT) {
         uint8_t buf[] = {pin_map[self->id], mode};
+        if (mode == NINA_GPIO_OUTPUT) {
+            if (NINA_GPIO_IS_INPUT_ONLY(buf[0])) {
+                mp_raise_ValueError("only Pin.IN is supported for this pin");
+            }
+            machine_pin_ext_set(self, value);
+        }
         nina_ioctl(NINA_GPIO_MODE, sizeof(buf), buf, 0);
     }
 }
