[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5198: xtensa: fix lack of float register save & resotre

2022-01-11 Thread GitBox
xiaoxiang781216 commented on pull request #5198: URL: https://github.com/apache/incubator-nuttx/pull/5198#issuecomment-1009713228 @Ouss4 could you take a look again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread GitBox
no1wudi commented on a change in pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r781958691 ## File path: arch/risc-v/src/common/riscv_swint.c ## @@ -116,6 +134,26 @@ static void dispatch_syscall(void) " ecall" /

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2822: arch/riscv: Unify 32/64 implementations and remove dead ports

2022-01-11 Thread GitBox
xiaoxiang781216 commented on pull request #2822: URL: https://github.com/apache/incubator-nuttx/pull/2822#issuecomment-1009767151 Let's track the merge in https://github.com/apache/incubator-nuttx/pull/2822 instead. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-nuttx] xiaoxiang781216 closed pull request #2822: arch/riscv: Unify 32/64 implementations and remove dead ports

2022-01-11 Thread GitBox
xiaoxiang781216 closed pull request #2822: URL: https://github.com/apache/incubator-nuttx/pull/2822 -- 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

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5202: libcxx-mini: operator new will assert when alloc failed.

2022-01-11 Thread GitBox
pkarashchenko merged pull request #5202: URL: https://github.com/apache/incubator-nuttx/pull/5202 -- 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:

[incubator-nuttx] branch master updated (40b1f0b -> 59c9620)

2022-01-11 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 40b1f0b libc: Add include/nuttx/input/x11_keysym.h add 59c9620 libcxx-mini: operator new will

[incubator-nuttx] branch master updated: xtensa: fix lack of float register save & resotre

2022-01-11 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 3cfc676 xtensa: fix lack of float

[GitHub] [incubator-nuttx] Ouss4 merged pull request #5198: xtensa: fix lack of float register save & resotre

2022-01-11 Thread GitBox
Ouss4 merged pull request #5198: URL: https://github.com/apache/incubator-nuttx/pull/5198 -- 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-

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r782059779 ## File path: arch/risc-v/include/irq.h ## @@ -31,21 +31,457 @@ /* Include chip-specific IRQ definitions (including IRQ numbers) */ +#i

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r782072589 ## File path: arch/risc-v/src/common/riscv_swint.c ## @@ -116,6 +134,26 @@ static void dispatch_syscall(void) " ecall"

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r782072589 ## File path: arch/risc-v/src/common/riscv_swint.c ## @@ -116,6 +134,26 @@ static void dispatch_syscall(void) " ecall"

[GitHub] [incubator-nuttx] yanyunyouyou commented on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
yanyunyouyou commented on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1009952699 ![无标题](https://user-images.githubusercontent.com/38059431/148948847-ab3f60f9-cd8b-4fbb-a63a-ce4cc6df2e55.png) It took me a long time to solve this problem. The m

[GitHub] [incubator-nuttx] yanyunyouyou removed a comment on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
yanyunyouyou removed a comment on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1009952699 ![无标题](https://user-images.githubusercontent.com/38059431/148948847-ab3f60f9-cd8b-4fbb-a63a-ce4cc6df2e55.png) It took me a long time to solve this proble

[GitHub] [incubator-nuttx] acassis commented on pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
acassis commented on pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#issuecomment-1009965058 @pkarashchenko please fix the coding style of "include/nuttx/tree.h" before apply this commit, since like the original file was already with coding style issues -- T

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #5203: xtensa/esp32s2: Fix missing parenthesis on macro expression

2022-01-11 Thread GitBox
gustavonihei opened a new pull request #5203: URL: https://github.com/apache/incubator-nuttx/pull/5203 ## Summary This PR intends to provide a simple fix to a missing parenthesis on the peripheral IRQ table description of ESP32-S2. ## Impact No impact, the fixed IRQ entry is cu

[GitHub] [incubator-nuttx] yanyunyouyou commented on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
yanyunyouyou commented on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1009968152 > @yanyunyouyou is the issue still relevant or you are satisfied with the answer? ![Uploading 无标题.png…]() It took me a long time to solve this problem. T

[GitHub] [incubator-nuttx] yanyunyouyou edited a comment on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
yanyunyouyou edited a comment on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1009968152 > @yanyunyouyou is the issue still relevant or you are satisfied with the answer? ![无标题](https://user-images.githubusercontent.com/38059431/14895252

[GitHub] [incubator-nuttx] acassis edited a comment on pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
acassis edited a comment on pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#issuecomment-1009965058 @pkarashchenko please fix the coding style of "include/nuttx/tree.h" before apply this commit, seams like the original file was already with coding style issues

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
pkarashchenko commented on pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#issuecomment-1009976604 > @pkarashchenko please fix the coding style of "include/nuttx/tree.h" before apply this commit, seams like the original file was already with coding style issues

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
acassis commented on a change in pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#discussion_r782161109 ## File path: arch/arm/include/armv6-m/irq.h ## @@ -219,8 +219,7 @@ struct xcptcontext /* Get/set the PRIMASK register */ -static inline uint

[incubator-nuttx] branch master updated: xtensa/esp32s2: Fix missing parenthesis on macro expression

2022-01-11 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new efca63e xtensa/esp32s2: Fix missing

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5203: xtensa/esp32s2: Fix missing parenthesis on macro expression

2022-01-11 Thread GitBox
xiaoxiang781216 merged pull request #5203: URL: https://github.com/apache/incubator-nuttx/pull/5203 -- 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

[incubator-nuttx] branch master updated: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 10bb48b arch/risc-v: Merge rv32im a

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

2022-01-11 Thread GitBox
xiaoxiang781216 merged pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 closed issue #2593: Merge rv32im and rv64gc into one

2022-01-11 Thread GitBox
xiaoxiang781216 closed issue #2593: URL: https://github.com/apache/incubator-nuttx/issues/2593 -- 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: com

[GitHub] [incubator-nuttx] hartmannathan commented on a change in pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
hartmannathan commented on a change in pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#discussion_r782311674 ## File path: arch/arm/include/armv6-m/irq.h ## @@ -219,8 +219,7 @@ struct xcptcontext /* Get/set the PRIMASK register */ -static inlin

[GitHub] [incubator-nuttx] hartmannathan commented on a change in pull request #5201: inline: switch from inline to inline_function

2022-01-11 Thread GitBox
hartmannathan commented on a change in pull request #5201: URL: https://github.com/apache/incubator-nuttx/pull/5201#discussion_r782312406 ## File path: arch/arm/include/armv6-m/irq.h ## @@ -219,8 +219,7 @@ struct xcptcontext /* Get/set the PRIMASK register */ -static inlin

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-01-11 Thread GitBox
xiaoxiang781216 opened a new pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204 ## Summary it's wrong to protect by au_exclsem in some case and remove ao_closing ## Impact ## Testing -- This is an automated message from the Apache Git Ser

[GitHub] [incubator-nuttx] AlanRosenthal opened a new issue #5205: Makefile Improvement Plan

2022-01-11 Thread GitBox
AlanRosenthal opened a new issue #5205: URL: https://github.com/apache/incubator-nuttx/issues/5205 Hi Folks, I've been working to improve the NuttX Makefiles over the last few weeks. See my contributions here: https://github.com/apache/incubator-nuttx/pulls?q=is%3Apr+is%3Aclosed+author%3AA

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5206: fgstm32: add FDCAN support

2022-01-11 Thread GitBox
raiden00pl opened a new pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206 ## Summary stm32: add FDCAN support based on PR #2987 For now only the character driver is supported. ## Impact ## Testing nucleo-g431rb and b-g431b-esc1

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5206: stm32: add FDCAN support

2022-01-11 Thread GitBox
acassis commented on a change in pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206#discussion_r782419571 ## File path: arch/arm/src/stm32/hardware/stm32_fdcan.h ## @@ -0,0 +1,586 @@ +/***

[GitHub] [incubator-nuttx] acassis commented on pull request #5206: stm32: add FDCAN support

2022-01-11 Thread GitBox
acassis commented on pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206#issuecomment-1010255659 @raiden00pl did you get it working on B-G431B-ESC1 board? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5207: fs/procfs: Remove the unnecessary strcmp

2022-01-11 Thread GitBox
xiaoxiang781216 opened a new pull request #5207: URL: https://github.com/apache/incubator-nuttx/pull/5207 ## Summary since the procfs already make the same check for us ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r782415335 ## File path: drivers/input/ajoystick.c ## @@ -331,10 +327,6 @@ static void ajoy_sample(FAR struct ajoy_upperhalf_s *priv) } }

[GitHub] [incubator-nuttx] AlanRosenthal closed pull request #5164: Add test to CI to verify that the `context` rule doesn't need to be rebuilt between runs

2022-01-11 Thread GitBox
AlanRosenthal closed pull request #5164: URL: https://github.com/apache/incubator-nuttx/pull/5164 -- 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:

[GitHub] [incubator-nuttx] AlanRosenthal commented on pull request #5164: Add test to CI to verify that the `context` rule doesn't need to be rebuilt between runs

2022-01-11 Thread GitBox
AlanRosenthal commented on pull request #5164: URL: https://github.com/apache/incubator-nuttx/pull/5164#issuecomment-1010432587 I've decided to approach this a bit different, see https://github.com/apache/incubator-nuttx/issues/5205 -- This is an automated message from the Apache Git Ser

[GitHub] [incubator-nuttx] AlanRosenthal closed pull request #5176: Cleanup Context/Dirlinks targets

2022-01-11 Thread GitBox
AlanRosenthal closed pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176 -- 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:

[GitHub] [incubator-nuttx] AlanRosenthal commented on pull request #5176: Cleanup Context/Dirlinks targets

2022-01-11 Thread GitBox
AlanRosenthal commented on pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#issuecomment-1010432893 I've decided to approach this a bit different, see https://github.com/apache/incubator-nuttx/issues/5205 -- This is an automated message from the Apache Git Ser

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782584997 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +944,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] AlanRosenthal opened a new pull request #5208: Cleanup boards/Makefile

2022-01-11 Thread GitBox
AlanRosenthal opened a new pull request #5208: URL: https://github.com/apache/incubator-nuttx/pull/5208 ## Summary There was logic in `tools/Config.mk` which was duplicated in `boards/Makefile`. This PR removes the duplicated logic in `boards/Makefile` ## Impact Small makefi

[GitHub] [incubator-nuttx] AlanRosenthal commented on pull request #5208: Cleanup boards/Makefile

2022-01-11 Thread GitBox
AlanRosenthal commented on pull request #5208: URL: https://github.com/apache/incubator-nuttx/pull/5208#issuecomment-1010461586 - [ ] update Win.mk after CI passes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [incubator-nuttx] pkarashchenko commented on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
pkarashchenko commented on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1010462306 Yes. `mount` does not know what is `-t fatfs` and `vfat` does not have `-o forceformat` option (it is from `littlefs`). So if I understand correctly the issue is re

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5206: stm32: add FDCAN support

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206#discussion_r782617537 ## File path: arch/arm/src/stm32/stm32_fdcan.h ## @@ -0,0 +1,91 @@ +/***

[incubator-nuttx-website] branch asf-site updated: Publishing web: adbba922a8890b650c27ecac31f9d26fe25e1ec4 docs: 10bb48b9b40a093a73bf04ce49582aeca441c590

2022-01-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new a45cf65 Publishing web

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5206: stm32: add FDCAN support

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206#discussion_r782619373 ## File path: arch/arm/src/stm32/stm32_fdcan.c ## @@ -0,0 +1,3458 @@ +/*

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5206: stm32: add FDCAN support

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5206: URL: https://github.com/apache/incubator-nuttx/pull/5206#discussion_r782619373 ## File path: arch/arm/src/stm32/stm32_fdcan.c ## @@ -0,0 +1,3458 @@ +/*

[GitHub] [incubator-nuttx] pkarashchenko commented on issue #4674: RISC-V Toolchain hardcoded in CROSSDEV to riscv64-unknown-elf-

2022-01-11 Thread GitBox
pkarashchenko commented on issue #4674: URL: https://github.com/apache/incubator-nuttx/issues/4674#issuecomment-1010542946 The best way to overcome link issue is to involve link process indirectly via compiler. Probably need to inspect makefiles and "translate" all link flags and add `-Xl

[GitHub] [incubator-nuttx] cederom commented on issue #4674: RISC-V Toolchain hardcoded in CROSSDEV to riscv64-unknown-elf-

2022-01-11 Thread GitBox
cederom commented on issue #4674: URL: https://github.com/apache/incubator-nuttx/issues/4674#issuecomment-1010548393 Thanks @pkarashchenko :-) At the moment I am playing with MicroPython for rapid development in my current project. When it works I will get back to NuttX and porting Micro

[GitHub] [incubator-nuttx] yanyunyouyou commented on issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
yanyunyouyou commented on issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020#issuecomment-1010548610 Yes,it can be closed now。Thank you。 -- 原始邮件 -- 发件人: "Petro ***@***.***>; 发送时间: 2022年1月12日(星期三) 上午7:34 收件人: *

[GitHub] [incubator-nuttx] pkarashchenko closed issue #5020: ramdisk can be created,but mkfatfs /dev/ram0 falied with errno = 6

2022-01-11 Thread GitBox
pkarashchenko closed issue #5020: URL: https://github.com/apache/incubator-nuttx/issues/5020 -- 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: commi

[GitHub] [incubator-nuttx] no1wudi opened a new pull request #5209: arch/risc-v: Refine riscv_vectors.S

2022-01-11 Thread GitBox
no1wudi opened a new pull request #5209: URL: https://github.com/apache/incubator-nuttx/pull/5209 ## Summary Remove duplicated file ## Impact Refactor only ## Testing CI -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5209: arch/risc-v: Refine riscv_vectors.S

2022-01-11 Thread GitBox
no1wudi commented on pull request #5209: URL: https://github.com/apache/incubator-nuttx/pull/5209#issuecomment-1010616723 Force push to trigger CI. -- 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 t

[GitHub] [incubator-nuttx] yamt commented on pull request #4977: arch/sim: add CONFIG_SIM_STACKSIZE_ADJUSTMENT to reduce variability

2022-01-11 Thread GitBox
yamt commented on pull request #4977: URL: https://github.com/apache/incubator-nuttx/pull/4977#issuecomment-1010632740 i'm not sure if this is a good idea because * this breaks existing configurations * it makes stack-related api very confusing (eg. pthread_attr_setstack and pthread_a

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5207: fs/procfs: Remove the unnecessary strcmp

2022-01-11 Thread GitBox
pkarashchenko merged pull request #5207: URL: https://github.com/apache/incubator-nuttx/pull/5207 -- 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:

[incubator-nuttx] branch master updated: fs/procfs: Remove the unnecessary strcmp

2022-01-11 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 1b77ae8 fs/procfs: Remove the u

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782747575 ## File path: sched/semaphore/sem_holder.c ## @@ -263,23 +276,16 @@ static int nxsem_foreachholder(FAR sem_t *sem, holderhandler_t handler,

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782747575 ## File path: sched/semaphore/sem_holder.c ## @@ -263,23 +276,16 @@ static int nxsem_foreachholder(FAR sem_t *sem, holderhandler_t handler,

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782747575 ## File path: sched/semaphore/sem_holder.c ## @@ -263,23 +276,16 @@ static int nxsem_foreachholder(FAR sem_t *sem, holderhandler_t handler,

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782747575 ## File path: sched/semaphore/sem_holder.c ## @@ -263,23 +276,16 @@ static int nxsem_foreachholder(FAR sem_t *sem, holderhandler_t handler,

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782775416 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +944,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb =

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782787550 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +944,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-11 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r782787550 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +944,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb