[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
pkarashchenko commented on PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#issuecomment-1278566600 Something is still wrong -- 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

[GitHub] [incubator-nuttx] ttnie commented on a diff in pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
ttnie commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r995399234 ## tools/ci/docker/linux/Dockerfile: ## @@ -78,6 +81,13 @@ FROM builder-base AS nuttx-toolchain-base RUN mkdir /tools WORKDIR /tools +# Download and install qe

[GitHub] [incubator-nuttx] jerpelea commented on pull request #7248: drivers: modem: update altair modem

2022-10-13 Thread GitBox
jerpelea commented on PR #7248: URL: https://github.com/apache/incubator-nuttx/pull/7248#issuecomment-1278538506 @pkarashchenko thanks for reviewing the code -- 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] pkarashchenko commented on a diff in pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
pkarashchenko commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r995351927 ## tools/ci/docker/linux/Dockerfile: ## @@ -78,6 +81,13 @@ FROM builder-base AS nuttx-toolchain-base RUN mkdir /tools WORKDIR /tools +# Download and in

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7305: Fixed NULL pointer use in tmpfs.

2022-10-13 Thread GitBox
pkarashchenko commented on PR #7305: URL: https://github.com/apache/incubator-nuttx/pull/7305#issuecomment-1278499165 Please rebase -- 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] jerpelea commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
jerpelea commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1278453035 > > > The change is already simple and consistent now, it's very error prone and time consuming to split the change into two commit again. @pkarashchenko and I already spend a lo

[GitHub] [incubator-nuttx] ttnie commented on a diff in pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
ttnie commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r995314879 ## tools/ci/docker/linux/Dockerfile: ## @@ -209,6 +218,7 @@ RUN cd renesas-tools/source/newlib && \ RUN cd /tools/renesas-tools/build/gcc && \ make && make ins

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7291: usrsock: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-13 Thread GitBox
masayuki2009 commented on PR #7291: URL: https://github.com/apache/incubator-nuttx/pull/7291#issuecomment-1278430283 This PR works now with https://github.com/apache/incubator-nuttx-apps/pull/1345 -- This is an automated message from the Apache Git Service. To respond to the message,

[incubator-nuttx] 02/03: usrsock:fix blocking socket connect operation hang issue

2022-10-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 commit c75253623d60fb034670e0921642caa1b339db9e Author: liangchaozhong AuthorDate: Fri Sep 16 18:43:34 2022 +0800

[incubator-nuttx] 01/03: usrsock: update state to connected when recv sento ready under connecting

2022-10-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 commit be06d00f85ce4c9d954eae8c82a71eddb4d7561c Author: zhanghongyu AuthorDate: Wed Oct 12 18:11:18 2022 +0800

[incubator-nuttx] 03/03: usrsock: combine response and events when socket setup case

2022-10-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 commit 403c657f4281ce737040d0323aada7d1b7a0e34b Author: zhanghongyu AuthorDate: Sat Oct 8 11:24:10 2022 +0800 u

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #7291: usrsock: Correct some status exceptions in non-blocking mode usrsock connect

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

[incubator-nuttx] branch master updated (28180b49d1 -> 403c657f42)

2022-10-13 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 28180b49d1 arm64: add Cortex-A57 and Cortex-A72 configurations new be06d00f85 usrsock: update state to

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#discussion_r995297542 ## arch/arm/src/samd5e5/sam_usb.c: ## @@ -8566,10 +8531,10 @@ static inline void sam_sw_initialize(struct sam_usbhost_s *priv) sam_reset_pipes(priv

[incubator-nuttx-apps] 01/03: usrsock_server: fix unblock socket connect error

2022-10-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-apps.git commit 8a2f29c9b5415846de5b7cf6407af68196830407 Author: zhanghongyu AuthorDate: Mon Aug 29 22:53:20 2022 +0800

[incubator-nuttx-apps] 03/03: usrsocktest: Fix default config mismatch in recv_block & no_block_connect

2022-10-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-apps.git commit 78541e9876f99d8131ff0961e11fc65b5b3fedef Author: zhanghongyu AuthorDate: Wed Oct 12 22:43:05 2022 +0800

[incubator-nuttx-apps] branch master updated (bcbf3ae89 -> 78541e987)

2022-10-13 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-apps.git from bcbf3ae89 Fixes in DHCPc. new 8a2f29c9b usrsock_server: fix unblock socket connect error new

[incubator-nuttx-apps] 02/03: usrsock_server: combine response and events when socket setup case

2022-10-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-apps.git commit 6e8ea78d70a6b62704ac0bbc1d37e57984ad0c23 Author: zhanghongyu AuthorDate: Sat Oct 8 10:54:12 2022 +0800

[GitHub] [incubator-nuttx-apps] masayuki2009 merged pull request #1345: usrsock_server: Correct some status exceptions in non-blocking mode usrsock connect

2022-10-13 Thread GitBox
masayuki2009 merged PR #1345: URL: https://github.com/apache/incubator-nuttx-apps/pull/1345 -- 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-unsu

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1278384108 > > The change is already simple and consistent now, it's very error prone and time consuming to split the change into two commit again. @pkarashchenko and I already spend

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7248: drivers: modem: update altair modem

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7248: URL: https://github.com/apache/incubator-nuttx/pull/7248#discussion_r995276791 ## drivers/modem/alt1250/alt1250.c: ## @@ -0,0 +1,1279 @@ +/ + * drivers/mode

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7266: ci: add qemu run support

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7266: URL: https://github.com/apache/incubator-nuttx/pull/7266#discussion_r995276353 ## boards/arm/imx6/sabre-6quad/configs/citest/defconfig: ## @@ -0,0 +1,70 @@ +# +# This file is autogenerated: PLEASE DO NOT EDIT IT. +# +# You can use "

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#discussion_r995275448 ## tools/ci/docker/linux/Dockerfile: ## @@ -78,6 +78,15 @@ FROM builder-base AS nuttx-toolchain-base RUN mkdir /tools WORKDIR /tools +# Install depen

[incubator-nuttx] branch master updated: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-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 28180b49d1 arm64: add Cortex-A57 and

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304 -- 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-website] branch asf-site updated: Publishing web: d696c67810809c3f81a4dd87b09d9e940db854b4 docs: e9db7bbb8c1c4a4f4cbce2522afa22651889057f

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

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1278272077 > This can be done by: > ./tools/refresh.sh --silent armv8a:nsh Thanks for the information. I rebased the PR. -- This is an automated message from t

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1278259748 >The change is already simple and consistent now, it's very error prone and time consuming to split the change into two commit again. @pkarashchenko and I already spend a lot

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
masayuki2009 commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1278245703 @HidenoriMatsubayashi Please rebase this PR to the latest master to fix the conflicts. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx-apps] pkarashchenko merged pull request #1346: Fixes in DHCP client.

2022-10-13 Thread GitBox
pkarashchenko merged PR #1346: URL: https://github.com/apache/incubator-nuttx-apps/pull/1346 -- 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: Fixes in DHCPc.

2022-10-13 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 bcbf3ae89 Fixes in DHCPc. b

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
pkarashchenko commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1278089177 Please rebase -- 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] pkarashchenko merged pull request #7303: arm64/qemu/qemu-a53: Ensure the purity of the defconfig

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

[incubator-nuttx] branch master updated (c22c07ba29 -> e9db7bbb8c)

2022-10-13 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 c22c07ba29 Minor fix in UBSan. add e9db7bbb8c arm64/qemu/qemu-a53: Ensure the purity of the defco

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
pkarashchenko commented on code in PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#discussion_r994989775 ## arch/arm/src/lpc54xx/lpc54_dma.c: ## @@ -437,8 +438,8 @@ int lpc54_dma_setup(int ch, uint32_t cfg, uint32_t xfrcfg, uint8_t trigsrc, putreg32(xfrcfg

[GitHub] [incubator-nuttx] pkarashchenko commented on issue #6046: BOARDIOC_SDCARD_SETNOTIFYCB callback from OS into application

2022-10-13 Thread GitBox
pkarashchenko commented on issue #6046: URL: https://github.com/apache/incubator-nuttx/issues/6046#issuecomment-1277995861 @SPRESENSE are there any plans to fix this hole in modularity? The automount signaling is already in place, so it would be good to fix this instead of keeping releasin

[GitHub] [incubator-nuttx] fjpanag opened a new issue, #7307: System does not start on STM32 when CONFIG_STACK_CANARIES is enabled.

2022-10-13 Thread GitBox
fjpanag opened a new issue, #7307: URL: https://github.com/apache/incubator-nuttx/issues/7307 When I enable the option `CONFIG_STACK_CANARIES` is not running. It builds correctly, I flash it to the board, but `main()` is never entered. Somehow this option breaks the build. This

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1277912195 Need reorder config in defconfig: https://user-images.githubusercontent.com/18066964/195658260-126727bc-eeb3-417c-9d4f-8103bcb822d8.png";> This can be done by: ./to

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7248: drivers: modem: update altair modem

2022-10-13 Thread GitBox
pkarashchenko commented on code in PR #7248: URL: https://github.com/apache/incubator-nuttx/pull/7248#discussion_r994871442 ## drivers/modem/alt1250/altcom_lwm2m_hdlr.c: ## @@ -0,0 +1,412 @@ +/ + * drive

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-13 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r994867932 ## mm/ubsan/ubsan.c: ## @@ -158,9 +158,11 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, if (is_inline_int(type)) { uns

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-129678 I merged https://github.com/apache/incubator-nuttx/pull/7303 into this change. -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-13 Thread GitBox
pkarashchenko commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r994777994 ## mm/ubsan/ubsan.c: ## @@ -158,9 +158,11 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, if (is_inline_int(type)) { uns

[incubator-nuttx] branch master updated: Minor fix in UBSan.

2022-10-13 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.git The following commit(s) were added to refs/heads/master by this push: new c22c07ba29 Minor fix in UBSan. c

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7301: Ubsan

2022-10-13 Thread GitBox
pkarashchenko merged PR #7301: URL: https://github.com/apache/incubator-nuttx/pull/7301 -- 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] HidenoriMatsubayashi commented on pull request #7303: arm64/qemu/qemu-a53: Ensure the purity of the defconfig

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7303: URL: https://github.com/apache/incubator-nuttx/pull/7303#issuecomment-1277752012 @zhhyu7 @xiaoxiang781216 `CONFIG_ARCH_CHIP_QEMU_A53` exists in the output file of `make savedefconfig`. And also, it's defined in `arch/arm64/src/qemu/Kconfig`

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7303: arm64/qemu/qemu-a53: Ensure the purity of the defconfig

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7303: URL: https://github.com/apache/incubator-nuttx/pull/7303#issuecomment-1277713999 > Normalize qemu-a53/nsh This may be my mistake, but what does this mean? -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [incubator-nuttx-apps] fjpanag commented on pull request #1346: Fixes in DHCP client.

2022-10-13 Thread GitBox
fjpanag commented on PR #1346: URL: https://github.com/apache/incubator-nuttx-apps/pull/1346#issuecomment-1277710525 Is CI fixed now? Can you restart it? -- 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 t

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1277699545 @xiaoxiang781216 Oh, sorry. It looks like my mistake. You mean should I merge it into this patch? -- This is an automated message from the Apache Git Servic

[GitHub] [incubator-nuttx] fjpanag opened a new issue, #7306: Failure of rename() may cause delete files.

2022-10-13 Thread GitBox
fjpanag opened a new issue, #7306: URL: https://github.com/apache/incubator-nuttx/issues/7306 There is a case were `rename()` may fail, but also delete files in the process that it shouldn't. This can happen when `oldpath` refers to a non-existent file, while `newpath` refers to a fi

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1277691979 @HidenoriMatsubayashi should we drop this patch: https://github.com/apache/incubator-nuttx/pull/7303 -- This is an automated message from the Apache Git Service. To resp

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1277670039 > @xiaoxiang781216 > > > As I said before, this patch want to distinguish sem_t usage which contain two part: > > I think the two parts are independent and if

[GitHub] [incubator-nuttx] fjpanag opened a new pull request, #7305: Fixed NULL pointer use in tmpfs.

2022-10-13 Thread GitBox
fjpanag opened a new pull request, #7305: URL: https://github.com/apache/incubator-nuttx/pull/7305 ## Summary When the file size is 0 and the data to be written is also 0 (in case of `tmpfs_write()`) then `tfo->tfo_data` will be NULL. Previously, the code used this buffer uncon

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304#issuecomment-1277664999 @xiaoxiang781216 @masayuki2009 Can you please review this change? -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-nuttx] HidenoriMatsubayashi opened a new pull request, #7304: arm64: add Cortex-A57 and Cortex-A72 configurations

