[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
pkarashchenko commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1206111060 > > > How do you think if we add a KCONFIG option to turn it on or off? like following: > > > > > > Although this is a improvement, if you bring in your TCP/IP rea

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r938501730 ## libs/libc/time/lib_localtime.c: ## @@ -307,14 +307,15 @@ struct rule_s /* The minimum and maximum finite time values. */ -static const time_t g_mi

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6788: LPC17xx_40xx PWM multichannel support

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6788: URL: https://github.com/apache/incubator-nuttx/pull/6788#discussion_r938494873 ## arch/arm/src/lpc17xx_40xx/lpc17_40_pwm.c: ## @@ -96,16 +96,22 @@ /* This structure represents the state of one PWM timer */ +struct lpc17_40_pwmcha

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6787: imxrt: Ethernet Cleanup and extend support for ETH1 or ETH2 and LAN8742A

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6787: URL: https://github.com/apache/incubator-nuttx/pull/6787#discussion_r938489101 ## arch/arm/src/imxrt/imxrt_enet.c: ## @@ -269,34 +314,48 @@ struct imxrt_driver_s struct net_driver_s dev; /* Interface understood by the network *

[incubator-nuttx] branch master updated: arch/stm32f0l0g0: add SPI3 support (STM32G0B0 chips)

2022-08-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 93584f8668 arch/stm32f0l0g0: add SPI

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6769: arch/stm32f0l0g0: add SPI3 support (STM32G0B0 chips)

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6769: URL: https://github.com/apache/incubator-nuttx/pull/6769 -- 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-unsubs

[incubator-nuttx-apps] branch master updated: netutils/iperf: fix string for iperf test interface option

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new 672b5ffbc netutils/iperf: fix s

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1251: netutils/iperf: fix string for iperf test interface option

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1251: URL: https://github.com/apache/incubator-nuttx-apps/pull/1251 -- 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-u

[incubator-nuttx-apps] branch master updated: system/cfgdata: add "format" command to description

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new 6d730e3c2 system/cfgdata: add "

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1252: system/cfgdata: add "format" command to description

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1252: URL: https://github.com/apache/incubator-nuttx-apps/pull/1252 -- 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-u

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1253: Embedded Template Library (ETL) Example application

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #1253: URL: https://github.com/apache/incubator-nuttx-apps/pull/1253#issuecomment-1206072999 @PetervdPerk it's better to enable ETL and example in some config to ensure that the future change don't break it. -- This is an automated message from the Apache

[incubator-nuttx-apps] branch master updated: Embedded Template Library (ETL) Example application

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new 917978390 Embedded Template Lib

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1253: Embedded Template Library (ETL) Example application

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1253: URL: https://github.com/apache/incubator-nuttx-apps/pull/1253 -- 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-u

[incubator-nuttx] 03/03: boards: Enable thread local storage for some config

2022-08-04 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 35fe1cd7d781a4669887faf0f05de8d91c117c30 Author: Xiang Xiao AuthorDate: Wed Aug 3 00:25:27 2022 +0800

[incubator-nuttx] 02/03: arch/risc-v: Fix error: invalid application of 'sizeof' to incomplete type 'struct tls_info_s'

2022-08-04 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit c61381da56babf62d1c130e2db374fe3ee159f4b Author: Xiang Xiao AuthorDate: Wed Aug 3 21:30:41 2022 +0800

[incubator-nuttx] branch master updated (33efeeeafd -> 35fe1cd7d7)

2022-08-04 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/incubator-nuttx.git from 33efeeeafd imxrt:EDMA add loop support new a623b5f31e tls: fix conflicting types for 'up_tls_ini

[incubator-nuttx] 01/03: tls: fix conflicting types for 'up_tls_initialize' compilation problem

2022-08-04 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit a623b5f31ee892beee5586d7522e6c0db3d38f64 Author: sheyunhu AuthorDate: Thu Aug 4 10:36:43 2022 +0800

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #6777: boards: Enable thread local storage for some config

2022-08-04 Thread GitBox
pkarashchenko merged PR #6777: URL: https://github.com/apache/incubator-nuttx/pull/6777 -- 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

[GitHub] [incubator-nuttx] michi-jung commented on pull request #6353: Add driver for WIZnet W5500 Ethernet controller

2022-08-04 Thread GitBox
michi-jung commented on PR #6353: URL: https://github.com/apache/incubator-nuttx/pull/6353#issuecomment-1206051094 > @michi-jung could you please try #6789 ? @pkarashchenko Gave it a try and works fine for me. Thanks! -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205995200 > It might not be obvious that the design patterns used in NuttX were intentionally very memory/resource conscious from the start. These patterns are extremely important t

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205993756 > > How do you think if we add a KCONFIG option to turn it on or off? like following: > > Although this is a improvement, if you bring in your TCP/IP reassemble cha

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6787: imxrt: Ethernet Cleanup and extend support for ETH1 or ETH2 and LAN8742A

2022-08-04 Thread GitBox
xiaoxiang781216 commented on code in PR #6787: URL: https://github.com/apache/incubator-nuttx/pull/6787#discussion_r938398751 ## arch/arm/src/imxrt/imxrt_enet.c: ## @@ -125,6 +125,12 @@ # error Write back D-Cache not yet supported #endif +/* TX poll delay = 1 seconds. CLK_T

[GitHub] [incubator-nuttx] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205989646 OK, I need a few days to prepare the draft version of IP reassembly before uploading -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
xiaoxiang781216 commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r938396437 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6777: boards: Enable thread local storage for some config

2022-08-04 Thread GitBox
xiaoxiang781216 commented on code in PR #6777: URL: https://github.com/apache/incubator-nuttx/pull/6777#discussion_r938394648 ## boards/arm/kinetis/kwikstik-k40/scripts/kwikstik-k40.ld: ## @@ -86,6 +86,18 @@ SECTIONS __exidx_end = ABSOLUTE(.); } > progflash +

[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
Donny9 commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r938390359 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[incubator-nuttx] branch master updated: imxrt:EDMA add loop support

2022-08-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 33efeeeafd imxrt:EDMA add loop suppo

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786 -- 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-unsubs

[incubator-nuttx] 02/03: imxrt:usdhc fix error when CONFIG_ARMV7M_DCACHE is off

2022-08-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 commit 5e902861d9a19f2d20ccf8ebea171f46dac89391 Author: David Sidrane AuthorDate: Mon Jun 6 15:35:41 2022 -0700

[incubator-nuttx] 01/03: imxrt:lowputc add extern "C"

2022-08-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 commit b9c6284750d367b3ad9d2bfbcefc710fda2cf135 Author: David Sidrane AuthorDate: Wed Mar 16 13:21:44 2022 -0700

[incubator-nuttx] 03/03: imxrt:Correct register usage in up_rtc_settime

2022-08-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 commit 1421bc58dbd8936fa5be8b1f53294666343e62c2 Author: David Sidrane AuthorDate: Mon Jul 18 13:58:41 2022 -0700

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6790: Several IMXRT fixes

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6790: URL: https://github.com/apache/incubator-nuttx/pull/6790 -- 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-unsubs

[incubator-nuttx] branch master updated (f64677b8c9 -> 1421bc58db)

2022-08-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 f64677b8c9 Add Embedded Template Library (ETL) support new b9c6284750 imxrt:lowputc add extern "C"

[incubator-nuttx] branch master updated: Add Embedded Template Library (ETL) support

2022-08-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 f64677b8c9 Add Embedded Template Lib

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6791: Add Embedded Template Library (ETL) support

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6791: URL: https://github.com/apache/incubator-nuttx/pull/6791 -- 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-unsubs

[incubator-nuttx-website] branch asf-site updated: Publishing web: 6758fc5efbbf5ed16c807ccdfb012d94d1db201a docs: 20bdd44e7b8bb233ab3055ace3c118c2a0061f21

2022-08-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 a7969d6 Publishing web

[GitHub] [incubator-nuttx-apps] PetervdPerk opened a new pull request, #1253: Embedded Template Library (ETL) Example application

2022-08-04 Thread GitBox
PetervdPerk opened a new pull request, #1253: URL: https://github.com/apache/incubator-nuttx-apps/pull/1253 ## Summary Example application for the Embedded Template Library (ETL). apache/incubator-nuttx#6791 -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-nuttx] PetervdPerk opened a new pull request, #6791: Add Embedded Template Library (ETL) support

2022-08-04 Thread GitBox
PetervdPerk opened a new pull request, #6791: URL: https://github.com/apache/incubator-nuttx/pull/6791 ## Summary Adds support for the Embedded Template Library (ETL). Which implements various embedded systems friendly C++ classes such as etl::vector without the need of the STL library

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6777: boards: Enable thread local storage for some config

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6777: URL: https://github.com/apache/incubator-nuttx/pull/6777#discussion_r938149598 ## boards/arm/kinetis/kwikstik-k40/scripts/kwikstik-k40.ld: ## @@ -86,6 +86,18 @@ SECTIONS __exidx_end = ABSOLUTE(.); } > progflash +.t

[GitHub] [incubator-nuttx] davids5 opened a new pull request, #6790: Several IMXRT fixes

2022-08-04 Thread GitBox
davids5 opened a new pull request, #6790: URL: https://github.com/apache/incubator-nuttx/pull/6790 ## Summary 2 compiler error fixes and one driver change making the RTC be set able ## Impact Code will build with CPP and if CONFIG_ARMV7M_DCACHE is off ## Testing

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786#discussion_r938103711 ## arch/arm/src/imxrt/imxrt_edma.c: ## @@ -404,13 +394,16 @@ static inline void imxrt_tcd_configure(struct imxrt_edmatcd_s *tcd, tcd->attr = EDMA_T

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6353: Add driver for WIZnet W5500 Ethernet controller

2022-08-04 Thread GitBox
pkarashchenko commented on PR #6353: URL: https://github.com/apache/incubator-nuttx/pull/6353#issuecomment-1205584357 @michi-jung could you please try https://github.com/apache/incubator-nuttx/pull/6789 ? -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [incubator-nuttx] pkarashchenko opened a new pull request, #6789: drivers/net: make sure that net driver d_buf is 16-bit aligned

2022-08-04 Thread GitBox
pkarashchenko opened a new pull request, #6789: URL: https://github.com/apache/incubator-nuttx/pull/6789 ## Summary - make sure that net driver `d_buf` is 16-bit aligned - fix buffer selection multi-instance netdev drivers - add missing `FAR` in multiple palces - perform some ren

[GitHub] [incubator-nuttx] davids5 commented on pull request #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
davids5 commented on PR #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786#issuecomment-1205565812 @pkarashchenko - Thank you for the review! -- 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

[GitHub] [incubator-nuttx] PetervdPerk opened a new pull request, #6788: LPC17xx_40xx PWM multichannel support

2022-08-04 Thread GitBox
PetervdPerk opened a new pull request, #6788: URL: https://github.com/apache/incubator-nuttx/pull/6788 ## Summary Adds support for multichannel in the LPC17xx_40xx PWM Furthermore some small changes - USB no softconnect support - SocketCAN Kconfig not fully implemented for

[GitHub] [incubator-nuttx] davids5 commented on a diff in pull request #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
davids5 commented on code in PR #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786#discussion_r938063697 ## arch/arm/src/imxrt/imxrt_edma.c: ## @@ -86,22 +90,16 @@ */ #ifdef CONFIG_ARMV7M_DCACHE -/* Align to the cache line size which we assume is >= 8 */ - -#

[GitHub] [incubator-nuttx] davids5 opened a new pull request, #6787: imxrt: Ethernet Cleanup and extend support for ETH1 or ETH2 and LAN8742A

2022-08-04 Thread GitBox
davids5 opened a new pull request, #6787: URL: https://github.com/apache/incubator-nuttx/pull/6787 ## Summary - Added Support for LAN8742A PHY - All boards ARCH_PHY_INTERRUPT is a board property - add IMXRT_PHY_POLLING - Support either ETH1 or ETH2 - pinmux Add ENET2 and co

[GitHub] [incubator-nuttx-apps] pkarashchenko opened a new pull request, #1252: system/cfgdata: add "format" command to description

2022-08-04 Thread GitBox
pkarashchenko opened a new pull request, #1252: URL: https://github.com/apache/incubator-nuttx-apps/pull/1252 ## Summary Add missing description to help message ## Impact None ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-nuttx-apps] pkarashchenko opened a new pull request, #1251: netutils/iperf: fix string for iperf test interface option

2022-08-04 Thread GitBox
pkarashchenko opened a new pull request, #1251: URL: https://github.com/apache/incubator-nuttx-apps/pull/1251 ## Summary Fix description as iperf can run not only with Wi-Fi network interface ## Impact None ## Testing Pass CI -- This is an automated message from the

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r938006992 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786#discussion_r937998801 ## arch/arm/src/imxrt/imxrt_edma.c: ## @@ -86,22 +90,16 @@ */ #ifdef CONFIG_ARMV7M_DCACHE -/* Align to the cache line size which we assume is >= 8 */

[GitHub] [incubator-nuttx] davids5 opened a new pull request, #6786: IMX.RT EDMA Support

2022-08-04 Thread GitBox
davids5 opened a new pull request, #6786: URL: https://github.com/apache/incubator-nuttx/pull/6786 ## Summary This PR cleans up the it IMX.RT DMA code and makes it functional - Header file clean up - API fixed DMACH_HANDLE is a value (blind pointer) not a pointer to the DMAC

[incubator-nuttx] branch master updated: Remove executable permission from source and build files.

2022-08-04 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 The following commit(s) were added to refs/heads/master by this push: new 20bdd44e7b Remove executable permissio

[GitHub] [incubator-nuttx] acassis merged pull request #6784: Remove executable permission from source and build files.

2022-08-04 Thread GitBox
acassis merged PR #6784: URL: https://github.com/apache/incubator-nuttx/pull/6784 -- 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...@nu

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6781: Multiple improvements for stm32wl5 board nucleo-stm32wl5jc

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #6781: URL: https://github.com/apache/incubator-nuttx/pull/6781#issuecomment-1205421458 @mlyszczek please fix the style warning. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6746: sched: Remove KEEP_ALIVE_HACK hack

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #6746: URL: https://github.com/apache/incubator-nuttx/pull/6746#issuecomment-1205420043 > @xiaoxiang781216 is the some testing to guarantee that sporadic scheduling is working? I found a good explanation here: https://www.embedded.com/scheduling-sporadic-even

[incubator-nuttx] branch master updated: bluetooth: fix net lock imbalance in drop casse

2022-08-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 ab33519b1b bluetooth: fix net lock i

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6782: bluetooth: fix net lock imbalance in drop casse

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6782: URL: https://github.com/apache/incubator-nuttx/pull/6782 -- 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-unsubs

[incubator-nuttx-apps] branch master updated: Remove executable permission from source file.

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new 30805a700 Remove executable per

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1249: Remove executable permission from source file.

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1249: URL: https://github.com/apache/incubator-nuttx-apps/pull/1249 -- 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-u

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1245: Use thread-safe ntoa functions

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1245: URL: https://github.com/apache/incubator-nuttx-apps/pull/1245 -- 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-u

[incubator-nuttx-apps] branch master updated: Use thread-safe ntoa functions

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new dcd60a2ab Use thread-safe ntoa

[GitHub] [incubator-nuttx] fjpanag opened a new issue, #6785: usleep() blocks when CONFIG_SIM_WALLTIME_SIGNAL is enabled

2022-08-04 Thread GitBox
fjpanag opened a new issue, #6785: URL: https://github.com/apache/incubator-nuttx/issues/6785 In simulator, when `CONFIG_SIM_WALLTIME_SIGNAL` is enabled, any call to `usleep()` blocks indefinitely and never returns. I have confirmed that the "tick" signal is delivered correctly and th

[GitHub] [incubator-nuttx-apps] acassis opened a new issue, #1250: Add to apps: Luos support

2022-08-04 Thread GitBox
acassis opened a new issue, #1250: URL: https://github.com/apache/incubator-nuttx-apps/issues/1250 This is a nice framework that should be integrated on NuttX, they already opened an issue to it: https://github.com/Luos-io/luos_engine/issues/293 -- This is an automated message from the

[incubator-nuttx] branch master updated (0083a2e4f2 -> fa9d352566)

2022-08-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 0083a2e4f2 optimize UNUSED macro add e0080766b3 FS Disable FS_LARGEFILE by default add 93f05c7e7

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6773: FS disable FS_LARGEFILE by default

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6773: URL: https://github.com/apache/incubator-nuttx/pull/6773 -- 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-unsubs

[GitHub] [incubator-nuttx] acassis commented on pull request #6746: sched: Remove KEEP_ALIVE_HACK hack

2022-08-04 Thread GitBox
acassis commented on PR #6746: URL: https://github.com/apache/incubator-nuttx/pull/6746#issuecomment-1205321445 @xiaoxiang781216 is the some testing to guarantee that sporadic scheduling is working? I found a good explanation here: https://www.embedded.com/scheduling-sporadic-events/ --

[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
Donny9 commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937840747 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937821141 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
Donny9 commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937783089 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937763289 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] hartmannathan commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
hartmannathan commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937761661 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937762094 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] hartmannathan commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
hartmannathan commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937759615 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx-apps] hartmannathan opened a new pull request, #1249: Remove executable permission from source file.

2022-08-04 Thread GitBox
hartmannathan opened a new pull request, #1249: URL: https://github.com/apache/incubator-nuttx-apps/pull/1249 ## Summary `examples/scd41/scd41_main.c` incorrectly had executable permission. This PR fixes it. ## Impact Prevents trying to execute a non-executable file.

[GitHub] [incubator-nuttx] hartmannathan opened a new pull request, #6784: Remove executable permission from source and build files.

2022-08-04 Thread GitBox
hartmannathan opened a new pull request, #6784: URL: https://github.com/apache/incubator-nuttx/pull/6784 ## Summary Numerous source and build files (.c, .h, .ld, Make.defs, Makefile, etc.) incorrectly had executable permissions. This PR fixes these incorrect permissions. ## Im

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
pkarashchenko commented on code in PR #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783#discussion_r937739405 ## libs/libc/time/lib_localtime.c: ## @@ -676,23 +668,22 @@ static int tzload(FAR const char *name, for (i = 0; i < sp->timecnt; ++i) {

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
pkarashchenko commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205203626 The main issue is that all the network and not only the network use a single IOB mechanism. That has more drawbacks that advantages IMO, but still it is what we have now. Fo

[GitHub] [incubator-nuttx] Donny9 opened a new pull request, #6783: libc/localtime: fix ats over time_t range

