[incubator-nuttx] branch master updated (367ec20 -> c230ede)

2021-03-02 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 367ec20 tools/convert-comments:Fix breakage from Sytle fixes add c230ede esp32_ummap: write back

[GitHub] [incubator-nuttx] Ouss4 merged pull request #2947: esp32_ummap: write back spiram cache before calling Cache_Flush

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

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #2947: esp32_ummap: write back spiram cache before calling Cache_Flush

2021-03-02 Thread GitBox
Ouss4 commented on pull request #2947: URL: https://github.com/apache/incubator-nuttx/pull/2947#issuecomment-788724814 > I couldn't find any documentation about Cache_Flush. `Cache_Flush` in ESP32 invalidates cache lines. -

[GitHub] [incubator-nuttx] yamt commented on pull request #2947: esp32_ummap: write back spiram cache before calling Cache_Flush

2021-03-02 Thread GitBox
yamt commented on pull request #2947: URL: https://github.com/apache/incubator-nuttx/pull/2947#issuecomment-788735253 > > I couldn't find any documentation about Cache_Flush. > > `Cache_Flush` in ESP32 invalidates cache lines. invalidates without writing back, you mean? it m

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #2947: esp32_ummap: write back spiram cache before calling Cache_Flush

2021-03-02 Thread GitBox
Ouss4 commented on pull request #2947: URL: https://github.com/apache/incubator-nuttx/pull/2947#issuecomment-788739605 > invalidates without writing back, you mean? @yamt yes, in ESP32, it only invalidates. That's the only cache operation available. For S2 and subsequent chips, ther

[GitHub] [incubator-nuttx] yamt commented on pull request #2947: esp32_ummap: write back spiram cache before calling Cache_Flush

2021-03-02 Thread GitBox
yamt commented on pull request #2947: URL: https://github.com/apache/incubator-nuttx/pull/2947#issuecomment-788744701 > > invalidates without writing back, you mean? > > @yamt yes, in ESP32, it only invalidates. That's the only cache operation available. For S2 and subsequent chips,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2946: sched: task: Call nxtask_flushstreams() without critical section

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2946: URL: https://github.com/apache/incubator-nuttx/pull/2946#discussion_r585377480 ## File path: sched/task/task_exithook.c ## @@ -645,7 +645,20 @@ void nxtask_exithook(FAR struct tcb_s *tcb, int status, bool nonblocking

[GitHub] [incubator-nuttx] michi-jung opened a new pull request #2949: libs/libc: License Clearance

2021-03-02 Thread GitBox
michi-jung opened a new pull request #2949: URL: https://github.com/apache/incubator-nuttx/pull/2949 ## Summary Change the copyright header of assorted source files in libs/libc to the ASF version. I was listed as the author in these files. I did also check the heritage of those f

[incubator-nuttx] branch master updated (c230ede -> fbfddda)

2021-03-02 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 c230ede esp32_ummap: write back spiram cache before calling Cache_Flush add fbfddda armv8-m: Fix EX

[GitHub] [incubator-nuttx] michi-jung commented on pull request #2944: armv8-m: Fix EXC_RETURN for non-secure usage

2021-03-02 Thread GitBox
michi-jung commented on pull request #2944: URL: https://github.com/apache/incubator-nuttx/pull/2944#issuecomment-788749176 Thanks for your feedback, @acassis and @xiaoxiang781216! Do I have to take any further steps to get this pull request merged? ---

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2944: armv8-m: Fix EXC_RETURN for non-secure usage

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2944: URL: https://github.com/apache/incubator-nuttx/pull/2944#issuecomment-788775754 Let wait @acassis feedback. This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-nuttx] jerpelea opened a new pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea opened a new pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950 ## Summary adds CONFIG_AUDIO_FORMAT_RAW as an option to the PCM audio format for devices expecting raw data without a header. ## Impact RAW support for Spresense board

[incubator-nuttx] branch master updated (fbfddda -> cbe3fb5)

2021-03-02 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 fbfddda armv8-m: Fix EXC_RETURN for non-secure usage add cbe3fb5 libs/libc: License Clearance No

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

2021-03-02 Thread GitBox
fjpanag commented on pull request #2347: URL: https://github.com/apache/incubator-nuttx/pull/2347#issuecomment-788829816 @yamt I fixed those in #2951. Nevertheless, I just wanted you to know. I didn't dig into the details of this PR, so maybe you know if this is expected side-effect?

