-
Notifications
You must be signed in to change notification settings - Fork 31
Upstream Chimera features #103
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
Open
sermazz
wants to merge
42
commits into
master
Choose a base branch
from
smazzola/dkeller-rebased
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or 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
The linker script has L1 address ORIGIN set to 0x10000004 even through in hardware it is set to 0x10000000. However the testbench assumes 64b alignment to initialize the L1. Thus, the data was shifted by 32b in the simulation. While the AXI bursts are set to 64b, the misalignment needs to be handled coming from the linker script.
Being unnecessarily unpacked, it was not compatible with other systems (like Cheshire)
It created problems in routing of requests through peripheral interconnect
… configurations based on WidePortShouldBeEnabled. Update AXI request/response handling and introduce a multiplexer for merging cluster bus and DMA narrow master requests. Enhance isolation and CDC instantiation for wide port scenarios.
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
…ncy in DMA configuration Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
…PE_PORT also in the package as reference Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
… for Questa Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
…ke hex; roll back to astral version Neureka; 3 HWPEs added Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
…lized Signed-off-by: Daniel Keller <daniel.kellermartinez@csem.ch>
…arameters - Update HCI interface parameter structure to correctly propagate the FIFO depth parameter to both the interconnect and the HWPE (Neureka) - Increase the FIFO depth parameter from 0 to 2 to fix the combinational loop due to iDMA being able to read+write from+to TCDM with its 2 backend - Bump versions of HCI and Neureka in bender for updated parameter structure
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.
This merges #89 and #99
dmac_wrap
for iDMAImportant notes
To do
@DanielKellerM