Xqci extensions: additional fixes (based on compiler team feedback)

Signed-off-by: Albert Yosher <[email protected]>

Author: ayosher

cfgs/qc_iu/arch_overlay/ext/Xqci.yaml

@@ -149,8 +149,12 @@ versions:
   changes:
     - Fix encoding of qc.c.extu instruction
     - Fix encoding of qc.swmi instruction
+    - Fix decoding of qc.pputci instruction
+    - Fix decoding of qc.delay instruction (state that immediate cannot be 0)
     - Rename qc.slasat -> qc.shlsat
     - Rename qc.sllsat -> qc.shlusat
+    - Add requirement to include Zca extension for Xqcisim since it has 16-bit instructions
+    - Add requirement to include Zca extension for Xqcisync since it has 16-bit instructions
   implies:
   - [Xqcia, "0.4.0"]
   - [Xqciac, "0.2.0"]
@@ -166,7 +170,9 @@ versions:
   - [Xqcilia, "0.2.0"]
   - [Xqcilo, "0.2.0"]
   - [Xqcilsm, "0.4.0"]
+  - [Xqcisim, "0.2.0"]
   - [Xqcisls, "0.2.0"]
+  - [Xqcisync, "0.2.0"]
   requires:
     name: Zca
     version: ">= 1.0.0"

cfgs/qc_iu/arch_overlay/ext/Xqcisim.yaml

@@ -16,6 +16,20 @@ versions:
   - name: Derek Hower
     company: Qualcomm Technologies, Inc.
     email: [email protected]
+- version: "0.2.0"
+  state: frozen
+  ratification_date: null
+  contributors:
+  - name: Albert Yosher
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  - name: Derek Hower
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  changes:
+    - Fix decoding of qc.pputci instruction
+    - Add requirement to include Zca extension since has 16-bit instructions
+  requires: { name: Zca, version: ">= 1.0.0" }
 description: |
   The Xqcisim extension includes ten hint instructions to interface simulation environment.
   On real target any instruction from this extension executed as "no-operation" and have no effect.

cfgs/qc_iu/arch_overlay/ext/Xqcisync.yaml

@@ -16,6 +16,20 @@ versions:
   - name: Derek Hower
     company: Qualcomm Technologies, Inc.
     email: [email protected]
+- version: "0.2.0"
+  state: frozen
+  ratification_date: null
+  contributors:
+  - name: Albert Yosher
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  - name: Derek Hower
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  changes:
+    - Fix decoding of qc.delay instruction (state that immediate cannot be 0)
+    - Add requirement to include Zca extension since has 16-bit instructions
+  requires: { name: Zca, version: ">= 1.0.0" }
 description: |
   The Xqcisync extension includes nine instructions, eight for non-memory-mapped devices synchronization and delay instruction.
   Synchronization instructions are kind of IO fences that work with special devices synchronization signals.

cfgs/qc_iu/arch_overlay/inst/Xqci/qc.c.delay.yaml

@@ -18,6 +18,7 @@ encoding:
   variables:
     - name: imm
       location: 6-2
+      not: 0
 access:
   s: always
   u: always

cfgs/qc_iu/arch_overlay/inst/Xqci/qc.pputci.yaml

@@ -18,7 +18,7 @@ encoding:
   match: 0100--------00000010000000010011
   variables:
     - name: imm
-      location: 29-20
+      location: 27-20
 access:
   s: always
   u: always