samples: peripheral: lpuart: verify sample output

nordic-piks · nordicjm · commit 008bf27053e4 · 2024-11-08T12:30:23.000Z
Check both ASYNC and IRQ mode.

Signed-off-by: Piotr Kosycarz &lt;piotr.kosycarz@nordicsemi.no&gt;
diff --git a/samples/peripheral/lpuart/sample.yaml b/samples/peripheral/lpuart/sample.yaml
@@ -1,6 +1,10 @@
 sample:
   description: Low Power UART sample
   name: Low Power UART
+
+common:
+  tags: ci_build sysbuild ci_samples_peripheral_lpuart
+
 tests:
   sample.peripheral.lpuart:
     sysbuild: true
@@ -22,10 +26,8 @@ tests:
       - nrf21540dk/nrf52840
       - nrf54l15dk/nrf54l15/cpuapp
     platform_exclude: native_posix
-    tags: sysbuild ci_samples_peripheral_lpuart
   sample.peripheral.lpuart.debug:
     sysbuild: true
-    build_only: true
     platform_allow:
       - nrf52833dk/nrf52833
       - nrf52840dk/nrf52840
@@ -41,7 +43,20 @@ tests:
       - CONFIG_UART_CONSOLE=y
       - CONFIG_LOG=y
       - CONFIG_NRF_SW_LPUART_LOG_LEVEL_DBG=y
-    tags: ci_build sysbuild ci_samples_peripheral_lpuart
+    harness: console
+    harness_config:
+      fixture: gpio_loopback
+      type: multi_line
+      ordered: true
+      regex:
+        - ".*app: ASYNC Tx sent 5 bytes"
+        - ".*app: ASYNC Received data 5 bytes"
+        - ".*app: ASYNC Received data 1 bytes"
+        - ".*app: ASYNC Tx sent 5 bytes"
+        - ".*app: ASYNC Received data 5 bytes"
+        - ".*app: ASYNC Received data 1 bytes"
+        - ".*app: ASYNC Tx sent 5 bytes"
+        - ".*app: ASYNC Received data 5 bytes"
   sample.peripheral.lpuart_int_driven:
     sysbuild: true
     build_only: true
@@ -57,4 +72,36 @@ tests:
       - nrf54l15dk/nrf54l15/cpuapp
     platform_allow: nrf52dk/nrf52832 nrf52833dk/nrf52833 nrf52840dk/nrf52840 nrf9160dk/nrf9160/ns
       nrf5340dk/nrf5340/cpuapp nrf21540dk/nrf52840 nrf54l15dk/nrf54l15/cpuapp
-    tags: ci_build sysbuild ci_samples_peripheral_lpuart
+  sample.peripheral.lpuart_int_driven.debug:
+    sysbuild: true
+    extra_configs:
+      - CONFIG_NRF_SW_LPUART_INT_DRIVEN=y
+      - CONFIG_ASSERT=y
+      - CONFIG_CONSOLE=y
+      - CONFIG_UART_CONSOLE=y
+      - CONFIG_LOG=y
+      - CONFIG_NRF_SW_LPUART_LOG_LEVEL_DBG=y
+    integration_platforms:
+      - nrf52dk/nrf52832
+      - nrf52833dk/nrf52833
+      - nrf52840dk/nrf52840
+      - nrf9160dk/nrf9160/ns
+      - nrf5340dk/nrf5340/cpuapp
+      - nrf21540dk/nrf52840
+      - nrf54l15dk/nrf54l15/cpuapp
+    platform_allow: nrf52dk/nrf52832 nrf52833dk/nrf52833 nrf52840dk/nrf52840 nrf9160dk/nrf9160/ns
+      nrf5340dk/nrf5340/cpuapp nrf21540dk/nrf52840 nrf54l15dk/nrf54l15/cpuapp
+    harness: console
+    harness_config:
+      fixture: gpio_loopback
+      type: multi_line
+      ordered: true
+      regex:
+        - ".*app: IRQ Tx sent 5 bytes"
+        - ".*app: IRQ Received data 5 bytes"
+        - ".*app: IRQ Received data 1 bytes"
+        - ".*app: IRQ Tx sent 5 bytes"
+        - ".*app: IRQ Received data 5 bytes"
+        - ".*app: IRQ Received data 1 bytes"
+        - ".*app: IRQ Tx sent 5 bytes"
+        - ".*app: IRQ Received data 5 bytes"
diff --git a/samples/peripheral/lpuart/src/main.c b/samples/peripheral/lpuart/src/main.c
@@ -4,6 +4,8 @@
  * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
  */
 
+#include <string.h>
+#include <stdio.h>
 #include <zephyr/kernel.h>
 #include <zephyr/device.h>
 #include <zephyr/devicetree.h>
