[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-05 Thread GitBox
xiaoxiang781216 opened a new pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179 ## Summary since it's common implementation can be used in other arch too ## Impact Minor, reorganize the layout ## Testing Pass CI -- This is an automated

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5178: arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S

2022-01-05 Thread GitBox
xiaoxiang781216 opened a new pull request #5178: URL: https://github.com/apache/incubator-nuttx/pull/5178 ## Summary to align with up_vfork_arm.S naming style ## Impact No, rename only ## Testing Pass CI -- This is an automated message from the Apache Git Servic

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4575: net/devif: Fix the memory leak in case of netdev isn't alive

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #4575: URL: https://github.com/apache/incubator-nuttx/pull/4575#issuecomment-1006261687 > > @xiaoxiang781216 I tried to find if changing dev->d_flags is synchronized to net_lock() anywhere. It looks like dev->d_flags can be modified (e.g. by psock_

[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-05 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779272625 ## File path: sched/semaphore/sem_holder.c ## @@ -1200,4 +1211,31 @@ int nxsem_nfreeholders(void) } #endif +/

[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-05 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779272504 ## File path: sched/semaphore/sem_holder.c ## @@ -187,35 +194,36 @@ nxsem_findorallocateholder(sem_t *sem, FAR struct tcb_s *htcb) static i

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

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#issuecomment-1006240215 @AlanRosenthal do you notice this warning from https://github.com/apache/incubator-nuttx/runs/4716428060?check_suite_focus=true: ``` Makefile:326: warning:

[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-05 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779261590 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #961: Typo fixes.

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

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #961: Typo fixes.

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #961: URL: https://github.com/apache/incubator-nuttx-apps/pull/961#discussion_r779261175 ## File path: fsutils/mkfatfs/Kconfig ## @@ -11,7 +11,7 @@ config FSUTILS_MKFATFS ---help--- Enables support f

[incubator-nuttx-apps] branch master updated: Typo fixes.

2022-01-05 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-apps.git The following commit(s) were added to refs/heads/master by this push: new bc5d803 Typo fixes. bc5d803 is

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5175: Minor fix for input Kconfig

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175#discussion_r779258518 ## File path: drivers/input/Kconfig ## @@ -14,7 +14,7 @@ menuconfig INPUT if INPUT config INPUT_MOUSE - bool "Enable mouse suppo

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
zhaoxiu-zeng commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1006219723 > > > Another real case: #5170, both case will generate hard fault. > > > > > > I run ostest with #5171, no such assertions were found. > > I think pri

[incubator-nuttx-website] branch asf-site updated: Publishing web: adbba922a8890b650c27ecac31f9d26fe25e1ec4 docs: 41c727f2513fdaf464e757424dc811b820f2425a

2022-01-05 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 a5c64d9 Publishing web

[incubator-nuttx] branch master updated: Revert "ci: Freeze pipenv version for avoiding issue on documentation generation"

2022-01-05 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 41c727f Revert "ci: Freeze pipenv

[GitHub] [incubator-nuttx] Ouss4 merged pull request #5177: Revert pipenv version freeze

2022-01-05 Thread GitBox
Ouss4 merged pull request #5177: URL: https://github.com/apache/incubator-nuttx/pull/5177 -- 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] Ouss4 commented on pull request #5176: Cleanup Context/Dirlinks targets

2022-01-05 Thread GitBox
Ouss4 commented on pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#issuecomment-1006157092 @AlanRosenthal The custom-board logic refactoring looks good, however, did you test with a custom out-of-tree board? I don't expect any issues but our CI only tests in-

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
pkarashchenko commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1006118747 > > Another real case: #5170, both case will generate hard fault. > > I run ostest with https://github.com/apache/incubator-nuttx/pull/5171, no such asserti

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

2022-01-05 Thread GitBox
AlanRosenthal commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779162088 ## File path: tools/Unix.mk ## @@ -328,13 +328,20 @@ arch/dummy/Kconfig: @echo "CP: $@ to $(CHIP_KCONFIG)" $(Q) cp -f $(CHIP

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5176: Cleanup Context/Dirlinks targets

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779159668 ## File path: tools/Unix.mk ## @@ -328,13 +328,20 @@ arch/dummy/Kconfig: @echo "CP: $@ to $(CHIP_KCONFIG)" $(Q) cp -f $(CHIP_

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #5177: Revert pipenv version freeze

2022-01-05 Thread GitBox
gustavonihei opened a new pull request #5177: URL: https://github.com/apache/incubator-nuttx/pull/5177 ## Summary This PR intends to revert the changes from https://github.com/apache/incubator-nuttx/pull/4807, then required to fix a regression introduced by the 2021.11.05 version of pip

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5176: Cleanup Context/Dirlinks targets

2022-01-05 Thread GitBox
pkarashchenko commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779149945 ## File path: tools/Unix.mk ## @@ -589,36 +579,43 @@ pass2dep: context tools/mkdeps$(HOSTEXEEXT) tools/cnvwindeps$(HOSTEXEEXT) config:

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

2022-01-05 Thread GitBox
AlanRosenthal commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779149800 ## File path: tools/Unix.mk ## @@ -589,36 +579,43 @@ pass2dep: context tools/mkdeps$(HOSTEXEEXT) tools/cnvwindeps$(HOSTEXEEXT) config:

[GitHub] [incubator-nuttx] davids5 commented on a change in pull request #5175: Minor fix for input Kconfig

2022-01-05 Thread GitBox
davids5 commented on a change in pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175#discussion_r779149602 ## File path: drivers/input/Kconfig ## @@ -14,7 +14,7 @@ menuconfig INPUT if INPUT config INPUT_MOUSE - bool "Enable mouse support" Rev

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5176: Cleanup Context/Dirlinks targets

2022-01-05 Thread GitBox
pkarashchenko commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779146121 ## File path: tools/Unix.mk ## @@ -589,36 +579,43 @@ pass2dep: context tools/mkdeps$(HOSTEXEEXT) tools/cnvwindeps$(HOSTEXEEXT) config:

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

2022-01-05 Thread GitBox
AlanRosenthal commented on a change in pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176#discussion_r779144686 ## File path: boards/Makefile ## @@ -94,19 +74,12 @@ makedepfile: $(CSRCS:.c=.ddc) $(ASRCS:.S=.dds) $(CXXSRCS:.cxx=.ddx) depend: .depend

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

2022-01-05 Thread GitBox
AlanRosenthal opened a new pull request #5176: URL: https://github.com/apache/incubator-nuttx/pull/5176 ## Summary Previously the context rule and dirlinks rule were doing the same thing. * the `context` target called other `context` targets in subdirectories. * the `context` target

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
acassis commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779108976 ## File path: arch/ceva/include/stdarg.h ## @@ -0,0 +1,41 @@ +/ + * ar

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
acassis commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779104609 ## File path: arch/ceva/include/xm6/irq.h ## @@ -0,0 +1,256 @@ +/ + *

[incubator-nuttx] branch master updated (6053de2 -> 42ffae2)

2022-01-05 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 6053de2 tools: Fix parallel build issue due to missing dependency add 10a8dc9 tools/esp: Prevent res

[GitHub] [incubator-nuttx] acassis merged pull request #5173: espressif: Fix build system issues related to Flash Encryption

2022-01-05 Thread GitBox
acassis merged pull request #5173: URL: https://github.com/apache/incubator-nuttx/pull/5173 -- 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: commit

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5175: Minor fix for input Kconfig

2022-01-05 Thread GitBox
acassis commented on a change in pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175#discussion_r779102212 ## File path: drivers/input/Kconfig ## @@ -14,7 +14,7 @@ menuconfig INPUT if INPUT config INPUT_MOUSE - bool "Enable mouse support" Rev

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779071327 ## File path: arch/ceva/include/arch.h ## @@ -0,0 +1,107 @@ +/ +

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779071100 ## File path: arch/ceva/include/arch.h ## @@ -0,0 +1,107 @@ +/ +

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779070913 ## File path: arch/ceva/Kconfig ## @@ -0,0 +1,133 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfi

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779070567 ## File path: arch/ceva/Kconfig ## @@ -0,0 +1,133 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfi

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779070441 ## File path: arch/ceva/Kconfig ## @@ -0,0 +1,133 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfi

[GitHub] [incubator-nuttx] a-lunev commented on pull request #4575: net/devif: Fix the memory leak in case of netdev isn't alive

2022-01-05 Thread GitBox
a-lunev commented on pull request #4575: URL: https://github.com/apache/incubator-nuttx/pull/4575#issuecomment-1005946086 > @xiaoxiang781216 I tried to find if changing dev->d_flags is synchronized to net_lock() anywhere. It looks like dev->d_flags can be modified (e.g. by psock_ioctl()) a

[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-05 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779019294 ## File path: sched/semaphore/sem_holder.c ## @@ -86,30 +87,36 @@ static inline FAR struct semholder_s *nxsem_allocholder(sem_t *sem)

[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-05 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779009113 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ 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-05 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r778982536 ## File path: sched/semaphore/sem_holder.c ## @@ -187,35 +194,36 @@ nxsem_findorallocateholder(sem_t *sem, FAR struct tcb_s *htcb) static

[GitHub] [incubator-nuttx-apps] fjpanag commented on a change in pull request #961: Typo fixes.

2022-01-05 Thread GitBox
fjpanag commented on a change in pull request #961: URL: https://github.com/apache/incubator-nuttx-apps/pull/961#discussion_r778964201 ## File path: fsutils/mkfatfs/Kconfig ## @@ -11,7 +11,7 @@ config FSUTILS_MKFATFS ---help--- Enables support for the m

[GitHub] [incubator-nuttx] AlanRosenthal commented on pull request #5174: tools: Fix parallel build issue due to missing dependency

2022-01-05 Thread GitBox
AlanRosenthal commented on pull request #5174: URL: https://github.com/apache/incubator-nuttx/pull/5174#issuecomment-1005871823 > @AlanRosenthal PTAL, we've identified this issue in our internal CI. LGTM! -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5175: Minor fix for input Kconfig

2022-01-05 Thread GitBox
pkarashchenko commented on a change in pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175#discussion_r778951401 ## File path: drivers/input/Kconfig ## @@ -14,7 +14,7 @@ menuconfig INPUT if INPUT config INPUT_MOUSE - bool "Enable mouse support

[GitHub] [incubator-nuttx-apps] davids5 commented on a change in pull request #961: Typo fixes.

2022-01-05 Thread GitBox
davids5 commented on a change in pull request #961: URL: https://github.com/apache/incubator-nuttx-apps/pull/961#discussion_r778926842 ## File path: fsutils/mkfatfs/Kconfig ## @@ -11,7 +11,7 @@ config FSUTILS_MKFATFS ---help--- Enables support for the m

[GitHub] [incubator-nuttx-apps] fjpanag opened a new pull request #961: Typo fixes.

2022-01-05 Thread GitBox
fjpanag opened a new pull request #961: URL: https://github.com/apache/incubator-nuttx-apps/pull/961 ## Summary Several typo fixes, as indicated by [typos](https://github.com/crate-ci/typos). ## Impact There shouldn't be any functional change. Mostly comments and d

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5174: tools: Fix parallel build issue due to missing dependency

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5174: URL: https://github.com/apache/incubator-nuttx/pull/5174 -- 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: tools: Fix parallel build issue due to missing dependency

2022-01-05 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 6053de2 tools: Fix parallel build i

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5175: Minor fix for input Kconfig

2022-01-05 Thread GitBox
xiaoxiang781216 opened a new pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175 ## Summary - input: Remove prompt string from INPUT_MOUSE - usbhost: Fix typo error in Kconfig(MOUSE->INPUT_MOUSE) ## Impact Minor ## Testing Pass CI

[GitHub] [incubator-nuttx] gustavonihei merged pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
gustavonihei merged pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530 -- 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: c

[incubator-nuttx] branch master updated: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread gustavonihei
This is an automated email from the ASF dual-hosted git repository. gustavonihei 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 fb92621 drivers/mmcsd: Add RPMB

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530#discussion_r778855651 ## File path: include/nuttx/mmcsd.h ## @@ -27,6 +27,47 @@ #include +#include +#include + +/**

[GitHub] [incubator-nuttx] gustavonihei commented on pull request #5174: tools: Fix parallel build issue due to missing dependency

2022-01-05 Thread GitBox
gustavonihei commented on pull request #5174: URL: https://github.com/apache/incubator-nuttx/pull/5174#issuecomment-1005724272 @AlanRosenthal PTAL, we've identified this issue in our internal CI. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #5174: tools: Fix parallel build issue due to missing dependency

2022-01-05 Thread GitBox
gustavonihei opened a new pull request #5174: URL: https://github.com/apache/incubator-nuttx/pull/5174 ## Summary This PR intends to fix an issue with parallel build due to missing dependency on the `include/arch/chip:` Make rule. ```bash $ ./tools/configure.sh esp32-wrover-ki

[incubator-nuttx] branch master updated (e27962a -> a983cd2)

2022-01-05 Thread davids5
This is an automated email from the ASF dual-hosted git repository. davids5 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from e27962a net/devif: Fix the memory leak in case of netdev isn't alive new 6fe95d8 stm32: add SocketCA

[incubator-nuttx] 02/02: boards/stm32: register CAN character device if configured

2022-01-05 Thread davids5
This is an automated email from the ASF dual-hosted git repository. davids5 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit a983cd23777fb4596a140a15375012874e81c9a0 Author: raiden00pl AuthorDate: Wed Jan 5 10:47:05 2022 +0100 boa

[incubator-nuttx] 01/02: stm32: add SocketCAN support, based on stm32_can.c

2022-01-05 Thread davids5
This is an automated email from the ASF dual-hosted git repository. davids5 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 6fe95d831499a6c907261394023c242651b65116 Author: raiden00pl AuthorDate: Wed Jan 5 10:28:47 2022 +0100 stm

[GitHub] [incubator-nuttx] davids5 merged pull request #5169: stm32: add SocketCAN support

2022-01-05 Thread GitBox
davids5 merged pull request #5169: URL: https://github.com/apache/incubator-nuttx/pull/5169 -- 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: commit

[GitHub] [incubator-nuttx] anchao merged pull request #4575: net/devif: Fix the memory leak in case of netdev isn't alive

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

[incubator-nuttx] branch master updated: net/devif: Fix the memory leak in case of netdev isn't alive

2022-01-05 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer 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 e27962a net/devif: Fix the memory leak

[GitHub] [incubator-nuttx] anchao commented on pull request #4575: net/devif: Fix the memory leak in case of netdev isn't alive

2022-01-05 Thread GitBox
anchao commented on pull request #4575: URL: https://github.com/apache/incubator-nuttx/pull/4575#issuecomment-1005709861 LGTM -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5172: fs/procfs: fix readdir loss last character

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5172: URL: https://github.com/apache/incubator-nuttx/pull/5172 -- 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: fix readdir loss last character

2022-01-05 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 f93c17b fs/procfs: fix readdir loss

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530#discussion_r778835894 ## File path: include/nuttx/mmcsd.h ## @@ -27,6 +27,47 @@ #include +#include +#include + +/***

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530#discussion_r778802563 ## File path: include/nuttx/mmcsd.h ## @@ -27,6 +27,47 @@ #include +#include +#include + +/**

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
gustavonihei commented on a change in pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530#discussion_r778802563 ## File path: include/nuttx/mmcsd.h ## @@ -27,6 +27,47 @@ #include +#include +#include + +/**

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #5173: espressif: Fix build system issues related to Flash Encryption

2022-01-05 Thread GitBox
gustavonihei opened a new pull request #5173: URL: https://github.com/apache/incubator-nuttx/pull/5173 ## Summary This PR intends to fix two build system issues related to the recently added feature of **Flash Encryption** on Espressif chips. Commit 23dec090 prevents the build system

[GitHub] [incubator-nuttx] Donny9 opened a new pull request #5172: fs/procfs: fix readdir loss last character

2022-01-05 Thread GitBox
Donny9 opened a new pull request #5172: URL: https://github.com/apache/incubator-nuttx/pull/5172 ## Summary fs/procfs: fix readdir loss last character nsh> ls /proc: 0/ 1/ 2/ 3/ 4/ 5/ meminf iobinf f/ ne/ sel/ uptim ver

[incubator-nuttx] branch master updated: signal: signal handler may cause task's state error

2022-01-05 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki 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 fb43fd7 signal: signal handler may c

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #5120: signal: signal handler may cause task's state error

2022-01-05 Thread GitBox
masayuki2009 merged pull request #5120: URL: https://github.com/apache/incubator-nuttx/pull/5120 -- 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: c

[GitHub] [incubator-nuttx-apps] masayuki2009 merged pull request #955: ostest: sighand.c: add sem_wait in signal handler

2022-01-05 Thread GitBox
masayuki2009 merged pull request #955: URL: https://github.com/apache/incubator-nuttx-apps/pull/955 -- 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-apps] branch master updated: ostest: sighand.c: add sem_wait in signal handler

2022-01-05 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 614b73d ostest: sighand.c: add

[GitHub] [incubator-nuttx-apps] masayuki2009 edited a comment on pull request #955: ostest: sighand.c: add sem_wait in signal handler

2022-01-05 Thread GitBox
masayuki2009 edited a comment on pull request #955: URL: https://github.com/apache/incubator-nuttx-apps/pull/955#issuecomment-1005647806 >@masayuki2009 cloud you try this patch with/without apache/incubator-nuttx#5120? I tried this PR with sabre-6quad:smp (QEMU) and confirmed that

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on pull request #955: ostest: sighand.c: add sem_wait in signal handler

2022-01-05 Thread GitBox
masayuki2009 commented on pull request #955: URL: https://github.com/apache/incubator-nuttx-apps/pull/955#issuecomment-1005647806 >@masayuki2009 cloud you try this patch with/without apache/incubator-nuttx#5120? I tried this PR with sabre-6quad:smp (QEMU) and confirmed that apache/

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4530: drivers/mmcsd: Add RPMB ioctl

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #4530: URL: https://github.com/apache/incubator-nuttx/pull/4530#issuecomment-1005643227 @Ouss4 and @gustavonihei could you help review this simple PR? -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5170: libc/pthread:pthread_barrierinit sem use pri_none

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5170: URL: https://github.com/apache/incubator-nuttx/pull/5170 -- 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 (f4d74b5 -> 9aa6916)

2022-01-05 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from f4d74b5 Rename `tools/Makefile.*` to have the `.mk` extension add 9aa6916 libc/pthread:pthread_bar

[incubator-nuttx-apps] branch master updated (04dd2d7 -> 4a14126)

2022-01-05 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git. from 04dd2d7 netutils/dhcpc: Fix dhcpc lease bug add 4a14126 ostest: fix assert when enable CONFIG

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #960: ostest: fix assert when enable CONFIG_PRIORITY_INHERITANCE

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

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
zhaoxiu-zeng commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1005623759 > Another real case: #5170, both case will generate hard fault. I run ostest with https://github.com/apache/incubator-nuttx/pull/5171, no such assertions wer

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
zhaoxiu-zeng commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1005614899 @pkarashchenko I have push the modification, let's discuss in the PR. https://github.com/apache/incubator-nuttx/pull/5171 -- This is an automated message f

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
zhaoxiu-zeng commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1005610175 > I have pushed the modificaiton, let's discuss in the PR. https://github.com/apache/incubator-nuttx/pull/5171 -- This is an automated message fro

[GitHub] [incubator-nuttx] zhaoxiu-zeng removed a comment on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
zhaoxiu-zeng removed a comment on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1005610175 > I have pushed the modificaiton, let's discuss in the PR. https://github.com/apache/incubator-nuttx/pull/5171 -- This is an automated mes

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

2022-01-05 Thread GitBox
zhaoxiu-zeng opened a new pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171 Add a list in TCB to track all semphores the task held, so we can release all holders when exit, so nxsched_verify_tcb is unnecessary. Signed-off-by: Zeng Zhaoxiu ## Su

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

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5164: URL: https://github.com/apache/incubator-nuttx/pull/5164#discussion_r778746479 ## File path: tools/testbuild.sh ## @@ -258,6 +258,10 @@ function build { fail=1 fi + # Verify that the context rule does not n

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1005603855 Another real case: https://github.com/apache/incubator-nuttx/pull/5170, both case will generate hard fault. -- This is an automated message from the Apache Gi

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

2022-01-05 Thread GitBox
Ouss4 commented on a change in pull request #5164: URL: https://github.com/apache/incubator-nuttx/pull/5164#discussion_r778719017 ## File path: tools/testbuild.sh ## @@ -258,6 +258,10 @@ function build { fail=1 fi + # Verify that the context rule does not need to be

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5167: stm32: add CAN error support

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5167: URL: https://github.com/apache/incubator-nuttx/pull/5167 -- 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] anjiahao1 opened a new pull request #5170: libc/pthread:pthread_barrierinit sem use pri_none

2022-01-05 Thread GitBox
anjiahao1 opened a new pull request #5170: URL: https://github.com/apache/incubator-nuttx/pull/5170 barrier is used protect resources. don't lock Signed-off-by: anjiahao ## Summary ## Impact ## Testing -- This is an automated message from the Apache Git Se

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5169: stm32: add SocketCAN support

2022-01-05 Thread GitBox
raiden00pl opened a new pull request #5169: URL: https://github.com/apache/incubator-nuttx/pull/5169 ## Summary - stm32: add SocketCAN support, based on stm32_can.c - boards/stm32: register CAN character device if configured ## Impact ## Testing nucleo-f302r8 and n

[incubator-nuttx] branch master updated: Rename `tools/Makefile.*` to have the `.mk` extension

2022-01-05 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 f4d74b5 Rename `tools/Makefile.*` t

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5163: Rename `tools/Makefile.*` to have the `.mk` extension

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5163: URL: https://github.com/apache/incubator-nuttx/pull/5163 -- 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: stm32: add CAN error support

2022-01-05 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 5b9b381 stm32: add CAN error suppor

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5168: SocketCAN: add CAN error definitions

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5168: URL: https://github.com/apache/incubator-nuttx/pull/5168 -- 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 (474205e -> a8ca1ec)

2022-01-05 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 474205e boards/nucleo-f446re: add procfs support new 09ae157 include/nuttx/can/can.h: fix typo

[incubator-nuttx] 01/02: include/nuttx/can/can.h: fix typo

2022-01-05 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 commit 09ae15755e3c34167ba4930861cae0568fec3d60 Author: raiden00pl AuthorDate: Wed Jan 5 10:10:20 2022 +0100 i

[incubator-nuttx] 02/02: SocketCAN: add CAN error definitions

2022-01-05 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 commit a8ca1ec026b4d61003f7d4ed3ea8d2f868f01b56 Author: raiden00pl AuthorDate: Wed Jan 5 10:11:16 2022 +0100 S

[incubator-nuttx] branch master updated: boards/nucleo-f446re: add procfs support

2022-01-05 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 474205e boards/nucleo-f446re: add p

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5166: boards/nucleo-f446re: add procfs support

2022-01-05 Thread GitBox
xiaoxiang781216 merged pull request #5166: URL: https://github.com/apache/incubator-nuttx/pull/5166 -- 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-apps] branch master updated (a1857b9 -> 04dd2d7)

2022-01-05 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git. from a1857b9 wireless/gs2200m: replace the preamble scratch to union add 04dd2d7 netutils/dhcpc: F

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #958: netutils/dhcpc: Fix dhcpc lease bug

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

  1   2   >