[PR] examples/module: allow module on hostfs [nuttx-apps]

2025-01-23 Thread via GitHub
yf13 opened a new pull request, #2972: URL: https://github.com/apache/nuttx-apps/pull/2972 ## Summary This allows loading kernel modules from hostfs file system. ## Impact None ## Testing - local checks with FLAT mode NuttX on qemu-armv7a - CI checks

Re: [PR] Use small lock to protect resources related to i2c master and slave. [nuttx]

2025-01-23 Thread via GitHub
wangzhi-art commented on code in PR #15662: URL: https://github.com/apache/nuttx/pull/15662#discussion_r1926513572 ## arch/arm/src/stm32/stm32_i2c_v2.c: ## @@ -1759,7 +1770,8 @@ static int stm32_i2c_isr_process(struct stm32_i2c_priv_s *priv) priv->dcnt--; #ifdef C

(nuttx) branch master updated (49460518a9 -> 801cd455ef)

2025-01-23 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/nuttx.git from 49460518a9 Doc: Fortify Instructions add 801cd455ef sim/posix/uart: add host_printf() to debug some critical i

Re: [PR] Use small lock to protect resources related to i2c master and slave. [nuttx]

2025-01-23 Thread via GitHub
wangzhi-art commented on code in PR #15662: URL: https://github.com/apache/nuttx/pull/15662#discussion_r1926582696 ## arch/arm/src/stm32/stm32_i2c_v2.c: ## @@ -1759,7 +1770,8 @@ static int stm32_i2c_isr_process(struct stm32_i2c_priv_s *priv) priv->dcnt--; #ifdef C

[I] [BUG] RP23XX compile error due to undeclared GPIO_FUNC_NULL [nuttx]

2025-01-23 Thread via GitHub
paolovolpi opened a new issue, #15664: URL: https://github.com/apache/nuttx/issues/15664 Enabling the adc support under RP23XX arch boards is not possible due to the following compilation error: chip/rp23xx_adc.c:475:44: error: 'GPIO_FUNC_NULL' undeclared (first use in this functio

Re: [PR] Because sched_lock is now called in spin_lock_irqsave, there is no ne… [nuttx]

2025-01-23 Thread via GitHub
nuttxpr commented on PR #15665: URL: https://github.com/apache/nuttx/pull/15665#issuecomment-2609202706 [**\[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.

Re: [PR] Doc: Fortify Instructions [nuttx]

2025-01-23 Thread via GitHub
raiden00pl commented on code in PR #15658: URL: https://github.com/apache/nuttx/pull/15658#discussion_r1926598397 ## Documentation/guides/fortify.rst: ## @@ -0,0 +1,77 @@ + +Fortify + + +Overview + Review Comment: please use heading levels according t

Re: [PR] spin_lock_irqsave+sched_lock [nuttx]

2025-01-23 Thread via GitHub
hujun260 commented on code in PR #14578: URL: https://github.com/apache/nuttx/pull/14578#discussion_r1926571146 ## sched/sched/sched_unlock.c: ## @@ -288,30 +137,23 @@ int sched_unlock(void) if ((rtcb->flags & TCB_FLAG_POLICY_MASK) == TCB_FLAG_SCHED_SPORADIC

Re: [PR] examples/module: allow module on hostfs [nuttx-apps]

2025-01-23 Thread via GitHub
nuttxpr commented on PR #2972: URL: https://github.com/apache/nuttx-apps/pull/2972#issuecomment-2609160270 [**\[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 requiremen

[PR] Because sched_lock is now called in spin_lock_irqsave, there is no ne… [nuttx]

2025-01-23 Thread via GitHub
wangzhi-art opened a new pull request, #15665: URL: https://github.com/apache/nuttx/pull/15665 …ed to call it explicitly. *Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Because sched_lock is now

[I] [BUG] RP23XX Return board unique ID feature is broken [nuttx]

2025-01-23 Thread via GitHub
paolovolpi opened a new issue, #15666: URL: https://github.com/apache/nuttx/issues/15666 ### Description / Steps to reproduce the issue Select any RP23XX board. Enable Board Selection/Return board unique ID. We have 2 problems: 1) Wrong include https://github.com/apache

Re: [PR] arm: Support hardware debugging of ARM architecture [nuttx]

