[GitHub] [incubator-nuttx-apps] pkarashchenko commented on a diff in pull request #1340: Fix the minor style issue

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #1340: URL: https://github.com/apache/incubator-nuttx-apps/pull/1340#discussion_r993070592 ## examples/netloop/lo_listener.c: ## @@ -297,7 +299,7 @@ void *lo_listener(pthread_addr_t pvarg) memset(&nls, 0, sizeof(struct net_listener_s));

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7289: newlib/impure: declare thread specific file only if __NEWLIB__ is defined

2022-10-12 Thread GitBox
anchao opened a new pull request, #7289: URL: https://github.com/apache/incubator-nuttx/pull/7289 ## Summary newlib/impure: declare thread specific file only if __NEWLIB__ is defined. Signed-off-by: chao an ## Impact N/A ## Testing ci-check -- Thi

[GitHub] [incubator-nuttx] anchao commented on pull request #7284: tools/ci: Update gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
anchao commented on PR #7284: URL: https://github.com/apache/incubator-nuttx/pull/7284#issuecomment-1275697599 > Anyway statement `defined(__NEWLIB__) || __NEWLIB__ < 4` seems to be wrong. If `__NEWLIB__` is not defined the `__NEWLIB__ < 4` will be evaluated and will produce compiler warnin

[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7284: tools/ci: Update gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
anchao commented on code in PR #7284: URL: https://github.com/apache/incubator-nuttx/pull/7284#discussion_r993079572 ## tools/ci/patch/arm-none-eabi-workaround-for-newlib-version-break.patch: ## @@ -0,0 +1,38 @@ +From 819ab4ac433b809fdaca33246e2bffa1426acc0f Mon Sep 17 00:00:00

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7114: Add CodeChecker to CI

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7114: URL: https://github.com/apache/incubator-nuttx/pull/7114 -- 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-unsubs

[incubator-nuttx] branch master updated: tools/ci/docker/linux/Dockerfile: Install CodeChecker and requirements(clang, clang-tidy)

2022-10-12 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 d8cff7d17a tools/ci/docker/linux/Doc

[GitHub] [incubator-nuttx] zyfeier commented on a diff in pull request #7288: armv7-a: add l2 page mapping interface

2022-10-12 Thread GitBox
zyfeier commented on code in PR #7288: URL: https://github.com/apache/incubator-nuttx/pull/7288#discussion_r993185388 ## arch/arm/src/armv7-a/mmu.h: ## @@ -605,10 +605,17 @@ #endif #define MMU_L1_DATAFLAGS (PMD_TYPE_PTE | PMD_PTE_PXN | PMD_PTE_DOM(0)) -#define MMU_L2_UD

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993202325 ## libs/libc/obstack/Make.defs: ## @@ -0,0 +1,30 @@ + +# libs/libc/obstack/Make.defs +

[GitHub] [incubator-nuttx] zyfeier opened a new pull request, #7290: armv7-a: set normal memory shareable in smp mode

2022-10-12 Thread GitBox
zyfeier opened a new pull request, #7290: URL: https://github.com/apache/incubator-nuttx/pull/7290 ## Summary In SMP mode, need set normal memory shareable between multiple cores. ## Impact NA ## Testing Test use V200Z-R-EVB board. -- This is an automate

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993207564 ## libs/libc/obstack/lib_obstack_copy.c: ## @@ -0,0 +1,47 @@ +/ + * libs/libc/obstack/

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993208741 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,77 @@ +/ + * libs/libc/obsta

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993209357 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,77 @@ +/ + * libs/libc/obsta

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993210210 ## libs/libc/obstack/lib_obstack_printf.c: ## @@ -0,0 +1,43 @@ +/ + * libs/libc/obstac

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993212361 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,77 @@ +/ + * libs/libc/obsta

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-12 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r993213435 ## libs/libc/obstack/lib_obstack_init.c: ## @@ -0,0 +1,41 @@ +/ + * libs/libc/obstack/

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7289: newlib/impure: declare thread specific file only if __NEWLIB__ is defined

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7289: URL: https://github.com/apache/incubator-nuttx/pull/7289 -- 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-unsubs

[incubator-nuttx] branch master updated (d8cff7d17a -> 36039c8526)

2022-10-12 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 d8cff7d17a tools/ci/docker/linux/Dockerfile: Install CodeChecker and requirements(clang, clang-tidy)

[GitHub] [incubator-nuttx] SrivamsiMalladi commented on issue #7193: Missing SoftAP documentation or feature for ESP32

2022-10-12 Thread GitBox
SrivamsiMalladi commented on issue #7193: URL: https://github.com/apache/incubator-nuttx/issues/7193#issuecomment-1275871827 Hello @tmedicci. Thank you so much for looking into the issue. I tired what you mentioned, but the behavior is still the same. Cannot connect to the AP from my And

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993229457 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) * 8 - t

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993274928 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) * 8 - t

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request, #7291: usrsock: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
zhhyu7 opened a new pull request, #7291: URL: https://github.com/apache/incubator-nuttx/pull/7291 ## Summary Correct some status exceptions in non-blocking mode usrsock connect ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-nuttx-apps] zhhyu7 opened a new pull request, #1345: usrsock_server: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
zhhyu7 opened a new pull request, #1345: URL: https://github.com/apache/incubator-nuttx-apps/pull/1345 ## Summary Correct some status exceptions in non-blocking mode usrsock connect ## Impact ## Testing Depends on https://github.com/apache/incubator-nuttx/pull/7291 -- T

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993286949 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993290268 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7291: usrsock: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
masayuki2009 commented on PR #7291: URL: https://github.com/apache/incubator-nuttx/pull/7291#issuecomment-1275954028 @zhhyu7 ``` NuttShell (NSH) NuttX-10.3.0-RC0 nsh> uname -a NuttX 10.3.0-RC0 2861124366-dirty Oct 12 2022 19:36:41 sim sim nsh> usrsocktest Starting unit

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993290268 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] zhhyu7 commented on pull request #7291: usrsock: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
zhhyu7 commented on PR #7291: URL: https://github.com/apache/incubator-nuttx/pull/7291#issuecomment-1275963272 > @zhhyu7 > > Please test before sending PR. > > ``` > NuttShell (NSH) NuttX-10.3.0-RC0 > nsh> uname -a > NuttX 10.3.0-RC0 2861124366-dirty Oc

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993301685 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993301685 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993301685 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993302583 ## net/tcp/tcp_recvfrom.c: ## @@ -706,7 +706,8 @@ ssize_t psock_tcp_recvfrom(FAR struct socket *psock, FAR struct msghdr *msg, *want for more because th

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993308230 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) *

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7290: armv7-a: set normal memory shareable in smp mode

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7290: URL: https://github.com/apache/incubator-nuttx/pull/7290 -- 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-unsubs

[incubator-nuttx] branch master updated (36039c8526 -> 466635a5e0)

2022-10-12 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 36039c8526 newlib/impure: declare thread specific file only if __NEWLIB__ is defined. add 466635a5e0

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7284: tools/ci: Update gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7284: URL: https://github.com/apache/incubator-nuttx/pull/7284 -- 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-unsubs

[incubator-nuttx] branch master updated (466635a5e0 -> 0f3f5d091a)

2022-10-12 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 466635a5e0 armv7-a: set normal memory shareable in smp mode add cfe2ec3d4d tools/ci: Update gcc-arm-n

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276084714 `sim:usrsocktest` with `CONFIG_DEBUG_ASSERTIONS=y` failed. ``` up_assert: Assertion failed at file:/mnt/m2ssd/opensource/github_anjiahao1/nuttx/include/nuttx/mutex.h

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276097887 @xiaoxiang781216 >But, the initial version doesn't exist in my local git anymore. If I can't retrieve the initial/early version of this patch from github, it's very ha

[GitHub] [incubator-nuttx] zouboan opened a new pull request, #7292: arch/sparc dismiss non-Unicode characters issue

2022-10-12 Thread GitBox
zouboan opened a new pull request, #7292: URL: https://github.com/apache/incubator-nuttx/pull/7292 ## Summary dismiss non-Unicode characters issue in sparc-v8 response to https://www.mail-archive.com/dev@nuttx.apache.org/msg08478.html ## Impact sparc-v8 systemreset ## Testing

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276107696 >sim:usrsocktest with CONFIG_DEBUG_ASSERTIONS=y failed. Same as `spresense:wifi_smp` with `CONFIG_DEBUG_ASSERTIONS=y` ``` AEG [0.44] [CPU1] up_asser

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
pkarashchenko commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276121674 Let me take an effort and separate FAR and some style changes and submit as a separate PR. I will need a couple of days based on my current work load -- This is an automat

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993433226 ## net/tcp/tcp_recvfrom.c: ## @@ -706,7 +706,8 @@ ssize_t psock_tcp_recvfrom(FAR struct socket *psock, FAR struct msghdr *msg, *want for more because th

