Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix wifi persistent #1404

Closed
wants to merge 2 commits into from
Closed

Fix wifi persistent #1404

wants to merge 2 commits into from

Conversation

everslick
Copy link
Contributor

WiFi config is currently written to NVS regardless of if WiFi.persistent(false); was called before or not. This PR moves some of the static functions in WiFiGeneric.cpp into the WiFiGenericClass to give them access to the _persistent member. Additionally WiFi.getMode() and WiFi.mode() have been made regular methods instead of static. Now esp_wifi_set_storage(WIFI_STORAGE_RAM); is called when persistent storage is switched off.

everslick added 2 commits May 12, 2018 12:50
  WiFi config is currently written to NVS regardless of if
  WiFi.persistent(false); was called before or not. This PR
  moves some of the static functions in WiFiGeneric.cpp into
  the WiFiGenericClass to give them access to the _persistent
  member. Additionally WiFi.getMode() and WiFi.mode() have
  been made regular methods instead of static. Now
  esp_wifi_set_storage(WIFI_STORAGE_RAM); is called when
  persistent storage is switched off.
@everslick
Copy link
Contributor Author

Rer: #548 and #1393

@everslick
Copy link
Contributor Author

this change breaks ETH, will come up with a better fix....

@everslick everslick closed this May 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant