add NB-IOT me3616 #91
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Lawlieta
approved these changes
Dec 31, 2019
|
|
||
| #ifdef AT_DEVICE_USING_ME3616 | ||
|
|
||
| #define ME3616_DEEP_SLEEP_EN 0//module support deep sleep mode |
Contributor
There was a problem hiding this comment.
DEEP_SLEEP 模式是否可以做成 menuconfig 中功能配置，或者 at_device_control 函数中控制，而非在代码中写死
Contributor
Author
There was a problem hiding this comment.
目前模块不支持DEEP_SLEEP 模式，现在只是将功能代码保留下，方便以后功能升级。可以修改这个宏定义为：
#ifndef ME3616_DEEP_SLEEP_EN
#define ME3616_DEEP_SLEEP_EN 0//module support deep sleep mode
#endif
这样可以在menuconfig 中增加功能配置。可好？
| goto __exit; | ||
| } | ||
|
|
||
| at_resp_set_info(resp, CONN_RESP_SIZE, 0, (45*RT_TICK_PER_SECOND)); |
Contributor
There was a problem hiding this comment.
45 秒的超时在实际使用中是不是有点长，可根据实际连接情况优化一下等待时间
Contributor
Author
There was a problem hiding this comment.
测试验证过的,最长返回结果时间为45秒.如果改短可能无法获取返回结果.
Contributor
|
感谢你的支持和贡献，确认一下，相关网络软件包和功能都有测试通过吗？ |
Contributor
Author
|
此软件包以经过2周测试优化过了。 |
Lawlieta
approved these changes
Dec 31, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.