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

Out of date URL in CSR DV Plan #2625

Open
MikeOpenHWGroup opened this issue Nov 21, 2024 · 1 comment
Open

Out of date URL in CSR DV Plan #2625

MikeOpenHWGroup opened this issue Nov 21, 2024 · 1 comment
Assignees
Labels
Component:Doc For issues in the Documentation (e.g. for README.md files) Component:Verif For issues in the verification environment or test cases (e.g. for testbench, C code, etc.) Good First Issue Good first issue to work on if you want to contribute Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system

Comments

@MikeOpenHWGroup
Copy link
Member

MikeOpenHWGroup commented Nov 21, 2024

In verif/docs/VerifPlans/csr_access/dvplan_csr-access.md there are multiple instances of references to an out-of-date DV Plan, namely https://docs.openhwgroup.org/projects/cva6-user-manual/01\_cva6\_user/CV32A6\_Control\_Status\_Registers.html.

An example of this appears on line 92.

@MikeOpenHWGroup MikeOpenHWGroup added Component:Doc For issues in the Documentation (e.g. for README.md files) Component:Verif For issues in the verification environment or test cases (e.g. for testbench, C code, etc.) Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system Good First Issue Good first issue to work on if you want to contribute labels Nov 21, 2024
ASintzoff added a commit to ThalesSiliconSecurity/cva6 that referenced this issue Nov 22, 2024
@MikeOpenHWGroup
Copy link
Member Author

Sorry @ASintzoff, the Issue is still not resolved. In the Markdown file, the URL contains backslashes (\) to escape the underscores (_). These are not needed and are in fact the issue. Line 91 of verif/docs/VerifPlans/csr_access/dvplan_csr-access.md is a good example. Instead of this:

 **Requirement location:** https://docs.openhwgroup.org/projects/cva6-user-manual/01\_cva6\_user/CSR\_CV32A60X.html

we need this:

 **Requirement location:** https://docs.openhwgroup.org/projects/cva6-user-manual/01_cva6_user/CSR_CV32A60X.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Doc For issues in the Documentation (e.g. for README.md files) Component:Verif For issues in the verification environment or test cases (e.g. for testbench, C code, etc.) Good First Issue Good first issue to work on if you want to contribute Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system
Projects
None yet
Development

No branches or pull requests

2 participants