Skip to content

Commit 1b59c67

Browse files
committed
[dv] Remove MISA from csr_description.yaml
The value of `misa` will change depending on whether M or B are enabled. The presence and read values of other CSRs may also depend upon the Ibex configuration. A fix is required to allow riscv_csr_test to deal with different CSR descriptions for different Ibex configurations. For now just comment out `misa` from the descriptions file to enable riscv_csr_test to run on a wider range of configurations.
1 parent 48a886a commit 1b59c67

File tree

1 file changed

+22
-20
lines changed

1 file changed

+22
-20
lines changed

dv/uvm/core_ibex/riscv_dv_extension/csr_description.yaml

+22-20
Original file line numberDiff line numberDiff line change
@@ -39,27 +39,29 @@
3939
# - ...
4040

4141

42+
# TODO: Enable multiple configurations for CSR test. MISA read value will depend
43+
# on whether B and M are available. https://github.com/lowRISC/ibex/issues/1333
4244
# Ibex MISA CSR
43-
- csr: misa
44-
description: >
45-
Machine ISA Register
46-
address: 0x301
47-
privilege_mode: M
48-
rv32:
49-
- field_name: MXL
50-
description: >
51-
Encodes native base ISA width
52-
type: R
53-
reset_val: 1
54-
msb: 31
55-
lsb: 30
56-
- field_name: Extensions
57-
description: >
58-
Encodes all supported ISA extensions
59-
type: R
60-
reset_val: 0x101106
61-
msb: 25
62-
lsb: 0
45+
#- csr: misa
46+
# description: >
47+
# Machine ISA Register
48+
# address: 0x301
49+
# privilege_mode: M
50+
# rv32:
51+
# - field_name: MXL
52+
# description: >
53+
# Encodes native base ISA width
54+
# type: R
55+
# reset_val: 1
56+
# msb: 31
57+
# lsb: 30
58+
# - field_name: Extensions
59+
# description: >
60+
# Encodes all supported ISA extensions
61+
# type: R
62+
# reset_val: 0x101106
63+
# msb: 25
64+
# lsb: 0
6365

6466
# Ibex's implementation of MHARTID is read-only
6567
# MHARTID

0 commit comments

Comments
 (0)