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] Fast Preload of binary for fast debug #57

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Conversation

arpansur
Copy link

@arpansur arpansur commented Dec 9, 2024

In the existing testbench the binary preload takes a lot of time due to JTAG load. A fast preload of binary is done by forcing the relevant signals in the narrow interconnect of the memory island. The method still relies on JTAG to halt and unhalt the core whereas the preload(significant portion) is done through forcing the relevant memory signals.

In the existing testbench the binary preload takes a lot of time due to JTAG load.
A fast preload of binary is done by forcing the relevant signals in the narrow interconnect
of the memory island. The method still relies on JTAG to halt and unhalt the core whereas the
preload(significant portion) is done through forcing the relevant memory signals.
@arpansur arpansur requested a review from Scheremo December 9, 2024 10:38
Copy link
Collaborator

@Scheremo Scheremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Arpan,

Looks good to me! Only very minor points

Comment on lines 36 to 39
static doub_bt write_address;
static doub_bt write_data;
write_address = addr;
write_data = data;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be automatic or not decorated instead of static

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using automatic variable leads to compilation error that it is not allowed in the force statement.

target/sim/src/vip_chimera_soc.sv Outdated Show resolved Hide resolved
target/sim/src/vip_chimera_soc.sv Outdated Show resolved Hide resolved
@arpansur arpansur force-pushed the prasadar/fast_debug branch 2 times, most recently from b4fd201 to 85a0fb8 Compare December 12, 2024 14:27
@arpansur arpansur force-pushed the prasadar/fast_debug branch from 85a0fb8 to 8f40e77 Compare December 12, 2024 14:52
Copy link
Collaborator

@Scheremo Scheremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor discussion point, feel free to decide for or against implementing it, but LGTM otherwise.

target/sim/src/tb_chimera_soc.sv Show resolved Hide resolved
@arpansur arpansur merged commit 68f54dc into devel Dec 12, 2024
6 checks passed
@arpansur arpansur deleted the prasadar/fast_debug branch December 12, 2024 16:31
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