lowRISC / lowrisc-chip

The root repo for lowRISC project and FPGA demos.
http://www.lowrisc.org/
Other
596 stars 148 forks source link

vcs make file error #60

Closed clare7 closed 7 years ago

clare7 commented 7 years ago

Hi when I tried to make for /lowrisc-chip/vcs I get the following errors:

Error-[IND] Identifier not declared .../lowrisc-chip/minion_subsystem/verilog/minion_soc.sv, 66 Identifier 'one_hot_rdata' has not been declared yet. If this error is not expected, please check if you have setdefault_nettype to none.`

any advice?

jrrk commented 7 years ago

Suppprt for vcs is experimental. Try moving the declaration of one_hot_rdata and friends earlier in the file (before first use).

Sent from my iPhone

On 10 Aug 2017, at 06:00, clare7 notifications@github.com wrote:

Hi when I tried to make for /lowrisc-chip/vcs I get the following errors:

Error-[IND] Identifier not declared .../lowrisc-chip/minion_subsystem/verilog/minion_soc.sv, 66 Identifier 'one_hot_rdata' has not been declared yet. If this error is not expected, please check if you have setdefault_nettype to none.`

any advice?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

clare7 commented 7 years ago

Alright noted! thanks

jrrk commented 7 years ago

No they should not be commented out. VCS has a stricter interpretation of Verizon ordering of statements than Vivado, so you need to move the declarations that VCS complains about earlier in the file. I have a feeling this bug was already fixed on some branch, but you do not say what branch you are working on.

The function of the signals is to multiplex between the control and data signals of the different Minion peripherals.

Sent from my iPhone

On 10 Aug 2017, at 07:58, clare7 notifications@github.com wrote:

What are the function of these one_hot_rdata and friends actually? So I just need to comment them out in the files?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

clare7 commented 7 years ago

Thanks! I have did what you have suggested and it works! With that I close the topic :)

On Thu, Aug 10, 2017 at 3:27 PM, jrrk notifications@github.com wrote:

No they should not be commented out. VCS has a stricter interpretation of Verizon ordering of statements than Vivado, so you need to move the declarations that VCS complains about earlier in the file. I have a feeling this bug was already fixed on some branch, but you do not say what branch you are working on.

The function of the signals is to multiplex between the control and data signals of the different Minion peripherals.

Sent from my iPhone

On 10 Aug 2017, at 07:58, clare7 notifications@github.com wrote:

What are the function of these one_hot_rdata and friends actually? So I just need to comment them out in the files?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/lowRISC/lowrisc-chip/issues/60#issuecomment-321473568, or mute the thread https://github.com/notifications/unsubscribe-auth/AcHzHLjewNp6KdT58a8LGKEP_vNi7zo_ks5sWrDPgaJpZM4Oy8k6 .

-- Regards, Ming