imx9/px4io: Add error checking for all the DMA operations

Check the return values from DMA operations and add a performance counter to detect if there are any DMA related errors. Signed-off-by: Jukka Laitinen <[email protected]>
tiiuae · Sep 30, 2024 · f79f7cf · f79f7cf
1 parent 36bddac
commit f79f7cf
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 11 deletions.
diff --git a/platforms/nuttx/src/px4/nxp/imx9/include/px4_arch/px4io_serial.h b/platforms/nuttx/src/px4/nxp/imx9/include/px4_arch/px4io_serial.h
@@ -134,6 +134,10 @@ class ArchPX4IOSerial : public PX4IO_serial
 
 	px4_sem_t		_recv_sem;
 
+	/** dma error perf counter */
+
+	perf_counter_t		_pc_dmaerrs;
+
 	/**
 	 * DMA completion handler.
 	 */

diff --git a/platforms/nuttx/src/px4/nxp/imx9/px4io_serial/px4io_serial.cpp b/platforms/nuttx/src/px4/nxp/imx9/px4io_serial/px4io_serial.cpp
@@ -38,6 +38,7 @@
  */
 
 #include <px4_arch/px4io_serial.h>
+#include <perf/perf_counter.h>
 
 #include <syslog.h>
 
@@ -63,7 +64,8 @@ static const struct uart_config_s g_px4io_config = {
 
 ArchPX4IOSerial::ArchPX4IOSerial() :
 	_current_packet(nullptr),
-	_recv_sem(SEM_INITIALIZER(0))
+	_recv_sem(SEM_INITIALIZER(0)),
+	_pc_dmaerrs(perf_alloc(PC_COUNT, "px4io: dmaerrs"))
 {
 	px4_sem_setprotocol(&_recv_sem, SEM_PRIO_NONE);
 
@@ -89,6 +91,8 @@ ArchPX4IOSerial::~ArchPX4IOSerial()
 
 	imx9_dmach_free(_rx_dma);
 	imx9_dmach_free(_tx_dma);
+
+	perf_free(_pc_dmaerrs);
 }
 
 int
@@ -162,6 +166,7 @@ ArchPX4IOSerial::ioctl(unsigned operation, unsigned &arg)
 						perf_print_counter(_pc_txns);
 						perf_print_counter(_pc_retries);
 						perf_print_counter(_pc_timeouts);
+						perf_print_counter(_pc_dmaerrs);
 						perf_print_counter(_pc_crcerrs);
 						perf_print_counter(_pc_protoerrs);
 						perf_print_counter(_pc_uerrs);
@@ -217,9 +222,7 @@ ArchPX4IOSerial::_bus_exchange(IOPacket *_packet)
 			(uintptr_t)_current_packet + DMA_ALIGN_UP(sizeof(IOPacket)));
 #endif
 
-	/* Start the RX DMA */
-
-	_waiting_for_dma = true;
+	/* Configure the RX and TX DMA */
 
 	struct imx9_edma_xfrconfig_s rx_config;
 	rx_config.saddr  = PX4IO_SERIAL_BASE + IMX9_LPUART_DATA_OFFSET;
@@ -235,11 +238,6 @@ ArchPX4IOSerial::_bus_exchange(IOPacket *_packet)
 	rx_config.linkch = NULL;
 #endif
 
-	imx9_dmach_xfrsetup(_rx_dma, &rx_config);
-	imx9_dmach_start(_rx_dma, _dma_callback, (void *)this);
-
-	/* Start the TX DMA */
-
 	struct imx9_edma_xfrconfig_s tx_config;
 
 	tx_config.saddr  = reinterpret_cast<uintptr_t>(_current_packet);
@@ -255,8 +253,29 @@ ArchPX4IOSerial::_bus_exchange(IOPacket *_packet)
 	tx_config.linkch = NULL;
 #endif
 
-	imx9_dmach_xfrsetup(_tx_dma, &tx_config);
-	imx9_dmach_start(_tx_dma, nullptr, nullptr);
+	/* If setup fails, just stop and exit */
+
+	if (imx9_dmach_xfrsetup(_rx_dma, &rx_config) != OK ||
+	    imx9_dmach_xfrsetup(_tx_dma, &tx_config) != OK) {
+		_stop_dma();
+		perf_count(_pc_dmaerrs);
+		perf_cancel(_pc_txns);
+		return -EIO;
+	}
+
+	/* Start DMA. imx9_dmach_start can only return OK at the time of writing this,
+	 * but keep the error checking to look proper
+	 */
+
+	_waiting_for_dma = true;
+
+	if (imx9_dmach_start(_rx_dma, _dma_callback, (void *)this) != OK ||
+	    imx9_dmach_start(_tx_dma, nullptr, nullptr) != OK) {
+		_stop_dma();
+		perf_count(_pc_dmaerrs);
+		perf_cancel(_pc_txns);
+		return -EIO;
+	}
 
 	/* Wait for response, max 10 ms */