Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Transition from debug_havereset to debug_halted not possible #410

Closed
silabs-oysteink opened this issue Jan 28, 2022 · 1 comment
Closed

Transition from debug_havereset to debug_halted not possible #410

silabs-oysteink opened this issue Jan 28, 2022 · 1 comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Bug For bugs in any content (RTL, Documentation, etc.)

Comments

@silabs-oysteink
Copy link
Contributor

The debug state fsm currently does not allow the transition from debug_havereset to debug_halted (see image).
image

The controller needs an update to ensure the transition will be possible. If debug_req_i is asserted early, the core will set debug_havereset -> debug_running -> debug_halted. Possible solution is to check for debug entry in the state BOOT_SET, and se debug_mode_n accordingly.
@Silabs-ArjanB Silabs-ArjanB added Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Type:Bug For bugs in any content (RTL, Documentation, etc.) labels Jan 28, 2022
Silabs-ArjanB added a commit that referenced this issue Dec 22, 2022
@Silabs-ArjanB
Merge pull request #747 from silabs-oysteink/silabs-oysteink_issue-410
32e8406 
Fix for issue #410
@Silabs-ArjanB
Copy link
Contributor

Fixed by #747

@Silabs-ArjanB Silabs-ArjanB added the Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation label Dec 22, 2022
silabs-oysteink added a commit to silabs-oysteink/cv32e40x that referenced this issue Mar 27, 2023
@silabs-oysteink
Only updating rvfi_mem_rmask and rvfi_mem_wmask if a transaction trul…
2790c50 
…y went into the write buffer (not blocked by WPT, MPU or alignment checker).

Fixes issue openhwgroup#410 on cv32e40s.

Signed-off-by: Oystein Knauserud <[email protected]>
@silabs-oysteink silabs-oysteink mentioned this issue Mar 27, 2023
Merged
Silabs-ArjanB added a commit that referenced this issue Mar 27, 2023
@Silabs-ArjanB
Merge pull request #816 from silabs-oysteink/silabs-oysteink_s-issue-410
c6f4759 
Fix for issue #410 on cv32e40s
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Bug For bugs in any content (RTL, Documentation, etc.)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Silabs-ArjanB @silabs-oysteink