ci(p4): p4 eco5 enable parts of special tests

components/esp_driver_spi/test_apps/.build-test-rules.yml

@@ -9,36 +9,20 @@
 components/esp_driver_spi/test_apps/master:
   disable:
     - if: SOC_GPSPI_SUPPORTED != 1
-  disable_test:
-    - if: IDF_TARGET == "esp32p4"
-      temporary: true
-      reason: p4 rev3 migration # TODO: IDF-14399
   <<: *spi_depends_default
 
 components/esp_driver_spi/test_apps/param:
   disable:
     - if: SOC_GPSPI_SUPPORTED != 1
-  disable_test:
-    - if: IDF_TARGET in ["esp32h21", "esp32p4"]
-      temporary: true
-      reason: p4 rev3 migration # TODO: IDF-14399
   <<: *spi_depends_default
 
 components/esp_driver_spi/test_apps/slave:
   disable:
     - if: SOC_GPSPI_SUPPORTED != 1
-  disable_test:
-    - if: IDF_TARGET in ["esp32h21", "esp32p4"]
-      temporary: true
-      reason: p4 rev3 migration # TODO: IDF-14399
   <<: *spi_depends_default
 
 components/esp_driver_spi/test_apps/slave_hd:
   disable:
     - if: SOC_GPSPI_SUPPORTED != 1
     - if: SOC_SPI_SUPPORT_SLAVE_HD_VER2 != 1
-  disable_test:
-    - if: IDF_TARGET in ["esp32h21", "esp32p4", "esp32c61"]
-      temporary: true
-      reason: p4 rev3 migration # TODO: IDF-14399
   <<: *spi_depends_default

components/esp_driver_spi/test_apps/master/main/test_spi_master.c

@@ -2159,7 +2159,9 @@ TEST_CASE("SPI_Master: PSRAM buffer transaction via EDMA", "[spi]")
         TEST_ASSERT(i ? (before - after) < 2 * TEST_EDMA_TRANS_LEN : (before - after) > 2 * TEST_EDMA_TRANS_LEN);
         spi_device_polling_end(dev_handle, portMAX_DELAY);
         printf("TX fail: %d, RX fail: %d\n", !!(trans_cfg.flags & SPI_TRANS_DMA_TX_FAIL), !!(trans_cfg.flags & SPI_TRANS_DMA_RX_FAIL));
+#if !CONFIG_IDF_TARGET_ESP32P4  // P4 can't reach error condition since it has powerful 16bits ddr psram
         TEST_ASSERT((!!i) == !!(trans_cfg.flags & (SPI_TRANS_DMA_TX_FAIL | SPI_TRANS_DMA_RX_FAIL)));
+#endif
         if (!i) { // data should be correct if using auto malloc
             spitest_cmp_or_dump(trans_cfg.tx_buffer, trans_cfg.rx_buffer, TEST_EDMA_TRANS_LEN);
         }

components/esp_driver_spi/test_apps/master/pytest_spi_master.py

@@ -17,7 +17,6 @@ from pytest_embedded_idf.utils import soc_filtered_targets
 @idf_parametrize(
     'target', soc_filtered_targets('SOC_GPSPI_SUPPORTED == 1 and IDF_TARGET not in ["esp32c5"]'), indirect=['target']
 )
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 def test_master_single_dev(case_tester) -> None:  # type: ignore
     for case in case_tester.test_menu:
         if 'test_env' in case.attributes:
@@ -76,7 +75,6 @@ def test_master_esp_flash(case_tester) -> None:  # type: ignore
 
 
 @pytest.mark.generic_multi_device
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 @pytest.mark.parametrize(
     'count, config',
     [

components/esp_driver_spi/test_apps/param/pytest_spi_param.py

@@ -5,7 +5,6 @@ from pytest_embedded_idf.utils import idf_parametrize
 
 
 @pytest.mark.generic
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 @idf_parametrize('target', ['supported_targets'], indirect=['target'])
 def test_param_single_dev(case_tester) -> None:  # type: ignore
     case_tester.run_all_normal_cases(reset=True)
@@ -14,6 +13,5 @@ def test_param_single_dev(case_tester) -> None:  # type: ignore
 @pytest.mark.generic_multi_device
 @pytest.mark.parametrize('count', [2], indirect=True)
 @idf_parametrize('target', ['supported_targets'], indirect=['target'])
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 def test_param_multi_dev(case_tester) -> None:  # type: ignore
     case_tester.run_all_multi_dev_cases(reset=True)

components/esp_driver_spi/test_apps/slave/pytest_spi_slave.py

@@ -7,7 +7,6 @@ from pytest_embedded_idf.utils import soc_filtered_targets
 
 @pytest.mark.generic
 @pytest.mark.parametrize('config', ['release', 'iram_safe'], indirect=True)
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 @idf_parametrize(
     'target', soc_filtered_targets('SOC_GPSPI_SUPPORTED == 1 and IDF_TARGET not in ["esp32c5"]'), indirect=['target']
 )
@@ -32,7 +31,6 @@ def test_slave_single_dev_esp32c5_rev1(case_tester) -> None:  # type: ignore
 
 @pytest.mark.generic_multi_device
 @pytest.mark.parametrize('count, config', [(2, 'release'), (2, 'iram_safe')], indirect=True)
-@pytest.mark.temp_skip_ci(targets=['esp32p4'], reason='p4 rev3 migration # TODO: IDF-14399')
 @idf_parametrize('target', ['supported_targets'], indirect=['target'])
 def test_slave_multi_dev(case_tester) -> None:  # type: ignore
     case_tester.run_all_multi_dev_cases(reset=True)

components/esp_driver_spi/test_apps/slave_hd/pytest_spi_slave_hd.py

@@ -8,7 +8,6 @@ from pytest_embedded_idf.utils import idf_parametrize
 
 @pytest.mark.generic
 @pytest.mark.parametrize('config', ['release'], indirect=True)
-@pytest.mark.temp_skip_ci(targets=['esp32p4', 'esp32c61'], reason='p4 rev3 migration # TODO: IDF-14399')
 @idf_parametrize(
     'target',
     ['esp32s2', 'esp32s3', 'esp32c2', 'esp32c3', 'esp32c6', 'esp32c61', 'esp32h2', 'esp32p4'],
@@ -29,7 +28,6 @@ def test_slave_hd_single_dev_esp32c5_rev1(case_tester) -> None:  # type: ignore
 # if `test_env` not defined, will run on `generic_multi_device` by default
 @pytest.mark.generic_multi_device
 @pytest.mark.parametrize('count, config', [(2, 'release')], indirect=True)
-@pytest.mark.temp_skip_ci(targets=['esp32p4', 'esp32c61'], reason='p4 rev3 migration # TODO: IDF-14399')
 @idf_parametrize(
     'target',
     ['esp32s2', 'esp32s3', 'esp32c2', 'esp32c3', 'esp32c5', 'esp32c6', 'esp32h2', 'esp32p4'],