Цитата(Kuzmi4 @ Nov 16 2009, 02:40)

"излишне усложненно "- это где?,
блок wr_logger, логика однократного срабатывания после сброса. Вы городите подобие КА, которое тут ну совершенно не к месту. ИМХО проще сделать так :
Код
if (rst)
pwr_srl <= 2'b01;
else
pwr_srl <= (pwr_srl << 1);
..
pwr_on_sig <= pwr_srl[1];
Цитата
чем описание КА страшное, .. ??
кодирование состояний у вас в голове и это цифры, которые нужно помнить и анализировать. Куда нагляднее читать в коде что то вроде STATE_INIT_RAM_CLEAR/STATE_RAM_CLEAR чем мифические 4'd02/4'd03 и т.д.
Цитата
что именно сумбурно
совершенно лишнее использование макросов для задания границ счетчиков, есть ненужный перерасход ресурса и комменты стоят странно, часть фсм полностью расписана вплоть до примитивных действий (что ИМХО не нужно), другая часть вообще не комментирована.