[GitHub] [incubator-nuttx] acassis commented on pull request #2944: armv8-m: Fix EXC_RETURN for non-secure usage

2021-03-02 Thread GitBox
acassis commented on pull request #2944: URL: https://github.com/apache/incubator-nuttx/pull/2944#issuecomment-788802525 thank you @michi-jung and @xiaoxiang781216 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-nuttx] acassis merged pull request #2944: armv8-m: Fix EXC_RETURN for non-secure usage

2021-03-02 Thread GitBox
acassis merged pull request #2944: URL: https://github.com/apache/incubator-nuttx/pull/2944 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 t

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2949: libs/libc: License Clearance

2021-03-02 Thread GitBox
xiaoxiang781216 merged pull request #2949: URL: https://github.com/apache/incubator-nuttx/pull/2949 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] fjpanag opened a new pull request #2951: stm32_wwdg debug log formatting

2021-03-02 Thread GitBox
fjpanag opened a new pull request #2951: URL: https://github.com/apache/incubator-nuttx/pull/2951 ## Summary After #2347 Nuttx is not building with `-Wall -Werror`. I get warnings in stm32_wwdg, due to incorrect formats in the various log messages in this driver. This PR fixe

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2946: sched: task: Call nxtask_flushstreams() without critical section

2021-03-02 Thread GitBox
masayuki2009 commented on a change in pull request #2946: URL: https://github.com/apache/incubator-nuttx/pull/2946#discussion_r585434684 ## File path: sched/task/task_exithook.c ## @@ -645,7 +645,20 @@ void nxtask_exithook(FAR struct tcb_s *tcb, int status, bool nonblocking)

[GitHub] [incubator-nuttx] jerpelea commented on pull request #2949: libs/libc: License Clearance

2021-03-02 Thread GitBox
jerpelea commented on pull request #2949: URL: https://github.com/apache/incubator-nuttx/pull/2949#issuecomment-788782899 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585500906 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /

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

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2347: URL: https://github.com/apache/incubator-nuttx/pull/2347#issuecomment-788855571 I am fine with either approach. This is an automated message from the Apache Git Service. To res

[GitHub] [incubator-nuttx] davids5 commented on pull request #2948: Remve the empty xxx_initialize functions

2021-03-02 Thread GitBox
davids5 commented on pull request #2948: URL: https://github.com/apache/incubator-nuttx/pull/2948#issuecomment-788875479 @xiaoxiang781216 nit Remve -> Remove (I do it all the time) This is an automated message from the Apach

[GitHub] [incubator-nuttx] davids5 commented on pull request #2942: mtd/progmem: Add up_progmem_read callback guarded by ARCH_HAVE_PROGMEM_READ

2021-03-02 Thread GitBox
davids5 commented on pull request #2942: URL: https://github.com/apache/incubator-nuttx/pull/2942#issuecomment-788879778 > What's testing do you ask? The Testing section of this PR blank. The Use case described is a complex series of events. We know is passes build tes

[GitHub] [incubator-nuttx] davids5 commented on a change in pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
davids5 commented on a change in pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#discussion_r585531047 ## File path: arch/arm/src/stm32f7/stm32_lse.c ## @@ -1,9 +1,9 @@ /

[incubator-nuttx] branch master updated (cbe3fb5 -> 9d370fc)

2021-03-02 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 cbe3fb5 libs/libc: License Clearance add 9d370fc sched: task: Call nxtask_flushstreams() without c

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2946: sched: task: Call nxtask_flushstreams() without critical section

