ПЛИС и всё что с ними связано

[HardWareMan]

pix_count - cчетчик вверх с параллельной загрузкой, со сбросом при значении 1056-1 - длина всей строки с со всеми причиндалами.

Это не синхронный дизайн здорового человека. У тебя MUX21 и так делает нормальное синхронное обнуление на шаге h4F1 = 1055. Так зачем этот асинхронный сброс?

- - - Добавлено - - -

И еще, ты старайся указывать размерность жгута. Ато он у тебя расширяется до стандартных 32 бит. Понятно, что фиттер потом урежет до используемого количества, но чётко указанная размерность делает схему в RTL чище.

[Advertiser]

[dosikus]

Это не синхронный дизайн здорового человека. У тебя MUX21 и так делает нормальное синхронное обнуление на шаге h4F1 = 1055. Так зачем этот асинхронный сброс?

Наверняка он здесь :
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

И еще, ты старайся указывать размерность жгута. Ато он у тебя расширяется до стандартных 32 бит. Понятно, что фиттер потом урежет до используемого количества, но чётко указанная размерность делает схему в RTL чище.

А это скорей всего я внес здесь :

localparam integer H_ACTIV = 800;
localparam integer V_ACTIV = 600;

Ибо рассчитывал адресc vram - H_ACTIV*V_ACTIV, синтез мне обрезал результат до 16бит.
Потом расчет переделал на v_addr <= ((pix_y*800)+pix_x);
и про то забыл..

[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

А, ну да. Для полностью синхронного дизайна никогда не используй список чувствительности более одного сигнала. Для этого тебе следует сделать вот так:

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

2 бита это если нужно отловить конкретно перепад, здесь я отлавливаю 1 => 0. Если перепад не нужно (сработка по уровню), то достаточно одного бита, главное это засинхронизировать его (перенести в текущий тактовый домен). В идеале, список чувствительности блока always должен содержать только тактовый сигнал, причем либо posedge, либо negedge. И не обязательно мастерклока. Т.е., если ведомый блок always у тебя не имеет перекликания с другими always блоками, то его тактовая может быть выходом триггера предыдущего блока. Но это уже с опытом придёт.

И ещё, обязательно усвой разницу инверсии между ~ и !. Она принципиально разная.

[dosikus]

Стало :

[HardWareMan]

Ай, красиво. А что сказал time_quest? Какой получился fmax?

[dosikus]

Угу, еще бы я им пользоваться умел ...

[omercury]

А это скорей всего я внес здесь :

localparam integer H_ACTIV = 800;
localparam integer V_ACTIV = 600;

Ибо рассчитывал адресc vram - H_ACTIV*V_ACTIV, синтез мне обрезал результат до 16бит.
Потом расчет переделал на v_addr <= ((pix_y*800)+pix_x);
и про то забыл..

Это автоматом проще рассчитывается:

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]);

Когда наводишь мышь в окне проект манагера на какой-либо модуль, в тултипе Квартус показывает значения parameter.
localparam не показывает(((

[omercury]

Угу, еще бы я им пользоваться умел ...

В простейшем случае ничего особо и не надо.
В Compilation Report в окне Table of content найди в дереве TimeQuest Timing Analiser и выбери в нём медленные модели.
Что-то он там по-любому наанализировал...

[dosikus]

Олег, ок порядок наведем, счас затык в другом - ранее как и писал - может сделаем знакогенератор ?

[HardWareMan]

omercury, нужно только SDC файл создать, со требуемой частотой.

dosikus, вот у тебя 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

И посмотри, какая красота появится в твоём RTL (только поправь настоящие константы и размерности, я там просто что-бы понятно было написал). Суть посыла такая: синтезатор, к сожалению, не умеет объединять некоторые события сам, справедливо полагая, что они имеют некоторый приоритет, описанный программистом. Однако, схемотехник сразу скажет - тут и тут лишнее, можно упростить. И если ты знаешь что и как - это не сложно объяснить синтезатору. Основная цель - сократить число управляющих цепей. В данном случае цепь сброса счётчика pix_count может быть объединена с цепью обнуления при счёте. Желательно всегда стремиться к уменьшению вложенности управляющих цепей (это те, которые условия), а так же стараться объединять цепи, ведущие к одному результату.

А в целом, тут ситуация как с блок-схемами алгоритма при обычном программировании - все настойчиво игнорируют этот момент, а потом получают спагетти-код. А ведь описать код по блок-схеме реально быстро и просто, я пробовал. Правда, в институте, когда на кафедре АСУ учился.