Browse Source

projects/memtest: Instance port formatting / alignment

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
Sylvain Munaut 3 years ago
parent
commit
ea2e9a3613

+ 11 - 11
projects/memtest/rtl/hdmi_buf.v

@@ -28,18 +28,18 @@ module hdmi_buf (
 	generate
 		for (i=0; i<4; i=i+1)
 			ice40_ebr #(
-				.READ_MODE(2),
-				.WRITE_MODE(1)
+				.READ_MODE  (2),	// 1024x4
+				.WRITE_MODE (1)		//  512x8
 			) ebr_wrap_I (
-				.wr_addr(waddr),
-				.wr_data({wdata[i*4+:4], wdata[16+i*4+:4]}),
-				.wr_mask(8'h00),
-				.wr_ena(wren),
-				.wr_clk(clk),
-				.rd_addr(raddr),
-				.rd_data(rdata[i*4+:4]),
-				.rd_ena(1'b1),
-				.rd_clk(clk)
+				.wr_addr (waddr),
+				.wr_data ({wdata[i*4+:4], wdata[16+i*4+:4]}),
+				.wr_mask (8'h00),
+				.wr_ena  (wren),
+				.wr_clk  (clk),
+				.rd_addr (raddr),
+				.rd_data (rdata[i*4+:4]),
+				.rd_ena  (1'b1),
+				.rd_clk  (clk)
 			);
 	endgenerate
 

+ 42 - 42
projects/memtest/rtl/hdmi_out.v

@@ -146,8 +146,8 @@ module hdmi_out #(
 
 		// Standard 1080p60
 	vid_tgen #(
-		.H_WIDTH(12),
-		.V_WIDTH(12),
+		.H_WIDTH  (12),
+		.V_WIDTH  (12),
 		.H_FP     (  88 / 4),
 		.H_SYNC   (  44 / 4),
 		.H_BP     ( 148 / 4),
@@ -157,15 +157,15 @@ module hdmi_out #(
 		.V_BP     (  36),
 		.V_ACTIVE (1080)
 	) hdmi_tgen_I (
-		.vid_hsync(vt_hsync),
-		.vid_vsync(vt_vsync),
-		.vid_active(vt_de),
-		.vid_h_first(vt_hfirst),
-		.vid_h_last(),
-		.vid_v_first(vt_vfirst),
-		.vid_v_last(vt_vlast),
-		.clk(clk_1x),
-		.rst(rst)
+		.vid_hsync   (vt_hsync),
+		.vid_vsync   (vt_vsync),
+		.vid_active  (vt_de),
+		.vid_h_first (vt_hfirst),
+		.vid_h_last  (),
+		.vid_v_first (vt_vfirst),
+		.vid_v_last  (vt_vlast),
+		.clk         (clk_1x),
+		.rst         (rst)
 	);
 
 	assign vt_trig = vt_de & vt_hfirst;
@@ -226,12 +226,12 @@ module hdmi_out #(
 
 	// Memory
 	hdmi_buf line_I (
-		.waddr({vb_pingpong, vb_waddr}),
-		.wdata(vb_wdata),
-		.wren (vb_wren),
-		.raddr({~vb_pingpong, vb_raddr}),
-		.rdata(vb_rdata),
-		.clk(clk_1x)
+		.waddr ({vb_pingpong, vb_waddr}),
+		.wdata (vb_wdata),
+		.wren  (vb_wren),
+		.raddr ({~vb_pingpong, vb_raddr}),
+		.rdata (vb_rdata),
+		.clk   (clk_1x)
 	);
 
 
@@ -257,42 +257,42 @@ module hdmi_out #(
 				.AWIDTH(6),
 				.DWIDTH(DW)
 			) pal_I (
-				.wr_addr(pal_waddr),
-				.wr_data(pal_wdata),
-				.wr_ena(pal_wren),
-				.rd_addr({frame_cnt, vb_rdata[(3-i)*4+:4]}),
-				.rd_data(vo_data[i]),
-				.rd_ena(1'b1),
-				.clk(clk_1x)
+				.wr_addr (pal_waddr),
+				.wr_data (pal_wdata),
+				.wr_ena  (pal_wren),
+				.rd_addr ({frame_cnt, vb_rdata[(3-i)*4+:4]}),
+				.rd_data (vo_data[i]),
+				.rd_ena  (1'b1),
+				.clk     (clk_1x)
 			);
 	endgenerate
 
 	// Control delay
 	delay_bus #(3, 3) dly_vs_I (
-		.d({vt_hsync, vt_vsync, vt_de}),
-		.q({vo_hsync, vo_vsync, vo_de}),
-		.clk(clk_1x)
+		.d   ({vt_hsync, vt_vsync, vt_de}),
+		.q   ({vo_hsync, vo_vsync, vo_de}),
+		.clk (clk_1x)
 	);
 
 	// PHY
 	hdmi_phy_4x #(
 		.DW(DW)
 	) phy_I (
-		.hdmi_data(hdmi_data),
-		.hdmi_hsync(hdmi_hsync),
-		.hdmi_vsync(hdmi_vsync),
-		.hdmi_de(hdmi_de),
-		.hdmi_clk(hdmi_clk),
-		.in_data0(vo_data[0]),
-		.in_data1(vo_data[1]),
-		.in_data2(vo_data[2]),
-		.in_data3(vo_data[3]),
-		.in_hsync(vo_hsync),
-		.in_vsync(vo_vsync),
-		.in_de(vo_de),
-		.clk_1x(clk_1x),
-		.clk_4x(clk_4x),
-		.clk_sync(sync_4x)
+		.hdmi_data  (hdmi_data),
+		.hdmi_hsync (hdmi_hsync),
+		.hdmi_vsync (hdmi_vsync),
+		.hdmi_de    (hdmi_de),
+		.hdmi_clk   (hdmi_clk),
+		.in_data0   (vo_data[0]),
+		.in_data1   (vo_data[1]),
+		.in_data2   (vo_data[2]),
+		.in_data3   (vo_data[3]),
+		.in_hsync   (vo_hsync),
+		.in_vsync   (vo_vsync),
+		.in_de      (vo_de),
+		.clk_1x     (clk_1x),
+		.clk_4x     (clk_4x),
+		.clk_sync   (sync_4x)
 	);
 
 endmodule

+ 14 - 14
projects/memtest/rtl/memtest.v

@@ -86,26 +86,26 @@ module memtest #(
 		.AWIDTH(8),
 		.DWIDTH(32)
 	) buf_wr_I (
-		.wr_addr(bw_waddr),
-		.wr_data(bw_wdata),
-		.wr_ena(bw_wren),
-		.rd_addr(bw_raddr),
-		.rd_data(bw_rdata),
-		.rd_ena(bw_rden),
-		.clk(clk)
+		.wr_addr (bw_waddr),
+		.wr_data (bw_wdata),
+		.wr_ena  (bw_wren),
+		.rd_addr (bw_raddr),
+		.rd_data (bw_rdata),
+		.rd_ena  (bw_rden),
+		.clk     (clk)
 	);
 
 	ram_sdp #(
 		.AWIDTH(8),
 		.DWIDTH(32)
 	) buf_rd_I (
-		.wr_addr(br_waddr),
-		.wr_data(br_wdata),
-		.wr_ena(br_wren),
-		.rd_addr(br_raddr),
-		.rd_data(br_rdata),
-		.rd_ena(br_rden),
-		.clk(clk)
+		.wr_addr (br_waddr),
+		.wr_data (br_wdata),
+		.wr_ena  (br_wren),
+		.rd_addr (br_raddr),
+		.rd_data (br_rdata),
+		.rd_ena  (br_rden),
+		.clk     (clk)
 	);
 
 

