Numerous iMX fixes and SDK update

* Fixes #4954. Leaving GPIO in output mode messed with ADC readings.
* Fixes #8018 and fixes #6537. UART rx was more complicated than it needed to be.
* Fixes #5581. uart reset is handled by finalizer. Added i2c_reset()
  call and reservation check.

Author: tannewt

ports/mimxrt10xx/common-hal/busio/I2C.c

@@ -138,6 +138,10 @@ void common_hal_busio_i2c_construct(busio_i2c_obj_t *self,
                 continue;
             }
 
+            if (reserved_i2c[mcu_i2c_scl_list[j].bank_idx - 1]) {
+                continue;
+            }
+
             self->sda = &mcu_i2c_sda_list[i];
             self->scl = &mcu_i2c_scl_list[j];
 
@@ -151,6 +155,9 @@ void common_hal_busio_i2c_construct(busio_i2c_obj_t *self,
         self->i2c = mcu_i2c_banks[self->sda->bank_idx - 1];
     }
 
+
+    reserved_i2c[self->sda->bank_idx - 1] = true;
+
     config_periph_pin(self->sda);
     config_periph_pin(self->scl);
 

ports/mimxrt10xx/common-hal/busio/I2C.h

@@ -25,8 +25,7 @@
  * THE SOFTWARE.
  */
 
-#ifndef MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_I2C_H
-#define MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_I2C_H
+#pragma once
 
 #include "common-hal/microcontroller/Pin.h"
 #include "fsl_common.h"
@@ -42,5 +41,3 @@ typedef struct {
 } busio_i2c_obj_t;
 
 void i2c_reset(void);
-
-#endif // MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_I2C_H

ports/mimxrt10xx/common-hal/busio/SPI.h

@@ -25,8 +25,7 @@
  * THE SOFTWARE.
  */
 
-#ifndef MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_SPI_H
-#define MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_SPI_H
+#pragma once
 
 #include "common-hal/microcontroller/Pin.h"
 #include "fsl_common.h"
@@ -44,5 +43,3 @@ typedef struct {
 } busio_spi_obj_t;
 
 void spi_reset(void);
-
-#endif // MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_SPI_H

ports/mimxrt10xx/common-hal/busio/UART.c

@@ -83,24 +83,6 @@ static void config_periph_pin(const mcu_periph_obj_t *periph) {
         | IOMUXC_SW_PAD_CTL_PAD_SRE(0));
 }
 
