feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

Merge branch 'test/add_test_spi_sio_master_all' into 'master'

Browse Source 
SPI Master: add sio multi-device test case for all chips

Closes IDF-4455

See merge request espressif/esp-idf!19249
This commit is contained in:
Wan Lei
2022-08-18 14:41:48 +08:00
parent 19d7afa47a 4757efe21d
commit 60d794501f
5 changed files with 456 additions and 75 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
.gitlab/ci/dependencies/dependencies.yml
View File

@@ -169,6 +169,7 @@ build:integration_test:
- ethernet # example_test*ethernet* - ethernet # example_test*ethernet*
- sdio # UT_044, UT_045 - sdio # UT_044, UT_045
- usb # USB Device & Host tests - usb # USB Device & Host tests
- spi_multi # UT_C2_SPI_DUAL
patterns: patterns:
- "{0}-{1}-{2}" - "{0}-{1}-{2}"
- "{0}-{2}" - "{0}-{2}"

298
.gitlab/ci/rules.yml
View File

@@ -229,6 +229,12 @@
- "examples/peripherals/usb/host/**/**/**/*" - "examples/peripherals/usb/host/**/**/**/*"
- "examples/peripherals/usb/device/**/**/*" - "examples/peripherals/usb/device/**/**/*"
# for job: unit_test-spi_multi which has only one runner
.patterns-unit_test-spi_multi: &patterns-unit_test-spi_multi
- "components/esp_serial_slave_link/**/*"
- "components/driver/**/*"
- "examples/peripherals/spi_slave*/**/*"
############## ##############
# if anchors # # if anchors #
@@ -1027,6 +1033,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test: .rules:build:unit_test:
rules: rules:
@@ -1051,6 +1059,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32: .rules:build:unit_test-esp32:
rules: rules:
@@ -1070,6 +1080,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32c2: .rules:build:unit_test-esp32c2:
rules: rules:
@@ -1086,6 +1098,8 @@
changes: *patterns-build_system changes: *patterns-build_system
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32c3: .rules:build:unit_test-esp32c3:
rules: rules:
@@ -1104,6 +1118,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32h2: .rules:build:unit_test-esp32h2:
rules: rules:
@@ -1122,6 +1138,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32s2: .rules:build:unit_test-esp32s2:
rules: rules:
@@ -1140,6 +1158,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:unit_test-esp32s3: .rules:build:unit_test-esp32s3:
rules: rules:
@@ -1158,6 +1178,8 @@
changes: *patterns-unit_test changes: *patterns-unit_test
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:build:windows: .rules:build:windows:
rules: rules:
@@ -1251,6 +1273,17 @@
- <<: *if-label-component_ut_esp32 - <<: *if-label-component_ut_esp32
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32
- <<: *if-label-target_test
.rules:test:component_ut-esp32-usb: .rules:test:component_ut-esp32-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1316,6 +1349,17 @@
- <<: *if-label-component_ut_esp32c2 - <<: *if-label-component_ut_esp32c2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32c2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32c2
- <<: *if-label-target_test
.rules:test:component_ut-esp32c2-usb: .rules:test:component_ut-esp32c2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1386,6 +1430,17 @@
- <<: *if-label-component_ut_esp32c3 - <<: *if-label-component_ut_esp32c3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32c3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32c3
- <<: *if-label-target_test
.rules:test:component_ut-esp32c3-usb: .rules:test:component_ut-esp32c3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1456,6 +1511,17 @@
- <<: *if-label-component_ut_esp32h2 - <<: *if-label-component_ut_esp32h2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32h2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32h2
- <<: *if-label-target_test
.rules:test:component_ut-esp32h2-usb: .rules:test:component_ut-esp32h2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1526,6 +1592,17 @@
- <<: *if-label-component_ut_esp32s2 - <<: *if-label-component_ut_esp32s2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32s2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32s2
- <<: *if-label-target_test
.rules:test:component_ut-esp32s2-usb: .rules:test:component_ut-esp32s2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1591,6 +1668,17 @@
- <<: *if-label-component_ut_esp32s3 - <<: *if-label-component_ut_esp32s3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:component_ut-esp32s3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-component_ut
- <<: *if-label-component_ut_esp32s3
- <<: *if-label-target_test
.rules:test:component_ut-esp32s3-usb: .rules:test:component_ut-esp32s3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1659,6 +1747,17 @@
- <<: *if-label-custom_test_esp32 - <<: *if-label-custom_test_esp32
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32
- <<: *if-label-target_test
.rules:test:custom_test-esp32-usb: .rules:test:custom_test-esp32-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1723,6 +1822,17 @@
- <<: *if-label-custom_test_esp32c2 - <<: *if-label-custom_test_esp32c2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32c2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32c2
- <<: *if-label-target_test
.rules:test:custom_test-esp32c2-usb: .rules:test:custom_test-esp32c2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1791,6 +1901,17 @@
- <<: *if-label-custom_test_esp32c3 - <<: *if-label-custom_test_esp32c3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32c3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32c3
- <<: *if-label-target_test
.rules:test:custom_test-esp32c3-usb: .rules:test:custom_test-esp32c3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1859,6 +1980,17 @@
- <<: *if-label-custom_test_esp32h2 - <<: *if-label-custom_test_esp32h2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32h2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32h2
- <<: *if-label-target_test
.rules:test:custom_test-esp32h2-usb: .rules:test:custom_test-esp32h2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1927,6 +2059,17 @@
- <<: *if-label-custom_test_esp32s2 - <<: *if-label-custom_test_esp32s2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32s2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32s2
- <<: *if-label-target_test
.rules:test:custom_test-esp32s2-usb: .rules:test:custom_test-esp32s2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -1991,6 +2134,17 @@
- <<: *if-label-custom_test_esp32s3 - <<: *if-label-custom_test_esp32s3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:custom_test-esp32s3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-custom_test
- <<: *if-label-custom_test_esp32s3
- <<: *if-label-target_test
.rules:test:custom_test-esp32s3-usb: .rules:test:custom_test-esp32s3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2074,6 +2228,17 @@
- <<: *if-label-example_test_esp32 - <<: *if-label-example_test_esp32
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32
- <<: *if-label-target_test
.rules:test:example_test-esp32-usb: .rules:test:example_test-esp32-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2146,6 +2311,17 @@
- <<: *if-label-example_test_esp32c2 - <<: *if-label-example_test_esp32c2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32c2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32c2
- <<: *if-label-target_test
.rules:test:example_test-esp32c2-usb: .rules:test:example_test-esp32c2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2233,6 +2409,17 @@
- <<: *if-label-example_test_esp32c3 - <<: *if-label-example_test_esp32c3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32c3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32c3
- <<: *if-label-target_test
.rules:test:example_test-esp32c3-usb: .rules:test:example_test-esp32c3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2311,6 +2498,17 @@
- <<: *if-label-example_test_esp32h2 - <<: *if-label-example_test_esp32h2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32h2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32h2
- <<: *if-label-target_test
.rules:test:example_test-esp32h2-usb: .rules:test:example_test-esp32h2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2389,6 +2587,17 @@
- <<: *if-label-example_test_esp32s2 - <<: *if-label-example_test_esp32s2
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32s2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32s2
- <<: *if-label-target_test
.rules:test:example_test-esp32s2-usb: .rules:test:example_test-esp32s2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2461,6 +2670,17 @@
- <<: *if-label-example_test_esp32s3 - <<: *if-label-example_test_esp32s3
- <<: *if-label-target_test - <<: *if-label-target_test
.rules:test:example_test-esp32s3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-example_test
- <<: *if-label-example_test_esp32s3
- <<: *if-label-target_test
.rules:test:example_test-esp32s3-usb: .rules:test:example_test-esp32s3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2569,6 +2789,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32-usb: .rules:test:unit_test-esp32-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2635,6 +2868,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32c2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32c2
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32c2-usb: .rules:test:unit_test-esp32c2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2705,6 +2951,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32c3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32c3
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32c3-usb: .rules:test:unit_test-esp32c3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2775,6 +3034,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32h2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32h2
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32h2-usb: .rules:test:unit_test-esp32h2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2845,6 +3117,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32s2-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32s2
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32s2-usb: .rules:test:unit_test-esp32s2-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch
@@ -2911,6 +3196,19 @@
- <<: *if-dev-push - <<: *if-dev-push
changes: *patterns-unit_test-sdio changes: *patterns-unit_test-sdio
.rules:test:unit_test-esp32s3-spi_multi:
rules:
- <<: *if-revert-branch
when: never
- <<: *if-protected
- <<: *if-label-build-only
when: never
- <<: *if-label-target_test
- <<: *if-label-unit_test
- <<: *if-label-unit_test_esp32s3
- <<: *if-dev-push
changes: *patterns-unit_test-spi_multi
.rules:test:unit_test-esp32s3-usb: .rules:test:unit_test-esp32s3-usb:
rules: rules:
- <<: *if-revert-branch - <<: *if-revert-branch