2021-03-02 Thread GitBox
xiaoxiang781216 merged pull request #2946: URL: https://github.com/apache/incubator-nuttx/pull/2946 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] xiaoxiang781216 commented on pull request #2948: Remove the empty xxx_initialize functions

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2948: URL: https://github.com/apache/incubator-nuttx/pull/2948#issuecomment-788901372 > @xiaoxiang781216 nit Remve -> Remove (I do it all the time) Done, thanks. This is an au

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585570251 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /* Parse

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585570251 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /* Parse

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

2021-03-02 Thread GitBox
patacongo commented on pull request #2938: URL: https://github.com/apache/incubator-nuttx/pull/2938#issuecomment-788921674 > > > There are many places inovke getpid, we need time to check each place and ensure the right function get called:(. Perhaps we could put the changes

[GitHub] [incubator-nuttx] patacongo commented on pull request #2518: sched: Implement getpid and gettid correctly

2021-03-02 Thread GitBox
patacongo commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-788924592 Perhaps we could put the changes on a branch. I could help out some with the conversion. However, we should wait until after the next 10.1.x release to give the

[GitHub] [incubator-nuttx] patacongo removed a comment on pull request #2518: sched: Implement getpid and gettid correctly

2021-03-02 Thread GitBox
patacongo removed a comment on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-788924592 Perhaps we could put the changes on a branch. I could help out some with the conversion. However, we should wait until after the next 10.1.x release to

[GitHub] [incubator-nuttx] patacongo commented on pull request #2518: sched: Implement getpid and gettid correctly

2021-03-02 Thread GitBox
patacongo commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-788925892 Perhaps we could put the changes on a branch. I could help out some with the conversion. However, we should wait until after the next 10.1.x release to give the

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

2021-03-02 Thread GitBox
patacongo removed a comment on pull request #2938: URL: https://github.com/apache/incubator-nuttx/pull/2938#issuecomment-788921674 > > > There are many places inovke getpid, we need time to check each place and ensure the right function get called:(. Perhaps we could put the

[GitHub] [incubator-nuttx] yunkya2 opened a new pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
yunkya2 opened a new pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952 ## Summary Add I2C driver for Raspberry Pi Pico. ## Impact rp2040 only ## Testing New config `raspberrypi-pico:ssd1306` is added to test I2C OLED display (SSD1306). Gr

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
acassis commented on a change in pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952#discussion_r585611438 ## File path: boards/arm/rp2040/common/src/rp2040_i2cdev.c ## @@ -0,0 +1,68 @@ +/*

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

2021-03-02 Thread GitBox
hartmannathan opened a new pull request #2953: URL: https://github.com/apache/incubator-nuttx/pull/2953 ## Summary Fix nxstyle errors in arch/arm/src/stm32f7/stm32_tim.h. In particular, please review how I eliminated the C++ style comments that nxstyle was complaining about. R

[GitHub] [incubator-nuttx] yunkya2 commented on a change in pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
yunkya2 commented on a change in pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952#discussion_r585621510 ## File path: boards/arm/rp2040/common/src/rp2040_i2cdev.c ## @@ -0,0 +1,68 @@ +/*

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585627633 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585627633 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585637775 ## File path: audio/pcm_decode.c ## @@ -1087,6 +1090,7 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /* Parse

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585650684 ## File path: audio/pcm_decode.c ## @@ -1087,7 +1090,8 @@ static int pcm_enqueuebuffer(FAR struct audio_lowerhalf_s *dev, { /

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585651761 ## File path: audio/pcm_decode.c ## @@ -326,7 +328,7 @@ static uint16_t pcm_leuint32(uint32_t value) * Return true if this is a valid

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585658237 ## File path: audio/pcm_decode.c ## @@ -151,8 +151,10 @@ static uint16_t pcm_leuint32(uint32_t value); # define pcm_leuint32(v) (v) #endif +

[GitHub] [incubator-nuttx] jerpelea opened a new pull request #2954: libs: Author Gregory Nutt: update licenses to Apache

2021-03-02 Thread GitBox
jerpelea opened a new pull request #2954: URL: https://github.com/apache/incubator-nuttx/pull/2954 ## Summary Gregory Nutt has submitted the SGA and we can migrate the licenses to Apache. ## Impact none ## Testing none

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585658237 ## File path: audio/pcm_decode.c ## @@ -151,8 +151,10 @@ static uint16_t pcm_leuint32(uint32_t value); # define pcm_leuint32(v) (v) #endif +

[GitHub] [incubator-nuttx] jerpelea commented on a change in pull request #2950: pcm_decode: add RAW format support

2021-03-02 Thread GitBox
jerpelea commented on a change in pull request #2950: URL: https://github.com/apache/incubator-nuttx/pull/2950#discussion_r585707575 ## File path: audio/pcm_decode.c ## @@ -151,8 +151,10 @@ static uint16_t pcm_leuint32(uint32_t value); # define pcm_leuint32(v) (v) #endif +

[GitHub] [incubator-nuttx] davids5 commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
davids5 commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789074133 @xiaoxiang781216, @acassis - Can this be merged? This is an automated message from the Apache Git Servi

[GitHub] [incubator-nuttx] btashton commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
btashton commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789079047 Sorry @davids5 I have been meaning to test this. Can I merge this tonight? I have hit what I think is this issue on one of my boards and it would be nice to give it a p

[GitHub] [incubator-nuttx] acassis commented on pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
acassis commented on pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952#issuecomment-789079779 Hi @yunkya2 please fix it: Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/arch/arm/src/rp2040/rp2040_i2c.c:515:86: error: Long line found --

[GitHub] [incubator-nuttx] btashton edited a comment on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
btashton edited a comment on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789079047 Sorry @davids5 I have been meaning to test this. Can I merge this tonight? I have hit what I think is this issue on one of my boards and it would be nice to give

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
acassis commented on a change in pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952#discussion_r585768582 ## File path: arch/arm/src/rp2040/rp2040_i2c.c ## @@ -0,0 +1,832 @@ +/

[GitHub] [incubator-nuttx] acassis commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
acassis commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789082085 > Sorry @davids5 I have been meaning to test this. Can I merge this tonight? I have hit what I think is this issue on one of my boards and it would be nice to give it a

[GitHub] [incubator-nuttx] davids5 commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
davids5 commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789081991 > Sorry @davids5 I have been meaning to test this. Can I merge this tonight? Works for me. Thank you @btashton and @acassis !

[GitHub] [incubator-nuttx] acassis merged pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
acassis merged pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943 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 t

[incubator-nuttx] branch master updated (9d370fc -> 1e57542)

2021-03-02 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 9d370fc sched: task: Call nxtask_flushstreams() without critical section new 9fbd7f9 stm32h7:Add op

[incubator-nuttx] 01/02: stm32h7:Add option to auto select LSE CAPABILITY

2021-03-02 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/incubator-nuttx.git commit 9fbd7f9dc5649f7e71db3a06fe4b86a464f9b3ae Author: David Sidrane AuthorDate: Wed Feb 3 15:23:13 2021 -0800

[incubator-nuttx] 02/02: stm32f7:Add option to auto select LSE CAPABILITY

2021-03-02 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/incubator-nuttx.git commit 1e5754232ae8edf936cee441885de44ca251ee9e Author: David Sidrane AuthorDate: Thu Feb 4 06:35:24 2021 -0800

[GitHub] [incubator-nuttx] acassis commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
acassis commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789083440 > Sorry @davids5 I have been meaning to test this. Can I merge this tonight? I have hit what I think is this issue on one of my boards and it would be nice to give it a

[GitHub] [incubator-nuttx] davids5 commented on pull request #2943: stm32x7:lse ensure it is started

2021-03-02 Thread GitBox
davids5 commented on pull request #2943: URL: https://github.com/apache/incubator-nuttx/pull/2943#issuecomment-789086997 @acassis - I think we are all good. No Risk: It is behind a Kconfig. It has been cross tested and more is just better. @btashton feedback is always appreciated! So pleas

[GitHub] [incubator-nuttx] davids5 opened a new pull request #2955: stm32h7:Add DBGMCU missing from master

2021-03-02 Thread GitBox
davids5 opened a new pull request #2955: URL: https://github.com/apache/incubator-nuttx/pull/2955 ## Summary This adds the DBGMCU ## Impact Missing from master. ## Testing with https://github.com/apache/incubator-nuttx/pull/2943 -

[GitHub] [incubator-nuttx] davids5 commented on pull request #2955: stm32h7:Add DBGMCU missing from master

2021-03-02 Thread GitBox
davids5 commented on pull request #2955: URL: https://github.com/apache/incubator-nuttx/pull/2955#issuecomment-789106551 @btashton - I forgot to get this in before https://github.com/apache/incubator-nuttx/pull/2943 It is needed, when the option is enabled.

[GitHub] [incubator-nuttx] Ouss4 merged pull request #2953: arch/stm32f7: Fix nxstyle errors

2021-03-02 Thread GitBox
Ouss4 merged pull request #2953: URL: https://github.com/apache/incubator-nuttx/pull/2953 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 (1e57542 -> a3f0923)

2021-03-02 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 1e57542 stm32f7:Add option to auto select LSE CAPABILITY add a3f0923 arch/stm32f7: Fix nxstyle e

[GitHub] [incubator-nuttx] yunkya2 commented on a change in pull request #2952: arm/rp2040: Add I2C driver support

2021-03-02 Thread GitBox
yunkya2 commented on a change in pull request #2952: URL: https://github.com/apache/incubator-nuttx/pull/2952#discussion_r585956470 ## File path: arch/arm/src/rp2040/rp2040_i2c.c ## @@ -0,0 +1,832 @@ +/

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

2021-03-02 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 35fc5ba Publishing web

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2951: stm32_wwdg debug log formatting

2021-03-02 Thread GitBox
yamt commented on a change in pull request #2951: URL: https://github.com/apache/incubator-nuttx/pull/2951#discussion_r586052480 ## File path: arch/arm/src/stm32/stm32_wwdg.c ## @@ -591,8 +591,8 @@ static int stm32_settimeout(FAR struct watchdog_lowerhalf_s *lower, priv->fw

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

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

[incubator-nuttx] 01/02: esp32/esp32_wifi_adapter.c: Fix the issue of WiFi internal malloc from PSRAM

2021-03-02 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 516c553b9744fa38d1eed42c272c0b2c87f0070d Author: chenwen AuthorDate: Mon Feb 8 10:18:23 2021 +0800 esp3

[incubator-nuttx] branch master updated (a3f0923 -> 1962709)

2021-03-02 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 a3f0923 arch/stm32f7: Fix nxstyle errors new 516c553 esp32/esp32_wifi_adapter.c: Fix the issue of

[incubator-nuttx] 02/02: esp32/esp32_allocateheap.c: Support the maximum available internal heap configuration

2021-03-02 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 19627095e4aa7146a0ebcbd9519b3eaffc7eb9b2 Author: chenwen AuthorDate: Mon Feb 8 17:16:04 2021 +0800 esp3

[incubator-nuttx] branch master updated (1962709 -> 41b0298)

2021-03-02 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 1962709 esp32/esp32_allocateheap.c: Support the maximum available internal heap configuration add

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2950: pcm_decode: add RAW format support

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

[incubator-nuttx] branch master updated: stm32h7:Add DBGMCU

2021-03-02 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 ab5f46d stm32h7:Add DBGMCU ab5f46d

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2955: stm32h7:Add DBGMCU missing from master

2021-03-02 Thread GitBox
xiaoxiang781216 merged pull request #2955: URL: https://github.com/apache/incubator-nuttx/pull/2955 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] xiaoxiang781216 commented on pull request #2518: sched: Implement getpid and gettid correctly

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-789383285 > Perhaps we could put the changes on a branch. I could help out some with the conversion. > Thanks for helping. > However, we should wait until

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2021-03-02 Thread GitBox
gustavonihei commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r558337841 ## File path: sched/task/task_gettid.c ## @@ -45,5 +50,42 @@ pid_t gettid(void) { - return getpid(); + FAR struct tcb_s *rtcb; + + /*

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2899: Unify socket into file system to avoid the code duplication

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2899: URL: https://github.com/apache/incubator-nuttx/pull/2899#issuecomment-789391282 This patch pass our internal test, does anyone have more concern to address? This is an automate

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2942: mtd/progmem: Add up_progmem_read callback guarded by ARCH_HAVE_PROGMEM_READ

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2942: URL: https://github.com/apache/incubator-nuttx/pull/2942#issuecomment-789397552 > > What's testing do you ask? > > The Testing section of this PR blank. > > The Use case described is a complex series of events. > > We know

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2956: fs/mount: fix mount cmd break out when meet stat error

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2956: URL: https://github.com/apache/incubator-nuttx/pull/2956 ## Summary fs/mount: fix mount cmd break out when meet stat error ## Impact when mount cmd meet some mountpoint statfs error, then continue. still return info->handler erro

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2957: mqueue: add poll support

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2957: URL: https://github.com/apache/incubator-nuttx/pull/2957 ## Summary mqueue: add poll support ## Impact mqeue, can support poll ## Testing This is an a

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2958: sched/task: fix pthread call exit() failed

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2958: URL: https://github.com/apache/incubator-nuttx/pull/2958 ## Summary sched/task: fix pthread call exit() failed ## Impact pthread call pthread_exit() => current pthread exit pthread call exit()=> current task exit

[GitHub] [incubator-nuttx] PeterBee97 opened a new pull request #2959: net/socket: move si_send/recv into sendmsg/recvmsg

2021-03-02 Thread GitBox
PeterBee97 opened a new pull request #2959: URL: https://github.com/apache/incubator-nuttx/pull/2959 ## Summary Implement si_send/sendto/recvfrom with si_sendmsg/recvmsg, instead of the other way round. Multi-iov cases has been handled in Unix domain sockets and inet sockets.

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2960: sigdeliver: fix system block when kill signal to idle in SMP

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2960: URL: https://github.com/apache/incubator-nuttx/pull/2960 ## Summary sigdeliver: fix system block when kill signal to idle in SMP Bug description: CONFIG_SMP=y Suppose we have 2 cores in SMP, here is the ps

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2961: syslog_rpmsg: fix system crash when open CONFIG_SCHED_HAVE_PARENT

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2961: URL: https://github.com/apache/incubator-nuttx/pull/2961 ## Summary syslog_rpmsg: fix system crash when open CONFIG_SCHED_HAVE_PARENT ## Impact Reason: nx_start use syslog at every early time event the idle thread not full

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2962: sim/up_uart.c: fix losting uart data when user paste long cmd

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2962: URL: https://github.com/apache/incubator-nuttx/pull/2962 ## Summary sim/up_uart.c: fix losting uart data when user paste long cmd ## Impact In SIM shell, when use ctrl+c crtl+v to paste long command to shell. Always cut off by si

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2963: debug tools: add heap & stack check in idle thread

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2963: URL: https://github.com/apache/incubator-nuttx/pull/2963 ## Summary debug tools: add heap & stack check in idle thread ## Impact We always meet heap corruption or stack overflow. Here provide one method to check heap & stack

[GitHub] [incubator-nuttx] GUIDINGLI opened a new pull request #2964: socket_rpmsg: add socket rpmsg support

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #2964: URL: https://github.com/apache/incubator-nuttx/pull/2964 ## Summary socket_rpmsg: add socket rpmsg support ## Impact This is for apps to use rpmsg easier, with the socket C/S mode, user can easy to realize communication between 2 or

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2958: sched/task: fix pthread call exit() failed

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2958: URL: https://github.com/apache/incubator-nuttx/pull/2958#issuecomment-789449338 https://github.com/apache/incubator-nuttx/pull/2518 This is an automated message from the Apache

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2964: socket_rpmsg: add socket rpmsg support

2021-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #2964: URL: https://github.com/apache/incubator-nuttx/pull/2964#issuecomment-789468867 @GUIDINGLI let merge the intermediate patches into one This is an automated message from the Apa

[GitHub] [incubator-nuttx-apps] GUIDINGLI opened a new pull request #604: apps/md5: fix md5 result error

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #604: URL: https://github.com/apache/incubator-nuttx-apps/pull/604 ## Summary apps/md5: fix md5 result error ## Impact ## Testing This is an automated messa

[GitHub] [incubator-nuttx-apps] GUIDINGLI opened a new pull request #605: rpmsgsocket: add rpmsgsocket test case

2021-03-02 Thread GitBox
GUIDINGLI opened a new pull request #605: URL: https://github.com/apache/incubator-nuttx-apps/pull/605 ## Summary rpmsgsocket: add rpmsgsocket test case ## Impact ## Testing This is an automat

[GitHub] [incubator-nuttx-apps] GUIDINGLI commented on pull request #605: rpmsgsocket: add rpmsgsocket test case

2021-03-02 Thread GitBox
GUIDINGLI commented on pull request #605: URL: https://github.com/apache/incubator-nuttx-apps/pull/605#issuecomment-789473712 Test case of https://github.com/apache/incubator-nuttx/pull/2964 This is an automated message from

[GitHub] [incubator-nuttx] GUIDINGLI commented on pull request #2964: socket_rpmsg: add socket rpmsg support

2021-03-02 Thread GitBox
GUIDINGLI commented on pull request #2964: URL: https://github.com/apache/incubator-nuttx/pull/2964#issuecomment-789473874 Test case: https://github.com/apache/incubator-nuttx-apps/pull/605 This is an automated message fro

[GitHub] [incubator-nuttx-apps] Donny9 opened a new pull request #606: netutils/thttpd: fix compile break

2021-03-02 Thread GitBox
Donny9 opened a new pull request #606: URL: https://github.com/apache/incubator-nuttx-apps/pull/606 ## Summary netutils/thttpd: fix compile break change NSOCKET_DESCRIPTORS to NFILE_DESCRIPTORS Signed-off-by: Jiuzhu Dong ## Impact ## Testing

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2960: sigdeliver: fix system block when kill signal to idle in SMP

2021-03-02 Thread GitBox
masayuki2009 commented on pull request #2960: URL: https://github.com/apache/incubator-nuttx/pull/2960#issuecomment-789517270 @GUIDINGLI I can reproduce the bug with sabre6quad:smp and confirmed that this PR fixes the problem. Could you please apply the same logic to armv6-m/arm_