On 1/21/22 7:07 AM, Atish Patra wrote:
Add the definition for ratified privileged specification version v1.12
Signed-off-by: Atish Patra <ati...@rivosinc.com>
---
target/riscv/cpu.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
index 4d630867650a..671f65100b1a 100644
--- a/target/riscv/cpu.h
+++ b/target/riscv/cpu.h
@@ -82,6 +82,7 @@ enum {
#define PRIV_VERSION_1_10_0 0x00011000
#define PRIV_VERSION_1_11_0 0x00011100
+#define PRIV_VERSION_1_12_0 0x00011200
Is there any good reason for defining things this way, as opposed to a simple
enumeration?
A simple enum would eliminate the need for
+ /* The default privilege specification version supported is 1.10 */
+ if (!csr_min_priv) {
+ csr_min_priv = PRIV_VERSION_1_10_0;
+ }
in patch 5.
r~
