djsundog/circuitpython
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

drivers/ninaw10: Fix ESP32 input-only pins.

Browse Source 
ESP32 pins 34, 35, 36 and 39 are input only, and should not be configured
as output.
This commit is contained in:
iabdalkader 2023-03-26 19:50:39 +02:00 committed by Damien George
parent 5652f1f661
commit 9025671e72
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/ninaw10/machine_pin_nina.c
View File

@ -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);
}
}