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

[TB]: Extend testbench to improve debugging #53

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Conversation

Lore0599
Copy link

@Lore0599 Lore0599 commented Nov 5, 2024

Extend Testbench for better observability

This PR introduces some few adjustment to the Testbench to improve the observability when printing a word with UART.
Additionally, it also modify some parameters due to some issues encountered when testing the Chimrea PD design.
the goal of this PR, is to end up with a shared version of the VIP module.

JTAG SBUSY Problem

The reason why the idle_cycles have been increased to 4000, is to work around an issue with the task jtag_poll_bit0. This task uses the Debug Module function read_dmi_exp_backoff, which should not be used for read operations with side effects, as reading SBData0 is one such operation.
For instance, if the clock signal is removed, something that can occur during FLL testing, the Debug Module may not complete the write operation before attempting to read SBData0. This results in the Debug Module appearing busy. Consequently, during the next iteration of the exponential backoff, the DMI resets without clearing the sbusy bit, which leaves the system in a perpetually unstable state.
This problem should be documented and addressed properly in a dedicated issue.

@Lore0599 Lore0599 requested review from Scheremo and sermazz November 5, 2024 14:11
@sermazz
Copy link

sermazz commented Nov 5, 2024

There's a failing test for Fatal: vsim caught SIGTERM. Closing vsimk kernel., we can merge after fixing the CI.
Also make sure to open an issue with what we discussed about the hotfix at line https://github.com/pulp-platform/chimera/pull/53/files#diff-e7ab86558bbfd35750a5495e656e391742070643bd26b942056310b2c72ed69cR332

@Lore0599 Lore0599 merged commit 233863c into devel Nov 6, 2024
6 checks passed
@Lore0599 Lore0599 deleted the lleone/extend-tb branch November 6, 2024 07:23
@Lore0599 Lore0599 mentioned this pull request Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants