Hi...
suppose i have a binary number i= 00110001
then i want to count the different run lengths present in this number and send it to output.
so i get three run lengths for given number as 2,0,3.
then for this i have written code which is as follows:

module runcount(i, o);
    input [7:0] i;
    output [3:0]o;
	 reg [3:0]o;
	 integer count=0;
	 integer j;
	

	 always@(i)
	 begin
	 for(j=7;j<=0;j=j-1)
	 begin
	 
	 if(i[j]==0)
	 begin
	 count=count+1;
	 end
	 else
	 begin
	 o=count;
	
	count=0;
	 end
	 end	
end 
endmodule

but in this code me getting syntax correct but as i tried to check its simulation using test bench waveform i get output o as always 'XXXX'.
so plz help me to find out the error..

 

Hello,

I found this post in the "completed projects" forum.
As the name told you, it is for completed projects.
Over there the post will stay invisible, until the mods approve or move it.

Bertus

 

Without knowing much about "run lengths", or the code, it appears as though you have a looping structure that is supposed to "look at" the current value loaded into "i", but your "always@" loop refers to "i" also. Did you mean always@(posedge clk) or something along those lines? As is, the code will only execute (I think) when "i" changes.

I think this is going to be a learning experience for both of us; I do not know verilog all that well...just very basic stuff.

 

Your problem is that you are approaching this as a software problem rather than a hardware circuit. You generally want to have a circuit in mind before you write any verilog or VHDL.

You need to totally re-think how to do this. But I don't know what you mean by 'run lengths' either. Your code looks like you're just trying to count the number of zeros in a byte (if it were software, that is...). Off the top of my head, a shift register, a 3-bit accumulator, an and gate, and an inverter will get you there with minimal logic and will cost you 8 clock cycles. You could also do it purely combinationally, but will cost you area.

Point is, design your circuit first. Then think about how to code up each individual building block. Then put it all together.

For the record, loops get unrolled by the synthesizer. And integers default to 32 bits.

 

I agree with both replies above, plus it appears that you may have a logic error in your "for" statement as shown.