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

rtl8720dn: add SetupAndConnectToAccessPoint() #436

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

sago35
Copy link
Member

@sago35 sago35 commented Jul 23, 2022

In many cases, the task of setting up and connecting the rtl8720dn is tedious.
SetupAndConnectToAccessPoint() and simplified the code.
This makes examples/rtl8720dn/webclient etc. as close to the same source code as possible as the PC.

@sago35
Copy link
Member Author

sago35 commented Jul 23, 2022

Perhaps it would be useful to create a similar structure for wifinina and others.
In that case, instead of examples/rtl8720dn, it might be better to use rtl8720dn/setup.

@scottfeldman
Copy link
Contributor

I believe this PR is subsumed by the greater netdev work in drivers:dev. Netdev gives drivers a common entry point to setup and connect to Wifi. Apps use that API, regardless of which network driver is being used. Consequently, the examples/net examples apps aren't specific to one network driver or the other.

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.

None yet

2 participants