[network] fixes Ethernet default config/validation, minor optimizations #2846
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.
Problem
PIN_INVALID
should not be used as a default pin setting for wiznet config. It's very easy to put ethernet driver into a not so performant mode due to lack of interrupt pin configuration.Solution
PIN_INVALID
should not be used to reset the settings, as currently callingwill result in
cs_pin
being properly overridden to defaults, but not RESET or INT. This is not in public docs, but a couple of test apps are using this.Reset to defaults should be:
until we introduce a nicer API.
When loading the config from filesystem, if we see
cs_pin = PIN_INVALID
- invalidate the whole config to the defaults. This is NOT a valid configuration and it's better to just revert to defaults.Setting
reset_pin
orint_pin
toPIN_INVALID
is still allowed.This PR also includes some minor optimizations to ethernet driver itself.
Steps to Test
N/A
Example App
N/A
References
Links to the Community, Docs, Other Issues, etc..
Completeness