-
Notifications
You must be signed in to change notification settings - Fork 777
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
[darjeeling] Instantiate Darjeeling on master
#25156
base: master
Are you sure you want to change the base?
Conversation
dc9910a
to
539cfad
Compare
The verible lint job fails because this PR is exceeding the maximum number of files. |
master
Here's the output of a manual Verible lint run on the files changed by this PR:
No real violations IMO |
Signed-off-by: Robert Schilling <[email protected]>
A manual run of Check for and block unauthorized changes with
passed; thus there are no changes to authorize in this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR seems huge, but most changes are autogenerated and contained within one commit, which contains the command to generate them. I cross-checked the autogenerated file by repeating the command, and it matches. (This is in addition to a CI check that should catch mismatches, I think.)
LGTM!
31b3416
to
1b61155
Compare
Add top_darjeeling.hjson and all xbar.hjson files. Signed-off-by: Robert Schilling <[email protected]>
Command run: make -C hw top_and_cmdgen Signed-off-by: Robert Schilling <[email protected]>
Signed-off-by: Robert Schilling <[email protected]>
Signed-off-by: Robert Schilling <[email protected]>
Signed-off-by: Robert Schilling <[email protected]>
1b61155
to
d0f0cd3
Compare
Add jtag_id_pkg.sv, padring.sv, scan_role_pkg, ibex_pmp_reset_pkg.sv Signed-off-by: Robert Schilling <[email protected]>
d10c3aa
to
fe8fbe2
Compare
The mi/dio signals are used in both cases, hardware strap sampling enabled an not. Signed-off-by: Robert Schilling <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes related to FuseSoC / virtual cores and linting / dvsim flows are needed to resolve conflicts with Earlgrey. This is ongoing work; withdrawing approval for now (architecturally still LGTM, though).
We are finally able to instantiate Darjeeling from the
master
branch. This commit is separated into 3 commits:SEC_CM
's in pinmux that are used only when hardware strap sampling is active. This is only the case for Earlgrey and English Breakfasthw/Makefile
and runmake -C hw top_and_cmdgen
to generate Darjeeling and all other artifacts.