Re: [PR] add spinlock_type.h [nuttx]

2024-11-21 Thread via GitHub
hujun260 commented on PR #14801: URL: https://github.com/apache/nuttx/pull/14801#issuecomment-2490976340 > let's hold this patch until the recursive happen? ok -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] add spinlock_type.h [nuttx]

2024-11-21 Thread via GitHub
xiaoxiang781216 commented on PR #14801: URL: https://github.com/apache/nuttx/pull/14801#issuecomment-2490966630 let's hold this patch until the recursive happen? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] add spinlock_type.h [nuttx]

2024-11-20 Thread via GitHub
hujun260 commented on PR #14801: URL: https://github.com/apache/nuttx/pull/14801#issuecomment-2489922917 > > add spinlock_type.h reason: Due to incomplete handling of spinlock_t in arch/spinlock.h, it should not be used directly by external code. Furthermore, because pthread.h and nuttx/spi

Re: [PR] add spinlock_type.h [nuttx]

2024-11-17 Thread via GitHub
xiaoxiang781216 commented on PR #14801: URL: https://github.com/apache/nuttx/pull/14801#issuecomment-2481262693 > add spinlock_type.h reason: Due to incomplete handling of spinlock_t in arch/spinlock.h, it should not be used directly by external code. Furthermore, because pthread.h and nutt

[PR] add spinlock_type.h [nuttx]

2024-11-15 Thread via GitHub
hujun260 opened a new pull request, #14801: URL: https://github.com/apache/nuttx/pull/14801 ## Summary add spinlock_type.h reason: Due to incomplete handling of spinlock_t in arch/spinlock.h, it should not be used directly by external code. Furthermore, because pthread.h and

Re: [PR] add spinlock_type.h [nuttx]

2024-11-14 Thread via GitHub
nuttxpr commented on PR #14801: URL: https://github.com/apache/nuttx/pull/14801#issuecomment-2477847770 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No. This PR description lacks crucial details required