2022-10-13 Thread GitBox
HidenoriMatsubayashi opened a new pull request, #7304: URL: https://github.com/apache/incubator-nuttx/pull/7304 ## Summary This change adds the followings: - Rename the board configuration name from `qemu-a53` `to qemu-v8a`. - Add the configurations for Cortex-A57 and Cortex-A72.

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request, #7303: arm64/qemu/qemu-a53: Ensure the purity of the defconfig

2022-10-13 Thread GitBox
zhhyu7 opened a new pull request, #7303: URL: https://github.com/apache/incubator-nuttx/pull/7303 Configuring... Building NuttX... Normalize qemu-a53/nsh 13a14 > CONFIG_ARCH_CHIP_QEMU_A53=y Saving the new configuration file On branch master Your branch is up to date

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1277628210 @xiaoxiang781216 >As I said before, this patch want to distinguish sem_t usage which contain two part: I think the two parts are independent and if the commit i

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1277528793 > > @masayuki2009 ok, the change split into two patch, please review again. > > @xiaoxiang781216 Thanks for the changes. As far as I can see, the first commit still

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1277507186 Thanks for the reviews. > I think we should also change the name CONFIG_ARCH_BOARD_QEMU_A53 (+ directory name). I'll create a new patch and send it later

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
masayuki2009 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1277481868 >@masayuki2009 ok, the change split into two patch, please review again. @xiaoxiang781216 Thanks for the changes. As far as I can see, the first commit still cont

[incubator-nuttx] branch master updated (83c76eaa93 -> 06cf41901f)

2022-10-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 83c76eaa93 Documentation file typo's corrected add 06cf41901f arch/arm64: cleanup defconfigs No new

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295 -- 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 #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#issuecomment-1277456023 Let's ignore the ci temp break. -- 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

[incubator-nuttx] branch master updated: Documentation file typo's corrected

2022-10-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 83c76eaa93 Documentation file typo's

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7302: Documentation file typo's corrected

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #7302: URL: https://github.com/apache/incubator-nuttx/pull/7302 -- 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: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-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 143d1322ea Added handling of MSG_WAI

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286 -- 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 #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#issuecomment-1277381560 Both sim and macos pass the ci which mean the network change in this patch should work as expect, so let's ignore the ci break. -- This is an automated message from the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#issuecomment-1277374084 @fjpanag could you rebase the change to the latest master? The ci break is fixed after your update. -- This is an automated message from the Apache Git Service. To respo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r994448097 ## net/tcp/tcp_recvfrom.c: ## @@ -412,7 +412,8 @@ static uint16_t tcp_recvhandler(FAR struct net_driver_s *dev, * next receive is performed.

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on a diff in pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on code in PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#discussion_r994445371 ## boards/arm64/qemu/qemu-a53/configs/nsh/defconfig: ## @@ -11,7 +11,6 @@ CONFIG_ARCH_BOARD="qemu-a53" CONFIG_ARCH_BOARD_QEMU_A53=y CONFIG_ARCH_CH

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1277368760 @masayuki2009 ok, the change split into two patch, please review again. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#discussion_r994442729 ## boards/arm64/qemu/qemu-a53/configs/nsh/defconfig: ## @@ -11,7 +11,6 @@ CONFIG_ARCH_BOARD="qemu-a53" CONFIG_ARCH_BOARD_QEMU_A53=y CONFIG_ARCH_CHIP="q

[GitHub] [incubator-nuttx] TimJTi opened a new pull request, #7302: Documentation file typo's corrected

2022-10-13 Thread GitBox
TimJTi opened a new pull request, #7302: URL: https://github.com/apache/incubator-nuttx/pull/7302 Correct "Nuttx" to "NuttX" ## Summary ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-nuttx] ttnie commented on pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
ttnie commented on PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#issuecomment-1277352392 > rebase the code and trigger ci again. -- 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] fjpanag commented on pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
fjpanag commented on PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#issuecomment-1277352136 > I haven't tested the latest changes on actual hardware, yet. > Please do not merge before confirmation. This patch was tested on actual hardware, and seems to behave cor

[GitHub] [incubator-nuttx-apps] fjpanag opened a new pull request, #1346: Fixes in DHCP client.

2022-10-13 Thread GitBox
fjpanag opened a new pull request, #1346: URL: https://github.com/apache/incubator-nuttx-apps/pull/1346 ## Summary Two fixes in DHCP client: 1. Previously, on every request the client was setting the current IP address to `INADDR_ANY`. This may be OK for the initial request, bu

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on a diff in pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on code in PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#discussion_r994418555 ## boards/arm64/qemu/qemu-a53/configs/nsh/defconfig: ## @@ -11,7 +11,6 @@ CONFIG_ARCH_BOARD="qemu-a53" CONFIG_ARCH_BOARD_QEMU_A53=y CONFIG_ARCH_CH

