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

[earlgrey_1.0.0] Ibex timing fix #24383

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

GregAC
Copy link
Contributor

@GregAC GregAC commented Aug 22, 2024

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 a cherry pick of commit 5952581
to branch earlgrey_1.0.0.

NOTE: This PR will be updated once the PR into master (#24403 ) has been merged. This has now been done.

@vogelpi vogelpi added the ECO Accepted as an ECO label Aug 23, 2024
@vogelpi vogelpi changed the title DO NOT MERGE: Potential timing fix DO NOT MERGE: Ibex timing fix Aug 23, 2024
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 a cherry pick of commit 5952581
to branch earlgrey_1.0.0.

Signed-off-by: Pirmin Vogel <vogelpi@lowrisc.org>
@vogelpi vogelpi changed the title DO NOT MERGE: Ibex timing fix [earlgrey_1.0.0] Ibex timing fix Aug 23, 2024
@vogelpi
Copy link
Contributor

vogelpi commented Aug 23, 2024

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

@vogelpi vogelpi marked this pull request as ready for review August 23, 2024 23:14
@vogelpi vogelpi self-requested a review August 23, 2024 23:14
@andreaskurth
Copy link
Contributor

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

@andreaskurth andreaskurth merged commit ca4bf7b into lowRISC:earlgrey_1.0.0 Aug 26, 2024
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ECO Accepted as an ECO
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants