[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1000: Fix error: 'strncpy' specified bound 16 equals destination size

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #1000: URL: https://github.com/apache/incubator-nuttx-apps/pull/1000#issuecomment-1037923833 Found by PR https://github.com/apache/incubator-nuttx/pull/5476 -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470#issuecomment-1037924718 Ok, Do you have plan to run NuttX on the secure side too? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470 -- 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

[incubator-nuttx] branch master updated (64130b4 -> 70dae3b)

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 64130b4 risc-v: Use _ebss instead of _default_stack_limit as idle stack base new 5c1c939 stm32u5:

[incubator-nuttx] 03/03: Add arm_tcbinfo.c to stm32u5's common C sources

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 70dae3bb3b5978c5bf49f5d2e9c05ee0fbe53425 Author: Michael Jung AuthorDate: Sat Feb 12 10:21:30 2022 +0100

[incubator-nuttx] 02/03: b-u585i-iot02a: Board Support

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 6086f42bfb0dfe946ddd3393fd1c76fb7f73468a Author: Michael Jung AuthorDate: Sat Feb 12 08:23:02 2022 +0100

[GitHub] [incubator-nuttx-apps] zouboan opened a new pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
zouboan opened a new pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001 ## Summary Add support park transform angle from sensorless observer, support both nolinear fluxlink observer and sliding mode observer ## Impact The former foc control lib in ind

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request #1002: tools/mksymtab.sh: Disable the mismatch warning of builtin declaration

2022-02-13 Thread GitBox
xiaoxiang781216 opened a new pull request #1002: URL: https://github.com/apache/incubator-nuttx-apps/pull/1002 ## Summary ## Impact ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5476: boards: Remove -fno-builtin

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1037967270 depends on https://github.com/apache/incubator-nuttx-apps/pull/1000 and https://github.com/apache/incubator-nuttx-apps/pull/1002. -- This is an automated mess

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
raiden00pl opened a new pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480 ## Summary net/can/can_setsockopt.c: fix assertion, value can be NULL ## Impact ## Testing canutils/cansend with CAN FD frames -- This is an automated message from

[GitHub] [incubator-nuttx] michi-jung commented on pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
michi-jung commented on pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470#issuecomment-1037970517 No, I do not plan to run NuttX on the secure side. The ARM Platform Security Architecture recommends to run the RTOS in the Non-secure state and run some less compl

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5481: Stm32 fdcan

2022-02-13 Thread GitBox
raiden00pl opened a new pull request #5481: URL: https://github.com/apache/incubator-nuttx/pull/5481 ## Summary - stm32/fdcan: use array indexes when accessing RX/TX FIFO - stm32/fdcan: add CAN FD frames support ## Impact ## Testing nucleo-g431rb and b-g431b-esc1 i

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5482: arch: Replace nx_vsyslog with vsyslog

2022-02-13 Thread GitBox
xiaoxiang781216 opened a new pull request #5482: URL: https://github.com/apache/incubator-nuttx/pull/5482 ## Summary Minor change ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#issuecomment-1038004024 It's great to see that someone starts working with FOC and NuttX :) I had done some of this earlier but I didn't find the motivation to make it ready to upstream.

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480#discussion_r805319740 ## File path: net/can/can_setsockopt.c ## @@ -76,7 +76,7 @@ int can_setsockopt(FAR struct socket *psock, int option, int ret = OK; i

[GitHub] [incubator-nuttx] raiden00pl commented on a change in pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
raiden00pl commented on a change in pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#discussion_r805320061 ## File path: libs/libdsp/lib_observer.c ## @@ -495,6 +504,190 @@ void motor_sobserver_div(FAR struct motor_observer_f32_s *o, so->angle_pr

[GitHub] [incubator-nuttx-apps] raiden00pl commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038055936 It's better to separate all observers and place them in different modules. Eg. one module for vel PLL observer, one for vel DIV observer etc. More important

[GitHub] [incubator-nuttx] raiden00pl commented on a change in pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
raiden00pl commented on a change in pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480#discussion_r805324601 ## File path: net/can/can_setsockopt.c ## @@ -76,7 +76,7 @@ int can_setsockopt(FAR struct socket *psock, int option, int ret = OK; int co

[GitHub] [incubator-nuttx] raiden00pl opened a new pull request #5483: libdsp: improvements for lib_observer.c

2022-02-13 Thread GitBox
raiden00pl opened a new pull request #5483: URL: https://github.com/apache/incubator-nuttx/pull/5483 ## Summary The main motivation of these changes is to improve modularity and enable various combinations of the motor controller logic. - libdsp/lib_observer.c: separate angle obse

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#issuecomment-1038120767 @zouboan you should look at https://github.com/apache/incubator-nuttx/pull/5483 which solve various problems that I found with the current observer interface. --

[GitHub] [incubator-nuttx-apps] raiden00pl commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038123804 https://github.com/apache/incubator-nuttx/pull/5483 This PR separate observers logic in libdsp. -- This is an automated message from the Apache Git Service

[GitHub] [incubator-nuttx-apps] raiden00pl commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038132031 Below you can find how I previously organized velocity observers into modules. I can't find my code with angle observers now, but it should look similar (use i

[GitHub] [incubator-nuttx] no1wudi opened a new pull request #5484: risc-v: Implement common up_cpu_idlestack

2022-02-13 Thread GitBox
no1wudi opened a new pull request #5484: URL: https://github.com/apache/incubator-nuttx/pull/5484 ## Summary * risc-v: Implement common up_cpu_idlestack * Add -mstrictalign to arch level flags since it don't support unaligned memory access * Use common cpu idle stack implementation

[GitHub] [incubator-nuttx] zouboan commented on pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
zouboan commented on pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#issuecomment-1038184019 > It's great to see that someone starts working with FOC and NuttX :) > I had done some of this earlier but I didn't find the motivation to make it ready to upstream.

[GitHub] [incubator-nuttx] zouboan commented on a change in pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
zouboan commented on a change in pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#discussion_r805349002 ## File path: libs/libdsp/lib_observer.c ## @@ -495,6 +504,190 @@ void motor_sobserver_div(FAR struct motor_observer_f32_s *o, so->angle_prev

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480 -- 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

[incubator-nuttx] branch master updated: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 5b678ab net/can/can_setsockopt.c: f

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5484: risc-v: Implement common up_cpu_idlestack

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5484: URL: https://github.com/apache/incubator-nuttx/pull/5484#discussion_r805349514 ## File path: arch/risc-v/src/common/riscv_cpuidlestack.c ## @@ -0,0 +1,177 @@ +/*

[GitHub] [incubator-nuttx-apps] zouboan commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
zouboan commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038204258 > [apache/incubator-nuttx#5483](https://github.com/apache/incubator-nuttx/pull/5483) > This PR separate observers logic in libdsp. thanks, let me see!

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 opened a new pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485 ## Summary - I noticed that the following error happened when loading the init 'elf_symvalue: SHN_COMMON: Re-compile with -fno-common' - This commit fixes this issue #

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5452: sim: detect clang native compiler on MacOS.

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5452: URL: https://github.com/apache/incubator-nuttx/pull/5452#discussion_r805433938 ## File path: arch/sim/src/Makefile ## @@ -285,11 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libboard$(LIBEXT

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #5474: libc: posix_openpty should use the absolute path("dev/ptmx")

2022-02-13 Thread GitBox
masayuki2009 merged pull request #5474: URL: https://github.com/apache/incubator-nuttx/pull/5474 -- 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

[incubator-nuttx] branch master updated: libc: posix_openpty should use the absolute path("dev/ptmx")

2022-02-13 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 34baf47 libc: posix_openpty should u

[incubator-nuttx-website] branch asf-site updated: Publishing web: 1b39f27b512cbb1fc294b70e0e70a67d0092b98d docs: 34baf47307d97835f2c3416b6b8a2830bc210d03

2022-02-13 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 73bfef8 Publishing web

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5382: Run sim:ostest on the CI

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5382: URL: https://github.com/apache/incubator-nuttx/pull/5382#discussion_r805455108 ## File path: tools/ci/testlist/sim-02.dat ## @@ -1,5 +1,8 @@ /sim/*/*/*/[o-z]* +# heavy test, moved to a separate set +-sim:ostest Review commen

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5382: Run sim:ostest on the CI

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5382: URL: https://github.com/apache/incubator-nuttx/pull/5382#discussion_r805455482 ## File path: tools/testbuild.sh ## @@ -276,6 +281,18 @@ function build { return $fail } +function run { + if [ ${RUN} -ne 0 ]; then +run_

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#issuecomment-1038004024 -- 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.

[GitHub] [incubator-nuttx] zouboan closed pull request #5472: libdsp:add the calculate of duty now

2022-02-13 Thread GitBox
zouboan closed pull request #5472: URL: https://github.com/apache/incubator-nuttx/pull/5472 -- 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: commit

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5484: risc-v: Implement common up_cpu_idlestack

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5484: URL: https://github.com/apache/incubator-nuttx/pull/5484#discussion_r805349514 ## File path: arch/risc-v/src/common/riscv_cpuidlestack.c ## @@ -0,0 +1,177 @@ +/*

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480#discussion_r805319740 ## File path: net/can/can_setsockopt.c ## @@ -76,7 +76,7 @@ int can_setsockopt(FAR struct socket *psock, int option, int ret = OK; i

[GitHub] [incubator-nuttx] ptka commented on a change in pull request #5401: sim: initial support sim:nsh on MacOS M1 based hosts.

2022-02-13 Thread GitBox
ptka commented on a change in pull request #5401: URL: https://github.com/apache/incubator-nuttx/pull/5401#discussion_r805203870 ## File path: arch/sim/include/setjmp.h ## @@ -111,7 +145,8 @@ #ifndef __ASSEMBLY__ -#if defined(CONFIG_HOST_X86_64) && !defined(CONFIG_SIM_M32)

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #5472: libdsp:add the calculate of duty now

2022-02-13 Thread GitBox
raiden00pl commented on pull request #5472: URL: https://github.com/apache/incubator-nuttx/pull/5472#issuecomment-1037372954 There is nothing wrong with park_transform in other places. That's why it's available in dsp.h/dspb16.h. `v_dq_mod` is still application specific data which can be

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5476: boards: Remove -fno-builtin

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1037967270 depends on https://github.com/apache/incubator-nuttx-apps/pull/1000 and https://github.com/apache/incubator-nuttx-apps/pull/1002. -- This is an automated mess

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480 -- 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

[GitHub] [incubator-nuttx] zouboan commented on pull request #5472: libdsp:add the calculate of duty now

2022-02-13 Thread GitBox
zouboan commented on pull request #5472: URL: https://github.com/apache/incubator-nuttx/pull/5472#issuecomment-1037268034 -- 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

[GitHub] [incubator-nuttx] michi-jung edited a comment on pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
michi-jung edited a comment on pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470#issuecomment-1037873160 -- 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 co

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5401: sim: initial support sim:nsh on MacOS M1 based hosts.

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5401: URL: https://github.com/apache/incubator-nuttx/pull/5401#issuecomment-1037471467 @ptka it's better that you could fix the stack alignment issue first. -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5459: mm/mm_heap: add mm_foreach to run handler for every memory node

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5459: URL: https://github.com/apache/incubator-nuttx/pull/5459 -- 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

[GitHub] [incubator-nuttx] zouboan commented on pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
zouboan commented on pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#issuecomment-1038184019 > It's great to see that someone starts working with FOC and NuttX :) > I had done some of this earlier but I didn't find the motivation to make it ready to upstream.

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #5468: arch: Add xxx_tcbinfo.c to SoC level Make.defs

2022-02-13 Thread GitBox
masayuki2009 merged pull request #5468: URL: https://github.com/apache/incubator-nuttx/pull/5468 -- 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

[GitHub] [incubator-nuttx-apps] zouboan commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
zouboan commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038204258 > [apache/incubator-nuttx#5483](https://github.com/apache/incubator-nuttx/pull/5483) > This PR separate observers logic in libdsp. thanks, let me see!

[GitHub] [incubator-nuttx-apps] raiden00pl commented on pull request #1001: industry/foc: add support park transform angle from sensorless observer

2022-02-13 Thread GitBox
raiden00pl commented on pull request #1001: URL: https://github.com/apache/incubator-nuttx-apps/pull/1001#issuecomment-1038055936 -- 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

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5382: Run sim:ostest on the CI

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5382: URL: https://github.com/apache/incubator-nuttx/pull/5382#discussion_r805455108 ## File path: tools/ci/testlist/sim-02.dat ## @@ -1,5 +1,8 @@ /sim/*/*/*/[o-z]* +# heavy test, moved to a separate set +-sim:ostest Review commen

[GitHub] [incubator-nuttx] zouboan commented on a change in pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
zouboan commented on a change in pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#discussion_r805349002 ## File path: libs/libdsp/lib_observer.c ## @@ -495,6 +504,190 @@ void motor_sobserver_div(FAR struct motor_observer_f32_s *o, so->angle_prev

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5401: sim: initial support sim:nsh on MacOS M1 based hosts.

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5401: URL: https://github.com/apache/incubator-nuttx/pull/5401#discussion_r805203341 ## File path: arch/sim/include/setjmp.h ## @@ -111,7 +145,8 @@ #ifndef __ASSEMBLY__ -#if defined(CONFIG_HOST_X86_64) && !defined(CONF

[GitHub] [incubator-nuttx] raiden00pl commented on a change in pull request #5480: net/can/can_setsockopt.c: fix assertion, value can be NULL

2022-02-13 Thread GitBox
raiden00pl commented on a change in pull request #5480: URL: https://github.com/apache/incubator-nuttx/pull/5480#discussion_r805324601 ## File path: net/can/can_setsockopt.c ## @@ -76,7 +76,7 @@ int can_setsockopt(FAR struct socket *psock, int option, int ret = OK; int co

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5478: risc-v: Use _ebss instead of _default_stack_limit as idle stack base

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5478: URL: https://github.com/apache/incubator-nuttx/pull/5478 -- 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

[GitHub] [incubator-nuttx] michi-jung commented on pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
michi-jung commented on pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470#issuecomment-1037873160 -- 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.

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470#issuecomment-1037475506 -- 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 comm

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5469: net/tcp: support for FIN+ACK case in tcp send event handlers

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5469: URL: https://github.com/apache/incubator-nuttx/pull/5469 -- 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

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1000: Fix error: 'strncpy' specified bound 16 equals destination size

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #1000: URL: https://github.com/apache/incubator-nuttx-apps/pull/1000#issuecomment-1037923833 Found by PR https://github.com/apache/incubator-nuttx/pull/5476 -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] raiden00pl commented on a change in pull request #5479: lib_observer: add nolinear fluxlink observer

2022-02-13 Thread GitBox
raiden00pl commented on a change in pull request #5479: URL: https://github.com/apache/incubator-nuttx/pull/5479#discussion_r805320061 ## File path: libs/libdsp/lib_observer.c ## @@ -495,6 +504,190 @@ void motor_sobserver_div(FAR struct motor_observer_f32_s *o, so->angle_pr

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5471: SAMv7: Added support for PWM driver

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5471: URL: https://github.com/apache/incubator-nuttx/pull/5471 -- 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

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5452: sim: detect clang native compiler on MacOS.

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5452: URL: https://github.com/apache/incubator-nuttx/pull/5452#discussion_r805433938 ## File path: arch/sim/src/Makefile ## @@ -285,11 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libboard$(LIBEXT

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5470: STM32U5 Architecture Support

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5470: URL: https://github.com/apache/incubator-nuttx/pull/5470 -- 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

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #5474: libc: posix_openpty should use the absolute path("dev/ptmx")

2022-02-13 Thread GitBox
masayuki2009 merged pull request #5474: URL: https://github.com/apache/incubator-nuttx/pull/5474 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5452: sim: detect clang native compiler on MacOS.

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5452: URL: https://github.com/apache/incubator-nuttx/pull/5452#discussion_r805468103 ## File path: arch/sim/src/Makefile ## @@ -285,11 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805468511 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -45,6 +45,9 @@ ARCHWARNINGS = -Wall -Wstrict-prototypes -Wshadow -Wundef

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5481: stm32/fdcan: add CAN FD frames support

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5481: URL: https://github.com/apache/incubator-nuttx/pull/5481 -- 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

[incubator-nuttx] branch master updated (34baf47 -> cbbe1c6)

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 34baf47 libc: posix_openpty should use the absolute path("dev/ptmx") add dd8408a stm32/fdcan: use

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5452: sim: detect clang native compiler on MacOS.

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5452: URL: https://github.com/apache/incubator-nuttx/pull/5452#discussion_r805470855 ## File path: arch/sim/src/Makefile ## @@ -285,11 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libboard$(LIBEXT

[GitHub] [incubator-nuttx] yamt opened a new pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
yamt opened a new pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486 ## Summary ``` /usr/bin/ld: cannot open linker script file nuttx.ld: No such file or directory clang: error: linker command failed with exit code 1 (use -v to see invocation)

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5452: sim: detect clang native compiler on MacOS.

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5452: URL: https://github.com/apache/incubator-nuttx/pull/5452#discussion_r805476529 ## File path: arch/sim/src/Makefile ## @@ -285,11 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libboard$(LIBEXT

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5487: binfmt: Decouple builtin from binfs file system

2022-02-13 Thread GitBox
xiaoxiang781216 opened a new pull request #5487: URL: https://github.com/apache/incubator-nuttx/pull/5487 ## Summary ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805479196 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -45,6 +45,9 @@ ARCHWARNINGS = -Wall -Wstrict-prototypes -Wshadow -Wundef A

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805480356 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS =

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5488: boards/sama5d4-ek: Move -fno-common to CELFFLAGS/CXXELFFLAGS

2022-02-13 Thread GitBox
xiaoxiang781216 opened a new pull request #5488: URL: https://github.com/apache/incubator-nuttx/pull/5488 ## Summary like other's Make.defs ## Impact No ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond to the message, pl

[incubator-nuttx] branch master updated (cbbe1c6 -> b8477f8)

2022-02-13 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from cbbe1c6 net/can/can_recvmsg.c: fix unused variable 'conn' error add 55d8862 risc-v: Implement comm

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5484: risc-v: Implement common up_cpu_idlestack

2022-02-13 Thread GitBox
xiaoxiang781216 merged pull request #5484: URL: https://github.com/apache/incubator-nuttx/pull/5484 -- 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

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805483079 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS = -mc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805483425 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS =

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5399: Add CONFIG_NDEBUG Kconfig to control NDEBUG definition

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5399: URL: https://github.com/apache/incubator-nuttx/pull/5399#issuecomment-1038601386 @davids5 more people prefer the default value is better to align with other debugging related option. How to avoid the PR stuck for a long time? -- This is an

[GitHub] [incubator-nuttx-apps] no1wudi opened a new pull request #1003: ftpd: Remove usage of deprecated macro __NUTTX__

2022-02-13 Thread GitBox
no1wudi opened a new pull request #1003: URL: https://github.com/apache/incubator-nuttx-apps/pull/1003 ## Summary Minor change ## Impact None ## Testing CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805491532 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS = -mc

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805491532 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS = -mc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#discussion_r805492565 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -39,8 +39,8 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) endif ARCHCPUFLAGS =

[GitHub] [incubator-nuttx] xiaoxiang781216 closed pull request #5488: boards/sama5d4-ek: Move -fno-common to CELFFLAGS/CXXELFFLAGS

2022-02-13 Thread GitBox
xiaoxiang781216 closed pull request #5488: URL: https://github.com/apache/incubator-nuttx/pull/5488 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5488: boards/sama5d4-ek: Move -fno-common to CELFFLAGS/CXXELFFLAGS

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5488: URL: https://github.com/apache/incubator-nuttx/pull/5488#issuecomment-1038617381 abandon see here: https://github.com/apache/incubator-nuttx/pull/5485 -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805493478 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805493618 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#issuecomment-1038620919 @yamt please reorganize your pachset and remove the intermediate change. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [incubator-nuttx] yamt commented on pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
yamt commented on pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#issuecomment-1038628351 > @yamt please reorganize your pachset and remove the intermediate change. why? this PR contains only three commits and i feel they are independent. -- This is

[GitHub] [incubator-nuttx] yamt commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
yamt commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805496906 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libboard$(LIBEXT

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#issuecomment-1038660562 Hmm, I don't know why the Build / macOS (macos) failed. ``` Config

[GitHub] [incubator-nuttx] yamt commented on pull request #5476: boards: Remove -fno-builtin

2022-02-13 Thread GitBox
yamt commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1038663994 for simple functions like memset it might make sense. for more complex functions, i'm not sure. isn't it safer to enable them selectively as we do for alloca? -- Th

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5485: boards: sabre-6quad: Add -fno-common to Make.defs

2022-02-13 Thread GitBox
masayuki2009 commented on pull request #5485: URL: https://github.com/apache/incubator-nuttx/pull/5485#issuecomment-1038665127 >Hmm, I don't know why the Build / macOS (macos) failed. My local build without this PR also fails. -- This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805520014 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805520014 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5486: sim: Fix a build with clang on linux

2022-02-13 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5486: URL: https://github.com/apache/incubator-nuttx/pull/5486#discussion_r805520014 ## File path: arch/sim/src/Makefile ## @@ -285,14 +285,14 @@ nuttx-names.dat: nuttx-names.in nuttx$(EXEEXT): libarch$(LIBEXT) board/libbo

  1   2   >