[GitHub] [incubator-nuttx] HidenoriMatsubayashi commented on a diff in pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
HidenoriMatsubayashi commented on code in PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#discussion_r994413328 ## boards/arm64/qemu/qemu-a53/configs/nsh/defconfig: ## @@ -11,7 +11,6 @@ CONFIG_ARCH_BOARD="qemu-a53" CONFIG_ARCH_BOARD_QEMU_A53=y CONFIG_ARCH_CH

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7295: arch/arm64: cleanup defconfigs

2022-10-13 Thread GitBox
xiaoxiang781216 commented on code in PR #7295: URL: https://github.com/apache/incubator-nuttx/pull/7295#discussion_r994404885 ## boards/arm64/qemu/qemu-a53/configs/nsh/defconfig: ## @@ -11,7 +11,6 @@ CONFIG_ARCH_BOARD="qemu-a53" CONFIG_ARCH_BOARD_QEMU_A53=y CONFIG_ARCH_CHIP="q

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7301: Ubsan

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7301: URL: https://github.com/apache/incubator-nuttx/pull/7301#issuecomment-1277312607 @fjpanag please rebase your change. -- 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] xiaoxiang781216 commented on pull request #7298: install qemu on docker

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7298: URL: https://github.com/apache/incubator-nuttx/pull/7298#issuecomment-1277311694 @ttnie please rebase the patch, the ci break is fixed here: https://github.com/apache/incubator-nuttx/pull/7299 -- This is an automated message from the Apache Git Servi

[incubator-nuttx] branch master updated: Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022"

2022-10-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 8150534a2f Revert "tools/ci: patch f

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7299: Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022"

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #7299: URL: https://github.com/apache/incubator-nuttx/pull/7299 -- 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 #7299: Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022"

2022-10-13 Thread GitBox
xiaoxiang781216 commented on PR #7299: URL: https://github.com/apache/incubator-nuttx/pull/7299#issuecomment-1277309562 let's merge the change directly to unlock ci. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-nuttx] fjpanag opened a new pull request, #7301: Ubsan

2022-10-13 Thread GitBox
fjpanag opened a new pull request, #7301: URL: https://github.com/apache/incubator-nuttx/pull/7301 ## Summary Minor fix in UBSan. See discussion in #7285. ## Impact Bug fix. ## Testing I run some test cases in simulator (on 64-bit host), and it yields t

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7285: Fixed undefined behavior in UBSan.

2022-10-13 Thread GitBox
fjpanag commented on code in PR #7285: URL: https://github.com/apache/incubator-nuttx/pull/7285#discussion_r994365145 ## mm/ubsan/ubsan.c: ## @@ -158,9 +158,11 @@ static int64_t get_signed_val(FAR struct type_descriptor *type, if (is_inline_int(type)) { unsigned

[GitHub] [incubator-nuttx] fjpanag commented on a diff in pull request #7286: Added handling of MSG_WAITALL flag in TCP recv.

2022-10-13 Thread GitBox
fjpanag commented on code in PR #7286: URL: https://github.com/apache/incubator-nuttx/pull/7286#discussion_r994331743 ## net/tcp/tcp_recvfrom.c: ## @@ -412,7 +412,8 @@ static uint16_t tcp_recvhandler(FAR struct net_driver_s *dev, * next receive is performed.

[incubator-nuttx-website] branch asf-site updated: Publishing web: d696c67810809c3f81a4dd87b09d9e940db854b4 docs: 53dd0819cf4317f24e081aa0175d1c4ef3fa4c57

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

[incubator-nuttx-website] branch master updated: Bump tzinfo from 1.2.6 to 1.2.10

2022-10-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-website.git The following commit(s) were added to refs/heads/master by this push: new d696c67 Bump tzinfo from 1.

[GitHub] [incubator-nuttx-website] xiaoxiang781216 merged pull request #71: Bump tzinfo from 1.2.6 to 1.2.10

2022-10-13 Thread GitBox
xiaoxiang781216 merged PR #71: URL: https://github.com/apache/incubator-nuttx-website/pull/71 -- 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-un