[GitHub] [nuttx-apps] pkarashchenko commented on a diff in pull request #1665: ostest: Introduce basic work queue test

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #1665: URL: https://github.com/apache/nuttx-apps/pull/1665#discussion_r1142952982 ## testing/ostest/wqueue.c: ## @@ -0,0 +1,188 @@ +/ + * apps/testing/ostest/wqueue.

[GitHub] [nuttx] pkarashchenko commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477356024 > > > > Ok. Let's move on with this change if the new PR is ready. I do not want to block the process despite I do not have a good explanation why this can't be done in a single PR.

[GitHub] [nuttx-apps] pkarashchenko commented on a diff in pull request #1672: netutils/ftpc: Cancel wdog on error state in ftpc_reconnect.

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #1672: URL: https://github.com/apache/nuttx-apps/pull/1672#discussion_r1142950589 ## netutils/ftpc/ftpc_connect.c: ## @@ -276,5 +276,6 @@ int ftpc_reconnect(FAR struct ftpc_session_s *session) errout_with_socket: ftpc_sockclose(&session-

[GitHub] [nuttx] anchao opened a new pull request, #8862: syscall/libc: add more syscall/libc symbols into csv file

2023-03-20 Thread via GitHub
anchao opened a new pull request, #8862: URL: https://github.com/apache/nuttx/pull/8862 ## Summary syscall/libc: add more syscall/libc symbols into csv file ## Impact N/A ## Testing ci-check -- This is an automated message from the Apache Git Service. To

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
xiaoxiang781216 commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477353899 > > > Ok. Let's move on with this change if the new PR is ready. I do not want to block the process despite I do not have a good explanation why this can't be done in a single PR.

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8779: URL: https://github.com/apache/nuttx/pull/8779#discussion_r1142941791 ## include/nuttx/serial/uart_16550.h: ## @@ -330,6 +330,9 @@ typedef uint64_t uart_addrwidth_t; * Public Functions Definitions *

[GitHub] [nuttx] pkarashchenko commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477352976 > > Ok. Let's move on with this change if the new PR is ready. I do not want to block the process despite I do not have a good explanation why this can't be done in a single PR. >

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
xiaoxiang781216 commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477350269 > Ok. Let's move on with this change if the new PR is ready. I do not want to block the process despite I do not have a good explanation why this can't be done in a single PR.

[nuttx] branch master updated (f57814999a -> 52525796ab)

2023-03-20 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/nuttx.git from f57814999a fix typo cyles -> cycles add 52525796ab openamp: libmetal compile pass with arm64 arch No new r

[GitHub] [nuttx] pkarashchenko merged pull request #8859: openamp: fix libmetal compile error with arm64 arch

2023-03-20 Thread via GitHub
pkarashchenko merged PR #8859: URL: https://github.com/apache/nuttx/pull/8859 -- 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...@nuttx.

[GitHub] [nuttx] pkarashchenko commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477348524 Ok. Let's move on with this change if the new PR is ready. I do not want to block the process despite I do not have a good explanation why this can't be done in a single PR. -- Thi

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8779: URL: https://github.com/apache/nuttx/pull/8779#discussion_r1141808005 ## arch/arm64/src/fvp-v8r/fvp_lowputc.S: ## @@ -50,24 +50,6 @@ * Public Functions /

[GitHub] [nuttx] anchao opened a new pull request, #8861: libc/settimeofday: correct prototype of settimeofday()

2023-03-20 Thread via GitHub
anchao opened a new pull request, #8861: URL: https://github.com/apache/nuttx/pull/8861 ## Summary libc/settimeofday: correct prototype of settimeofday() https://man.openbsd.org/OpenBSD-5.9/settimeofday.2 ## Impact N/A ## Testing ci-check -- Th

[GitHub] [nuttx] no1wudi opened a new pull request, #8860: Revert "driver/uart_rpmsg: Passthrough data by default"

2023-03-20 Thread via GitHub
no1wudi opened a new pull request, #8860: URL: https://github.com/apache/nuttx/pull/8860 ## Summary This reverts commit 64e611d71eb31c401c28c3b31fa48a4170588abd. ## Impact uart-rpmsg ## Testing sim:rpserver/sim:rpproxy Notice: This patch must works with https://

[GitHub] [nuttx-apps] no1wudi opened a new pull request, #1673: system/cu: Let remote machine handle special character except Ctrl-C

2023-03-20 Thread via GitHub
no1wudi opened a new pull request, #1673: URL: https://github.com/apache/nuttx-apps/pull/1673 ## Summary Let remote machine handle special character except Ctrl-C, make cu's behavior more close to linux/bsd system. ## Impact cu ## Testing sim:rpserver/sim:rpproxy -- Thi

[GitHub] [nuttx-apps] wengzhe opened a new pull request, #1672: netutils/ftpc: Cancel wdog on error state in ftpc_reconnect.

2023-03-20 Thread via GitHub
wengzhe opened a new pull request, #1672: URL: https://github.com/apache/nuttx-apps/pull/1672 ## Summary The timer is for connect timeout, but is not cancelled on connect error, then it may be triggered after ftpc quit and cause heap-use-after-free. ## Impact ftpc won't trigger

[GitHub] [nuttx] wengzhe commented on a diff in pull request #8610: net/netdev: Add upper half of netdev and simplify sim driver

2023-03-20 Thread via GitHub
wengzhe commented on code in PR #8610: URL: https://github.com/apache/nuttx/pull/8610#discussion_r1142859285 ## arch/sim/src/sim/sim_netdriver.c: ## @@ -66,288 +66,161 @@ #include #include #include -#include +#include #include #include "sim_internal.h" -#if CONFIG

[GitHub] [nuttx] CV-Bowen opened a new pull request, #8859: openamp: fix libmetal compile error with arm64 arch

2023-03-20 Thread via GitHub
CV-Bowen opened a new pull request, #8859: URL: https://github.com/apache/nuttx/pull/8859 ## Summary Error log: ```c /home/wangbowen/project/community/nuttx_rpmsg_server/nuttx/include/metal/cpu.h:15:11: fatal error: metal/processor/arm64/cpu.h: No such file or directory 15 |

[GitHub] [nuttx] qinwei2004 commented on pull request #8779: arch: Rename up_[early]serialinit to [arm64|riscv|x86_64][early]serialinit

2023-03-20 Thread via GitHub
qinwei2004 commented on PR #8779: URL: https://github.com/apache/nuttx/pull/8779#issuecomment-1477211242 @pkarashchenko ,My new patch must wait for this patch to be merged. at new patch, I will call assembly code directly -- This is an automated message from the Apache Git Servi

[GitHub] [nuttx-apps] no1wudi commented on pull request #1671: interpreters/wamr: New option to dump call stack

2023-03-20 Thread via GitHub
no1wudi commented on PR #1671: URL: https://github.com/apache/nuttx-apps/pull/1671#issuecomment-1477198328 > what would be the implementation? :-) Emm, the implementation is in WAMR side https://github.com/bytecodealliance/wasm-micro-runtime/pull/2042 and already done, just need a op

[GitHub] [nuttx-apps] cederom commented on pull request #1671: interpreters/wamr: New option to dump call stack

2023-03-20 Thread via GitHub
cederom commented on PR #1671: URL: https://github.com/apache/nuttx-apps/pull/1671#issuecomment-1477196271 what would be the implementation? :-) -- 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 th

[GitHub] [nuttx-apps] no1wudi opened a new pull request, #1671: interpreters/wamr: New option to dump call stack

2023-03-20 Thread via GitHub
no1wudi opened a new pull request, #1671: URL: https://github.com/apache/nuttx-apps/pull/1671 ## Summary New option to dump call stack ## Impact WAMR only ## Testing QEMU -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [nuttx] patacongo opened a new issue, #8858: Issues with adjtime implementation.

2023-03-20 Thread via GitHub
patacongo opened a new issue, #8858: URL: https://github.com/apache/nuttx/issues/8858 adjtime() is implemented in sched/clock/clock_timekeeping. A quick review reveals several problems: 1. Missing system call. The implemented of adjtime() is provided, but there is no call gate impl

[GitHub] [nuttx] realprocrastinator opened a new issue, #8857: use after free in `poll()` implementation

2023-03-20 Thread via GitHub
realprocrastinator opened a new issue, #8857: URL: https://github.com/apache/nuttx/issues/8857 In `fd/vfs/fs_poll.c`, the semaphore object is allocated on the stack meaning that its memory is valid until this function returns. However, there is a `poll_default_cb()` which may also access th

[nuttx-website] branch asf-site updated: Publishing web: b50f5ca4ed22743959c1160aab5dcc9ae18b3eff docs: f57814999a5015e628ad359e2af1515606f4e94b

2023-03-20 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/nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new a4763dfb Publishing web: b50f5ca4

[GitHub] [nuttx] pkarashchenko merged pull request #8855: fix typo cyles -> cycles

2023-03-20 Thread via GitHub
pkarashchenko merged PR #8855: URL: https://github.com/apache/nuttx/pull/8855 -- 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...@nuttx.

[nuttx] branch master updated: fix typo cyles -> cycles

2023-03-20 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/nuttx.git The following commit(s) were added to refs/heads/master by this push: new f57814999a fix typo cyles -> cycles f57814

[nuttx-apps] branch master updated: nxplayer: read file completely until 0 bytes are returned

2023-03-20 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 3a1893ba7 nxplayer: read file completely u

[GitHub] [nuttx-apps] masayuki2009 merged pull request #1652: nxplayer: read file completely until 0 bytes are returned

2023-03-20 Thread via GitHub
masayuki2009 merged PR #1652: URL: https://github.com/apache/nuttx-apps/pull/1652 -- 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

[nuttx] branch master updated: drivers/pty: Echo input by default

2023-03-20 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new fab77cd322 drivers/pty: Echo input by default f

[GitHub] [nuttx] masayuki2009 merged pull request #8846: drivers/pty: Echo input by default

2023-03-20 Thread via GitHub
masayuki2009 merged PR #8846: URL: https://github.com/apache/nuttx/pull/8846 -- 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...@nuttx.a

[GitHub] [nuttx] TimJTi commented on a diff in pull request #8789: Add Broadcom APDS9922 ambient light and proximity sensor

2023-03-20 Thread via GitHub
TimJTi commented on code in PR #8789: URL: https://github.com/apache/nuttx/pull/8789#discussion_r1142676546 ## include/nuttx/sensors/apds9922.h: ## @@ -0,0 +1,315 @@ +/ + * include/nuttx/sensors/apds9922

[GitHub] [nuttx] TimJTi commented on a diff in pull request #8789: Add Broadcom APDS9922 ambient light and proximity sensor

2023-03-20 Thread via GitHub
TimJTi commented on code in PR #8789: URL: https://github.com/apache/nuttx/pull/8789#discussion_r1142675186 ## drivers/sensors/apds9922.c: ## @@ -0,0 +1,2598 @@ +/ + * drivers/sensors/apds9922.c + * + *

[GitHub] [nuttx] TimJTi commented on a diff in pull request #8789: Add Broadcom APDS9922 ambient light and proximity sensor

2023-03-20 Thread via GitHub
TimJTi commented on code in PR #8789: URL: https://github.com/apache/nuttx/pull/8789#discussion_r1142674775 ## include/nuttx/sensors/apds9922.h: ## @@ -0,0 +1,315 @@ +/ + * include/nuttx/sensors/apds9922

[GitHub] [nuttx] TimJTi commented on a diff in pull request #8789: Add Broadcom APDS9922 ambient light and proximity sensor

2023-03-20 Thread via GitHub
TimJTi commented on code in PR #8789: URL: https://github.com/apache/nuttx/pull/8789#discussion_r1142674294 ## .gitignore: ## @@ -59,3 +59,4 @@ uImage .dirlinks .vscode .DS_Store +nuttx Default (copy).launch Review Comment: Nope - sometimes my git commits include this and

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8789: Add Broadcom APDS9922 ambient light and proximity sensor

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8789: URL: https://github.com/apache/nuttx/pull/8789#discussion_r1142670739 ## .gitignore: ## @@ -59,3 +59,4 @@ uImage .dirlinks .vscode .DS_Store +nuttx Default (copy).launch Review Comment: I'm not sure if we should commit this

[nuttx] branch master updated: xtensa/esp32: Update bootloader patch to recent ESP-IDF version

2023-03-20 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/nuttx.git The following commit(s) were added to refs/heads/master by this push: new d4e6d9ab77 xtensa/esp32: Update bootloader

[GitHub] [nuttx] pkarashchenko merged pull request #8838: xtensa/esp32: Update bootloader patch to recent ESP-IDF version

2023-03-20 Thread via GitHub
pkarashchenko merged PR #8838: URL: https://github.com/apache/nuttx/pull/8838 -- 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...@nuttx.

[GitHub] [nuttx] pkarashchenko commented on pull request #8856: armv7-m irq: initialize mask to avoid compiler warning/error

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8856: URL: https://github.com/apache/nuttx/pull/8856#issuecomment-1476868098 Since this code is located in the header file we can go with this fix, but maybe it is worth putting a comment in the code as init is actually not needed. -- This is an automated me

[GitHub] [nuttx] pkarashchenko commented on pull request #8856: armv7-m irq: initialize mask to avoid compiler warning/error

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8856: URL: https://github.com/apache/nuttx/pull/8856#issuecomment-1476864829 In general GCC 12 gives many false positive warnings like this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [nuttx] ldube commented on pull request #8854: boards/sama5d3-xplained: Fix OHCI clock.

2023-03-20 Thread via GitHub
ldube commented on PR #8854: URL: https://github.com/apache/nuttx/pull/8854#issuecomment-1476846532 @acassis, I would like to see the iPAQ running NuttX too. If I find an excuse to do the port I will. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [nuttx] TheCranston commented on issue #8839: Photon P0 WiFi not functional

2023-03-20 Thread via GitHub
TheCranston commented on issue #8839: URL: https://github.com/apache/nuttx/issues/8839#issuecomment-1476799693 Ok. much bisecting later I believe, if I'm doing this correctly, I have the commit in question. ``` fa6ea23101bf9470253c2f60ca422027abdef537 is the first bad commit

[GitHub] [nuttx] MaEtUgR opened a new pull request, #8856: armv7-m irq: initialize mask to avoid compiler warning/error

2023-03-20 Thread via GitHub
MaEtUgR opened a new pull request, #8856: URL: https://github.com/apache/nuttx/pull/8856 ## Summary arm-none-eabi-gcc 12.2.0 gives the following warning: error: 'primask' is used uninitialized We use Werror and the file is indirectly included in different places. I suggest ini

[GitHub] [nuttx] christianrauch opened a new pull request, #8855: fix typo cyles -> cycles

2023-03-20 Thread via GitHub
christianrauch opened a new pull request, #8855: URL: https://github.com/apache/nuttx/pull/8855 ## Summary Fix a typo. ## Impact None. ## Testing None. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [nuttx] TheCranston commented on issue #8839: Photon P0 WiFi not functional

2023-03-20 Thread via GitHub
TheCranston commented on issue #8839: URL: https://github.com/apache/nuttx/issues/8839#issuecomment-1476679159 git bisect log.. not sure if this helps ``` $ git bisect log git bisect start # status: waiting for both good and bad commits # good: [725bdfb0e8c704823669d20931fd

[GitHub] [nuttx] TheCranston commented on issue #8839: Photon P0 WiFi not functional

2023-03-20 Thread via GitHub
TheCranston commented on issue #8839: URL: https://github.com/apache/nuttx/issues/8839#issuecomment-1476668405 Looks like the issue is potentially here (If I did the git bisect thing correctly) ``` 561cbbca96a749e4d2e9ec2ea672f257dc5348aa is the first bad commit commit 561cbbca9

[GitHub] [nuttx] acassis commented on pull request #8854: boards/sama5d3-xplained: Fix OHCI clock.

2023-03-20 Thread via GitHub
acassis commented on PR #8854: URL: https://github.com/apache/nuttx/pull/8854#issuecomment-1476662047 Hi @ldube thank you for finding and fixing this issue! I took a look at your github and discovered your iPaq H3800 eCos port (not EcoSport hehehe). That was inspiring, maybe it could be a g

[GitHub] [nuttx] ldube opened a new pull request, #8854: boards/sama5d3-xplained: Fix OHCI clock.

