Force enable and reset to be low when deiniting chip #285
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, when deiniting the WiFi chip via
WiFi.end()
, the enable pin is left floating.As per the data sheet section 8.4 (Power up/down sequence), both the
enable
andreset
pins must be asserted high or low, and never left floating:This PR addresses this by ensuring both pins are output (enable was INPUT_PULLUP), and force asserting both pins low on deinit. This appears safe, as the enable and reset pins do not appear to be used by any other piece of code in the WiFi101 library.