Вид для печати
Это не синхронный дизайн здорового человека. У тебя MUX21 и так делает нормальное синхронное обнуление на шаге h4F1 = 1055. Так зачем этот асинхронный сброс?Цитата:
Сообщение от dosikus
- - - Добавлено - - -
И еще, ты старайся указывать размерность жгута. Ато он у тебя расширяется до стандартных 32 бит. Понятно, что фиттер потом урежет до используемого количества, но чётко указанная размерность делает схему в RTL чище.
Цитата:
Сообщение от HardWareMan
Наверняка он здесь :
always @( posedge clk_in or negedge reset )
begin
if (!reset) begin
line_count <= 0; // On a reset, restart counters from 0
pix_count <= 0;
v_addr <=0;
end else begin
А это скорей всего я внес здесь :Цитата:
Сообщение от HardWareMan
localparam integer H_ACTIV = 800;
localparam integer V_ACTIV = 600;
Ибо рассчитывал адресc vram - H_ACTIV*V_ACTIV, синтез мне обрезал результат до 16бит.
Потом расчет переделал на v_addr <= ((pix_y*800)+pix_x);
и про то забыл..
А, ну да. Для полностью синхронного дизайна никогда не используй список чувствительности более одного сигнала. Для этого тебе следует сделать вот так:Цитата:
Сообщение от dosikus
2 бита это если нужно отловить конкретно перепад, здесь я отлавливаю 1 => 0. Если перепад не нужно (сработка по уровню), то достаточно одного бита, главное это засинхронизировать его (перенести в текущий тактовый домен). В идеале, список чувствительности блока always должен содержать только тактовый сигнал, причем либо posedge, либо negedge. И не обязательно мастерклока. Т.е., если ведомый блок always у тебя не имеет перекликания с другими always блоками, то его тактовая может быть выходом триггера предыдущего блока. Но это уже с опытом придёт.Код:reg [1:0]SynRes;
always @( posedge clk_in )
begin
// Синхронизация сброса
SynRes[1:0] <= {SynRes[0],reset};
if (SynRes[1] & ~SynRes[0]) begin // Либо так: if (SynRes[1:0] == 2'b10) begin
line_count <= 0; // On a reset, restart counters from 0
pix_count <= 0;
v_addr <=0;
end else begin
И ещё, обязательно усвой разницу инверсии между ~ и !. Она принципиально разная.
Ай, красиво. А что сказал time_quest? Какой получился fmax?
Угу, еще бы я им пользоваться умел ...
Это автоматом проще рассчитывается:Цитата:
Сообщение от dosikus
Когда наводишь мышь в окне проект манагера на какой-либо модуль, в тултипе Квартус показывает значения parameter.Код:parameter hcnt_Width = $clog2(LINE_END); //Ширина горизонтального счётчика, вычисляемое
parameter hAct_Width = $clog2(H_ACTIV); //Ширина шины адреса по горизонтали, вычисляемое
parameter vcnt_Width = $clog2(FRAME_END); //Ширина вертикального счётчика, вычисляемое
parameter vAct_Width = $clog2(V_ACTIV); //Ширина шины адреса по вертикали, вычисляемое
parameter VRAM_Width = vAct_Width + hAct_Width; //Ширина шины адреса памяти, вычисляемое
reg pix_count [hcnt_Width-1:0] = 0; // Горизонтальный счетчик
reg line_count [vcnt_Width-1:0] = 0; // Вертикальный счетчик
wire [VRAM_Width-1: 0] VRAM_Addr = {line_count [vAct_Width-1:0], pix_count [hAct_Width-1:0]);
localparam не показывает(((
В простейшем случае ничего особо и не надо.Цитата:
Сообщение от dosikus
В Compilation Report в окне Table of content найди в дереве TimeQuest Timing Analiser и выбери в нём медленные модели.
Что-то он там по-любому наанализировал...
Вложение 70073
Олег, ок порядок наведем, счас затык в другом - ранее как и писал - может сделаем знакогенератор ?
omercury, нужно только SDC файл создать, со требуемой частотой.
dosikus, вот у тебя pix_count только либо обнуляется, либо считает в плюс, верно?
https://b.radikal.ru/b21/1909/18/d8297ca84942.jpg
А ты вынеси его за скобки, типа вот так:
И посмотри, какая красота появится в твоём RTL (только поправь настоящие константы и размерности, я там просто что-бы понятно было написал). Суть посыла такая: синтезатор, к сожалению, не умеет объединять некоторые события сам, справедливо полагая, что они имеют некоторый приоритет, описанный программистом. Однако, схемотехник сразу скажет - тут и тут лишнее, можно упростить. И если ты знаешь что и как - это не сложно объяснить синтезатору. Основная цель - сократить число управляющих цепей. В данном случае цепь сброса счётчика pix_count может быть объединена с цепью обнуления при счёте. Желательно всегда стремиться к уменьшению вложенности управляющих цепей (это те, которые условия), а так же стараться объединять цепи, ведущие к одному результату.Код:reg [1:0]SynRes;
always @( posedge clk_in )
begin
// Синхронизация сброса
SynRes[1:0] <= {SynRes[0],reset};
// Счетчик пикселей
if ( (SynRes[1] & ~SynRes[0]) | (pix_count == h41F) ) pix_count <= 0; else pix_count <= pix_count + 1;
// Остальная логика
if (SynRes[1] & ~SynRes[0]) begin // Либо так: if (SynRes[1:0] == 2'b10) begin
line_count <= 0; // On a reset, restart counters from 0
v_addr <=0;
end else begin
А в целом, тут ситуация как с блок-схемами алгоритма при обычном программировании - все настойчиво игнорируют этот момент, а потом получают спагетти-код. А ведь описать код по блок-схеме реально быстро и просто, я пробовал. Правда, в институте, когда на кафедре АСУ учился. :)