2023-03-20 Thread via GitHub
ldube opened a new pull request, #8854: URL: https://github.com/apache/nuttx/pull/8854 Choose a divider value that matches the description provided within the same header file. Include stddef.h to fix compiler errors because NULL is not defined. Make logs print protocol, vid and pid cons

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8726: Replace the unsafe function(strcat, sprintf) with the safe one(strlcat, snprintf)

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8726: URL: https://github.com/apache/nuttx/pull/8726#discussion_r1142239721 ## arch/arm/src/phy62xx/uart.h: ## @@ -165,7 +165,12 @@ int hal_uart_set_tx_buf(UART_INDEX_e uart_index, uint8_t *buf, uint16_t size); int hal_uart_get_tx_re

[GitHub] [nuttx] gustavonihei commented on issue #8853: distclean issue with esp32-wrover-kit:lvgl

2023-03-20 Thread via GitHub
gustavonihei commented on issue #8853: URL: https://github.com/apache/nuttx/issues/8853#issuecomment-1476374477 Also reproducible with `sim:lvgl_lcd` and `sim:lvgl_fb`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8610: net/netdev: Add upper half of netdev and simplify sim driver

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8610: URL: https://github.com/apache/nuttx/pull/8610#discussion_r1142223643 ## arch/sim/src/sim/sim_netdriver.c: ## @@ -66,288 +66,161 @@ #include #include #include -#include +#include #include #include "sim_internal.h" -#if

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8783: xtensa/esp32: Tasks use PSRAM as stack can do SPI flash read/write/erase/map/unmap

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8783: URL: https://github.com/apache/nuttx/pull/8783#discussion_r1142216503 ## arch/xtensa/src/esp32/esp32_spiflash.c: ## @@ -460,6 +514,33 @@ static inline void IRAM_ATTR leave_critical_section(state->flags); } +/

[GitHub] [nuttx] pkarashchenko commented on pull request #8836: tools/ci: upgrade to the latest Xtensa ESP32 toolchains

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8836: URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1476339863 @tmedicci I've tested many Xtensa configurations today (definitely not all available). Could you please try this PR? The issues in `esp32-devkitc:audio` and `:cxx` are fixed now --

[GitHub] [nuttx] pkarashchenko opened a new issue, #8853: distclean issue with esp32-wrover-kit:lvgl

2023-03-20 Thread via GitHub
pkarashchenko opened a new issue, #8853: URL: https://github.com/apache/nuttx/issues/8853 I tried to run ``` ./tools/configure.sh esp32-wrover-kit:lvgl make make distclean ``` and got next output with `distclean`: ``` make distclean find: ./lvgl/demos: No such file

[GitHub] [nuttx] slorquet commented on pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
slorquet commented on PR #8852: URL: https://github.com/apache/nuttx/pull/8852#issuecomment-1476320351 Thanks for merging. -- 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. T

[nuttx] branch master updated (947100c5b8 -> 13d823f307)

2023-03-20 Thread btashton
This is an automated email from the ASF dual-hosted git repository. btashton pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 947100c5b8 sim: Fix iic/spi bus open failed add 13d823f307 spi_xx25xx: Repair the spi bus locking mechanism when

[GitHub] [nuttx] btashton merged pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
btashton merged PR #8852: URL: https://github.com/apache/nuttx/pull/8852 -- 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...@nuttx.apach

[GitHub] [nuttx] slorquet commented on pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
slorquet commented on PR #8852: URL: https://github.com/apache/nuttx/pull/8852#issuecomment-1476286307 it does not really matter. this is just frustration to see small fixes included fast. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [nuttx] pkarashchenko commented on pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
pkarashchenko commented on PR #8852: URL: https://github.com/apache/nuttx/pull/8852#issuecomment-1476282232 > Why are test faster today? Last time it took two days. I'm not sure what PR are you referring to. Sometimes CI is stopped due to various failures like HTTP timeout during down

[GitHub] [nuttx] slorquet commented on pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
slorquet commented on PR #8852: URL: https://github.com/apache/nuttx/pull/8852#issuecomment-1476275559 Why are test faster today? Last time it took two days. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[nuttx] branch master updated: sim: Fix iic/spi bus open failed

2023-03-20 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/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 947100c5b8 sim: Fix iic/spi bus open faile

[GitHub] [nuttx] pkarashchenko merged pull request #8850: sim: Fix iic/spi bus open failed

2023-03-20 Thread via GitHub
pkarashchenko merged PR #8850: URL: https://github.com/apache/nuttx/pull/8850 -- 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...@nuttx.

