[GitHub] [incubator-nuttx] acassis opened a new pull request #2937: stm32f103-minimum: Update README.txt to use MCP2515 module

2021-02-28 Thread GitBox
acassis opened a new pull request #2937: URL: https://github.com/apache/incubator-nuttx/pull/2937 ## Summary Update README.txt to use MCP2515 module ## Impact Make it easier to people testing MCP2515 how to wire it ## Testing STM32F103-Minimum ---

[GitHub] [incubator-nuttx] acassis commented on pull request #2937: stm32f103-minimum: Update README.txt to use MCP2515 module

2021-02-28 Thread GitBox
acassis commented on pull request #2937: URL: https://github.com/apache/incubator-nuttx/pull/2937#issuecomment-787473565 MacOS is failing: ``` ==> Downloading https://homebrew.bintray.com/bottles/u-boot-tools-2021.01.catalina.bottle.tar.gz curl: (35) error:1400410B:SSL routines

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #2930: tools/version.sh: If the version is not available, use 0.0.0

2021-02-28 Thread GitBox
Ouss4 commented on pull request #2930: URL: https://github.com/apache/incubator-nuttx/pull/2930#issuecomment-787500153 The thing with adding a new command line option is that we will have to control it somehow from the Makefile. Are we going to pass a variable with `make`? Or introduce a

[GitHub] [incubator-nuttx] fjpanag commented on pull request #2347: Enable format check for syslog

2021-02-28 Thread GitBox
fjpanag commented on pull request #2347: URL: https://github.com/apache/incubator-nuttx/pull/2347#issuecomment-787505848 @yamt @xiaoxiang781216 This effectively broke NuttX for me. After this I am no longer able to build NuttX with `-Wall -Werror`, as I was before. I get a b

[GitHub] [incubator-nuttx] yamt commented on pull request #2347: Enable format check for syslog

2021-02-28 Thread GitBox
yamt commented on pull request #2347: URL: https://github.com/apache/incubator-nuttx/pull/2347#issuecomment-787538226 @fjpanag you can fix those formats. or you can give me your kconfig and let me fix it for you. This is an

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2885: Ensure the kernel component don't call userspace API

2021-02-28 Thread GitBox
masayuki2009 commented on pull request #2885: URL: https://github.com/apache/incubator-nuttx/pull/2885#issuecomment-787556091 LGTM This is an automated message from the Apache Git Service. To respond to the message, pleas

[incubator-nuttx] 01/03: fs/fs.h: Reference the argumnet 'r' in _NX_SETERRNO

2021-02-28 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 commit cf73496d9e643cbbe846a51a6a048634fb30df69 Author: Xiang Xiao AuthorDate: Mon Feb 22 03:00:14 2021 +0800 f

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #2885: Ensure the kernel component don't call userspace API

2021-02-28 Thread GitBox
masayuki2009 merged pull request #2885: URL: https://github.com/apache/incubator-nuttx/pull/2885 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

[incubator-nuttx] 03/03: Fix nxstyle warning

2021-02-28 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 commit 151cf495621bd09177e94e7c3c3d415c52af6f0f Author: Xiang Xiao AuthorDate: Mon Feb 15 16:36:03 2021 +0800 F

[incubator-nuttx] branch master updated (59a5d03 -> 151cf49)

2021-02-28 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 59a5d03 boards: raspberrypi-pico: Update defconfigs new cf73496 fs/fs.h: Reference the argumnet 'r'

[incubator-nuttx-website] branch asf-site updated: Publishing web: 59c4fb7620bc9aabf46df895f9e5cdb361f50177 docs: 151cf495621bd09177e94e7c3c3d415c52af6f0f

2021-02-28 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 080962f Publishing web

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2347: Enable format check for syslog

2021-02-28 Thread GitBox
xiaoxiang781216 commented on pull request #2347: URL: https://github.com/apache/incubator-nuttx/pull/2347#issuecomment-787591671 @fjpanag it's better to enable wwdg in some defconfig to catch the similar problem in the future. -

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #2938: libs: misc: Remove critical section in lib_filesem.c for SMP

2021-02-28 Thread GitBox
masayuki2009 opened a new pull request #2938: URL: https://github.com/apache/incubator-nuttx/pull/2938 ## Summary - The code was added in Mar 2018 to stabilize the SMP kernel - I confirmed that the code is no longer needed now. ## Impact - SMP only ## Testing

[GitHub] [incubator-nuttx] patacongo commented on pull request #2938: libs: misc: Remove critical section in lib_filesem.c for SMP

2021-02-28 Thread GitBox
patacongo commented on pull request #2938: URL: https://github.com/apache/incubator-nuttx/pull/2938#issuecomment-787594702 Unrelated to your change, but shouldn't the calls to getpid() be changed to calls to gettid()? This i

[GitHub] [incubator-nuttx] patacongo edited a comment on pull request #2938: libs: misc: Remove critical section in lib_filesem.c for SMP

2021-02-28 Thread GitBox
patacongo edited a comment on pull request #2938: URL: https://github.com/apache/incubator-nuttx/pull/2938#issuecomment-787594702 Unrelated to your change, but shouldn't the calls to getpid() be changed to calls to gettid()? The logic implements that thread-reentrant lock. gettid() is th

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2937: stm32f103-minimum: Update README.txt to use MCP2515 module

2021-02-28 Thread GitBox
xiaoxiang781216 commented on pull request #2937: URL: https://github.com/apache/incubator-nuttx/pull/2937#issuecomment-787615831 So, let's ignore the false alarm and merge it. This is an automated message from the Apache Git

[incubator-nuttx] branch master updated: stm32f103-minimum: Update README.txt to use MCP2515 module

2021-02-28 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 b1dc219 stm32f103-minimum: Update R

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2937: stm32f103-minimum: Update README.txt to use MCP2515 module

2021-02-28 Thread GitBox
xiaoxiang781216 merged pull request #2937: URL: https://github.com/apache/incubator-nuttx/pull/2937 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL