Re: [PR] include/strings: replace ffs from an inline function to a macro [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 commented on code in PR #14028: URL: https://github.com/apache/nuttx/pull/14028#discussion_r1794792486 ## include/strings.h: ## @@ -73,73 +73,46 @@ extern "C" / #ifdef CONFIG_HAVE_BU

Re: [PR] serial:update some bug fix,and functional improvements [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #13914: URL: https://github.com/apache/nuttx/pull/13914#issuecomment-2404204357 Could this be the problem: CONFIG_16550_REGINCR is referenced twice, so it's double-incrementing the UART Address? https://github.com/apache/nuttx/blob/master/drivers/serial/uart_16

(nuttx) branch master updated: arch/x86_64:Add allsymbol functionality

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new f858026819 arch/x86_64:Add allsymbol functiona

Re: [PR] arch/x86_64:Add allsymbol functionality [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #13980: URL: https://github.com/apache/nuttx/pull/13980 -- 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] arch/x86_64:Synchronize comments and data type information in the com… [nuttx]

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

[PR] sched/pthread: save pid before task activate [nuttx]

2024-10-09 Thread via GitHub
zyfeier opened a new pull request, #14032: URL: https://github.com/apache/nuttx/pull/14032 ## Summary When a high-priority pthread is created, ptcb may be released after nxtask_activate. ## Impact pthread_create ## Testing sim board pipe test pass -- Thi

[PR] arch/x86_64:Synchronize comments and data type information in the com… [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14031: URL: https://github.com/apache/nuttx/pull/14031 ## Summary Synchronize comments and data type information in the common/intel64 ## Impact no impact ## Testing ostset -- This is an automated message from the Apache Git S

Re: [PR] serial:update some bug fix,and functional improvements [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #13914: URL: https://github.com/apache/nuttx/pull/13914#issuecomment-2404172009 Sorry @anjiahao1: This commit seems to hang the Console Output on Milk-V Duo S (based on 16550 UART): ```text Bytes transferred = 21575 (5447 hex) ## Flattened Device Tree blo

Re: [I] esp32s3 : float calc error or stack error [nuttx]

2024-10-09 Thread via GitHub
w2016561536 commented on issue #12490: URL: https://github.com/apache/nuttx/issues/12490#issuecomment-2404144881 Perhaps this bug needs Espressif staff to work on. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] include/strings: replace ffs from an inline function to a macro [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14028: URL: https://github.com/apache/nuttx/pull/14028#issuecomment-2404141051 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The PR summary provides context, explains the issue (sh

Re: [PR] {Bp-13688} sched optimise for sim, CONFIG_SCHED_INSTRUMENTATION to CONFIG_SCHED_INSTRUMENTATION_SWITCH [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14029: URL: https://github.com/apache/nuttx/pull/14029#issuecomment-2404138802 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) ## NuttX PR Requirements Review **Verdict:** The

Re: [PR] arch/x86_64:Add some macro definitions [nuttx]

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

[PR] arch/x86_64:Add some macro definitions [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14030: URL: https://github.com/apache/nuttx/pull/14030 ## Summary Add some macro definitions ## Impact no impact ## Testing ostest -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[PR] {Bp-13688} sched optimise for sim, CONFIG_SCHED_INSTRUMENTATION to CONFIG_SCHED_INSTRUMENTATION_SWITCH [nuttx]

2024-10-09 Thread via GitHub
jerpelea opened a new pull request, #14029: URL: https://github.com/apache/nuttx/pull/14029 ## Summary sched_smp:sync refcount before enqueue smp call queue sched_smp:adjust the critical section to protect refcount from multi cores access fix build error when sim SMP CC:

[PR] include/strings: replace ffs from an inline function to a macro [nuttx]

2024-10-09 Thread via GitHub
Gary-Hobson opened a new pull request, #14028: URL: https://github.com/apache/nuttx/pull/14028 ## Summary using static inline functions sometimes generates compilation warnings: nxfonts/nxfonts_bitmaps_sans23x27.c:168:50: warning: declaration of ‘ffs’ shadows a built-in function [-Wshado

(nuttx) branch master updated (20cba94a86 -> 93890321c5)

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from 20cba94a86 risc-v/espressif: Fix alert message in `esp_setup_irq()` add 93890321c5 sched/wdog: Fix code style

Re: [PR] sched/wdog: Fix code style [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14013: URL: https://github.com/apache/nuttx/pull/14013 -- 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] {Bp-2658} fsutils/mkmbr/mkmbr.c: fix null pointer access when not all device sp… [nuttx-apps]

2024-10-09 Thread via GitHub
jerpelea opened a new pull request, #2673: URL: https://github.com/apache/nuttx-apps/pull/2673 ## Summary …ace is defined in mbr argv[argn] is accessed out of range when there are neither four partitions are specified nor the last partition is of auto size. Add a number of partitio

Re: [PR] group: access group in critical section [nuttx]

2024-10-09 Thread via GitHub
pussuw commented on code in PR #13944: URL: https://github.com/apache/nuttx/pull/13944#discussion_r1794708043 ## sched/group/group_argvstr.c: ## @@ -60,16 +61,20 @@ size_t group_argvstr(FAR struct tcb_s *tcb, FAR char *args, size_t size) Review Comment: Also the name grou

Re: [PR] {Bp-13716} change nxsched_islocked_global to nxsched_islocked_tcb [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14025: URL: https://github.com/apache/nuttx/pull/14025#issuecomment-2404095089 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) ## NuttX PR Requirements Review This PR appears

Re: [I] esp32s3 : float calc error or stack error [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 commented on issue #12490: URL: https://github.com/apache/nuttx/issues/12490#issuecomment-2404094824 but the change look like FreeRTOS specific: https://github.com/espressif/esp-idf/commit/b03c8912c73fa59061d97a2f5fd5acddcc3fa356#diff-db429b5abb80b87b6da1abb1ecd103c81fc2d9

Re: [PR] {Bp-13647} can: Merge cd_error and rx_overflow into rx_error so the error could … [nuttx]

2024-10-09 Thread via GitHub
jerpelea commented on PR #14027: URL: https://github.com/apache/nuttx/pull/14027#issuecomment-2404093526 @xiaoxiang781216 there will be several can updates which will fail to compile please merge -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [I] [BUG] esp32 pid controller + psram issue [nuttx]

2024-10-09 Thread via GitHub
yamt commented on issue #13761: URL: https://github.com/apache/nuttx/issues/13761#issuecomment-2404090666 > ESP32 PID Controller is (or should be) experimental and must not be used for production. why? is there a known hardware bug or something? do you have a reference? >

(nuttx) branch master updated (e3b3a6145a -> 20cba94a86)

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git from e3b3a6145a arch/x86_64: idle convert all asm() to __asm__() add 20cba94a86 risc-v/espressif: Fix alert message

[PR] {Bp-13716} change nxsched_islocked_global to nxsched_islocked_tcb [nuttx]

2024-10-09 Thread via GitHub
jerpelea opened a new pull request, #14025: URL: https://github.com/apache/nuttx/pull/14025 ## Summary 1 To improve efficiency, we mimic Linux's behavior where preemption disabling is only applicable to the current CPU and does not affect other CPUs. 2 In the future, we will impleme

Re: [PR] risc-v/espressif: Fix alert message in `esp_setup_irq()` [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14020: URL: https://github.com/apache/nuttx/pull/14020 -- 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] sched: fix omitted merging the pending queue regression [nuttx]

2024-10-09 Thread via GitHub
hujun260 opened a new pull request, #14026: URL: https://github.com/apache/nuttx/pull/14026 ## Summary sched: fix omitted merging the pending queue regression This commit fixes the regression from https://github.com/apache/nuttx/pull/13995 ## Impact sched ## Testin

(nuttx) branch master updated: arch/x86_64: idle convert all asm() to __asm__()

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new e3b3a6145a arch/x86_64: idle convert all asm()

Re: [PR] arch/x86_64: idle convert all asm() to __asm__() [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14021: URL: https://github.com/apache/nuttx/pull/14021 -- 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] arch: support customized up_cpu_index() in AMP mode [nuttx]

2024-10-09 Thread via GitHub
hujun260 commented on PR #13886: URL: https://github.com/apache/nuttx/pull/13886#issuecomment-2404066173 https://github.com/apache/nuttx/pull/14024 -- 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

[PR] sim: only POSIX implementation (ARCH_HAVE_MULTICPU) is provided [nuttx]

2024-10-09 Thread via GitHub
hujun260 opened a new pull request, #14024: URL: https://github.com/apache/nuttx/pull/14024 ## Summary sim: only POSIX implementation (ARCH_HAVE_MULTICPU) is provided This commit fixes the regression from https://github.com/apache/nuttx/pull/13886 ## Impact sim ## Test

Re: [PR] arch: support customized up_cpu_index() in AMP mode [nuttx]

2024-10-09 Thread via GitHub
hujun260 commented on code in PR #13886: URL: https://github.com/apache/nuttx/pull/13886#discussion_r1794665519 ## arch/sim/src/sim/CMakeLists.txt: ## @@ -174,6 +174,9 @@ endif() if(CONFIG_SMP) list(APPEND SRCS sim_smpsignal.c sim_cpuidlestack.c) +endif() + +if(CONFIG_ARCH

Re: [PR] Rpmsg ping: improve the rpmsg ping [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14023: URL: https://github.com/apache/nuttx/pull/14023#issuecomment-2403986793 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This pull request appears to meet the NuttX requirement

[PR] Rpmsg ping: improve the rpmsg ping [nuttx]

2024-10-09 Thread via GitHub
CV-Bowen opened a new pull request, #14023: URL: https://github.com/apache/nuttx/pull/14023 ## Summary -rpmsg_ping.c: fix msg data memset length The range of memset exceeds the size of the buffer -rpmsg_ping.c: change msg cmd type for more compatible

Re: [I] esp32s3 : float calc error or stack error [nuttx]

2024-10-09 Thread via GitHub
yamt commented on issue #12490: URL: https://github.com/apache/nuttx/issues/12490#issuecomment-2403977033 i guess he meant this one. https://github.com/espressif/esp-idf/issues/11690 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [I] esp32s3 : float calc error or stack error [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 commented on issue #12490: URL: https://github.com/apache/nuttx/issues/12490#issuecomment-2403939911 @ProfFan could you point out the patch? we can apply the change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [I] esp32s3 : float calc error or stack error [nuttx]

2024-10-09 Thread via GitHub
ProfFan commented on issue #12490: URL: https://github.com/apache/nuttx/issues/12490#issuecomment-2403928958 Hey guys, I am the reporter of the original problem in ESP-IDF FreeRTOS. Yes, this is a silent data corruption and the current ESP-IDF's interrupt vector assembly file has a fix. Reg

(nuttx) branch master updated: arch/x86_64: cpuid expect 32 bit variables

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 1af831e139 arch/x86_64: cpuid expect 32 bit va

Re: [PR] arch/x86_64: cpuid expect 32 bit variables [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14019: URL: https://github.com/apache/nuttx/pull/14019 -- 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] arch/arm64: Fix clock drift from tick timer [nuttx]

2024-10-09 Thread via GitHub
jlaitine commented on PR #13984: URL: https://github.com/apache/nuttx/pull/13984#issuecomment-2403904559 What is the +1 bug you are referring to? Are you getting an error print and assertion with the test case above? Re-starting the timer after systick has already passed means that servic

(nuttx) branch master updated: arch/x86_64:Add perf tool

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 2448e8a59e arch/x86_64:Add perf tool 2448e8a59

Re: [PR] arch/x86_64:Add perf tool [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14016: URL: https://github.com/apache/nuttx/pull/14016 -- 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] sched/wdog: Fix code style [nuttx]

2024-10-09 Thread via GitHub
GUIDINGLI commented on PR #14013: URL: https://github.com/apache/nuttx/pull/14013#issuecomment-2403886353 Download error, re-run the CI -- 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] CI: Skip the CI Builds that don't match the Board Label [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14022: URL: https://github.com/apache/nuttx/pull/14022#issuecomment-2403884503 [**\[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.

[PR] CI: Skip the CI Builds that don't match the Board Label [nuttx]

2024-10-09 Thread via GitHub
lupyuen opened a new pull request, #14022: URL: https://github.com/apache/nuttx/pull/14022 ## Summary This PR extends the CI Build Rules to eliminate more unnecessary builds. We use the Board Label for PRs: - Previously: "Arch: arm" will build only `arm-01` to `arm-14` - Now: "B

(nuttx) branch master updated: arch/x86_64:Use the checkstack function

2024-10-09 Thread ligd
This is an automated email from the ASF dual-hosted git repository. ligd 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 f81c7dbc93 arch/x86_64:Use the checkstack function

Re: [PR] arch/x86_64:Use the checkstack function [nuttx]

2024-10-09 Thread via GitHub
GUIDINGLI merged PR #14017: URL: https://github.com/apache/nuttx/pull/14017 -- 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.ap

Re: [PR] arch/x86_64: idle convert all asm() to __asm__() [nuttx]

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

Re: [PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx]

2024-10-09 Thread via GitHub
anchao commented on PR #14014: URL: https://github.com/apache/nuttx/pull/14014#issuecomment-2403871190 Why not fix this issue directly? Ninja will be much faster than Makefile backend(CMake) -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] build ci:add MSVC windows native ci workflow [nuttx]

2024-10-09 Thread via GitHub
lupyuen commented on code in PR #13894: URL: https://github.com/apache/nuttx/pull/13894#discussion_r1794561766 ## .github/workflows/build.yml: ## @@ -331,3 +331,54 @@ jobs: name: msys2-${{matrix.boards}}-builds path: buildartifacts/ continue-on-err

[PR] arch/x86_64: idle convert all asm() to __asm__() [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14021: URL: https://github.com/apache/nuttx/pull/14021 ## Summary idle convert all asm() to __asm__() ## Impact no impact ## Testing ostest -- This is an automated message from the Apache Git Service. To respond to the message, please

[PR] risc-v/espressif: Fix alert message in `esp_setup_irq()` [nuttx]

2024-10-09 Thread via GitHub
no1wudi opened a new pull request, #14020: URL: https://github.com/apache/nuttx/pull/14020 ## Summary Correct the alert message in `esp_setup_irq()` if irq number allocation fails, the parameter number is not matched with format specifier. ## Impact riscv/espressif ##

Re: [PR] arch: support customized up_cpu_index() in AMP mode [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 commented on code in PR #13886: URL: https://github.com/apache/nuttx/pull/13886#discussion_r1794557532 ## arch/sim/src/sim/CMakeLists.txt: ## @@ -174,6 +174,9 @@ endif() if(CONFIG_SMP) list(APPEND SRCS sim_smpsignal.c sim_cpuidlestack.c) +endif() + +if(CONF

[PR] arch/x86_64: cpuid expect 32 bit variables [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14019: URL: https://github.com/apache/nuttx/pull/14019 ## Summary cpuid expect 32 bit variables ## Impact no impact ## Testing ostest -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794538000 ## drivers/perf/CMakeLists.txt: ## @@ -0,0 +1,34 @@ +# ## +# drivers/perf/CMakeLists.tx

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794538756 ## arch/arm64/src/common/arm64_hwdebug.h: ## @@ -0,0 +1,337 @@ +/ + * arch/arm64/src/com

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794538944 ## arch/arm64/src/common/arm64_hwdebug.c: ## @@ -0,0 +1,1311 @@ +/ + * arch/arm64/src/co

Re: [PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx-apps]

2024-10-09 Thread via GitHub
cederom commented on PR #2671: URL: https://github.com/apache/nuttx-apps/pull/2671#issuecomment-2403839781 Okay I filled in a new Issue to remember this needs a fix :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[I] [BUG] MSYS2/CMAKE/NINJA: Argument list too long Build Broken. [nuttx-apps]

2024-10-09 Thread via GitHub
cederom opened a new issue, #2672: URL: https://github.com/apache/nuttx-apps/issues/2672 ### Description / Steps to reproduce the issue * As found in https://github.com/apache/nuttx-apps/pull/2671 the msys2 / cmake / ninja build is broken due to 8192 chars limit on command line length

Re: [PR] net/local: fix the return address is incorrect when accept [nuttx]

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

[PR] net/local: fix the return address is incorrect when accept [nuttx]

2024-10-09 Thread via GitHub
zhhyu7 opened a new pull request, #14018: URL: https://github.com/apache/nuttx/pull/14018 ## Summary The argument addr is a pointer to a sockaddr structure. This structure is filled in with the address of the peer socket, as known to the communications layer. ## Impact ## T

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794540878 ## arch/arm64/src/common/arm64_pmuv3.h: ## @@ -0,0 +1,176 @@ +/ + * arch/arm64/src/commo

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794539106 ## arch/arm64/src/common/Make.defs: ## @@ -119,3 +119,7 @@ endif ifeq ($(CONFIG_ARM64_SEMIHOSTING_HOSTFS),y) CMN_CSRCS += arm64_hostfs.c endif + +ifeq ($(CONFI

Re: [PR] build ci:add MSVC windows native ci workflow [nuttx]

2024-10-09 Thread via GitHub
xuxin930 commented on PR #13894: URL: https://github.com/apache/nuttx/pull/13894#issuecomment-2403829475 hi @hujun260 Can you help me look at the MSVC issues caused by this PR? https://github.com/apache/nuttx/pull/13886 -- This is an automated message from the Apache Git Service. To

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794538000 ## drivers/perf/CMakeLists.txt: ## @@ -0,0 +1,34 @@ +# ## +# drivers/perf/CMakeLists.tx

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794537748 ## include/perf/arm_pmu.h: ## @@ -0,0 +1,175 @@ +/ + * include/perf/arm_pmu.h + * + * Li

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794537365 ## arch/arm64/src/common/arm64_registerdump.c: ## @@ -50,7 +50,7 @@ uintptr_t up_getusrsp(void *regs) { struct regs_context *ptr = regs; - return ptr->regs[RE

Re: [PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx-apps]

2024-10-09 Thread via GitHub
xuxin930 commented on PR #2671: URL: https://github.com/apache/nuttx-apps/pull/2671#issuecomment-2403825866 > Thanks @xuxin930 :-) > > * If the problem is on CI then developers on their local machines will have that problem too? Does that mean CMake + Ninja is broken on windoze? >

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794530898 ## arch/arm64/src/common/arm64_fatal.c: ## @@ -527,10 +527,10 @@ static int arm64_exception_handler(struct regs_context *regs) if (ret != 0) { - sin

Re: [PR] arch: support customized up_cpu_index() in AMP mode [nuttx]

2024-10-09 Thread via GitHub
xuxin930 commented on code in PR #13886: URL: https://github.com/apache/nuttx/pull/13886#discussion_r1794527383 ## arch/sim/src/sim/CMakeLists.txt: ## @@ -174,6 +174,9 @@ endif() if(CONFIG_SMP) list(APPEND SRCS sim_smpsignal.c sim_cpuidlestack.c) +endif() + +if(CONFIG_ARCH

Re: [PR] arch/x86_64:Use the checkstack function [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14017: URL: https://github.com/apache/nuttx/pull/14017#issuecomment-2403795815 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This Pull Request (PR) description, as it stands, **doe

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794505119 ## arch/arm64/src/common/arm64_pmuv3.h: ## @@ -0,0 +1,176 @@ +/ + * arch/arm64/src/commo

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794502329 ## arch/arm64/src/common/arm64_hwdebug.h: ## @@ -0,0 +1,337 @@ +/ + * arch/arm64/src/com

Re: [PR] Arm64 arch [nuttx]

2024-10-09 Thread via GitHub
lipengfei28 commented on code in PR #13978: URL: https://github.com/apache/nuttx/pull/13978#discussion_r1794502193 ## arch/arm64/src/common/Make.defs: ## @@ -119,3 +119,7 @@ endif ifeq ($(CONFIG_ARM64_SEMIHOSTING_HOSTFS),y) CMN_CSRCS += arm64_hostfs.c endif + +ifeq ($(CONFI

Re: [PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx-apps]

2024-10-09 Thread via GitHub
cederom commented on PR #2671: URL: https://github.com/apache/nuttx-apps/pull/2671#issuecomment-2403784315 Thanks @xuxin930 :-) * If the problem is on CI then developers on their local machines will have that problem too? Does that mean CMake + Ninja is broken on windoze? * Does this c

[PR] arch/x86_64:Use the checkstack function [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14017: URL: https://github.com/apache/nuttx/pull/14017 ## Summary create stack use the checkstack function ## Impact no impact ## Testing ostest -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [PR] arch/x86_64:Add nanosecond delay interface to TSC [nuttx]

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

Re: [PR] arch/x86_64:Add perf tool [nuttx]

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

[PR] arch/x86_64:Add perf tool [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14016: URL: https://github.com/apache/nuttx/pull/14016 ## Summary Add perf tool ## Impact no impact ## Testing perf test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[PR] arch/x86_64:Add nanosecond delay interface to TSC [nuttx]

2024-10-09 Thread via GitHub
xianglyc opened a new pull request, #14015: URL: https://github.com/apache/nuttx/pull/14015 ## Summary Add nanosecond delay interface to TSC ## Impact no impact ## Testing ostest -- This is an automated message from the Apache Git Service. To respond to the message, plea

[PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx-apps]

2024-10-09 Thread via GitHub
xuxin930 opened a new pull request, #2671: URL: https://github.com/apache/nuttx-apps/pull/2671 ## Summary CI in the msys2 environment, the CMake ninja generator will have a problem with parameters being too long ## Impact Msys2 ## Testing CI build

[PR] build.yml[bugfix]:disable CMake Ninja for Msys2 [nuttx]

2024-10-09 Thread via GitHub
xuxin930 opened a new pull request, #14014: URL: https://github.com/apache/nuttx/pull/14014 ## Summary CI in the msys2 environment, the CMake ninja generator will have a problem with parameters being too long ## Impact Msys2 ## Testing Ci build -- This i

Re: [PR] build ci:add MSVC windows native ci workflow [nuttx]

2024-10-09 Thread via GitHub
lupyuen commented on PR #13894: URL: https://github.com/apache/nuttx/pull/13894#issuecomment-2403733220 @xuxin930 Wonder if we can extend the Build Rules `arch.yml` + Build Workflow `build.yml`, which are described here: https://github.com/apache/nuttx/issues/13775 -- This is an automate

Re: [PR] Adding partitioning support for OTA [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 commented on code in PR #13951: URL: https://github.com/apache/nuttx/pull/13951#discussion_r1794456953 ## boards/risc-v/esp32c6/common/src/esp_board_spiflash.c: ## @@ -58,18 +58,102 @@ * Private Types **

Re: [PR] build ci:add MSVC windows native ci workflow [nuttx]

2024-10-09 Thread via GitHub
xuxin930 commented on PR #13894: URL: https://github.com/apache/nuttx/pull/13894#issuecomment-2403727351 hi @lupyuen how can i integrate labels with this MSVC ci, for example, if we only build sim/windows here, should the changes only affect the sim change? -- This is an automated message

Re: [PR] arch/arm64: Fix clock drift from tick timer [nuttx]

2024-10-09 Thread via GitHub
Fix-Point commented on PR #13984: URL: https://github.com/apache/nuttx/pull/13984#issuecomment-2403727329 I am using this test case to reproduce the bug. ```c #include #include #include #define read_sysreg(reg) \ ({

[PR] sched/wdog: Fix code style [nuttx]

2024-10-09 Thread via GitHub
Fix-Point opened a new pull request, #14013: URL: https://github.com/apache/nuttx/pull/14013 ## Summary This commit fixed code style issue left by the previous patch. ## Impact None. ## Testing None. -- This is an automated message from the Apache Git Service. To res

Re: [PR] arch/libc: Integrate vfork into fork, and vfork directly call up_fork [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #13967: URL: https://github.com/apache/nuttx/pull/13967 -- 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: sched: remove param in nxsched_remove_readytorun

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 67c9a7aabd sched: remove param in nxsched_remo

Re: [PR] sched: remove param in nxsched_remove_readytorun [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #13995: URL: https://github.com/apache/nuttx/pull/13995 -- 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: arch/libc: Integrate vfork into fork, and vfork directly call up_fork

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 2f26323388 arch/libc: Integrate vfork into for

Re: [PR] nuttx/drivers: add ept_release_cb for destroy server resource [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14009: URL: https://github.com/apache/nuttx/pull/14009 -- 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: CI: Split the RISC-V Build Jobs into smaller jobs

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 80d03cb296 CI: Split the RISC-V Build Jobs int

(nuttx) branch master updated: nuttx/drivers: add ept_release_cb for destroy server resource

2024-10-09 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 37a30023f5 nuttx/drivers: add ept_release_cb f

Re: [PR] CI: Split the RISC-V Build Jobs into smaller jobs [nuttx]

2024-10-09 Thread via GitHub
xiaoxiang781216 merged PR #14011: URL: https://github.com/apache/nuttx/pull/14011 -- 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-website) branch asf-site updated: Publishing web: 6f43cfb03f4b427e9277e6d3064476fa7ea61660 docs: 110ab50f50c8c7f1dc704dee01400eac7994a46b

2024-10-09 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 8fc79fe49 Publishing web: 6f43cfb

Re: [PR] CI: Split the RISC-V Build Jobs into smaller jobs [nuttx]

2024-10-09 Thread via GitHub
nuttxpr commented on PR #14011: URL: https://github.com/apache/nuttx/pull/14011#issuecomment-2403534585 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR meets the NuttX requirements. **He

[I] [HELP] nimBLE central (companion to examples/nimble) on a nrf52840-dk [nuttx]

2024-10-09 Thread via GitHub
maxikrie opened a new issue, #14012: URL: https://github.com/apache/nuttx/issues/14012 ### Description Hello, I have successfully tested the examples/nimble app on a nrf52840-dk acting as a BLE peripheral. I am connecting to the board with a smartphone and the nrf connect app. I a

[PR] CI: Split the RISC-V Build Jobs into smaller jobs [nuttx]

2024-10-09 Thread via GitHub
lupyuen opened a new pull request, #14011: URL: https://github.com/apache/nuttx/pull/14011 ## Summary To speed up the CI Workflow, this PR splits the CI Build Jobs for RISC-V into smaller jobs. Each job will now complete within 1 hour. __Before the PR:__ There are 2 jobs for RI

[I] [HELP] Making HTTPS calls using MBEDTLS [nuttx]

2024-10-09 Thread via GitHub
engdavidiogo opened a new issue, #14010: URL: https://github.com/apache/nuttx/issues/14010 ### Description I need help understanding what is wrong with this implementation attempt. I am trying to make **HTTPS** calls in my project using **MBEDTLS**, and for that, I am using the

  1   2   3   4   5   >