xiaoxiang781216 commented on code in PR #10605: URL: https://github.com/apache/nuttx/pull/10605#discussion_r1348459819
########## include/nuttx/spinlock.h: ########## @@ -84,6 +107,10 @@ typedef uint8_t spinlock_t; # define __SP_UNLOCK_FUNCTION 1 #endif +#ifdef CONFIG_TICKET_SPINLOCK +# define __SP_UNLOCK_FUNCTION 1 Review Comment: I mean this: ``` #if !defined(__SP_UNLOCK_FUNCTION) && (defined(CONFIG_TICKET_SPINLOCK) || \\ defined(CONFIG_SCHED_INSTRUMENTATION_SPINLOCKS)) # define __SP_UNLOCK_FUNCTION 1 #endif ``` ########## include/nuttx/spinlock.h: ########## @@ -84,6 +107,10 @@ typedef uint8_t spinlock_t; # define __SP_UNLOCK_FUNCTION 1 #endif +#ifdef CONFIG_TICKET_SPINLOCK +# define __SP_UNLOCK_FUNCTION 1 Review Comment: I mean this: ``` #if !defined(__SP_UNLOCK_FUNCTION) && (defined(CONFIG_TICKET_SPINLOCK) || \\ defined(CONFIG_SCHED_INSTRUMENTATION_SPINLOCKS)) # define __SP_UNLOCK_FUNCTION 1 #endif ``` -- 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