[GitHub] [incubator-nuttx] btashton commented on issue #2594: Should we remove the support of too old and incompatible risc-v chip(gap8 and nr5m100)?

2021-01-04 Thread GitBox
btashton commented on issue #2594: URL: https://github.com/apache/incubator-nuttx/issues/2594#issuecomment-754449876 Ken, I know you did a lot of work getting the first RISC-V support going, and that included the nr5m100 which I understood is based on work you did with IQ-Analog.

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
yamt commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551758507 ## File path: tools/configure.sh ## @@ -296,7 +296,7 @@ done if [ "X${defappdir}" = "Xy" ]; then # In-place edit can mess up permissions on

[GitHub] [incubator-nuttx] yamt commented on pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
yamt commented on pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#issuecomment-754457877 > > @yamt can you merge all patch into one? it's important to keep the history clean without the intermediate commit > > i will merge them later done

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
yamt commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551747059 ## File path: boards/risc-v/nr5m100/nr5m100-nexys4/README.txt ## @@ -143,7 +143,7 @@ Debugger shadowed variable warnings, so disable the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551755975 ## File path: boards/risc-v/nr5m100/nr5m100-nexys4/README.txt ## @@ -143,7 +143,7 @@ Debugger shadowed variable warnings, so

[GitHub] [incubator-nuttx] btashton commented on a change in pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
btashton commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551760293 ## File path: tools/configure.sh ## @@ -296,7 +296,7 @@ done if [ "X${defappdir}" = "Xy" ]; then # In-place edit can mess up permissions

[incubator-nuttx] branch master updated (8940646 -> c8784fa)

2021-01-04 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 8940646 sched: sched: Don't call sched_lock()/unlock() in nx_waitid()/waitpid() for SMP add

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2650: ctype/iscntrl: correct the control character function

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

[incubator-nuttx-apps] branch master updated: wireless/wapi: add set country code support

2021-01-04 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/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new bd9174a wireless/wapi: add set

[GitHub] [incubator-nuttx-apps] jerpelea merged pull request #552: wireless/wapi: add set country code support

2021-01-04 Thread GitBox
jerpelea merged pull request #552: URL: https://github.com/apache/incubator-nuttx-apps/pull/552 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] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #551: make/archive: replace the dependency to preprequisite list

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #551: URL: https://github.com/apache/incubator-nuttx-apps/pull/551#discussion_r551757718 ## File path: Application.mk ## @@ -138,11 +138,11 @@ $(CXXOBJS): %$(SUFFIX)$(OBJEXT): %$(CXXEXT) $(if $(and

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
yamt commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551758572 ## File path: tools/testbuild.sh ## @@ -244,11 +244,11 @@ function configure { varname=`echo $setting | cut -d'=' -f1` if [ ! -z

[GitHub] [incubator-nuttx-apps] jerpelea merged pull request #550: system/readline: check the control ascii only to support multi-languages

2021-01-04 Thread GitBox
jerpelea merged pull request #550: URL: https://github.com/apache/incubator-nuttx-apps/pull/550 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] [incubator-nuttx] btashton commented on a change in pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
btashton commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551760001 ## File path: tools/configure.sh ## @@ -296,7 +296,7 @@ done if [ "X${defappdir}" = "Xy" ]; then # In-place edit can mess up permissions

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2654: Deal with "sed -i" portability

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551760041 ## File path: tools/configure.sh ## @@ -296,7 +296,7 @@ done if [ "X${defappdir}" = "Xy" ]; then # In-place edit can mess up

[GitHub] [incubator-nuttx] btashton commented on a change in pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
btashton commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551716159 ## File path: tools/Makefile.unix ## @@ -505,7 +505,7 @@ gconfig: apps_preconfig savedefconfig: apps_preconfig $(Q)

[GitHub] [incubator-nuttx] yamt commented on pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
yamt commented on pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#issuecomment-754440010 > @yamt can you merge all patch into one? it's important to keep the history clean without the intermediate commit i will merge them later

[GitHub] [incubator-nuttx] yamt opened a new pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
yamt opened a new pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654 ## Summary An empty extension for sed -i is not widely available. * Where it isn't available (eg. macOS's sed) "sed -i -e" will create a backup file with the "-e" suffix. It isn't

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2654: Always specify an extension for "sed -i"

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2654: URL: https://github.com/apache/incubator-nuttx/pull/2654#discussion_r551729511 ## File path: boards/risc-v/nr5m100/nr5m100-nexys4/README.txt ## @@ -143,7 +143,7 @@ Debugger shadowed variable warnings, so

[incubator-nuttx-apps] branch master updated: system/readline: check the control ascii only to support multi-languages

2021-01-04 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/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 54bbfee system/readline: check

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request #550: system/readline: check the control ascii only to support multi-languages

2021-01-04 Thread GitBox
anchao opened a new pull request #550: URL: https://github.com/apache/incubator-nuttx-apps/pull/550 ## Summary system/readline: check the control ascii only to support multi-languages ## Impact ## Testing ``` nsh> echo hello world hello world nsh> echo

[GitHub] [incubator-nuttx] anchao opened a new pull request #2650: ctype/iscntrl: correct the control character function

2021-01-04 Thread GitBox
anchao opened a new pull request #2650: URL: https://github.com/apache/incubator-nuttx/pull/2650 ## Summary ctype/iscntrl: correct the control character function all the characters placed before the space on the ASCII table and the 0x7F character (DEL) are control

[incubator-nuttx] branch master updated: Updated README.txt

2021-01-04 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 4e6ce1f Updated README.txt 4e6ce1f

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2649: Updated README.txt for RSK2M

2021-01-04 Thread GitBox
xiaoxiang781216 merged pull request #2649: URL: https://github.com/apache/incubator-nuttx/pull/2649 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-apps] anchao opened a new pull request #551: make/archive: replace the dependency to preprequisite list

2021-01-04 Thread GitBox
anchao opened a new pull request #551: URL: https://github.com/apache/incubator-nuttx-apps/pull/551 ## Summary make/archive: replace the dependency to preprequisite list Change-Id: I3c3f9354cedb9f4f69c66698cd7556f1de95c660 Signed-off-by: chao.an ## Impact

[GitHub] [incubator-nuttx] anchao opened a new pull request #2651: make/archive: replace the dependency to preprequisite list

2021-01-04 Thread GitBox
anchao opened a new pull request #2651: URL: https://github.com/apache/incubator-nuttx/pull/2651 ## Summary make/archive: replace the dependency to preprequisite list Signed-off-by: chao.an ## Impact N/A ## Testing N/A

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2631: mqueue: simplify the mqueue reailize

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2631: URL: https://github.com/apache/incubator-nuttx/pull/2631#discussion_r551271645 ## File path: fs/mqueue/mq_open.c ## @@ -196,19 +241,18 @@ int nxmq_open(FAR const char *mq_name, int oflags, mode_t mode,

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request #552: wireless/wapi: add set country code support

2021-01-04 Thread GitBox
anchao opened a new pull request #552: URL: https://github.com/apache/incubator-nuttx-apps/pull/552 ## Summary wireless/wapi: add set country code support Signed-off-by: chao.an ## Impact N/A ## Testing wapi wlan0 country CN

[GitHub] [incubator-nuttx] Ouss4 merged pull request #2647: xtensa/esp32: Fix ESP32 serial UART tx ready check error

2021-01-04 Thread GitBox
Ouss4 merged pull request #2647: URL: https://github.com/apache/incubator-nuttx/pull/2647 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

[incubator-nuttx] branch master updated: xtensa/esp32: Fix ESP32 serial UART tx ready check error

2021-01-04 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche 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 fadae0b xtensa/esp32: Fix ESP32

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2063: Fix libcxx for sim/macOS