[GitHub] [incubator-nuttx] fjpanag commented on pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#issuecomment-1276142303 I haven't tested the latest changes on actual hardware, yet. Please do not merge before confirmation. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993441377 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) * 8 - t

[GitHub] [incubator-nuttx] acassis opened a new pull request, #7293: esp32s2: Add missing ESP32S2 SPIRAM config

2022-10-12 Thread GitBox
acassis opened a new pull request, #7293: URL: https://github.com/apache/incubator-nuttx/pull/7293 ## Summary Add missing ESP32S2 SPIRAM config ## Impact Users will be able to select the SPIRAM support ## Testing CI -- This is an automated message from the Apache Git Ser

[incubator-nuttx] branch master updated (0f3f5d091a -> 7d34ebdd4e)

2022-10-12 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 0f3f5d091a tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022 add 7d34ebdd4e armv7-a

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7288: armv7-a: add l2 page mapping interface

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7288: URL: https://github.com/apache/incubator-nuttx/pull/7288 -- 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-unsubs

[GitHub] [incubator-nuttx] tmedicci commented on issue #7193: Missing SoftAP documentation or feature for ESP32

2022-10-12 Thread GitBox
tmedicci commented on issue #7193: URL: https://github.com/apache/incubator-nuttx/issues/7193#issuecomment-1276263662 > Hello @tmedicci. Thank you so much for looking into the issue. I tired what you mentioned, but the behavior is still the same. Cannot connect to the AP from my Android ph

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276265560 > At least, you need to test with `CONFIG_DEBUG_ASSERTIONS=y` Ok, the problem is nxrmutex_breaklock forget to clear ret value: https://github.com/apache/incubator

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276274881 > Let me take an effort and separate FAR and some style changes and submit as a separate PR. I will need a couple of days based on my current work load I find a meth

[GitHub] [incubator-nuttx] acassis merged pull request #7292: arch/sparc dismiss non-Unicode characters issue

2022-10-12 Thread GitBox
acassis merged PR #7292: URL: https://github.com/apache/incubator-nuttx/pull/7292 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nu

[incubator-nuttx] branch master updated (7d34ebdd4e -> d3f84ba724)

2022-10-12 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 7d34ebdd4e armv7-a: add l2 page mapping interface add d3f84ba724 arch/sparc dismiss non-Unicode charact

[GitHub] [incubator-nuttx] HidenoriMatsubayashi opened a new pull request, #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
HidenoriMatsubayashi opened a new pull request, #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295 ## Summary `Use CONFIG_ARCH_CHIP_QEMU` instead of `CONFIG_ARCH_CHIP_QEMU_A53`. This is because these configurations depend on the hardware configuration of qemu (memory map, ir

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
HidenoriMatsubayashi commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276296855 Hi @MasayukiIshikawa @xiaoxiang781216 Can you please review? -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993562518 ## mm/ubsan/ubsan.c: ## @@ -160,7 +160,7 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, unsigned extra_bits = sizeof(int64_t) * 8 - t

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
HidenoriMatsubayashi commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276304245 Or, perhaps we should create a new config for qemu virt (e.g. CONFIG_ARCH_CHIP_QEMU_VIRT). -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276328045 @masayuki2009 @pkarashchenko the style change is revert, please review again. -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-nuttx-apps] zhhyu7 commented on pull request #1345: usrsock_server: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
zhhyu7 commented on PR #1345: URL: https://github.com/apache/incubator-nuttx-apps/pull/1345#issuecomment-1276352621 Hi, @masayuki2009 https://github.com/apache/incubator-nuttx/pull/7291 will pass the test with this change. Please help to check whether this change is reasonable, Thinks. -

[GitHub] [incubator-nuttx] TimJTi opened a new pull request, #7296: Documentation for Custom Boards and Apps

2022-10-12 Thread GitBox
TimJTi opened a new pull request, #7296: URL: https://github.com/apache/incubator-nuttx/pull/7296 ## Summary Adds new RST files to document creating custom boards out of tree, and custom apps in and out of tree ## Impact Improved documentation ## Testing HTML created a

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#issuecomment-1276523635 let's ignore the temp ci broken and merge this simple change directly. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] xiaoxiang781216 closed issue #7274: Undefined behaviour within UBSAN.

2022-10-12 Thread GitBox
xiaoxiang781216 closed issue #7274: Undefined behaviour within UBSAN. URL: https://github.com/apache/incubator-nuttx/issues/7274 -- 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 #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285 -- 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-unsubs

[incubator-nuttx] branch master updated (d3f84ba724 -> eec7f5da00)

2022-10-12 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 d3f84ba724 arch/sparc dismiss non-Unicode characters issue add eec7f5da00 Fixed undefined behavior in

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7293: esp32s2: Add missing ESP32S2 SPIRAM config

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7293: URL: https://github.com/apache/incubator-nuttx/pull/7293#issuecomment-1276524900 Let's ignore the temp ci broken and merge this simple patch directly -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7293: esp32s2: Add missing ESP32S2 SPIRAM config

2022-10-12 Thread GitBox
xiaoxiang781216 merged PR #7293: URL: https://github.com/apache/incubator-nuttx/pull/7293 -- 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-unsubs

[incubator-nuttx] branch master updated (eec7f5da00 -> c297ecaa5d)

2022-10-12 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 eec7f5da00 Fixed undefined behavior in UBSan. add c297ecaa5d esp32s2: Add missing ESP32S2 SPIRAM conf

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276526338 > Or, perhaps we should create a new config for qemu virt? (e.g. `CONFIG_ARCH_CHIP_QEMU_VIRT`). it's more simple to remove _A53 suffix. -- This is an automated me

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276526891 let's ignore the temp ci broken and merge this simple change directly. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276528820 > > Or, perhaps we should create a new config for qemu virt? (e.g. `CONFIG_ARCH_CHIP_QEMU_VIRT`). > > it's more simple to remove _A53 suffix. @HidenoriMatsuba

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
xiaoxiang781216 commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993745202 ## net/tcp/tcp_recvfrom.c: ## @@ -706,7 +706,8 @@ ssize_t psock_tcp_recvfrom(FAR struct socket *psock, FAR struct msghdr *msg, *want for more be

[GitHub] [incubator-nuttx] acassis commented on pull request #7293: esp32s2: Add missing ESP32S2 SPIRAM config

2022-10-12 Thread GitBox
acassis commented on PR #7293: URL: https://github.com/apache/incubator-nuttx/pull/7293#issuecomment-1276538999 Thank you @xiaoxiang781216 ! Any idea why our CI is failing too often recently? -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-nuttx] acassis commented on pull request #7296: Documentation for Custom Boards and Apps

2022-10-12 Thread GitBox
acassis commented on PR #7296: URL: https://github.com/apache/incubator-nuttx/pull/7296#issuecomment-1276539781 Thank you @TimJTi !!! -- 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

[GitHub] [incubator-nuttx] acassis merged pull request #7296: Documentation for Custom Boards and Apps

2022-10-12 Thread GitBox
acassis merged PR #7296: URL: https://github.com/apache/incubator-nuttx/pull/7296 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nu

[incubator-nuttx] branch master updated (c297ecaa5d -> 538365e362)

2022-10-12 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 c297ecaa5d esp32s2: Add missing ESP32S2 SPIRAM config add 538365e362 Documentation for Custom Boards a

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7296: Documentation for Custom Boards and Apps

2022-10-12 Thread GitBox
acassis commented on code in PR #7296: URL: https://github.com/apache/incubator-nuttx/pull/7296#discussion_r993752918 ## Documentation/guides/customapps.rst: ## @@ -0,0 +1,346 @@ +== +Custom Apps How-to +== + +Nuttx comes with a large number of Ap

[GitHub] [incubator-nuttx] TimJTi opened a new pull request, #7297: Nuttx corrected to NuttX

2022-10-12 Thread GitBox
TimJTi opened a new pull request, #7297: URL: https://github.com/apache/incubator-nuttx/pull/7297 As per title -- 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 un

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7293: esp32s2: Add missing ESP32S2 SPIRAM config

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7293: URL: https://github.com/apache/incubator-nuttx/pull/7293#issuecomment-1276557778 tool change upgrade issue, here is the fix: https://github.com/apache/incubator-nuttx/pull/7294 -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7294: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #7294: URL: https://github.com/apache/incubator-nuttx/pull/7294#issuecomment-1276559554 @acassis please merge this patch if Docker-Linux can pass CI, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993870810 ## mm/ubsan/ubsan.c: ## @@ -158,9 +158,11 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, if (is_inline_int(type)) { uns

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-12 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r993876542 ## mm/ubsan/ubsan.c: ## @@ -158,9 +158,11 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, if (is_inline_int(type)) { uns

[GitHub] [incubator-nuttx] TimJTi closed pull request #7297: Nuttx corrected to NuttX

