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

Update register map for hw and sw test & submodule version #10

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

zhukun-wang
Copy link

Update the register map for hardware test and software test

@zhukun-wang zhukun-wang deleted the update branch March 18, 2024 15:05
@zhukun-wang zhukun-wang restored the update branch March 18, 2024 15:05
@zhukun-wang zhukun-wang reopened this Mar 18, 2024
@zhukun-wang zhukun-wang changed the title Update Update register map for hw and sw test & submodule version Mar 18, 2024
@zhukun-wang
Copy link
Author

Update register map for hardware and software test. And also update submodule to the latest version.

@GregWatson
Copy link

@zhukun-wang what was the motivation for changing the base addresses? Just curious

@zhukun-wang
Copy link
Author

zhukun-wang commented Mar 18, 2024

@zhukun-wang what was the motivation for changing the base addresses? Just curious

For *_define.tcl, the base address is changed due to the new version onic shell changing their register map from
Box0: 0x1_0000-0x3_FFFF
Box1: 0x4_0000-0xF_FFFF
to
Box0: 0x10_0000-0x1F_FFFF
Box1: 0x20_0000-0x2F_FFFF
So it needs to change the base address to ensure that the correct address can be accessed during testing.

For *_sim.tcl, the base address of crossbar ip core is changed because in actual hardware, the address is first processed by the onic shell and then sent to the corresponding box. But there is no onic shell in software testing, so it needs to directly use the base address corresponding to register.

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