[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
pkarashchenko commented on code in PR #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159#discussion_r977269758 ## sched/semaphore/sem_waitirq.c: ## @@ -97,10 +97,6 @@ void nxsem_wait_irq(FAR struct tcb_s *wtcb, int errcode) sem->semcount++; - /* Indi

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
pkarashchenko commented on PR #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159#issuecomment-1254602836 Saving the word number of bytes is always a good idea :) Regarding another change that moves the global list to sync objects, that goes in conflict with my PR that is inte

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159#issuecomment-1254583467 So, @masayuki2009 how about we merge this PR and focus on the real change(https://github.com/apache/incubator-nuttx/pull/6987)? -- This is an automated message from the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7160: arch: Change the linker generated symbols from uint32_t to uint8_t *

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7160: URL: https://github.com/apache/incubator-nuttx/pull/7160#issuecomment-1254581011 Please ignore this warning: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/arch/arm/src/phy62xx/start.c:169:6: error: Mixed case identifier found -- This is

[incubator-nuttx-apps] branch master updated: apps/nshlib: add the missing FAR macro for all source files

2022-09-21 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 79e770b0c apps/nshlib: add the

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1320: Add the missing FAR macro for all source files under apps/nshlib

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #1320: URL: https://github.com/apache/incubator-nuttx-apps/pull/1320 -- 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-u

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159#issuecomment-1254557804 > @zyfeier What is your intention for this PR? https://github.com/apache/incubator-nuttx/pull/6987 depends on this PR, the major objective is replacing global wait l

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
masayuki2009 commented on PR #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159#issuecomment-1254520453 @zyfeier What is your intention for this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-nuttx-apps] Junbo-Zheng opened a new pull request, #1320: Add the missing FAR macro for all source files

2022-09-21 Thread GitBox
Junbo-Zheng opened a new pull request, #1320: URL: https://github.com/apache/incubator-nuttx-apps/pull/1320 ## Summary Add the missing **`FAR`** macro for all source files under `apps/nshlib` Signed-off-by: Junbo Zheng ## Impact ## Testing -- This is an

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7160: arch: Change the linker generated symbols from uint32_t to uint8_t *

2022-09-21 Thread GitBox
xiaoxiang781216 opened a new pull request, #7160: URL: https://github.com/apache/incubator-nuttx/pull/7160 ## Summary The symbol generated by linker is always used as an address not a value. ## Impact Code refactor ## Testing -- This is an automated message from

[GitHub] [incubator-nuttx] zyfeier opened a new pull request, #7159: sched: merge waitsem and msgwaitq

2022-09-21 Thread GitBox
zyfeier opened a new pull request, #7159: URL: https://github.com/apache/incubator-nuttx/pull/7159 ## Summary 1. Combine waitsem and msgwaitq in struct tcb_s; 2. Indicate wait is over after tcb removed from block list; ## Impact NA ## Testing sabre-6quad:

[incubator-nuttx] 01/02: libc: Remove the reference of _stext/_etext from lib_cxx_initialize.c

2022-09-21 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 commit a7bbd33f1ba3222c5d088e6498707060da1df281 Author: Xiang Xiao AuthorDate: Wed Sep 21 04:57:15 2022 +0800 lib

[incubator-nuttx] 02/02: arch.h: Make _sinit/_einit as an offical interface

2022-09-21 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 commit 4c44c6f70914471cffa4267c5c67755a37c70fd3 Author: Xiang Xiao AuthorDate: Wed Sep 21 06:12:43 2022 +0800 arc

[incubator-nuttx] branch master updated (3c1c29f2c4 -> 4c44c6f709)

2022-09-21 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 3c1c29f2c4 arch: move non arm g_current_regs defintion to common place new a7bbd33f1b libc: Remove the r

[GitHub] [incubator-nuttx] anchao merged pull request #7151: libc: Remove the reference of _stext/_etext from lib_cxx_initialize.c

2022-09-21 Thread GitBox
anchao merged PR #7151: URL: https://github.com/apache/incubator-nuttx/pull/7151 -- 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...@nut

[incubator-nuttx-website] branch asf-site updated: Publishing web: 6758fc5efbbf5ed16c807ccdfb012d94d1db201a docs: 3c1c29f2c4bb675ecb14d2ee72364209d33e0bc1

2022-09-21 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 f4559ed Publishing web

[incubator-nuttx] 03/03: arch: move non arm g_current_regs defintion to common place

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 3c1c29f2c4bb675ecb14d2ee72364209d33e0bc1 Author: Xiang Xiao AuthorDate: Tue Sep 20 08:14:31 2022 +0800

[incubator-nuttx] branch master updated (6274e19257 -> 3c1c29f2c4)

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 6274e19257 fix conditional compilation term (#else) new 47bd07034c arch/misoc: Implement up_initi

[incubator-nuttx] 02/03: arch/armv[6|7|8]-m: Move _vectors to arm_internal.h to avoid the duplication

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 229b57d6cb1c87868e0ca86f6035e56226979fa5 Author: Xiang Xiao AuthorDate: Tue Sep 20 06:41:52 2022 +0800

[incubator-nuttx] 01/03: arch/misoc: Implement up_initialize in the common place

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 47bd07034c540387806888c73940dbf49d5f7421 Author: Xiang Xiao AuthorDate: Tue Sep 20 14:24:29 2022 +0800

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7140: arch: Move _vector and g_current_regs definition to common place

2022-09-21 Thread GitBox
pkarashchenko merged PR #7140: URL: https://github.com/apache/incubator-nuttx/pull/7140 -- 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

[GitHub] [incubator-nuttx] acassis commented on pull request #7109: add ci run on sim

2022-09-21 Thread GitBox
acassis commented on PR #7109: URL: https://github.com/apache/incubator-nuttx/pull/7109#issuecomment-1254109423 @ttnie it is failing to run the SIM (nuttx file), is this a chicken-egg dilemma ? -- This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-nuttx] branch master updated (fb4c593e89 -> 6274e19257)

2022-09-21 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 fb4c593e89 armv7-r/tms570: fix build break add 6274e19257 fix conditional compilation term (#else) No

[GitHub] [incubator-nuttx] acassis merged pull request #7158: fix conditional compilation term (#else)

2022-09-21 Thread GitBox
acassis merged PR #7158: URL: https://github.com/apache/incubator-nuttx/pull/7158 -- 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

[GitHub] [incubator-nuttx] acassis commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
acassis commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1254102930 @SrivamsiMalladi are you using the ```esp32-devkitc``` board or you created your own board based on it? Seems like something is mis-defined/mis-configured. Try to figure-out

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7103: tools/configure.sh: Update USAGE for custom out-of-tree boards

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7103: URL: https://github.com/apache/incubator-nuttx/pull/7103#issuecomment-1254070314 @hartmannathan let's finish this PR? -- 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

[incubator-nuttx] branch master updated: armv7-r/tms570: fix build break

2022-09-21 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 fb4c593e89 armv7-r/tms570: fix build

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157 -- 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] xiaoxiang781216 merged pull request #7156: Remove executable permissions from source files

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #7156: URL: https://github.com/apache/incubator-nuttx/pull/7156 -- 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: Remove executable permissions from source files

2022-09-21 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 ac20a5c0d6 Remove executable permiss

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7155: arm/armv7-r: remove incorrect include header "addrenv.h"

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #7155: URL: https://github.com/apache/incubator-nuttx/pull/7155 -- 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: arm/armv7-r: remove incorrect include header "addrenv.h"

2022-09-21 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 de1ad746d8 arm/armv7-r: remove incor

[GitHub] [incubator-nuttx] ASM3 opened a new pull request, #7158: fix conditional compilation term (#else)

2022-09-21 Thread GitBox
ASM3 opened a new pull request, #7158: URL: https://github.com/apache/incubator-nuttx/pull/7158 ## Summary Code fixing for the conditional compilation term (#else) ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
acassis commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976684332 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sram.ld A

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976678887 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sr

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
acassis commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976676954 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sram.ld A

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7154: arch/arm: fallback to common toolchain if armeb(endian big) is unavailable

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154#discussion_r976676895 ## arch/arm/src/common/Toolchain.defs: ## @@ -87,30 +87,34 @@ endif # NuttX buildroot under Linux or Cygwin -ifeq ($(CONFIG_ENDIAN_BIG),y) Review C

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7154: arch/arm: fallback to common toolchain if armeb(endian big) is unavailable

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154#discussion_r976676895 ## arch/arm/src/common/Toolchain.defs: ## @@ -87,30 +87,34 @@ endif # NuttX buildroot under Linux or Cygwin -ifeq ($(CONFIG_ENDIAN_BIG),y) Review C

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
acassis commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976672891 ## arch/arm/src/tms570/Make.defs: ## @@ -23,7 +23,7 @@ include armv7-r/Make.defs # SAMA5-specific C source files CHIP_CSRCS = tms570_boot.c tms570_clockconf

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976668790 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sr

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976667047 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sr

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7131: poll: add poll_notify() api and call it in all drivers.

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7131: URL: https://github.com/apache/incubator-nuttx/pull/7131#discussion_r976633836 ## drivers/analog/comp.c: ## @@ -91,35 +91,12 @@ static const struct comp_callback_s g_comp_callback = static void comp_pollnotify(FAR struct comp_dev_s

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
acassis commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976657405 ## arch/arm/src/tms570/Make.defs: ## @@ -23,7 +23,7 @@ include armv7-r/Make.defs # SAMA5-specific C source files CHIP_CSRCS = tms570_boot.c tms570_clockconf

[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
anchao commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976634512 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,7 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sram.ld AR

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7148: build(deps): bump urllib3 from 1.26.4 to 1.26.5 in /Documentation

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7148: URL: https://github.com/apache/incubator-nuttx/pull/7148#issuecomment-1253826816 Thanks. -- 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 commen

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157#discussion_r976610553 ## boards/arm/tms570/launchxl-tms57004/scripts/Make.defs: ## @@ -25,7 +25,7 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs LDSCRIPT = flash-sr

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7157: armv7-r/tms570: fix build break

2022-09-21 Thread GitBox
anchao opened a new pull request, #7157: URL: https://github.com/apache/incubator-nuttx/pull/7157 ## Summary armv7-r/tms570: fix build break 1. `make[1]: *** No rule to make target 'tms570_spi.c', needed by '.depend'. Stop.` 2. ``` In file included from

[GitHub] [incubator-nuttx] SrivamsiMalladi commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1253815521 Apparently `CONFIG_DEV_CONSOLE` is enabled and `CONFIG_DEV_CONSOLE` is disabled. -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-nuttx] btashton commented on pull request #7148: build(deps): bump urllib3 from 1.26.4 to 1.26.5 in /Documentation

2022-09-21 Thread GitBox
btashton commented on PR #7148: URL: https://github.com/apache/incubator-nuttx/pull/7148#issuecomment-1253813531 The quickest way is to pin jinja In the Pipfile to: ``` jinja2<3.1.0 ``` And then regenerate the Pipfile.lock I can likely do this tonight if no one else

[GitHub] [incubator-nuttx] acassis commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
acassis commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1253810892 Please look the inside esp32_serial.c file, since CONFIG_UART0_SERIAL_CONSOLE is defined, the only way to CONSOLE_DEV be undefined is if USE_SERIALDRIVER is false. And this

[incubator-nuttx-apps] branch master updated (ac4b42fee -> 45ef7f948)

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git from ac4b42fee testing:support openbsd crypto test for nuttx new 06f39d10f apps/nshlib: add uptim

[incubator-nuttx-apps] 02/02: apps/nshlib: fix CI error

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git commit 45ef7f948fa66a975fc33e9270f2ebc9855b1aeb Author: Junbo Zheng AuthorDate: Mon Sep 19 19:29:57 2022 +

[GitHub] [incubator-nuttx-apps] pkarashchenko merged pull request #1319: apps/nshlib: add uptime command support

2022-09-21 Thread GitBox
pkarashchenko merged PR #1319: URL: https://github.com/apache/incubator-nuttx-apps/pull/1319 -- 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-uns

[incubator-nuttx-apps] 01/02: apps/nshlib: add uptime command support

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git commit 06f39d10f15c7e68b3bd1c528bb7bb89fe5a8bd0 Author: Junbo Zheng AuthorDate: Mon Sep 12 15:43:36 2022 +

[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7154: arch/arm: fallback to common toolchain if armeb(endian big) is unavailable

2022-09-21 Thread GitBox
anchao commented on code in PR #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154#discussion_r976590105 ## arch/arm/src/common/Toolchain.defs: ## @@ -87,30 +87,34 @@ endif # NuttX buildroot under Linux or Cygwin -ifeq ($(CONFIG_ENDIAN_BIG),y) - ARCHCPUFLAGS +=

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7148: build(deps): bump urllib3 from 1.26.4 to 1.26.5 in /Documentation

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7148: URL: https://github.com/apache/incubator-nuttx/pull/7148#issuecomment-1253798945 @btashton do you know how to make this PR pass CI? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7140: arch: Move _vector and g_current_regs definition to common place

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7140: URL: https://github.com/apache/incubator-nuttx/pull/7140#discussion_r976587617 ## boards/arm/imxrt/teensy-4.x/src/imxrt_flexspi_nor_boot.h: ## @@ -91,7 +91,7 @@ /* Located in Destination Memory */ -#define IMAGE_ENTRY_ADDRESS

[GitHub] [incubator-nuttx] hartmannathan opened a new pull request, #7156: Remove executable permissions from source files

2022-09-21 Thread GitBox
hartmannathan opened a new pull request, #7156: URL: https://github.com/apache/incubator-nuttx/pull/7156 ## Summary Remove executable permissions from two source files, accidentally added in PR #7141 ## Impact Fixes permissions ## Testing `$ find . -type f

[GitHub] [incubator-nuttx] vshymanskyy commented on issue #7153: Make use of multiplexing (GSM 07.10) protocol for PPPD and GSM modules

2022-09-21 Thread GitBox
vshymanskyy commented on issue #7153: URL: https://github.com/apache/incubator-nuttx/issues/7153#issuecomment-1253788799 @acassis I'm currently evaluating multiple platforms with PPP support, so I created this issue in process of documenting my findings. I have no immediate need for mux su

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7154: arch/arm: fallback to common toolchain if armeb(endian big) is unavailable

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154#discussion_r976543655 ## arch/arm/src/common/Toolchain.defs: ## @@ -87,30 +87,34 @@ endif # NuttX buildroot under Linux or Cygwin -ifeq ($(CONFIG_ENDIAN_BIG),y) - ARCHCP

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7155: arm/armv7-r: remove incorrect include header "addrenv.h"

2022-09-21 Thread GitBox
anchao opened a new pull request, #7155: URL: https://github.com/apache/incubator-nuttx/pull/7155 ## Summary arm/armv7-r: remove incorrect include header "addrenv.h" since armv7-r does not support kernel mode ``` ./armv7-r/arm_syscall.c:36:10: fatal error: addrenv.h: N

[GitHub] [incubator-nuttx] SrivamsiMalladi commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1253749170 The UART0 is chosen as serial console by default (Maybe because I am using the nsh config of ESP32). Also, when I check the `.config` file, I can see `CONFIG_UART0_S

[GitHub] [incubator-nuttx] SrivamsiMalladi commented on issue #7152: Using GPIO with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi commented on issue #7152: URL: https://github.com/apache/incubator-nuttx/issues/7152#issuecomment-1253743800 That really helped! Changed GPIO_OUT1 to 13 and the LED is now accessible using the gpio example. Thanks -- This is an automated message from the Apache Git Ser

[GitHub] [incubator-nuttx] SrivamsiMalladi closed issue #7152: Using GPIO with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi closed issue #7152: Using GPIO with ESP32 URL: https://github.com/apache/incubator-nuttx/issues/7152 -- 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 unsubscr

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7154: arch/arm: fallback to common toolchain if armeb(endian big) is unavailable

2022-09-21 Thread GitBox
anchao opened a new pull request, #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154 ## Summary arch/arm: fallback to common toolchain if armeb(endian big) is unavailable Signed-off-by: chao an ## Impact N/A ## Testing armeb -- This i

[GitHub] [incubator-nuttx] acassis commented on issue #7153: Make use of multiplexing (GSM 07.10) protocol for PPPD and GSM modules

2022-09-21 Thread GitBox
acassis commented on issue #7153: URL: https://github.com/apache/incubator-nuttx/issues/7153#issuecomment-1253739348 Hi @vshymanskyy AFAIK NuttX doesn't have PPP serial multiplexing, you need to implement the mux protocol. This should be a nice feature to have. If you need help to implemen

[GitHub] [incubator-nuttx] acassis commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
acassis commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1253735485 @SrivamsiMalladi you need to define which UART will be your serial console, you do it on menuconfig (**$ make menuconfig**) then if you need UART0 as serial console your nee

[GitHub] [incubator-nuttx] acassis commented on issue #7152: Using GPIO with ESP32

2022-09-21 Thread GitBox
acassis commented on issue #7152: URL: https://github.com/apache/incubator-nuttx/issues/7152#issuecomment-1253728396 Please look the code at boards/xtensa/esp32/esp32-devkitc/src/esp32_gpio.c it will give you some idea where these /dev/gpioX come from. Basically you will modify the GPIO_OU

[GitHub] [incubator-nuttx] vshymanskyy opened a new issue, #7153: Make use of multiplexing (GSM 07.10) protocol for PPPD and GSM modules

2022-09-21 Thread GitBox
vshymanskyy opened a new issue, #7153: URL: https://github.com/apache/incubator-nuttx/issues/7153 While the PPPD example works, one needs to exit PPP mode (possibly power-resetting the modem) to send additional AT commands or read NMEA information from the GSM modems, which proves diffic

[GitHub] [incubator-nuttx] SrivamsiMalladi opened a new issue, #7152: Using GPIO with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi opened a new issue, #7152: URL: https://github.com/apache/incubator-nuttx/issues/7152 I am using [Adafruit HUZZAH32 – ESP32 Feather Board](https://www.adafruit.com/product/3405). It has an inbuilt LED connected to pin 13. I am trying to write high and low to it using the [a

[GitHub] [incubator-nuttx] CV-Bowen commented on a diff in pull request #7131: poll: add poll_notify() api and call it in all drivers.

2022-09-21 Thread GitBox
CV-Bowen commented on code in PR #7131: URL: https://github.com/apache/incubator-nuttx/pull/7131#discussion_r976480245 ## net/netlink/netlink_sockif.c: ## @@ -595,7 +593,7 @@ static int netlink_poll(FAR struct socket *psock, FAR struct pollfd *fds, if (revents != 0)

[incubator-nuttx] branch master updated: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 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 ae378872c2 arch: armv7-a: Remove the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7149: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7149: URL: https://github.com/apache/incubator-nuttx/pull/7149#discussion_r976413779 ## arch/arm/src/armv7-a/pgalloc.h: ## @@ -228,9 +197,7 @@ uintptr_t arm_physpgaddr(uintptr_t vaddr); * **

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7149: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #7149: URL: https://github.com/apache/incubator-nuttx/pull/7149 -- 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] masayuki2009 commented on a diff in pull request #7149: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 Thread GitBox
masayuki2009 commented on code in PR #7149: URL: https://github.com/apache/incubator-nuttx/pull/7149#discussion_r976394218 ## arch/arm/src/armv7-a/pgalloc.h: ## @@ -228,9 +197,7 @@ uintptr_t arm_physpgaddr(uintptr_t vaddr); * *

[GitHub] [incubator-nuttx] SrivamsiMalladi commented on issue #7150: Using UART with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi commented on issue #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150#issuecomment-1253573012 I found https://nuttx.apache.org/docs/latest/components/drivers/character/serial.html where it says > Each serial device driver must implement an instance of str

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7151: libc: Remove the reference of _stext/_etext from lib_cxx_initialize.c

2022-09-21 Thread GitBox
xiaoxiang781216 opened a new pull request, #7151: URL: https://github.com/apache/incubator-nuttx/pull/7151 ## Summary ## Impact Remove the impossible condition ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx] anjiahao1 commented on pull request #6920: support /dev/crypto for nuttx

2022-09-21 Thread GitBox
anjiahao1 commented on PR #6920: URL: https://github.com/apache/incubator-nuttx/pull/6920#issuecomment-1253523715 @pkarashchenko Thanks for the codereivew, I just submitted it once -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[incubator-nuttx] branch master updated (6903f02d41 -> 53466c6d46)

2022-09-21 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 6903f02d41 armv7-r: add VBAR cp15 opearation add 53466c6d46 net: Add IN6_IS_ADDR_LINKLOCAL macro No new

[GitHub] [incubator-nuttx] anchao merged pull request #7143: net: Add IN6_IS_ADDR_LINKLOCAL macro

2022-09-21 Thread GitBox
anchao merged PR #7143: URL: https://github.com/apache/incubator-nuttx/pull/7143 -- 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...@nut

[GitHub] [incubator-nuttx] SrivamsiMalladi opened a new issue, #7150: Using UART with ESP32

2022-09-21 Thread GitBox
SrivamsiMalladi opened a new issue, #7150: URL: https://github.com/apache/incubator-nuttx/issues/7150 I would like to use NuttX with ESP32 and I am looking for a UART driver. I see that there is the [nuttx/include/nuttx/serial/serial.h](https://github.com/apache/incubator-nuttx/blob/master/

[incubator-nuttx] 02/04: armv7a/r: refact cp15_cache functions

2022-09-21 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 6d92810d5a8d6a02e8737a91821f7ef82c8ee1b2 Author: ligd AuthorDate: Fri Sep 16 14:31:52 2022 +0800 armv7a

[incubator-nuttx] 04/04: armv7-r: add VBAR cp15 opearation

2022-09-21 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 6903f02d4111b9815aa77159132ebd86c9276a6a Author: ligd AuthorDate: Mon Sep 19 12:00:54 2022 +0800 armv7-

[incubator-nuttx] branch master updated (f4812e0a83 -> 6903f02d41)

2022-09-21 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 f4812e0a83 net: Implement SO_RCVBUF and SO_SNDBUF for getsockopt new c866b6be9a armv7a/r: add common

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7113: armv7-a/r: refact cp15_cache functions

2022-09-21 Thread GitBox
xiaoxiang781216 merged PR #7113: URL: https://github.com/apache/incubator-nuttx/pull/7113 -- 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] 01/04: armv7a/r: add common operation CP15_SET/GET()

2022-09-21 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 c866b6be9ae5af8516b55008fe307810b4077def Author: ligd AuthorDate: Thu Sep 15 22:10:12 2022 +0800 armv7a

[incubator-nuttx] 03/04: armv7a/r: add isb to cp15_invalidate_icache()

2022-09-21 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 3477c347e805c74e3879b408156dbc817f9cd45a Author: ligd AuthorDate: Fri Sep 16 23:29:37 2022 +0800 armv7a

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7149: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 Thread GitBox
xiaoxiang781216 commented on code in PR #7149: URL: https://github.com/apache/incubator-nuttx/pull/7149#discussion_r976318391 ## arch/arm/src/armv7-a/pgalloc.h: ## @@ -228,9 +197,7 @@ uintptr_t arm_physpgaddr(uintptr_t vaddr); * **

[GitHub] [incubator-nuttx] anjiahao1 commented on a diff in pull request #6920: support /dev/crypto for nuttx

2022-09-21 Thread GitBox
anjiahao1 commented on code in PR #6920: URL: https://github.com/apache/incubator-nuttx/pull/6920#discussion_r976297307 ## crypto/castsb.h: ## @@ -0,0 +1,554 @@ +/ + * crypto/castsb.h + * $OpenBSD: casts

[GitHub] [incubator-nuttx] anjiahao1 commented on a diff in pull request #6920: support /dev/crypto for nuttx

2022-09-21 Thread GitBox
anjiahao1 commented on code in PR #6920: URL: https://github.com/apache/incubator-nuttx/pull/6920#discussion_r976267424 ## include/crypto/xform.h: ## @@ -0,0 +1,111 @@ +/ + * include/crypto/xform.h + * $

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request, #7149: arch: armv7-a: Remove the code for CONFIG_ARCH_PGPOOL_MAPPING=n

2022-09-21 Thread GitBox
masayuki2009 opened a new pull request, #7149: URL: https://github.com/apache/incubator-nuttx/pull/7149 ## Summary - Currently, CONFIG_ARCH_PGPOOL_MAPPING=y is necessary for CONFIG_BUILD_KERNEL=y. - This commit removes the code for CONFIG_ARCH_PGPOOL_MAPPING=n ## Impact

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7140: arch: Move _vector and g_current_regs definition to common place

2022-09-21 Thread GitBox
pkarashchenko commented on code in PR #7140: URL: https://github.com/apache/incubator-nuttx/pull/7140#discussion_r976128863 ## boards/arm/imxrt/teensy-4.x/src/imxrt_flexspi_nor_boot.h: ## @@ -91,7 +91,7 @@ /* Located in Destination Memory */ -#define IMAGE_ENTRY_ADDRESS

[GitHub] [incubator-nuttx] CV-Bowen commented on a diff in pull request #7131: poll: add poll_notify() api and call it in all drivers.

2022-09-21 Thread GitBox
CV-Bowen commented on code in PR #7131: URL: https://github.com/apache/incubator-nuttx/pull/7131#discussion_r976127867 ## net/netlink/netlink_sockif.c: ## @@ -595,7 +593,7 @@ static int netlink_poll(FAR struct socket *psock, FAR struct pollfd *fds, if (revents != 0)

[GitHub] [incubator-nuttx-apps] Junbo-Zheng commented on a diff in pull request #1319: apps/nshlib: add uptime command support

2022-09-21 Thread GitBox
Junbo-Zheng commented on code in PR #1319: URL: https://github.com/apache/incubator-nuttx-apps/pull/1319#discussion_r976119567 ## nshlib/nsh_proccmds.c: ## @@ -45,6 +47,15 @@ # define CONFIG_NSH_PROC_MOUNTPOINT "/proc" #endif +#ifndef CONFIG_NSH_DISABLE_UPTIME + #ifndef F

[incubator-nuttx] branch master updated (25d5cd11a1 -> f4812e0a83)

2022-09-21 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 25d5cd11a1 mmu: mmu enable should after enable SMP add f4812e0a83 net: Implement SO_RCVBUF and SO

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7124: net: Implement SO_RCVBUF and SO_SNDBUF for getsockopt

2022-09-21 Thread GitBox
pkarashchenko merged PR #7124: URL: https://github.com/apache/incubator-nuttx/pull/7124 -- 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

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7131: poll: add poll_notify() api and call it in all drivers.

2022-09-21 Thread GitBox
pkarashchenko commented on code in PR #7131: URL: https://github.com/apache/incubator-nuttx/pull/7131#discussion_r976117835 ## net/netlink/netlink_sockif.c: ## @@ -595,7 +593,7 @@ static int netlink_poll(FAR struct socket *psock, FAR struct pollfd *fds, if (revents != 0)

[GitHub] [incubator-nuttx] xiaoxiang781216 closed pull request #7147: arch: move non arm g_current_regs defintion to common place

2022-09-21 Thread GitBox
xiaoxiang781216 closed pull request #7147: arch: move non arm g_current_regs defintion to common place URL: https://github.com/apache/incubator-nuttx/pull/7147 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7147: arch: move non arm g_current_regs defintion to common place

2022-09-21 Thread GitBox
xiaoxiang781216 commented on PR #7147: URL: https://github.com/apache/incubator-nuttx/pull/7147#issuecomment-1253289496 Close, let's track by https://github.com/apache/incubator-nuttx/pull/7140 -- This is an automated message from the Apache Git Service. To respond to the message, please l