mirror of
https://github.com/espressif/esp-idf.git
synced 2024-10-05 20:47:46 -04:00
Merge branch 'bugfix/fix_wps_example_issue' into 'master'
example: We should not check the return value of esp_wifi_connect() in any case See merge request espressif/esp-idf!11897
This commit is contained in:
commit
755b5d617c
@ -133,7 +133,7 @@ static void start(const char *ssid, const char *passwd)
|
|||||||
ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
|
ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
|
||||||
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config));
|
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config));
|
||||||
ESP_ERROR_CHECK(esp_wifi_start());
|
ESP_ERROR_CHECK(esp_wifi_start());
|
||||||
ESP_ERROR_CHECK(esp_wifi_connect());
|
esp_wifi_connect();
|
||||||
s_connection_name = ssid;
|
s_connection_name = ssid;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -167,7 +167,7 @@ static bool wifi_cmd_sta_join(const char *ssid, const char *pass)
|
|||||||
esp_wifi_disconnect();
|
esp_wifi_disconnect();
|
||||||
//ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) ); //by snake
|
//ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) ); //by snake
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_connect() );
|
esp_wifi_connect();
|
||||||
|
|
||||||
xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT, 0, 1, 5000 / portTICK_RATE_MS);
|
xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT, 0, 1, 5000 / portTICK_RATE_MS);
|
||||||
|
|
||||||
|
@ -271,7 +271,7 @@ static esp_netif_t *wifi_start(void)
|
|||||||
ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
|
ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
|
||||||
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config));
|
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config));
|
||||||
ESP_ERROR_CHECK(esp_wifi_start());
|
ESP_ERROR_CHECK(esp_wifi_start());
|
||||||
ESP_ERROR_CHECK(esp_wifi_connect());
|
esp_wifi_connect();
|
||||||
return netif;
|
return netif;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -72,7 +72,7 @@ static bool wifi_join(const char *ssid, const char *pass, int timeout_ms)
|
|||||||
|
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) );
|
ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_connect() );
|
esp_wifi_connect();
|
||||||
|
|
||||||
int bits = xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT,
|
int bits = xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT,
|
||||||
pdFALSE, pdTRUE, timeout_ms / portTICK_PERIOD_MS);
|
pdFALSE, pdTRUE, timeout_ms / portTICK_PERIOD_MS);
|
||||||
|
@ -159,7 +159,7 @@ static bool wifi_cmd_sta_join(const char* ssid, const char* pass)
|
|||||||
reconnect = true;
|
reconnect = true;
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) );
|
ESP_ERROR_CHECK( esp_wifi_set_mode(WIFI_MODE_STA) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_connect() );
|
esp_wifi_connect();
|
||||||
|
|
||||||
xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT, 0, 1, 5000/portTICK_RATE_MS);
|
xEventGroupWaitBits(wifi_event_group, CONNECTED_BIT, 0, 1, 5000/portTICK_RATE_MS);
|
||||||
|
|
||||||
|
@ -70,7 +70,7 @@ static void event_handler(void* arg, esp_event_base_t event_base,
|
|||||||
|
|
||||||
ESP_ERROR_CHECK( esp_wifi_disconnect() );
|
ESP_ERROR_CHECK( esp_wifi_disconnect() );
|
||||||
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
ESP_ERROR_CHECK( esp_wifi_set_config(ESP_IF_WIFI_STA, &wifi_config) );
|
||||||
ESP_ERROR_CHECK( esp_wifi_connect() );
|
esp_wifi_connect();
|
||||||
} else if (event_base == SC_EVENT && event_id == SC_EVENT_SEND_ACK_DONE) {
|
} else if (event_base == SC_EVENT && event_id == SC_EVENT_SEND_ACK_DONE) {
|
||||||
xEventGroupSetBits(s_wifi_event_group, ESPTOUCH_DONE_BIT);
|
xEventGroupSetBits(s_wifi_event_group, ESPTOUCH_DONE_BIT);
|
||||||
}
|
}
|
||||||
|
@ -64,7 +64,7 @@ static void wifi_event_handler(void* arg, esp_event_base_t event_base,
|
|||||||
case WIFI_EVENT_STA_DISCONNECTED:
|
case WIFI_EVENT_STA_DISCONNECTED:
|
||||||
ESP_LOGI(TAG, "WIFI_EVENT_STA_DISCONNECTED");
|
ESP_LOGI(TAG, "WIFI_EVENT_STA_DISCONNECTED");
|
||||||
if (s_retry_num < MAX_RETRY_ATTEMPTS) {
|
if (s_retry_num < MAX_RETRY_ATTEMPTS) {
|
||||||
ESP_ERROR_CHECK(esp_wifi_connect());
|
esp_wifi_connect();
|
||||||
s_retry_num++;
|
s_retry_num++;
|
||||||
} else if (ap_idx < s_ap_creds_num) {
|
} else if (ap_idx < s_ap_creds_num) {
|
||||||
/* Try the next AP credential if first one fails */
|
/* Try the next AP credential if first one fails */
|
||||||
@ -73,7 +73,7 @@ static void wifi_event_handler(void* arg, esp_event_base_t event_base,
|
|||||||
ESP_LOGI(TAG, "Connecting to SSID: %s, Passphrase: %s",
|
ESP_LOGI(TAG, "Connecting to SSID: %s, Passphrase: %s",
|
||||||
wps_ap_creds[ap_idx].sta.ssid, wps_ap_creds[ap_idx].sta.password);
|
wps_ap_creds[ap_idx].sta.ssid, wps_ap_creds[ap_idx].sta.password);
|
||||||
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wps_ap_creds[ap_idx++]) );
|
ESP_ERROR_CHECK(esp_wifi_set_config(ESP_IF_WIFI_STA, &wps_ap_creds[ap_idx++]) );
|
||||||
ESP_ERROR_CHECK(esp_wifi_connect());
|
esp_wifi_connect();
|
||||||
}
|
}
|
||||||
s_retry_num = 0;
|
s_retry_num = 0;
|
||||||
} else {
|
} else {
|
||||||
@ -107,7 +107,7 @@ static void wifi_event_handler(void* arg, esp_event_base_t event_base,
|
|||||||
* with legacy apps. So directly attempt connection here.
|
* with legacy apps. So directly attempt connection here.
|
||||||
*/
|
*/
|
||||||
ESP_ERROR_CHECK(esp_wifi_wps_disable());
|
ESP_ERROR_CHECK(esp_wifi_wps_disable());
|
||||||
ESP_ERROR_CHECK(esp_wifi_connect());
|
esp_wifi_connect();
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case WIFI_EVENT_STA_WPS_ER_FAILED:
|
case WIFI_EVENT_STA_WPS_ER_FAILED:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user