fix(eppp_link): Per review comments

Author: david-cermak

components/eppp_link/README.md

@@ -35,10 +35,10 @@ Tested with WiFi-NAPT example, no IRAM optimizations
 
 ### UART @ 3Mbauds
 
-* TCP - 2Mbits
-* UDP - 2Mbits
+* TCP - 2Mbits/s
+* UDP - 2Mbits/s
 
 ### SPI @ 20MHz
 
-* TCP - 6Mbits
-* UDP - 10Mbits
+* TCP - 6Mbits/s
+* UDP - 10Mbits/s

components/eppp_link/eppp_link.c

@@ -43,7 +43,7 @@ struct eppp_handle {
     uart_port_t uart_port;
 #endif
     esp_netif_t *netif;
-    enum eppp_type role;
+    eppp_type_t role;
     bool stop;
     bool exited;
     bool netif_stop;
@@ -109,7 +109,7 @@ static void netif_deinit(esp_netif_t *netif)
     }
 }
 
-static esp_netif_t *netif_init(enum eppp_type role)
+static esp_netif_t *netif_init(eppp_type_t role)
 {
     if (s_eppp_netif_count > 9) {
         ESP_LOGE(TAG, "Cannot create more than 10 instances");
@@ -575,15 +575,6 @@ esp_err_t eppp_perform(esp_netif_t *netif)
     return ESP_OK;
 }
 
-static void ppp_task(void *args)
-{
-    esp_netif_t *netif = args;
-    while (eppp_perform(netif) != ESP_ERR_TIMEOUT) {}
-    struct eppp_handle *h = esp_netif_get_io_driver(netif);
-    h->exited = true;
-    vTaskDelete(NULL);
-}
-
 #elif CONFIG_EPPP_LINK_DEVICE_UART
 #define BUF_SIZE (1024)
 
@@ -636,17 +627,17 @@ esp_err_t eppp_perform(esp_netif_t *netif)
     return ESP_OK;
 }
 
+#endif // CONFIG_EPPP_LINK_DEVICE_SPI / UART
+
 static void ppp_task(void *args)
 {
     esp_netif_t *netif = args;
-    while (eppp_perform(netif) == ESP_OK) {}
+    while (eppp_perform(netif) != ESP_ERR_TIMEOUT) {}
     struct eppp_handle *h = esp_netif_get_io_driver(netif);
     h->exited = true;
     vTaskDelete(NULL);
 }
 
-#endif // CONFIG_EPPP_LINK_DEVICE_SPI / UART
-
 static bool have_some_eppp_netif(esp_netif_t *netif, void *ctx)
 {
     return get_netif_num(netif) > 0;
@@ -679,7 +670,7 @@ void eppp_deinit(esp_netif_t *netif)
     netif_deinit(netif);
 }
 
-esp_netif_t *eppp_init(enum eppp_type role, eppp_config_t *config)
+esp_netif_t *eppp_init(eppp_type_t role, eppp_config_t *config)
 {
     esp_netif_t *netif = netif_init(role);
     if (!netif) {
@@ -710,7 +701,7 @@ esp_netif_t *eppp_init(enum eppp_type role, eppp_config_t *config)
     return netif;
 }
 
-esp_netif_t *eppp_open(enum eppp_type role, eppp_config_t *config, TickType_t connect_timeout)
+esp_netif_t *eppp_open(eppp_type_t role, eppp_config_t *config, TickType_t connect_timeout)
 {
 #if CONFIG_EPPP_LINK_DEVICE_UART
     if (config->transport != EPPP_TRANSPORT_UART) {

components/eppp_link/eppp_link_types.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: Unlicense OR CC0-1.0
  */
@@ -46,18 +46,14 @@ static struct {
 static int ppp_cmd_iperf(int argc, char **argv)
 {
     int nerrors = arg_parse(argc, argv, (void **)&iperf_args);
-    iperf_cfg_t cfg;
+    // ethernet iperf only support IPV4 address
+    iperf_cfg_t cfg = {.type = IPERF_IP_TYPE_IPV4};
 
     if (nerrors != 0) {
         arg_print_errors(stderr, iperf_args.end, argv[0]);
         return 0;
     }
 
-    memset(&cfg, 0, sizeof(cfg));
-
-    // ethernet iperf only support IPV4 address
-    cfg.type = IPERF_IP_TYPE_IPV4;
-
     /* iperf -a */
     if (iperf_args.abort->count != 0) {
         iperf_stop();

components/eppp_link/examples/host/main/register_iperf.c

@@ -92,11 +92,11 @@ esp_netif_t *eppp_listen(eppp_config_t *config);
 
 void eppp_close(esp_netif_t *netif);
 
-esp_netif_t *eppp_init(enum eppp_type role, eppp_config_t *config);
+esp_netif_t *eppp_init(eppp_type_t role, eppp_config_t *config);
 
 void eppp_deinit(esp_netif_t *netif);
 
-esp_netif_t *eppp_open(enum eppp_type role, eppp_config_t *config, TickType_t connect_timeout);
+esp_netif_t *eppp_open(eppp_type_t role, eppp_config_t *config, TickType_t connect_timeout);
 
 esp_err_t eppp_netif_stop(esp_netif_t *netif, TickType_t stop_timeout);