@@ -15,21 +17,46 @@ LOG_MODULE_REGISTER(app);
 #define BUF_SIZE 64
 static K_MEM_SLAB_DEFINE(uart_slab, BUF_SIZE, 3, 4);
 
-static void uart_irq_handler(const struct device *dev, void *context)
+#define TX_DATA_SIZE 5
+static const uint8_t tx_buf[TX_DATA_SIZE] = {1, 2, 3, 4, 5};
+static uint8_t buffer_print[TX_DATA_SIZE * 5 + 1];
+
+static void to_display_format(const uint8_t *src, size_t size, char *dst)
+{
+	size_t i;
+
+	for (i = 0; i < size; i++) {
+		sprintf(dst + 5 * i, "0x%02x,", src[i]);
+	}
+}
+
+static void verify_buffers(const uint8_t *tx_data, char *rx_data, size_t size)
 {
-	uint8_t buf[] = {1, 2, 3, 4, 5};
+	if (memcmp(tx_data, rx_data, size)) {
+		to_display_format(tx_data, size, buffer_print);
+		LOG_ERR("Buffer contents are different: %s", buffer_print);
+		to_display_format(rx_data, size, buffer_print);
+		LOG_ERR("                           vs: %s", buffer_print);
+		__ASSERT(false, "Buffer contents are different");
+	}
+}
 
+static void uart_irq_handler(const struct device *dev, void *context)
+{
 	if (uart_irq_tx_ready(dev)) {
-		(void)uart_fifo_fill(dev, buf, sizeof(buf));
+		(void)uart_fifo_fill(dev, tx_buf, TX_DATA_SIZE);
 		uart_irq_tx_disable(dev);
+		LOG_INF("IRQ Tx sent %d bytes", TX_DATA_SIZE);
 	}
 
 	if (uart_irq_rx_ready(dev)) {
-		uint8_t buf[10];
+		uint8_t buf[TX_DATA_SIZE + 1];
 		int len = uart_fifo_read(dev, buf, sizeof(buf));
 
-		if (len) {
-			printk("read %d bytes\n", len);
+		LOG_INF("IRQ Received data %d bytes", len);
+		__ASSERT(len == 1 || len == TX_DATA_SIZE, "Received unexpected data");
+		if (len == TX_DATA_SIZE) {
+			verify_buffers(tx_buf, buf, TX_DATA_SIZE);
 		}
 	}
 }
@@ -49,28 +76,31 @@ static void interrupt_driven(const struct device *dev)
 	}
 }
 
-static void uart_callback(const struct device *dev,
-			  struct uart_event *evt,
-			  void *user_data)
+static void uart_callback(const struct device *dev, struct uart_event *evt, void *user_data)
 {
 	struct device *uart = user_data;
 	int err;
 
 	switch (evt->type) {
 	case UART_TX_DONE:
-		LOG_INF("Tx sent %d bytes", evt->data.tx.len);
+		LOG_INF("ASYNC Tx sent %d bytes", evt->data.tx.len);
 		break;
 
 	case UART_TX_ABORTED:
-		LOG_ERR("Tx aborted");
+		LOG_ERR("ASYNC Tx aborted");
 		break;
 
 	case UART_RX_RDY:
-		LOG_INF("Received data %d bytes", evt->data.rx.len);
+		LOG_INF("ASYNC Received data %d bytes", evt->data.rx.len);
+		__ASSERT(evt->data.rx.len == 1 || evt->data.rx.len == TX_DATA_SIZE,
+			 "Received unexpected data");
+		if (evt->data.rx.len == TX_DATA_SIZE) {
+			verify_buffers(tx_buf, evt->data.rx.buf + evt->data.rx.offset,
+				       TX_DATA_SIZE);
+		}
 		break;
 
-	case UART_RX_BUF_REQUEST:
-	{
+	case UART_RX_BUF_REQUEST: {
 		uint8_t *buf;
 
 		err = k_mem_slab_alloc(&uart_slab, (void **)&buf, K_NO_WAIT);
@@ -95,7 +125,6 @@ static void uart_callback(const struct device *dev,
 
 static void async(const struct device *lpuart)
 {
-	uint8_t txbuf[5] = {1, 2, 3, 4, 5};
 	int err;
 	uint8_t *buf;
 
@@ -109,12 +138,12 @@ static void async(const struct device *lpuart)
 	__ASSERT(err == 0, "Failed to enable RX");
 
 	while (1) {
-		err = uart_tx(lpuart, txbuf, sizeof(txbuf), 10000);
+		err = uart_tx(lpuart, tx_buf, sizeof(tx_buf), 10000);
 		__ASSERT(err == 0, "Failed to initiate transmission");
 
 		k_sleep(K_MSEC(500));
 
-		uart_poll_out(lpuart, txbuf[0]);
+		uart_poll_out(lpuart, tx_buf[0]);
 		k_sleep(K_MSEC(100));
 	}
 }
