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

16-bit imp_id field seems too small #46

Closed
MarkHillCodasip opened this issue Mar 14, 2024 · 3 comments
Closed

16-bit imp_id field seems too small #46

MarkHillCodasip opened this issue Mar 14, 2024 · 3 comments

Comments

@MarkHillCodasip
Copy link

The allocation of just 16-bits to vendor_n_imp_id.imp_id to cover all of a companies portfolio of IPs and versions seems quite small to me, especially if a vendor wants to break this broken down into subfields for classes of IP and major/minor/patch versions etc. Specifically in the case of RISCV processor implementations it seems reasonable to support a one-one mapping between RERI registers and mvendorid, marchid and mimpid.

@ved-rivos
Copy link
Collaborator

Its reasonable to widen the imp_id to 32-bits by expanding into the 16 reserved bits adjacent to it. I will post a PR shortly with a change.

@MarkHillCodasip
Copy link
Author

MarkHillCodasip commented Mar 21, 2024 via email

@ved-rivos
Copy link
Collaborator

Update in PR #47

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