diff --git a/drivers/ninaw10/machine_pin_nina.c b/drivers/ninaw10/machine_pin_nina.c index 3d98af0ef..025ecb66a 100644 --- 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); } }