pkarashchenko commented on a change in pull request #5192:
URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r780531230



##########
File path: arch/risc-v/src/qemu-rv32/qemu_rv32_head.S
##########
@@ -154,21 +139,6 @@ exception_common:
   lw   s0, 32*4(sp) /* restore mstatus */
   csrw mstatus, s0
 
-#if defined(INT_XCPT_REGS) && INT_XCPT_REGS >= 39
-  lw x28, 36*4(sp)
-  lw x29, 37*4(sp)
-  lw x30, 38*4(sp)
-  csrrw x0, 0x7b4, x28
-  csrrw x0, 0x7b5, x29
-  csrrw x0, 0x7b6, x30
-  lw x28, 33*4(sp)
-  lw x29, 34*4(sp)
-  lw x30, 35*4(sp)
-  csrrw x0, 0x7b0, x28
-  csrrw x0, 0x7b1, x29
-  csrrw x0, 0x7b2, x30
-#endif

Review comment:
       Why this is removed?

##########
File path: arch/risc-v/src/qemu-rv32/qemu_rv32_head.S
##########
@@ -105,21 +105,6 @@ exception_common:
   sw   x30, 30*4(sp)  /* t5 */
   sw   x31, 31*4(sp)  /* t6 */
 
-#if defined(INT_XCPT_REGS) && INT_XCPT_REGS >= 39
-  csrr x28, 0x7b0
-  csrr x29, 0x7b1
-  csrr x30, 0x7b2
-  sw x28, 33*4(sp)
-  sw x29, 34*4(sp)
-  sw x30, 35*4(sp)
-  csrr x28, 0x7b4
-  csrr x29, 0x7b5
-  csrr x30, 0x7b6
-  sw x28, 36*4(sp)
-  sw x29, 37*4(sp)
-  sw x30, 38*4(sp)
-#endif
-

Review comment:
       Why this is removed?

##########
File path: arch/risc-v/src/rv32m1/rv32m1_irq_dispatch.c
##########
@@ -53,11 +53,11 @@ volatile uint32_t * g_current_regs;
  ****************************************************************************/
 
 LOCATE_ITCM
-void *rv32m1_dispatch_irq(uint32_t vector, uint32_t *regs)
+void *rv32m1_dispatch_irq(uintptr_t vector, uintptr_t *regs)
 {
   int vec = vector & 0x1f;
   int irq = (vector >> 27) + vec;

Review comment:
       ```suggestion
     uintptr_t vec = vector & 0x1f;
     uintptr_t irq = (vector >> 27) + vec;
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to