andrew_b 16 19 июня, 2020 Опубликовано 19 июня, 2020 · Жалоба Только что, Maverick_ сказал: правильное ? все сигналы в списке чувствительности? Да. Но процесс тут ни к чему. Тут просится when else. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Maverick_ 15 19 июня, 2020 Опубликовано 19 июня, 2020 · Жалоба Еще есть замечания/предложения? Возможно что-то не понятно... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Nick_K 0 19 июня, 2020 Опубликовано 19 июня, 2020 · Жалоба Из быстрого просмотра: if (to_integer(unsigned(avs_csr_address)) = 1) then ... в интеджер не обязательно преобразовывать - и меньше работы и код чище, достаточно сравнить unsigned: if (unsigned(avs_csr_address) = 1) then А также в строках: avs_s5_readdata <= std_logic_vector(to_unsigned(0, 14))&b_dout_aX_gdata; и avs_s6_readdata <= std_logic_vector(to_unsigned(0, 14))&b_dout_aY_gdata; присваеваемые вектора имеют разрядность 32 бита. Не помню уже как себя будет вести синтезатор, но оставлять в воздухе половину вектора - плохая практика. Тем более что всё обявлено через generic'и, вот лучше разрядности задать через их родных. Это касательно чистоты кода. С остальным не работал, увы не подскажу Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться