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

Add macros to disable Mem and SimJTAG DPI-C functions #259

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

klin02
Copy link
Member

@klin02 klin02 commented Jan 18, 2024

We add macros to turn off DPIC on MemRWHelper and SimJTAG without SYNTHESIS. That means we can reduce number of synchronizations if needed. The new macros is defined in palladium.mk by default.

To add build-in decleration of system task for Palladium, we need to add $.

We declare fwrite in tb_top as Gfifo to reduce synchronizations and speed up simulation.

We add Macros to turn off DPIC on MemRWHelper and SimJTAG without
SYNTHESIS. That means we can reduce number of synchronizations if
needed. The new Macros is defined in palladium.mk by default.

To add build-in decleration of system task for Palladium, we need
to add $.

We declare fwrite in tb_top as Gfifo to reduce synchronizations and
speed up simulation.
@klin02 klin02 requested a review from poemonsense January 18, 2024 02:47
@poemonsense poemonsense changed the title Fix Palladium Add macros to disable Mem and SimJTAG DPI-C functions Jan 18, 2024
@klin02 klin02 merged commit 8673c65 into OpenXiangShan:master Jan 18, 2024
3 checks passed
pxk27 pushed a commit to pxk27/difftest that referenced this pull request Jan 30, 2024
)

We add Macros to turn off DPIC on MemRWHelper and SimJTAG without
SYNTHESIS. That means we can reduce number of synchronizations if
needed. The new Macros is defined in palladium.mk by default.

To add build-in decleration of system task for Palladium, we need
to add $.

We declare fwrite in tb_top as Gfifo to reduce synchronizations and
speed up simulation.
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