Re: [PR] libs/libc/modlib/modlib_bind.c: Fix system crash if modlib_bind fails [nuttx]

2025-02-12 Thread via GitHub
xiaoxiang781216 commented on code in PR #15830: URL: https://github.com/apache/nuttx/pull/15830#discussion_r1953995022 ## libs/libc/modlib/modlib_bind.c: ## @@ -1076,6 +1076,8 @@ int modlib_bind(FAR struct module_s *modp, } #endif + errout_with_addrenv: Review Comment:

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
michallenc commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655780963 @cederom @anchao @lupyuen @acassis Hey guys, I am not quite sure what's the problem here. The discussion was resolved, Alan just forgot to mark it, but he approved the PR after the cha

[PR] libs/libc/modlib/modlib_bind.c: Fix system crash if modlib_bind fails [nuttx]

2025-02-12 Thread via GitHub
jlaitine opened a new pull request, #15830: URL: https://github.com/apache/nuttx/pull/15830 ## Summary Addrenv is changed to the newly created process' one in the beginning of modlib_bind, and needs to be changed always when returning from the function; also in error cases.

Re: [PR] Add support for the RN903 and RN2483 LoRa modules [nuttx]

2025-02-12 Thread via GitHub
lupyuen commented on PR #15828: URL: https://github.com/apache/nuttx/pull/15828#issuecomment-2655660749 Thanks @linguini1! I suggest we rename the PR Title to "drivers/wireless: Add support for..."? So it looks nicer in the NuttX Release Notes :-) https://nuttx.apache.org/releases/12.8.0/