2022-08-04 Thread GitBox
Donny9 opened a new pull request, #6783: URL: https://github.com/apache/incubator-nuttx/pull/6783 ## Summary libc/localtime: fix ats over time_t range Fix time error for "set TZ :Pacific/Honolulu". Signed-off-by: Jiuzhu Dong ## Impact tzset correct ## Testin

[GitHub] [incubator-nuttx] mkeydevelop opened a new pull request, #6782: bluetooth: fix net lock imbalance in drop casse

2022-08-04 Thread GitBox
mkeydevelop opened a new pull request, #6782: URL: https://github.com/apache/incubator-nuttx/pull/6782 ## Summary Net lock is imbalance if network is down or receive a bad HCI packet ## Impact ## Testing -- This is an automated message from the Apache Git Service. To r

[incubator-nuttx-apps] branch master updated: ifconfig support set IPv6 DNS server

2022-08-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-apps.git The following commit(s) were added to refs/heads/master by this push: new c5102fe50 ifconfig support set

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1248: ifconfig support set IPv6 DNS server

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #1248: URL: https://github.com/apache/incubator-nuttx-apps/pull/1248 -- 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-u

[GitHub] [incubator-nuttx] mlyszczek opened a new pull request, #6781: Multiple improvements for stm32wl5 board nucleo-stm32wl5jc

