-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update Python Debug Modules and fix Virtual Timer clock issue #61
Merged
loiclefort
merged 27 commits into
lowRISC:ot-darjeeling-8.2.0
from
rivos-eblot:dev/ebl/dm_update
Apr 16, 2024
Merged
Update Python Debug Modules and fix Virtual Timer clock issue #61
loiclefort
merged 27 commits into
lowRISC:ot-darjeeling-8.2.0
from
rivos-eblot:dev/ebl/dm_update
Apr 16, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ensure there could not be more than one handler assigned to each logger, to avoid duplication of messages. Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
- use a cache to track transitions for TMS sequences - use a cache to update current state from new transitions Signed-off-by: Emmanuel Blot <[email protected]>
- reduce log evaluation for low level comm traces - use local bytearray to build TCP packet - use local variables for JTAG signal management Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Output is similar to `hexdump -C` Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Should only report the first line of the module documentation Signed-off-by: Emmanuel Blot <[email protected]>
Basic features, limited to RV32 and 32-bit operations: - read/write DM registers - read/write CSR and registers or RISC-V core - read/write memory - load and execute ELF file Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
….misc Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
…er DM Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
VIRTUAL timer is paced with icount setting; VIRTUAL_RT is not. This means that as the higher icount, the less instruction executed in a given time slot, VIRTUAL_RT makes HW timer relatively much faster from the guest standpoint. Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
… bug - CSR value is not required when no CSR check is requested - remove useless update of TDI signal - implement System Bus "fast mode" transfer Signed-off-by: Emmanuel Blot <[email protected]>
- remove useless setter/getter for TMS and TDI - remove useless TMS update, already performed by state selection Signed-off-by: Emmanuel Blot <[email protected]>
* TAP IR length * DMI base address Signed-off-by: Emmanuel Blot <[email protected]>
Signed-off-by: Emmanuel Blot <[email protected]>
loiclefort
approved these changes
Apr 16, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (already reviewed and tested at Rivos)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.