Revert "Merge branch 'espressif:master' into master"

This reverts commit 7384a13, reversing
changes made to 1830ebf.

.gitmodules

@@ -1,6 +1,3 @@
 [submodule "components/asio/asio"]
 	path = components/asio/asio
 	url = https://github.com/espressif/asio
-[submodule "components/mosq/mosquitto"]
-	path = components/mosquitto/mosquitto
-	url = https://github.com/eclipse/mosquitto

.pre-commit-config.yaml

@@ -62,7 +62,7 @@ repos:
     hooks:
       - id: commit message scopes
         name: "commit message must be scoped with: mdns, modem, websocket, asio, mqtt_cxx, console, common, eppp, wifi_remote, tls_cxx"
-        entry: '\A(?!(feat|fix|ci|bump|test|docs)\((mdns|modem|common|console|websocket|asio|mqtt_cxx|examples|eppp|wifi_remote|tls_cxx|mosq)\)\:)'
+        entry: '\A(?!(feat|fix|ci|bump|test|docs)\((mdns|modem|common|console|websocket|asio|mqtt_cxx|examples|eppp|wifi_remote|tls_cxx)\)\:)'
         language: pygrep
         args: [--multiline]
         stages: [commit-msg]

ci/check_copyright_config.yaml

@@ -47,13 +47,6 @@ asio_component:
     - Apache-2.0
     - BSL-1.0
 
-mosquitto_component:
-  include:
-    - 'components/mosquitto/port/**'
-  allowed_licenses:
-    - EPL-2.0
-    - Apache-2.0
-
 slim_modem_examples:
   include:
     - 'examples/esp_netif/slip_custom_netif/**'

components/esp_modem/Kconfig

@@ -67,13 +67,7 @@ menu "esp-modem"
         int "Size in bytes for response from AT commands returning textual values (C-API)"
         default 128
         help
-            Some AT commands returns textual values which C-API copy as c-string to user allocated space,
+            Some AT commands returns textrual values which C-API copy as c-string to user allocated space,
             it also truncates the output data to this size. Increase this if some AT answers are truncated.
 
-    config ESP_MODEM_URC_HANDLER
-        bool "Enable support for adding URC handler"
-        default n
-        help
-            If enabled, APIs to add URC handler are available
-
 endmenu

components/esp_modem/include/cxx_include/esp_modem_dce.hpp

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2021-2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2021-2022 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -89,13 +89,6 @@ class DCE_T {
         return dte->recover();
     }
 
-#ifdef CONFIG_ESP_MODEM_URC_HANDLER
-    void set_urc(got_line_cb on_read_cb)
-    {
-        dte->set_urc_cb(on_read_cb);
-    }
-#endif
-
 protected:
     std::shared_ptr<DTE> dte;
     std::shared_ptr<SpecificModule> device;

components/esp_modem/include/cxx_include/esp_modem_dte.hpp

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2021-2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2021-2023 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -94,17 +94,6 @@ class DTE : public CommandableIf {
      */
     void set_error_cb(std::function<void(terminal_error err)> f);
 
-#ifdef CONFIG_ESP_MODEM_URC_HANDLER
-    /**
-     * @brief Allow setting a line callback for all incoming data
-     * @param line_cb
-     */
-    void set_urc_cb(got_line_cb line_cb)
-    {
-        command_cb.urc_handler = std::move(line_cb);
-    }
-#endif
-
     /**
      * @brief Sets the DTE to desired mode (Command/Data/Cmux)
      * @param m Desired operation mode
@@ -202,9 +191,6 @@ class DTE : public CommandableIf {
      * @brief This abstracts command callback processing and implements its locking, signaling of completion and timeouts.
      */
     struct command_cb {
-#ifdef CONFIG_ESP_MODEM_URC_HANDLER
-        got_line_cb urc_handler {};                             /*!< URC callback if enabled */
-#endif
         static const size_t GOT_LINE = SignalGroup::bit0;       /*!< Bit indicating response available */
         got_line_cb got_line;                                   /*!< Supplied command callback */
         Lock line_lock{};                                       /*!< Command callback locking mechanism */

components/esp_modem/src/esp_modem_dte.cpp

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2021-2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2021-2023 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -50,11 +50,9 @@ void DTE::set_command_callbacks()
 {
     primary_term->set_read_cb([this](uint8_t *data, size_t len) {
         Scoped<Lock> l(command_cb.line_lock);
-#ifndef CONFIG_ESP_MODEM_URC_HANDLER
-        if (command_cb.got_line == nullptr || command_cb.result != command_result::TIMEOUT) {
-            return false;   // this line has been processed already (got OK or FAIL previously)
+        if (command_cb.got_line == nullptr) {
+            return false;
         }
-#endif
         if (data) {
             // For terminals which post data directly with the callback (CMUX)
             // we cannot defragment unless we allocate, but
@@ -349,14 +347,9 @@ void DTE::on_read(got_line_cb on_read_cb)
 
 bool DTE::command_cb::process_line(uint8_t *data, size_t consumed, size_t len)
 {
-#ifdef CONFIG_ESP_MODEM_URC_HANDLER
-    if (urc_handler) {
-        urc_handler(data, consumed + len);
-    }
-    if (result != command_result::TIMEOUT || got_line == nullptr) {
+    if (result != command_result::TIMEOUT) {
         return false;   // this line has been processed already (got OK or FAIL previously)
     }
-#endif
     if (memchr(data + consumed, separator, len)) {
         result = got_line(data, consumed + len);
         if (result == command_result::OK || result == command_result::FAIL) {

components/esp_modem/test/target/main/CMakeLists.txt

@@ -1,5 +1,6 @@
 idf_component_register(SRCS "pppd_test.cpp"
                             "NetworkDCE.cpp"
+        INCLUDE_DIRS "$ENV{IDF_PATH}/tools/catch"
         REQUIRES esp_modem)
 
 set_target_properties(${COMPONENT_LIB} PROPERTIES

components/esp_modem/test/target/main/pppd_test.cpp

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2022-2023 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Unlicense OR CC0-1.0
  */
@@ -17,6 +17,9 @@
 #include "freertos/FreeRTOS.h"
 #include "freertos/event_groups.h"
 
+#define CATCH_CONFIG_MAIN
+#include "catch.hpp"
+
 static const char *TAG = "pppd_test";
 static EventGroupHandle_t event_group = NULL;
 
@@ -73,9 +76,6 @@ esp_err_t modem_init_network(esp_netif_t *netif);
 void modem_start_network();
 void modem_stop_network();
 
-bool test_connect();
-bool test_disconnect();
-
 extern "C" void app_main(void)
 {
 
@@ -99,27 +99,41 @@ extern "C" void app_main(void)
 #endif
 
     modem_start_network();
-
-    bool t1 = test_connect();
-    bool t2 = test_disconnect();
-
-    if (t1 && t2) {
-        ESP_LOGI(TAG, "All tests passed");
-    } else {
+    Catch::Session session;
+    int numFailed = session.run();
+    if (numFailed > 0) {
         ESP_LOGE(TAG, "Test FAILED!");
+    } else {
+        ESP_LOGI(TAG, "Test passed!");
     }
 
 }
 
-bool test_connect() //("Connect test", "[esp_modem]")
+TEST_CASE("Connect test", "[esp_modem]")
 {
     EventBits_t b = xEventGroupWaitBits(event_group, 1, pdTRUE, pdFALSE, pdMS_TO_TICKS(15000));
-    return b == 1;
+    CHECK(b == 1);
 }
 
-bool test_disconnect() //("Disconnection test", "[esp_modem]")
+TEST_CASE("Disconnection test", "[esp_modem]")
 {
     modem_stop_network();
     EventBits_t b = xEventGroupWaitBits(event_group, 2, pdTRUE, pdFALSE, pdMS_TO_TICKS(15000));
-    return b == 2;
+    CHECK(b == 2);
+}
+
+
+extern "C" {
+
+    static void handle(int nr)
+    {
+        ESP_LOGE(TAG, "Signal handler %d", nr);
+    }
+
+    _sig_func_ptr signal (int nr, _sig_func_ptr)
+    {
+        return handle;
+    }
+
+
 }