Forum Discussion

Altera_Forum's avatar
Altera_Forum
Icon for Honored Contributor rankHonored Contributor
9 years ago

Difference between simulation and real test.

Hi all, I am a student and I am working on a homework. Shortly, I will summary the hw's requirement. It said some thing like that: At first, when counter goes from 0->7 then led 1 on; when the cou...
Altera_Forum's avatar
Altera_Forum
Icon for Honored Contributor rankHonored Contributor
9 years ago

I (too) think you should be more explicit with your case statement's 'default'. Why not:

default: Y=fer;

This line:
if (y==transfer && count==15) //This to reset the counter
stands out to me. 'y' is moved on to 'flush' when count = 14. So, I don't think this condition can ever be true.

On a more general note - use more descriptive signal/register names. I'd suggest 'y' and 'Y' are particularly poor. Use 'present_state' & 'next_state', as per your notes. Much clearer for anyone reviewing or you coming back to your code later.