STM32

有什么办法可以改善这个ISR - STM32F401RE吗？

2019-07-14 17:23发布

站内问答 / STM32/STM8
13102 3 1109
我目前正在从头编写飞行控制器，目前我正在使用该模块来解码传入的PPM信号。请记住，这是一个ISR，我的功能仅限于从接收器读取通道值。它工作完全没问题，但我想要一些反馈和改进（如果可能的话）。ISR如下：
  1. void measureChannel(){
  2.    int elapsedtime = t.read_us();
  3.           //if any pulse shorter than 1000us, discard as garbage
  4.    if(elapsedTime < 900){
  5.         t.reset();
  6.         return;
  7.     }

  9.   else{
  10.         //if start/stop sequence, then set currentChannel = 1 and begin to read
  11.         if(elapsedTime > 2100) {
  12.                 currentChannel = 1;
  13.                 t.reset();
  14.                 return;
  15.             }
  16.         //read channel value
  17.         else if(elapsedTime >= 1000 && elapsedTime <= 2000){
  18.                 if(currentChannel == 1){
  19.                         channelVal[0] = elapsedTime;
  20.                         currentChannel = 2;
  21.                         t.reset();
  22.                         return;
  23.                     }

  25.                 else if(currentChannel == 2){
  26.                         channelVal[1] = elapsedTime;
  27.                         currentChannel = 3;
  28.                         t.reset();
  29.                         return;
  30.                     }

  32.                 else if(currentChannel == 3){
  33.                         channelVal[2] = elapsedTime;
  34.                         currentChannel = 4;
  35.                         t.reset();
  36.                         return;
  37.                     }

  39.                 else if(currentChannel == 4){
  40.                         channelVal[3] = elapsedTime;
  41.                         currentChannel = 5;
  42.                         t.reset();
  43.                         return;
  44.                     }

  46.                 else if(currentChannel == 5){
  47.                         channelVal[4] = elapsedTime;
  48.                         currentChannel = 6;
  49.                         t.reset();
  50.                         return;
  51.                     }

  53.                 else if(currentChannel == 6){
  54.                         channelVal[5] = elapsedTime;
  55.                         currentChannel = 0;
  56.                         t.reset();
  57.                         return;
  58.                     }
  59.             }
  60.       }
  61. }
复制代码
友情提示: 此问题已得到解决，问题已经关闭,关闭后问题禁止继续编辑，回答。
该问题目前已经被作者或者管理员关闭, 无法添加新回复
3条回答
msvdsufsdf
1楼-- · 2019-07-14 23:53
假设这些全局变量是在ISR之外访问的，那么它们应该被声明为volatile
sfafa
2楼-- · 2019-07-15 00:16
 精彩回答 2  元偷偷看……
随行者011011
3楼-- · 2019-07-15 04:45
不需要返回语句

为了清晰起见，最好使用一些数组来最小化代码

读取通道值部分可写为：

  //read channel value
  else if(elapsedTime >= 1000 && elapsedTime <= 2000)
  {
       if((currentChannel >= 1) && (currentChannel <= 6))
       {
             channelVal[currentChannel - 1] = elapsedTime;
             currentChannel = (currentChannel + 1) % 7;
       }
  }

一周热门 更多>