2025-01-23 Thread via GitHub
hujun260 commented on code in PR #15616: URL: https://github.com/apache/nuttx/pull/15616#discussion_r1926611892 ## arch/arm/src/armv7-a/Make.defs: ## @@ -43,7 +43,7 @@ CMN_ASRCS += arm_cpuhead.S arm_vectors.S arm_saveusercontext.S # Common C source files CMN_CSRCS += arm_cac

[I] [FEATURE] Documentation: move debugging tools in one place [nuttx]

2025-01-23 Thread via GitHub
raiden00pl opened a new issue, #15667: URL: https://github.com/apache/nuttx/issues/15667 ### Is your feature request related to a problem? Please describe. There are many useful debugging tools in NuttX, and many of them already have their documentation pages. Maybe we should crea

Re: [PR] arm: Support hardware debugging of ARM architecture [nuttx]

2025-01-23 Thread via GitHub
W-M-R commented on code in PR #15616: URL: https://github.com/apache/nuttx/pull/15616#discussion_r1926626113 ## arch/arm/src/armv7-a/Make.defs: ## @@ -43,7 +43,7 @@ CMN_ASRCS += arm_cpuhead.S arm_vectors.S arm_saveusercontext.S # Common C source files CMN_CSRCS += arm_cache.

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
raiden00pl commented on code in PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#discussion_r1926663701 ## audioutils/speexdsp/Kconfig: ## @@ -0,0 +1,26 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfig-language.txt in the

(nuttx) branch master updated: drivers/rpmsg_port_uart: remove debug message of datalen

2025-01-23 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/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 45f2a4b2cb drivers/rpmsg_port_uart: remove deb

