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

open_esplibs libnet80211: source code for sdk_wifi_softap_start paths. #286

Closed

Conversation

ourairquality
Copy link
Contributor

The function sdk_wifi_softap_start is one of two paths that allocates a struct netif and also accesses the netif->hwaddr so is needed for lwip development. This function depends on static data and functions so a good bit so other code was needed.

This was a difficult set, challenging to review, but it gets us the allocation of the netif structure needed for LwIP development, plus the remaining uses of the ets_timer, so feedback welcomed.

The function sdk_wifi_softap_start is one of two paths that allocates a struct netif and also accesses the netif->hwaddr so is needed for lwip development. This function depends on static data and functions so a good bit so other code was needed.
@vlad-ivanov-name
Copy link
Contributor

By the way, some of decompiled net80211 functions can be found at https://github.com/ernacktob/esp8266_wifi_raw

Copy link
Contributor

@sheinz sheinz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ourairquality How much testing have you done of this code?
I think we should merge it even if we can't be sure it doesn't have a regression. We can always disable parts of open_esplibs to check if some issue appears.
Thanks for the great work!

@ourairquality
Copy link
Contributor Author

@sheinz Testing is limited, it appears to work in all the code I am working with, but some of these functions were a struggle to translate by hand so I would like to re-check them. Very hard to review these. I am exploring some automation to help translate the code, it still needs a lot of manual work, but might give a different view for comparison. Still need to get the wl_cnx.o translated for lwip v2 to be practical, which could take time, so there is no rush on this PR.

@UncleRus UncleRus added the core label Nov 29, 2016
@ourairquality
Copy link
Contributor Author

Have been over this many times so hope it is all ok, and it was necessary to bundle it in with the lwip v2 support #389 so closing this PR.

@ourairquality ourairquality deleted the ieee80211-hostap branch June 16, 2017 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants