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

[doc,reset] Fix references to reset behavior #20738

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

matutem
Copy link
Contributor

@matutem matutem commented Dec 27, 2023

  • Don't mention the reset topology in the lc_ctrl theory of operation doc's description of CPU_EN since lc_cpu_en_o is independent of reset, and rv_core_ibex gets a separate fetch_en input from pwrmgr that determines whether the cpu is effectively enabled.
  • Fix the life cycle control description in the rv_dm interfaces doc: all it needs to say is that NDM reset will not reset rv_dm but will reset lc_ctrl. The prior text is inaccurate.

Fixes #20700

- Don't mention the reset topology in the lc_ctrl theory of operation doc's
  description of CPU_EN since lc_cpu_en_o is independent of reset, and
  rv_core_ibex gets a separate fetch_en input from pwrmgr that determines
  whether the cpu is effectively enabled.
- Fix the life cycle control description in the rv_dm interfaces doc: all
  it needs to say is that NDM reset will not reset rv_dm but will reset
  lc_ctrl. The prior text is inaccurate.

Fixes lowRISC#20700

Signed-off-by: Guillermo Maturana <[email protected]>
@matutem
Copy link
Contributor Author

matutem commented Jan 3, 2024

There is no way these doc changes can be related to the CI failure:
//sw/device/silicon_creator/manuf/skus/earlgrey_a0/sival_bringup:ft_provision_fpga_cw310_sival TIMEOUT in 2 out of 2 in 60.5s

Merging this.

@matutem matutem merged commit 637d1b8 into lowRISC:master Jan 3, 2024
30 of 32 checks passed
@matutem matutem deleted the rstmgr_doc branch January 3, 2024 20:02
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.

[ip,doc] Fix inaccurate references to reset behavior
2 participants