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

Change default eth key #502

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Change default eth key #502

merged 1 commit into from
Oct 21, 2024

Conversation

DaneSlattery
Copy link
Contributor

According to this issue, espressif/esp-idf#10359

The default name for an ethernet netif is ETH_DEF, but it is hardcoded as ETH_CL_DEF. The mDNS service attempts to look up the netif based on the name during initialisation, but it won't find it unless the user wraps their own netif with the new name.

@ivmarkov
Copy link
Collaborator

Hmmmm, what about WIFI? Is there any "default" key which is expected there? We currently have "WIFI_STA_DEF" and "WIFI_AP_DEF". Are these correct?

What about the so called "router" Eth, which is nothing else but a fixed-conf Eth?

@DaneSlattery
Copy link
Contributor Author

DaneSlattery commented Oct 17, 2024

@ivmarkov
Copy link
Collaborator

What about the so called "router" Eth, which is nothing else but a fixed-conf Eth?

What about this? I think you should change it there as well.

@DaneSlattery
Copy link
Contributor Author

It doesn't seem to interact with the DNS system at all, so I'm not sure what it should be. The ETH_RT_DEF name seems fine to me

@ivmarkov
Copy link
Collaborator

OK fine, I'll merge as is, and will change the RT key if necessary later.

@ivmarkov ivmarkov merged commit 7cf3a9a into esp-rs:master Oct 21, 2024
15 checks passed
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.

2 participants