[GitHub] [incubator-nuttx-apps] coffee809721232 opened a new pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
coffee809721232 opened a new pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958 ## Summary netutils/dhcpc: Fix dhcpc lease bug Signed-off-by: songlinzhang songlinzh...@xiaomi.com ## Impact dhcpc lease management ## Testing call dhcpc_request

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4881: libc: Update include/nuttx/input/x11_keysymdef.h to the latest version

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#discussion_r778614713 ## File path: drivers/video/vnc/vnc_keymap.c ## @@ -29,13 +29,8 @@ #include #include - -#define XK_MISCELLANY 1 /* Select X11 charac

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4881: libc: Update include/nuttx/input/x11_keysymdef.h to the latest version

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#discussion_r778614713 ## File path: drivers/video/vnc/vnc_keymap.c ## @@ -29,13 +29,8 @@ #include #include - -#define XK_MISCELLANY 1 /* Select X11 charac

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958#discussion_r778620373 ## File path: netutils/dhcpc/dhcpc.c ## @@ -458,6 +458,7 @@ static void *dhcpc_run(void *args) struct dhcpc_state result; int re

[GitHub] [incubator-nuttx-apps] coffee809721232 commented on a change in pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
coffee809721232 commented on a change in pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958#discussion_r778622368 ## File path: netutils/dhcpc/dhcpc.c ## @@ -458,6 +458,7 @@ static void *dhcpc_run(void *args) struct dhcpc_state result; int re

[GitHub] [incubator-nuttx-apps] coffee809721232 commented on pull request #904: netutils/dhcpc: add non-blocking interface

2022-01-05 Thread GitBox
coffee809721232 commented on pull request #904: URL: https://github.com/apache/incubator-nuttx-apps/pull/904#issuecomment-1005475381 > Was there a plan to update `netinit_net_bringup` to use `dhcpc_request_async`? and/or add some built-in commands like `ntpc` has to start/stop a dhcpc dae

[GitHub] [incubator-nuttx-apps] coffee809721232 edited a comment on pull request #904: netutils/dhcpc: add non-blocking interface

2022-01-05 Thread GitBox
coffee809721232 edited a comment on pull request #904: URL: https://github.com/apache/incubator-nuttx-apps/pull/904#issuecomment-1005475381 > Was there a plan to update `netinit_net_bringup` to use `dhcpc_request_async`? and/or add some built-in commands like `ntpc` has to start/stop a dh

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958#discussion_r778628718 ## File path: netutils/dhcpc/dhcpc.c ## @@ -458,6 +458,7 @@ static void *dhcpc_run(void *args) struct dhcpc_state result; int re

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
anchao opened a new pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959 ## Summary wireless/gs2200m: replace the preamble scratch to union ## Impact fix the preamble scratch buffer issue ## Testing N/A, I do not have the wifi mo

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
xiaoxiang781216 commented on a change in pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958#discussion_r778629324 ## File path: netutils/dhcpc/dhcpc.c ## @@ -458,6 +458,7 @@ static void *dhcpc_run(void *args) struct dhcpc_state result; int re

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959#issuecomment-1005480661 @masayuki2009 could you try the change? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] anchao commented on pull request #5154: net/usrsock: replace xid type to uint64_t to avoid limitation of connections

2022-01-05 Thread GitBox
anchao commented on pull request #5154: URL: https://github.com/apache/incubator-nuttx/pull/5154#issuecomment-1005480711 > @xiaoxiang781216 @anchao Yes, I use the latest apps but the renew command still does not work with gs2200m daemon. @masayuki2009 san, could you please to try thi

[GitHub] [incubator-nuttx] anchao edited a comment on pull request #5154: net/usrsock: replace xid type to uint64_t to avoid limitation of connections

2022-01-05 Thread GitBox
anchao edited a comment on pull request #5154: URL: https://github.com/apache/incubator-nuttx/pull/5154#issuecomment-1005480711 > @xiaoxiang781216 @anchao Yes, I use the latest apps but the renew command still does not work with gs2200m daemon. @masayuki2009 san, could you please to

[GitHub] [incubator-nuttx-apps] coffee809721232 commented on a change in pull request #958: netutils/dhcpc: Fix dhcpc lease bug

2022-01-05 Thread GitBox
coffee809721232 commented on a change in pull request #958: URL: https://github.com/apache/incubator-nuttx-apps/pull/958#discussion_r778636018 ## File path: netutils/dhcpc/dhcpc.c ## @@ -458,6 +458,7 @@ static void *dhcpc_run(void *args) struct dhcpc_state result; int re

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on a change in pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
masayuki2009 commented on a change in pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959#discussion_r778640247 ## File path: wireless/gs2200m/gs2200m_main.c ## @@ -389,17 +406,17 @@ static int usrsock_request(int fd, FAR struct gs2200m_s *priv)

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5166: boards/nucleo-f446re: add procfs support

2022-01-05 Thread GitBox
raiden00pl opened a new pull request #5166: URL: https://github.com/apache/incubator-nuttx/pull/5166 ## Summary boards/nucleo-f446re: add procfs support ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-nuttx-apps] anchao commented on a change in pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
anchao commented on a change in pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959#discussion_r778644757 ## File path: wireless/gs2200m/gs2200m_main.c ## @@ -389,17 +406,17 @@ static int usrsock_request(int fd, FAR struct gs2200m_s *priv) r

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5167: stm32: add CAN error support

2022-01-05 Thread GitBox
raiden00pl opened a new pull request #5167: URL: https://github.com/apache/incubator-nuttx/pull/5167 ## Summary stm32: add CAN error support. RX/TX buffer overflow errors not supported yet ## Impact ## Testing nucleo-f302r8 and nucleo-f446re -- This is an automa

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

2022-01-05 Thread GitBox
xiaoxiang781216 commented on pull request #5163: URL: https://github.com/apache/incubator-nuttx/pull/5163#issuecomment-1005506757 @Ouss4 do you think the rename is good to merge? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5168: SocketCAN: add CAN error definitions

2022-01-05 Thread GitBox
raiden00pl opened a new pull request #5168: URL: https://github.com/apache/incubator-nuttx/pull/5168 ## Summary SocketCAN: add CAN error definitions ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on a change in pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
masayuki2009 commented on a change in pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959#discussion_r778658794 ## File path: wireless/gs2200m/gs2200m_main.c ## @@ -389,17 +406,17 @@ static int usrsock_request(int fd, FAR struct gs2200m_s *priv)

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on pull request #957: ntpc: optimize stack used

2022-01-05 Thread GitBox
masayuki2009 commented on pull request #957: URL: https://github.com/apache/incubator-nuttx-apps/pull/957#issuecomment-1005523807 @Donny9 The latest commit reduces the stack size. Thanks for the fix. ``` 1414 100 RR Task--- Waiting Signal 0

[incubator-nuttx-apps] branch master updated (b9a6dc2 -> 64ffdb4)

2022-01-05 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git. from b9a6dc2 netutils/netcat: implemented NETUTILS_NETCAT_SENDFILE option. This option enables using sen

[GitHub] [incubator-nuttx-apps] masayuki2009 merged pull request #957: ntpc: optimize stack used

2022-01-05 Thread GitBox
masayuki2009 merged pull request #957: URL: https://github.com/apache/incubator-nuttx-apps/pull/957 -- 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] 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_r778687313 ## 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-apps] Donny9 opened a new pull request #960: ostest: fix assert when enable CONFIG_PRIORITY_INHERITANCE

2022-01-05 Thread GitBox
Donny9 opened a new pull request #960: URL: https://github.com/apache/incubator-nuttx-apps/pull/960 ## Summary ostest: fix assert when enable CONFIG_PRIORITY_INHERITANCE [ 96.555456] [265] ap: up_assert: Assertion failed at file:semaphore/sem_holder.c line: 112 task: pt-0x5b09

[incubator-nuttx-apps] branch master updated (64ffdb4 -> a1857b9)

2022-01-05 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git. from 64ffdb4 ntpc: optimize stack used add a1857b9 wireless/gs2200m: replace the preamble scratch t

[GitHub] [incubator-nuttx-apps] masayuki2009 merged pull request #959: wireless/gs2200m: replace the preamble scratch to union

2022-01-05 Thread GitBox
masayuki2009 merged pull request #959: URL: https://github.com/apache/incubator-nuttx-apps/pull/959 -- 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 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_r778700904 ## 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-100900 Here is a case: https://github.com/apache/incubator-nuttx-apps/pull/960 -- This is an automated message from the Apache Git Service. To respond to the message,

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

2022-01-05 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-100900 Here is a real case: https://github.com/apache/incubator-nuttx-apps/pull/960 -- This is an automated message from the Apache Git Service. To respond to

[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

[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] 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] 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

[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 (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

[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: 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 #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: 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] 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

[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] 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 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] 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] 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] 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 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-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

[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

[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

[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

[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-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-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

[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 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

[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

[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] 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

[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] 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] 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 + +/***

[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 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

[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

[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 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

[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

[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

[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] 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

[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

[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 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 + +/**

[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 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

[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

[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 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

[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-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] 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] 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-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] 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] 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_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] 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] 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] 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_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_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_r779071327 ## File path: arch/ceva/include/arch.h ## @@ -0,0 +1,107 @@ +/ +

  1   2   >