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.

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

[DOC] User Manual: Update the debug description #258

Closed
silabs-robin opened this issue Oct 4, 2021 · 4 comments
Closed

[DOC] User Manual: Update the debug description #258

silabs-robin opened this issue Oct 4, 2021 · 4 comments
Assignees
Labels
Component:Doc For issues in the Documentation (e.g. for User Manual, README.md files) 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-robin
Copy link
Contributor

The user manual does not include new (planned) 40x debug capabilities. E.g. DBG_NUM_TRIGGERS etc.

@Silabs-ArjanB Silabs-ArjanB added Component:Doc For issues in the Documentation (e.g. for User Manual, README.md files) Type:Bug For bugs in any content (RTL, Documentation, etc.) labels Oct 4, 2021
@Silabs-ArjanB
Copy link
Contributor

Also fix the following statement:

debug_req_i is the “debug interrupt”, issued by the debug module when the core should enter Debug Mode. The debug_req_i is synchronous to clk_i and requires a minimum assertion of one clock period to enter Debug Mode. The instruction being decoded during the same cycle that debug_req_i is first asserted shall not be executed before entering Debug Mode.

We don't want remarks as 'the instruction being decoded' as this is not externally visible. There just needs to be a remark somewhere how to enter debug immediately when coming out of reset / fetch enable = 1.

Also update the remark about 'one clock period' if no longer correct after fixing #341.

@Silabs-ArjanB Silabs-ArjanB changed the title User Manual: Update the debug description [DOC] User Manual: Update the debug description Mar 13, 2023
@silabs-oysteink
Copy link
Contributor

I belive the user manual is up to date with a good description of the debug features already.

@Silabs-ArjanB
Copy link
Contributor

@silabs-robin Can you please close this ticket if agreed? (The points I raised have been addressed.)

@Silabs-ArjanB Silabs-ArjanB added the Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation label Sep 27, 2023
@silabs-robin
Copy link
Contributor Author

Closing based on:
Lots of commits with improvements to the debug documentation have been pushed since the issue got created.
Arjan reports his points have been addressed.
The particular example of DBG_NUM_TRIGGERS is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Doc For issues in the Documentation (e.g. for User Manual, README.md files) 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
Development

No branches or pull requests

3 participants