[GitHub] [incubator-nuttx] codebje opened a new pull request #2860: eZ80: fix a variety of build errors

2021-02-16 Thread GitBox
codebje opened a new pull request #2860: URL: https://github.com/apache/incubator-nuttx/pull/2860 ## Summary A small number of issues prevent successful builds of arch z80, chip eZ80. Most are compile-time problems, typos and the like, though the change to `irq.h` is required to cor

[GitHub] [incubator-nuttx-apps] yamt commented on a change in pull request #594: ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage

2021-02-16 Thread GitBox
yamt commented on a change in pull request #594: URL: https://github.com/apache/incubator-nuttx-apps/pull/594#discussion_r577328781 ## File path: netutils/ntpclient/ntpclient.c ## @@ -625,14 +699,15 @@ static void ntpc_settime(int64_t offset, FAR struct timespec *start_realti

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #594: ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage

2021-02-16 Thread GitBox
xiaoxiang781216 commented on a change in pull request #594: URL: https://github.com/apache/incubator-nuttx-apps/pull/594#discussion_r577326062 ## File path: netutils/ntpclient/ntpclient.c ## @@ -625,14 +699,15 @@ static void ntpc_settime(int64_t offset, FAR struct timespec *s

[incubator-nuttx] branch master updated: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread yamamoto
This is an automated email from the ASF dual-hosted git repository. yamamoto 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 102adaf arch: esp32: Fix a memory le

[GitHub] [incubator-nuttx] yamt merged pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
yamt merged pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858 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 g

[GitHub] [incubator-nuttx] yamt commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
yamt commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-780279338 > > > I noticed that esp_wifi_free_eb() must be called before calling work_queue(), otherwise Wi-Fi stops in SMP mode after 5mins. > > > > > > do you understand w

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-780245519 > > I noticed that esp_wifi_free_eb() must be called before calling work_queue(), otherwise Wi-Fi stops in SMP mode after 5mins. > > do you understand why? i

[GitHub] [incubator-nuttx] yamt commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
yamt commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-780240056 > I noticed that esp_wifi_free_eb() must be called before calling work_queue(), otherwise Wi-Fi stops in SMP mode after 5mins. do you understand why? i don't. -

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

2021-02-16 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 0adc220 Publishing web

[GitHub] [incubator-nuttx-apps] yamt commented on a change in pull request #594: ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage

2021-02-16 Thread GitBox
yamt commented on a change in pull request #594: URL: https://github.com/apache/incubator-nuttx-apps/pull/594#discussion_r577214282 ## File path: netutils/ntpclient/ntpclient.c ## @@ -625,14 +699,15 @@ static void ntpc_settime(int64_t offset, FAR struct timespec *start_realti

[GitHub] [incubator-nuttx-apps] btashton commented on pull request #595: canutils: changed print_usage function to static

2021-02-16 Thread GitBox
btashton commented on pull request #595: URL: https://github.com/apache/incubator-nuttx-apps/pull/595#issuecomment-780156863 > Oh, wasn´t expecting that much nx style errors. Gonna have to get some sleep before fixing this, I´ll push nx style changes tomorrow. I think for canutils w

[GitHub] [incubator-nuttx-apps] michallenc commented on pull request #595: canutils: changed print_usage function to static

2021-02-16 Thread GitBox
michallenc commented on pull request #595: URL: https://github.com/apache/incubator-nuttx-apps/pull/595#issuecomment-780154546 Oh, wasn´t expecting that much nx style errors. Gonna have to get some sleep before fixing this, I´ll push nx style changes tomorrow. ---

[GitHub] [incubator-nuttx-apps] michallenc opened a new pull request #595: canutils: changed print_usage function to static

2021-02-16 Thread GitBox
michallenc opened a new pull request #595: URL: https://github.com/apache/incubator-nuttx-apps/pull/595 ## Summary Global print_usage function in canutils/candump/ was causing double definition error while compiling NuttX with some extern apps, like pysimCoder for example. Therefore the

[GitHub] [incubator-nuttx-apps] btashton commented on issue #592: nimble sim configuration is breaking CI beacuse of pointer-to-int-cast error

2021-02-16 Thread GitBox
btashton commented on issue #592: URL: https://github.com/apache/incubator-nuttx-apps/issues/592#issuecomment-779941110 I think I would like to avoid it if possible, they already have a PR up to fix it. In this case it actually is calling out a bug. I suspect they did not see it because t

[GitHub] [incubator-nuttx-apps] v01d commented on issue #592: nimble sim configuration is breaking CI beacuse of pointer-to-int-cast error

2021-02-16 Thread GitBox
v01d commented on issue #592: URL: https://github.com/apache/incubator-nuttx-apps/issues/592#issuecomment-779847135 Sorry for the issue, don't know why I didn't do it sooner. Thanks for the fix. I think that this kind of errors will have to be dealt with disabling certainly warnings wh

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #2833: Refine the interaction between fs and net

2021-02-16 Thread GitBox
xiaoxiang781216 commented on issue #2833: URL: https://github.com/apache/incubator-nuttx/issues/2833#issuecomment-779818934 The more detail design can found here: https://github.com/apache/incubator-nuttx/issues/2834 This i

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue #2859: Migrate all protocol from si_send/si_sendto/si_recvfrom to si_sendmsg/si_recvmsg

2021-02-16 Thread GitBox
xiaoxiang781216 opened a new issue #2859: URL: https://github.com/apache/incubator-nuttx/issues/2859 si_sendmsg/si_recmsg is added to support CAN bus by: https://github.com/apache/incubator-nuttx/pull/581 Since si_sendmsg/si_recmsg is superset si_send/si_sendto/si_recvfrom, it's bette

