Skip to content

Commit 246914d

Browse files
fix: Change Zicntr from mandatory to optional in MC100 (#1223)
Keep mandatory in MC200 (for now). Seems like package-lock.json also is updated (maybe because I am running in a new devcontainer?). See CSC Requirements WG discussion about Zicntr at https://lists.riscv.org/g/csc-requirements/topic/116005126 Signed-off-by: James Ball <jameball@qti.qualcomm.com>
1 parent fcae1f7 commit 246914d

File tree

3 files changed

+207
-5
lines changed

3 files changed

+207
-5
lines changed

package-lock.json

Lines changed: 202 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

spec/std/isa/proc_cert_model/MC100-32.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,10 @@ in_scope_priv_modes:
2323
# History of this certificate.
2424
revision_history:
2525
- revision: "0.9.0"
26-
date: "2025-02-06"
26+
date: "2025-10-29"
2727
changes:
2828
- |
29-
Provided to CSC Microcontroller WG for review given the following list of known issues:
30-
** abc
31-
** def
29+
Made Zicntr from mandatory to optional since recent published ISA manuals have it as "recommended"
3230
- revision: "0.8.0"
3331
date: "2025-01-19"
3432
changes:
@@ -118,7 +116,7 @@ extensions:
118116
presence: mandatory
119117
Zicntr:
120118
version: "~> 2.0"
121-
presence: mandatory
119+
presence: optional
122120
param_constraints:
123121
TIME_CSR_IMPLEMENTED: {} # Unconstrained
124122
Sm:

spec/std/isa/proc_cert_model/MC200-32.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,8 @@ debug_manual_revision: "0.13.2"
4646
# TODO: No ratified CLIC yet. It will be multiple extensions.
4747

4848
extensions:
49+
Zicntr:
50+
presence: mandatory
4951
M:
5052
presence: mandatory
5153
B:

0 commit comments

Comments
 (0)