Collective PR to combine tests increasing coverage #944
verible-format.yml
on: pull_request_target
format-review
45s
Annotations
10 warnings
format-review:
design/dbg/el2_dbg.sv#L27
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/dbg/el2_dbg.sv:27:-`include "el2_param.vh"
design/dbg/el2_dbg.sv:28:- )(
design/dbg/el2_dbg.sv:29:- // outputs to the core for command and data interface
design/dbg/el2_dbg.sv:30:- output logic [31:0] dbg_cmd_addr,
design/dbg/el2_dbg.sv:31:- output logic [31:0] dbg_cmd_wrdata,
design/dbg/el2_dbg.sv:32:- output logic dbg_cmd_valid,
design/dbg/el2_dbg.sv:33:- output logic dbg_cmd_write, // 1: write command, 0: read_command
design/dbg/el2_dbg.sv:34:- output logic [1:0] dbg_cmd_type, // 0:gpr 1:csr 2: memory
design/dbg/el2_dbg.sv:35:- output logic [1:0] dbg_cmd_size, // size of the abstract mem access debug command
design/dbg/el2_dbg.sv:36:- output logic dbg_core_rst_l, // core reset from dm
design/dbg/el2_dbg.sv:37:-
design/dbg/el2_dbg.sv:38:- // inputs back from the core/dec
design/dbg/el2_dbg.sv:39:- input logic [31:0] core_dbg_rddata,
design/dbg/el2_dbg.sv:40:- input logic core_dbg_cmd_done, // This will be treated like a valid signal
design/dbg/el2_dbg.sv:41:- input logic core_dbg_cmd_fail, // Exception during command run
design/dbg/el2_dbg.sv:42:-
design/dbg/el2_dbg.sv:43:- // Signals to dma to get a bubble
design/dbg/el2_dbg.sv:44:- output logic dbg_dma_bubble, // Debug needs a bubble to send a valid
design/dbg/el2_dbg.sv:45:- input logic dma_dbg_ready, // DMA is ready to accept debug request
design/dbg/el2_dbg.sv:46:-
design/dbg/el2_dbg.sv:47:- // interface with the rest of the core to halt/resume handshaking
design/dbg/el2_dbg.sv:48:- output logic dbg_halt_req, // This is a pulse
design/dbg/el2_dbg.sv:49:- output logic dbg_resume_req, // Debug sends a resume requests. Pulse
design/dbg/el2_dbg.sv:50:- input logic dec_tlu_debug_mode, // Core is in debug mode
design/dbg/el2_dbg.sv:51:- input logic dec_tlu_dbg_halted, // The core has finished the queiscing sequence. Core is halted now
design/dbg/el2_dbg.sv:52:- input logic dec_tlu_mpc_halted_only, // Only halted due to MPC
design/dbg/el2_dbg.sv:53:- input logic dec_tlu_resume_ack, // core sends back an ack for the resume (pulse)
design/dbg/el2_dbg.sv:54:-
design/dbg/el2_dbg.sv:55:- // inputs from the JTAG
design/dbg/el2_dbg.sv:56:- input logic dmi_reg_en, // read or write
design/dbg/el2_dbg.sv:57:- input logic [6:0] dmi_reg_addr, // address of DM register
design/dbg/el2_dbg.sv:58:- input logic dmi_reg_wr_en, // write instruction
design/dbg/el2_dbg.sv:59:- input logic [31:0] dmi_reg_wdata, // write data
design/dbg/el2_dbg.sv:60:-
design/dbg/el2_dbg.sv:61:- // output
design/dbg/el2_dbg.sv:62:- output logic [31:0] dmi_reg_rdata, // read data
design/dbg/el2_dbg.sv:63:-
design/dbg/el2_dbg.sv:64:- // AXI Write Channels
design/dbg/el2_dbg.sv:65:- output logic sb_axi_awvalid,
design/dbg/el2_dbg.sv:66:- input logic sb_axi_awready,
design/dbg/el2_dbg.sv:67:- /* exclude signals that are tied to constant value in this file */
design/dbg/el2_dbg.sv:68:- /*verilator coverage_off*/
design/dbg/el2_dbg.sv:69:- output logic [pt.SB_BUS_TAG-1:0] sb_axi_awid,
design/dbg/el2_dbg.sv:70:- /*verilator coverage_on*/
design/dbg/el2_dbg.sv:71:- output logic [31:0] sb_axi_awaddr,
design/dbg/el2_dbg.sv:72:- output logic [3:0] sb_axi_awregion,
design/dbg/el2_dbg.sv:73:- /* exclude signals that are t
|
format-review:
design/dbg/el2_dbg.sv#L508
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/dbg/el2_dbg.sv:508:- dbg_state_en = (dmstatus_reg[9] & resumereq) | execute_command | ~(dmstatus_reg[9] | dec_tlu_mpc_halted_only);
design/dbg/el2_dbg.sv:509:- abstractcs_busy_wren = dbg_state_en & ((dbg_nxtstate == CORE_CMD_START) | (dbg_nxtstate == SB_CMD_START)); // write busy when a new command was written by jtag
design/dbg/el2_dbg.sv:510:- abstractcs_busy_din = 1'b1;
design/dbg/el2_dbg.sv:511:- dbg_resume_req = dbg_state_en & (dbg_nxtstate == RESUMING); // single cycle pulse to core if resuming
design/dbg/el2_dbg.sv:512:- end
design/dbg/el2_dbg.sv:513:- CORE_CMD_START: begin
design/dbg/el2_dbg.sv:514:- // Don't execute the command if cmderror or transfer=0 for abstract register access
design/dbg/el2_dbg.sv:515:- dbg_nxtstate = ((|abstractcs_reg[10:8]) | ((command_reg[31:24] == 8'h0) & ~command_reg[17])) ? CMD_DONE : CORE_CMD_WAIT; // new command sent to the core
design/dbg/el2_dbg.sv:516:- dbg_state_en = dbg_cmd_valid | (|abstractcs_reg[10:8]) | ((command_reg[31:24] == 8'h0) & ~command_reg[17]);
design/dbg/el2_dbg.sv:517:- end
design/dbg/el2_dbg.sv:518:- CORE_CMD_WAIT: begin
design/dbg/el2_dbg.sv:519:- dbg_nxtstate = CMD_DONE;
design/dbg/el2_dbg.sv:520:- dbg_state_en = core_dbg_cmd_done; // go to done state for one cycle after completing current command
design/dbg/el2_dbg.sv:521:- end
design/dbg/el2_dbg.sv:522:- SB_CMD_START: begin
design/dbg/el2_dbg.sv:523:- dbg_nxtstate = (|abstractcs_reg[10:8]) ? CMD_DONE : SB_CMD_SEND;
design/dbg/el2_dbg.sv:524:- dbg_state_en = (dbg_bus_clk_en & ~sb_cmd_pending) | (|abstractcs_reg[10:8]);
design/dbg/el2_dbg.sv:525:- end
design/dbg/el2_dbg.sv:526:- SB_CMD_SEND: begin
design/dbg/el2_dbg.sv:527:- sb_abmem_cmd_done_in = 1'b1;
design/dbg/el2_dbg.sv:528:- sb_abmem_data_done_in= 1'b1;
design/dbg/el2_dbg.sv:529:- sb_abmem_cmd_done_en = (sb_bus_cmd_read | sb_bus_cmd_write_addr) & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:530:- sb_abmem_data_done_en= (sb_bus_cmd_read | sb_bus_cmd_write_data) & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:531:- dbg_nxtstate = SB_CMD_RESP;
design/dbg/el2_dbg.sv:532:- dbg_state_en = (sb_abmem_cmd_done | sb_abmem_cmd_done_en) & (sb_abmem_data_done | sb_abmem_data_done_en) & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:533:- end
design/dbg/el2_dbg.sv:534:- SB_CMD_RESP: begin
design/dbg/el2_dbg.sv:535:- dbg_nxtstate = CMD_DONE;
design/dbg/el2_dbg.sv:536:- dbg_state_en = (sb_bus_rsp_read | sb_bus_rsp_write) & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:537:- dbg_sb_bus_error = (sb_bus_rsp_read | sb_bus_rsp_write) & sb_bus_rsp_error & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:538:- data0_reg_wren2 = dbg_state_en & ~sb_abmem_cmd_write & ~dbg_sb_bus_error;
design/dbg/el2_dbg.sv:539:- end
design/dbg/el2_dbg.sv:540:- CMD_DONE: begin
design/dbg/el2_dbg.sv:541:- dbg_nxtstate = HALTED;
design/dbg/el2_dbg.sv:542:- dbg_state_en = 1'b1;
design/dbg/el2_dbg.sv:543:- abstractcs_busy_wren = dbg_state_en; // remove the busy bit from the abstracts ( bit 12 )
design/dbg/el2_dbg.sv:544:- abstractcs_busy_din = 1'b0;
design/dbg/el2_dbg.sv:545:- sb_abmem_cmd_done_in = 1'b0;
design/dbg/el2_dbg.sv:546:- sb_abmem_data_done_in= 1'b0;
design/dbg/el2_dbg.sv:547:- sb_abmem
|
format-review:
design/dbg/el2_dbg.sv#L620
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/dbg/el2_dbg.sv:620:- sb_nxtstate = SBIDLE;
design/dbg/el2_dbg.sv:621:- sb_state_en = 1'b0;
design/dbg/el2_dbg.sv:622:- sbcs_sbbusy_wren = 1'b0;
design/dbg/el2_dbg.sv:623:- sbcs_sbbusy_din = 1'b0;
design/dbg/el2_dbg.sv:624:- sbcs_sberror_wren = 1'b0;
design/dbg/el2_dbg.sv:625:- sbcs_sberror_din[2:0] = 3'b0;
design/dbg/el2_dbg.sv:626:- sbaddress0_reg_wren1 = 1'b0;
design/dbg/el2_dbg.sv:627:- case (sb_state)
design/dbg/el2_dbg.sv:628:- SBIDLE: begin
design/dbg/el2_dbg.sv:629:- sb_nxtstate = sbdata0wr_access ? WAIT_WR : WAIT_RD;
design/dbg/el2_dbg.sv:630:- sb_state_en = (sbdata0wr_access | sbreadondata_access | sbreadonaddr_access) & ~(|sbcs_reg[14:12]) & ~sbcs_reg[22];
design/dbg/el2_dbg.sv:631:- sbcs_sbbusy_wren = sb_state_en; // set the single read bit if it is a singlread command
design/dbg/el2_dbg.sv:632:- sbcs_sbbusy_din = 1'b1;
design/dbg/el2_dbg.sv:633:- sbcs_sberror_wren = sbcs_wren & (|dmi_reg_wdata[14:12]); // write to clear the error bits
design/dbg/el2_dbg.sv:634:- sbcs_sberror_din[2:0] = ~dmi_reg_wdata[14:12] & sbcs_reg[14:12];
design/dbg/el2_dbg.sv:635:- end
design/dbg/el2_dbg.sv:636:- WAIT_RD: begin
design/dbg/el2_dbg.sv:637:- sb_nxtstate = (sbcs_unaligned | sbcs_illegal_size) ? DONE : CMD_RD;
design/dbg/el2_dbg.sv:638:- sb_state_en = (dbg_bus_clk_en & ~sb_abmem_cmd_pending) | sbcs_unaligned | sbcs_illegal_size;
design/dbg/el2_dbg.sv:639:- sbcs_sberror_wren = sbcs_unaligned | sbcs_illegal_size;
design/dbg/el2_dbg.sv:640:- sbcs_sberror_din[2:0] = sbcs_unaligned ? 3'b011 : 3'b100;
design/dbg/el2_dbg.sv:641:- end
design/dbg/el2_dbg.sv:642:- WAIT_WR: begin
design/dbg/el2_dbg.sv:643:- sb_nxtstate = (sbcs_unaligned | sbcs_illegal_size) ? DONE : CMD_WR;
design/dbg/el2_dbg.sv:644:- sb_state_en = (dbg_bus_clk_en & ~sb_abmem_cmd_pending) | sbcs_unaligned | sbcs_illegal_size;
design/dbg/el2_dbg.sv:645:- sbcs_sberror_wren = sbcs_unaligned | sbcs_illegal_size;
design/dbg/el2_dbg.sv:646:- sbcs_sberror_din[2:0] = sbcs_unaligned ? 3'b011 : 3'b100;
design/dbg/el2_dbg.sv:647:- end
design/dbg/el2_dbg.sv:648:- CMD_RD : begin
design/dbg/el2_dbg.sv:649:- sb_nxtstate = RSP_RD;
design/dbg/el2_dbg.sv:650:- sb_state_en = sb_bus_cmd_read & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:651:- end
design/dbg/el2_dbg.sv:652:- CMD_WR : begin
design/dbg/el2_dbg.sv:653:- sb_nxtstate = (sb_bus_cmd_write_addr & sb_bus_cmd_write_data) ? RSP_WR : (sb_bus_cmd_write_data ? CMD_WR_ADDR : CMD_WR_DATA);
design/dbg/el2_dbg.sv:654:- sb_state_en = (sb_bus_cmd_write_addr | sb_bus_cmd_write_data) & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:655:- end
design/dbg/el2_dbg.sv:656:- CMD_WR_ADDR : begin
design/dbg/el2_dbg.sv:657:- sb_nxtstate = RSP_WR;
design/dbg/el2_dbg.sv:658:- sb_state_en = sb_bus_cmd_write_addr & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:659:- end
design/dbg/el2_dbg.sv:660:- CMD_WR_DATA : begin
design/dbg/el2_dbg.sv:661:- sb_nxtstate = RSP_WR;
design/dbg/el2_dbg.sv:662:- sb_state_en = sb_bus_cmd_write_data & dbg_bus_clk_en;
design/dbg/el2_dbg.sv:663:- end
design/dbg/el2_dbg.sv:664:- RSP_RD: begin
design/dbg/el2_dbg.sv:665:- sb_nxtstate
|
format-review:
design/el2_veer.sv#L26
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer.sv:26:-`include "el2_param.vh"
design/el2_veer.sv:27:- )
design/el2_veer.sv:28:- (
design/el2_veer.sv:29:- input logic clk,
design/el2_veer.sv:30:- input logic rst_l,
design/el2_veer.sv:31:- input logic dbg_rst_l,
design/el2_veer.sv:32:- input logic [31:1] rst_vec,
design/el2_veer.sv:33:- input logic nmi_int,
design/el2_veer.sv:34:- input logic [31:1] nmi_vec,
design/el2_veer.sv:35:- output logic core_rst_l, // This is "rst_l | dbg_rst_l"
design/el2_veer.sv:36:-
design/el2_veer.sv:37:- output logic active_l2clk,
design/el2_veer.sv:38:- output logic free_l2clk,
design/el2_veer.sv:39:-
design/el2_veer.sv:40:- output logic [31:0] trace_rv_i_insn_ip,
design/el2_veer.sv:41:- output logic [31:0] trace_rv_i_address_ip,
design/el2_veer.sv:42:- output logic trace_rv_i_valid_ip,
design/el2_veer.sv:43:- output logic trace_rv_i_exception_ip,
design/el2_veer.sv:44:- output logic [4:0] trace_rv_i_ecause_ip,
design/el2_veer.sv:45:- output logic trace_rv_i_interrupt_ip,
design/el2_veer.sv:46:- output logic [31:0] trace_rv_i_tval_ip,
design/el2_veer.sv:47:-
design/el2_veer.sv:48:-
design/el2_veer.sv:49:- output logic dccm_clk_override,
design/el2_veer.sv:50:- output logic icm_clk_override,
design/el2_veer.sv:51:- output logic dec_tlu_core_ecc_disable,
design/el2_veer.sv:52:-
design/el2_veer.sv:53:- // external halt/run interface
design/el2_veer.sv:54:- input logic i_cpu_halt_req, // Asynchronous Halt request to CPU
design/el2_veer.sv:55:- input logic i_cpu_run_req, // Asynchronous Restart request to CPU
design/el2_veer.sv:56:- output logic o_cpu_halt_ack, // Core Acknowledge to Halt request
design/el2_veer.sv:57:- output logic o_cpu_halt_status, // 1'b1 indicates processor is halted
design/el2_veer.sv:58:- output logic o_cpu_run_ack, // Core Acknowledge to run request
design/el2_veer.sv:59:- output logic o_debug_mode_status, // Core to the PMU that core is in debug mode. When core is in debug mode, the PMU should refrain from sendng a halt or run request
design/el2_veer.sv:60:-
design/el2_veer.sv:61:- input logic [31:4] core_id, // CORE ID
design/el2_veer.sv:62:-
design/el2_veer.sv:63:- // external MPC halt/run interface
design/el2_veer.sv:64:- input logic mpc_debug_halt_req, // Async halt request
design/el2_veer.sv:65:- input logic mpc_debug_run_req, // Async run request
design/el2_veer.sv:66:- input logic mpc_reset_run_req, // Run/halt after reset
design/el2_veer.sv:67:- output logic mpc_debug_halt_ack, // Halt ack
design/el2_veer.sv:68:- output logic mpc_debug_run_ack, // Run ack
design/el2_veer.sv:69:- output logic debug_brkpt_status, // debug breakpoint
design/el2_veer.sv:70:-
design/el2_veer.sv:71:- output logic dec_tlu_perfcnt0, // toggles when slot0 perf counter 0 has an event inc
design/el2_veer.sv:72:- output logic dec_tlu_perfcnt1,
design/el2_veer.sv:73:- output logic dec_tlu_perfcnt2,
design/el2_veer.sv:74:- output logic dec_tlu_perfcnt3,
design/el2_veer.sv:75:-
design/el2_veer.sv:76:- // DCCM ports
design/el2_veer.sv:77:- output logic dccm_wren,
design/el2_veer.sv:78:- output logic dccm_rden,
design/el2_veer.sv:79:- output logic [pt.DCCM_BITS-1:0] dccm_wr_addr_lo,
design/el2_veer.sv:80:- output logic [pt.DCCM_BITS-1:0] dccm_wr_addr_hi,
design/el2_veer.sv:81:- output logic [pt.DCCM_BITS-1:0] dccm_rd_addr_lo,
design/el2_veer.sv:82:- output logic [pt.DCCM_BITS-1:0] dccm_rd_addr_hi,
design/el2_veer.sv:83:- output logic [pt.DCCM_FDATA_WIDTH-1:0] dccm_wr_data_lo,
design/el2_veer.sv:84:- output logic [pt.DCCM_FDATA_WIDTH-1:0] dccm_wr_data_hi,
design/el2_veer.sv:85:-
design/el2_veer.sv:86:- input logic [pt.DCCM_FDATA_WIDTH-1:0] dccm_rd_data_lo,
design/el2_veer.sv
|
format-review:
design/el2_veer.sv#L467
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer.sv:467:- logic [63:0] hwdata_nc;
design/el2_veer.sv:468:- //----------------------------------------------------------------------
design/el2_veer.sv:469:- //
design/el2_veer.sv:470:- //----------------------------------------------------------------------
design/el2_veer.sv:471:-
design/el2_veer.sv:472:- logic ifu_pmu_instr_aligned;
design/el2_veer.sv:473:- logic ifu_ic_error_start;
design/el2_veer.sv:474:- logic ifu_iccm_dma_rd_ecc_single_err;
design/el2_veer.sv:475:- logic ifu_iccm_rd_ecc_single_err;
design/el2_veer.sv:476:- logic ifu_iccm_rd_ecc_double_err;
design/el2_veer.sv:477:- logic lsu_dccm_rd_ecc_single_err;
design/el2_veer.sv:478:- logic lsu_dccm_rd_ecc_double_err;
design/el2_veer.sv:479:-
design/el2_veer.sv:480:- logic lsu_axi_awready_ahb;
design/el2_veer.sv:481:- logic lsu_axi_wready_ahb;
design/el2_veer.sv:482:- logic lsu_axi_bvalid_ahb;
design/el2_veer.sv:483:- logic lsu_axi_bready_ahb;
design/el2_veer.sv:484:- logic [1:0] lsu_axi_bresp_ahb;
design/el2_veer.sv:485:- logic [pt.LSU_BUS_TAG-1:0] lsu_axi_bid_ahb;
design/el2_veer.sv:486:- logic lsu_axi_arready_ahb;
design/el2_veer.sv:487:- logic lsu_axi_rvalid_ahb;
design/el2_veer.sv:488:- logic [pt.LSU_BUS_TAG-1:0] lsu_axi_rid_ahb;
design/el2_veer.sv:489:- logic [63:0] lsu_axi_rdata_ahb;
design/el2_veer.sv:490:- logic [1:0] lsu_axi_rresp_ahb;
design/el2_veer.sv:491:- logic lsu_axi_rlast_ahb;
design/el2_veer.sv:492:-
design/el2_veer.sv:493:- logic lsu_axi_awready_int;
design/el2_veer.sv:494:- logic lsu_axi_wready_int;
design/el2_veer.sv:495:- logic lsu_axi_bvalid_int;
design/el2_veer.sv:496:- logic lsu_axi_bready_int;
design/el2_veer.sv:497:- logic [1:0] lsu_axi_bresp_int;
design/el2_veer.sv:498:- logic [pt.LSU_BUS_TAG-1:0] lsu_axi_bid_int;
design/el2_veer.sv:499:- logic lsu_axi_arready_int;
design/el2_veer.sv:500:- logic lsu_axi_rvalid_int;
design/el2_veer.sv:501:- logic [pt.LSU_BUS_TAG-1:0] lsu_axi_rid_int;
design/el2_veer.sv:502:- logic [63:0] lsu_axi_rdata_int;
design/el2_veer.sv:503:- logic [1:0] lsu_axi_rresp_int;
design/el2_veer.sv:504:- logic lsu_axi_rlast_int;
design/el2_veer.sv:505:-
design/el2_veer.sv:506:- logic ifu_axi_awready_ahb;
design/el2_veer.sv:507:- logic ifu_axi_wready_ahb;
design/el2_veer.sv:508:- logic ifu_axi_bvalid_ahb;
design/el2_veer.sv:509:- logic ifu_axi_bready_ahb;
design/el2_veer.sv:510:- logic [1:0] ifu_axi_bresp_ahb;
design/el2_veer.sv:511:- logic [pt.IFU_BUS_TAG-1:0] ifu_axi_bid_ahb;
design/el2_veer.sv:512:- logic ifu_axi_arready_ahb;
design/el2_veer.sv:513:- logic ifu_axi_rvalid_ahb;
design/el2_veer.sv:514:- logic [pt.IFU_BUS_TAG-1:0] ifu_axi_rid_ahb;
design/el2_veer.sv:515:- logic [63:0] ifu_axi_rdata_ahb;
design/el2_veer.sv:516:- logic [1:0] ifu_axi_rresp_ahb;
design/el2_veer.sv:517:- logic ifu_axi_rlast_ahb;
design/el2_veer.sv:518:-
design/el2_veer.sv:519:- logic ifu_axi_awready_int;
design/el2_veer.sv:520:- logic ifu_axi_wready_int;
design/el2_veer.sv:521:- logic ifu_axi_bvalid_int;
design/el2_veer.sv:522:- logic
|
format-review:
design/el2_veer_wrapper.sv#L49
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer_wrapper.sv:49:- //-------------------------- LSU AXI signals--------------------------
design/el2_veer_wrapper.sv:50:- // AXI Write Channels
design/el2_veer_wrapper.sv:51:- output logic lsu_axi_awvalid,
design/el2_veer_wrapper.sv:52:- input logic lsu_axi_awready,
design/el2_veer_wrapper.sv:53:- output logic [pt.LSU_BUS_TAG-1:0] lsu_axi_awid,
design/el2_veer_wrapper.sv:54:- output logic [31:0] lsu_axi_awaddr,
design/el2_veer_wrapper.sv:55:- output logic [3:0] lsu_axi_awregion,
design/el2_veer_wrapper.sv:56:- /* exclude signals that are tied to constant value in el2_lsu_bus_buffer.sv */
design/el2_veer_wrapper.sv:57:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:58:- output logic [7:0] lsu_axi_awlen,
design/el2_veer_wrapper.sv:59:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:60:- output logic [2:0] lsu_axi_awsize,
design/el2_veer_wrapper.sv:61:- /* exclude signals that are tied to constant value in el2_lsu_bus_buffer.sv */
design/el2_veer_wrapper.sv:62:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:63:- output logic [1:0] lsu_axi_awburst,
design/el2_veer_wrapper.sv:64:- output logic lsu_axi_awlock,
design/el2_veer_wrapper.sv:65:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:66:- output logic [3:0] lsu_axi_awcache,
design/el2_veer_wrapper.sv:67:- /* exclude signals that are tied to constant value in el2_lsu_bus_buffer.sv */
design/el2_veer_wrapper.sv:68:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:69:- output logic [2:0] lsu_axi_awprot,
design/el2_veer_wrapper.sv:70:- output logic [3:0] lsu_axi_awqos,
design/el2_veer_wrapper.sv:71:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:72:-
design/el2_veer_wrapper.sv:73:- output logic lsu_axi_wvalid,
design/el2_veer_wrapper.sv:74:- input logic lsu_axi_wready,
design/el2_veer_wrapper.sv:75:- output logic [63:0] lsu_axi_wdata,
design/el2_veer_wrapper.sv:76:- output logic [7:0] lsu_axi_wstrb,
design/el2_veer_wrapper.sv:77:- output logic lsu_axi_wlast,
design/el2_veer_wrapper.sv:78:-
design/el2_veer_wrapper.sv:79:- input logic lsu_axi_bvalid,
design/el2_veer_wrapper.sv:80:- /* exclude signals that are tied to constant value in el2_lsu_bus_buffer.sv */
design/el2_veer_wrapper.sv:81:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:82:- output logic lsu_axi_bready,
design/el2_veer_wrapper.sv:83:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:84:- input logic [1:0] lsu_axi_bresp,
design/el2_veer_wrapper.sv:85:- input logic [pt.LSU_BUS_TAG-1:0] lsu_axi_bid,
design/el2_veer_wrapper.sv:86:-
design/el2_veer_wrapper.sv:87:- // AXI Read Channels
design/el2_veer_wrapper.sv:88:- output logic lsu_axi_arvalid,
design/el2_veer_wrapper.sv:89:- input logic lsu_axi_arready,
design/el2_veer_wrapper.sv:90:- output logic [pt.LSU_BUS_TAG-1:0] lsu_axi_arid,
design/el2_veer_wrapper.sv:91:- output logic [31:0] lsu_axi_araddr,
design/el2_veer_wrapper.sv:92:- output logic [3:0] lsu_axi_arregion,
design/el2_veer_wrapper.sv:93:- /* exclude signals that are tied to constant value in el2_lsu_bus_buffer.sv */
design/el2_veer_wrapper.sv:94:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:95:- output logic [7:0] lsu_axi_arlen,
design/el2_veer_wrapper.sv:96:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:97:- output logic [2:0] lsu_axi_arsize,
design/el2_veer_wrapper.sv:98:-
|
format-review:
design/el2_veer_wrapper.sv#L301
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer_wrapper.sv:301:- //// AHB LITE BUS
design/el2_veer_wrapper.sv:302:- output logic [31:0] haddr,
design/el2_veer_wrapper.sv:303:- /* exclude signals that are tied to constant value in axi4_to_ahb.sv */
design/el2_veer_wrapper.sv:304:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:305:- output logic [2:0] hburst,
design/el2_veer_wrapper.sv:306:- output logic hmastlock,
design/el2_veer_wrapper.sv:307:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:308:- output logic [3:0] hprot,
design/el2_veer_wrapper.sv:309:- output logic [2:0] hsize,
design/el2_veer_wrapper.sv:310:- output logic [1:0] htrans,
design/el2_veer_wrapper.sv:311:- output logic hwrite,
design/el2_veer_wrapper.sv:312:-
design/el2_veer_wrapper.sv:313:- /* exclude signals that are tied to constant value in this file */
design/el2_veer_wrapper.sv:314:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:315:- input logic [63:0] hrdata,
design/el2_veer_wrapper.sv:316:- input logic hready,
design/el2_veer_wrapper.sv:317:- input logic hresp,
design/el2_veer_wrapper.sv:318:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:319:-
design/el2_veer_wrapper.sv:320:- // LSU AHB Master
design/el2_veer_wrapper.sv:321:- output logic [31:0] lsu_haddr,
design/el2_veer_wrapper.sv:322:- /* exclude signals that are tied to constant value in axi4_to_ahb.sv */
design/el2_veer_wrapper.sv:323:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:324:- output logic [2:0] lsu_hburst,
design/el2_veer_wrapper.sv:325:- output logic lsu_hmastlock,
design/el2_veer_wrapper.sv:326:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:327:- output logic [3:0] lsu_hprot,
design/el2_veer_wrapper.sv:328:- output logic [2:0] lsu_hsize,
design/el2_veer_wrapper.sv:329:- output logic [1:0] lsu_htrans,
design/el2_veer_wrapper.sv:330:- output logic lsu_hwrite,
design/el2_veer_wrapper.sv:331:- output logic [63:0] lsu_hwdata,
design/el2_veer_wrapper.sv:332:-
design/el2_veer_wrapper.sv:333:- /* exclude signals that are tied to constant value in this file */
design/el2_veer_wrapper.sv:334:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:335:- input logic [63:0] lsu_hrdata,
design/el2_veer_wrapper.sv:336:- input logic lsu_hready,
design/el2_veer_wrapper.sv:337:- input logic lsu_hresp,
design/el2_veer_wrapper.sv:338:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:339:- // Debug Syster Bus AHB
design/el2_veer_wrapper.sv:340:- output logic [31:0] sb_haddr,
design/el2_veer_wrapper.sv:341:- /* exclude signals that are tied to constant value in axi4_to_ahb.sv */
design/el2_veer_wrapper.sv:342:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:343:- output logic [2:0] sb_hburst,
design/el2_veer_wrapper.sv:344:- output logic sb_hmastlock,
design/el2_veer_wrapper.sv:345:- /*verilator coverage_on*/
design/el2_veer_wrapper.sv:346:- output logic [3:0] sb_hprot,
design/el2_veer_wrapper.sv:347:- output logic [2:0] sb_hsize,
design/el2_veer_wrapper.sv:348:- output logic [1:0] sb_htrans,
design/el2_veer_wrapper.sv:349:- output logic sb_hwrite,
design/el2_veer_wrapper.sv:350:- output logic [63:0] sb_hwdata,
design/el2_veer_wrapper.sv:351:-
design/el2_veer_wrapper.sv:352:- /* exclude signals that are tied to constant value in this file */
design/el2_veer_wrappe
|
format-review:
design/el2_veer_wrapper.sv#L378
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer_wrapper.sv:378:- // clk ratio signals
design/el2_veer_wrapper.sv:379:- input logic lsu_bus_clk_en, // Clock ratio b/w cpu core clk & AHB master interface
design/el2_veer_wrapper.sv:380:- input logic ifu_bus_clk_en, // Clock ratio b/w cpu core clk & AHB master interface
design/el2_veer_wrapper.sv:381:- input logic dbg_bus_clk_en, // Clock ratio b/w cpu core clk & AHB master interface
design/el2_veer_wrapper.sv:382:- input logic dma_bus_clk_en, // Clock ratio b/w cpu core clk & AHB slave interface
design/el2_veer_wrapper.sv:383:-
design/el2_veer_wrapper.sv:384:- // ICCM/DCCM ECC status
design/el2_veer_wrapper.sv:385:- output logic iccm_ecc_single_error,
design/el2_veer_wrapper.sv:386:- output logic iccm_ecc_double_error,
design/el2_veer_wrapper.sv:387:- output logic dccm_ecc_single_error,
design/el2_veer_wrapper.sv:388:- output logic dccm_ecc_double_error,
design/el2_veer_wrapper.sv:389:-
design/el2_veer_wrapper.sv:390:- // all of these test inputs are brought to top-level; must be tied off based on usage by physical design (ie. icache or not, iccm or not, dccm or not)
design/el2_veer_wrapper.sv:391:-
design/el2_veer_wrapper.sv:392:- input el2_ic_data_ext_in_pkt_t [pt.ICACHE_NUM_WAYS-1:0][pt.ICACHE_BANKS_WAY-1:0] ic_data_ext_in_pkt,
design/el2_veer_wrapper.sv:393:- input el2_ic_tag_ext_in_pkt_t [pt.ICACHE_NUM_WAYS-1:0] ic_tag_ext_in_pkt,
design/el2_veer_wrapper.sv:394:-
design/el2_veer_wrapper.sv:395:- input logic timer_int,
design/el2_veer_wrapper.sv:396:- input logic soft_int,
design/el2_veer_wrapper.sv:397:- input logic [pt.PIC_TOTAL_INT:1] extintsrc_req,
design/el2_veer_wrapper.sv:398:-
design/el2_veer_wrapper.sv:399:- output logic dec_tlu_perfcnt0, // toggles when slot0 perf counter 0 has an event inc
design/el2_veer_wrapper.sv:400:- output logic dec_tlu_perfcnt1,
design/el2_veer_wrapper.sv:401:- output logic dec_tlu_perfcnt2,
design/el2_veer_wrapper.sv:402:- output logic dec_tlu_perfcnt3,
design/el2_veer_wrapper.sv:403:-
design/el2_veer_wrapper.sv:404:- // ports added by the soc team
design/el2_veer_wrapper.sv:405:- input logic jtag_tck, // JTAG clk
design/el2_veer_wrapper.sv:406:- input logic jtag_tms, // JTAG TMS
design/el2_veer_wrapper.sv:407:- input logic jtag_tdi, // JTAG tdi
design/el2_veer_wrapper.sv:408:- input logic jtag_trst_n, // JTAG Reset
design/el2_veer_wrapper.sv:409:- output logic jtag_tdo, // JTAG TDO
design/el2_veer_wrapper.sv:410:- output logic jtag_tdoEn, // JTAG Test Data Output enable
design/el2_veer_wrapper.sv:411:-
design/el2_veer_wrapper.sv:412:- input logic [31:4] core_id,
design/el2_veer_wrapper.sv:413:-
design/el2_veer_wrapper.sv:414:- // Memory Export Interface
design/el2_veer_wrapper.sv:415:- el2_mem_if.veer_sram_src el2_mem_export,
design/el2_veer_wrapper.sv:416:-
design/el2_veer_wrapper.sv:417:- // external MPC halt/run interface
design/el2_veer_wrapper.sv:418:- input logic mpc_debug_halt_req, // Async halt request
design/el2_veer_wrapper.sv:419:- input logic mpc_debug_run_req, // Async run request
design/el2_veer_wrapper.sv:420:- input logic mpc_reset_run_req, // Run/halt after reset
design/el2_veer_wrapper.sv:421:- output logic mpc_debug_halt_ack, // Halt ack
design/el2_veer_wrapper.sv
|
format-review:
design/el2_veer_wrapper.sv#L447
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer_wrapper.sv:447:- logic active_l2clk;
design/el2_veer_wrapper.sv:448:- logic free_l2clk;
design/el2_veer_wrapper.sv:446:+ logic active_l2clk;
design/el2_veer_wrapper.sv:447:+ logic free_l2clk;
|
format-review:
design/el2_veer_wrapper.sv#L515
[verible-verilog-format] reported by reviewdog 🐶
Raw Output:
design/el2_veer_wrapper.sv:515:- // Since all the signals in this block are tied to constant, we exclude this from coverage analysis
design/el2_veer_wrapper.sv:516:- /*verilator coverage_off*/
design/el2_veer_wrapper.sv:517:-
design/el2_veer_wrapper.sv:518:- //// AHB LITE BUS
design/el2_veer_wrapper.sv:519:- logic [31:0] haddr;
design/el2_veer_wrapper.sv:520:- logic [2:0] hburst;
design/el2_veer_wrapper.sv:521:- logic hmastlock;
design/el2_veer_wrapper.sv:522:- logic [3:0] hprot;
design/el2_veer_wrapper.sv:523:- logic [2:0] hsize;
design/el2_veer_wrapper.sv:524:- logic [1:0] htrans;
design/el2_veer_wrapper.sv:525:- logic hwrite;
design/el2_veer_wrapper.sv:526:-
design/el2_veer_wrapper.sv:527:- logic [63:0] hrdata;
design/el2_veer_wrapper.sv:528:- logic hready;
design/el2_veer_wrapper.sv:529:- logic hresp;
design/el2_veer_wrapper.sv:530:-
design/el2_veer_wrapper.sv:531:- // LSU AHB Master
design/el2_veer_wrapper.sv:532:- logic [31:0] lsu_haddr;
design/el2_veer_wrapper.sv:533:- logic [2:0] lsu_hburst;
design/el2_veer_wrapper.sv:534:- logic lsu_hmastlock;
design/el2_veer_wrapper.sv:535:- logic [3:0] lsu_hprot;
design/el2_veer_wrapper.sv:536:- logic [2:0] lsu_hsize;
design/el2_veer_wrapper.sv:537:- logic [1:0] lsu_htrans;
design/el2_veer_wrapper.sv:538:- logic lsu_hwrite;
design/el2_veer_wrapper.sv:539:- logic [63:0] lsu_hwdata;
design/el2_veer_wrapper.sv:540:-
design/el2_veer_wrapper.sv:541:- logic [63:0] lsu_hrdata;
design/el2_veer_wrapper.sv:542:- logic lsu_hready;
design/el2_veer_wrapper.sv:543:- logic lsu_hresp;
design/el2_veer_wrapper.sv:544:- // Debug Syster Bus AHB
design/el2_veer_wrapper.sv:545:- logic [31:0] sb_haddr;
design/el2_veer_wrapper.sv:546:- logic [2:0] sb_hburst;
design/el2_veer_wrapper.sv:547:- logic sb_hmastlock;
design/el2_veer_wrapper.sv:548:- logic [3:0] sb_hprot;
design/el2_veer_wrapper.sv:549:- logic [2:0] sb_hsize;
design/el2_veer_wrapper.sv:550:- logic [1:0] sb_htrans;
design/el2_veer_wrapper.sv:551:- logic sb_hwrite;
design/el2_veer_wrapper.sv:552:- logic [63:0] sb_hwdata;
design/el2_veer_wrapper.sv:553:-
design/el2_veer_wrapper.sv:554:- logic [63:0] sb_hrdata;
design/el2_veer_wrapper.sv:555:- logic sb_hready;
design/el2_veer_wrapper.sv:556:- logic sb_hresp;
design/el2_veer_wrapper.sv:557:-
design/el2_veer_wrapper.sv:558:- // DMA Slave
design/el2_veer_wrapper.sv:559:- logic dma_hsel;
design/el2_veer_wrapper.sv:560:- logic [31:0] dma_haddr;
design/el2_veer_wrapper.sv:561:- logic [2:0] dma_hburst;
design/el2_veer_wrapper.sv:562:- logic dma_hmastlock;
design/el2_veer_wrapper.sv:563:- logic [3:0] dma_hprot;
design/el2_veer_wrapper.sv:564:- logic [2:0] dma_hsize;
design/el2_veer_wrapper.sv:565:- logic [1:0] dma_htrans;
design/el2_veer_wrapper.sv:566:- logic dma_hwrite;
design/el2_veer_wrapper.sv:567:- logic [63:0] dma_hwdata;
design/el2_veer_wrapper.sv:568:- logic dma_hreadyin;
design/el2_veer_wrapper.sv:569:-
design/el2_veer_wrapper.sv:570:- logic [63:0] dma_hrdata;
design/el2_veer_wrapper.sv:571:- logic dma_hreadyout;
design/el2_veer_wrapper.sv:572:- logic dma_hresp;
design/el2_veer_wrapper.sv:573:-
design/el2_veer_wrapper.sv:574:-
design/el2_veer_wrapper.sv:575:-
design/el2_veer_wrapper.sv:576:- // AHB
design/el2_veer_wra
|