9
.gitlab/ci/target-test.yml
View File

@@ -972,6 +972,15 @@ UT_C2:
- UT_T1_1 - UT_T1_1
- xtal_40mhz - xtal_40mhz
UT_C2_SPI_DUAL:
extends:
- .unit_test_esp32c2_template
- .rules:test:unit_test-esp32c2-spi_multi
tags:
- ESP32C2_IDF
- Example_SPI_Multi_device
- xtal_40mhz
UT_C2_26M: UT_C2_26M:
extends: .unit_test_esp32c2_template extends: .unit_test_esp32c2_template
tags: tags:

2
components/driver/spi_master.c
View File

@@ -714,7 +714,9 @@ static SPI_MASTER_ISR_ATTR esp_err_t check_trans_valid(spi_device_handle_t handl
SPI_CHECK(!is_half_duplex || !bus_attr->dma_enabled || !rx_enabled || !tx_enabled, "SPI half duplex mode does not support using DMA with both MOSI and MISO phases.", ESP_ERR_INVALID_ARG ); SPI_CHECK(!is_half_duplex || !bus_attr->dma_enabled || !rx_enabled || !tx_enabled, "SPI half duplex mode does not support using DMA with both MOSI and MISO phases.", ESP_ERR_INVALID_ARG );
#endif #endif
#if !SOC_SPI_HD_BOTH_INOUT_SUPPORTED #if !SOC_SPI_HD_BOTH_INOUT_SUPPORTED
//On these chips, HW doesn't support using both TX and RX phases when in halfduplex mode
SPI_CHECK(!is_half_duplex || !tx_enabled || !rx_enabled, "SPI half duplex mode is not supported when both MOSI and MISO phases are enabled.", ESP_ERR_INVALID_ARG); SPI_CHECK(!is_half_duplex || !tx_enabled || !rx_enabled, "SPI half duplex mode is not supported when both MOSI and MISO phases are enabled.", ESP_ERR_INVALID_ARG);
SPI_CHECK(!is_half_duplex || !trans_desc->length || !trans_desc->rxlength, "SPI half duplex mode is not supported when both MOSI and MISO phases are enabled.", ESP_ERR_INVALID_ARG);
#endif #endif
//MOSI phase is skipped only when both tx_buffer and SPI_TRANS_USE_TXDATA are not set. //MOSI phase is skipped only when both tx_buffer and SPI_TRANS_USE_TXDATA are not set.
SPI_CHECK(trans_desc->length != 0 || !tx_enabled, "trans tx_buffer should be NULL and SPI_TRANS_USE_TXDATA should be cleared to skip MOSI phase.", ESP_ERR_INVALID_ARG); SPI_CHECK(trans_desc->length != 0 || !tx_enabled, "trans tx_buffer should be NULL and SPI_TRANS_USE_TXDATA should be cleared to skip MOSI phase.", ESP_ERR_INVALID_ARG);

221
components/driver/test/test_spi_sio.c
View File

@@ -84,7 +84,7 @@ TEST_CASE("SPI Single Board Test SIO", "[spi]")
spi_transaction_t mst_trans; spi_transaction_t mst_trans;
spi_slave_transaction_t slv_trans; spi_slave_transaction_t slv_trans;
spi_slave_transaction_t* ret; spi_slave_transaction_t *ret;
for (int i = 0; i < 8; i ++) { for (int i = 0; i < 8; i ++) {
int tlen = i * 2 + 1; int tlen = i * 2 + 1;
@@ -143,123 +143,194 @@ TEST_CASE("SPI Single Board Test SIO", "[spi]")
#endif //#if (TEST_SPI_PERIPH_NUM >= 2) #endif //#if (TEST_SPI_PERIPH_NUM >= 2)
//TODO IDF-4455
#if !TEMPORARY_DISABLED_FOR_TARGETS(ESP32S2, ESP32C3, ESP32S3, ESP32C2, ESP32H2)
//These tests are ESP32 only due to lack of runners
/******************************************************************************** /********************************************************************************
* Test SIO Master & Slave * Test SIO Master
* SIO Slave is not suported, and one unit test is limited to one feature, so,,,
* sio master test can be splited to singal-input and single-output
*
* for single-output: master slave
* cs-----cs ------------- cs
* clk----clk ------------- clk
* d------mosi------------- mosi
* q miso------------- miso
* master can get input on mosi pin after output finish in sio mode, but in this
* case, master can get no data from slave, so check assert on the slave.
*
* ------------------------------------------------------------------------------
* for single-input: master slave
* cs-----cs ------------- cs
* clk----clk ------------- clk
* d-\ mosi------------- mosi
* q \\--miso------------- miso
* In this case, master can get input data from slave after output finish, but
* slave can get no data from master due to internal broke, besides output data
* from both master and slave on miso line will get conflict in master's output
* frame.
********************************************************************************/ ********************************************************************************/
//if test_mosi is false, test on miso of slave, otherwise test on mosi of slave #define TRANS_LEN 1024
void test_sio_master_round(bool test_mosi) #define MAX_TRANS_BUFF 64
{ #define TEST_NUM 8
spi_device_handle_t spi;
WORD_ALIGNED_ATTR uint8_t rx_buffer[320];
if (test_mosi) { WORD_ALIGNED_ATTR uint8_t sio_master_rx_buff[TRANS_LEN];
ESP_LOGI(MASTER_TAG, "======== TEST MOSI ==========="); WORD_ALIGNED_ATTR uint8_t sio_slave_rx_buff [TRANS_LEN];
} else {
ESP_LOGI(MASTER_TAG, "======== TEST MISO ==========="); void test_sio_master_trans(bool sio_master_in)
{
spi_device_handle_t dev_0;
uint8_t *master_tx_max = heap_caps_calloc(TRANS_LEN * 2, 1, MALLOC_CAP_DMA);
TEST_ASSERT_NOT_NULL_MESSAGE(master_tx_max, "malloc failed, exit.\n");
// write somethin to a long buffer for test long transmition
for (uint16_t i = 0; i < TRANS_LEN; i++) {
master_tx_max[i] = i;
master_tx_max[TRANS_LEN * 2 - i - 1] = i;
} }
spi_bus_config_t bus_cfg = SPI_BUS_TEST_DEFAULT_CONFIG(); spi_bus_config_t bus_cfg = SPI_BUS_TEST_DEFAULT_CONFIG();
if (!test_mosi) bus_cfg.mosi_io_num = bus_cfg.miso_io_num; if (sio_master_in) {
// normally, spi read data from port Q and write data to port D
// test master input from port D (output default.), so link port D (normally named mosi) to miso pin.
bus_cfg.mosi_io_num = bus_cfg.miso_io_num;
printf("\n====================Test sio master input====================\n");
} else {
printf("\n============Test sio master output, data checked by slave.=============\n");
}
bus_cfg.miso_io_num = -1; bus_cfg.miso_io_num = -1;
TEST_ESP_OK(spi_bus_initialize(TEST_SPI_HOST, &bus_cfg, 0)); TEST_ESP_OK(spi_bus_initialize(TEST_SPI_HOST, &bus_cfg, SPI_DMA_CH_AUTO));
spi_device_interface_config_t dev_cfg = SPI_DEVICE_TEST_DEFAULT_CONFIG(); spi_device_interface_config_t dev_cfg = SPI_DEVICE_TEST_DEFAULT_CONFIG();
dev_cfg.flags = SPI_DEVICE_HALFDUPLEX | SPI_DEVICE_3WIRE; dev_cfg.flags = SPI_DEVICE_HALFDUPLEX | SPI_DEVICE_3WIRE;
dev_cfg.clock_speed_hz = 1*1000*1000; dev_cfg.clock_speed_hz = 1 * 1000 * 1000;
TEST_ESP_OK(spi_bus_add_device(TEST_SPI_HOST, &dev_cfg, &spi)); TEST_ESP_OK(spi_bus_add_device(TEST_SPI_HOST, &dev_cfg, &dev_0));
printf("CS:CLK:MO:MI: %d\t%d\t%d\t%d\n", dev_cfg.spics_io_num, bus_cfg.sclk_io_num, bus_cfg.mosi_io_num, bus_cfg.miso_io_num);
for (int i = 0; i < 8; i ++) { for (int i = 0; i < TEST_NUM; i ++) {
int tlen = i*2+1; spi_transaction_t trans = {};
int rlen = 9-i; if (sio_master_in) {
spi_transaction_t t = { // master input only section
.length = tlen*8, trans.rxlength = (i + 1) * 8 * 8;
.tx_buffer = spitest_master_send+i, // test a huge data for last transmition
.rxlength = rlen*8, if (i >= TEST_NUM - 1) {
.rx_buffer = rx_buffer+i, trans.rxlength = TRANS_LEN * 8;
}; }
memset(rx_buffer, 0x66, sizeof(rx_buffer)); trans.rx_buffer = sio_master_rx_buff;
trans.length = 0;
trans.tx_buffer = NULL;
memset(sio_master_rx_buff, 0, sizeof(sio_master_rx_buff));
} else {
// master output only section
trans.length = MAX_TRANS_BUFF / (i + 1) * 8;
// test a huge data for last transmition
if (i >= TEST_NUM - 1) {
trans.length = TRANS_LEN * 8;
}
trans.tx_buffer = master_tx_max;
trans.rxlength = 0;
trans.rx_buffer = NULL;
// use some differnt data
trans.tx_buffer += (i % 2) ? TRANS_LEN : 0;
}
//get signal //get signal
unity_wait_for_signal("slave ready"); unity_wait_for_signal("slave ready");
TEST_ESP_OK(spi_device_transmit(spi, &t)); TEST_ESP_OK(spi_device_transmit(dev_0, &trans));
uint8_t* exp_ptr = spitest_slave_send+i; if (sio_master_in) {
ESP_LOG_BUFFER_HEXDUMP("master tx", t.tx_buffer, tlen, ESP_LOG_INFO); ESP_LOG_BUFFER_HEXDUMP("master rx", trans.rx_buffer, trans.rxlength / 8, ESP_LOG_INFO);
ESP_LOG_BUFFER_HEXDUMP("exp tx", exp_ptr, rlen, ESP_LOG_INFO); TEST_ASSERT_EQUAL_HEX8_ARRAY(master_tx_max + i, trans.rx_buffer, trans.rxlength / 8);
ESP_LOG_BUFFER_HEXDUMP("master rx", t.rx_buffer, rlen, ESP_LOG_INFO); } else {
if (!test_mosi) { printf("%d master output\n", trans.length / 8);
TEST_ASSERT_EQUAL_HEX8_ARRAY(exp_ptr+tlen, t.rx_buffer, rlen); ESP_LOG_BUFFER_HEXDUMP("master tx", trans.tx_buffer, trans.length / 8, ESP_LOG_INFO);
} }
} }
master_free_device_bus(spi); free(master_tx_max);
master_free_device_bus(dev_0);
} }
void test_sio_master(void) void test_sio_slave_emulate(bool sio_master_in)
{ {
test_sio_master_round(true); uint8_t *slave_tx_max = heap_caps_calloc(TRANS_LEN * 2, 1, MALLOC_CAP_DMA);
unity_send_signal("master ready"); TEST_ASSERT_NOT_NULL_MESSAGE(slave_tx_max, "malloc failed, exit.\n");
test_sio_master_round(false);
}
void test_sio_slave_round(bool test_mosi) // write somethin to a long buffer for test long transmition
{ for (uint16_t i = 0; i < TRANS_LEN; i++) {
WORD_ALIGNED_ATTR uint8_t rx_buffer[320]; slave_tx_max[i] = i;
slave_tx_max[TRANS_LEN * 2 - i - 1] = i;
}
if (test_mosi) { if (sio_master_in) {
ESP_LOGI(SLAVE_TAG, "======== TEST MOSI ==========="); printf("\n==================Test sio master input.================\n");
} else { } else {
ESP_LOGI(SLAVE_TAG, "======== TEST MISO ==========="); printf("\n==================Test sio master output.=================\n");
} }
spi_bus_config_t bus_cfg = SPI_BUS_TEST_DEFAULT_CONFIG(); spi_bus_config_t bus_cfg = SPI_BUS_TEST_DEFAULT_CONFIG();
spi_slave_interface_config_t slv_cfg = SPI_SLAVE_TEST_DEFAULT_CONFIG();
#if CONFIG_IDF_TARGET_ESP32
// esp32 use different pin for slave in current runner
bus_cfg.mosi_io_num = spi_periph_signal[TEST_SLAVE_HOST].spid_iomux_pin; bus_cfg.mosi_io_num = spi_periph_signal[TEST_SLAVE_HOST].spid_iomux_pin;
bus_cfg.miso_io_num = spi_periph_signal[TEST_SLAVE_HOST].spiq_iomux_pin; bus_cfg.miso_io_num = spi_periph_signal[TEST_SLAVE_HOST].spiq_iomux_pin;
bus_cfg.sclk_io_num = spi_periph_signal[TEST_SLAVE_HOST].spiclk_iomux_pin; bus_cfg.sclk_io_num = spi_periph_signal[TEST_SLAVE_HOST].spiclk_iomux_pin;
spi_slave_interface_config_t slv_cfg = SPI_SLAVE_TEST_DEFAULT_CONFIG();
slv_cfg.spics_io_num = spi_periph_signal[TEST_SLAVE_HOST].spics0_iomux_pin; slv_cfg.spics_io_num = spi_periph_signal[TEST_SLAVE_HOST].spics0_iomux_pin;
TEST_ESP_OK(spi_slave_initialize(TEST_SLAVE_HOST, &bus_cfg, &slv_cfg, 0)); #endif
TEST_ESP_OK(spi_slave_initialize(TEST_SLAVE_HOST, &bus_cfg, &slv_cfg, SPI_DMA_CH_AUTO));
printf("CS:CLK:MO:MI: %d\t%d\t%d\t%d\n", slv_cfg.spics_io_num, bus_cfg.sclk_io_num, bus_cfg.mosi_io_num, bus_cfg.miso_io_num);
for (int i = 0; i < 8; i++) { for (int i = 0; i < TEST_NUM; i++) {
int tlen = 9-i; spi_slave_transaction_t trans = {};
int rlen = i*2+1; if (sio_master_in) {
spi_slave_transaction_t t = { // slave output only section
.length = (tlen+rlen)*8, trans.length = (i + 1) * 8 * 8;
.tx_buffer = spitest_slave_send+i, // test a huge data for last transmition
.rx_buffer = rx_buffer, if (i >= TEST_NUM - 1) {
}; trans.length = TRANS_LEN * 8;
}
trans.tx_buffer = slave_tx_max + i;
trans.rx_buffer = NULL;
} else {
// slave input only section
trans.length = MAX_TRANS_BUFF / (i + 1) * 8;
// test a huge data for last transmition
if (i >= TEST_NUM - 1) {
trans.length = TRANS_LEN * 8;
}
trans.tx_buffer = NULL;
trans.rx_buffer = sio_slave_rx_buff;
memset(sio_slave_rx_buff, 0, sizeof(sio_slave_rx_buff));
}
TEST_ESP_OK(spi_slave_queue_trans(TEST_SLAVE_HOST, &t, portMAX_DELAY)); TEST_ESP_OK(spi_slave_queue_trans(TEST_SLAVE_HOST, &trans, portMAX_DELAY));
ESP_LOG_BUFFER_HEXDUMP("slave tx", t.tx_buffer, tlen+rlen, ESP_LOG_INFO);
//send signal_idx
unity_send_signal("slave ready"); unity_send_signal("slave ready");
uint8_t *exp_ptr = spitest_master_send+i; spi_slave_transaction_t *p_slave_ret;
spi_slave_transaction_t* ret_t; TEST_ESP_OK(spi_slave_get_trans_result(TEST_SLAVE_HOST, &p_slave_ret, portMAX_DELAY));
TEST_ESP_OK(spi_slave_get_trans_result(TEST_SLAVE_HOST, &ret_t, portMAX_DELAY));
ESP_LOG_BUFFER_HEXDUMP("exp tx", exp_ptr, tlen+rlen, ESP_LOG_INFO); if (sio_master_in) {
ESP_LOG_BUFFER_HEXDUMP("slave rx", t.rx_buffer, tlen+rlen, ESP_LOG_INFO); ESP_LOG_BUFFER_HEXDUMP("slave tx", trans.tx_buffer, trans.length / 8, ESP_LOG_INFO);
if (test_mosi) { } else {
TEST_ASSERT_EQUAL_HEX8_ARRAY(exp_ptr, t.rx_buffer, rlen); ESP_LOG_BUFFER_HEXDUMP("slave rx", trans.rx_buffer, trans.length / 8, ESP_LOG_INFO);
TEST_ASSERT_EQUAL_HEX8_ARRAY(slave_tx_max + TRANS_LEN * (i % 2), trans.rx_buffer, trans.length / 8);
} }
} }
free(slave_tx_max);
spi_slave_free(TEST_SLAVE_HOST); spi_slave_free(TEST_SLAVE_HOST);
} }
void test_sio_slave(void) void test_master_run(void)
{ {
test_sio_slave_round(true); test_sio_master_trans(false);
unity_wait_for_signal("master ready"); unity_send_signal("master ready");
test_sio_slave_round(false); test_sio_master_trans(true);
} }
TEST_CASE_MULTIPLE_DEVICES("sio mode", "[spi][test_env=Example_SPI_Multi_device]", test_sio_master, test_sio_slave); void test_slave_run(void)
#endif //#if !TEMPORARY_DISABLED_FOR_TARGETS(ESP32S2, ESP32C3, ESP32C2, ESP32H2) {
test_sio_slave_emulate(false);
unity_wait_for_signal("master ready");
test_sio_slave_emulate(true);
}
TEST_CASE_MULTIPLE_DEVICES("test sio all", "[spi_ms][test_env=Example_SPI_Multi_device]", test_master_run, test_slave_run);