tests: drivers: uart: Adding wide data cfg tests

Extending configuration tests with wide data support.

Signed-off-by: Jeroen van Dooren <[email protected]>

Author: JvanDooren

tests/drivers/uart/uart_basic_api/CMakeLists.txt

@@ -11,3 +11,4 @@ target_sources(app PRIVATE
     )
 target_sources_ifdef(CONFIG_UART_INTERRUPT_DRIVEN app PRIVATE src/test_uart_fifo.c)
 target_sources_ifdef(CONFIG_UART_INTERRUPT_DRIVEN app PRIVATE src/test_uart_pending.c)
+target_sources_ifdef(CONFIG_UART_WIDE_DATA app PRIVATE src/test_uart_config_wide.c)

tests/drivers/uart/uart_basic_api/boards/nucleo_h743zi.overlay

@@ -0,0 +1,10 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+dut: &usart2 {
+	pinctrl-0 = <&usart2_tx_pd5 &usart2_rx_pd6>;
+	pinctrl-names = "default";
+	current-speed = <115200>;
+	status = "okay";
+};

tests/drivers/uart/uart_basic_api/src/test_uart.h

@@ -23,11 +23,16 @@ void test_uart_configure(void);
 void test_uart_config_get(void);
 void test_uart_poll_out(void);
 void test_uart_poll_in(void);
+#if CONFIG_UART_WIDE_DATA
+void test_uart_configure_wide(void);
+void test_uart_config_get_wide(void);
+#endif
 #if CONFIG_UART_INTERRUPT_DRIVEN
 void test_uart_fifo_fill(void);
 void test_uart_fifo_read(void);
 void test_uart_pending(void);
 #endif
+
 #endif
 
 #endif /* __TEST_UART_H__ */

tests/drivers/uart/uart_basic_api/src/test_uart_config_wide.c

@@ -0,0 +1,114 @@
+/*
+ * Copyright (c) 2023 Jeroen van Dooren
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+/*
+ * @addtogroup t_uart_basic
+ * @{
+ * @defgroup t_uart_config test_uart_config_wide
+ * @brief TestPurpose: verify UART configure API settings for wide data support
+ * @details
+ * - Test Steps
+ *   - Configure: test_uart_configure_wide( )
+ *   - Configure Get: test_uart_config_get_wide( )
+ * - Expected Results
+ *   -# When test UART CONFIG Configure, the value of configurations actually
+ *      set will be equal to the original configuration values (from device
+ *      tree or run-time configuration to modify those loaded initially from
+ *      device tree)
+ *   -# When test UART CONFIG Configure Get, the app will get/retrieve the
+ *      value of configurations stored at location and to be passed to UART
+ *      CONFIG Configure
+ * @}
+ */
+
+#include "test_uart.h"
+const struct uart_config uart_cfg_wide = {
+		.baudrate = 115200,
+		.parity = UART_CFG_PARITY_NONE,
+		.stop_bits = UART_CFG_STOP_BITS_1,
+		.data_bits = UART_CFG_DATA_BITS_9,
+		.flow_ctrl = UART_CFG_FLOW_CTRL_NONE
+	};
+
+static int test_configure_wide(void)
+{
+	const struct device *const uart_dev = DEVICE_DT_GET(DT_NODELABEL(dut));
+
+	if (!device_is_ready(uart_dev)) {
+		TC_PRINT("UART device not ready\n");
+		return TC_FAIL;
+	}
+
+	/* Verify configure() - set device configuration using data in cfg */
+	int ret = uart_configure(uart_dev, &uart_cfg_wide);
+
+	if (ret == -ENOSYS) {
+		return TC_SKIP;
+	}
+
+	/* 0 if successful, - error code otherwise */
+	return (ret == 0) ? TC_PASS : TC_FAIL;
+
+}
+
+/* test UART configure get (retrieve configuration) */
+static int test_config_get_wide(void)
+{
+	struct uart_config uart_cfg_check;
+	const struct device *const uart_dev = DEVICE_DT_GET(DT_NODELABEL(dut));
+
+	if (!device_is_ready(uart_dev)) {
+		TC_PRINT("UART device not ready\n");
+		return TC_FAIL;
+	}
+
+	TC_PRINT("This is a configure_get_wide test.\n");
+
+	/* Verify configure() - set device configuration using data in cfg */
+	/* 0 if successful, - error code otherwise */
+	int ret = uart_configure(uart_dev, &uart_cfg_wide);
+
+	if (ret == -ENOSYS) {
+		return TC_SKIP;
+	}
+
+	zassert_true(ret == 0, "set config error");
+
+	/* Verify config_get() - get device configuration, put in cfg */
+	/* 0 if successful, - error code otherwise */
+	/* so get the configurations from the device and check */
+	ret = uart_config_get(uart_dev, &uart_cfg_check);
+	zassert_true(ret == 0, "get config error");
+
+	/* Confirm the values from device are the values put in cfg*/
+	if (memcmp(&uart_cfg_wide, &uart_cfg_check, sizeof(uart_cfg_wide)) != 0) {
+		return TC_FAIL;
+	} else {
+		return TC_PASS;
+	}
+}
+
+#if CONFIG_SHELL
+void test_uart_configure_wide(void)
+#else
+ZTEST(uart_basic_api, test_uart_configure_wide)
+#endif
+{
+	int ret = test_configure_wide();
+
+	zassert_true((ret == TC_PASS) || (ret == TC_SKIP));
+}
+
+#if CONFIG_SHELL
+void test_uart_config_get_wide(void)
+#else
+ZTEST(uart_basic_api, test_uart_config_get_wide)
+#endif
+{
+	int ret = test_config_get_wide();
+
+	zassert_true((ret == TC_PASS) || (ret == TC_SKIP));
+}

tests/drivers/uart/uart_basic_api/testcase.yaml

@@ -7,6 +7,16 @@ tests:
     harness: keyboard
     integration_platforms:
       - mps2_an385
+  drivers.uart.basic_api.wide:
+    extra_configs:
+      - CONFIG_UART_WIDE_DATA=y
+    tags: drivers uart
+    filter: CONFIG_UART_CONSOLE
+    arch_allow: arm
+    platform_allow: nucleo_h743zi
+    integration_platforms:
+      - nucleo_h743zi
+    extra_args: DTC_OVERLAY_FILE="boards/nucleo_h743zi.overlay"
   drivers.uart.basic_api.poll:
     extra_args: CONF_FILE=prj_poll.conf
     tags: