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

[rv_timer] Wrong offset from QEMU or documentation #26030

Closed
SubeLucas opened this issue Jan 27, 2025 · 1 comment
Closed

[rv_timer] Wrong offset from QEMU or documentation #26030

SubeLucas opened this issue Jan 27, 2025 · 1 comment

Comments

@SubeLucas
Copy link

There appears to be a discrepancy between the actual usage of the offset in QEMU and the information provided in the documentation. Could someone with knowledge on this topic confirm whether the QEMU implementation needs to be modified to align with the documentation, or if the documentation itself needs to be updated to reflect the current implementation?

Link of the corresponding issue on the QEMU Gitlab :
https://gitlab.com/qemu-project/qemu/-/issues/2787

Link to the current documentation :
https://github.com/lowRISC/opentitan/blob/master/hw/ip/rv_timer/doc/registers.md

@rswarbrick
Copy link
Contributor

Yep, the OpenTitan documentation is correct here. (Indeed, it gets automatically generated from the same thing as the actual implementation RTL so it will definitely match!)

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

No branches or pull requests

2 participants