stnolting / neorv32

:desktop_computer: A small, customizable and extensible MCU-class 32-bit RISC-V soft-core CPU and microcontroller-like SoC written in platform-independent VHDL.
https://neorv32.org
BSD 3-Clause "New" or "Revised" License
1.55k stars 214 forks source link

A question on simple testbech #866

Closed mahdi259 closed 5 months ago

mahdi259 commented 5 months ago

Hi

My question I imported NeoRV32 micro controller into a questa simulation project and used sim/simple/neorv32_tb.simple.vhd and sim/simple/uart_rx.simple.vhd for simulation. I wondered why UART doesn't show activity. Here int_bootloader is disabled and application_image is for hello_world.

To Reproduce Using sim/simple/neorv32_tb.simple.vhd and sim/simple/uart_rx.simple.vhd files for simulation in questasim.

Expected behavior UART activity for printing hello world!

Screenshots image

My VHDL version is 2008. (I changed it from modelsim.ini)

Warnings are attached below. warnings.txt

stnolting commented 5 months ago

Hey there!

Your warning.txt looks fine so far. How did you compile/install the hello_world memory image?

mahdi259 commented 5 months ago

I use this command make clean MARCH=rv32imc_zicsr_zifencei install and copy application_image content into neorv32_application_image.vhd.

stnolting commented 5 months ago

Ah I see, that might be the problem.

According to your warnings.txt the CPU is configured without the C extension:

Note: [NEORV32] CPU ISA: rv32imabu_zicsr_zicntr_zicond_zifencei_zfinx_zihpm_zxcfu_sdext_sdtrig_smpmp

So if you use the same ISA configuration (no "c") for compiling the code the simulation should work (better use clean_all here to remove all artifacts from previous builds):

$ make clean_all MARCH=rv32im_zicsr_zifencei install

By the way, the install target already copies the local memory image into core/rtl overriding the default neorv32_application_image.vhd.

mahdi259 commented 5 months ago

You're right. Thanks a lot. image

stnolting commented 5 months ago

Looks better now! :wink:

I guess we can close this?

mahdi259 commented 5 months ago

Yes thanks.