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

Skip to content

Conversation

jrvanwhy
Copy link

@jrvanwhy jrvanwhy commented Oct 5, 2023

This PR makes use of the new configuration mechanism introduced in tock/tock#3640 to handle the new cw310 frequencies (changed in #19479). This change fixes the basic Tock test.

I also updated our board definition to work with the changes in tock/tock#3551.

@jrvanwhy jrvanwhy requested review from cfrantz and a team as code owners October 5, 2023 17:41
@jrvanwhy
Copy link
Author

jrvanwhy commented Oct 5, 2023

After sending this PR for review, I discovered that we need an additional change to support running with an upstream kernel. In particular, we need to specify the fpga_cw310 feature on the earlgrey crate. I'll try to figure out how to do that later today.

@jrvanwhy
Copy link
Author

jrvanwhy commented Oct 5, 2023

Okay, fpga_cw310 is in place. I've confirmed this works with an external Tock repository, provided you manually adjust the clock frequencies in the Tock repository. I'll send the frequency changes upstream.

This sets the clock frequencies to match earlgrey on cw310, allowing us to re-enable our Tock test.

Signed-off-by: Johnathan Van Why <[email protected]>
@cfrantz cfrantz merged commit 36fe175 into lowRISC:master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants