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

[top_earlgrey,tests] Fix clkmgr_external_clk_src_for_sw_slow_test_fpga_cw310_test_rom #19620

Closed
matutem opened this issue Sep 6, 2023 · 2 comments · Fixed by #21096
Closed
Assignees
Labels

Comments

@matutem
Copy link
Contributor

matutem commented Sep 6, 2023

Description

This test is failing on the cw310 fpga: only the main clk gets a measurement error, all others clocks are okay. The ...for_sw_fast..." test works okay.

@matutem matutem added this to the Earlgrey ES SV2 milestone Sep 6, 2023
@matutem
Copy link
Contributor Author

matutem commented Jan 26, 2024

It turns out the measurements of main in slow speed correspond to the expected values for fast. This means the main clk seems to be running at 96 MHz rather than the expected 48 MHz. In sim_dv main runs at the expected 48 MHz. Seems like an FPGA bug, since in silicon the external clock is used for calibration, and that flow works.

@a-will
Copy link
Contributor

a-will commented Jan 26, 2024

The FPGA doesn't support a physical external clock, nor having the main clock at a lower frequency. I think we can close this and accept that the test is not supported on FPGA, then (or at least CW310). Throwing in an alternate clock and muxing it out would consume more of already very impacted clocking resources on the CW310.

When we're in "48 MHz external clock" mode, though, what does work is compensating for clkmgr's switching of dividers. The I/O frequencies remain at the expected values.

matutem added a commit to matutem/opentitan that referenced this issue Jan 29, 2024
…low_test

The fpga doesn't support lowering the main clock frequency.

Fixes lowRISC#19620

Signed-off-by: Guillermo Maturana <[email protected]>
matutem added a commit that referenced this issue Jan 29, 2024
…low_test

The fpga doesn't support lowering the main clock frequency.

Fixes #19620

Signed-off-by: Guillermo Maturana <[email protected]>
github-actions bot pushed a commit that referenced this issue Mar 5, 2024
…low_test

The fpga doesn't support lowering the main clock frequency.

Fixes #19620

Signed-off-by: Guillermo Maturana <[email protected]>
(cherry picked from commit 1aa5e3d)
jwnrt pushed a commit that referenced this issue May 14, 2024
…low_test

The fpga doesn't support lowering the main clock frequency.

Fixes #19620

Signed-off-by: Guillermo Maturana <[email protected]>
(cherry picked from commit 1aa5e3d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants