[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5647: assert.h: Don't use UNUSED macro since it's very easy happen conflict

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5647: URL: https://github.com/apache/incubator-nuttx/pull/5647#issuecomment-1056508269 > > @xiaoxiang781216 I think that we can drop dependency on `nuttx/compiler.h` now. > > I think so before, but assert reference noreturn_function:(.

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5647: assert.h: Don't use UNUSED macro since it's very easy happen conflict

2022-03-02 Thread GitBox
pkarashchenko merged pull request #5647: URL: https://github.com/apache/incubator-nuttx/pull/5647 -- 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 (4a16cf7 -> 0dd844d)

2022-03-02 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 4a16cf7 Fix -Werror=nonnull-compare and -Werror=format-truncation= add 0dd844d assert.h: Don't

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056517606 Does GCC11 for arm release? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5637: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5637: URL: https://github.com/apache/incubator-nuttx/pull/5637#issuecomment-1056522650 I close #5659, it strange that this one is closed too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056570617 > Does GCC11 for arm release? https://developer.arm.com/-/media/Files/downloads/gnu/11.2-2022.02/binrel/gcc-arm-11.2-2022.02-x86_64-arm-none-eabi.tar.xz -

[GitHub] [incubator-nuttx] michi-jung opened a new pull request #5661: Include nuttx/compiler.h from string.h

2022-03-02 Thread GitBox
michi-jung opened a new pull request #5661: URL: https://github.com/apache/incubator-nuttx/pull/5661 NuttX' string.h is using the FAR preprocessor definition, which is defined in nuttx/compiler.h. Thus, it should include it. Signed-off-by: Michael Jung -- This is an automa

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056604711 But I only see 10.3 on download page: https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm/downloads -- This is

[GitHub] [incubator-nuttx] no1wudi opened a new pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi opened a new pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662 ## Summary Reopen #5637 ## 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 U

[GitHub] [incubator-nuttx] no1wudi edited a comment on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi edited a comment on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056604711 But I only see 10.3 on download page: https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm/downloads -- This

[GitHub] [incubator-nuttx] no1wudi edited a comment on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi edited a comment on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056604711 But I only see 10.3 in download page: https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm/downloads -- This

[GitHub] [incubator-nuttx] no1wudi edited a comment on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi edited a comment on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056604711 But I only see 10.3 in download page: https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm/downloads Emm,

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056634500 ![Screenshot from 2022-03-02 10-08-42](https://user-images.githubusercontent.com/17704713/156330535-753893cb-d120-413f-9d90-893a399e0f14.png) The button on

[GitHub] [incubator-nuttx] jerpelea opened a new pull request #5663: LICENSE: document licenses found in libs/libc

2022-03-02 Thread GitBox
jerpelea opened a new pull request #5663: URL: https://github.com/apache/incubator-nuttx/pull/5663 ## Summary Document licenses found in libs/libc ## Impact LICENSE ## Testing NONE -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1056660366 Should we add softfp flag in case is GCC version is 9 or lower? What do you think? Also with this PR we drop some parts related to "old" toolchains. I think we ne

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1056662309 Also neon flag is dropped. Is this expected? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817500804 ## File path: boards/arm/lpc43xx/bambino-200e/README.txt ## @@ -111,18 +111,6 @@ ports. CONFIG_ARCH_FPU=y CONFIG_ARMV7M_LAZYFPU=

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1056669540 > Also neon flag is dropped. Is this expected? It's due to the incomplete SoC feature setting since there is a option `CONFIG_ARM_HAVE_NEON`, I'll submit new comm

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817503006 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -34,7 +34,6 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) ARCHOPTIMIZATION += $(MA

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817503319 ## File path: boards/arm/lpc43xx/bambino-200e/README.txt ## @@ -111,18 +111,6 @@ ports. CONFIG_ARCH_FPU=y CONFIG_ARMV7M_LAZYFPU=y -C

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817504494 ## File path: boards/arm/imx6/sabre-6quad/scripts/Make.defs ## @@ -34,7 +34,6 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) ARCHOPTIMIZATION += $(MAXOPTIM

[incubator-nuttx] branch master updated (0dd844d -> 4deaa6d)

2022-03-02 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 0dd844d assert.h: Don't use UNUSED macro since it's very easy happen conflict add 4deaa6d sys

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5660: syscal: Remove .context form .gitignore

2022-03-02 Thread GitBox
pkarashchenko merged pull request #5660: URL: https://github.com/apache/incubator-nuttx/pull/5660 -- 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] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056720815 @xiaoxiang781216 seems that after toolchain update CI is not passing anymore -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [incubator-nuttx] pkarashchenko edited a comment on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko edited a comment on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056720815 @xiaoxiang781216 seems that after toolchain update CI is not passing anymore ``` mmcsd/mmcsd_spi.c:223:28: note: while referencing 'g_mmcsdslot'

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056740504 It seems code check is more strict in this GCC, in this error: ``` /* A slot structure allocated for each configured slot */ static struct mmcsd_slot_s g_mmc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5661: Include nuttx/compiler.h from string.h

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5661: URL: https://github.com/apache/incubator-nuttx/pull/5661#issuecomment-1056748898 let's ignore the false alarm. -- 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] xiaoxiang781216 merged pull request #5661: Include nuttx/compiler.h from string.h

2022-03-02 Thread GitBox
xiaoxiang781216 merged pull request #5661: URL: https://github.com/apache/incubator-nuttx/pull/5661 -- 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 #5663: LICENSE: document licenses found in libs/libc

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5663: URL: https://github.com/apache/incubator-nuttx/pull/5663#issuecomment-1056749316 let's ignore the false alarm -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5663: LICENSE: document licenses found in libs/libc

2022-03-02 Thread GitBox
xiaoxiang781216 merged pull request #5663: URL: https://github.com/apache/incubator-nuttx/pull/5663 -- 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] no1wudi opened a new pull request #5664: nuttx/boards/arm/tiva/ekk-lm3s9b96: Disable unused mmcsd option

2022-03-02 Thread GitBox
no1wudi opened a new pull request #5664: URL: https://github.com/apache/incubator-nuttx/pull/5664 ## Summary Remove them to pass CI since they are not really used in board level logic. ## Impact None ## Testing CI -- This is an automated message from the Apache Git Serv

[incubator-nuttx] branch master updated (4deaa6d -> 9114f84)

2022-03-02 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 4deaa6d syscal: Remove .context form .gitignore add 9114f84 Include nuttx/compiler.h from string.h

[incubator-nuttx] branch master updated (9114f84 -> 6a6b0aa)

2022-03-02 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 9114f84 Include nuttx/compiler.h from string.h add c58ac94 libs: libc: lib_ubsqrt: migrate license

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
no1wudi commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056753057 @pkarashchenko Please see #5664 -- 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

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5664: nuttx/boards/arm/tiva/ekk-lm3s9b96: Disable unused mmcsd option

2022-03-02 Thread GitBox
no1wudi commented on pull request #5664: URL: https://github.com/apache/incubator-nuttx/pull/5664#issuecomment-1056759863 Commit message updated -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5664: ekk-lm3s9b96: Remove unused mmcsd option

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5664: URL: https://github.com/apache/incubator-nuttx/pull/5664#issuecomment-1056763093 Let's directly merge this simple patch to unblock CI. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5664: ekk-lm3s9b96: Remove unused mmcsd option

2022-03-02 Thread GitBox
xiaoxiang781216 merged pull request #5664: URL: https://github.com/apache/incubator-nuttx/pull/5664 -- 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 (6a6b0aa -> 8caa496)

2022-03-02 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 6a6b0aa LICENSE: add license for libs/libc/wctype add 8caa496 ekk-lm3s9b96: Remove unused mmcsd op

[GitHub] [incubator-nuttx] Ouss4 opened a new pull request #5665: Add ESP32-S3 SPI-Flash support

2022-03-02 Thread GitBox
Ouss4 opened a new pull request #5665: URL: https://github.com/apache/incubator-nuttx/pull/5665 ## Summary Add support for the SPI-Flash for ESP32-S3. ## Impact New feature -- no impact on existing code. ## Testing Provided defconfig. -- This is an automated message from th

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #5641: sim: add loop thread to handle dev loops

2022-03-02 Thread GitBox
masayuki2009 merged pull request #5641: URL: https://github.com/apache/incubator-nuttx/pull/5641 -- 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 (8caa496 -> fe6aa4a)

2022-03-02 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 8caa496 ekk-lm3s9b96: Remove unused mmcsd option add eb1cc02 sched/group: Move files_initlist/lib_s

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5624: joystick/buttons: Fix the event lose between the invocation of poll

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5624: URL: https://github.com/apache/incubator-nuttx/pull/5624#discussion_r81760 ## File path: drivers/input/ajoystick.c ## @@ -722,6 +720,19 @@ static int ajoy_poll(FAR struct file *filep, FAR struct pollfd *fds,

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5650: armv7-a/armv7-r:cache: modify hardcode in cache set/way operation

2022-03-02 Thread GitBox
masayuki2009 commented on pull request #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-1056827391 >Test on qemu sabre-6quad:smp @zhuyanlinzyl Did you test this PR with your dual Cortex-A7 platform? -- This is an automated message from the Ap

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1056831143 > It's due to the incomplete SoC feature setting since there is a option `CONFIG_ARM_HAVE_NEON`, I'll submit new commit to this PR to keep its origin flags.

[GitHub] [incubator-nuttx] juniskane opened a new pull request #5666: arch/arm/src/stm32l4: STM32L4+ might need flash data cache corruption workaround

2022-03-02 Thread GitBox
juniskane opened a new pull request #5666: URL: https://github.com/apache/incubator-nuttx/pull/5666 ## Summary The STM32L4+ has flash data cache and errata documents say this issue exists, so enable workaround option for those chips. The "depends on" condition is also simplified t

[GitHub] [incubator-nuttx] zhuyanlinzyl commented on pull request #5650: armv7-a/armv7-r:cache: modify hardcode in cache set/way operation

2022-03-02 Thread GitBox
zhuyanlinzyl commented on pull request #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-1056833676 Yes, I also have test on dual Cortex-A7 platform @masayuki2009 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] jerpelea opened a new pull request #5667: LICENSES: document licenses for include files

2022-03-02 Thread GitBox
jerpelea opened a new pull request #5667: URL: https://github.com/apache/incubator-nuttx/pull/5667 ## Summary Document licenses found in libs/libc ## Impact LICENSE ## Testing NONE -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
gustavonihei opened a new pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668 ## Summary This PR intends to add support for Tickless operation of the NuttX kernel using the Systimer component. ## Impact Extended OS support for `esp32s3-devkit` platform.

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668#discussion_r817661244 ## File path: arch/xtensa/src/esp32s3/esp32s3_tickless.c ## @@ -0,0 +1,484 @@ +/

[GitHub] [incubator-nuttx-apps] gustavonihei commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056897731 > @nandojve why to move mcuboot to examples? I think boot/ is a better place to keep it, what do you think @gustavonihei ? Actually there are many "examples" i

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
gustavonihei commented on a change in pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668#discussion_r817665684 ## File path: arch/xtensa/src/esp32s3/esp32s3_tickless.c ## @@ -0,0 +1,484 @@ +/*

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668#discussion_r817666273 ## File path: arch/xtensa/src/esp32s3/esp32s3_tickless.c ## @@ -0,0 +1,484 @@ +/

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
gustavonihei commented on a change in pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668#discussion_r817671140 ## File path: arch/xtensa/src/esp32s3/esp32s3_tickless.c ## @@ -0,0 +1,484 @@ +/*

[GitHub] [incubator-nuttx] gustavonihei commented on a change in pull request #5668: xtensa/esp32s3: Add support for Tickless kernel using Systimer

2022-03-02 Thread GitBox
gustavonihei commented on a change in pull request #5668: URL: https://github.com/apache/incubator-nuttx/pull/5668#discussion_r817671140 ## File path: arch/xtensa/src/esp32s3/esp32s3_tickless.c ## @@ -0,0 +1,484 @@ +/*

[GitHub] [incubator-nuttx-apps] nandojve commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
nandojve commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056917598 Hi Folks, This change of feelings is that create bad projects. I invested a lot of time to put things in the correct way and, after weeks, now, it is about

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5624: joystick/buttons: Fix the event lose between the invocation of poll

2022-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5624: URL: https://github.com/apache/incubator-nuttx/pull/5624#discussion_r817692400 ## File path: drivers/input/ajoystick.c ## @@ -722,6 +720,19 @@ static int ajoy_poll(FAR struct file *filep, FAR struct pollfd *fds,

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5624: joystick/buttons: Fix the event lose between the invocation of poll

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5624: URL: https://github.com/apache/incubator-nuttx/pull/5624#discussion_r817695866 ## File path: drivers/input/ajoystick.c ## @@ -722,6 +720,19 @@ static int ajoy_poll(FAR struct file *filep, FAR struct pollfd *fds,

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817698716 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817702007 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817703636 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2 = $(NX

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817704650 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2 = $(NX

[GitHub] [incubator-nuttx-apps] gustavonihei commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056941297 > Hi Folks, > > This change of feelings is that create bad projects. I invested a lot of time to put things in the correct way and, after weeks, now, it

[GitHub] [incubator-nuttx-apps] gustavonihei edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056941297 > Hi Folks, > > This change of feelings is that create bad projects. I invested a lot of time to put things in the correct way and, after weeks, n

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817699643 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2

[GitHub] [incubator-nuttx] no1wudi opened a new pull request #5669: boards: Remove -nostartfiles -nodefaultlibs from phy62xx

2022-03-02 Thread GitBox
no1wudi opened a new pull request #5669: URL: https://github.com/apache/incubator-nuttx/pull/5669 ## Summary -nostartfiles -nodefaultlibs can't be recognized by gcc 10.3, and already removed from other socs. ## Impact phy62xx ## Testing CI -- This is an automated messa

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817702007 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2

[GitHub] [incubator-nuttx-apps] gustavonihei edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056941297 > Hi Folks, > > This change of feelings is that create bad projects. I invested a lot of time to put things in the correct way and, after weeks, n

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5669: boards: Remove -nostartfiles -nodefaultlibs from phy62xx

2022-03-02 Thread GitBox
no1wudi commented on pull request #5669: URL: https://github.com/apache/incubator-nuttx/pull/5669#issuecomment-1056952488 @pkarashchenko Please see this PR, we should wait for CI pass to ensure there don't have other break. -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r817707282 ## File path: boards/arm/phy62xx/phy6222/scripts/Make.defs ## @@ -53,9 +53,6 @@ NXFLATLDFLAGS1 = -r -Wl,-d -Wl,-warn-common NXFLATLDFLAGS2

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5667: LICENSES: document licenses for include files

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5667: URL: https://github.com/apache/incubator-nuttx/pull/5667#issuecomment-1056955489 Let's ignore the false ci alarm by the new toolchain: https://github.com/apache/incubator-nuttx/pull/5659 -- This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056956073 > ![Screenshot from 2022-03-02 10-08-42](https://user-images.githubusercontent.com/17704713/156330535-753893cb-d120-413f-9d90-893a399e0f14.png) > > The bu

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5650: armv7-a/armv7-r:cache: modify hardcode in cache set/way operation

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-1056957211 @zhuyanlinzyl please rebase the latest mainline which fix the build break generated by other PR. -- This is an automated message from the Apache Git Service.

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5666: arch/arm/src/stm32l4: STM32L4+ might need flash data cache corruption workaround

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5666: URL: https://github.com/apache/incubator-nuttx/pull/5666#issuecomment-1056959171 @juniskane please rebase to the latest mainline to fix the broken ci. -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5669: boards: Remove -nostartfiles -nodefaultlibs from phy62xx

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5669: URL: https://github.com/apache/incubator-nuttx/pull/5669#issuecomment-1056952423 Let's merge this simple patch first to unblock the broken ci. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5669: boards: Remove -nostartfiles -nodefaultlibs from phy62xx

2022-03-02 Thread GitBox
xiaoxiang781216 merged pull request #5669: URL: https://github.com/apache/incubator-nuttx/pull/5669 -- 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 (fe6aa4a -> 31da1fa)

2022-03-02 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 fe6aa4a sim: add loop thread to handle dev loops add 31da1fa boards: Remove -nostartfiles -nodefau

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5667: LICENSES: document licenses for include files

2022-03-02 Thread GitBox
xiaoxiang781216 merged pull request #5667: URL: https://github.com/apache/incubator-nuttx/pull/5667 -- 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] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056960172 currently I'm downloading the toolchain. It was released very recently so need to make some tests, but since ARM Developer marked other versions as `discontinued`

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5650: armv7-a/armv7-r:cache: modify hardcode in cache set/way operation

2022-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#discussion_r817716029 ## File path: arch/arm/src/armv7-a/cp15_clean_dcache_all.S ## @@ -91,19 +91,30 @@ cp15_clean_dcache_all: + push{r4,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5650: armv7-a/armv7-r:cache: modify hardcode in cache set/way operation

2022-03-02 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#discussion_r817716732 ## File path: arch/arm/src/armv7-a/cp15_clean_dcache_all.S ## @@ -91,19 +91,30 @@ cp15_clean_dcache_all: + push{r4,

[GitHub] [incubator-nuttx-apps] otavio commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
otavio commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1056984566 @gustavonihei I think @nandojve is a valid concern. However, this cannot be based on personal feelings but on a general project guideline that must be documented som

[incubator-nuttx] branch master updated (31da1fa -> d61476f)

2022-03-02 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 31da1fa boards: Remove -nostartfiles -nodefaultlibs from phy62xx add 48f6149 LICENSE: add include/

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1056999840 The first run is not too optimistic: ``` stdlib/lib_strtold.c: In function 'strtold': stdlib/lib_strtold.c:137:7: internal compiler error: Illegal instruct

[GitHub] [incubator-nuttx-apps] gustavonihei commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057002869 > @gustavonihei I think @nandojve is a valid concern. However, this cannot be based on personal feelings but on a general project guideline that must be docume

[GitHub] [incubator-nuttx-apps] gustavonihei edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057002869 > @gustavonihei I think @nandojve is a valid concern. However, this cannot be based on personal feelings but on a general project guideline that must be

[GitHub] [incubator-nuttx-apps] gustavonihei edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057002869 -- 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 speci

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
no1wudi commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1057009180 > > It's due to the incomplete SoC feature setting since there is a option `CONFIG_ARM_HAVE_NEON`, I'll submit new commit to this PR to keep its origin flags. >

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1057010384 Another error with LIBM=y ``` CC: math/lib_expf.c math/lib_expf.c:48:3: internal compiler error: Illegal instruction 48 | 1.0 / 120.0,

[GitHub] [incubator-nuttx-apps] gustavonihei edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
gustavonihei edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057002869 > @gustavonihei I think @nandojve is a valid concern. However, this cannot be based on personal feelings but on a general project guideline that must be

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5662: arch/arm: Move ARCHCPUFLAGS to Toolchain.defs

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#issuecomment-1057019045 > > > It's due to the incomplete SoC feature setting since there is a option `CONFIG_ARM_HAVE_NEON`, I'll submit new commit to this PR to keep its origin flags.

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057018989 I am fine with either place. I personally follow this order: 1. If this program is general enough to be used in many case without the code modificatio

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
xiaoxiang781216 edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057018989 -- 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 sp

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 edited a comment on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
xiaoxiang781216 edited a comment on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057018989 I personally follow this order: 1. If this program is general enough to be used in many case without the code modification, it's better to move

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1057023756 Look like it's the compiler bug? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5659: ci: Update gcc-arm-none-eabi to 10.3-2021.10

2022-03-02 Thread GitBox
pkarashchenko commented on pull request #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1057026796 > Look like it's the compiler bug? Yes. I mean that seems like toolchain was built in the wrong way. -- This is an automated message from the Apache Git S

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

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #5476: URL: https://github.com/apache/incubator-nuttx/pull/5476#issuecomment-1057033297 > > so you want I to define all possible built-in function one by one? > > ``` > > 1. The potential number is huge > > > > 2. Where the official list

[GitHub] [incubator-nuttx-apps] otavio commented on pull request #929: MCUboot move apps to examples

2022-03-02 Thread GitBox
otavio commented on pull request #929: URL: https://github.com/apache/incubator-nuttx-apps/pull/929#issuecomment-1057048343 @gustavonihei, as O.S. Systems, we have been sponsoring the work on NuttX in this area, and this lost time and energy doesn't make sense. Furthermore, it causes cont

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request #1038: system/adb: Replace NuttX special shell service with microADB builtin one

2022-03-02 Thread GitBox
xiaoxiang781216 opened a new pull request #1038: URL: https://github.com/apache/incubator-nuttx-apps/pull/1038 ## Summary since the new libuv porting support uv_proccess_t now ## Impact Unify the shell implementation ## Testing sim:adb(https://github.com/apache/incubat

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1038: system/adb: Replace NuttX special shell service with microADB builtin one

2022-03-02 Thread GitBox
xiaoxiang781216 commented on pull request #1038: URL: https://github.com/apache/incubator-nuttx-apps/pull/1038#issuecomment-1057055997 @spiriou could you take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [incubator-nuttx] mogenson opened a new pull request #5670: imxrt: fix userled config in appinit

2022-03-02 Thread GitBox
mogenson opened a new pull request #5670: URL: https://github.com/apache/incubator-nuttx/pull/5670 ## Summary The imxrt1060, imxrt1064, and teensy-4.x board configs would fail to build when `CONFIG_USERLED_LOWER` was enabled, due to a missing header in imxrt_appinit.c and an uns

  1   2   >