2022-08-04 Thread GitBox
mlyszczek opened a new pull request, #6781: URL: https://github.com/apache/incubator-nuttx/pull/6781 ## Summary This pull requests contains multiple commits with improvements for stm32wl5 chip and nucleo-stm32wl5jc board. In short it adds: - fixes unbuffered IPCC communication -

[GitHub] [incubator-nuttx] PetervdPerk-NXP commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
PetervdPerk-NXP commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205098524 > How do you think if we add a KCONFIG option to turn it on or off? like following: Although this is a improvement, if you bring in your TCP/IP reassemble changes i

[GitHub] [incubator-nuttx] davidtso1219 commented on issue #5530: libxx "C++ Example using CMake" guide "No thread API" error

2022-08-04 Thread GitBox
davidtso1219 commented on issue #5530: URL: https://github.com/apache/incubator-nuttx/issues/5530#issuecomment-1205089543 kind of off-topic. But how do you guys flash `hellocpp.bin` to your board using `openocd`? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-nuttx] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205073094 How do you think if we add a KCONFIG option to turn it on or off? like following: #if defined(CONFIG_IOB_PRIVATE) FAR uintptr_t iob_private; #endif This configuratio

[GitHub] [incubator-nuttx] davids5 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
davids5 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204990858 It might not be obvious that the design patterns used in NuttX were intentionally very memory/resource conscious from the start. These patterns are extremely important to preserve

