Skip to content

chip: rename Ibex to EarlGrey#1997

Merged
bors[bot] merged 1 commit into
tock:masterfrom
silvestrst:silvestrs/tock-ibex-to-earlgrey
Jul 6, 2020
Merged

chip: rename Ibex to EarlGrey#1997
bors[bot] merged 1 commit into
tock:masterfrom
silvestrst:silvestrs/tock-ibex-to-earlgrey

Conversation

@silvestrst

@silvestrst silvestrst commented Jul 2, 2020 •
edited
Loading

Copy link
Copy Markdown

Currently EarlGrey is the first and only OpenTitan SoC design, with Ibex CPU at the heart.

Ibex being a CPU does not have any knowledge of EarlGrey peripherals, and hence was misrepresented in Tock. This has caused some confusion to how exactly it was intended to be used, and did not achieve the goals of being stand alone crate, as it was coupled to heavily with the OpenTitan board.

Pull Request Overview

This pull request renames Ibex crate to EarlGrey, because de facto - that's what it is.

This change also makes #1741 more reasonable.

Testing Strategy

  • Should be tested on FPGA or verilator before merging.

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

@silvestrst silvestrst added the WG-OpenTitan In the purview of the OpenTitan working group. label Jul 2, 2020
@silvestrst silvestrst requested a review from a team July 2, 2020 10:15
bradjc
bradjc previously approved these changes Jul 2, 2020
View reviewed changes
alistair23
alistair23 previously approved these changes Jul 2, 2020
View reviewed changes
@silvestrst silvestrst requested a review from a team July 2, 2020 15:57
ppannuto
ppannuto previously approved these changes Jul 2, 2020
View reviewed changes
@ppannuto ppannuto added the last-call Final review period for a pull request. label Jul 2, 2020
@hudson-ayers

hudson-ayers commented Jul 2, 2020

Copy link
Copy Markdown
Contributor

It looks like there are a few references to ibex still around that should probably change (especially the ones in doc comments and the one in tools)

hudson: ~/tmp_tock ((HEAD detached at s2/silvestrs/tock-ibex-to-earlgrey)) $ rg ibex
tools/update_chip_support.py
27:            'lowrisc': ['ibex'],

capsules/src/hmac.rs
7://! let hmac = &ibex::hmac::HMAC;
10://! digest::Digest::set_client(&ibex::hmac::HMAC, mux_hmac);

capsules/src/mlx90614.rs
9://! let mux_i2c = components::i2c::I2CMuxComponent::new(&ibex::i2c::I2C)

boards/components/src/hmac.rs
9://!    let mux_hmac = components::hmac::HmacMuxComponent::new(&ibex::hmac::HMAC).finalize(

doc/wg/opentitan/README.md
55:- `chips/ibex`

capsules/src/usb/cdc.rs
39:/// Platform-specific packet length for the `ibex` USB hardware.

capsules/src/usb/usbc_client.rs
32:/// Platform-specific packet length for the `ibex` USB hardware.

hudson: ~/tmp_tock ((HEAD detached at s2/silvestrs/tock-ibex-to-earlgrey)) $ rg Ibex
boards/README.md
23:| [Nexys Video OpenTitan](opentitan/README.md)                         | RISC-V RV32IMC  | Ibex           | custom     | custom         |

chips/earlgrey/README.md
4:Ibex RISC-V compliant processor.

boards/opentitan/src/main.rs
102:    // Ibex-specific handler

@silvestrst

silvestrst commented Jul 3, 2020

Copy link
Copy Markdown
Author

@hudson-ayers good catch, thank you!

@silvestrst silvestrst mentioned this pull request Jul 3, 2020
Merged
2 tasks
@silvestrst silvestrst dismissed stale reviews from ppannuto, alistair23, and bradjc via ece7a4d July 3, 2020 14:28
@silvestrst silvestrst force-pushed the silvestrs/tock-ibex-to-earlgrey branch from 5d7c8c8 to ece7a4d Compare July 3, 2020 14:28
chip: rename Ibex to EarlGrey
356e461 
Currently EarlGrey is the first and only OpenTitan SoC design, with
Ibex CPU at the heart.

Ibex being a CPU does not have any knowledge of EarlGrey peripherals,
and hence was misrepresented in Tock. This has caused some confusion
to how exactly is was intended to be used, and did not achieve the
goals of being stand alone crate, as it was coupled to heavily with
the OpenTitan board.
@silvestrst silvestrst force-pushed the silvestrs/tock-ibex-to-earlgrey branch from ece7a4d to 356e461 Compare July 3, 2020 14:30
@silvestrst

silvestrst commented Jul 3, 2020

Copy link
Copy Markdown
Author

I have changed most references of ibex to earlgrey, I think the ones that are left - are justified.

@ppannuto

ppannuto commented Jul 6, 2020

Copy link
Copy Markdown
Member

bors r+

@bors

bors Bot commented Jul 6, 2020

Copy link
Copy Markdown
Contributor

Build succeeded:

@bors bors Bot merged commit 2694f5b into tock:master Jul 6, 2020
sirchnik pushed a commit to sirchnik/tock that referenced this pull request May 12, 2026
@bors
Merge tock#1997
97348dd 
1997: chip: rename Ibex to EarlGrey r=ppannuto a=silvestrst

Currently EarlGrey is the first and only OpenTitan SoC design, with Ibex CPU at the heart.

Ibex being a CPU does not have any knowledge of EarlGrey peripherals, and hence was misrepresented in Tock. This has caused some confusion to how exactly it was intended to be used, and did not achieve the goals of being stand alone crate, as it was coupled to heavily with the OpenTitan board.

### Pull Request Overview

This pull request renames Ibex crate to EarlGrey, because de facto - that's what it is.

This change also makes tock#1741 more reasonable.

### Testing Strategy

- [x] Should be tested on FPGA or verilator before merging.

### Documentation Updated

- [x] Updated the relevant files in `/docs`, or no updates are required.

### Formatting

- [x] Ran `make prepush`.


Co-authored-by: Silvestrs Timofejevs <silvestrst@lowrisc.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alevy alevy alevy approved these changes

@ppannuto ppannuto ppannuto approved these changes

@hudson-ayers hudson-ayers hudson-ayers approved these changes

@alistair23 alistair23 Awaiting requested review from alistair23

@bradjc bradjc Awaiting requested review from bradjc

Assignees

No one assigned

Labels

last-call Final review period for a pull request. WG-OpenTitan In the purview of the OpenTitan working group.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@silvestrst @hudson-ayers @ppannuto @alevy @alistair23 @bradjc