-
Notifications
You must be signed in to change notification settings - Fork 3k
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
ESP8266: Country code API #9927
ESP8266: Country code API #9927
Conversation
|
||
bool ESP8266::set_country_code_policy(bool track_ap, const char *country_code, int channel_start, int channels) | ||
{ | ||
int _track_ap = track_ap ? 0 : 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[minor] isn't an underscore reserved for private class members?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if there would be a way to smuggle in a link to the lit of acceptable country codes somewhere in the comment? I checked that the Espressif's documentation does not seem to have it.
My thinking is that "ISO 3166-1 Alpha-2 coded country code" is a big enough hint. Of course the documentation doesn't really state that it would be followed. I tried the 3-character codes as the documentation mentioned those. Seems to be that allowing 3-characters is not the same as supporting 3-character codes... 'AUT' isn't the same as 'AT' as it is supposed to. The firmware just seems to chop off the last character. |
@VeijoPesonen, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@ARMmbed/mbed-os-maintainers Here's a question. 5.12.1 or 5.13-rc1? Is this closer to a target update or feature addition? |
starting CI |
Test run: SUCCESSSummary: 13 of 13 test jobs passed |
@VeijoPesonen : CI has passed , please answer the question by @cmonr above. |
@cmonr , @NirSonnenschein Lets treat it as a Functionality change, a new API for setting the country code was introduced. |
…e_policy ESP8266: Country code API
@VeijoPesonen @SeppoTakalo We did recently find that some files were being improperly excluded (#10140 (comment)), and have since fixed the astyle job. After this is merged (no need to hold merge on this glitch), either y'all or I can create a PR to get the repo back clean. |
Description
ESP8266Interface API updated with possibility to configure country code and channels to be used.
Pull request type
Reviewers
@SeppoTakalo
@kjbracey-arm
@michalpasztamobica
Release Notes
ESP8266Interface: new API to configure country code and channels to be used.