[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396575435 Since iattr and ATTR_xxx isn't defined in nuttx. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396572081 > > what's the point of this change? > > There is code which redefines DT_(e.g., http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blob;f=direct/yportenv.h;h=5a79f93583b963d715b0dee9a0b

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8121: arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8121: URL: https://github.com/apache/nuttx/pull/8121#issuecomment-1396564883 > @xiaoxiang781216 Could you please share the motivation for disabling this warning? In which scenario does it manifest? @gustavonihei here is the warning generated by Candanc

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396559754 > what's the point of this change? There is code which redefines DT_(e.g., http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blob;f=direct/yportenv.h;h=5a79f93583b963d715b0de

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396556478 what's the point of this change? are you going to remove DTYPE_xxx? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [nuttx] pkarashchenko commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-18 Thread GitBox
pkarashchenko commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1396543499 But I do not see that pholder code modifies sem count value. How is this different from the case without priority inheritance enabled? -- This is an automated message from the Apach

[nuttx] branch master updated (6b1e0ad1c4 -> 82550887c2)

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 6b1e0ad1c4 tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported add 82550887c2 c++

[GitHub] [nuttx] jerpelea merged pull request #8187: c++: Change the default value of CXX_STANDARD from c++17 to gnu++17

2023-01-18 Thread GitBox
jerpelea merged PR #8187: URL: https://github.com/apache/nuttx/pull/8187 -- 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...@nuttx.apach

[nuttx] branch master updated: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 6b1e0ad1c4 tm4c1294-launchpad/Kconfig: Fix warn

[GitHub] [nuttx] jerpelea merged pull request #8178: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread GitBox
jerpelea merged PR #8178: URL: https://github.com/apache/nuttx/pull/8178 -- 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...@nuttx.apach

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8178: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8178: URL: https://github.com/apache/nuttx/pull/8178#issuecomment-1396541453 Can anybody merge this PR? The CI still spew many useless logs now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[nuttx-apps] branch master updated: Add *.su to .gitignore

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 44d064233 Add *.su to .gitignore 44d064233

[GitHub] [nuttx-apps] jerpelea merged pull request #1498: Add *.su to .gitignore

2023-01-18 Thread GitBox
jerpelea merged PR #1498: URL: https://github.com/apache/nuttx-apps/pull/1498 -- 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...@nuttx.

[nuttx] branch master updated (11e401c0bc -> 138f3faeef)

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 11e401c0bc esp32[c3|s3]: Fix relative path in file header add 138f3faeef boards: sabre-6quad: Adjust TCP and UDP

[GitHub] [nuttx] jerpelea merged pull request #8186: boards: sabre-6quad: Adjust TCP and UDP configurations

2023-01-18 Thread GitBox
jerpelea merged PR #8186: URL: https://github.com/apache/nuttx/pull/8186 -- 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...@nuttx.apach

[GitHub] [nuttx] pussuw opened a new pull request, #8188: mm/mm_extend: Increase total heap size accordingly

2023-01-18 Thread GitBox
pussuw opened a new pull request, #8188: URL: https://github.com/apache/nuttx/pull/8188 ## Summary When adding more heap memory, the total heap size was not updated. This results in a crash in mm_mallinfo: DEBUGASSERT((size_t)info->uordblks + info->fordblks == heap->mm_heapsize);

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
jlaitine commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1080851962 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_num_lo

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396494107 > For 15.0.0, it seems libcxx moved to using `__attribute__((using_if_exists))` in order to enable or disable features. See [this commit on LLVM](https://github.com/llvm/llvm-proje

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080846046 ## libs/libxx/Kconfig: ## @@ -35,10 +35,11 @@ config LIBCXXMINI config LIBCXX bool "LLVM libc++ C++ Standard Library" + depends on CXX_STANDARD="c++17" ||

[GitHub] [nuttx] g2gps commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396487086 > For 15.0.0, it seems libcxx moved to using `__attribute__((using_if_exists))` in order to enable or disable features. See [this commit on LLVM](https://github.com/llvm/llvm-project

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396485568 Please reorganize the patch set and remove the intermediate change, @g2gps . -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080842496 ## libs/libxx/Kconfig: ## @@ -35,10 +35,11 @@ config LIBCXXMINI config LIBCXX bool "LLVM libc++ C++ Standard Library" + depends on CXX_STANDARD="

[GitHub] [nuttx] PeterBee97 commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-18 Thread GitBox
PeterBee97 commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1396484817 > @PeterBee97 Do I need to have the "Infineon 43439 firmware file" to get the WiFi working? There is not documentation about it in the README.tx file Sorry Alan, I only have t

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396479291 BTW, @g2gps do you plan to upgrade libcxx to 15.0.x -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396470872 > Would you prefer that I squash the commits into one while applying the amendments? It's fine since github provide UI to compare the change between the new patch and the old

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080830955 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

[GitHub] [nuttx] g2gps commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396468518 Would you prefer that I squash the commits into one while applying the amendments? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8187: c++: Change the default value of CXX_STANDARD from c++17 to gnu++17

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #8187: URL: https://github.com/apache/nuttx/pull/8187 ## Summary since many 3rd party code use some gnu c++ extension ## Impact C++ default compiler option ## Testing Pass CI -- This is an automated message from

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080829205 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..13bc65dfb30

[GitHub] [nuttx-apps] xiaoxiang781216 opened a new pull request, #1498: Add *.su to .gitignore

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #1498: URL: https://github.com/apache/nuttx-apps/pull/1498 ## Summary *.su is generated by gcc/clang when we pass -fstack-usage to them ## Impact Minor ## Testing Pass CI -- This is an automated message from the

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080817564 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396454597 @g2gps it's better to fuse the last patch to related individual patch instead. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080817564 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080817564 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

[GitHub] [nuttx] masayuki2009 opened a new pull request, #8186: boards: sabre-6quad: Adjust TCP and UDP configurations

2023-01-18 Thread GitBox
masayuki2009 opened a new pull request, #8186: URL: https://github.com/apache/nuttx/pull/8186 ## Summary - This commit adjusts TCP and UDP configurations based on sabre-6quad/configs/netnsh/defconfig ## Impact - sabre-6quad only ## Testing - Tested with QEM

[GitHub] [nuttx] dnschwa opened a new pull request, #8185: Acknowledge interrupt before installing callback

2023-01-18 Thread GitBox
dnschwa opened a new pull request, #8185: URL: https://github.com/apache/nuttx/pull/8185 ## Summary Acknowledge the interrupt before installing the callback... ## Impact ...otherwise it will fire unexpectedly if it's already pending. ## Testing Impact no longer observed

[nuttx] branch master updated (530e67ba4b -> 11e401c0bc)

2023-01-18 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/nuttx.git from 530e67ba4b fs: Define DT_xxx to number directly add ed91c97f93 xtensa/esp32: Remove deprecated CONFIGFILE varia

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8184: Espressif: Clean ups and minor fixes to comments

2023-01-18 Thread GitBox
xiaoxiang781216 merged PR #8184: URL: https://github.com/apache/nuttx/pull/8184 -- 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...@nutt

[GitHub] [nuttx] zyfeier commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-18 Thread GitBox
zyfeier commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1396333791 > @zyfeier > > > Increase the sem count when the holder task exit, to avoid lock failure at next time. > > Could you explain the lock failure more concretely? Is it possible to

[nuttx-website] branch asf-site updated: Publishing web: 94ac6a38fee09a8d54ee96573f16ea8fc881f5d4 docs: 530e67ba4b8400a229be6bf084796d92e65398f5

2023-01-18 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/nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 1ac44735 Publishing web: 94ac6a38

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080647201 ## libs/libxx/.gitignore: ## @@ -4,3 +4,4 @@ /libcxx-*.src.tar.xz /libcxxabi-*.src.tar.xz /etl +/cmake Review Comment: Removed from `.gitignore`. The directory is no

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080646797 ## libs/libxx/__config_site: ## @@ -0,0 +1,69 @@ +/ + * libs/libxx/__config_site + * + * License

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080646614 ## boards/sim/sim/sim/scripts/Make.defs: ## @@ -164,6 +164,10 @@ ifeq ($(CONFIG_LIBCXX),y) # It makes libcxx __availability header unhappy. # https://github.com/llvm

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080646183 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..13bc65dfb30

[nuttx] branch master updated (85f57fb50c -> 530e67ba4b)

2023-01-18 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/nuttx.git from 85f57fb50c boards/arm/rp2040: add waveshare rp2040 lcd 1.28 add 530e67ba4b fs: Define DT_xxx to number dire

[GitHub] [nuttx] pkarashchenko merged pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
pkarashchenko merged PR #8163: URL: https://github.com/apache/nuttx/pull/8163 -- 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...@nuttx.

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
pkarashchenko commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1080643434 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_n

[GitHub] [nuttx] acassis commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-18 Thread GitBox
acassis commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1396181775 @PeterBee97 Do I need to have the "Infineon 43439 firmware file" to get the WiFi working? There is not documentation about it in the README.tx file -- This is an automated message fr

[nuttx] 03/03: boards/arm/rp2040: add waveshare rp2040 lcd 1.28

2023-01-18 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git commit 85f57fb50cac0f573460840841710b788814087a Author: Peter Bee AuthorDate: Wed Jan 18 14:37:05 2023 +0800 boards/arm/rp

[nuttx] branch master updated (dc454765fb -> 85f57fb50c)

2023-01-18 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/nuttx.git from dc454765fb Revert "add holder for mutex" new 5d08844413 boards/arm/rp2040: move duplicate peripheral configs to

[nuttx] 01/03: boards/arm/rp2040: move duplicate peripheral configs to common

2023-01-18 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git commit 5d0884441328a586befc92af9d2fd9503069f3a0 Author: Peter Bee AuthorDate: Wed Jan 18 16:05:04 2023 +0800 boards/arm/rp

[nuttx] 02/03: boards/arm/rp2040: add gc9a01 drivers

2023-01-18 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git commit 9deeae0dee552733a5d5922b9b750095ed2709fa Author: Peter Bee AuthorDate: Wed Jan 18 15:35:09 2023 +0800 boards/arm/rp

[GitHub] [nuttx] acassis merged pull request #8179: boards/arm/rp2040: add waveshare rp2040 lcd 1.28

2023-01-18 Thread GitBox
acassis merged PR #8179: URL: https://github.com/apache/nuttx/pull/8179 -- 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...@nuttx.apache

[GitHub] [nuttx] acassis commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-18 Thread GitBox
acassis commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1396154039 Still failing here! I enabled the debug network and got more info about the error: ``` nsh> ifconfig lo Link encap:Local Loopback at RUNNING inet addr:127.0

[nuttx] branch master updated (5920a8b673 -> dc454765fb)

2023-01-18 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/nuttx.git from 5920a8b673 mm/mempool/mempool_multiple.c: Remove void * arithmetic add dc454765fb Revert "add holder for mutex"

[GitHub] [nuttx] masayuki2009 merged pull request #8181: Revert "add holder for mutex"

2023-01-18 Thread GitBox
masayuki2009 merged PR #8181: URL: https://github.com/apache/nuttx/pull/8181 -- 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...@nuttx.a

[GitHub] [nuttx] gustavonihei opened a new pull request, #8184: Espressif: Clean ups and minor fixes to comments

2023-01-18 Thread GitBox
gustavonihei opened a new pull request, #8184: URL: https://github.com/apache/nuttx/pull/8184 ## Summary This PR brings some changes to Makefiles and Linker Scripts for cleaning up and synchronizing documentation across Espressif's chips. ## Impact No changes to source code, no

[GitHub] [nuttx] Cynerd commented on a diff in pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-18 Thread GitBox
Cynerd commented on code in PR #8183: URL: https://github.com/apache/nuttx/pull/8183#discussion_r1073878712 ## include/limits.h: ## @@ -309,4 +309,9 @@ #define HOST_NAME_MAX 32 +/* ptrdiff_t limits */ + +#define PTRDIFF_MAX INT_MAX Review Comment: Oh.. you are right. I

[GitHub] [nuttx] Cynerd commented on a diff in pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-18 Thread GitBox
Cynerd commented on code in PR #8183: URL: https://github.com/apache/nuttx/pull/8183#discussion_r1073878712 ## include/limits.h: ## @@ -309,4 +309,9 @@ #define HOST_NAME_MAX 32 +/* ptrdiff_t limits */ + +#define PTRDIFF_MAX INT_MAX Review Comment: Oh.. you are right. I

[GitHub] [nuttx] TA1DB commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-18 Thread GitBox
TA1DB commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1387482379 Any news guys? -- 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 u

[GitHub] [nuttx] Cynerd commented on a diff in pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-18 Thread GitBox
Cynerd commented on code in PR #8183: URL: https://github.com/apache/nuttx/pull/8183#discussion_r1073878712 ## include/limits.h: ## @@ -309,4 +309,9 @@ #define HOST_NAME_MAX 32 +/* ptrdiff_t limits */ + +#define PTRDIFF_MAX INT_MAX Review Comment: Oh.. you are right. I

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8183: URL: https://github.com/apache/nuttx/pull/8183#discussion_r1073831131 ## include/limits.h: ## @@ -309,4 +309,9 @@ #define HOST_NAME_MAX 32 +/* ptrdiff_t limits */ + +#define PTRDIFF_MAX INT_MAX Review Comment: PTRDIFF_MAX

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
jlaitine commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1073767626 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_num_lo

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
jlaitine commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1073767626 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_num_lo

[GitHub] [nuttx] masayuki2009 commented on pull request #8182: sched/mutex: Return the error code correctly in nxmutex_timedlock

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8182: URL: https://github.com/apache/nuttx/pull/8182#issuecomment-1387221181 >Ok, let's revert https://github.com/apache/nuttx/pull/8164 utils we found why spresense:rndis_smp stop working. Revert PR is https://github.com/apache/nuttx/pull/8181 --

[GitHub] [nuttx] Cynerd opened a new pull request, #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-18 Thread GitBox
Cynerd opened a new pull request, #8183: URL: https://github.com/apache/nuttx/pull/8183 ptrdiff_t is defined as int in Nuttx and thus these are same as int limits. By defining these we cover platforms where this might not be the same. -- This is an automated message from the Apache Gi

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1073576249 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col

[nuttx] branch master updated (0922121bc0 -> 5920a8b673)

2023-01-18 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/nuttx.git from 0922121bc0 riscv/addrenv: Do not free physical memory for SHM area new 6ca75deec8 drivers/can/can.c: Change siz

[nuttx] 01/02: drivers/can/can.c: Change size_t type printf formatter %d -> %zu

2023-01-18 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/nuttx.git commit 6ca75deec88c5af28df401c776cea18d8ea95391 Author: To Doan Ngoc Hai AuthorDate: Mon Oct 31 13:36:40 2022 +0200 driv

[nuttx] 02/02: mm/mempool/mempool_multiple.c: Remove void * arithmetic

2023-01-18 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/nuttx.git commit 5920a8b673d1be77b5f9b8e3e5c971b8cdd1ab80 Author: Jukka Laitinen AuthorDate: Wed Jan 18 14:36:48 2023 +0400 mm/mem

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
xiaoxiang781216 merged PR #8176: URL: https://github.com/apache/nuttx/pull/8176 -- 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...@nutt

[nuttx] branch master updated (c19b5eeb8d -> 0922121bc0)

2023-01-18 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/nuttx.git from c19b5eeb8d Add support of CAN character device for tm4c1294 launchpad. add 0922121bc0 riscv/addrenv: Do not fre

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8180: riscv/addrenv: Do not free physical memory for SHM area

2023-01-18 Thread GitBox
xiaoxiang781216 merged PR #8180: URL: https://github.com/apache/nuttx/pull/8180 -- 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...@nutt

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8182: sched/mutex: Return the error code correctly in nxmutex_timedlock

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8182: URL: https://github.com/apache/nuttx/pull/8182#issuecomment-1387107751 > Hmm, spresense:rndis_smp and spresesnse:wifi_smp are still unstable. Ok, let's revert #8164 utils we found why spresense:rndis_smp stop working. -- This is an automated me

[GitHub] [nuttx] masayuki2009 commented on pull request #8181: Revert "add holder for mutex"

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8181: URL: https://github.com/apache/nuttx/pull/8181#issuecomment-1387103097 >Let's try whether this patch can fix the problem: https://github.com/apache/nuttx/pull/8182? The PR still can not fix issues with spresense:rndis_smp and spresense:wifi_smp.

[GitHub] [nuttx] masayuki2009 commented on pull request #8182: sched/mutex: Return the error code correctly in nxmutex_timedlock

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8182: URL: https://github.com/apache/nuttx/pull/8182#issuecomment-1387097263 Hmm, spresense:rndis_smp and spresesnse:wifi_smp are still unstable. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8181: Revert "add holder for mutex"

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8181: URL: https://github.com/apache/nuttx/pull/8181#issuecomment-1387089735 Let's try where this patch can fix the problem: https://github.com/apache/nuttx/pull/8182? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8164: URL: https://github.com/apache/nuttx/pull/8164#discussion_r1073545950 ## include/semaphore.h: ## @@ -42,6 +42,7 @@ #define SEM_PRIO_INHERIT 1 #define SEM_PRIO_PROTECT 2 #define SEM_PRIO_MASK 3 +#defi

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387085158 > @xiaoxiang781216 > > > I just tried sim:usrsocktest with the last master([c19b5ee](https://github.com/apache/nuttx/commit/c19b5eeb8d8f37394c79ff85876c7ba6d846fc12)), all te

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8182: sched/mutex: Return the error code correctly in nxmutex_timedlock

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #8182: URL: https://github.com/apache/nuttx/pull/8182 ## Summary - sched/semaphore: Change SEM_PRIO_MUTEX to SEM_TYPE_MUTEX - sim/usrsocktest: Enable CONFIG_DEBUG_ASSERTIONS Fix regression report here: https://github.com/apache/nutt

[GitHub] [nuttx] jerpelea opened a new pull request, #8181: Revert "add holder for mutex"

2023-01-18 Thread GitBox
jerpelea opened a new pull request, #8181: URL: https://github.com/apache/nuttx/pull/8181 Reverts apache/nuttx#8164 -- 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 unsub

[GitHub] [nuttx] jerpelea commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
jerpelea commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387050591 +1 for revert -- 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 unsu

[nuttx] branch revert-8164-addholder_mutex created (now 51213b6a27)

2023-01-18 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch revert-8164-addholder_mutex in repository https://gitbox.apache.org/repos/asf/nuttx.git at 51213b6a27 Revert "add holder for mutex" This branch includes the following new commits:

[nuttx] 01/01: Revert "add holder for mutex"

2023-01-18 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch revert-8164-addholder_mutex in repository https://gitbox.apache.org/repos/asf/nuttx.git commit 51213b6a27721b10a91deadcfe5be950288612a7 Author: Masayuki Ishikawa AuthorDate: Wed Jan 18 22:10:06

[GitHub] [nuttx] masayuki2009 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387049087 Shall we revert this PR because there are some issues? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [nuttx] masayuki2009 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387048178 @flyingfish89 spresense:rndis_smp is unstable now (i.e. ping does not work) -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [nuttx] masayuki2009 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387024832 @xiaoxiang781216 >I just tried sim:usrsocktest with the last master(https://github.com/apache/nuttx/commit/c19b5eeb8d8f37394c79ff85876c7ba6d846fc12), all test can pass.

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1387016242 > @flyingfish89 > > I noticed that sim:usrsocktest causes assert with this PR. > > ``` > nsh> uname -a > NuttX 12.0.0 a4c9da9280 Jan 18 2023 20:32:01 sim sim >

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
jlaitine commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1073475284 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_num_lo

[GitHub] [nuttx] masayuki2009 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1386970935 @flyingfish89 >I noticed that sim:usrsocktest causes assert with this PR. It seems that this PR causes deadlock with ostest (e.g. rp2040:smp with CONFIG_DEBUG_ASSERTIONS=

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
pkarashchenko commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1073452370 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_n

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
pkarashchenko commented on code in PR #8164: URL: https://github.com/apache/nuttx/pull/8164#discussion_r1073443964 ## include/semaphore.h: ## @@ -42,6 +42,7 @@ #define SEM_PRIO_INHERIT 1 #define SEM_PRIO_PROTECT 2 #define SEM_PRIO_MASK 3 +#define

[GitHub] [nuttx] PeterBee97 opened a new pull request, #8179: boards/arm/rp2040: add waveshare rp2040 lcd 1.28

2023-01-18 Thread GitBox
PeterBee97 opened a new pull request, #8179: URL: https://github.com/apache/nuttx/pull/8179 ## Summary Added a new board with the RP2040 SoC and GC9A01 LCD screen. ## Impact New board support. Fixes a bug with Kconfig along the way. The RP2040_LCD_SPI_CH config doesn't work for boar

[GitHub] [nuttx] masayuki2009 commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1386920351 @zyfeier >Increase the sem count when the holder task exit, to avoid lock failure at next time. Could you explain the lock failure more concretely? Is it possible to d

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8178: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #8178: URL: https://github.com/apache/nuttx/pull/8178 ## Summary Generated by https://github.com/apache/nuttx/pull/8152 ## Impact Minor ## Testing Pass CI without warning -- This is an automated message from

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8177: build: Add STACK_USAGE(-fstack-usage) to assist the stack analysis

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #8177: URL: https://github.com/apache/nuttx/pull/8177 ## Summary https://github.com/HBehrens/puncover can give the graph report. ## Impact New optiion ## Testing Pass CI -- This is an automated message from the Apach

[GitHub] [nuttx] masayuki2009 commented on pull request #8164: add holder for mutex

2023-01-18 Thread GitBox
masayuki2009 commented on PR #8164: URL: https://github.com/apache/nuttx/pull/8164#issuecomment-1386910504 @flyingfish89 I noticed that sim:usrsocktest causes assert with this PR? ``` nsh> uname -a NuttX 12.0.0 a4c9da9280 Jan 18 2023 20:32:01 sim sim nsh> usrsocktest

[nuttx] 01/02: Bugfix of typo in tiva_can.c

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git commit 5dbd082fadc14e770b289ce5298a9fcfb74381b1 Author: David AuthorDate: Mon Jan 16 14:18:29 2023 +0100 Bugfix of typo i

[nuttx] branch master updated (a4c9da9280 -> c19b5eeb8d)

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from a4c9da9280 xtensa/esp32: Fix ESP32_SPIRAM_USER_HEAP under Protected mode new 5dbd082fad Bugfix of typo in tiva_

[nuttx] 02/02: Add support of CAN character device for tm4c1294 launchpad.

2023-01-18 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git commit c19b5eeb8d8f37394c79ff85876c7ba6d846fc12 Author: David AuthorDate: Mon Jan 16 14:29:31 2023 +0100 Add support of C

  1   2   >