[nuttx-apps] 22/23: apps/netutils/webserver/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 7560a5e80d9ec77b00fa04b9cb78402db88a2b9f Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 16/23: apps/netutils/smtp/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 17103a6b3d89830abdc8629262744e44b7583d56 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 13/23: apps/netutils/pppd/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit af4e1777229e6be72044520f4574b55b99d460f4 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 19/23: apps/netutils/tftpc/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit b2d24f76885873d06ed8e37fea27d85e1aef0643 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 01/23: apps/netutils/codecs/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 6978a65293590f6c64d0213fcfdf9d0a754d1e4d Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 23/23: apps/netutils/xmlrpc/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit d3c3a6ebe08fe38c67083a6c92e97dd8cc102d72 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 18/23: apps/netutils/telnetd/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 98a161820218c038c938593007d249e8b394f284 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 17/23: apps/netutils/telnetc/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 4b401d95b11ee2dfb6d250945572f478e5926ede Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 15/23: apps/netutils/rexecd/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 27a17df69d00117ad4e8e89ae354a7c358e7cf06 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 09/23: apps/netutils/libcurl4nx/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 6529e74326850ec657096a96c75e8f6c1c8439cf Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 11/23: apps/netutils/netlib/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 804762c72d1fc5b27d6ae46f17345fc6fb009bf5 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 07/23: apps/netutils/ftpc/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 8bc93e07cef1e214687387d7a896d86663db7a9a Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 12/23: apps/netutils/ntpclient/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 257719d65aabdf096e6ac7753e8da47c32221137 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 20/23: apps/netutils/thttpd/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit b2f1ab31c7635e908aca366a9fc1fd733de0e17f Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 10/23: apps/netutils/netinit/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit e1357fa6c9876788c9ea10d38924c293643a6c4e Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 21/23: apps/netutils/webclient/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 79b72c030227f6ad7b7b3d5ea3a4f257e8cec852 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 08/23: apps/netutils/ftpd/Make.defs: nxstyle

2023-03-20 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/nuttx-apps.git commit 2c2577fc753675b24a6138d717e333c566a55728 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 06/23: apps/netutils/esp8266/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit a75dd4be405afd2584c06b0f6532745b5f7f57c9 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 02/23: apps/netutils/dhcp6c/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 5736d2adfde113d365329816ded238fda6af54d0 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 04/23: apps/netutils/dhcpd/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 585d0e083b1be405d05d022a1a127083ed3e Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 05/23: apps/netutils/discover/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit cf0ee4354c091fd621d7c3ec00d568b472968622 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] branch master updated (f3c104b99 -> d3c3a6ebe)

2023-03-20 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/nuttx-apps.git from f3c104b99 Fix nxstyle warning new 6978a6529 apps/netutils/codecs/Make.defs: fix nxstyle new 5736d

[nuttx-apps] 14/23: apps/netutils/rexec/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 19d6f4076734adcdf57501240fbb0780f245b2b1 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[nuttx-apps] 03/23: apps/netutils/dhcpc/Make.defs: fix nxstyle

2023-03-20 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/nuttx-apps.git commit 5137e6f3db8510923503aa55646a7eb8a94e1e43 Author: simbit18 <101105604+simbi...@users.noreply.github.com> Author

[GitHub] [nuttx-apps] pkarashchenko merged pull request #1670: apps/netutils/: fix nxstyle

2023-03-20 Thread via GitHub
pkarashchenko merged PR #1670: URL: https://github.com/apache/nuttx-apps/pull/1670 -- 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...@n

[GitHub] [nuttx-apps] gustavonihei commented on a diff in pull request #1649: Add another batch of missing headers throughout the repository

2023-03-20 Thread via GitHub
gustavonihei commented on code in PR #1649: URL: https://github.com/apache/nuttx-apps/pull/1649#discussion_r1142062330 ## industry/foc/float/foc_handler.c: ## @@ -27,6 +27,7 @@ #include #include #include +#include Review Comment: Yes. I don't recall the exact warning

[GitHub] [nuttx-apps] pussuw commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pussuw commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142052695 ## nshlib/nsh_alias.c: ## @@ -0,0 +1,454 @@ +/ + * apps/nshlib/nsh_alias.c + * + * License

