[GitHub] [incubator-nuttx] v01d commented on a change in pull request #2566: arch/risc-v: Add BL602 support

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

v01d commented on a change in pull request #2566:
URL: https://github.com/apache/incubator-nuttx/pull/2566#discussion_r616841863



##########
File path: include/nuttx/arch.h
##########
@@ -1353,6 +1354,39 @@ void up_irqinitialize(void);
 
 bool up_interrupt_context(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.
+ *
+ ****************************************************************************/
+
+irqstate_t up_irq_save(void);
+
+/****************************************************************************
+ * Name: up_irq_restore
+ *
+ * Description:
+ *   Restore the previous irq state (i.e., the one previously
+ *   returned by up_irq_save())
+ *
+ * Input Parameters:
+ *   irqstate - The interrupt state to be restored.
+ *
+ * Returned Value:
+ *   None
+ *
+ ****************************************************************************/
+
+void up_irq_restore(irqstate_t irqstate);
+

Review comment:
       I guess we missed this during review. From the looks of it this declaration should only be in arch-dependant header, as it is done for other arch. You can make a PR to remove these and we can see if BL602 still builds or it requires further fix.




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