在线咨询
eetop公众号 创芯大讲堂 创芯人才网
切换到宽版

EETOP 创芯网论坛 (原名:电子顶级开发网)

手机号码,快捷登录

手机号码,快捷登录

找回密码

  登录   注册  

快捷导航
搜帖子
查看: 9568|回复: 33

[讨论] 请大家看一下这种代码会有什么样的风险?

[复制链接]
发表于 2013-9-1 13:15:21 | 显示全部楼层 |阅读模式

马上注册,结交更多好友,享用更多功能,让你轻松玩转社区。

您需要 登录 才可以下载或查看,没有账号?注册

x
小弟正在做一个模块的设计,模块中用到了一些诸如下面写法的逻辑,领导看完后说这样写会有风险,过不了我们内部的代码审查,问他有什么风险,他又解释不清楚,代码如下:
always @ (posedge clk or negedge rst_n)
begin
    if (!rst_n)
    begin
        a <= 'b0;
    end
    else if (flag)
    begin
        a <= a + 1'b1;
    end
    else
    begin
        a <= a;
    end
end
不明白会出现什么样的风险,请各位大神前辈知无不言言无不尽,都说说这种计数器写法存在的问题,如何能改进,谢谢!
发表于 2013-9-1 14:09:25 | 显示全部楼层
回复 1# yangxiaoyong


   感觉没什么问题。看代码审查会给什么意见吧!
发表于 2013-9-1 15:05:46 | 显示全部楼层
我看也没什么问题啊。
 楼主| 发表于 2013-9-1 15:06:41 | 显示全部楼层
回复 2# rvnistelrooy


   谢谢,我也是实在看不出什么问题, 以前在上家单位的时候基本会用到很多这样的写法,都没有问题的。
 楼主| 发表于 2013-9-1 15:08:43 | 显示全部楼层
回复 3# bright888

谢谢,我也觉得没啥问题,领导说else分支的赋值是把输出接到输入上了。。。
发表于 2013-9-1 18:02:46 | 显示全部楼层
那是组合逻辑环,你这个是时序电路
 楼主| 发表于 2013-9-1 20:27:01 | 显示全部楼层
回复 6# wgej1987


   谢谢,我也知道组合逻辑环是不可以,时序逻辑我也没听说这样有什么不可以~只以为自己孤陋寡闻了~而且以前都这样写也一直没问题。。。
发表于 2013-9-2 08:52:18 | 显示全部楼层
应该正如你说的产生了反馈应该多少会影响,建议你再看看反馈是什么回事吧
发表于 2013-9-2 09:29:50 | 显示全部楼层
这样应该没有设么问题的
发表于 2013-9-2 12:16:16 | 显示全部楼层
最后一个else可以不写,不过写上去可读性更直观。
综合后,效率是一样的。
您需要登录后才可以回帖 登录 | 注册

本版积分规则

关闭

站长推荐 上一条 /1 下一条

小黑屋| 手机版| 关于我们| 联系我们| 在线咨询| 隐私声明| EETOP 创芯网
( 京ICP备:10050787号 京公网安备:11010502037710 )

GMT+8, 2024-11-5 14:48 , Processed in 0.031817 second(s), 8 queries , Gzip On, Redis On.

eetop公众号 创芯大讲堂 创芯人才网
快速回复 返回顶部 返回列表