2022-10-12 Thread GitBox
TimJTi closed pull request #7297: Nuttx corrected to NuttX URL: https://github.com/apache/incubator-nuttx/pull/7297 -- 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 unsubscri

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993895275 ## net/tcp/tcp_recvfrom.c: ## @@ -412,7 +412,8 @@ static uint16_t tcp_recvhandler(FAR struct net_driver_s *dev, * next receive is performed.

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993895716 ## net/tcp/tcp_recvfrom.c: ## @@ -698,43 +699,53 @@ ssize_t psock_tcp_recvfrom(FAR struct socket *psock, FAR struct msghdr *msg, /* We get here when we we de

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-12 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r993898747 ## net/tcp/tcp_recvfrom.c: ## @@ -698,43 +699,53 @@ ssize_t psock_tcp_recvfrom(FAR struct socket *psock, FAR struct msghdr *msg, /* We get here when we we de

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
HidenoriMatsubayashi commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276808629 > could you remove all _A53, if the code work with other armv8a core. Ok, I'll do try that. -- This is an automated message from the Apache Git Service. To re

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
masayuki2009 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276815494 @HidenoriMatsubayashi Please use @masayuki2009 the next time. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
masayuki2009 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276821787 >let's ignore the temp ci broken and merge this simple change directly. @xiaoxiang781216 Do you know why the CI breaks now? -- This is an automated message fr

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7294: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
masayuki2009 commented on PR #7294: URL: https://github.com/apache/incubator-nuttx/pull/7294#issuecomment-1276844888 Let me merge this PR to fix the CI issue. -- 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] masayuki2009 merged pull request #7294: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 Thread GitBox
masayuki2009 merged PR #7294: URL: https://github.com/apache/incubator-nuttx/pull/7294 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr.

[incubator-nuttx] branch master updated: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

2022-10-12 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 53dd0819cf tools/ci: patch for docker

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7295: arch/arm64: cleanup configs independent of CPU core types

2022-10-12 Thread GitBox
masayuki2009 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1276850452 >Do you know why the CI breaks now? I've just merged the following PR. https://github.com/apache/incubator-nuttx/pull/7294 -- This is an automated message from

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on pull request #1345: usrsock_server: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
masayuki2009 commented on PR #1345: URL: https://github.com/apache/incubator-nuttx-apps/pull/1345#issuecomment-1276862903 Though I merged https://github.com/apache/incubator-nuttx/pull/7294, the CI still fails. -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276920822 >@masayuki2009 @pkarashchenko the style change is revert, please review again. @xiaoxiang781216 Thanks for your efforts and updates. It seems much easier to

[GitHub] [incubator-nuttx] ttnie opened a new pull request, #7298: install qemu on docker

2022-10-12 Thread GitBox
ttnie opened a new pull request, #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298 Signed-off-by: nietingting ## Summary install qemu on docker ## Impact no ## Testing test on local host -- This is an automated message from the Apache Git Service. T

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7298: install qemu on docker

2022-10-12 Thread GitBox
xiaoxiang781216 commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r994047320 ## tools/ci/docker/linux/Dockerfile: ## @@ -212,6 +212,20 @@ RUN cd renesas-tools/source/newlib && \ RUN cd /tools/renesas-tools/build/gcc && \ make

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276949593 > > @masayuki2009 @pkarashchenko the style change is revert, please review again. > > @xiaoxiang781216 Thanks for your efforts and updates. > > It seems much

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-12 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1276961009 >nxmutex_t is initialized directly at the variables(g_i2cxdev) definition, not in cxd56_i2cbus_initialize, and then the code can be protected by mutex_t self, so it's redunda

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7299: tools/ci: fetch patch from github

2022-10-12 Thread GitBox
anchao opened a new pull request, #7299: URL: https://github.com/apache/incubator-nuttx/pull/7299 ## Summary tools/ci: fetch patch from github Signed-off-by: chao an ## Impact N/A ## Testing ci check -- This is an automated message from the Apache

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1345: usrsock_server: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-12 Thread GitBox
xiaoxiang781216 commented on PR #1345: URL: https://github.com/apache/incubator-nuttx-apps/pull/1345#issuecomment-1276971082 > Though I merged [apache/incubator-nuttx#7294](https://github.com/apache/incubator-nuttx/pull/7294), the CI still fails. > > ``` > nothing to commit, work

[GitHub] [incubator-nuttx] ttnie commented on a diff in pull request #7298: install qemu on docker

2022-10-12 Thread GitBox
ttnie commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r994083497 ## tools/ci/docker/linux/Dockerfile: ## @@ -212,6 +212,20 @@ RUN cd renesas-tools/source/newlib && \ RUN cd /tools/renesas-tools/build/gcc && \ make && make in

  1   2   >