lowRISC / ibex

Ibex is a small 32 bit RISC-V CPU core, previously known as zero-riscy.
https://www.lowrisc.org
Apache License 2.0
1.38k stars 542 forks source link

Fix Xcelium warnings and errors with simple system #1196

Open imphil opened 3 years ago

imphil commented 3 years ago
      .test_en_i             ('b0),
                                |
xmelab: *W,CUVMPW (../src/lowrisc_ibex_ibex_simple_system_0/rtl/ibex_simple_system.sv,185|32): port sizes differ in port connection (32/1).
      .debug_req_i           ('b0),
                                |
xmelab: *W,CUVMPW (../src/lowrisc_ibex_ibex_simple_system_0/rtl/ibex_simple_system.sv,214|32): port sizes differ in port connection (32/1).
      .fetch_enable_i        ('b1),
                                |
xmelab: *W,CUVMPW (../src/lowrisc_ibex_ibex_simple_system_0/rtl/ibex_simple_system.sv,216|32): port sizes differ in port connection (32/1).
  always_ff @(posedge clk_i or negedge rst_ni) begin
          |
xmelab: *E,MULAXX (../src/lowrisc_ibex_sim_shared_0/rtl/sim/simulator_ctrl.sv,59|10): Multiple drivers to always_ff output variable sim_finish detected.
  always_ff @(posedge clk_i or negedge rst_ni) begin
          |
xmelab: *E,MULAXX (../src/lowrisc_ibex_sim_shared_0/rtl/sim/simulator_ctrl.sv,59|10): Multiple drivers to always_ff output variable sim_finish detected.
  always_ff @(posedge clk_i or negedge rst_ni) begin
          |
xmelab: *E,MULAXX (../src/lowrisc_ibex_sim_shared_0/rtl/sim/simulator_ctrl.sv,59|10): Multiple drivers to always_ff output variable sim_finish detected.
xrun: *E,ELBERR: Error during elaboration (status 1), exiting.
rahulraveendran15-coder commented 3 years ago

@imphil I have solved the errors due to LRM violation that were causing with xcelium. Do I need to push the changes?

imphil commented 3 years ago

I have solved the errors due to LRM violation that were causing with xcelium. Do I need to push the changes?

Yes, please do a pull request. You can find more information on how to do that in the GitHub documentation (e.g. at https://docs.github.com/en/free-pro-team@latest/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request#changing-the-branch-range-and-destination-repository), and by Googling for "GitHub pull requests".

Fresher14 commented 3 years ago

@imphil I get exactly same results. Replacing 'always_ff' with with a simple 'always' was enough to pass hello_test in my environment. On top of this, I had to retrieve 2 files generated by FuseSoc: prim_clock_gating.sv prim_ram_2p.sv Then force a define RVFI Then source dv/uvm/core_ibex/common/prim/prim_pkg.sv

rahulraveendran15-coder commented 3 years ago

@Fresher14

>> On top of this, I had to retrieve 2 files generated by FuseSoc:
prim_clock_gating.sv
prim_ram_2p.sv
Then force a define RVFI
Then source dv/uvm/core_ibex/common/prim/prim_pkg.sv

What you mean by this could you please explain? For what purpose you changed the files,Because I didnt do any changes on these files.

Fresher14 commented 3 years ago

See topic #1187