Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jrvanwhy
Copy link
Contributor

@jrvanwhy jrvanwhy commented Oct 5, 2023

Pull Request Overview

OpenTitan increased some of Earlgrey's clock frequencies on cw310: lowRISC/opentitan#19479. This PR adjusts Tock's Earlgrey config on cw310 to match the new values.

Replaces #3639

Testing Strategy

This pull request was tested using OpenTitan's //sw/device/silicon_owner/tock/tests/basic:basic_test with lowRISC/opentitan#19895 applied.

Documentation Updated

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

Formatting

  • Ran make prepush.

@github-actions github-actions bot added the WG-OpenTitan In the purview of the OpenTitan working group. label Oct 5, 2023
@jrvanwhy jrvanwhy changed the title Earlgrey: Update clock cw310 FPGA clock frequencies Earlgrey: Update cw310 FPGA clock frequencies Oct 5, 2023
@bradjc
Copy link
Contributor

bradjc commented Oct 6, 2023

Don't we also need to specify the SHA?

…requencies.

I failed to test this myself (I was unable to find the rsa.tbf that board/opentitan/README.md thought would exist) so I used the already-tested commit here: tock#3639 (comment)
@jrvanwhy
Copy link
Contributor Author

jrvanwhy commented Oct 6, 2023

Don't we also need to specify the SHA?

I think so, done.

@bradjc bradjc enabled auto-merge October 6, 2023 18:18
@bradjc bradjc added this pull request to the merge queue Oct 6, 2023
Merged via the queue into tock:master with commit 3df5b00 Oct 6, 2023
@jrvanwhy jrvanwhy deleted the ot-freq-update branch February 23, 2024 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WG-OpenTitan In the purview of the OpenTitan working group.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants