cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

[TASK] Upgrade Spike tandem with DPI functions to access CSRs and to update with Interrupt/Debug signals

Open AnouarZajni opened this issue 1 year ago • 4 comments

Is there an existing CVA6 task for this?

  • [X] I have searched the existing task issues

Task Description

To align RTL and Spike-Tandem in some test scenarios (example CSR tests, interrupt tests, ...) and for functional verification purposes, upgrade Spike tandem is required to implement DPI functions to access internal spike CSR registers in read and write modes. Also, Spike should be aware about external interrupt/debug signals.

Required Changes

Required DPI functions are: 1- DPI function to write spike internal CSR registers 2- DPI function to read spike internal CSR registers 3- DPI function to update spike with interrupt asserted/de-asserted 4- DPI function to update spike with debug signals

Current Status

N/A

Risks

Some registers are not updated by spike so read return values will be wrong

Prerequisites

No response

KPI (KEY Performance Indicators)

No response

Description of Done

DPI function implemented and tested in UVM testbench

Associated PRs

No response

AnouarZajni avatar Jul 03 '24 10:07 AnouarZajni

👋 Hi there!

This issue seems inactive. Need more help? Feel free to update us. If there are no updates within the next few days, we'll go ahead and close this issue. 😊

github-actions[bot] avatar Aug 23 '24 01:08 github-actions[bot]

#2407 The feature will be included when this PR is merged

MarioOpenHWGroup avatar Aug 23 '24 13:08 MarioOpenHWGroup

For CV32A65X there are no debug features supported. This leaves the interrupt aspect open, see PR combo CVA6 #2502 + https://github.com/openhwgroup/core-v-verif/pull/2532 for a proposed solution.

zchamski avatar Sep 19 '24 12:09 zchamski

👋 Hi there!

This issue seems inactive. Need more help? Feel free to update us. If there are no updates within the next few days, we'll go ahead and close this issue. 😊

github-actions[bot] avatar Oct 20 '24 02:10 github-actions[bot]

Task completed with the merge of PR combo CVA6 https://github.com/openhwgroup/cva6/pull/2502 + https://github.com/openhwgroup/core-v-verif/pull/2532. No debug feature needed as there is no debug support in CV32A65X.

zchamski avatar Oct 28 '24 08:10 zchamski