[GitHub] [nuttx-apps] pussuw commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pussuw commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142051723 ## nshlib/nsh.h: ## @@ -715,6 +715,25 @@ struct nsh_parser_s #endif }; +#ifdef CONFIG_NSH_ALIAS +struct nsh_alias_s +{ + FAR struct nsh_alias_s *next;/* Sing

[GitHub] [nuttx-apps] pussuw commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pussuw commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142049374 ## nshlib/nsh_parse.c: ## @@ -331,6 +372,57 @@ static void nsh_memlist_free(FAR struct nsh_memlist_s *memlist) } #endif +/***

[GitHub] [nuttx] slorquet commented on a diff in pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
slorquet commented on code in PR #8852: URL: https://github.com/apache/nuttx/pull/8852#discussion_r1142049166 ## drivers/eeprom/spi_xx25xx.c: ## @@ -392,6 +393,7 @@ static void ee25xx_waitwritecomplete(struct ee25xx_dev_s *priv) /* Deselect the FLASH */ SPI_SELE

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8783: xtensa/esp32: Tasks use PSRAM as stack can do SPI flash read/write/erase/map/unmap

2023-03-20 Thread via GitHub
gustavonihei commented on code in PR #8783: URL: https://github.com/apache/nuttx/pull/8783#discussion_r1142046038 ## arch/xtensa/src/esp32/esp32_spiflash.c: ## @@ -1470,6 +1551,219 @@ static int IRAM_ATTR esp32_readdata_encrypted( return OK; } +/***

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8783: xtensa/esp32: Tasks use PSRAM as stack can do SPI flash read/write/erase/map/unmap

2023-03-20 Thread via GitHub
gustavonihei commented on code in PR #8783: URL: https://github.com/apache/nuttx/pull/8783#discussion_r1142045606 ## arch/xtensa/src/esp32/esp32_spiflash.c: ## @@ -1470,6 +1551,219 @@ static int IRAM_ATTR esp32_readdata_encrypted( return OK; } +/***

[GitHub] [nuttx-apps] pkarashchenko commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142016064 ## nshlib/nsh_alias.c: ## @@ -0,0 +1,454 @@ +/ + * apps/nshlib/nsh_alias.c + * + *

[GitHub] [nuttx-apps] pkarashchenko commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142014300 ## nshlib/nsh.h: ## @@ -715,6 +715,25 @@ struct nsh_parser_s #endif }; +#ifdef CONFIG_NSH_ALIAS +struct nsh_alias_s +{ + FAR struct nsh_alias_s *next;

[GitHub] [nuttx-apps] pussuw commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pussuw commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1142010456 ## nshlib/nsh.h: ## @@ -715,6 +715,25 @@ struct nsh_parser_s #endif }; +#ifdef CONFIG_NSH_ALIAS +struct nsh_alias_s +{ + FAR struct nsh_alias_s *next;/* Sing

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #8852: URL: https://github.com/apache/nuttx/pull/8852#discussion_r1142008319 ## drivers/eeprom/spi_xx25xx.c: ## @@ -392,6 +393,7 @@ static void ee25xx_waitwritecomplete(struct ee25xx_dev_s *priv) /* Deselect the FLASH */ SPI

[GitHub] [nuttx-apps] pkarashchenko commented on a diff in pull request #1654: nsh/alias: Add shell alias support for NSH

2023-03-20 Thread via GitHub
pkarashchenko commented on code in PR #1654: URL: https://github.com/apache/nuttx-apps/pull/1654#discussion_r1141999277 ## nshlib/nsh.h: ## @@ -715,6 +715,25 @@ struct nsh_parser_s #endif }; +#ifdef CONFIG_NSH_ALIAS +struct nsh_alias_s +{ + FAR struct nsh_alias_s *next;

[GitHub] [nuttx] slorquet opened a new pull request, #8852: spi_xx25xx: Repair the spi bus locking mechanism when waiting for write completion

2023-03-20 Thread via GitHub
slorquet opened a new pull request, #8852: URL: https://github.com/apache/nuttx/pull/8852 ## Summary The spi_xx25xx EEPROM driver had a bug (probably from its inception in 2014) where the waitwritecomplete() function was expected to be called within an SPI bus exclusion lock, but it was

  1   2   >