-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Remove CONFIG_LWIP_LOCAL_HOSTNAME from S3 boards #9656
base: main
Are you sure you want to change the base?
Conversation
For completeness: I checked an S2 board with an empty |
I submitted this not really knowing as much as I should have about core Circuitpython WiFi. It seems to me if the variable is used somewhere that is visible via Circuitpython we should probably leave it alone. Feel free to re-open if there's a reason to pursue this. I made these changes using a script so it would be relatively painless, if for some reason we wanted to expand to other chips or make a different change to the files/default values. |
I didn't mean to shut this down, only highlight what the new hostname would be. We haven't been consistent with having |
I do want to get rid of these settings so that our IDF builds are more uniform across boards. I think we should set it to the board id by default though. That's basically what folks use this setting for anyway. |
Perhaps we should unify this with the MDNS name too. It could be |
Any pointers as to where the initialization should take place. It looks like common_hal_wifi_init happens when wifi is imported which sounds too late to me. |
I would think |
As discussed in #9583 this removes the CONFIG_LWIP_LOCAL_HOSTNAME parameter from the sdkconfig files of all ESP32-S3 boards.
I actually just commented out the configuration setting but can easily update this to completely remove the lines if that is preferred.