[GitHub] [incubator-nuttx] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204976649 > Still we should consider that NuttX runs on small microcontrollers with SRAM varying between 8KB up to 1MB. Hence a small increases can still have big impact on smaller platf

[GitHub] [incubator-nuttx] PetervdPerk-NXP commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
PetervdPerk-NXP commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204960766 Still we should consider that NuttX runs on small microcontrollers with SRAM varying between 8KB up to 1MB. Hence a small increases can still have big impact on smaller

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6773: FS disable FS_LARGEFILE by default

2022-08-04 Thread GitBox
pkarashchenko commented on PR #6773: URL: https://github.com/apache/incubator-nuttx/pull/6773#issuecomment-1204955414 Seems that SPIFS also need to select an option -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [incubator-nuttx] luojun1234 closed pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 closed pull request #6780: iob_s add io_private filed, user can use it to keep context URL: https://github.com/apache/incubator-nuttx/pull/6780 -- 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] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204947703 > > However the impact of this would increase memory usage overall for all IOB users. For example an imxrt config that has following config `CONFIG_IOB_NBUFFERS=128` Which give

[incubator-nuttx] branch master updated: optimize UNUSED macro

2022-08-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 0083a2e4f2 optimize UNUSED macro 008

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6779: optimize UNUSED macro

