Forum Discussion
Altera_Forum
Honored Contributor
12 years agoThere is nothing that strikes me as bad here, except that you are using both numeric_std.all and std_logic_unsigned.all at the same time, which is usually a bad idea and confuses the synthesizer. I would suggest to only use numeric_std.all and declare your cnt_r signal as an unsigned instead of std_logic_vector. If you want to keep blinker_o a std_logic_vector, you can cast it with this line outside of the process:
blinker_o <= std_logic_vector(cnt_r); -- update the outputs, pickoff last bit for output signalAre you sure the FPGA is picking the clock correctly? You could compile a simple design that would recopy the clk_i signal to an outside pin and check with an oscilloscope that it is indeed 24MHz.