[GitHub] [incubator-nuttx] patacongo commented on pull request #2857: boards: esp32: Set CONFIG_NET_ETH_PKTSIZE to 1514

2021-02-16 Thread GitBox
patacongo commented on pull request #2857: URL: https://github.com/apache/incubator-nuttx/pull/2857#issuecomment-779804941 Sounds good to me.  Or MM_SMALL?Sent from my Galaxy Original message From: Xiang Xiao Date: 2/15/21 11:43 PM (GMT-06:00) To: apache/incubator-nu

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #590: nshlib: Fix dependency on FSUTILS_MKFATFS

2021-02-16 Thread GitBox
xiaoxiang781216 merged pull request #590: URL: https://github.com/apache/incubator-nuttx-apps/pull/590 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] xiaoxiang781216 edited a comment on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
xiaoxiang781216 edited a comment on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779787240 > Hmm there are some important changes in mine that are not in that implemention both for the framebuffer and lcd driver. How do you want to bring the

[incubator-nuttx-apps] branch master updated: nshlib: Fix dependency on FSUTILS_MKFATFS

2021-02-16 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 5cddb39 nshlib: Fix dependency

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 edited a comment on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
xiaoxiang781216 edited a comment on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779787240 > Hmm there are some important changes in mine that are not in that implemention both for the framebuffer and lcd driver. How do you want to bring the

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
xiaoxiang781216 commented on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779787240 > Hmm there are some important changes in mine that are not in that implemention both for the framebuffer and lcd driver. How do you want to bring these toge

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779735021 The latest commit works in both non-SMP and SMP mode. Stress tests are still running over 1 hour. ---

[GitHub] [incubator-nuttx-apps] btashton edited a comment on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
btashton edited a comment on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779719760 Hmm there are some important changes in mine that are not in that implemention both for the framebuffer and lcd driver. How do you want to bring these togeth

[GitHub] [incubator-nuttx-apps] btashton commented on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
btashton commented on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779719760 Hmm there are some important changes in mine that are not in that implemention both for the framebuffer and lcd driver. How do you want to bring these together then

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #591: lvgldemo: Add support for lcddev

2021-02-16 Thread GitBox
xiaoxiang781216 commented on pull request #591: URL: https://github.com/apache/incubator-nuttx-apps/pull/591#issuecomment-779712668 lcd driver duplicate with this: https://github.com/apache/incubator-nuttx-apps/pull/580 Thi

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #594: ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage

2021-02-16 Thread GitBox
xiaoxiang781216 commented on a change in pull request #594: URL: https://github.com/apache/incubator-nuttx-apps/pull/594#discussion_r576680088 ## File path: netutils/ntpclient/ntpclient.c ## @@ -625,14 +699,15 @@ static void ntpc_settime(int64_t offset, FAR struct timespec *s

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779699897 >Let me check if the code works in SMP mode which I've just started for testing Hmm, I need to modify the code again. I noticed that esp_wifi_free_eb() mus

[GitHub] [incubator-nuttx] masayuki2009 edited a comment on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 edited a comment on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779694233 @yamt Let me check if the code works in SMP mode which I've just started for testing ---

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779694233 @yamt Let me check if the code works in SMP mode. This is an automated message from the

[GitHub] [incubator-nuttx-apps] Ouss4 merged pull request #593: Pin Nimble release to working commit until stable tag is ready

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

[incubator-nuttx-apps] branch master updated: Pin Nible release to working commit until stable tag is ready

2021-02-16 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-apps.git The following commit(s) were added to refs/heads/master by this push: new 75690bc Pin Nible release to

[GitHub] [incubator-nuttx] masayuki2009 edited a comment on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 edited a comment on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779688028 @yamt I've just refactored the code and pushed with -f. Also, updated the commit log and summary for this PR -

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#issuecomment-779688028 @yamt I've just refactored the code and pushed with -f. This is an automated message fr

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
masayuki2009 commented on a change in pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#discussion_r576639268 ## File path: arch/xtensa/src/esp32/esp32_wlan.c ## @@ -493,6 +493,12 @@ static int wlan_rx_done(void *buffer, uint16_t len, void *eb)

[GitHub] [incubator-nuttx-apps] Ouss4 commented on pull request #593: Pin Nimble release to working commit until stable tag is ready

2021-02-16 Thread GitBox
Ouss4 commented on pull request #593: URL: https://github.com/apache/incubator-nuttx-apps/pull/593#issuecomment-779675872 @xiaoxiang781216 let's not wait for the macOS builds please. This will take hours and we need to fix the CI.

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2858: arch: esp32: Fix a memory leak when discarding a large packet.

2021-02-16 Thread GitBox
yamt commented on a change in pull request #2858: URL: https://github.com/apache/incubator-nuttx/pull/2858#discussion_r576629818 ## File path: arch/xtensa/src/esp32/esp32_wlan.c ## @@ -493,6 +493,12 @@ static int wlan_rx_done(void *buffer, uint16_t len, void *eb) {

[GitHub] [incubator-nuttx-apps] yamt opened a new pull request #594: ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage

2021-02-16 Thread GitBox
yamt opened a new pull request #594: URL: https://github.com/apache/incubator-nuttx-apps/pull/594 ## Summary ntpclient.c: Replace home-grown ntp_addr_u with sockaddr_storage ## Impact ## Testing It tested IPv4 with lm3s6965-ek:qemu-protected on qemu. IPv6 is not tested