-
Notifications
You must be signed in to change notification settings - Fork 4.6k
feat:新增blufi配网协议 #1321
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
base: main
Are you sure you want to change the base?
feat:新增blufi配网协议 #1321
Conversation
|
建议使用 network_provisioning,维护性会更好一些。 |
|
感谢您的建议,这是一个很棒的优化,我将进行重构 |
0399577 to
b9c28ca
Compare
|
请问现在这个blufi配网协议已经优化过了嘛?为啥还不合并到主干捏 |
我已经完成了优化,将blufi写到了一个class中,避免混乱。并且我注意到由network_provisioning提供的蓝牙配网协议与blufi似乎不同。因此我重新实现了一个class而非使用network_provisioning |
|
CMakeLists.txt 的改动会不会太大了,另外需要提供一个文档来说明用法。可以放在 docs 下。 |
很抱歉我的自动格式化程序将CMakeLists.txt进行了格式化,我将避免格式化程序引入的改动。 |
|
编译的时候会有如下错误 |
|
刚刚我已经编译成功了,需要在json文件中添加,但是如果开启蓝牙5.0是无法编译通过的,希望可以在文档中描述一下,稍后我测试一下blufi配网是否可以正常使用 |
|
测试可以获得密码但是连不上网,我暂时的解决方案是保存到nvs然后重启 |
|
最近 esp-wifi-connect 这个组件会更新一个3.0的版本, 支持灵活的开关wifi,以及和ap模式之间的切换,无需重启。 |
感谢您的测试报告。我将修改代码使用esp-wifi-connect组件,它会处理保存到nvs相关的逻辑。 |
e9af38f to
5dab6a0
Compare
这样的话那就太棒了,可以直接更加优雅的使用这个组件获取blufi配网的结果。我已经提交了一个更新的版本。 |
|
对CMakeLists.txt的修改是不是忘记提交了,目前blufi.cpp是不会被编译的。另外,我现在把blufi.cpp添加进编译后依然会编译失败。我看了一下是switch 语句中的局部变量初始化问题,我添加了一个大括号解决了编译问题,以下是我的变更记录: 另外,仅仅添加CONFIG_USE_ESP_BLUFI_WIFI_PROVISIONING配置是不够的,需要添加这些: 能不能改成在添加CONFIG_USE_ESP_BLUFI_WIFI_PROVISIONING后自动引用对下面几个的添加呢? 此外,上述操作只是配网成功并且写入nvs了,并不会自动重启,需要手动重启才可以联网成功, esp-wifi-connect 这个组件更新3.0的版本后应该可以解决,暂时可以在代码里面添加esp_restart()重启。 这个feat是非常有用的,希望可以尽快修改后合并如主线。 |
感谢您的反馈。很抱歉我犯了这样的错误。我将会尽快修改。对于自动引用的问题,我计划修改配置为如下的方式: 来告诉构建系统应该自动勾选这些内容。 |
|
感谢您的贡献,我还有一个问题,就是当我在json中添加USE_ESP_BLUFI_WIFI_PROVISIONING后,下面的那些select并不会被选中,这些自动引用是只能够手动idf.py menuconfig选中才能自动引用吗? |
我检查了esp-idf的文档,似乎只有idf.py menuconfig的时候才会自动引用。我正在尝试寻找更多的有关信息,如有进展我会尽快反馈给您。 |
非常感谢,我这边的配网现象是不能自动重启的,看日志会无限打印一条日志,具体日志信息稍后提供给您. |
|
日志如下,会一直打印,我看虾哥已经提交了pr,会和这个pr有冲突,等合并后再修复吧 |
|
我在release.py中新增了_apply_auto_selects用于处理这种情况。现在逻辑应该是正确的。以及我修复了自动重启的逻辑。 |
|
现在可以正常配网了,十分感谢 |
很高兴听到所有功能正常工作,希望对大家能有所帮助。 |
今天2.10版本发布了,WiFi的代码有修改,与这个commit有冲突了,需要解一下冲突 |
明白,我将会解决冲突并且切换到新的Wi-Fi连接逻辑 |
|
我已经解决了所有的冲突。切换到新的Wi-Fi链接逻辑后,现在配网成功后不再需要重启。 |
README.md 应该还没更新,写的是需要等待重启 |
|
关于CONFIG_BT_NIMBLE_ENABLED这个宏的代码需不需要删除呢?或者在文档说明一下?我打开这个宏好像编译有问题 |
感谢指正 |
CONFIG_BT_BLUEDROID_ENABLED和CONFIG_BT_NIMBLE_ENABLED理应二选一,它们都是对ble的实现。 |
Signed-off-by: WhereAreBugs <[email protected]>
|
重新修正了一处意外的头文件引入,现在CI将不再失败 |
#546
简单新增了blufi协议,支持通过官方app或其他使用esp-blufi协议的app进行配网操作。