[GitHub] [incubator-nuttx] davids5 commented on a change in pull request #3580: arch: move up_irq{save|restore} to irq.h

[GitBox <gi...@apache.org>]

davids5 commented on a change in pull request #3580:
URL: https://github.com/apache/incubator-nuttx/pull/3580#discussion_r616956499



##########
File path: arch/risc-v/include/bl602/irq.h
##########
@@ -224,8 +224,49 @@ extern "C"
  * Public Function Prototypes
  ****************************************************************************/
 
+/****************************************************************************
+ * Name: up_irq_enable
+ *
+ * Description:
+ *   Return the current interrupt state and enable interrupts
+ *
+ ****************************************************************************/
+
 EXTERN irqstate_t up_irq_enable(void);
 
+/****************************************************************************
+ * Name: up_irq_save
+ *
+ * Description:
+ *   Save the current interrupt state and disable interrupts.
+ *
+ * Input Parameters:
+ *   None
+ *
+ * Returned Value:
+ *   Interrupt state prior to disabling interrupts.
+ *
+ ****************************************************************************/
+
+EXTERN irqstate_t up_irq_save(void);

Review comment:
       @gustavonihei - it may very well be that the added lines in arch/risc-v/include/bl602/irq.h  are not needed but the inclusion into arch.h is conflicting with the arch specific deceleration.  




-- 
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.

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