Re: [PR] sched/spin_lock: rename raw_*lock to *_notrace [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15729: URL: https://github.com/apache/nuttx/pull/15729#issuecomment-2655651378 @hujun260 please rebase this PR -- 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

Re: [PR] Revert some changes to restore the original semantics of spin_lock_irqsave. [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15767: URL: https://github.com/apache/nuttx/pull/15767#issuecomment-2655636838 @hujun260 please resolve the conflict with master and revert https://github.com/apache/nuttx/pull/15587 together -- This is an automated message from the Apache Git Service. To respond t

(nuttx) branch master updated (99c7b64072 -> 05ec313bc4)

2025-02-12 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 99c7b64072 mtd/w25q: ensure the correct behavior if erase sector fails add 45422e2ad9 Revert "No need to call sche

Re: [PR] Revert changes related to "enter_critical_section -> spin_lock_irqsave" [nuttx]

2025-02-12 Thread via GitHub
anchao merged PR #15766: URL: https://github.com/apache/nuttx/pull/15766 -- 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

Re: [PR] Add support for the RN903 and RN2483 LoRa modules [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15828: URL: https://github.com/apache/nuttx/pull/15828#issuecomment-2655533231 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR appears to meet the NuttX requirements, al

[PR] xtensa/esp32: set cpuint to initial value after deallocate [nuttx]

2025-02-12 Thread via GitHub
royfengsss opened a new pull request, #15829: URL: https://github.com/apache/nuttx/pull/15829 *Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Please to https://github.com/apache/nuttx/pull/15433 cpui

[PR] Add support for the RN903 and RN2483 LoRa modules [nuttx]

2025-02-12 Thread via GitHub
linguini1 opened a new pull request, #15828: URL: https://github.com/apache/nuttx/pull/15828 ## Summary Add support for the RN903 and RN2483 family of LoRa radio transceivers. This initial support includes transmit and receive functionality and configuration and reading of radio para

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655458126 > @anchao so after reading the code you saw that not only return fix is there but also introduction of new blocking behavior with arbitrary delay with no polling for success and still

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
cederom commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655376910 > @cederom: @anchao did you see there is ongoing discussion in progress before you merged? > @anchao: I saw @acassis approved this PR, this is a valid response Okay, I will mark

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655337253 IMHO that maintainers should spend more time on reading and learning code and software architecture instead of commenting, which will make it more efficient. Nuttx is already a very big pr

Re: [PR] Audio: modify audio driver satisfy nuttx alsa [nuttx]

2025-02-12 Thread via GitHub
yangyalei commented on PR #15657: URL: https://github.com/apache/nuttx/pull/15657#issuecomment-2655310193 > ping @yangyalei done -- 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 specifi

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655318513 > @anchao did you see there is ongoing discussion in progress before you merged? I saw @acassis approved this PR, this is a valid response -- This is an automated message from th

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
lupyuen commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655303765 Sorry I approved the PR too quickly. Next time I'll mark it as "Change Requested" until all pending concerns are resolved. -- This is an automated message from the Apache Git Service. T

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
CV-Bowen commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2655300722 [Documentation/platforms/arm/qemu/boards/qemu-armv7a/index.rst](https://github.com/apache/nuttx/pull/15818/files#diff-49614bc0f05338725fe51be6689bc18419efb45e5bbf6a272893f6f223845b58)

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
cederom commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2655269041 @anchao did you see there is ongoing discussion in progress before you merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

(nuttx) branch master updated (b57f3bb840 -> 99c7b64072)

2025-02-12 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from b57f3bb840 Fix macOS build error. add 99c7b64072 mtd/w25q: ensure the correct behavior if erase sector fails No n

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
anchao merged PR #15812: URL: https://github.com/apache/nuttx/pull/15812 -- 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

Re: [PR] syslog/ramlog: add support of ramlog flush worker [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #13599: URL: https://github.com/apache/nuttx/pull/13599#issuecomment-2655260828 > @anchao what to do with this PR? Did you address Xiang Xiao comments? please hold this PR, I will enhance this capability in future. -- This is an automated message from the Apac

Re: [PR] qemu-armv8a/ivshmem: add ivshmem chapter [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15827: URL: https://github.com/apache/nuttx/pull/15827#issuecomment-2655252149 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR mostly meets the NuttX requirements, but is lac

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2655251994 > @CV-Bowen I cannot find it: > > https://nuttx.apache.org/docs/latest/search.html?q=rpserver_ivshmem&check_keywords=yes&area=default > > https://nuttx.apache.org/docs/latest

[PR] qemu-armv8a/ivshmem: add ivshmem chapter [nuttx]

2025-02-12 Thread via GitHub
anchao opened a new pull request, #15827: URL: https://github.com/apache/nuttx/pull/15827 ## Summary qemu-armv8a/ivshmem: add ivshmem chapter An example implementation for OpenAMP based on the Inter-VM share memory(ivshmem):: rpproxy_ivshmem: Remote slave(client) proxy

Re: [PR] Initial STM32H5 ICACHE Commit [nuttx]

2025-02-12 Thread via GitHub
cederom commented on PR #15718: URL: https://github.com/apache/nuttx/pull/15718#issuecomment-2655179775 Thank you @kywwilson11, amazing work! :-) * Please use signed commit (`git commit -s`) and we are ready to go :-) * This is initial commit, are there other things to do? * Do you

Re: [PR] mpfs/mpfs_irq.c: Enable interrupts on all harts [nuttx]

2025-02-12 Thread via GitHub
cederom commented on PR #15820: URL: https://github.com/apache/nuttx/pull/15820#issuecomment-2655084050 Thank you @pussuw, some questions below :-) * Is it safe to enable this irq on all harts? Can you provide a reference to specification or advise that this is common practice? * Would

[I] [BUG] Floating point issues on STM32H743 [nuttx]

2025-02-12 Thread via GitHub
linguini1 opened a new issue, #15826: URL: https://github.com/apache/nuttx/issues/15826 ### Description / Steps to reproduce the issue I am encountering numerous strange issues on the STM32H743. Floating point numbers seem to not work for the MS56xx driver, used on an MS5607.

Re: [PR] xtensa/esp32s3: allow moving .bss data to the external PSRAM [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15825: URL: https://github.com/apache/nuttx/pull/15825#issuecomment-2654781305 @tmedicci please include a Documentation/ with the information you added here, this way users will know how to use it in their applications -- This is an automated message from the Apac

Re: [PR] xtensa/esp32s3: allow moving .bss data to the external PSRAM [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15825: URL: https://github.com/apache/nuttx/pull/15825#issuecomment-2654769480 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR description mostly meets the NuttX requirements

Re: [PR] xtensa: use misc register to implement up_this_task macro [nuttx]

2025-02-12 Thread via GitHub
tmedicci commented on PR #15821: URL: https://github.com/apache/nuttx/pull/15821#issuecomment-2654765610 I will let it testing on our internal CI. Please don't merge it yet. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[PR] xtensa/esp32s3: allow moving .bss data to the external PSRAM [nuttx]

2025-02-12 Thread via GitHub
tmedicci opened a new pull request, #15825: URL: https://github.com/apache/nuttx/pull/15825 ## Summary * xtensa/esp32s3: allow moving .bss data to the external PSRAM This commit allows placing .bss data into the external PSRAM. Previously, the PSRAM was fully allocated

Re: [PR] Audio: modify audio driver satisfy nuttx alsa [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15657: URL: https://github.com/apache/nuttx/pull/15657#issuecomment-2654516871 ping @yangyalei -- 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

Re: [PR] interpreters/python: fix patch to set `_PyRuntime` attribute [nuttx-apps]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #2992: URL: https://github.com/apache/nuttx-apps/pull/2992#issuecomment-2654708161 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR mostly meets the NuttX requirements, bu

[PR] interpreters/python: fix patch to set `_PyRuntime` attribute [nuttx-apps]

2025-02-12 Thread via GitHub
tmedicci opened a new pull request, #2992: URL: https://github.com/apache/nuttx-apps/pull/2992 ## Summary * interpreters/python: fix patch to set `_PyRuntime` attribute This commit also adds the check for the `__NuttX__` macro to the patch file that allows setting an attribute

Re: [PR] risc-v/backtrace: Support backtrace dump during IRQ [nuttx]

2025-02-12 Thread via GitHub
tmedicci commented on code in PR #15815: URL: https://github.com/apache/nuttx/pull/15815#discussion_r1953265966 ## arch/risc-v/src/common/riscv_doirq.c: ## @@ -88,13 +88,24 @@ uintreg_t *riscv_doirq(int irq, uintreg_t *regs) (*running_task)->xcp.regs = regs; } - /

Re: [PR] mm: Add CONFIG_MM_NODE_PENDING configuration [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15484: URL: https://github.com/apache/nuttx/pull/15484#issuecomment-2654523839 ping @W-M-R -- 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 unsu

Re: [PR] risc-v/backtrace: Support backtrace dump during IRQ [nuttx]

2025-02-12 Thread via GitHub
xiaoxiang781216 commented on code in PR #15815: URL: https://github.com/apache/nuttx/pull/15815#discussion_r1953242553 ## arch/risc-v/src/common/riscv_exception_common.S: ## @@ -207,25 +207,46 @@ handle_irq: .cfi_offset x2, 8 /* Toolchain not support macro, is R

Re: [PR] syslog/ramlog: add support of ramlog flush worker [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #13599: URL: https://github.com/apache/nuttx/pull/13599#issuecomment-2654568186 @anchao what to do with this PR? Did you address Xiang Xiao comments? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] tools: Add cmux command line tool [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #14823: URL: https://github.com/apache/nuttx/pull/14823#issuecomment-2654560156 @Gary-Hobson any news about the CMUX driver? -- 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 t

Re: [PR] [PATCH] stm32 timers: add set/get prescaler operations and get period operation to ll_ops. [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15482: URL: https://github.com/apache/nuttx/pull/15482#issuecomment-2654551960 @joaoVLAndrade please include a better commit log message, it should include the motivation to add this new features. I think set/get prescaler could be useful in some application. --

Re: [PR] drivers/rptun: add rptun trace feature for debug [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15300: URL: https://github.com/apache/nuttx/pull/15300#issuecomment-2654553970 ping @luckyyaojin -- 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

Re: [I] [BUG] Mirror of NuttX: All macOS Jobs are broken [nuttx]

2025-02-12 Thread via GitHub
acassis closed issue #15491: [BUG] Mirror of NuttX: All macOS Jobs are broken URL: https://github.com/apache/nuttx/issues/15491 -- 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.

(nuttx) branch master updated (df65980273 -> b57f3bb840)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from df65980273 docs: enhance Rust guide with CPU optimization instructions add b57f3bb840 Fix macOS build error. No

Re: [PR] Fix macOS build error. [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15774: URL: https://github.com/apache/nuttx/pull/15774 -- 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.apac

Re: [PR] docs: enhance Rust guide [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15754: URL: https://github.com/apache/nuttx/pull/15754 -- 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.apac

Re: [I] [BUG] Mirror of NuttX: All macOS Jobs are broken [nuttx]

2025-02-12 Thread via GitHub
acassis closed issue #15491: [BUG] Mirror of NuttX: All macOS Jobs are broken URL: https://github.com/apache/nuttx/issues/15491 -- 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.

(nuttx) branch master updated (3dcedd507b -> df65980273)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 3dcedd507b h7: eth: add checks for PHYID in stm32_phyinit add 081588c1c5 docs: update rust guide to include X86 a

(nuttx) branch master updated (e5388558a6 -> 3dcedd507b)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from e5388558a6 qemu/ivshmem: switch proxy/server roles add 3dcedd507b h7: eth: add checks for PHYID in stm32_phyinit

Re: [PR] h7: eth: add checks for PHYID in stm32_phyinit [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15757: URL: https://github.com/apache/nuttx/pull/15757 -- 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.apac

Re: [PR] LSM6DSO32 driver [nuttx]

2025-02-12 Thread via GitHub
linguini1 closed pull request #15823: LSM6DSO32 driver URL: https://github.com/apache/nuttx/pull/15823 -- 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: c

[PR] LSM6DSO32 driver [nuttx]

2025-02-12 Thread via GitHub
linguini1 opened a new pull request, #15823: URL: https://github.com/apache/nuttx/pull/15823 ## Summary Resolves issue where invalid address 0x6c was included instead of 0x6a. ## Impact Boards using 0x6a as the address will no longer receive a DEBUGASSERT failure when re

[PR] sensors/lsm6dso32: Fixed typo in device address registration. [nuttx]

2025-02-12 Thread via GitHub
linguini1 opened a new pull request, #15824: URL: https://github.com/apache/nuttx/pull/15824 ## Summary Resolves issue where invalid address 0x6c was included instead of 0x6a. ## Impact Boards using 0x6a as the address will no longer receive a DEBUGASSERT failure when re

Re: [PR] risc-v/backtrace: Support backtrace dump during IRQ [nuttx]

2025-02-12 Thread via GitHub
fdcavalcanti commented on code in PR #15815: URL: https://github.com/apache/nuttx/pull/15815#discussion_r1953025438 ## arch/risc-v/src/common/riscv_doirq.c: ## @@ -88,13 +88,24 @@ uintreg_t *riscv_doirq(int irq, uintreg_t *regs) (*running_task)->xcp.regs = regs; }

Re: [PR] risc-v/backtrace: Support backtrace dump during IRQ [nuttx]

2025-02-12 Thread via GitHub
tmedicci commented on code in PR #15815: URL: https://github.com/apache/nuttx/pull/15815#discussion_r1953004237 ## arch/risc-v/src/common/riscv_doirq.c: ## @@ -88,13 +88,24 @@ uintreg_t *riscv_doirq(int irq, uintreg_t *regs) (*running_task)->xcp.regs = regs; } - /

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2654028876 @CV-Bowen I cannot find it: https://nuttx.apache.org/docs/latest/search.html?q=rpserver_ivshmem&check_keywords=yes&area=default https://nuttx.apache.org/docs/latest/search.h

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
CV-Bowen commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2653997613 @anchao Done @jerpelea I found this warning when I enable RPMSGBLK_SERVER, so I think place this patch in this PR is reasonable. @acassis There is alreay a document about the rpserv

Re: [PR] Add support to Serial Error Reporting ioctl [nuttx]

2025-02-12 Thread via GitHub
raiden00pl commented on code in PR #15780: URL: https://github.com/apache/nuttx/pull/15780#discussion_r1952780235 ## arch/arm/src/stm32/stm32_serial.c: ## @@ -465,6 +465,9 @@ struct up_dev_s #ifdef CONFIG_SERIAL_OFLOWCONTROL const uint32_tcts_gpio; /* U[S]ART CTS GPIO p

Re: [PR] STM32L4: Fix all boards to support proper stm32_bringup.c [nuttx]

2025-02-12 Thread via GitHub
cederom commented on PR #15817: URL: https://github.com/apache/nuttx/pull/15817#issuecomment-2653909471 Thank you @acassis !! I played a bit some time ago with STM32 boards and noticed the inconsistencies, thank you for the cleanup!! I have some of these boards, will test on hardware and re

(nuttx) branch master updated (729f8a75f7 -> e5388558a6)

2025-02-12 Thread raiden00
This is an automated email from the ASF dual-hosted git repository. raiden00 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 729f8a75f7 drivers/sensors: Added uORB driver for LSM6DSO32 IMU. add e5388558a6 qemu/ivshmem: switch proxy/serve

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
raiden00pl merged PR #15810: URL: https://github.com/apache/nuttx/pull/15810 -- 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

Re: [PR] DOC: Fix documentation to include checkpatch.sh [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15794: URL: https://github.com/apache/nuttx/pull/15794#issuecomment-2653884945 @LuchianMihai I fixed the previous paragraph to make it clear -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] [BUG] [nuttx]

2025-02-12 Thread via GitHub
fdcavalcanti commented on issue #15807: URL: https://github.com/apache/nuttx/issues/15807#issuecomment-2653851828 Run `git clean -xfdn` on your apps folder after you `make distclean` on `nuttx`. See if there are pending files there. -- This is an automated message from the Apache Git S

Re: [PR] xtensa/esp32s3: use common Espressif wireless source [nuttx]

2025-02-12 Thread via GitHub
fdcavalcanti commented on PR #15816: URL: https://github.com/apache/nuttx/pull/15816#issuecomment-2653801449 Fixed CI build error. -- 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 com

Re: [PR] mtd/w25q: ensure the correct behavior if erase sector fails [nuttx]

2025-02-12 Thread via GitHub
michallenc commented on PR #15812: URL: https://github.com/apache/nuttx/pull/15812#issuecomment-2653754558 @acassis any more concerns? -- 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

(nuttx) branch master updated (597b3e6ca7 -> 729f8a75f7)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 597b3e6ca7 mpfs_i2c: Increase TTOA margin to 1ms add 729f8a75f7 drivers/sensors: Added uORB driver for LSM6DSO32

Re: [PR] drivers/sensors: Added uORB driver for LSM6DSO32 IMU. [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15789: URL: https://github.com/apache/nuttx/pull/15789 -- 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.apac

(nuttx) branch master updated (e657b35720 -> 597b3e6ca7)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from e657b35720 nuttx/can: Modify poll logic to bind can_reader_s and pollfd. add 255d8c57c0 mpfs_i2c: Clean up the l

Re: [PR] mpfs_i2c: Tweaks and fixes for the i2c driver [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15819: URL: https://github.com/apache/nuttx/pull/15819 -- 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.apac

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
acassis commented on PR #15810: URL: https://github.com/apache/nuttx/pull/15810#issuecomment-2653728329 > > @anchao, I tried both qemu-armv7a and qemu-armv8a on Ubuntu 22.04 with QEMU v6.2. rpsock stream block test sessions all complete. only qemu-armv8a server side doesn't return back to n

Re: [PR] nshlib/builtin: check background task before restore the signal [nuttx-apps]

2025-02-12 Thread via GitHub
acassis merged PR #2990: URL: https://github.com/apache/nuttx-apps/pull/2990 -- 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

(nuttx-apps) branch master updated: nshlib/builtin: check background task before restore the signal

2025-02-12 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/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 7f424c3e8 nshlib/builtin: check background

(nuttx) branch master updated (31270a8479 -> e657b35720)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 31270a8479 net: fix ipv6_input for 6lowpan add e657b35720 nuttx/can: Modify poll logic to bind can_reader_s and

Re: [PR] nuttx/can: Modify poll logic to bind can_reader_s and pollfd. [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15809: URL: https://github.com/apache/nuttx/pull/15809 -- 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.apac

Re: [PR] net: fix ipv6_input for 6lowpan [nuttx]

2025-02-12 Thread via GitHub
acassis merged PR #15822: URL: https://github.com/apache/nuttx/pull/15822 -- 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.apac

(nuttx) branch master updated (381d3fe64f -> 31270a8479)

2025-02-12 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 381d3fe64f arch/arm/max326xx: add max32690 icc updates add 31270a8479 net: fix ipv6_input for 6lowpan No new rev

Re: [PR] net: fix ipv6_input for 6lowpan [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15822: URL: https://github.com/apache/nuttx/pull/15822#issuecomment-2653305035 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR appears to meet the basic NuttX requiremen

[PR] net: fix ipv6_input for 6lowpan [nuttx]

2025-02-12 Thread via GitHub
Laczen opened a new pull request, #15822: URL: https://github.com/apache/nuttx/pull/15822 ## Summary 6lowpan response for tcp and icmp was missing because of wrong evaluation of `dev->d_lltype`. ## Impact Allows use of ipv6 tcp and icmp over 6lowpan. ## Testing

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
yf13 commented on PR #15810: URL: https://github.com/apache/nuttx/pull/15810#issuecomment-2653138297 > > You can try adjusting the stack size of test task,arm64 may require more stack sizes. @anchao it turned out that I need run the rpsock_server in background from nsh.

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
CV-Bowen commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2653097563 @anchao OK, I missed this PR, let's merge https://github.com/apache/nuttx/pull/15810 first and I will rebase this PR. -- This is an automated message from the Apache Git Service. To re

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
raiden00pl commented on PR #15810: URL: https://github.com/apache/nuttx/pull/15810#issuecomment-2653075068 > So, problems in the community won't be solved in a timely manner, right? The maintainers will come and approve them when they are happy? There is no owner for any of the subsystem an

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15810: URL: https://github.com/apache/nuttx/pull/15810#issuecomment-2653054523 > > I don't understand why this PR has not been merged,How many people will only get your attention when they encounter problems? Is this still an active community? > > @anchao you

Re: [PR] xtensa: use misc register to implement up_this_task macro [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15821: URL: https://github.com/apache/nuttx/pull/15821#issuecomment-2653040516 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No, this PR does not fully meet the NuttX requirements.

[PR] xtensa: use misc register to implement up_this_task macro [nuttx]

2025-02-12 Thread via GitHub
chirping78 opened a new pull request, #15821: URL: https://github.com/apache/nuttx/pull/15821 ## Summary This is the follow-up work of https://github.com/apache/nuttx/pull/13726 (percpu reg store this_task) on xtensa. To help accelerate this_task(), use misc0 register to store the

[PR] mpfs/mpfs_irq.c: Enable interrupts on all harts [nuttx]

2025-02-12 Thread via GitHub
pussuw opened a new pull request, #15820: URL: https://github.com/apache/nuttx/pull/15820 ## Summary Enable interrupts on all cores, when enabling an external interrupt source. Instead of enabling an interrupt on the calling hart, enable it on every hart. This should balance th

Re: [PR] mpfs/mpfs_irq.c: Enable interrupts on all harts [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15820: URL: https://github.com/apache/nuttx/pull/15820#issuecomment-2653009509 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No, this PR summary does not fully meet the NuttX requi

Re: [PR] qemu/ivshmem: switch proxy/server roles [nuttx]

2025-02-12 Thread via GitHub
raiden00pl commented on PR #15810: URL: https://github.com/apache/nuttx/pull/15810#issuecomment-2652990344 > I don't understand why this PR has not been merged,How many people will only get your attention when they encounter problems? Is this still an active community? @anchao you sh

Re: [PR] mpfs_i2c: Tweaks and fixes for the i2c driver [nuttx]

2025-02-12 Thread via GitHub
nuttxpr commented on PR #15819: URL: https://github.com/apache/nuttx/pull/15819#issuecomment-2652979459 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR mostly meets the NuttX requirements, but c

Re: [PR] qemu-armv7a and armv8a Ivshmem configs update [nuttx]

2025-02-12 Thread via GitHub
anchao commented on PR #15818: URL: https://github.com/apache/nuttx/pull/15818#issuecomment-2652976505 please discard roles changes: https://github.com/apache/nuttx/pull/15810 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] mpfs_i2c: Tweaks and fixes for the i2c driver [nuttx]

2025-02-12 Thread via GitHub
pussuw opened a new pull request, #15819: URL: https://github.com/apache/nuttx/pull/15819 ## Summary This removes several error conditions that can be present after a failed i2c transaction. The most important of them is the priv->sem_isr semaphore which can be signaled due to a