From aa80dbe2511c235ab2f4632c3e0983f6623ed071 Mon Sep 17 00:00:00 2001
From: muhaidong <muhaidong@espressif.com>
Date: Tue, 25 Jun 2024 15:16:39 +0800
Subject: [PATCH] fix(phy): fix esp32 build fail issue

Closes https://github.com/espressif/esp-idf/issues/11972
---
 components/esp_phy/CMakeLists.txt             | 26 +++++++++----------
 .../esp_phy/include/esp_phy_cert_test.h       |  6 ++---
 examples/phy/.build-test-rules.yml            |  8 +++---
 examples/phy/cert_test/sdkconfig.defaults     |  1 +
 4 files changed, 22 insertions(+), 19 deletions(-)

diff --git a/components/esp_phy/CMakeLists.txt b/components/esp_phy/CMakeLists.txt
index 9c333eda46..d7bec39ef5 100644
--- a/components/esp_phy/CMakeLists.txt
+++ b/components/esp_phy/CMakeLists.txt
@@ -73,6 +73,19 @@ if(CONFIG_ESP_PHY_ENABLED)
 
         idf_component_get_property(esp_phy_lib esp_phy COMPONENT_LIB)
 
+        if(CONFIG_ESP_PHY_ENABLE_CERT_TEST)
+            if(CONFIG_IDF_TARGET_ESP32)
+                target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libbttestmode.a
+                librftest.a $<TARGET_FILE:${esp_phy_lib}>)
+            elseif(CONFIG_IDF_TARGET_ESP32S2)
+                target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> librftest.a
+                librfate.a $<TARGET_FILE:${esp_phy_lib}>)
+            else()
+                target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libbttestmode.a
+                librfate.a librftest.a $<TARGET_FILE:${esp_phy_lib}>)
+            endif()
+        endif()
+
         if(CONFIG_IDF_TARGET_ESP32)
             target_link_libraries(${COMPONENT_LIB} PUBLIC rtc)
             target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libphy.a librtc.a
@@ -85,20 +98,7 @@ if(CONFIG_ESP_PHY_ENABLED)
             target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libphy.a
             $<TARGET_FILE:${esp_phy_lib}>)
         endif()
-
-    if(CONFIG_ESP_PHY_ENABLE_CERT_TEST)
-        if(CONFIG_IDF_TARGET_ESP32)
-            target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libbttestmode.a
-            librftest.a $<TARGET_FILE:${esp_phy_lib}>)
-        elseif(CONFIG_IDF_TARGET_ESP32S2)
-            target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> librftest.a
-            librfate.a $<TARGET_FILE:${esp_phy_lib}>)
-        else()
-            target_link_libraries(${COMPONENT_LIB} INTERFACE $<TARGET_FILE:${esp_phy_lib}> libbttestmode.a
-            librfate.a librftest.a $<TARGET_FILE:${esp_phy_lib}>)
-        endif()
     endif()
-endif()
 
    if(CONFIG_ESP_PHY_INIT_DATA_IN_PARTITION)
         idf_component_get_property(esp_common_dir esp_common COMPONENT_DIR)
diff --git a/components/esp_phy/include/esp_phy_cert_test.h b/components/esp_phy/include/esp_phy_cert_test.h
index 461295683c..75e6b5e093 100644
--- a/components/esp_phy/include/esp_phy_cert_test.h
+++ b/components/esp_phy/include/esp_phy_cert_test.h
@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -63,7 +63,7 @@ typedef struct {
     uint32_t phy_rx_correct_count; /*!< The number of desired packets received */
     int phy_rx_rssi;               /*!< Average RSSI of desired packets */
     uint32_t phy_rx_total_count;   /*!< The number of total packets received */
-    uint32_t phy_rx_result_flag;   /*!< 0 means no RX info; 1 means the lastest Wi-Fi RX info; 2 means the lastest BLE RX info. */
+    uint32_t phy_rx_result_flag;   /*!< 0 means no RX info; 1 means the latest Wi-Fi RX info; 2 means the latest BLE RX info. */
 } esp_phy_rx_result_t;
 
 /**
@@ -120,7 +120,7 @@ void esp_phy_cbw40m_en(bool en);
 void esp_phy_wifi_tx(uint32_t chan, esp_phy_wifi_rate_t rate, int8_t backoff, uint32_t length_byte, uint32_t packet_delay, uint32_t packet_num);
 
 /**
- * @brief Test start/stop command, used to stop transmitting or reciving state.
+ * @brief Test start/stop command, used to stop transmitting or receiving state.
  *
  * @param value:
  *     Value should be set to 3 before TX/RX.
diff --git a/examples/phy/.build-test-rules.yml b/examples/phy/.build-test-rules.yml
index 60dcb493ef..bd914b1c5e 100644
--- a/examples/phy/.build-test-rules.yml
+++ b/examples/phy/.build-test-rules.yml
@@ -8,7 +8,9 @@ examples/phy/antenna:
     - if: IDF_TARGET in ["esp32p4", "esp32h2"]
       reason: not supported
 examples/phy/cert_test:
-  enable:
-    - if: IDF_TARGET in ["esp32", "esp32c2", "esp32c3", "esp32c6", "esp32h2", "esp32s2", "esp32s3"]
+  disable:
+    - if: IDF_TARGET in ["esp32p4"]
+      reason: not supported
+    - if: IDF_TARGET == "esp32c5"
       temporary: true
-      reason: the other targets are not tested yet
+      reason: not supported yet  # TODO: [ESP32C5] IDF-8851
diff --git a/examples/phy/cert_test/sdkconfig.defaults b/examples/phy/cert_test/sdkconfig.defaults
index b0ce7b5c29..13e6bed13d 100644
--- a/examples/phy/cert_test/sdkconfig.defaults
+++ b/examples/phy/cert_test/sdkconfig.defaults
@@ -2,3 +2,4 @@ CONFIG_ESP_TASK_WDT_EN=n
 
 CONFIG_ESP_PHY_INIT_DATA_IN_PARTITION=y
 CONFIG_ESP_PHY_ENABLE_CERT_TEST=y
+CONFIG_BT_ENABLED=y