search

RT-Thread-packages / at_device

AT component porting or samples for different devices
Apache License 2.0
214 stars 181 forks source link

fix: at_device_air720.c 编译错误 #134

Closed wendal closed 3 years ago

wendal commented 3 years ago
  1. AT_DEVICE_CTRL_REBOOT 已经被移除? 注释掉吧
  2. air720_reboot声明有误, 一个声明为static,一个不是, 修正之
xiangxistu commented 3 years ago
  1. AT_DEVICE_CTRL_REBOOT 是 AT_DEVICE 提供的一个统一的接口；理论上所有的设备均应实现这个接口；所以这里应当继续保留。
  2. static 修饰该函数在其他文件内不可调用，不建议去除。若声明不一致，建议调整为一致，均由 static 修饰。
wendal commented 3 years ago

@xiangxistu rtt主干代码里面, AT_DEVICE没有AT_DEVICE_CTRL_REBOOT, 其他模块也没实现AT_DEVICE_CTRL_REBOOT

xiangxistu commented 3 years ago

@wendal 感觉注释起来还是不优雅，可以在 at_device.h 里添加 AT_DEVICE_CTRL_REBOOT 宏定义；

wendal commented 3 years ago

那是给rt-thread发个PR? 不晓得为啥删掉了...

xiangxistu commented 3 years ago

那是给rt-thread发个PR? 不晓得为啥删掉了...

在 at_device 里添加就可以，需要向 RT-Thread 的主分支里提交代码吗？

xiangxistu commented 3 years ago

there have a conflict that ‘reset’ and ‘reboot’.