cv32e40x icon indicating copy to clipboard operation
cv32e40x copied to clipboard

User Manual: Update the debug description

Open silabs-robin opened this issue 3 years ago • 1 comments

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

silabs-robin avatar Oct 04 '21 13:10 silabs-robin

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 https://github.com/openhwgroup/cv32e40x/issues/341.

Silabs-ArjanB avatar Sep 28 '22 05:09 Silabs-ArjanB

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

silabs-oysteink avatar Sep 26 '23 17:09 silabs-oysteink

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

Silabs-ArjanB avatar Sep 27 '23 06:09 Silabs-ArjanB

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.

silabs-robin avatar Sep 27 '23 08:09 silabs-robin