2021-01-04 Thread GitBox
yamt commented on a change in pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#discussion_r551175295 ## File path: libs/libc/sched/task_startup.c ## @@ -94,6 +94,11 @@ extern uintptr_t _etext; static void cxx_initialize(void) { #ifdef

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2063: Fix libcxx for sim/macOS

2021-01-04 Thread GitBox
yamt commented on a change in pull request #2063: URL: https://github.com/apache/incubator-nuttx/pull/2063#discussion_r551175295 ## File path: libs/libc/sched/task_startup.c ## @@ -94,6 +94,11 @@ extern uintptr_t _etext; static void cxx_initialize(void) { #ifdef

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request #549: Fix pty_test issue during testing

2021-01-04 Thread GitBox
xiaoxiang781216 opened a new pull request #549: URL: https://github.com/apache/incubator-nuttx-apps/pull/549 ## Summary examples/pty_test: Remove O_NONBLOCK from open nsh: Pass the correct command lines to nsh_consolemain ## Impact pts test can run sucesfully ##

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2628: Add generic efuse driver for NuttX

2021-01-04 Thread GitBox
acassis commented on a change in pull request #2628: URL: https://github.com/apache/incubator-nuttx/pull/2628#discussion_r551315486 ## File path: include/nuttx/efuse/efuse.h ## @@ -0,0 +1,294 @@ +/ + *

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2628: Add generic efuse driver for NuttX

2021-01-04 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2628: URL: https://github.com/apache/incubator-nuttx/pull/2628#discussion_r551321958 ## File path: include/nuttx/efuse/efuse.h ## @@ -0,0 +1,294 @@

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2628: Add generic efuse driver for NuttX

2021-01-04 Thread GitBox
acassis commented on a change in pull request #2628: URL: https://github.com/apache/incubator-nuttx/pull/2628#discussion_r551313793 ## File path: include/nuttx/efuse/efuse.h ## @@ -0,0 +1,294 @@ +/ + *

[GitHub] [incubator-nuttx] hartmannathan opened a new pull request #2652: arch/stm32h7: Fix nxstyle errors

2021-01-04 Thread GitBox
hartmannathan opened a new pull request #2652: URL: https://github.com/apache/incubator-nuttx/pull/2652 ## Summary arch/arm/src/stm32h7/stm32_adc.c: arch/arm/src/stm32h7/stm32_adc.h: - Fix nxstyle issues. ## Impact Removes nxstyle errors. ## Testing

[incubator-nuttx] branch master updated: arch/stm32h7: Fix nxstyle errors

2021-01-04 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 4ccaedf arch/stm32h7: Fix nxstyle

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2652: arch/stm32h7: Fix nxstyle errors

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

[incubator-nuttx] branch master updated: sched: sched: Don't call sched_lock()/unlock() in nx_waitid()/waitpid() for SMP

2021-01-04 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 8940646 sched: sched: Don't call

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2653: sched: sched: Don't call sched_lock()/unlock() in nx_waitid()/waitpid() for SMP

2021-01-04 Thread GitBox
xiaoxiang781216 merged pull request #2653: URL: https://github.com/apache/incubator-nuttx/pull/2653 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 opened a new pull request #2653: sched: sched: Don't call sched_lock()/unlock() in nx_waitid()/waitpid() for SMP

2021-01-04 Thread GitBox
masayuki2009 opened a new pull request #2653: URL: https://github.com/apache/incubator-nuttx/pull/2653 ## Summary - Calling sched_lock()/unlock() is unnecessary for SMP because we've been using the critical section API ## Impact - SMP only ## Testing

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2653: sched: sched: Don't call sched_lock()/unlock() in nx_waitid()/waitpid() for SMP

2021-01-04 Thread GitBox
masayuki2009 commented on pull request #2653: URL: https://github.com/apache/incubator-nuttx/pull/2653#issuecomment-754321531 See https://github.com/apache/incubator-nuttx/pull/2298 as well This is an automated message

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

2021-01-04 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 9eb1baa Publishing