diff --git a/.github/workflows/modem__build-host-tests.yml b/.github/workflows/modem__build-host-tests.yml index d28a30d11..fcc1d4cc9 100644 --- a/.github/workflows/modem__build-host-tests.yml +++ b/.github/workflows/modem__build-host-tests.yml @@ -15,11 +15,6 @@ jobs: matrix: idf_ver: ["latest", "release-v5.0", "release-v5.1", "release-v5.2", "release-v5.3", "release-v5.4"] example: ["pppos_client", "modem_console", "modem_tcp_client", "ap_to_pppos", "simple_cmux_client"] - include: - - idf_ver: "release-v5.0" - example: "simple_cmux_client" - warning: "Warning: The smallest app partition is nearly full" - runs-on: ubuntu-22.04 container: espressif/idf:${{ matrix.idf_ver }} steps: @@ -30,9 +25,9 @@ jobs: - if: ${{ matrix.skip_config }} run: rm -f $GITHUB_WORKSPACE/protocols/components/esp_modem/examples/${{ matrix.example }}/sdkconfig.ci.${{ matrix.skip_config }}* - name: Build ${{ matrix.example }} with IDF-${{ matrix.idf_ver }} - env: - EXPECTED_WARNING: ${{ matrix.warning }} shell: bash + env: + EXPECTED_WARNING: "Warning: The smallest app partition is nearly full\nwarning: unknown kconfig symbol 'ESP32_PANIC_PRINT_HALT'" run: | . ${IDF_PATH}/export.sh python -m pip install idf-build-apps diff --git a/ci/build_apps.py b/ci/build_apps.py index 806a53605..1dc680d39 100644 --- a/ci/build_apps.py +++ b/ci/build_apps.py @@ -56,7 +56,7 @@ if __name__ == '__main__': build_dir='build_@t_@w', config_rules_str=args.rules, build_log_filename='build_log.txt', - size_json_filename='size.json' if not args.linux else None, + size_json_filename=None, check_warnings=True, manifest_files=args.manifests, default_build_targets=SUPPORTED_TARGETS, diff --git a/components/esp_modem/CMakeLists.txt b/components/esp_modem/CMakeLists.txt index c0e37154f..5d9edf0d4 100644 --- a/components/esp_modem/CMakeLists.txt +++ b/components/esp_modem/CMakeLists.txt @@ -11,7 +11,12 @@ else() src/esp_modem_uart.cpp src/esp_modem_term_uart.cpp src/esp_modem_netif.cpp) - set(dependencies driver esp_event esp_netif) + set(dependencies esp_event esp_netif) + if("${IDF_VERSION_MAJOR}.${IDF_VERSION_MINOR}" VERSION_GREATER "5.3") + list(APPEND dependencies esp_driver_uart) + else() + list(APPEND dependencies driver) + endif() endif() diff --git a/components/esp_modem/examples/ap_to_pppos/main/ap_to_pppos.c b/components/esp_modem/examples/ap_to_pppos/main/ap_to_pppos.c index a6ac822d4..d8ba31567 100644 --- a/components/esp_modem/examples/ap_to_pppos/main/ap_to_pppos.c +++ b/components/esp_modem/examples/ap_to_pppos/main/ap_to_pppos.c @@ -1,5 +1,5 @@ /* - * SPDX-FileCopyrightText: 2022-2023 Espressif Systems (Shanghai) CO LTD + * SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD * * SPDX-License-Identifier: Unlicense OR CC0-1.0 */ @@ -100,10 +100,10 @@ void wifi_init_softap(void) ESP_ERROR_CHECK(esp_wifi_init(&cfg)); ESP_ERROR_CHECK(esp_event_handler_instance_register(WIFI_EVENT, - ESP_EVENT_ANY_ID, - &wifi_event_handler, - NULL, - NULL)); + ESP_EVENT_ANY_ID, + &wifi_event_handler, + NULL, + NULL)); wifi_config_t wifi_config = { .ap = { @@ -120,7 +120,7 @@ void wifi_init_softap(void) } ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_AP)); - ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_AP, &wifi_config)); + ESP_ERROR_CHECK(esp_wifi_set_config(WIFI_IF_AP, &wifi_config)); ESP_ERROR_CHECK(esp_wifi_start()); ESP_LOGI(TAG, "wifi_init_softap finished. SSID:%s password:%s channel:%d", diff --git a/components/esp_modem/examples/modem_console/main/CMakeLists.txt b/components/esp_modem/examples/modem_console/main/CMakeLists.txt index 67de0841f..67ed4f6c5 100644 --- a/components/esp_modem/examples/modem_console/main/CMakeLists.txt +++ b/components/esp_modem/examples/modem_console/main/CMakeLists.txt @@ -9,5 +9,4 @@ idf_component_register(SRCS "modem_console_main.cpp" "${command_dir}/my_module_dce.cpp" "httpget_handle.c" "ping_handle.c" - REQUIRES console esp_http_client nvs_flash INCLUDE_DIRS "." "${command_dir}") diff --git a/components/esp_modem/port/linux/esp_netif_linux/lwipopts.h b/components/esp_modem/port/linux/esp_netif_linux/lwipopts.h index edeed034e..b77fce432 100644 --- a/components/esp_modem/port/linux/esp_netif_linux/lwipopts.h +++ b/components/esp_modem/port/linux/esp_netif_linux/lwipopts.h @@ -89,7 +89,7 @@ #define MPPE_SUPPORT 0 -#define PPP_MAXIDLEFLAG 0 +#define PPP_MAXIDLEFLAG 100 #define PRINTPKT_SUPPORT 1 #define PPP_PROTOCOLNAME 1 diff --git a/components/esp_modem/port/linux/esp_netif_linux/tun_io.c b/components/esp_modem/port/linux/esp_netif_linux/tun_io.c index 16f57e713..4ad4a5b1d 100644 --- a/components/esp_modem/port/linux/esp_netif_linux/tun_io.c +++ b/components/esp_modem/port/linux/esp_netif_linux/tun_io.c @@ -104,7 +104,7 @@ static void ppp_link_status_cb(ppp_pcb *pcb, int err_code, void *ctx) } } -static u32_t ppp_output_cb(struct ppp_pcb_s *pcb, const void *data, u32_t len, void *ctx) +static u32_t ppp_output_cb(struct ppp_pcb_s *pcb, u8_t *data, u32_t len, void *ctx) { esp_netif_t *netif = (esp_netif_t *)ctx; if (netif->transmit) {