Мгновенный модуль не выполняет вычитание должным образом - PullRequest
0 голосов
/ 21 апреля 2019

Я пытаюсь выполнить вычитание, когда ALX равен 1, а ALY равен 0, используя сумматор с мгновенным переносом, но он не работает должным образом. Это прекрасно работает для дополнения.

  • если ALX равен 0, а ALY также равен 0, он загрузит вход для вывода
  • если ALX равен 0, а ALY равен 1, это добавит ввод к выводу
  • если ALX равен 1, а ALY равен 0, он будет вводиться на выход
  • если ALX равен 1, а ALY также равен 1, он будет и введен для вывода

Вот некоторые из моих модулей alu

module alu (CLK, RST, ALE, ALX , ALY, iDATA, oDATA , flags,aDATA);
input wire CLK, ALE, ALX, ALY,RST;
input wire [3:0] iDATA;
output reg [3:0] oDATA;
output reg [3:0] flags;
output reg [2:0] aDATA;

//declare variables

reg [3:0] nextflags;
wire [3:0] ALU;
wire [3:0] SUM;
wire CIN;
wire claCF,claZF,claNF,claVF;

cla4 test(iDATA,oDATA,CIN,SUM,claCF,claZF,claNF,claVF);
assign CIN = ALX;
assign ALU = ALX ? (ALY ? iDATA & oDATA : SUM) : (ALY ? SUM : iDATA ); //loop 
always @ (posedge CLK, posedge RST) begin
if (RST) begin
oDATA <= 0;
flags <= 0;
end

else if(ALE) begin
oDATA <= ALU;
flags <= nextflags;
end

else begin
oDATA <= oDATA;
flags <= flags;
end

always @ (*) begin
aDATA[2:0] = {ALE,ALX,~CIN};
nextflags[3] = ALX ? ( ALY ? flags[3]: claCF ) : (ALY ? claCF : 0 );
nextflags[2] = (ALU==0);
nextflags[1] = ALU[3];
nextflags[0] = ALX ? ( ALY ? flags[0] : claVF ) : (ALY ? claVF : 0 );
end
endmodule

Вот мой вывод

 time:                   0 OUTPUT:0000 ADATA:001 C:0 Z:0 N:0 V:0 
 time:                   1 OUTPUT:0000 ADATA:110 C:0 Z:0 N:0 V:0 
 time:                   2 OUTPUT:0001 ADATA:110 C:1 Z:0 N:0 V:0 
 time:                   4 OUTPUT:0000 ADATA:110 C:1 Z:1 N:0 V:0 
 time:                   6 OUTPUT:0001 ADATA:110 C:1 Z:0 N:0 V:0 
 time:                   8 OUTPUT:0000 ADATA:110 C:1 Z:1 N:0 V:0 
 time:                  10 OUTPUT:0001 ADATA:110 C:1 Z:0 N:0 V:0 
 time:                  12 OUTPUT:0000 ADATA:110 C:1 Z:1 N:0 V:0 
 time:                  14 OUTPUT:0001 ADATA:110 C:1 Z:0 N:0 V:0 
 time:                  16 OUTPUT:0000 ADATA:110 C:1 Z:1 N:0 V:0 
 time:                  18 OUTPUT:0001 ADATA:110 C:1 Z:0 N:0 V:0 
 time:                  20 OUTPUT:0000 ADATA:110 C:1 Z:1 N:0 V:0 

Ответы [ 2 ]

0 голосов
/ 22 апреля 2019

Я допустил глупую ошибку и исправил ее, изменив местоположение oDATA и iDATA для функции сумматора упреждающего просмотра, вызывая

cla4 test(oDATA,iDATA,CIN,SUM,claCF,claZF,claNF,claVF);

, потому что в моем модуле сумматора упреждающего просмотра я объявляю первую переменнуюв качестве переменной, которую нужно добавить в.

module cla4 (X,Y,M,S,CF,ZF,NF,VF);

assign G = X & (Y ^ {4{CIN}});
assign P = X ^ (Y ^ {4{CIN}});

, поэтому выход будет работать в цикле, потому что он будет в основном перезаписывать его с переносом в себя.Значение просто меняется вперед и назад между значением xor и его исходным значением.

0 голосов
/ 21 апреля 2019

Вы бы предоставили полный код?трудно найти ошибку из-за именования IO.

у вас есть последовательная схема, поэтому вы должны сначала дождаться нарастающего фронта тактовой частоты, затем проверить изменения в управляющих сигналах (ALY, ALX), а затем, наконец,распространять сигналы данных в комбинационную схему (пункт 4).Таким образом, вы должны реализовать четвертую строку в блоке always.

btw, лучше избегать использования нескольких условий в операторе assign, если вы не уверены на 100%, что это правильно.

в блоке alwaysВы можете использовать регистр:

   always @ (posedge CLK, posedge RST) begin
if (RST) begin
oDATA <= 0;
flags <= 0;
end

else 
    case({ALX ,ALY}) 
        2'b00 : begin /* */ end
        2'b10 : begin /* */ end
        2'b01 : begin /* */ end
        2'b11 : begin /* */ end



    endcase
   end
...