Re: [PR] serial/uart_ram: update lower buffer offset if previous send completes [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15660: URL: https://github.com/apache/nuttx/pull/15660 -- 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 (45f2a4b2cb -> 5de6f7f6fd)

2025-01-23 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/nuttx.git from 45f2a4b2cb drivers/rpmsg_port_uart: remove debug message of datalen add 5de6f7f6fd serial/uart_ram: update lowe

(nuttx-apps) branch master updated: examples/module: add hostfs support

2025-01-23 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/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 048b3e6f7 examples/module: add hostfs sup

Re: [PR] examples/module: allow module on hostfs [nuttx-apps]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #2972: URL: https://github.com/apache/nuttx-apps/pull/2972 -- 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...

Re: [I] [BUG] `CONFIG_DEBUG_UORB` should enable `CONFIG_LIBC_FLOATINGPOINT` [nuttx]

2025-01-23 Thread via GitHub
raiden00pl commented on issue #15599: URL: https://github.com/apache/nuttx/issues/15599#issuecomment-2609301576 > My concern with depends is that as far as I'm aware, the user has to enable all dependencies before the option becomes visible to select. Please correct me if I'm wrong. Otherwi

Re: [PR] drivers/rpmsg_port_uart: remove debug message of datalen [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15663: URL: https://github.com/apache/nuttx/pull/15663 -- 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

Re: [PR] debug/0 address: Add 0 address access panic configuration [nuttx]

2025-01-23 Thread via GitHub
GUIDINGLI commented on code in PR #15562: URL: https://github.com/apache/nuttx/pull/15562#discussion_r1926745628 ## sched/irq/irq_initialize.c: ## @@ -87,5 +96,11 @@ void irq_initialize(void) #endif up_irqinitialize(); + +#ifdef CONFIG_ARCH_PROTECT_ZERO_ADDRESS + up_debug

Re: [PR] debug/0 address: Add 0 address access panic configuration [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15562: URL: https://github.com/apache/nuttx/pull/15562#discussion_r1926796531 ## sched/irq/irq_initialize.c: ## @@ -87,5 +96,11 @@ void irq_initialize(void) #endif up_irqinitialize(); + +#ifdef CONFIG_ARCH_PROTECT_ZERO_ADDRESS + up_debugpoi

(nuttx) branch master updated (c6858061ed -> b69111d16a)

2025-01-23 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/nuttx.git from c6858061ed nuttx/boards:fix kconfig under apps/testing folder that change config-testing-mm to config-testing-heap

Re: [PR] Use small lock to protect resources related to ethernet. [nuttx]

2025-01-23 Thread via GitHub
nuttxpr commented on PR #15668: URL: https://github.com/apache/nuttx/pull/15668#issuecomment-2609625361 [**\[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

(nuttx) 01/03: spinlock: Due to semantic inconsistency, remove/rename some functions with the _wo_note suffix.

2025-01-23 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/nuttx.git commit b49f4286fb60a2b1ceaefc0ce2b712d1992bf2e6 Author: hujun5 AuthorDate: Mon Jan 20 12:36:29 2025 +0800 spinlock: Due

(nuttx) 02/03: sched: remove csection and reduce the interrupt disabling time in sched_[un]lock

2025-01-23 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/nuttx.git commit 914ae532e68bf4ca75c758d6f541a1d2fcdce8c3 Author: hujun5 AuthorDate: Sun Jan 19 15:52:00 2025 +0800 sched: remove

(nuttx) 03/03: spinlock: add sched_lock to spin_lock_irqsave

2025-01-23 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/nuttx.git commit b69111d16a2a330fa272af8175c832e08881844b Author: hujun5 AuthorDate: Thu Jan 23 16:14:18 2025 +0800 spinlock: add

Re: [I] Thread can be unexpectedly suspended within a critical section. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 closed issue #1138: Thread can be unexpectedly suspended within a critical section. URL: https://github.com/apache/nuttx/issues/1138 -- 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 t

Re: [PR] spin_lock_irqsave+sched_lock [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #14578: URL: https://github.com/apache/nuttx/pull/14578 -- 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

Re: [I] Thread holding spinlock can be blocked. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 closed issue #9531: Thread holding spinlock can be blocked. URL: https://github.com/apache/nuttx/issues/9531 -- 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

[PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
XuNeo opened a new pull request, #15669: URL: https://github.com/apache/nuttx/pull/15669 *Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Coredump doens't need CONFIG_ELF to be enabled, but need elf.h to

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965 -- 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...

Re: [PR] nuttx/boards:fix kconfig under apps/testing folder that change config-testing-mm to config-testing-heap [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15659: URL: https://github.com/apache/nuttx/pull/15659 -- 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-apps) branch master updated (acc2b390c -> 077c3461d)

2025-01-23 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/nuttx-apps.git from acc2b390c examples/gps: Allow GPS serial port to be specified as command line argument, mark MINMEA dependency

(nuttx) branch master updated: nuttx/boards:fix kconfig under apps/testing folder that change config-testing-mm to config-testing-heap

2025-01-23 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/nuttx.git The following commit(s) were added to refs/heads/master by this push: new c6858061ed nuttx/boards:fix kconfig under apps

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#issuecomment-2609386582 > what is the license of this code? is it somehow based on `alsa-lib` ? Is it GPL? > > If this code is GPL then it can't be merged. GPL licensed code cannot be hosted in

Re: [PR] drivers/serial/serial.c: only adapt readv api to avoid block on secound read in readv() [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15604: URL: https://github.com/apache/nuttx/pull/15604#issuecomment-2609421630 @yamt do you have more suggestion for this patch? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] Use small lock to protect resources related to ethernet. [nuttx]

2025-01-23 Thread via GitHub
wangzhi-art opened a new pull request, #15668: URL: https://github.com/apache/nuttx/pull/15668 *Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Use small lock to protect resources related to ethernet.

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1926893869 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO +

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1926895587 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO +

Re: [PR] spin_lock_irqsave+sched_lock [nuttx]

2025-01-23 Thread via GitHub
anchao commented on PR #14578: URL: https://github.com/apache/nuttx/pull/14578#issuecomment-2609691441 @hujun260 Great work, I think we can prepare a report showing the benefits of SMP mode after all spinlock changes -- This is an automated message from the Apache Git Service. To respond

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1926914861 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO

(nuttx) branch master updated (b69111d16a -> b0af946b7a)

2025-01-23 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/nuttx.git from b69111d16a spinlock: add sched_lock to spin_lock_irqsave add b0af946b7a Because sched_lock is now called in spi

Re: [PR] Because sched_lock is now called in spin_lock_irqsave, there is no ne… [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15665: URL: https://github.com/apache/nuttx/pull/15665 -- 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

[PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
simbit18 opened a new pull request, #15671: URL: https://github.com/apache/nuttx/pull/15671 ## Summary Remove spaces from Kconfig files Add TABs ## Impact none ## Testing -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] x86_64: fix regression https://github.com/apache/nuttx/pull/15651 [nuttx]

2025-01-23 Thread via GitHub
nuttxpr commented on PR #15670: URL: https://github.com/apache/nuttx/pull/15670#issuecomment-2609786041 [**\[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

Re: [PR] Use small lock to protect resources related to i2c master and slave. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15662: URL: https://github.com/apache/nuttx/pull/15662#discussion_r1926541668 ## arch/arm/src/stm32/stm32_i2c_v2.c: ## @@ -1759,7 +1770,8 @@ static int stm32_i2c_isr_process(struct stm32_i2c_priv_s *priv) priv->dcnt--; #ifd

Re: [PR] sim/posix/uart: add host_printf() to debug some critical issue [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15661: URL: https://github.com/apache/nuttx/pull/15661 -- 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

Re: [PR] Use small lock to protect resources related to i2c master and slave. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15662: URL: https://github.com/apache/nuttx/pull/15662#discussion_r1926541668 ## arch/arm/src/stm32/stm32_i2c_v2.c: ## @@ -1759,7 +1770,8 @@ static int stm32_i2c_isr_process(struct stm32_i2c_priv_s *priv) priv->dcnt--; #ifd

Re: [I] [BUG] ESP32C6 - UsbConsole feature not working [nuttx]

2025-01-23 Thread via GitHub
eren-terzioglu commented on issue #15656: URL: https://github.com/apache/nuttx/issues/15656#issuecomment-2609129360 > Hello. > > I tried to build exactly commit that this feature was added: git checkout [86efbcb](https://github.com/apache/nuttx/commit/86efbcbee77bc0a29af07807080f4804a

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
simbit18 commented on PR #15671: URL: https://github.com/apache/nuttx/pull/15671#issuecomment-2609974268 Hi @xiaoxiang781216 unfortunately, no !!! I use on Windows simple Notepad++ editor. https://notepad-plus-plus.org/ -- This is an automated message from the Apache Git Service. To

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927057268 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO +

(nuttx) branch master updated (5c02379548 -> f8501fa3ea)

2025-01-23 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/nuttx.git from 5c02379548 Fix Kconfig style add a0fdb7f63b arm/goldfish: add ARCH_HAVE_ELF_EXECUTABLE add f8501fa3ea arch

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15671: URL: https://github.com/apache/nuttx/pull/15671#issuecomment-2610121861 @simbit18 fixed here: https://github.com/apache/nuttx/pull/15674. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [I] [BUG] `CONFIG_DEBUG_UORB` should enable `CONFIG_LIBC_FLOATINGPOINT` [nuttx]

2025-01-23 Thread via GitHub
linguini1 commented on issue #15599: URL: https://github.com/apache/nuttx/issues/15599#issuecomment-2609977924 > But I see this more like Kconfig limitation, where there really is no perfect solution. That makes sense. I am surprised that something as widely used as Kconfig does not

[PR] samv7/adc: fix handling of ANIOC_TRIGGER ioctl [nuttx]

2025-01-23 Thread via GitHub
michallenc opened a new pull request, #15675: URL: https://github.com/apache/nuttx/pull/15675 ## Summary ADC peripheral can be configured also for PWM trigger, so the `ifdef` should depend only on `CONFIG_SAMV7_AFEC_SWTRIG`. Also handle the situation when `ANIOC_TRIGGER` is called and th

Re: [I] [FEATURE] Documentation: move debugging tools in one place [nuttx]

2025-01-23 Thread via GitHub
acassis commented on issue #15667: URL: https://github.com/apache/nuttx/issues/15667#issuecomment-2610188606 @raiden00pl I think this is a good idea! Even debugging tools can be categorized, i.e.: debug initialization, memory stack, performance, etc. -- This is an automated message from t

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
raiden00pl commented on code in PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#discussion_r1927466673 ## audioutils/alsa-lib/pcm.c: ## @@ -0,0 +1,808 @@ +/ + * apps/audioutils/alsa-lib/pcm

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
raiden00pl commented on PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#issuecomment-2610652877 > it's rewrite from scratch, but follow alsa-lib public API, just like nuttx v4l2 driver framework: If this is **rewriten** based on alsa-lib then this is derivative work. If

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927099342 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
simbit18 commented on PR #15671: URL: https://github.com/apache/nuttx/pull/15671#issuecomment-2610001235 @xiaoxiang781216 the sim/citest build is broken ``` Configuration/Tool: sim/citest 202

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927099342 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15671: URL: https://github.com/apache/nuttx/pull/15671 -- 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

Re: [PR] syscall: add missing epoll API [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15672: URL: https://github.com/apache/nuttx/pull/15672#discussion_r1927166415 ## syscall/syscall.csv: ## @@ -19,9 +19,12 @@ "connect","sys/socket.h","defined(CONFIG_NET)","int","int","FAR const struct sockaddr *","socklen_t" "dup","uni

Re: [PR] add Elf executable support for arm/goldfish/x86 [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15673: URL: https://github.com/apache/nuttx/pull/15673#issuecomment-2610119601 ci break is fixed by https://github.com/apache/nuttx/pull/15674, let's merge this simple patch directly to avoid wasting the ci resource. -- This is an automated message from th

Re: [PR] add Elf executable support for arm/goldfish/x86 [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15673: URL: https://github.com/apache/nuttx/pull/15673 -- 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

[PR] boards/stm32h7: Fix wrong usbnsh name and fix Documentation [nuttx]

2025-01-23 Thread via GitHub
acassis opened a new pull request, #15678: URL: https://github.com/apache/nuttx/pull/15678 ## Summary Fixes the incorrect board profile name to 'usbnsh' used on WeAct-STM32H743 board. ## Impact User will have a better Documentation/ ## Testing WeAct-STM

Re: [PR] boards/stm32h7: Fix wrong usbnsh name and fix Documentation [nuttx]

2025-01-23 Thread via GitHub
acassis commented on PR #15678: URL: https://github.com/apache/nuttx/pull/15678#issuecomment-2610830628 @TOMASHEVSKIYIVAN I rename the board profile. I know you based on Linum board to use the name, but all NSH over USB on NuttX are named "usbnsh". I contacted @JorgeGzm and he also will fix

Re: [PR] Use small lock to protect resources related to irq in arch ARM. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15625: URL: https://github.com/apache/nuttx/pull/15625 -- 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

[PR] nuttx:fix tools-mm to heap [nuttx]

2025-01-23 Thread via GitHub
txy-21 opened a new pull request, #15674: URL: https://github.com/apache/nuttx/pull/15674 ## Summary config-testing-heap is changed to conofig-testing-heap, so the citest tools need to change. ## Impact nuttx/tools/ci ## Testing CI test. -- T

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927099342 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927099342 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO

Re: [I] [BUG] `CONFIG_DEBUG_UORB` should enable `CONFIG_LIBC_FLOATINGPOINT` [nuttx]

2025-01-23 Thread via GitHub
acassis commented on issue #15599: URL: https://github.com/apache/nuttx/issues/15599#issuecomment-2610155485 @raiden00pl I think kconfig-frontends also support to see all the entries, but I think it is the [Z] instead of [A]. Unfortunately these tools are diverging (extend and kill approach

Re: [PR] move coredump info to note [nuttx-apps]

2025-01-23 Thread via GitHub
nuttxpr commented on PR #2973: URL: https://github.com/apache/nuttx-apps/pull/2973#issuecomment-2610418681 [**\[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 requiremen

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927052955 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO +

[PR] add Elf executable support for arm/goldfish/x86 [nuttx]

2025-01-23 Thread via GitHub
jasonbu opened a new pull request, #15673: URL: https://github.com/apache/nuttx/pull/15673 ## Summary The executable format able to support add-symbol-file in kernel build. Will benifit when we tring to debug in qemu. ## Impact Kernel build apps able to select the ## T

Re: [PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
XuNeo commented on PR #15677: URL: https://github.com/apache/nuttx/pull/15677#issuecomment-2610399095 The documentation for coredump is actually here https://nuttx.apache.org/docs/latest/guides/coredump.html Maybe a reorganization is needed, as suggested by https://github.com/apache/nutt

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15671: URL: https://github.com/apache/nuttx/pull/15671#issuecomment-2610040333 @txy-21 please fix the break as soon as possible. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] arch: select LIBC_ARCH_ELF when using COREDUMP [nuttx]

2025-01-23 Thread via GitHub
anchao commented on code in PR #15669: URL: https://github.com/apache/nuttx/pull/15669#discussion_r1927123069 ## sched/Kconfig: ## @@ -2011,6 +2011,7 @@ config ASSERT_PAUSE_CPU_TIMEOUT config COREDUMP bool "Coredump support" depends on ARCH_HAVE_TCBINFO +

(nuttx) branch master updated (aa9f366eec -> 5c02379548)

2025-01-23 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/nuttx.git from aa9f366eec x86_64: fix regression https://github.com/apache/nuttx/pull/15651 add 5c02379548 Fix Kconfig style

Re: [PR] Fix Kconfig style [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15671: URL: https://github.com/apache/nuttx/pull/15671#issuecomment-2610044688 Before @txy-21 fix the problem, let's merge your trivial change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [PR] syscall: add missing epoll API [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 commented on PR #15672: URL: https://github.com/apache/nuttx/pull/15672#issuecomment-2610093976 ci break is fixed by https://github.com/apache/nuttx/pull/15674, let's merge this simple patch directly to avoid wasting the ci resource. -- This is an automated message from th

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
raiden00pl commented on code in PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#discussion_r1927245486 ## audioutils/alsa-lib/bits_convert.c: ## @@ -0,0 +1,172 @@ +/ + * apps/audioutils/als

[PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
XuNeo opened a new pull request, #15677: URL: https://github.com/apache/nuttx/pull/15677 *Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Add a custom note for NuttX information including magic and c

Re: [PR] misc/coredump: move coredump info to note [nuttx]

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

Re: [PR] Add vela alsa lib interface [nuttx-apps]

2025-01-23 Thread via GitHub
raiden00pl commented on PR #2970: URL: https://github.com/apache/nuttx-apps/pull/2970#issuecomment-2610307732 I'm definitely not licences expert so I don't know if licensing this code to Apache is OK. But for sure caution is required here. GPL is like virus. Where does "interface

Re: [PR] Use small lock to protect resources related to i2c master and slave. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15662: URL: https://github.com/apache/nuttx/pull/15662 -- 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

Re: [PR] syscall: add missing epoll API [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15672: URL: https://github.com/apache/nuttx/pull/15672 -- 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

Re: [PR] samv7/adc: fix handling of ANIOC_TRIGGER ioctl [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15675: URL: https://github.com/apache/nuttx/pull/15675 -- 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 (2c4fe28d4e -> 758b3baef0)

2025-01-23 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/nuttx.git from 2c4fe28d4e Use small lock to protect resources related to i2c master and slave. add 758b3baef0 samv7/adc: fix

(nuttx) branch master updated (758b3baef0 -> 7a94a4553d)

2025-01-23 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/nuttx.git from 758b3baef0 samv7/adc: fix handling of ANIOC_TRIGGER ioctl add 7a94a4553d syscall: add missing epoll API No new

(nuttx) branch master updated (7a94a4553d -> 919ed2e3d4)

2025-01-23 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/nuttx.git from 7a94a4553d syscall: add missing epoll API add 919ed2e3d4 Use small lock to protect resources related to i2c. N

Re: [PR] Use small lock to protect resources related to i2c. [nuttx]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #15648: URL: https://github.com/apache/nuttx/pull/15648 -- 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

Re: [PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
acassis commented on PR #15677: URL: https://github.com/apache/nuttx/pull/15677#issuecomment-2610556242 > The documentation for coredump is actually here https://nuttx.apache.org/docs/latest/guides/coredump.html Maybe a reorganization is needed, as suggested by #15667 Agreed! Maybe t

(nuttx) branch master updated (f8501fa3ea -> 2c4fe28d4e)

2025-01-23 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/nuttx.git from f8501fa3ea arch/x86_64: add ARCH_HAVE_ELF_EXECUTABLE add 2c4fe28d4e Use small lock to protect resources related

Re: [PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
W-M-R commented on code in PR #15677: URL: https://github.com/apache/nuttx/pull/15677#discussion_r1927437452 ## sched/misc/coredump.c: ## @@ -507,6 +513,39 @@ static void elf_emit_memory(FAR struct elf_dumpinfo_s *cinfo, int memsegs) } } +/**

Re: [PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
W-M-R commented on code in PR #15677: URL: https://github.com/apache/nuttx/pull/15677#discussion_r1927437452 ## sched/misc/coredump.c: ## @@ -507,6 +513,39 @@ static void elf_emit_memory(FAR struct elf_dumpinfo_s *cinfo, int memsegs) } } +/**

Re: [PR] misc/coredump: move coredump info to note [nuttx]

2025-01-23 Thread via GitHub
W-M-R commented on code in PR #15677: URL: https://github.com/apache/nuttx/pull/15677#discussion_r1927437452 ## sched/misc/coredump.c: ## @@ -507,6 +513,39 @@ static void elf_emit_memory(FAR struct elf_dumpinfo_s *cinfo, int memsegs) } } +/**

  1   2   >