Вид для печати
dosikus, он должен быть как логическая сумма обоих гашений:
blank <= ~((pix_count < d800) & (line_count < d600))
Инверсия - потому что гашение, а не окно. Ну и ты dXXX замени на константы свои, при этом -1 делать не надо. -1 только в счётчиках быть должно. Ты понимаешь, что твоё "line_count>=0" и "pix_count>=0" лишнее даже с точки зрения программирования? Ведь счетчик - число целочисленное от 0 и до константы. А лишнее условие = лишние ресурсы.
На константы типа d800 - ругается.
Переделал так - растр пропал и сигнал не видит...
Код:if ( pix_count < 800 ) pix_x <= pix_count; else pix_x <= 0;
if ( line_count < 600) pix_y <= line_count; else pix_y <= 0;
disp_enable <= ~((pix_count < 800) & (line_count < 600));
if(disp_enable)v_addr <= ((pix_y*800)+pix_x);
Весь код:
Код:module vga_sync(
input clk_in, // Input 40 MHz clock, this is a pixel clock for this VGA mode
input reset, // Input async. active low reset signal
output reg vga_hsync, // Output horizontal sync signal
output reg vga_vsync, // Output vertical sync signal
output reg disp_enable, // Set when a writable portion of display is enabled:
output reg[9:0] pix_x, // x-coordinate of an active pixel
output reg[9:0] pix_y, // y-coordinate of an active pixel
output wire[7:0] data_out
);
//======================================================================
localparam SYNC_ON = 1'b1; // Define the polarity of sync pulses (psitive)
localparam SYNC_OFF = 1'b0;
localparam HSYNC_START = (840-1);
localparam HSYNC_END = (968-1);
localparam LINE_END = (1056-1);
localparam VSYNC_START = (601-1);
localparam VSYNC_END = (605-1);
localparam FRAME_END = (628-1);
localparam H_ACTIV = 800;
localparam V_ACTIV = 600;
reg [7:0] vram [0:4095]; initial $readmemh("vram.dat", vram);
reg[9:0] line_count; // Line counter, current line
reg[15:0] pix_count; // Pixel counter, current pixel
reg[19:0] v_addr; // vaddres counter
reg [1:0]SynRes;
always @( posedge clk_in )
begin
// reset
SynRes[1:0] <= {SynRes[0],reset};
// pixel count
if ( (SynRes[1] & ~SynRes[0]) | (pix_count == LINE_END) ) pix_count <= 0; else pix_count <= pix_count + 1;
// line count
if ( (SynRes[1] & ~SynRes[0]) | (line_count == FRAME_END ) ) line_count <= 0;
else if ( pix_count == LINE_END ) line_count <= line_count + 1;
// synhroimpulse
if (SynRes[1] & ~SynRes[0]) begin vga_hsync <=0; vga_vsync<=0; end
if(pix_count== HSYNC_START)vga_hsync <= SYNC_ON;else if (pix_count==HSYNC_END)vga_hsync<=SYNC_OFF;
if(line_count== VSYNC_START)vga_vsync <= SYNC_ON;else if (line_count==VSYNC_END)vga_vsync<=SYNC_OFF;
if (SynRes[1] & ~SynRes[0]) v_addr <=0;// or this: if (SynRes[1:0] == 2'b10)
else begin
if ( pix_count < 800 ) pix_x <= pix_count; else pix_x <= 0;
if ( line_count < 600) pix_y <= line_count; else pix_y <= 0;
disp_enable <= ~((pix_count < 800) & (line_count < 600));
if(disp_enable)v_addr <= ((pix_y*800)+pix_x);
end
end
assign data_out = vram[v_addr];
endmodule
dosikus, :v2_dizzy_facepalm:
Я имел в виду поменять на твои именованные константы:
Цитата:
Сообщение от dosikus
Дык все равно что-то я сломал - пропала синхронизация с изображением.
dosikus, ну конечно сломал. Верни константы на место, а -1 поставь в те условия, где они требуются.
HardWareMan, не все проще , я пропустил что ты нормальный blanc сделал а не мой disp_enable, счас все работает.
У меня же и HDMI с инвертированным blanc.
Позже перепишу ...
Счас так:
RTLКод:module vga_sync(
input clk_in, // Input 40 MHz clock, this is a pixel clock for this VGA mode
input reset, // Input async. active low reset signal
output reg vga_hsync, // Output horizontal sync signal
output reg vga_vsync, // Output vertical sync signal
output reg disp_enable, // Set when a writable portion of display is enabled:
output reg[9:0] pix_x, // x-coordinate of an active pixel
output reg[9:0] pix_y, // y-coordinate of an active pixel
//output reg[19:0] v_addr, // VRAM address count
output wire[7:0] data_out
);
//======================================================================
localparam SYNC_ON = 1'b1; // Define the polarity of sync pulses (psitive)
localparam SYNC_OFF = 1'b0;
localparam HSYNC_START = (840-1);
localparam HSYNC_END = (968-1);
localparam LINE_END = (1056-1);
localparam VSYNC_START = (601-1);
localparam VSYNC_END = (605-1);
localparam FRAME_END = (628-1);
localparam H_ACTIV = 800;
localparam V_ACTIV = 600;
reg [7:0] vram [0:4095]; initial $readmemh("vram.dat", vram);
reg[9:0] line_count; // Line counter, current line
reg[15:0] pix_count; // Pixel counter, current pixel
reg[19:0] v_addr; // vaddres counter
reg [1:0]SynRes;
always @( posedge clk_in )
begin
// reset
SynRes[1:0] <= {SynRes[0],reset};
// pixel count
if ( (SynRes[1] & ~SynRes[0]) | (pix_count == LINE_END) ) pix_count <= 0; else pix_count <= pix_count + 1;
// line count
if ( (SynRes[1] & ~SynRes[0]) | (line_count == FRAME_END ) ) line_count <= 0;
else if ( pix_count == LINE_END ) line_count <= line_count + 1;
// synhroimpulse
if (SynRes[1] & ~SynRes[0]) begin vga_hsync <=0; vga_vsync<=0; end
if(pix_count== HSYNC_START)vga_hsync <= SYNC_ON;else if (pix_count==HSYNC_END)vga_hsync<=SYNC_OFF;
if(line_count== VSYNC_START)vga_vsync <= SYNC_ON;else if (line_count==VSYNC_END)vga_vsync<=SYNC_OFF;
if (SynRes[1] & ~SynRes[0]) v_addr <=0;// or this: if (SynRes[1:0] == 2'b10)
if ( pix_count < H_ACTIV ) pix_x <= pix_count; else pix_x <= 0;
if ( line_count < V_ACTIV) pix_y <= line_count; else pix_y <= 0;
//disp_enable <= ~((pix_count < 800) & (line_count < 600));
disp_enable <= ((pix_count < H_ACTIV) & (line_count < V_ACTIV));
if(disp_enable)v_addr <= ((pix_y*H_ACTIV)+pix_x);
end
assign data_out = vram[v_addr];
endmodule
Скрытый текст
Ааа, ну вообще-то я конкретно указал, что инверсия. Для disp_enable надо убрать инверсию.
Осталось решить соответствие pix_x - v_addr.
https://a.radikal.ru/a33/1909/ce/132c514059b4.png
https://a.radikal.ru/a01/1909/db/081a492879f5.png
Чего НЕ вижу - отсутствует blank, т.е. не будет привязки к уровню черного, да и гашения видеосигнала тоже.Цитата:
Сообщение от dosikus
Чего вижу - регистры pix_x и pix_y. Зачем они?
На счетчик адреса у тебя всё равно отдельный регистр v_addr, ну так его по НЕ_бланку и инкременируй.Цитата:
Сообщение от dosikus
На 1 такт раньше LINE_END, в дальнейшем будет на 2, так как тебе ещё знакогенератор читать.
В результате у тебя добавятся 2 флага:
1 - разрешение инкремента счётчика адреса, он же открывает доступ к видеопамяти;
2 - blank
blank отстаёт от первого (сейчас) ровнёхонько на 1 такт, проще сделать просто задержку на триггере, чтоб не плодить компараторы.
- - - Добавлено - - -
Вероятно я хреново объясняю...
Неприятность в том, что у тебя происходит задержка счёта адреса относительно развёртки. Вот здесь:
первый такт
второй тактКод:disp_enable <= ((pix_count < H_ACTIV) & (line_count < V_ACTIV));//строка 56
трерий тактКод:if ( pix_count < H_ACTIV ) pix_x <= pix_count; else pix_x <= 0;//строка 53
Если считать с нуля и использовать счётчики развёрток, то задержка будет всегда.Код:if(disp_enable)v_addr <= ((pix_y*H_ACTIV)+pix_x);// строка 57
Ты хочешь видеопамять линейную, то есть с 0 до 799 строка 1, а с 800 до 1599 строка 2 и т.д. Это пожалуйста, это сколько угодно.
У тебя счетчик адреса видеопамяти отдельный и нет никакого смысла привязываться к счетчикам синхросигналов.
Также нет никакого смысла начинать растр сразу с видеоданных, более того, это неприятно ещё и тем, что когда счётчик строк используется в формировании адреса видеопамяти, то первая строка растра остаётся без преамбулы (для спека это бордюр), приходится немного "колдовать", чтоб это выправить - инкремент счетчика строк делать в определённом месте.
Сейчас у тебя горизонтальный счётчик по первому варианту на картинке, чтобы не вывихивать моск при расчётах, я бы сдвинул растр как во втором или в третьем варианте, тогда не надо будет колдовать с разрядностью парамов/локалпарамрв так как не будет отрицательных значений в расчётах и будет проще забить координаты всех синхросигналов в автомат.
omercury, Да не, все нормально объясняешь,
просто вчера немного позволил , продолжу только в понедельник.
Спасибо вам всем, начинаю уже въезжать...