Problem in using shift operator

I encountered a problem in using the shift operator. In the example below, ser_data is one bit wide and data_reg, input_data, temp_reg are 60 bits wide. This is an example of serializing data. Data is first read in data_reg. It is then shifted and sent out via ser_data.


// Read data @ encoder clock
always_ff @(posedge clk_en) 
	begin
	if (reset_n)
		data_reg <= input_data;
	end

// Write data to output @ 60*encoder clock
always_ff @(clk_ser) 
	begin
	if (reset_n)
	    begin
	    ser_data <= data_reg[59]; 
	    data_reg <= data_reg<< 1; // **THIS DOESN’T WORK: data_reg is always 0.**
	    end
	end 

For debugging purposes, I used: temp_reg <= data_reg <<1
Now I can see the shifted data in temp_reg. Why doesn’t “data_reg <= data_reg<< 1” work ?

You are making assignments to data_reg in two different always_ff blocks. Perhaps you have a race condition.

Dave,
Actually, 1800-2009 section 9.2.2.4 Sequential logic always_ff procedure says

"Variables on the left-hand side of assignments within an always_ff procedure,
including variables from the contents of a called function, shall not be written to by any other process.

Thus, the
“data_reg <= data_reg<< 1; // THIS DOESN’T WORK: data_reg is always 0.” is illegal.


Ben Cohen, Design and verification expert (310) 997-2187
http://www.systemverilog.us/ ben@systemverilog.us

  • SystemVerilog Assertions Handbook, 3rd Edition, 2013 ISBN 0-9705394-3-6
  • A Pragmatic Approach to VMM Adoption 2006 ISBN 0-9705394-9-5
  • Using PSL/SUGAR for Formal and Dynamic Verification 2nd Edition, 2004, ISBN 0-9705394-6-0
  • Real Chip Design and Verification Using Verilog and VHDL, 2002 isbn 0-9705394-2-8
  • Component Design by Example, 2001 ISBN 0-9705394-0-1
  • VHDL Coding Styles and Methodologies, 2nd Edition, 1999 ISBN 0-7923-8474-1
  • VHDL Answers to Frequently Asked Questions, 2nd Edition ISBN 0-7923-8115

In reply to ben@SystemVerilog.us:

@ben, dave - thank you