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.

Sign up for GitHub

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

Jump to bottom

Update lowrisc_ibex to lowRISC/ibex@38c07093 #24403

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Update lowrisc_ibex to lowRISC/ibex@38c07093 #24403

merged 1 commit into from
Aug 23, 2024

Conversation

vogelpi
Copy link
Contributor

@vogelpi vogelpi commented Aug 23, 2024 •
edited
Loading

Update code from upstream repository
https://github.com/lowRISC/ibex.git to revision
38c070939183cc10940b66c8e9e04eeca6b65470

  • [rtl] Remove ECC related data_rdata_i -> instr_X_o feedthroughs (Greg Chadwick)

This is related to #24404.

@vogelpi
Update lowrisc_ibex to lowRISC/ibex@38c07093
3de831c 
Update code from upstream repository
https://github.com/lowRISC/ibex.git to revision
38c070939183cc10940b66c8e9e04eeca6b65470

* [rtl] Remove ECC related data_rdata_i -> instr_X_o feedthroughs
  (Greg Chadwick)

Signed-off-by: Pirmin Vogel <vogelpi@lowrisc.org>
@vogelpi vogelpi added ECO Accepted as an ECO ManuallyCherryPick:earlgrey_1.0.0 This PR should be cherry-picked to the earlgrey_1.0.0 branch (no automation, manual coordination). labels Aug 23, 2024
@vogelpi vogelpi mentioned this pull request Aug 23, 2024
Merged
@vogelpi
Copy link
Contributor Author

vogelpi commented Aug 23, 2024

CHANGE AUTHORIZED: hw/vendor/lowrisc_ibex/rtl/ibex_controller.sv

@moidx
Copy link
Contributor

moidx commented Aug 23, 2024

CHANGE AUTHORIZED: hw/vendor/lowrisc_ibex/rtl/ibex_controller.sv

@moidx
Copy link
Contributor

moidx commented Aug 23, 2024

Can we start an ECO issue for tracking purposes? This will make it easier to document the debug process once we get a chance. Thanks!

@vogelpi vogelpi mentioned this pull request Aug 23, 2024
Closed
@vogelpi
Copy link
Contributor Author

vogelpi commented Aug 23, 2024

Can we start an ECO issue for tracking purposes? This will make it easier to document the debug process once we get a chance. Thanks!

Good point, I've created the issue here #24404 and will link it in the tracker.

@vogelpi vogelpi merged commit 5952581 into lowRISC:master Aug 23, 2024
42 checks passed
@vogelpi vogelpi deleted the ibex-eco branch September 17, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moidx moidx moidx approved these changes

@GregAC GregAC Awaiting requested review from GregAC

@andreaskurth andreaskurth Awaiting requested review from andreaskurth

@OTshimeon OTshimeon Awaiting requested review from OTshimeon

Assignees
No one assigned
Labels
ECO Accepted as an ECO ManuallyCherryPick:earlgrey_1.0.0 This PR should be cherry-picked to the earlgrey_1.0.0 branch (no automation, manual coordination).
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vogelpi @moidx