[GitHub] [nuttx] tmedicci commented on a diff in pull request #8415: esp32c3: fix cpuint issue

["tmedicci (via GitHub)" <gi...@apache.org>]

tmedicci commented on code in PR #8415:
URL: https://github.com/apache/nuttx/pull/8415#discussion_r1094991958


##########
arch/risc-v/src/esp32c3/esp32c3_irq.c:
##########
@@ -82,10 +82,18 @@
 #define ESP32C3_PRIO_INDEX(p)   ((p) - ESP32C3_MIN_PRIORITY)
 
 #ifdef CONFIG_ESP32C3_WIFI
-#  define ESP32C3_WIFI_RESERVE_INT  ((1 << ESP32C3_CPUINT_MAC | \
-                                      1 << ESP32C3_CPUINT_MAC_NMI))
+#  define ESP32C3_WIFI_RESERVE_INT  ((1 << ESP32C3_CPUINT_ALWAYS_RSVD | \
+                                      1 << ESP32C3_CPUINT_WMAC))
 #else
-#  define ESP32C3_WIFI_RESERVE_INT  0
+#  define ESP32C3_WIFI_RESERVE_INT  (1 << ESP32C3_CPUINT_ALWAYS_RSVD)
+#endif
+
+#ifdef CONFIG_ESP32C3_BLE
+#  define ESP32C3_BLE_RESERVE_INT ((1 << ESP32C3_CPUINT_ALWAYS_RSVD) | \
+                                   (1 << ESP32C3_CPUINT_BT_BB) | \
+                                   (1 << ESP32C3_CPUINT_RWBLE))
+#else
+#  define ESP32C3_BLE_RESERVE_INT  (1 << ESP32C3_CPUINT_ALWAYS_RSVD)

Review Comment:
   I think we should consider the Wi-Fi + BLE situation, shouldn't we?



-- 
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-unsubscribe@nuttx.apache.org

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