[GitHub] [incubator-nuttx] yamt commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
yamt commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573502159 ## File path: arch/xtensa/Kconfig ## @@ -128,15 +128,23 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of the int

[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
cwespressif commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573495198 ## File path: arch/xtensa/Kconfig ## @@ -128,15 +128,23 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of

[incubator-nuttx-apps] branch master updated: ntpclient.c: Avoid integer overflows in offset calculation

2021-02-09 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 ca6907c ntpclient.c: Avoid int

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #583: ntpclient.c: Avoid integer overflows in offset calculation

2021-02-09 Thread GitBox
xiaoxiang781216 merged pull request #583: URL: https://github.com/apache/incubator-nuttx-apps/pull/583 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] yamt commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
yamt commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573480357 ## File path: arch/xtensa/Kconfig ## @@ -128,15 +128,23 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of the int

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on issue #2834: Dynamically extend fd table as needed

2021-02-09 Thread GitBox
xiaoxiang781216 edited a comment on issue #2834: URL: https://github.com/apache/incubator-nuttx/issues/2834#issuecomment-776470675 Yes, we plan to address this issue first: https://github.com/apache/incubator-nuttx/issues/2833, the rough idea is that: 1. Keep the kernel psock API as

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on issue #2834: Dynamically extend fd table as needed

2021-02-09 Thread GitBox
xiaoxiang781216 edited a comment on issue #2834: URL: https://github.com/apache/incubator-nuttx/issues/2834#issuecomment-776470675 Yes, we plan to address this issue first: https://github.com/apache/incubator-nuttx/issues/2833, the rough idea is that: 1. Keep the kernel psock API as

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #2834: Dynamically extend fd table as needed

2021-02-09 Thread GitBox
xiaoxiang781216 commented on issue #2834: URL: https://github.com/apache/incubator-nuttx/issues/2834#issuecomment-776470675 We plan to address this issue first: https://github.com/apache/incubator-nuttx/issues/2833, the rough idea is that: 1. Keep the kernel psock API as before 2

[GitHub] [incubator-nuttx] masayuki2009 commented on issue #2823: Possible memory problems with Ethernet and CDC-ACM

2021-02-09 Thread GitBox
masayuki2009 commented on issue #2823: URL: https://github.com/apache/incubator-nuttx/issues/2823#issuecomment-776442314 > Hi @masayuki2009, I think imrt1060-evk:netnsh doesn´t use CDC-ACM settings, it communicates via serial port (there is a converter on board as I recall, so from NuttX

[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
cwespressif commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573421825 ## File path: arch/xtensa/Kconfig ## @@ -128,9 +128,17 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of t

[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
cwespressif commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573421300 ## File path: arch/xtensa/Kconfig ## @@ -128,9 +128,17 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of t

[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
cwespressif commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573420702 ## File path: arch/xtensa/src/esp32/esp32_allocateheap.c ## @@ -130,6 +132,10 @@ void up_allocate_heap(FAR void **heap_start, size_t *heap_si

[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
cwespressif commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r573420348 ## File path: arch/xtensa/src/esp32/esp32_imm.c ## @@ -73,6 +73,46 @@ FAR void *xtensa_imm_malloc(size_t size) return mm_malloc(&g_iheap, s

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r573374355 ## File path: include/nuttx/spinlock.h ## @@ -30,7 +30,11 @@ #include #include -#ifdef CONFIG_SPINLOCK +#ifndef CONFIG_SPINLOCK Review

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

2021-02-09 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 3343162 Publishing web

[GitHub] [incubator-nuttx-apps] yamt opened a new pull request #583: ntpclient.c: Avoid integer overflows in offset calculation

2021-02-09 Thread GitBox
yamt opened a new pull request #583: URL: https://github.com/apache/incubator-nuttx-apps/pull/583 ## Summary ntpclient.c: Avoid integer overflows in offset calculation The current calculation easily overflows if the local time is around the unix epoch. I guess it isn't too

[GitHub] [incubator-nuttx] patacongo commented on issue #2834: Dynamically extend fd table as needed

2021-02-09 Thread GitBox
patacongo commented on issue #2834: URL: https://github.com/apache/incubator-nuttx/issues/2834#issuecomment-776312386 From top-level TODO list (under " UNIFIED DESCRIPTOR REPRESENTATION"): Another example of how the current implementation limits funct

[GitHub] [incubator-nuttx] Ouss4 merged pull request #2835: esp32: Make esptool output non-silent on POSTBUILD

2021-02-09 Thread GitBox
Ouss4 merged pull request #2835: URL: https://github.com/apache/incubator-nuttx/pull/2835 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: esp32: Make esptool output non-silent on POSTBUILD

2021-02-09 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new ea3eccb esp32: Make esptool outpu

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
Ouss4 commented on pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#issuecomment-776310417 @gustavonihei @acassis PTAL again... I almost slept and forgot this one ;) This is an automated message f

[GitHub] [incubator-nuttx] gauravsharma-bc commented on issue #1494: NXP imxrt1064 dose not support

2021-02-09 Thread GitBox
gauravsharma-bc commented on issue #1494: URL: https://github.com/apache/incubator-nuttx/issues/1494#issuecomment-776308407 Hi @degree900 , I'm on the same path as you- what did you change in the LUT in the 'imxrt_flexspi_nor_flash.c'? I don't really see anything address or peripheral spe

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
masayuki2009 commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r573311282 ## File path: include/nuttx/spinlock.h ## @@ -30,7 +30,11 @@ #include #include -#ifdef CONFIG_SPINLOCK +#ifndef CONFIG_SPINLOCK Review

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r573184352 ## File path: include/nuttx/spinlock.h ## @@ -30,7 +30,11 @@ #include #include -#ifdef CONFIG_SPINLOCK +#ifndef CONFIG_SPINLOCK Review

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r573184352 ## File path: include/nuttx/spinlock.h ## @@ -30,7 +30,11 @@ #include #include -#ifdef CONFIG_SPINLOCK +#ifndef CONFIG_SPINLOCK Review

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r573184352 ## File path: include/nuttx/spinlock.h ## @@ -30,7 +30,11 @@ #include #include -#ifdef CONFIG_SPINLOCK +#ifndef CONFIG_SPINLOCK Review

[incubator-nuttx] 02/02: arch: cxd56xx: Introduce driver-specific spinlock in cxd56_serial.c

2021-02-09 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 c024b414f8fce84b3e69d1e28515817c3743732f Author: Masayuki Ishikawa AuthorDate: Tue Feb 9 13:15:32 2021 +0900

[incubator-nuttx] branch master updated (dcec04f -> c024b41)

2021-02-09 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 dcec04f xtensa/esp32: Writeback PSRAM data when mapping SPI Flash address to ESP32's address bus n

[incubator-nuttx] 01/02: include: nuttx: Introduce spinlock_t for #ifndef CONFIG_SPINLOCK

2021-02-09 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 611dbb6d22ae33be3075623ed18da57729f7d108 Author: Masayuki Ishikawa AuthorDate: Tue Feb 9 13:07:03 2021 +0900

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
xiaoxiang781216 merged pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830 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] gustavonihei opened a new pull request #2835: esp32: Make esptool output non-silent on POSTBUILD

2021-02-09 Thread GitBox
gustavonihei opened a new pull request #2835: URL: https://github.com/apache/incubator-nuttx/pull/2835 ## Summary Just a small refactor on ESP32-specific POSTBUILD action to make the output of esptool.py visible. ## Impact No impact, just some extra messages on the build proces

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
Ouss4 commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573082460 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # archive.

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
acassis commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573067580 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # archive.

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
Ouss4 commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573059282 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # archive.

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
Ouss4 commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573059282 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # archive.

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573048572 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # arc

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2826: tools/esp32/Config.mk: Make the blob directory generic and not dependant on any board.

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2826: URL: https://github.com/apache/incubator-nuttx/pull/2826#discussion_r573048572 ## File path: tools/esp32/Config.mk ## @@ -22,9 +22,9 @@ # and assemble source files and to insert the resulting object files into an # arc

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
gustavonihei commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r572954028 ## File path: arch/xtensa/Kconfig ## @@ -128,9 +128,17 @@ config XTENSA_USE_SEPARATE_IMEM This separate heap will be part of

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
acassis commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r572948095 ## File path: arch/xtensa/src/esp32/esp32_allocateheap.c ## @@ -130,6 +132,10 @@ void up_allocate_heap(FAR void **heap_start, size_t *heap_size)

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue #2834: Dynamically extend fd table as needed

2021-02-09 Thread GitBox
xiaoxiang781216 opened a new issue #2834: URL: https://github.com/apache/incubator-nuttx/issues/2834 Now, all tasks can have hold at most CONFIG_NFILE_DESCRIPTORS file handles. But the complexity for each task is totally different, so it's better to extend the file table size at runtime to

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #2832: xtensa/esp32: Fix the issue of WiFi internal malloc from PSRAM

2021-02-09 Thread GitBox
acassis commented on a change in pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832#discussion_r572880126 ## File path: arch/xtensa/src/esp32/esp32_imm.c ## @@ -73,6 +73,46 @@ FAR void *xtensa_imm_malloc(size_t size) return mm_malloc(&g_iheap, size)

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
masayuki2009 commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r572830251 ## File path: include/nuttx/spinlock.h ## @@ -30,6 +30,12 @@ #include #include +#ifndef CONFIG_SPINLOCK +typedef struct +{ +} spinlock_

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue #2833: Refine the interaction between fs and net

2021-02-09 Thread GitBox
xiaoxiang781216 opened a new issue #2833: URL: https://github.com/apache/incubator-nuttx/issues/2833 fs layer has many places to check socket handler explicitly: ``` int nx_close(int fd) { /* Did we get a valid file descriptor? */ if (fd >= CONFIG_NFILE_DESCRIPTORS)

[GitHub] [incubator-nuttx] acassis merged pull request #2831: xtensa/esp32: Writeback PSRAM data when mapping SPI Flash address to ESP32's address bus

2021-02-09 Thread GitBox
acassis merged pull request #2831: URL: https://github.com/apache/incubator-nuttx/pull/2831 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: xtensa/esp32: Writeback PSRAM data when mapping SPI Flash address to ESP32's address bus

2021-02-09 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 dcec04f xtensa/esp32: Writeback PSRAM

[GitHub] [incubator-nuttx] michallenc commented on issue #2823: Possible memory problems with Ethernet and CDC-ACM

2021-02-09 Thread GitBox
michallenc commented on issue #2823: URL: https://github.com/apache/incubator-nuttx/issues/2823#issuecomment-775817183 Hi @masayuki2009, I think imrt1060-evk:netnsh doesn´t use CDC-ACM settings, it communicates via serial port (there is a converter on board as I recall, so from NuttX side

[GitHub] [incubator-nuttx] cwespressif opened a new pull request #2832: xtensa/esp32: Fix the issue of wifi internal malloc from psram

2021-02-09 Thread GitBox
cwespressif opened a new pull request #2832: URL: https://github.com/apache/incubator-nuttx/pull/2832 ## Summary `SPIRAM` is not DMA-capable, while WiFi `internal malloc` from heap must be DMA-capable, We must avoid `internal malloc` from the heap of `PSRAM` ## Impact I

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2830: Driver-specific spinlock for cxd56_serial.c

2021-02-09 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2830: URL: https://github.com/apache/incubator-nuttx/pull/2830#discussion_r572726009 ## File path: include/nuttx/spinlock.h ## @@ -30,6 +30,12 @@ #include #include +#ifndef CONFIG_SPINLOCK +typedef struct +{ +} spinlo

[incubator-nuttx] 05/05: sched: nxstyle fixes

2021-02-09 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 dee641828f49eb021ccf59a6d29c09021025e7e5 Author: Alin Jerpelea AuthorDate: Mon Feb 8 18:46:31 2021 +0100

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2825: sched: Author Gregory Nutt: update licenses to Apache

2021-02-09 Thread GitBox
xiaoxiang781216 merged pull request #2825: URL: https://github.com/apache/incubator-nuttx/pull/2825 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] 02/05: mm: nxstyle fixes

2021-02-09 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 acb8ad4c7b1e92f35f24727de14bddeb7a21d1ae Author: Alin Jerpelea AuthorDate: Mon Feb 8 16:19:12 2021 +0100

[incubator-nuttx] 03/05: passl: Author Gregory Nutt: update licenses to Apache

2021-02-09 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 6797d66a5090b2543ac90222743fe64107cb6d2a Author: Alin Jerpelea AuthorDate: Mon Feb 8 16:29:31 2021 +0100

[incubator-nuttx] branch master updated (2220827 -> dee6418)

2021-02-09 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 2220827 esp32_allocateheap.c: Add a sanity check new bcee9c3 mm: Author Gregory Nutt: update licen

[GitHub] [incubator-nuttx] cwespressif opened a new pull request #2831: xtensa/esp32: Writeback PSRAM data when mapping SPI Flash address to ESP32's address bus

2021-02-09 Thread GitBox
cwespressif opened a new pull request #2831: URL: https://github.com/apache/incubator-nuttx/pull/2831 ## Summary Fix the issue of `PSRAM` heap being destroyed when operating `SPI flash` APIs. ## Impact Enable `SPI flash` encryption in `Partition`, `PSRAM` heap will be