[GitHub] [incubator-nuttx-apps] pkarashchenko commented on a diff in pull request #851: examples/camera: Fix some issues and update license

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #851: URL: https://github.com/apache/incubator-nuttx-apps/pull/851#discussion_r1004065498 ## examples/camera/camera_main.c: ## @@ -525,74 +536,182 @@ int main(int argc, FAR char *argv[]) * order from the captured frame buffer and a new

[GitHub] [incubator-nuttx-apps] pkarashchenko commented on a diff in pull request #1384: examples/camera: Dynamically get the connected device

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #1384: URL: https://github.com/apache/incubator-nuttx-apps/pull/1384#discussion_r1004064480 ## examples/camera/camera_main.c: ## @@ -510,16 +529,40 @@ int main(int argc, FAR char *argv[]) * waits for new VIDIOC_QBUFed frame buffer.

[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7419: Makefile: Remove INCDIROPT

2022-10-24 Thread GitBox
Gary-Hobson commented on code in PR #7419: URL: https://github.com/apache/incubator-nuttx/pull/7419#discussion_r1004038467 ## drivers/motor/foc/Make.defs: ## @@ -30,4 +30,4 @@ endif DEPPATH += --dep-path motor$(DELIM)foc VPATH += :motor$(DELIM)foc -CFLAGS += ${shell $(INCDIR

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7076: Remove duplicate calls and optimize compilation speed

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7076: URL: https://github.com/apache/incubator-nuttx/pull/7076#discussion_r1004034307 ## tools/Config.mk: ## @@ -237,13 +237,13 @@ OBJPATH ?= . # CONFIG_WINDOWS_NATIVE - Defined for a Windows native build ifeq ($(CONFIG_WINDOWS_NATI

[incubator-nuttx-apps] branch master updated: Makefile: Remove INCDIROPT

2022-10-24 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-apps.git The following commit(s) were added to refs/heads/master by this push: new 2c3c2edcb Makefile: Remove INCD

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1385: Makefile: Remove INCDIROPT

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #1385: URL: https://github.com/apache/incubator-nuttx-apps/pull/1385 -- 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-u

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1290019823 The description looks good, @hartmannathan thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290015336 @tito97sp should we merge this patch instead yours(#7397)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-nuttx] hartmannathan commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-10-24 Thread GitBox
hartmannathan commented on PR #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1289974488 I have documented the mitigation entry for the next release's Release Notes here: https://cwiki.apache.org/confluence/display/NUTTX/NuttX+11.1.0#NuttX11.1.0-DefaultSemaphore

[incubator-nuttx] branch master updated (86e3bd9d75 -> 1feb0e759d)

2022-10-24 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 86e3bd9d75 arm/toolchain: update toolchain comment to avoid confusion add 1feb0e759d boards/stm32wb:

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7418: boards/stm32wb: Flipper Zero initial support

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7418: URL: https://github.com/apache/incubator-nuttx/pull/7418 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7418: boards/stm32wb: Flipper Zero initial support

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7418: URL: https://github.com/apache/incubator-nuttx/pull/7418#issuecomment-1289968383 ci failure is fixed by: https://github.com/apache/incubator-nuttx/pull/7422 let's merge this patch directly to save the server loading. -- This is an automated

[incubator-nuttx] branch master updated (9804320cae -> 86e3bd9d75)

2022-10-24 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 9804320cae arm/armv6-m: fix build break if enable syntax unified add 86e3bd9d75 arm/toolchain: update

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7423: arm/toolchain: update toolchain comment to avoid confusion

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7423: URL: https://github.com/apache/incubator-nuttx/pull/7423 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7423: arm/toolchain: update toolchain comment to avoid confusion

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7423: URL: https://github.com/apache/incubator-nuttx/pull/7423#issuecomment-1289965287 ci break is fixed here: https://github.com/apache/incubator-nuttx/pull/7422 let's merge this simple change to save the server loading. -- This is an automated messa

[incubator-nuttx] branch master updated: arm/armv6-m: fix build break if enable syntax unified

2022-10-24 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 9804320cae arm/armv6-m: fix build br

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7422: arm/armv6-m: fix build break if enable syntax unified

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7422: URL: https://github.com/apache/incubator-nuttx/pull/7422 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7415: sched: Pass idle thread envionment variables to the child task through copy

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7415: URL: https://github.com/apache/incubator-nuttx/pull/7415#issuecomment-1289963226 > @xiaoxiang781216 > > Hmm, the `hello` (ELF) application does not work with `spresense:wifi_smp` > > nsh> hello > nsh: hello: command not found > ```

[GitHub] [incubator-nuttx] anchao commented on pull request #7364: Support the stack size and address for posix_spawn and kernel mode

2022-10-24 Thread GitBox
anchao commented on PR #7364: URL: https://github.com/apache/incubator-nuttx/pull/7364#issuecomment-1289933927 @masayuki2009 Could you please verify this PR again? -- 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] anchao opened a new pull request, #7423: arm/toolchain: update toolchain comment to avoid confusion

2022-10-24 Thread GitBox
anchao opened a new pull request, #7423: URL: https://github.com/apache/incubator-nuttx/pull/7423 ## Summary arm/toolchain: update toolchain comment to avoid confusion ``` 1. add 'ARM' prefix to choice menu 2. rename 'Generic Clang toolchain' to 'LLVM Clang toolchain'

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7422: arm/armv6-m: fix build break if enable syntax unified

2022-10-24 Thread GitBox
anchao opened a new pull request, #7422: URL: https://github.com/apache/incubator-nuttx/pull/7422 ## Summary arm/armv6-m: fix build break if enable syntax unified armv6-m/arm_exception.S: Assembler messages: armv6-m/arm_exception.S:171: Error: cannot honor width suffix --

[incubator-nuttx-website] branch asf-site updated: Publishing web: d696c67810809c3f81a4dd87b09d9e940db854b4 docs: 42a0f444eebab3a9c85992e70d8d569c32eb6e9f

2022-10-24 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 b32c868 Publishing web

[GitHub] [incubator-nuttx] acassis commented on issue #7366: how to add microkernelized characteristics on nuttx 8.2

2022-10-24 Thread GitBox
acassis commented on issue #7366: URL: https://github.com/apache/incubator-nuttx/issues/7366#issuecomment-1289635291 Hi @yuening1999 really cool!!! Please send the normal support to this new chip to NuttX mainline, it will help more people to discover about this new chip. I think the micro

[GitHub] [incubator-nuttx] tmedicci commented on pull request #7421: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-24 Thread GitBox
tmedicci commented on PR #7421: URL: https://github.com/apache/incubator-nuttx/pull/7421#issuecomment-1289502232 Closing it to re-test the supported sample rates of the CS4344. The datasheet does not mention support for any sample rate under 32KHz, but it worked for 22,05KHz. -- This is

[GitHub] [incubator-nuttx] tmedicci closed pull request #7421: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-24 Thread GitBox
tmedicci closed pull request #7421: audio/i2s: add i2s_mclkfrequency method to the I2S lower half URL: https://github.com/apache/incubator-nuttx/pull/7421 -- 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 g

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7410: boards: spresense: enable lte modem

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7410: URL: https://github.com/apache/incubator-nuttx/pull/7410 -- 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-unsubs

[incubator-nuttx] branch master updated (09cc29af4d -> 42a0f444ee)

2022-10-24 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 09cc29af4d arch/armv6-m: fix compile error on LLVM clang add 42a0f444ee boards: spresense: enable lte

[GitHub] [incubator-nuttx] acassis commented on pull request #7416: drivers: video: Update some features

2022-10-24 Thread GitBox
acassis commented on PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#issuecomment-1289464160 Please fix the CI reported error: ``` drivers/video/isx019.c:1109:38: error: Operator/assignment must be followed with whitespace ``` -- This is an automated message fr

[GitHub] [incubator-nuttx] tmedicci opened a new pull request, #7421: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-24 Thread GitBox
tmedicci opened a new pull request, #7421: URL: https://github.com/apache/incubator-nuttx/pull/7421 ## Summary This method allows the codec driver to set a specific mclk frequency for the I2S lower half, allowing to change it on run time according to the supported master clock freque

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7417: stm32fh7:Allow the use of the Network Monitor via polling

2022-10-24 Thread GitBox
acassis commented on code in PR #7417: URL: https://github.com/apache/incubator-nuttx/pull/7417#discussion_r1003645678 ## arch/arm/src/stm32h7/Kconfig: ## @@ -5087,6 +5092,16 @@ config STM32H7_PHYINIT provide stm32_phyinitialize(); The STM32 Ethernet driver will

[GitHub] [incubator-nuttx] acassis commented on pull request #7418: boards/stm32wb: Flipper Zero initial support

2022-10-24 Thread GitBox
acassis commented on PR #7418: URL: https://github.com/apache/incubator-nuttx/pull/7418#issuecomment-1289453145 Amazing @onegray !!! Please fix the incorrect path in this file: ``` Error: boards/arm/stm32wb/flipperzero/include/flipperzero-clocking.h:2:1: error: Relative file path does

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7420: Update errno to match asm-generic

2022-10-24 Thread GitBox
pkarashchenko commented on PR #7420: URL: https://github.com/apache/incubator-nuttx/pull/7420#issuecomment-1289401124 What is the motivation for this PR? Why should we align with Linux and not Open/Free BSD for example? I will need some more time to look into these changes. -- This is

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7418: boards/stm32wb: Flipper Zero initial support

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #7418: URL: https://github.com/apache/incubator-nuttx/pull/7418#discussion_r1003590412 ## boards/arm/stm32wb/flipperzero/include/board.h: ## @@ -0,0 +1,115 @@ +/ + *

[GitHub] [incubator-nuttx] nmn35175 opened a new pull request, #7420: Update errno to match asm-generic

2022-10-24 Thread GitBox
nmn35175 opened a new pull request, #7420: URL: https://github.com/apache/incubator-nuttx/pull/7420 ## Summary Update NuttX error codes to match the values in asm-generic/errno-base.h and asm-generic/errno.h that are used in Linux kernel. ## Impact No impact ## Te

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7419: Makefile: Remove INCDIROPT

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7419: URL: https://github.com/apache/incubator-nuttx/pull/7419#discussion_r1003542897 ## drivers/motor/foc/Make.defs: ## @@ -30,4 +30,4 @@ endif DEPPATH += --dep-path motor$(DELIM)foc VPATH += :motor$(DELIM)foc -CFLAGS += ${shell $(IN

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7414: arch/armv6-m: fix compile error on LLVM clang

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7414: URL: https://github.com/apache/incubator-nuttx/pull/7414 -- 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-unsubs

[incubator-nuttx] branch master updated: arch/armv6-m: fix compile error on LLVM clang

2022-10-24 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 09cc29af4d arch/armv6-m: fix compile

[GitHub] [incubator-nuttx] Gary-Hobson opened a new pull request, #7419: Makefile: Remove INCDIROPT

2022-10-24 Thread GitBox
Gary-Hobson opened a new pull request, #7419: URL: https://github.com/apache/incubator-nuttx/pull/7419 This parameter is invalid See git commit # 5eae32577e5d5226e5d3027c169eeb369f83f77d ## Summary ## Impact ## Testing -- This is an automated message fro

[GitHub] [incubator-nuttx-apps] Gary-Hobson opened a new pull request, #1385: Makefile: Remove as used INCDIROPT

2022-10-24 Thread GitBox
Gary-Hobson opened a new pull request, #1385: URL: https://github.com/apache/incubator-nuttx-apps/pull/1385 ## Summary This parameter is not used, it has been removed in this commit See git commit # 5eae32577e5d5226e5d3027c169eeb369f83f77d ## Impact ## Testing

[GitHub] [incubator-nuttx] onegray opened a new pull request, #7418: boards/stm32wb: Flipper Zero initial support

2022-10-24 Thread GitBox
onegray opened a new pull request, #7418: URL: https://github.com/apache/incubator-nuttx/pull/7418 ## Summary This is an initial support of FlipperZero device. The device is developer-friendly - it has public hardware documentation, SWD pins for debugging, serial interface pins. See

[incubator-nuttx] branch master updated: arm/cache: fix build warning on LLVM clang

2022-10-24 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 539f9a862e arm/cache: fix build warn

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7413: arm/cache: fix build warning on LLVM clang

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7413: URL: https://github.com/apache/incubator-nuttx/pull/7413 -- 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-unsubs

[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7076: Remove duplicate calls and optimize compilation speed

2022-10-24 Thread GitBox
Gary-Hobson commented on PR #7076: URL: https://github.com/apache/incubator-nuttx/pull/7076#issuecomment-1289211785 > This looks very promising. @Gary-Hobson any progress on this? Yes,I am perfecting it -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7076: Remove duplicate calls and optimize compilation speed

2022-10-24 Thread GitBox
Gary-Hobson commented on PR #7076: URL: https://github.com/apache/incubator-nuttx/pull/7076#issuecomment-1289208066 This line of code looks ugly, but I didn't think of any proper solution It is the prefix used to get define Do you guys have any better suggestions ``` expor

[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7076: Remove duplicate calls and optimize compilation speed

2022-10-24 Thread GitBox
Gary-Hobson commented on code in PR #7076: URL: https://github.com/apache/incubator-nuttx/pull/7076#discussion_r1003448325 ## drivers/segger/Make.defs: ## @@ -27,11 +27,11 @@ ifeq ($(CONFIG_SEGGER_RTT),y) CFLAGS += -Wno-shadow -Wno-array-bounds ifneq ($(CONFIG_SEGGER_RTT

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #7076: Remove duplicate calls and optimize compilation speed

2022-10-24 Thread GitBox
gustavonihei commented on code in PR #7076: URL: https://github.com/apache/incubator-nuttx/pull/7076#discussion_r1003437939 ## drivers/segger/Make.defs: ## @@ -27,11 +27,11 @@ ifeq ($(CONFIG_SEGGER_RTT),y) CFLAGS += -Wno-shadow -Wno-array-bounds ifneq ($(CONFIG_SEGGER_RT

[incubator-nuttx] branch master updated: boards: cxd56xx: Update imageproc driver

2022-10-24 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 00d1a7db29 boards: cxd56xx: Update i

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7412: boards: cxd56xx: Update imageproc driver

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7412: URL: https://github.com/apache/incubator-nuttx/pull/7412 -- 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-unsubs

[GitHub] [incubator-nuttx-apps] SPRESENSE commented on pull request #1384: examples/camera: Dynamically get the connected device

2022-10-24 Thread GitBox
SPRESENSE commented on PR #1384: URL: https://github.com/apache/incubator-nuttx-apps/pull/1384#issuecomment-1289114527 Please ignore a below error. This is standard usage of `timercmp()` and has already been discussed in https://github.com/apache/incubator-nuttx-apps/pull/851#discussion_r7

[GitHub] [incubator-nuttx-apps] SPRESENSE opened a new pull request, #1384: examples/camera: Dynamically get the connected device

2022-10-24 Thread GitBox
SPRESENSE opened a new pull request, #1384: URL: https://github.com/apache/incubator-nuttx-apps/pull/1384 ## Summary This example can support for both ISX012 and ISX019 image sensors. ## Impact None ## Testing ./tools/configure.sh spresense:example_camera -- This

[GitHub] [incubator-nuttx] jerpelea commented on a diff in pull request #7410: boards: spresense: enable lte modem

2022-10-24 Thread GitBox
jerpelea commented on code in PR #7410: URL: https://github.com/apache/incubator-nuttx/pull/7410#discussion_r1003316912 ## boards/arm/cxd56xx/spresense/include/cxd56_alt1250.h: ## @@ -46,47 +46,57 @@ extern "C" * Public Function Prototypes ***

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003314673 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,97 @@ +/ + * libs/

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7410: boards: spresense: enable lte modem

2022-10-24 Thread GitBox
SPRESENSE commented on code in PR #7410: URL: https://github.com/apache/incubator-nuttx/pull/7410#discussion_r1003311458 ## boards/arm/cxd56xx/spresense/include/cxd56_alt1250.h: ## @@ -46,47 +46,57 @@ extern "C" * Public Function Prototypes **

[GitHub] [incubator-nuttx] davids5 opened a new pull request, #7417: stm32fh7:Allow the use of the Network Monitor via polling

2022-10-24 Thread GitBox
davids5 opened a new pull request, #7417: URL: https://github.com/apache/incubator-nuttx/pull/7417 ## Summary Not all boards have an interrupt line from the phy to the Soc. This commit allows the phy to be polled for link status. This may not work on all MAC/PH

[GitHub] [incubator-nuttx] SPRESENSE opened a new pull request, #7416: drivers: video: Update some features

2022-10-24 Thread GitBox
SPRESENSE opened a new pull request, #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416 ## Summary Update video driver with isx012 and isx019 image sensor - Add missing FAR qualifier - Add get frame interval API - Update API to dump register for debug - isx012: Sup

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003278101 ## libs/libc/obstack/lib_obstack_printf.c: ## @@ -0,0 +1,43 @@ +/ + * libs/libc/obsta

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003277823 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,77 @@ +/ + * libs/libc/obst

[incubator-nuttx] branch master updated (c629fd1b00 -> d6423b5527)

2022-10-24 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 c629fd1b00 arm/phy62xx: fix compile warning ('while' clause does not guard) add d6423b5527 romfs: ex

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7404: romfs: expand file cache by CONFIG_FS_ROMFS_FCACHE_NSECTORS

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7404: URL: https://github.com/apache/incubator-nuttx/pull/7404 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7404: romfs: expand file cache by CONFIG_FS_ROMFS_FCACHE_NSECTORS

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7404: URL: https://github.com/apache/incubator-nuttx/pull/7404#issuecomment-1288984795 let ignore the ci broken, which is fixed in: https://github.com/apache/incubator-nuttx/pull/7409 -- This is an automated message from the Apache Git Service. To respon

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003258629 ## libs/libc/obstack/lib_obstack_finish.c: ## @@ -0,0 +1,57 @@ +/ + * libs/l

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003258369 ## libs/libc/obstack/lib_obstack_printf.c: ## @@ -0,0 +1,43 @@ +/ + * libs/l

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
xiaoxiang781216 commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003258152 ## libs/libc/obstack/lib_obstack_vprintf.c: ## @@ -0,0 +1,77 @@ +/ + * libs/

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#issuecomment-1288965053 Let's me reply in the related comment. -- 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 opened a new pull request, #7415: sched: Pass idle thread envionment variables to the child task through copy

2022-10-24 Thread GitBox
xiaoxiang781216 opened a new pull request, #7415: URL: https://github.com/apache/incubator-nuttx/pull/7415 ## Summary because if we pass predefined environment variables table explicitly, the environment variables created by the board code will be lost. ## Impact restore the beh

[incubator-nuttx] branch master updated (fa0577ebfb -> c629fd1b00)

2022-10-24 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 fa0577ebfb boards/arm/gcc: disable coverage analysis temporarily add c629fd1b00 arm/phy62xx: fix comp

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7411: arm/phy62xx: fix compile warning on gcc-11.3 ('while' clause does not guard)

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7411: URL: https://github.com/apache/incubator-nuttx/pull/7411 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7411: arm/phy62xx: fix compile warning on gcc-11.3 ('while' clause does not guard)

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7411: URL: https://github.com/apache/incubator-nuttx/pull/7411#issuecomment-1288955257 let's merge this patch to unblock 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 abov

[incubator-nuttx] branch master updated (a4df3c0330 -> fa0577ebfb)

2022-10-24 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 a4df3c0330 arch: cxd56xx: Fix deadlock by using GNSS CEP file on SPI-Flash add fa0577ebfb boards/arm

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7409: boards/arm/gcc: disable coverage analysis temporarily

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7409: URL: https://github.com/apache/incubator-nuttx/pull/7409 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7409: boards/arm/gcc: disable coverage analysis temporarily

2022-10-24 Thread GitBox
xiaoxiang781216 commented on PR #7409: URL: https://github.com/apache/incubator-nuttx/pull/7409#issuecomment-1288954583 Let's merge this patch to unblock 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 abov

[GitHub] [incubator-nuttx] Cynerd commented on pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
Cynerd commented on PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#issuecomment-1288944123 > @Cynerd my last comment isn't resolved. The last comment I see is this one https://github.com/apache/incubator-nuttx/pull/7198#discussion_r987280006, and that is resolved

[GitHub] [incubator-nuttx] Cynerd commented on a diff in pull request #7198: libs/libc: add obstack

2022-10-24 Thread GitBox
Cynerd commented on code in PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198#discussion_r1003240966 ## libs/libc/obstack/lib_obstack_grow.c: ## @@ -0,0 +1,93 @@ +/ + * libs/libc/obstack

[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7414: arch/armv6-m: fix compile error on LLVM clang

2022-10-24 Thread GitBox
anchao commented on code in PR #7414: URL: https://github.com/apache/incubator-nuttx/pull/7414#discussion_r1003233917 ## boards/arm/phy62xx/phy6222/src/bringup.c: ## @@ -70,7 +70,7 @@ extern int phyplus_timer_initialize(const char *devpath, int timer); int phy62xx_bringup(vo

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7414: arch/armv6-m: fix compile error on LLVM clang

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #7414: URL: https://github.com/apache/incubator-nuttx/pull/7414#discussion_r1003206207 ## boards/arm/phy62xx/phy6222/src/bringup.c: ## @@ -70,7 +70,7 @@ extern int phyplus_timer_initialize(const char *devpath, int timer); int phy62xx_bri

[incubator-nuttx] branch master updated (07a2fa8d57 -> a4df3c0330)

2022-10-24 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 07a2fa8d57 ci/docker: copy `clang-extdef-mapping` to clang-arm add a4df3c0330 arch: cxd56xx: Fix dead

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7405: arch: cxd56xx: Fix deadlock by using GNSS CEP file on SPI-Flash

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7405: URL: https://github.com/apache/incubator-nuttx/pull/7405 -- 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-unsubs

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7414: arch/armv6-m: fix compile error on LLVM clang

2022-10-24 Thread GitBox
anchao opened a new pull request, #7414: URL: https://github.com/apache/incubator-nuttx/pull/7414 ## Summary arch/armv6-m: fix compile error on LLVM clang ``` armv6-m/arm_exception.S:139:2: error: invalid instruction, any one of the following would fix this: sub r1, #(4

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7413: arm/cache: fix build warning on LLVM clang

2022-10-24 Thread GitBox
anchao opened a new pull request, #7413: URL: https://github.com/apache/incubator-nuttx/pull/7413 ## Summary arm/cache: fix build warning on LLVM clang ``` armv7-m/arm_cache.c:93:24: warning: unused function 'arm_clz' [-Wunused-function] static inline uint32_t arm_clz

[GitHub] [incubator-nuttx] SPRESENSE opened a new pull request, #7412: boards: cxd56xx: Update imageproc driver

2022-10-24 Thread GitBox
SPRESENSE opened a new pull request, #7412: URL: https://github.com/apache/incubator-nuttx/pull/7412 ## Summary Fix raster operation bug in non-scaling case, and change the type of return value to int with error code for some functions. ## Impact Only spresense board ## T

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7410: boards: spresense: enable lte modem

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #7410: URL: https://github.com/apache/incubator-nuttx/pull/7410#discussion_r1003118050 ## boards/arm/cxd56xx/common/src/cxd56_alt1250.c: ## @@ -75,36 +73,34 @@ # error "Select LTE SPI 4 or 5" #endif -#define WAIT_READY_TO_GPIO_INTERRUPT

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7411: arm/phy62xx: fix compile warning on gcc-11.3 ('while' clause does not guard)

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #7411: URL: https://github.com/apache/incubator-nuttx/pull/7411#discussion_r1003114740 ## arch/arm/src/phy62xx/flash.c: ## @@ -43,7 +43,7 @@ #define SPIF_STATUS_WAIT_IDLE(n)\ do

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7411: arm/phy62xx: fix compile warning on gcc-11.3 ('while' clause does not guard)

2022-10-24 Thread GitBox
anchao opened a new pull request, #7411: URL: https://github.com/apache/incubator-nuttx/pull/7411 ## Summary arm/phy62xx: fix compile warning on gcc-11.3 ('while' clause does not guard) chip/flash.c: In function '_spif_read_status_reg_x': chip/flash.c:46:9: warning: this 'w

[GitHub] [incubator-nuttx] anchao commented on pull request #7409: boards/arm/gcc: disable coverage analysis temporarily

2022-10-24 Thread GitBox
anchao commented on PR #7409: URL: https://github.com/apache/incubator-nuttx/pull/7409#issuecomment-1288723427 > Anyway I still think that we are too fast with toolchain upgrade, but I'm more conservative. Do we have a significant gain in optimisation with new compiler? Or what is the rush

[incubator-nuttx] branch master updated (e66bb207aa -> 07a2fa8d57)

2022-10-24 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 e66bb207aa drivers/modem/alt1250: Change return type of bool function add 07a2fa8d57 ci/docker: copy

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7407: ci/docker: copy `clang-extdef-mapping` to clang-arm

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #7407: URL: https://github.com/apache/incubator-nuttx/pull/7407 -- 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-unsubs

[incubator-nuttx-apps] branch master updated: usrsock:fix stack-buffer-overflow issue when running basic_send test case

2022-10-24 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-apps.git The following commit(s) were added to refs/heads/master by this push: new ffab06c6c usrsock:fix stack-buf

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1369: usrsock:fix stack-buffer-overflow issue when running basic_send test case

2022-10-24 Thread GitBox
xiaoxiang781216 merged PR #1369: URL: https://github.com/apache/incubator-nuttx-apps/pull/1369 -- 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-u

[incubator-nuttx] branch master updated (22a29fdc97 -> e66bb207aa)

2022-10-24 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 22a29fdc97 arch: cxd56xx: Fix stall bulk xfer when sending 512 byte data add e66bb207aa drivers/

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7408: drivers/modem/alt1250: Change return type of bool function

2022-10-24 Thread GitBox
pkarashchenko merged PR #7408: URL: https://github.com/apache/incubator-nuttx/pull/7408 -- 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

[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #7404: romfs: expand file cache by CONFIG_FS_ROMFS_FCACHE_NSECTORS

2022-10-24 Thread GitBox
Donny9 commented on code in PR #7404: URL: https://github.com/apache/incubator-nuttx/pull/7404#discussion_r1003024391 ## fs/romfs/Kconfig: ## @@ -20,4 +20,10 @@ config FS_ROMFS_CACHE_NODE is mounted so that we can quick access entry of ROMFS files

[GitHub] [incubator-nuttx] jerpelea opened a new pull request, #7410: boards: spresense: enable lte modem

2022-10-24 Thread GitBox
jerpelea opened a new pull request, #7410: URL: https://github.com/apache/incubator-nuttx/pull/7410 ## Summary Enable and configure ALT1250 LTE modem ## Impact SPRESENSE LTE board ## Testing spresense -- This is an automated message from the Apache Git Service. T

[GitHub] [incubator-nuttx-apps] pkarashchenko merged pull request #1383: wireless: gs2200m: Change usrsock xid from uint64_t to uint32_t

2022-10-24 Thread GitBox
pkarashchenko merged PR #1383: URL: https://github.com/apache/incubator-nuttx-apps/pull/1383 -- 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-uns

[incubator-nuttx-apps] branch master updated: wireless: gs2200m: Change usrsock xid from uint64_t to uint32_t

2022-10-24 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 2717ba4f8 wireless: gs2200m

[GitHub] [incubator-nuttx-apps] pkarashchenko commented on pull request #1383: wireless: gs2200m: Change usrsock xid from uint64_t to uint32_t

2022-10-24 Thread GitBox
pkarashchenko commented on PR #1383: URL: https://github.com/apache/incubator-nuttx-apps/pull/1383#issuecomment-1288514601 Merging this directly since CI is not fully recovered -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[incubator-nuttx] branch master updated (306029e6e4 -> 22a29fdc97)

2022-10-24 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 306029e6e4 lte: Include nuttx/fs/ioctl.h instead of wireless.h add 22a29fdc97 arch: cxd56xx: Fix

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7406: arch: cxd56xx: Fix stall bulk xfer when sending 512 byte data

2022-10-24 Thread GitBox
pkarashchenko merged PR #7406: URL: https://github.com/apache/incubator-nuttx/pull/7406 -- 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

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7406: arch: cxd56xx: Fix stall bulk xfer when sending 512 byte data

2022-10-24 Thread GitBox
pkarashchenko commented on PR #7406: URL: https://github.com/apache/incubator-nuttx/pull/7406#issuecomment-1288512475 Merging it directly since CI is still not fully recovered -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7404: romfs: expand file cache by CONFIG_FS_ROMFS_FCACHE_NSECTORS

2022-10-24 Thread GitBox
pkarashchenko commented on code in PR #7404: URL: https://github.com/apache/incubator-nuttx/pull/7404#discussion_r1002937958 ## fs/romfs/Kconfig: ## @@ -20,4 +20,10 @@ config FS_ROMFS_CACHE_NODE is mounted so that we can quick access entry of ROMFS