2022-08-04 Thread GitBox
xiaoxiang781216 merged PR #6779: URL: https://github.com/apache/incubator-nuttx/pull/6779 -- 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-unsubs

[GitHub] [incubator-nuttx] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204937054 > However the impact of this would increase memory usage overall for all IOB users. For example an imxrt config that has following config `CONFIG_IOB_NBUFFERS=128` Which gives

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6779: optimize UNUSED macro

2022-08-04 Thread GitBox
pkarashchenko commented on PR #6779: URL: https://github.com/apache/incubator-nuttx/pull/6779#issuecomment-1204923543 If we are sure that there is no stack impact, then I'm fully fine with this change -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-nuttx] PetervdPerk-NXP commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
PetervdPerk-NXP commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204913589 However the impact of this would increase memory usage overall for all IOB users. For example an imxrt config that has following config `CONFIG_IOB_NBUFFERS=128` Whi

[GitHub] [incubator-nuttx] luojun1234 commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
luojun1234 commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204910447 It is optional, some modules may use it to store the pointer of running context. For example, in the IP reassemble function we are developing, multiple fragments are orga

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6779: optimize UNUSED macro

2022-08-04 Thread GitBox
xiaoxiang781216 commented on PR #6779: URL: https://github.com/apache/incubator-nuttx/pull/6779#issuecomment-1204892664 > I'm not sure about this change. For example if the parameter is `int` then would this change force the compiler to use the stack to obtain the parameter address (I mean

[GitHub] [incubator-nuttx] PetervdPerk-NXP commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

2022-08-04 Thread GitBox
PetervdPerk-NXP commented on PR #6780: URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1204889482 What's the particular use case for this? I only see the addition of this field but there's no example. Maybe the particular data can just be appended to the io_dat

  1   2   >