+ 63 - 63
projects/memtest/rtl/sysmgr.v

@@ -24,73 +24,73 @@ module sysmgr (
 	wire       pll_lock;
 
 	SB_PLL40_2F_PAD #(
-		.FEEDBACK_PATH("SIMPLE"),
-		.DIVR(4'b0000),
+		.FEEDBACK_PATH                  ("SIMPLE"),
+		.DIVR                           (4'b0000),
 
 	// 48
-//		.DIVF(7'b0111111),
-//		.DIVQ(3'b100),
+//		.DIVF                           (7'b0111111),
+//		.DIVQ                           (3'b100),
 
 	// 96
-//		.DIVF(7'b0111111),
-//		.DIVQ(3'b011),
+//		.DIVF                           (7'b0111111),
+//		.DIVQ                           (3'b011),
 
 	// 144
-//		.DIVF(7'b0101111),
-//		.DIVQ(3'b010),
+//		.DIVF                           (7'b0101111),
+//		.DIVQ                           (3'b010),
 
 	// 147
-		.DIVF(7'b0110000),
-		.DIVQ(3'b010),
+		.DIVF                           (7'b0110000),
+		.DIVQ                           (3'b010),
 
 	// 200
-//		.DIVF(7'b1000010),
-//		.DIVQ(3'b010),
-
-		.FILTER_RANGE(3'b001),
-		.DELAY_ADJUSTMENT_MODE_RELATIVE("DYNAMIC"),
-		.FDA_RELATIVE(15),
-		.SHIFTREG_DIV_MODE(0),
-		.PLLOUT_SELECT_PORTA("GENCLK"),
-		.PLLOUT_SELECT_PORTB("GENCLK")
+//		.DIVF                           (7'b1000010),
+//		.DIVQ                           (3'b010),
+
+		.FILTER_RANGE                   (3'b001),
+		.DELAY_ADJUSTMENT_MODE_RELATIVE ("DYNAMIC"),
+		.FDA_RELATIVE                   (15),
+		.SHIFTREG_DIV_MODE              (0),
+		.PLLOUT_SELECT_PORTA            ("GENCLK"),
+		.PLLOUT_SELECT_PORTB            ("GENCLK")
 	) pll_I (
-		.PACKAGEPIN(clk_in),
-		.DYNAMICDELAY({delay, 4'h0}),
-		.PLLOUTGLOBALA(clk_rd),
-		.PLLOUTGLOBALB(clk_4x),
-		.RESETB(1'b1),
-		.LOCK(pll_lock)
+		.PACKAGEPIN    (clk_in),
+		.DYNAMICDELAY  ({delay, 4'h0}),
+		.PLLOUTGLOBALA (clk_rd),
+		.PLLOUTGLOBALB (clk_4x),
+		.RESETB        (1'b1),
+		.LOCK          (pll_lock)
 	);
 
 	ice40_serdes_crg #(
 		.NO_CLOCK_2X(0)
 	) crg_I (
-		.clk_4x(clk_4x),
-		.pll_lock(pll_lock),
-		.clk_1x(clk_1x),
-		.clk_2x(clk_2x),
-		.rst(rst)
+		.clk_4x   (clk_4x),
+		.pll_lock (pll_lock),
+		.clk_1x   (clk_1x),
+		.clk_2x   (clk_2x),
+		.rst      (rst)
 	);
 
 `ifdef MEM_spi
 	ice40_serdes_sync #(
-		.PHASE(2),
-		.NEG_EDGE(0),
+		.PHASE      (2),
+		.NEG_EDGE   (0),
 `ifdef VIDEO_none
-		.GLOBAL_BUF(0),
-		.LOCAL_BUF(0),
-		.BEL_COL("X22"),
-		.BEL_ROW("Y4"),
+		.GLOBAL_BUF (0),
+		.LOCAL_BUF  (0),
+		.BEL_COL    ("X22"),
+		.BEL_ROW    ("Y4"),
 `else
-		.GLOBAL_BUF(0),
-		.LOCAL_BUF(1),
-		.BEL_COL("X15")
+		.GLOBAL_BUF (0),
+		.LOCAL_BUF  (1),
+		.BEL_COL    ("X15")
 `endif
 	) sync_4x_I (
-		.clk_slow(clk_1x),
-		.clk_fast(clk_4x),
-		.rst(rst),
-		.sync(sync_4x)
+		.clk_slow (clk_1x),
+		.clk_fast (clk_4x),
+		.rst      (rst),
+		.sync     (sync_4x)
 	);
 
 	assign sync_rd = 1'b0;
@@ -98,31 +98,31 @@ module sysmgr (
 
 `ifdef MEM_hyperram
 	ice40_serdes_sync #(
-		.PHASE(2),
-		.NEG_EDGE(0),
-		.GLOBAL_BUF(0),
-		.LOCAL_BUF(1),
-		.BEL_COL("X12"),
-		.BEL_ROW("Y15")
+		.PHASE      (2),
+		.NEG_EDGE   (0),
+		.GLOBAL_BUF (0),
+		.LOCAL_BUF  (1),
+		.BEL_COL    ("X12"),
+		.BEL_ROW    ("Y15")
 	) sync_4x_I (
-		.clk_slow(clk_1x),
-		.clk_fast(clk_4x),
-		.rst(rst),
-		.sync(sync_4x)
+		.clk_slow (clk_1x),
+		.clk_fast (clk_4x),
+		.rst      (rst),
+		.sync     (sync_4x)
 	);
 
 	ice40_serdes_sync #(
-		.PHASE(2),
-		.NEG_EDGE(0),
-		.GLOBAL_BUF(0),
-		.LOCAL_BUF(1),
-		.BEL_COL("X13"),
-		.BEL_ROW("Y15")
+		.PHASE      (2),
+		.NEG_EDGE   (0),
+		.GLOBAL_BUF (0),
+		.LOCAL_BUF  (1),
+		.BEL_COL    ("X13"),
+		.BEL_ROW    ("Y15")
 	) sync_rd_I (
-		.clk_slow(clk_1x),
-		.clk_fast(clk_rd),
-		.rst(rst),
-		.sync(sync_rd)
+		.clk_slow (clk_1x),
+		.clk_fast (clk_rd),
+		.rst      (rst),
+		.sync     (sync_rd)
 	);
 `endif
 

+ 188 - 188
projects/memtest/rtl/top.v

@@ -127,18 +127,18 @@ module top (
 	uart2wb #(
 		.WB_N(3)
 	) if_I (
-		.uart_rx(uart_rx),
-		.uart_tx(uart_tx),
-		.uart_div(8'd16),
-		.wb_wdata(wb_wdata),
-		.wb_rdata(wb_rdata),
-		.wb_addr(wb_addr),
-		.wb_we(wb_we),
-		.wb_cyc(wb_cyc),
-		.wb_ack(wb_ack),
-		.aux_csr(aux_csr),
-		.clk(clk_1x),
-		.rst(rst)
+		.uart_rx  (uart_rx),
+		.uart_tx  (uart_tx),
+		.uart_div (8'd16),
+		.wb_wdata (wb_wdata),
+		.wb_rdata (wb_rdata),
+		.wb_addr  (wb_addr),
+		.wb_we    (wb_we),
+		.wb_cyc   (wb_cyc),
+		.wb_ack   (wb_ack),
+		.aux_csr  (aux_csr),
+		.clk      (clk_1x),
+		.rst      (rst)
 	);
 
 	assign dma_run = aux_csr[0];
@@ -162,95 +162,95 @@ module top (
 
 	// Controller
 	qpi_memctrl #(
-		.CMD_READ(16'hEBEB),
-		.CMD_WRITE(16'h0202),
-		.DUMMY_CLK(6),
-		.PAUSE_CLK(8),
-		.FIFO_DEPTH(1),
-		.N_CS(2),
-		.PHY_SPEED(PHY_SPEED),
-		.PHY_WIDTH(1),
-		.PHY_DELAY((PHY_SPEED == 1) ? 2 : ((PHY_SPEED == 2) ? 3 : 4))
+		.CMD_READ   (16'hEBEB),
+		.CMD_WRITE  (16'h0202),
+		.DUMMY_CLK  (6),
+		.PAUSE_CLK  (8),
+		.FIFO_DEPTH (1),
+		.N_CS       (2),
+		.PHY_SPEED  (PHY_SPEED),
+		.PHY_WIDTH  (1),
+		.PHY_DELAY  ((PHY_SPEED == 1) ? 2 : ((PHY_SPEED == 2) ? 3 : 4))
 	) memctrl_I (
-		.phy_io_i(phy_io_i),
-		.phy_io_o(phy_io_o),
-		.phy_io_oe(phy_io_oe),
-		.phy_clk_o(phy_clk_o),
-		.phy_cs_o(phy_cs_o),
-		.mi_addr_cs(mi_addr[31:30]),
-		.mi_addr({mi_addr[21:0], 2'b00 }),	/* 32 bits aligned */
-		.mi_len(mi_len),
-		.mi_rw(mi_rw),
-		.mi_valid(mi_valid),
-		.mi_ready(mi_ready),
-		.mi_wdata(mi_wdata),
-		.mi_wack(mi_wack),
-		.mi_wlast(mi_wlast),
-		.mi_rdata(mi_rdata),
-		.mi_rstb(mi_rstb),
-		.mi_rlast(mi_rlast),
-		.wb_wdata(wb_wdata),
-		.wb_rdata(wb_rdata[31:0]),
-		.wb_addr(wb_addr[4:0]),
-		.wb_we(wb_we),
-		.wb_cyc(wb_cyc[0]),
-		.wb_ack(wb_ack[0]),
-		.clk(clk_1x),
-		.rst(rst)
+		.phy_io_i   (phy_io_i),
+		.phy_io_o   (phy_io_o),
+		.phy_io_oe  (phy_io_oe),
+		.phy_clk_o  (phy_clk_o),
+		.phy_cs_o   (phy_cs_o),
+		.mi_addr_cs (mi_addr[31:30]),
+		.mi_addr    ({mi_addr[21:0], 2'b00 }),	/* 32 bits aligned */
+		.mi_len     (mi_len),
+		.mi_rw      (mi_rw),
+		.mi_valid   (mi_valid),
+		.mi_ready   (mi_ready),
+		.mi_wdata   (mi_wdata),
+		.mi_wack    (mi_wack),
+		.mi_wlast   (mi_wlast),
+		.mi_rdata   (mi_rdata),
+		.mi_rstb    (mi_rstb),
+		.mi_rlast   (mi_rlast),
+		.wb_wdata   (wb_wdata),
+		.wb_rdata   (wb_rdata[31:0]),
+		.wb_addr    (wb_addr[4:0]),
+		.wb_we      (wb_we),
+		.wb_cyc     (wb_cyc[0]),
+		.wb_ack     (wb_ack[0]),
+		.clk        (clk_1x),
+		.rst        (rst)
 	);
 
 	// PHY
 	generate
 		if (PHY_SPEED == 1)
 			qpi_phy_ice40_1x #(
-				.N_CS(2),
-				.WITH_CLK(1),
-				.NEG_IN(0)
+				.N_CS     (2),
+				.WITH_CLK (1),
+				.NEG_IN   (0)
 			) phy_I (
-				.pad_io(spi_io),
-				.pad_clk(spi_sck),
-				.pad_cs_n(spi_cs_n),
-				.phy_io_i(phy_io_i),
-				.phy_io_o(phy_io_o),
-				.phy_io_oe(phy_io_oe),
-				.phy_clk_o(phy_clk_o),
-				.phy_cs_o(phy_cs_o),
-				.clk(clk_1x)
+				.pad_io    (spi_io),
+				.pad_clk   (spi_sck),
+				.pad_cs_n  (spi_cs_n),
+				.phy_io_i  (phy_io_i),
+				.phy_io_o  (phy_io_o),
+				.phy_io_oe (phy_io_oe),
+				.phy_clk_o (phy_clk_o),
+				.phy_cs_o  (phy_cs_o),
+				.clk       (clk_1x)
 			);
 
 		else if (PHY_SPEED == 2)
 			qpi_phy_ice40_2x #(
-				.N_CS(2),
-				.WITH_CLK(1),
+				.N_CS     (2),
+				.WITH_CLK (1),
 			) phy_I (
-				.pad_io(spi_io),
-				.pad_clk(spi_sck),
-				.pad_cs_n(spi_cs_n),
-				.phy_io_i(phy_io_i),
-				.phy_io_o(phy_io_o),
-				.phy_io_oe(phy_io_oe),
-				.phy_clk_o(phy_clk_o),
-				.phy_cs_o(phy_cs_o),
-				.clk_1x(clk_1x),
-				.clk_2x(clk_2x)
+				.pad_io    (spi_io),
+				.pad_clk   (spi_sck),
+				.pad_cs_n  (spi_cs_n),
+				.phy_io_i  (phy_io_i),
+				.phy_io_o  (phy_io_o),
+				.phy_io_oe (phy_io_oe),
+				.phy_clk_o (phy_clk_o),
+				.phy_cs_o  (phy_cs_o),
+				.clk_1x    (clk_1x),
+				.clk_2x    (clk_2x)
 			);
 
 		else if (PHY_SPEED == 4)
 			qpi_phy_ice40_4x #(
-				.N_CS(2),
-				.WITH_CLK(1),
+				.N_CS     (2),
+				.WITH_CLK (1),
 			) phy_I (
-				.pad_io(spi_io),
-				.pad_clk(spi_sck),
-				.pad_cs_n(spi_cs_n),
-				.phy_io_i(phy_io_i),
-				.phy_io_o(phy_io_o),
-				.phy_io_oe(phy_io_oe),
-				.phy_clk_o(phy_clk_o),
-				.phy_cs_o(phy_cs_o),
-				.clk_1x(clk_1x),
-				.clk_4x(clk_4x),
-				.clk_sync(sync_4x)
+				.pad_io    (spi_io),
+				.pad_clk   (spi_sck),
+				.pad_cs_n  (spi_cs_n),
+				.phy_io_i  (phy_io_i),
+				.phy_io_o  (phy_io_o),
+				.phy_io_oe (phy_io_oe),
+				.phy_clk_o (phy_clk_o),
+				.phy_cs_o  (phy_cs_o),
+				.clk_1x    (clk_1x),
+				.clk_4x    (clk_4x),
+				.clk_sync  (sync_4x)
 			);
 	endgenerate
 
@@ -282,65 +282,65 @@ module top (
 
 	// Controller
 	hbus_memctrl hram_ctrl_I (
-		.phy_ck_en(phy_ck_en),
-		.phy_rwds_in(phy_rwds_in),
-		.phy_rwds_out(phy_rwds_out),
-		.phy_rwds_oe(phy_rwds_oe),
-		.phy_dq_in(phy_dq_in),
-		.phy_dq_out(phy_dq_out),
-		.phy_dq_oe(phy_dq_oe),
-		.phy_cs_n(phy_cs_n),
-		.phy_rst_n(phy_rst_n),
-		.phy_cfg_wdata(phy_cfg_wdata),
-		.phy_cfg_rdata(phy_cfg_rdata),
-		.phy_cfg_stb(phy_cfg_stb),
-		.mi_addr_cs(mi_addr[31:30]),
-		.mi_addr({1'b0, mi_addr[29:0], 1'b0}),	/* 32b aligned */
-		.mi_len(mi_len),
-		.mi_rw(mi_rw),
-		.mi_linear(1'b0),
-		.mi_valid(mi_valid),
-		.mi_ready(mi_ready),
-		.mi_wdata(mi_wdata),
-		.mi_wmsk(4'h0),
-		.mi_wack(mi_wack),
-		.mi_rdata(mi_rdata),
-		.mi_rstb(mi_rstb),
-		.wb_wdata(wb_wdata),
-		.wb_rdata(wb_rdata[31:0]),
-		.wb_addr(wb_addr[3:0]),
-		.wb_we(wb_we),
-		.wb_cyc(wb_cyc[0]),
-		.wb_ack(wb_ack[0]),
-		.clk(clk_1x),
-		.rst(rst)
+		.phy_ck_en     (phy_ck_en),
+		.phy_rwds_in   (phy_rwds_in),
+		.phy_rwds_out  (phy_rwds_out),
+		.phy_rwds_oe   (phy_rwds_oe),
+		.phy_dq_in     (phy_dq_in),
+		.phy_dq_out    (phy_dq_out),
+		.phy_dq_oe     (phy_dq_oe),
+		.phy_cs_n      (phy_cs_n),
+		.phy_rst_n     (phy_rst_n),
+		.phy_cfg_wdata (phy_cfg_wdata),
+		.phy_cfg_rdata (phy_cfg_rdata),
+		.phy_cfg_stb   (phy_cfg_stb),
+		.mi_addr_cs    (mi_addr[31:30]),
+		.mi_addr       ({1'b0, mi_addr[29:0], 1'b0}),	/* 32b aligned */
+		.mi_len        (mi_len),
+		.mi_rw         (mi_rw),
+		.mi_linear     (1'b0),
+		.mi_valid      (mi_valid),
+		.mi_ready      (mi_ready),
+		.mi_wdata      (mi_wdata),
+		.mi_wmsk       (4'h0),
+		.mi_wack       (mi_wack),
+		.mi_rdata      (mi_rdata),
+		.mi_rstb       (mi_rstb),
+		.wb_wdata      (wb_wdata),
+		.wb_rdata      (wb_rdata[31:0]),
+		.wb_addr       (wb_addr[3:0]),
+		.wb_we         (wb_we),
+		.wb_cyc        (wb_cyc[0]),
+		.wb_ack        (wb_ack[0]),
+		.clk           (clk_1x),
+		.rst           (rst)
 	);
 
 	// PHY
 	hbus_phy_ice40 hram_phy_I (
-		.hbus_dq(hram_dq),
-		.hbus_rwds(hram_rwds),
-		.hbus_ck(hram_ck),
-		.hbus_cs_n(hram_cs_n),
-		.hbus_rst_n(hram_rst_n),
-		.phy_ck_en(phy_ck_en),
-		.phy_rwds_in(phy_rwds_in),
-		.phy_rwds_out(phy_rwds_out),
-		.phy_rwds_oe(phy_rwds_oe),
-		.phy_dq_in(phy_dq_in),
-		.phy_dq_out(phy_dq_out),
-		.phy_dq_oe(phy_dq_oe),
-		.phy_cs_n(phy_cs_n),
-		.phy_rst_n(phy_rst_n),
-		.phy_cfg_wdata(phy_cfg_wdata),
-		.phy_cfg_rdata(phy_cfg_rdata),
-		.phy_cfg_stb(phy_cfg_stb),
-		.clk_rd_delay(clk_rd_delay),
-		.clk_1x(clk_1x),
-		.clk_4x(clk_4x),
-		.clk_rd(clk_rd),
-		.sync_4x(sync_4x),
-		.sync_rd(sync_rd)
+		.hbus_dq       (hram_dq),
+		.hbus_rwds     (hram_rwds),
+		.hbus_ck       (hram_ck),
+		.hbus_cs_n     (hram_cs_n),
+		.hbus_rst_n    (hram_rst_n),
+		.phy_ck_en     (phy_ck_en),
+		.phy_rwds_in   (phy_rwds_in),
+		.phy_rwds_out  (phy_rwds_out),
+		.phy_rwds_oe   (phy_rwds_oe),
+		.phy_dq_in     (phy_dq_in),
+		.phy_dq_out    (phy_dq_out),
+		.phy_dq_oe     (phy_dq_oe),
+		.phy_cs_n      (phy_cs_n),
+		.phy_rst_n     (phy_rst_n),
+		.phy_cfg_wdata (phy_cfg_wdata),
+		.phy_cfg_rdata (phy_cfg_rdata),
+		.phy_cfg_stb   (phy_cfg_stb),
+		.clk_rd_delay  (clk_rd_delay),
+		.clk_1x        (clk_1x),
+		.clk_4x        (clk_4x),
+		.clk_rd        (clk_rd),
+		.sync_4x       (sync_4x),
+		.sync_rd       (sync_rd)
 	);
 `endif
 
@@ -351,23 +351,23 @@ module top (
 	memtest #(
 		.ADDR_WIDTH(32)
 	) memtest_I (
-		.mi_addr(mi0_addr),
-		.mi_len(mi0_len),
-		.mi_rw(mi0_rw),
-		.mi_valid(mi0_valid),
-		.mi_ready(mi0_ready),
-		.mi_wdata(mi0_wdata),
-		.mi_wack(mi0_wack),
-		.mi_rdata(mi0_rdata),
-		.mi_rstb(mi0_rstb),
-		.wb_wdata(wb_wdata),
-		.wb_rdata(wb_rdata[63:32]),
-		.wb_addr(wb_addr[8:0]),
-		.wb_we(wb_we),
-		.wb_cyc(wb_cyc[1]),
-		.wb_ack(wb_ack[1]),
-		.clk(clk_1x),
-		.rst(rst)
+		.mi_addr  (mi0_addr),
+		.mi_len   (mi0_len),
+		.mi_rw    (mi0_rw),
+		.mi_valid (mi0_valid),
+		.mi_ready (mi0_ready),
+		.mi_wdata (mi0_wdata),
+		.mi_wack  (mi0_wack),
+		.mi_rdata (mi0_rdata),
+		.mi_rstb  (mi0_rstb),
+		.wb_wdata (wb_wdata),
+		.wb_rdata (wb_rdata[63:32]),
+		.wb_addr  (wb_addr[8:0]),
+		.wb_we    (wb_we),
+		.wb_cyc   (wb_cyc[1]),
+		.wb_ack   (wb_ack[1]),
+		.clk      (clk_1x),
+		.rst      (rst)
 	);
 
 
@@ -407,30 +407,30 @@ module top (
 		.DW(12)
 `endif
 	) hdmi_I (
-		.hdmi_data(hdmi_data),
-		.hdmi_hsync(hdmi_hsync),
-		.hdmi_vsync(hdmi_vsync),
-		.hdmi_de(hdmi_de),
-		.hdmi_clk(hdmi_clk),
-		.wb_wdata(wb_wdata),
-		.wb_rdata(wb_rdata[95:64]),
-		.wb_addr(wb_addr[6:0]),
-		.wb_we(wb_we),
-		.wb_cyc(wb_cyc[2]),
-		.wb_ack(wb_ack[2]),
-		.mi_addr(mi1_addr),
-		.mi_len(mi1_len),
-		.mi_rw(mi1_rw),
-		.mi_valid(mi1_valid),
-		.mi_ready(mi1_ready),
-		.mi_wdata(mi1_wdata),
-		.mi_wack(mi1_wack),
-		.mi_rdata(mi1_rdata),
-		.mi_rstb(mi1_rstb),
-		.clk_1x(clk_1x),
-		.clk_4x(clk_4x),
-		.sync_4x(sync_4x),
-		.rst(rst)
+		.hdmi_data  (hdmi_data),
+		.hdmi_hsync (hdmi_hsync),
+		.hdmi_vsync (hdmi_vsync),
+		.hdmi_de    (hdmi_de),
+		.hdmi_clk   (hdmi_clk),
+		.wb_wdata   (wb_wdata),
+		.wb_rdata   (wb_rdata[95:64]),
+		.wb_addr    (wb_addr[6:0]),
+		.wb_we      (wb_we),
+		.wb_cyc     (wb_cyc[2]),
+		.wb_ack     (wb_ack[2]),
+		.mi_addr    (mi1_addr),
+		.mi_len     (mi1_len),
+		.mi_rw      (mi1_rw),
+		.mi_valid   (mi1_valid),
+		.mi_ready   (mi1_ready),
+		.mi_wdata   (mi1_wdata),
+		.mi_wack    (mi1_wack),
+		.mi_rdata   (mi1_rdata),
+		.mi_rstb    (mi1_rstb),
+		.clk_1x     (clk_1x),
+		.clk_4x     (clk_4x),
+		.sync_4x    (sync_4x),
+		.rst        (rst)
 	);
 `else
 	// Dummy wishbone
@@ -450,15 +450,15 @@ module top (
 	// -------------
 
 	sysmgr sysmgr_I (
-		.delay(clk_rd_delay),
-		.clk_in(clk_in),
-		.clk_1x(clk_1x),
-		.clk_2x(clk_2x),
-		.clk_4x(clk_4x),
-		.clk_rd(clk_rd),
-		.sync_4x(sync_4x),
-		.sync_rd(sync_rd),
-		.rst(rst)
+		.delay   (clk_rd_delay),
+		.clk_in  (clk_in),
+		.clk_1x  (clk_1x),
+		.clk_2x  (clk_2x),
+		.clk_4x  (clk_4x),
+		.clk_rd  (clk_rd),
+		.sync_4x (sync_4x),
+		.sync_rd (sync_rd),
+		.rst     (rst)
 	);
 
 endmodule