OpenEdv-开源电子网

 找回密码
 立即注册
正点原子全套STM32/Linux/FPGA开发资料,上千讲STM32视频教程免费下载...
查看: 3961|回复: 11

串口中断处理modbus协议时遇到错误帧该怎么处理??

[复制链接]

7

主题

19

帖子

0

精华

新手上路

积分
35
金钱
35
注册时间
2020-8-26
在线时间
13 小时
发表于 2023-8-8 17:25:52 | 显示全部楼层 |阅读模式
5金钱
各位大佬好!小弟目前在写串口中断处理modbus协议是程序,使用串口调试软件没问题,但是加了一个DTU透传时,透传为了发送心跳包会发送给一个心跳包代码,代码发送到单片机就会导致程序一直卡在中断里面,我找到原因但是不知道怎么写,感觉写在哪儿都不对劲。请各位大佬指点一下。 1691486563839.png 核心代码如图,当错误代码进入不到(crc==rccrc)里面时,我的count1就清不了0,脑子就卡在这个,请各位大佬给个方案

最佳答案

查看完整内容[请看2#楼]

count1=0放到下面大括号外,也就是函数末尾,反正每次都得清0的。 上面的if(count1==0)建议改为if(count1
正点原子逻辑分析仪DL16劲爆上市
回复

使用道具 举报

12

主题

3344

帖子

1

精华

论坛元老

Rank: 8Rank: 8

积分
8461
金钱
8461
注册时间
2020-5-11
在线时间
3904 小时
发表于 2023-8-8 17:25:53 | 显示全部楼层
本帖最后由 LcwSwust 于 2023-8-9 10:02 编辑

count1=0放到下面大括号外,也就是函数末尾,反正每次都得清0的。
上面的if(count1==0)建议改为if(count1<2),
否则当count1=1时Rx_buff1[count1-1]会越界;
考虑到还有地址码、功能码,一帧数据最少应为4字节,所以最好改为
if(count1<4){count1=0;return;}

专治疑难杂症
回复

使用道具 举报

530

主题

11万

帖子

34

精华

管理员

Rank: 12Rank: 12Rank: 12

积分
165371
金钱
165371
注册时间
2010-12-1
在线时间
2110 小时
发表于 2023-8-9 00:28:25 | 显示全部楼层
否则就丢弃数据/做其他处理
回复

使用道具 举报

1

主题

10

帖子

0

精华

新手上路

积分
37
金钱
37
注册时间
2020-2-2
在线时间
6 小时
发表于 2023-8-9 18:37:39 | 显示全部楼层
if(crc == rccrc){
//数据处理
}
else
{
//异常处理回复异常报文
//01 83 等等
//清除变量,进入下次接收
}
回复

使用道具 举报

7

主题

19

帖子

0

精华

新手上路

积分
35
金钱
35
注册时间
2020-8-26
在线时间
13 小时
 楼主| 发表于 2023-8-10 15:39:10 | 显示全部楼层
Genson 发表于 2023-8-9 18:37
if(crc == rccrc){
//数据处理
}

我把数据处理放在主循环里面了 else的话就会导致count一直清零
回复

使用道具 举报

7

主题

19

帖子

0

精华

新手上路

积分
35
金钱
35
注册时间
2020-8-26
在线时间
13 小时
 楼主| 发表于 2023-8-10 15:40:38 | 显示全部楼层
LcwSwust 发表于 2023-8-9 09:55
count1=0放到下面大括号外,也就是函数末尾,反正每次都得清0的。
上面的if(count1==0)建议改为if(count1

我把数据放在主函数里面处理了,这样导致我数据一直在判断,在这个情况下能对else进行处理么?
回复

使用道具 举报

12

主题

3344

帖子

1

精华

论坛元老

Rank: 8Rank: 8

积分
8461
金钱
8461
注册时间
2020-5-11
在线时间
3904 小时
发表于 2023-8-10 15:59:58 | 显示全部楼层
hellobye123 发表于 2023-8-10 15:40
我把数据放在主函数里面处理了,这样导致我数据一直在判断,在这个情况下能对else进行处理么?

我猜,Mosbus_Event()函数中的Event表示事件,即满足某种条件才会调用这个函数,比如收到一帧数据时。我不知道你所谓“把数据放在主函数里面处理”是怎么处理的,如果循环调用Mosbus_Event()函数那肯定是不行的。所以建议把程序传上来。
专治疑难杂症
回复

使用道具 举报

1

主题

10

帖子

0

精华

新手上路

积分
37
金钱
37
注册时间
2020-2-2
在线时间
6 小时
发表于 2023-8-12 16:08:57 | 显示全部楼层
hellobye123 发表于 2023-8-10 15:39
我把数据处理放在主循环里面了 else的话就会导致count一直清零

那你再好好想想,数据处理要单独受控的,不能来什么数据都处理
回复

使用道具 举报

7

主题

19

帖子

0

精华

新手上路

积分
35
金钱
35
注册时间
2020-8-26
在线时间
13 小时
 楼主| 发表于 2023-8-14 20:23:43 | 显示全部楼层
LcwSwust 发表于 2023-8-10 15:59
我猜,Mosbus_Event()函数中的Event表示事件,即满足某种条件才会调用这个函数,比如收到一帧数据时。我 ...

void main(void)
{
  //int i;

  SGP_halMcuWaitMs(5000);
  SGP_halMcuWaitMs(5000);
  InitUart(); //串口0初始化
  InitUart1();//串口1初始化
  Time_Init();//定时器0初始化
  IO_init();
  //UartSendByte(0X01);
  while(1)               
  {   
    Mosbus_Event();  
  }
}

//串口1
#pragma vector = URX1_VECTOR
__interrupt void UART1_ISR(void)
{
        URX1IF = 0;

        Rx_buff1[count1++] = U1DBUF;  

} 就这两段加上面的图片
回复

使用道具 举报

7

主题

19

帖子

0

精华

新手上路

积分
35
金钱
35
注册时间
2020-8-26
在线时间
13 小时
 楼主| 发表于 2023-8-14 20:25:46 | 显示全部楼层
Genson 发表于 2023-8-12 16:08
那你再好好想想,数据处理要单独受控的,不能来什么数据都处理

串口中断也能改,我想问问在不改变的情况下加什么判断可以优化这种一直循环的代码
回复

使用道具 举报

12

主题

3344

帖子

1

精华

论坛元老

Rank: 8Rank: 8

积分
8461
金钱
8461
注册时间
2020-5-11
在线时间
3904 小时
发表于 2023-8-15 09:36:00 | 显示全部楼层
hellobye123 发表于 2023-8-14 20:23
void main(void)
{
  //int i;

这样是不行的,Mosbus_Event()只能收到数据且在一段时间(如5ms,或利用空闲中断)内没有再收到串口数据(表示一帧数据完成)后调用一次。
专治疑难杂症
回复

使用道具 举报

1

主题

10

帖子

0

精华

新手上路

积分
37
金钱
37
注册时间
2020-2-2
在线时间
6 小时
发表于 2023-8-15 17:58:34 | 显示全部楼层
hellobye123 发表于 2023-8-14 20:25
串口中断也能改,我想问问在不改变的情况下加什么判断可以优化这种一直循环的代码

做分帧,加逻辑判断
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则



关闭

原子哥极力推荐上一条 /2 下一条

正点原子公众号

QQ|手机版|OpenEdv-开源电子网 ( 粤ICP备12000418号-1 )

GMT+8, 2025-2-24 06:57

Powered by OpenEdv-开源电子网

© 2001-2030 OpenEdv-开源电子网

快速回复 返回顶部 返回列表