-STATIC void LPUART_UserCallback(LPUART_Type *base, lpuart_handle_t *handle, status_t status, void *user_data) {
-    busio_uart_obj_t *self = (busio_uart_obj_t *)user_data;
-
-    if (status == kStatus_LPUART_RxIdle) {
-        self->rx_ongoing = false;
-    }
-}
-
-void uart_reset(void) {
-    for (uint i = 0; i < MP_ARRAY_SIZE(mcu_uart_banks); i++) {
-        if (never_reset_uart[i]) {
-            continue;
-        }
-        reserved_uart[i] = false;
-        LPUART_Deinit(mcu_uart_banks[i]);
-    }
-}
-
 void common_hal_busio_uart_never_reset(busio_uart_obj_t *self) {
     never_reset_uart[self->index] = true;
     common_hal_never_reset_pin(self->tx);
@@ -348,6 +330,8 @@ void common_hal_busio_uart_construct(busio_uart_obj_t *self,
 
     if (self->rx != NULL) {
         if (receiver_buffer == NULL) {
+            // One byte is used internally so add one to make sure we have enough space.
+            receiver_buffer_size += 1;
             self->ringbuf = gc_alloc(receiver_buffer_size, false);
         } else {
             self->ringbuf = receiver_buffer;
@@ -359,7 +343,8 @@ void common_hal_busio_uart_construct(busio_uart_obj_t *self,
             m_malloc_fail(receiver_buffer_size);
         }
 
-        LPUART_TransferCreateHandle(self->uart, &self->handle, LPUART_UserCallback, self);
+        // Use the internal ring buffer implementation.
+        LPUART_TransferCreateHandle(self->uart, &self->handle, NULL, NULL);
         // Pass actual allocated size; the LPUART routines are cognizant that
         // the capacity is one less than the size.
         LPUART_TransferStartRingBuffer(self->uart, &self->handle, self->ringbuf, receiver_buffer_size);
@@ -378,12 +363,13 @@ void common_hal_busio_uart_deinit(busio_uart_obj_t *self) {
         return;
     }
 
-    reserved_uart[self->index] = false;
-    never_reset_uart[self->index] = false;
-
+    LPUART_TransferStopRingBuffer(self->uart, &self->handle);
     LPUART_Deinit(self->uart);
     gc_free(self->ringbuf);
 
+    reserved_uart[self->index] = false;
+    never_reset_uart[self->index] = false;
+
     common_hal_reset_pin(self->rx);
     common_hal_reset_pin(self->tx);
     common_hal_reset_pin(self->cts);
@@ -395,7 +381,6 @@ void common_hal_busio_uart_deinit(busio_uart_obj_t *self) {
     self->cts = NULL;
     self->rts = NULL;
     self->rs485_dir = NULL;
-
 }
 
 // Read characters.
@@ -409,18 +394,24 @@ size_t common_hal_busio_uart_read(busio_uart_obj_t *self, uint8_t *data, size_t
         return 0;
     }
 
-    lpuart_transfer_t xfer = {
-        .data = data,
-        .dataSize = len,
-    };
-
-    self->rx_ongoing = true;
-    LPUART_TransferReceiveNonBlocking(self->uart, &self->handle, &xfer, NULL);
-
+    size_t received = 0;
     uint64_t start_ticks = supervisor_ticks_ms64();
-
     // Wait for all bytes received or timeout
-    while (self->rx_ongoing && (supervisor_ticks_ms64() - start_ticks < self->timeout_ms)) {
+    while (received < len && (supervisor_ticks_ms64() - start_ticks < self->timeout_ms)) {
+        lpuart_transfer_t xfer = {
+            .data = data + received
+        };
+        size_t remaining = len - received;
+        xfer.dataSize = MIN(remaining, LPUART_TransferGetRxRingBufferLength(self->uart, &self->handle));
+        // Only request as much as has already been received. Otherwise, we need to deal with
+        // callbacks.
+        size_t additional_read = 0;
+        LPUART_TransferReceiveNonBlocking(self->uart, &self->handle, &xfer, &additional_read);
+        received += additional_read;
+        // Break early when we're done to skip background tasks.
+        if (received == len) {
+            break;
+        }
         RUN_BACKGROUND_TASKS;
 
         // Allow user to break out of a timeout with a KeyboardInterrupt.
@@ -429,30 +420,12 @@ size_t common_hal_busio_uart_read(busio_uart_obj_t *self, uint8_t *data, size_t
         }
     }
 
-    // if we timed out, stop the transfer
-    if (self->rx_ongoing) {
-        uint32_t recvd = 0;
-        LPUART_TransferGetReceiveCount(self->uart, &self->handle, &recvd);
-        LPUART_TransferAbortReceive(self->uart, &self->handle);
-        if (recvd == 0) {
-            *errcode = EAGAIN;
-            return MP_STREAM_ERROR;
-        }
-        return recvd;
-    }
-
-    // No data left, we got it all
-    if (self->handle.rxData == NULL) {
-        return len;
+    if (received == 0) {
+        *errcode = EAGAIN;
+        return MP_STREAM_ERROR;
     }
 
-    // The only place we can reliably tell how many bytes have been received is from the current
-    // wp in the handle (because the abort nukes rxDataSize, and reading it before abort is a race.)
-    if (self->handle.rxData > data) {
-        return self->handle.rxData - data;
-    } else {
-        return len;
-    }
+    return received;
 }
 
 // Write characters.

ports/mimxrt10xx/common-hal/busio/UART.h

@@ -25,8 +25,7 @@
  * THE SOFTWARE.
  */
 
-#ifndef MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_UART_H
-#define MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_UART_H
+#pragma once
 
 #include "common-hal/microcontroller/Pin.h"
 
@@ -43,7 +42,6 @@ typedef struct {
     uint8_t *ringbuf;
     uint32_t baudrate;
     uint32_t timeout_ms;
-    bool rx_ongoing;
     uint8_t character_bits;
     uint8_t index;
     const mcu_pin_obj_t *rx;
@@ -54,6 +52,3 @@ typedef struct {
     bool rs485_invert;
 
 } busio_uart_obj_t;
-
-void uart_reset(void);
-#endif // MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_BUSIO_UART_H

ports/mimxrt10xx/common-hal/digitalio/DigitalInOut.h

@@ -25,8 +25,7 @@
  * THE SOFTWARE.
  */
 
-#ifndef MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_DIGITALIO_DIGITALINOUT_H
-#define MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_DIGITALIO_DIGITALINOUT_H
+#pragma once
 
 #include "common-hal/microcontroller/Pin.h"
 #include "shared-bindings/digitalio/Pull.h"
@@ -41,5 +40,3 @@ typedef struct {
 } digitalio_digitalinout_obj_t;
 
 void pin_config(const mcu_pin_obj_t *pin, bool open_drain, digitalio_pull_t pull);
-
-#endif // MICROPY_INCLUDED_MIMXRT10XX_COMMON_HAL_DIGITALIO_DIGITALINOUT_H

ports/mimxrt10xx/common-hal/microcontroller/Pin.c

@@ -29,6 +29,8 @@
 #include "shared-bindings/microcontroller/Pin.h"
 #include "shared-bindings/microcontroller/__init__.h"
 
+#include "sdk/drivers/igpio/fsl_gpio.h"
+
 #include "py/gc.h"
 
 STATIC bool claimed_pins[PAD_COUNT];
@@ -91,6 +93,11 @@ void common_hal_reset_pin(const mcu_pin_obj_t *pin) {
     never_reset_pins[pin->mux_idx] = false;
     claimed_pins[pin->mux_idx] = false;
 
+    // Make sure this pin's GPIO is set to input. Otherwise, output values could interfere
+    // with the ADC.
+    const gpio_pin_config_t config = { kGPIO_DigitalInput, 0, kGPIO_NoIntmode };
+    GPIO_PinInit(pin->gpio, pin->number, &config);
+
     // This should never be true, but protect against it anyway.
     if (pin->mux_reg == 0) {
         return;

ports/mimxrt10xx/common-hal/pwmio/PWMOut.c

@@ -213,7 +213,6 @@ pwmout_result_t common_hal_pwmio_pwmout_construct(pwmio_pwmout_obj_t *self,
 
         /*
          * pwmConfig.enableDebugMode = false;
-         * pwmConfig.enableWait = false;
          * pwmConfig.reloadSelect = kPWM_LocalReload;
          * pwmConfig.faultFilterCount = 0;
          * pwmConfig.faultFilterPeriod = 0;
@@ -228,7 +227,6 @@ pwmout_result_t common_hal_pwmio_pwmout_construct(pwmio_pwmout_obj_t *self,
         PWM_GetDefaultConfig(&pwmConfig);
 
         pwmConfig.reloadLogic = kPWM_ReloadPwmFullCycle;
-        pwmConfig.enableWait = true;
         pwmConfig.enableDebugMode = true;
 
         pwmConfig.prescale = self->prescaler;

ports/mimxrt10xx/sdk

@@ -43,6 +43,7 @@
 #include "common-hal/microcontroller/Pin.h"
 #include "common-hal/pwmio/PWMOut.h"
 #include "common-hal/rtc/RTC.h"
+#include "common-hal/busio/I2C.h"
 #include "common-hal/busio/SPI.h"
 #include "shared-bindings/microcontroller/__init__.h"
 
@@ -447,6 +448,7 @@ safe_mode_t port_init(void) {
 
 void reset_port(void) {
     #if CIRCUITPY_BUSIO
+    i2c_reset();
     spi_reset();
     #endif