From a6ed05e76532210b6e53124a897243f15e4f9e9e Mon Sep 17 00:00:00 2001 From: Emilio Benavente Date: Wed, 4 Sep 2024 15:24:32 +0000 Subject: [PATCH] tests: drivers: uart_async_api: fix userspace fault Updated overlay files to test using nocache. and run outside of usersapce mode to avoid inaccessible memory regions when using nocache on RT parts. Also updated testcase file to reflect nocache testing. Signed-off-by: Emilio Benavente --- .../uart/uart_async_api/boards/mimxrt1010_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1015_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1020_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1024_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1050_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1060_evk.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1060_evkb.overlay | 6 ------ .../uart/uart_async_api/boards/mimxrt1064_evk.overlay | 6 ------ .../boards/mimxrt1160_evk_mimxrt1166_cm7.overlay | 6 ------ .../boards/mimxrt1170_evk_mimxrt1176_cm7_A.overlay | 6 ------ tests/drivers/uart/uart_async_api/testcase.yaml | 10 +++++++++- 11 files changed, 9 insertions(+), 61 deletions(-) diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1010_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1010_evk.overlay index 02c9390abbc296..4dc4c119d905f7 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1010_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1010_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart4 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1015_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1015_evk.overlay index da52ae8d9d12c2..f82d4c29082ffd 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1015_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1015_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart4 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1020_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1020_evk.overlay index cceca9150179ec..4f031193d15229 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1020_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1020_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart2 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1024_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1024_evk.overlay index b8b4e96bc15d84..3349704181a00d 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1024_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1024_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart2 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1050_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1050_evk.overlay index 1a0c35e198aa76..22ecfdfa712aea 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1050_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1050_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart3 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evk.overlay index 1a0c35e198aa76..22ecfdfa712aea 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart3 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evkb.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evkb.overlay index 1a0c35e198aa76..22ecfdfa712aea 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evkb.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1060_evkb.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart3 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1064_evk.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1064_evk.overlay index 1a0c35e198aa76..22ecfdfa712aea 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1064_evk.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1064_evk.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart3 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1160_evk_mimxrt1166_cm7.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1160_evk_mimxrt1166_cm7.overlay index cceca9150179ec..4f031193d15229 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1160_evk_mimxrt1166_cm7.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1160_evk_mimxrt1166_cm7.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart2 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/boards/mimxrt1170_evk_mimxrt1176_cm7_A.overlay b/tests/drivers/uart/uart_async_api/boards/mimxrt1170_evk_mimxrt1176_cm7_A.overlay index cceca9150179ec..4f031193d15229 100644 --- a/tests/drivers/uart/uart_async_api/boards/mimxrt1170_evk_mimxrt1176_cm7_A.overlay +++ b/tests/drivers/uart/uart_async_api/boards/mimxrt1170_evk_mimxrt1176_cm7_A.overlay @@ -1,11 +1,5 @@ /* SPDX-License-Identifier: Apache-2.0 */ -/ { - chosen { - zephyr,sram = &dtcm; - }; -}; - dut: &lpuart2 { status = "okay"; current-speed = <115200>; diff --git a/tests/drivers/uart/uart_async_api/testcase.yaml b/tests/drivers/uart/uart_async_api/testcase.yaml index e7a970d4a58abe..91ef347af34250 100644 --- a/tests/drivers/uart/uart_async_api/testcase.yaml +++ b/tests/drivers/uart/uart_async_api/testcase.yaml @@ -58,9 +58,17 @@ tests: integration_platforms: - qemu_cortex_m0 drivers.uart.async_api.lpuart: - filter: CONFIG_SERIAL_SUPPORT_ASYNC and CONFIG_UART_MCUX_LPUART + filter: CONFIG_SERIAL_SUPPORT_ASYNC and CONFIG_UART_MCUX_LPUART and not CONFIG_CPU_HAS_DCACHE harness: ztest depends_on: dma + drivers.uart.async_api.lpuart.rt_nocache: + filter: CONFIG_SERIAL_SUPPORT_ASYNC and CONFIG_UART_MCUX_LPUART and CONFIG_CPU_HAS_DCACHE + harness: ztest + depends_on: dma + extra_configs: + - CONFIG_DCACHE=y + - CONFIG_NOCACHE_MEMORY=y + - CONFIG_USERSPACE=n drivers.uart.async_api.sam0: filter: CONFIG_SERIAL_SUPPORT_ASYNC and CONFIG_SOC_FAMILY_ATMEL_SAM0 platform_allow: