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

Add embedded csr description and CSR table list #1662

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

JeanRochCoulon
Copy link
Contributor

docs: create CSR chapter for embedded config
Add table of embedded CSRs

docs: create CSR chapter for embedded config
Add table of embedded CSRs
@JeanRochCoulon JeanRochCoulon marked this pull request as draft November 29, 2023 14:28
Copy link
Contributor

✔️ successful run, report available here.

@MikeOpenHWGroup
Copy link
Member

MikeOpenHWGroup commented Nov 29, 2023

Two questions:

  1. Can we rename the "embedded" directory to the device name? (e.g CV32-something-something-something)
  2. Are there plans to incorporate these files into one or more of the index files such as docs/01_cva6_user/index.rst so that it is rendered to HTML by ReadTheDocs?

@jquevremont
Copy link
Contributor

  1. Can we rename the "embedded" directory to the device name? (e.g CV32-something-something-something)

@MikeOpenHWGroup, based on the Cores TG feedback, we need to know if the this configuration will be single or dual issue before we know the part number: CV32E60X if it is single issue, CV32E6?X (e.g. CV32E62X, CV32E65X) if it is dual issue.

@JeanRochCoulon
Copy link
Contributor Author

2. Are there plans to incorporate these files into one or more of the index files such as docs/01_cva6_user/index.rst so that it is rendered to HTML by ReadTheDocs?

This question can be addressed during Tomorrow meeting. To my mind, this CSR specification is in a file which can be integrated in UM and/or design documentation. This question can be addressed during Tomorrow meeting.

@JeanRochCoulon
Copy link
Contributor Author

@MikeOpenHWGroup, based on the Cores TG feedback, we need to know if the this configuration will be single or dual issue before we know the part number: CV32E60X if it is single issue, CV32E6?X (e.g. CV32E62X, CV32E65X) if it is dual issue.

Unfortunately the decision to use Dual issue will be taken when the PPA results will be known, probably January. Until this time, the definitive name will be unknown, that's why I am currently use "embedded" name.

Copy link
Contributor

github-actions bot commented Dec 1, 2023

✔️ successful run, report available here.

@ASintzoff
Copy link
Contributor

LGTM

@JeanRochCoulon JeanRochCoulon marked this pull request as ready for review December 6, 2023 08:14
@JeanRochCoulon JeanRochCoulon merged commit 9952bce into openhwgroup:master Dec 6, 2023
13 checks passed
@JeanRochCoulon JeanRochCoulon deleted the csrdoc2 branch January 26, 2024 15:00
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.

4 participants