0

我正在创建一个具有 4 个空的 4 位寄存器的寄存器文件，并且在每个时钟周期，一个寄存器获取一个推送到它的值，然后使用 7 段解码器显示该值。如果寄存器为空，则不应该有显示，但是一旦将值加载到寄存器中，则应该有显示。我尝试为 7 段解码器编写 verilog 代码，如果 En = 1，则只有输出。我的问题是，7 段解码器不会显示任何内容，而是显示零，直到将值添加到寄存器中，在在这种情况下，解码器将显示加载的值。这是verilog代码。

module seven_seg_decoder(S, Z, Y, X, W,En);
input Z,Y,X,W,En; 
output [6:0] S;
reg [6:0] S;

always @(En)
    begin
        case({Z,Y,X,W,En})
            5'b00001: S = 'b0000001;
            5'b00011: S = 'b1001111;
            5'b00101: S = 'b0010010;
            5'b00111: S = 'b0000110;
            5'b01001: S = 'b1001100;
            5'b01011: S = 'b0100100;
            5'b01101: S = 'b0100000;
            5'b01111: S = 'b0001111;
            5'b10001: S = 'b0000000;
            5'b10011: S = 'b0000100;
            5'b10101: S = 'b0001000;
            5'b10111: S = 'b1100000;
            5'b11001: S = 'b0110001;
            5'b11011: S = 'b1000010;
            5'b11101: S = 'b0110000;
            5'b11111: S = 'b0111000;

        endcase
    end
endmodule

当寄存器加载一个值时，我有一个来自寄存器的输出，表明它已加载，并用作此解码器上启用的输入

4

1 回答 1

1

查看{Z, Y, X, W, En}您列举的可能的编码：

5'b00001: S = 'b0000001;
5'b00011: S = 'b1001111;
5'b00101: S = 'b0010010;
5'b00111: S = 'b0000110;
5'b01001: S = 'b1001100;
5'b01011: S = 'b0100100;
5'b01101: S = 'b0100000;
5'b01111: S = 'b0001111;
5'b10001: S = 'b0000000;
5'b10011: S = 'b0000100;
5'b10101: S = 'b0001000;
5'b10111: S = 'b1100000;
5'b11001: S = 'b0110001;
5'b11011: S = 'b1000010;
5'b11101: S = 'b0110000;
5'b11111: S = 'b0111000;
//     ^ Look here

的所有值En都是1！你从来没有告诉模块如果En是0. 它不知道要关闭显示。在这种情况下，你需要给它一些事情做。

你有几个选择：

简单的解决方案
您可以添加一个默认子句，在所有未枚举的情况下使显示空白：

 default: S = 'b0000000; //change to whatever means "turn everything off"

不必要的复杂解决方案
您可以通过使用 5'bxxxx0 作为子句和casex语句来获得更精确：

casex ({Z,Y,X,W,En})
    // all other enumerated cases
    5'bxxxx0: S = 'b0000000; //change to "everything off"
    default:  S = 'b0000000; //change to "everything off"

但是要知道，有一些潜在的陷阱casex可能会使其难以正确使用并导致令人惊讶的行为。另请注意，这最终与第一个简单的解决方案相同。我暂时避免它。

（我的）首选解决方案

这实际上与前两个相同，但我认为意图更加明确：

always @(*)
    begin
        if (!En)
            S = 'b0000000; // change to "everything off"

        else
            case({Z,Y,X,W,En})
                5'b00001: S = 'b0000001;
                5'b00011: S = 'b1001111;
                5'b00101: S = 'b0010010;
                5'b00111: S = 'b0000110;
                5'b01001: S = 'b1001100;
                5'b01011: S = 'b0100100;
                5'b01101: S = 'b0100000;
                5'b01111: S = 'b0001111;
                5'b10001: S = 'b0000000;
                5'b10011: S = 'b0000100;
                5'b10101: S = 'b0001000;
                5'b10111: S = 'b1100000;
                5'b11001: S = 'b0110001;
                5'b11011: S = 'b1000010;
                5'b11101: S = 'b0110000;
                5'b11111: S = 'b0111000;
                default: S = 'b0000000; // change to some error code
            endcase       
    end

此解决方案的优点是代码的目标更加明确：如果En不正确，则其余信号无关紧要。我们清除显示。其他解决方案也会发生这种情况，但是这种结构使正在发生的事情变得更加明显。

从技术上讲，这里的default情况是不必要的。不应该有没有列举的情况En为真。但是要输入很多数字。如果你打错了，在这些情况下，有一个强制错误显示的默认值（可能只是 7seg 显示的中心线？）将帮助你识别错误。

其他一些注意事项：

  1. 您应该避免使用X和Z作为变量名。他们可能会因为“不关心”和“高阻抗”而感到困惑。它使代码比它需要的更难遵循。

  2. 我已经更改了你的敏感度列表。Greg 正确地指出，如果您希望在输入 、 和 更改时正确更新输出，则W需要在更改X时触发块。操作员将告诉 Verilog 编译器根据块的内容推断正确的敏感度列表。（请注意，这不会查看任务或功能来确定敏感度列表。）YZalways*always

  3. 我强烈建议您将所有位模式放入参数中，以使您的代码更具可读性。

于 2016-12-08T23:07:59.090 回答