[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #5996: stdlib: generate uniformly distributed pseudo-random numbers

2022-04-15 Thread GitBox
Donny9 commented on code in PR #5996: URL: https://github.com/apache/incubator-nuttx/pull/5996#discussion_r851594721 ## include/stdlib.h: ## @@ -128,6 +128,15 @@ extern "C" void srand(unsigned int seed); int rand(void); +doubledrand48(void); +doubleerand48

[GitHub] [incubator-nuttx] Donny9 commented on a diff in pull request #5996: stdlib: generate uniformly distributed pseudo-random numbers

2022-04-15 Thread GitBox
Donny9 commented on code in PR #5996: URL: https://github.com/apache/incubator-nuttx/pull/5996#discussion_r851594678 ## libs/libc/stdlib/lib_rand48.c: ## @@ -0,0 +1,194 @@ +/ + * libs/libc/stdlib/lib_ran

[GitHub] [incubator-nuttx] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851592871 ## net/can/can_recvmsg.c: ## @@ -410,8 +410,7 @@ static uint16_t can_recvfrom_eventhandler(FAR struct net_driver_s *dev,

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100584725 Ok. Let's proceed with this pr -- 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 t

[incubator-nuttx-apps] branch master updated: usrsock: Don't clear recv and send available flag

2022-04-15 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 b80ef1df8 usrsock: Don't clear

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1132: usrsock: Don't clear recv and send available flag

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #1132: URL: https://github.com/apache/incubator-nuttx-apps/pull/1132 -- 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 #6031: usrsock: Don't clear recvfrom available flag

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6031: URL: https://github.com/apache/incubator-nuttx/pull/6031#issuecomment-1100582703 Ok, let's wait WIFI board arrival. -- 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] xiaoxiang781216 commented on pull request #6078: Stm32f746 audio

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100582501 @okayserh I rebase your patch on top of apache mainline, please take a look. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100581075 > > I'd like to notice that the the new chip can not use the arm common context switch code because the different assembly grammar. The reason we want to put this chip in

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851589813 ## boards/arm/stm32h7/nucleo-h743zi2/configs/socketcan/defconfig: ## @@ -0,0 +1,75 @@ +# +# This file is autogenerated: PLEASE DO NOT EDIT IT. +# +# You

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851589334 ## include/net/if.h: ## @@ -196,36 +210,37 @@ struct lifreq struct lifconf { - size_t lifc_len; /* Size of buff

[GitHub] [incubator-nuttx] Donny9 commented on pull request #6081: boards/sim: support rc.sysinit

2022-04-15 Thread GitBox
Donny9 commented on PR #6081: URL: https://github.com/apache/incubator-nuttx/pull/6081#issuecomment-1100578592 This PR is associated with https://github.com/apache/incubator-nuttx-apps/pull/1142 -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx] Donny9 opened a new pull request, #6081: boards/sim: support rc.sysinit

2022-04-15 Thread GitBox
Donny9 opened a new pull request, #6081: URL: https://github.com/apache/incubator-nuttx/pull/6081 ## Summary Follow: http://glennastory.net/boot/sysinit.html This is first script that init runs is rc.sysinit. This script does serval initialization tasks about basic service.

[GitHub] [incubator-nuttx-apps] Donny9 opened a new pull request, #1142: nsh/script: support rc.sysinit script

2022-04-15 Thread GitBox
Donny9 opened a new pull request, #1142: URL: https://github.com/apache/incubator-nuttx-apps/pull/1142 ## Summary Follow: http://glennastory.net/boot/sysinit.html This is first script that init runs is rc.sysinit. This script does serval initialization tasks about basic servi

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

2022-04-15 Thread GitBox
masayuki2009 commented on PR #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-1100569224 @zhuyanlinzyl >It hang in there and not any other log print. > >Can you give me some advice? How about enabling debug options and also disables ram log as

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

2022-04-15 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 ff4507d Publishing web

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

2022-04-15 Thread GitBox
pkarashchenko commented on PR #5659: URL: https://github.com/apache/incubator-nuttx/pull/5659#issuecomment-1100466839 I've filled a bug on ARM GCC 11: https://bugs.linaro.org/show_bug.cgi?id=5844 -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx] JacobCrabill commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100443424 Regarding GPL, I don't see any issues with referring to the use of _external_ GPL'd code in order to build a specific board config. If the GPL'd code was within this reposit

[GitHub] [incubator-nuttx] JacobCrabill commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851537433 ## net/can/can_recvmsg.c: ## @@ -410,8 +410,7 @@ static uint16_t can_recvfrom_eventhandler(FAR struct net_driver_s *dev,

[GitHub] [incubator-nuttx] JacobCrabill commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851537126 ## arch/arm/src/stm32h7/stm32_fdcan_sock.c: ## @@ -0,0 +1,2422 @@ +/ + * arch/ar

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6078: Stm32f746 audio

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100384845 No. Seems like the merge commit is still present. Let me try it locally. Maybe interactive rebase is needed in case if branch already contains a "merge" commit. -- This i

[GitHub] [incubator-nuttx] okayserh commented on pull request #6078: Stm32f746 audio

2022-04-15 Thread GitBox
okayserh commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100378640 Thanks for the quick help! I've entered the commands as suggested. Did this do the job? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6078: Stm32f746 audio

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100327159 ``` git remote add upstream g...@github.com:apache/incubator-nuttx.git git checkout stm32f746_audio git fetch upstream master git rebase upstream/master git reb

[GitHub] [incubator-nuttx] okayserh commented on pull request #6078: Stm32f746 audio

2022-04-15 Thread GitBox
okayserh commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100305165 Happy to do that, when I have figured out how to do it :-). Have circumvented more serious use of git for some time :-/. I'll go through the docs to figure out ... -- This is

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100252807 > The ucans32k146 board already in this repo also contains CONFIG_CANUTILS_x=y. Not sure what the policy is on board configs that refer to the use of GPL'd code in an independe

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100247303 > I had it split out into smaller pieces, but @xiaoxiang781216 requested I consolidate them into one commit. Should I split it back out? Sorry, I but I didn't see the pre

[GitHub] [incubator-nuttx] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851390636 ## arch/arm/src/stm32h7/stm32_fdcan_sock.c: ## @@ -0,0 +1,2422 @@ +/ + * arch/arm/

[GitHub] [incubator-nuttx] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851385873 ## net/can/can_recvmsg.c: ## @@ -410,8 +410,7 @@ static uint16_t can_recvfrom_eventhandler(FAR struct net_driver_s *dev,

[GitHub] [incubator-nuttx] JacobCrabill commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851380541 ## arch/arm/src/stm32h7/stm32_fdcan_sock.c: ## @@ -0,0 +1,2422 @@ +/ + * arch/ar

[GitHub] [incubator-nuttx] JacobCrabill commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851377763 ## net/can/can_recvmsg.c: ## @@ -410,8 +410,7 @@ static uint16_t can_recvfrom_eventhandler(FAR struct net_driver_s *dev,

[GitHub] [incubator-nuttx] JacobCrabill commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100225847 > The CAN-utils library requires ALLOW_GPL_COMPONENTS=y. Should we allow this in upstream NuttX ? I'm not sure what is Apache policy in this matter. The `ucans32k146` b

[GitHub] [incubator-nuttx] JacobCrabill commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100223182 > Commit [b16ad66](https://github.com/apache/incubator-nuttx/commit/b16ad669bc6a6a1a7863082c33da01f48dc866cb) should be definitely divided into smaller patches. There are man

[GitHub] [incubator-nuttx] JacobCrabill commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851373336 ## arch/arm/src/stm32/hardware/stm32_fdcan.h: ## @@ -43,7 +43,6 @@ #define STM32_FDCAN_CREL_OFFSET0x /* FDCAN core release register */ #define

[GitHub] [incubator-nuttx] JacobCrabill commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
JacobCrabill commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1100221925 > @JacobCrabill the follow error need be fixed before merging: > The `slcan` portion, at least, is coming from `apps` when I renamed `irfu_can_data_s` per @pkarashc

[incubator-nuttx-apps] branch master updated (460f631fa -> ab5e7496d)

2022-04-15 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-apps.git from 460f631fa netutils/usrsock_rpmsg: broadcast the client DNS server to server add ab5e7496d Makef

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140 -- 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-apps] anchao commented on a diff in pull request #1141: testing/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE

2022-04-15 Thread GitBox
anchao commented on code in PR #1141: URL: https://github.com/apache/incubator-nuttx-apps/pull/1141#discussion_r851356731 ## testing/ostest/fpu.c: ## @@ -211,8 +179,8 @@ static int fpu_task(int argc, char *argv[]) * that we can verify that reading of the registers actua

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080#discussion_r851346663 ## arch/risc-v/src/common/riscv_fpucmp.c: ## @@ -37,58 +34,27 @@ * Pre-processor Definitions

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a diff in pull request #1141: testing/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #1141: URL: https://github.com/apache/incubator-nuttx-apps/pull/1141#discussion_r851332511 ## testing/ostest/fpu.c: ## @@ -211,8 +179,8 @@ static int fpu_task(int argc, char *argv[]) * that we can verify that reading of the regist

[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
anchao commented on code in PR #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080#discussion_r851340807 ## include/nuttx/arch.h: ## @@ -2563,6 +2563,16 @@ uint32_t up_perf_gettime(void); uint32_t up_perf_getfreq(void); void up_perf_convert(uint32_t elapsed, FAR st

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080#discussion_r851330574 ## arch/arm/src/common/arm_fpucmp.c: ## @@ -25,70 +25,32 @@ #include #include -#include #include -#include - #include -#include #include

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
xiaoxiang781216 commented on code in PR #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080#discussion_r851329673 ## arch/arm/include/armv7-m/irq_cmnvector.h: ## @@ -133,6 +133,8 @@ #define XCPTCONTEXT_REGS(HW_XCPT_REGS + SW_XCPT_REGS) #define XCPTCONTEXT_SIZE

[GitHub] [incubator-nuttx-apps] anchao commented on pull request #1141: testing/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE

2022-04-15 Thread GitBox
anchao commented on PR #1141: URL: https://github.com/apache/incubator-nuttx-apps/pull/1141#issuecomment-1100169023 Depends on: https://github.com/apache/incubator-nuttx/pull/6080 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [incubator-nuttx] anchao commented on pull request #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
anchao commented on PR #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080#issuecomment-1100168903 Depends on https://github.com/apache/incubator-nuttx-apps/pull/1141 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request, #1141: testing/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE

2022-04-15 Thread GitBox
anchao opened a new pull request, #1141: URL: https://github.com/apache/incubator-nuttx-apps/pull/1141 ## Summary testing/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE fptest should not need to care about the size of FPU Signed-off-by: chao.an ## Impact fpu

[GitHub] [incubator-nuttx] anchao opened a new pull request, #6080: arch/ostest: remove CONFIG_TESTING_OSTEST_FPUSIZE & board level implement

2022-04-15 Thread GitBox
anchao opened a new pull request, #6080: URL: https://github.com/apache/incubator-nuttx/pull/6080 ## Summary board/arch_fpu*: move arch_[get|cmp]fpu to common arch boards/ostest: remove board ostest implement arch/arm: export arm_saveusercontext() ## Impact N/A

[incubator-nuttx] 04/05: boards/risc-v/esp32c3: ESP32C3 TWAI (CAN) controller bringup for DevKitM-1.

2022-04-15 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 0d9d2f4bf8d45b5380f61947855c46f9dd2fc8ae Author: Jan Charvat AuthorDate: Wed Apr 6 12:22:18 2022 +0200

[incubator-nuttx] 01/05: arch/riscv/esp32c3: esp32c3_twai.h TWAI (CAN) controller registers provided by Espressif.

2022-04-15 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 0fa1541a8025934216f209697f3758d6d141b4ad Author: Jan Charvat AuthorDate: Wed Apr 6 12:01:16 2022 +0200

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6005: ESP32C3 TWAI (CAN) controller support.

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #6005: URL: https://github.com/apache/incubator-nuttx/pull/6005 -- 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] 05/05: boards/risc-v/esp32c3: ESP32C3 TWAI (CAN) controller config for DevKitM-1.

2022-04-15 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 6d566e0fda6e920444e152ebfa86df1572636338 Author: Jan Charvat AuthorDate: Wed Apr 6 12:56:39 2022 +0200

[incubator-nuttx] 03/05: arch/riscv/esp32c3: ESP32C3 TWAI (CAN) controller included into Kconfig.

2022-04-15 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 6ec86361a867d84063d1e6a74540f5914100a34c Author: Jan Charvat AuthorDate: Wed Apr 6 12:15:04 2022 +0200

[incubator-nuttx] 02/05: arch/riscv/esp32c3: ESP32C3 TWAI (CAN) controller driver.

2022-04-15 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit e28584131dc5b3d39fd9c103eb7e2615da275c40 Author: Jan Charvat AuthorDate: Wed Apr 6 12:10:45 2022 +0200

[incubator-nuttx] branch master updated (a94c865791 -> 6d566e0fda)

2022-04-15 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 a94c865791 arch/risc-v: Use fs status definition from csr.h new 0fa1541a80 arch/riscv/esp32c3: esp32c

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100143698 > I'd like to notice that the the new chip can not use the arm common context switch code because the different assembly grammar. The reason we want to put this chip in arch

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100140309 Yeah. Also `__builtin_frame_address(0)` can generate different code from `mov %0, sp`. I just tried to compile a sample code and got `movr0, fp`, so it depends of optimi

[GitHub] [incubator-nuttx] CV-Bowen commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
CV-Bowen commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100139783 I'd like to notice that the the new chip can not use the arm common context switch code because the different assembly grammar. The reason we want to put this chip in arch/arm is

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100136259 > > > This seems to be GCC specific implementation. > > > > > > Yes, but `__asm__` keyword also GCC specific too:), not worse than before. > > Yeah, most

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100128496 I mean what is the benefit of this change? Is it clang related or current implementation brings some issues? -- This is an automated message from the Apache Git Service. T

[GitHub] [incubator-nuttx] zouboan commented on issue #6012: Low priority tasks interrupt high priority threads

2022-04-15 Thread GitBox
zouboan commented on issue #6012: URL: https://github.com/apache/incubator-nuttx/issues/6012#issuecomment-1100127005 @xiaoxiang781216 I implemented a read-write lock, based on semaphore: [https://github.com/zouboan/incubator-fs_files.cnuttx/blob/fft/sched/semaphore/rwlock.c](url) and

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100123056 > > This seems to be GCC specific implementation. > > Yes, but `__asm__` keyword also GCC specific too:), not worse than before. Yeah, most of other compilers ha

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100117013 > This seems to be GCC specific implementation. Yes, but __asm__ also GCC specific too:), not worse than before. -- This is an automated message from the Apache Gi

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100108393 This seems to be GCC specific implementation. -- 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] CV-Bowen commented on pull request #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
CV-Bowen commented on PR #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079#issuecomment-1100108095 @xiaoxiang781216 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific com

[GitHub] [incubator-nuttx] CV-Bowen opened a new pull request, #6079: arm/arch: using __builtin_frame_address(0) implement up_getsp().

2022-04-15 Thread GitBox
CV-Bowen opened a new pull request, #6079: URL: https://github.com/apache/incubator-nuttx/pull/6079 ## Summary using compile builtin function __builtin_frame_address(0) to implement the up_getsp() ## Test stm32f7 -- This is an automated message from the Apache Git Service

[GitHub] [incubator-nuttx] charvj commented on pull request #6005: ESP32C3 TWAI (CAN) controller support.

2022-04-15 Thread GitBox
charvj commented on PR #6005: URL: https://github.com/apache/incubator-nuttx/pull/6005#issuecomment-1100093492 @xiaoxiang781216 I applied the suggested changes from @pkarashchenko -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5966: arch:tcbinfo: update tcbinfo as xcpcontext update

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #5966: URL: https://github.com/apache/incubator-nuttx/pull/5966#issuecomment-1100061332 > @zhuyanlinzyl > > > So I'd like to known why this PR does't work, as I think the info threads and info stack can work > > `info threads` does not work becau

[incubator-nuttx] branch master updated (09b3fb25ab -> a94c865791)

2022-04-15 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 09b3fb25ab drivers: remove unimplemented open/close/ioctl interfaces add a94c865791 arch/risc-v: Use

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6077: arch/risc-v: Use fs status definition from csr.h

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #6077: URL: https://github.com/apache/incubator-nuttx/pull/6077 -- 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 #6078: Stm32f746 audio

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078#issuecomment-1100056935 @okayserh could you drop the last merge patch? -- 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] okayserh opened a new pull request, #6078: Stm32f746 audio

2022-04-15 Thread GitBox
okayserh opened a new pull request, #6078: URL: https://github.com/apache/incubator-nuttx/pull/6078 ## Summary STM32_RCC_DCKCFGR is not defined for the stm32f746, since the stm32f746 has two configuration registers STM32_RCC_DCKCFGR1 and STM32_RCC_DCKCFGR2. Since the LTDC is only affecte

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6031: usrsock: Don't clear recvfrom available flag

2022-04-15 Thread GitBox
masayuki2009 commented on PR #6031: URL: https://github.com/apache/incubator-nuttx/pull/6031#issuecomment-1100023019 >@masayuki2009 can https://github.com/apache/incubator-nuttx-apps/pull/1139 fix this issue too? @xiaoxiang781216 Unfortunately no. The PR fixes recvfrom() + TCP pas

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #3455: Use CMake

2022-04-15 Thread GitBox
xiaoxiang781216 commented on issue #3455: URL: https://github.com/apache/incubator-nuttx/issues/3455#issuecomment-1100021684 The work is inactive for a while. From the feedback, the community want to support both method. Welcome to contrib. -- This is an automated message from the Apache

[GitHub] [incubator-nuttx] easonxiang commented on issue #3455: Use CMake

2022-04-15 Thread GitBox
easonxiang commented on issue #3455: URL: https://github.com/apache/incubator-nuttx/issues/3455#issuecomment-1100016784 Any progress of this topic? It is not easy to understand and maintain the current build system :( -- This is an automated message from the Apache Git Service. To resp

[GitHub] [incubator-nuttx] no1wudi opened a new pull request, #6077: arch/risc-v: Use fs status definition from csr.h

2022-04-15 Thread GitBox
no1wudi opened a new pull request, #6077: URL: https://github.com/apache/incubator-nuttx/pull/6077 ## Summary Remove local definition from riscv_fpu.S ## Impact Refactor only ## Testing CI/ rv-virt -- This is an automated message from the Apache Git Service. To respond t

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

2022-04-15 Thread GitBox
zhuyanlinzyl commented on PR #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-114609 @masayuki2009 Is the loadaddr(0x1080) error ? I just use same address from you -- This is an automated message from the Apache Git Service. To respond to the message,

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

2022-04-15 Thread GitBox
zhuyanlinzyl commented on PR #5650: URL: https://github.com/apache/incubator-nuttx/pull/5650#issuecomment-114028 hi @masayuki2009 : I have a sabre-6quad board and use it to investigate this problem. The board name is `Freescale i.MX6 Quad SABRE Smart Device Board`, and I use `s

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6076: arch/armv[7|8]m: enhance the clang support

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6076: URL: https://github.com/apache/incubator-nuttx/pull/6076#issuecomment-1099985973 Nice! I just open a tick to ask the new release(https://github.com/ARM-software/LLVM-embedded-toolchain-for-Arm/issues/108). It could be closed now:). -- This is an aut

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6076: arch/armv[7|8]m: enhance the clang support

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6076: URL: https://github.com/apache/incubator-nuttx/pull/6076#issuecomment-1099984527 > > 14.0.0 release was started 20h ago. I will check if `--print-libgcc-file-name` bug is still there as soon as artifacts will be available for downloading, > > wher

[GitHub] [incubator-nuttx] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851161126 ## net/can/can_recvmsg.c: ## @@ -410,8 +410,7 @@ static uint16_t can_recvfrom_eventhandler(FAR struct net_driver_s *dev,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6076: arch/armv[7|8]m: enhance the clang support

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6076: URL: https://github.com/apache/incubator-nuttx/pull/6076#issuecomment-1099977946 > 14.0.0 release was started 20h ago. I will check if `--print-libgcc-file-name` bug is still there as soon as artifacts will be available for downloading, where do

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6076: arch/armv[7|8]m: enhance the clang support

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6076: URL: https://github.com/apache/incubator-nuttx/pull/6076#issuecomment-1099977187 14.0.0 release was started 20h ago. I will check if `--print-libgcc-file-name` bug is still there as soon as artifacts will be available for downloading, -- This is an au

[incubator-nuttx] branch master updated (6041a2f0db -> 09b3fb25ab)

2022-04-15 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 6041a2f0db boards/esp32_twai: Remove arm_arch.h from comment add 09b3fb25ab drivers: remove unimpleme

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6053: drivers: remove unimplemented open/close/ioctl interfaces

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #6053: URL: https://github.com/apache/incubator-nuttx/pull/6053 -- 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 #6053: drivers: remove unimplemented open/close/ioctl interfaces

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6053: URL: https://github.com/apache/incubator-nuttx/pull/6053#issuecomment-1099977059 Yes, I was thought that the compiler error doesn't fix yet. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6053: drivers: remove unimplemented open/close/ioctl interfaces

2022-04-15 Thread GitBox
pkarashchenko commented on PR #6053: URL: https://github.com/apache/incubator-nuttx/pull/6053#issuecomment-1099975903 @xiaoxiang781216 could you please merge the PR? -- 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] pkarashchenko merged pull request #6075: boards/esp32_twai: Remove arm_arch.h from comment

2022-04-15 Thread GitBox
pkarashchenko merged PR #6075: URL: https://github.com/apache/incubator-nuttx/pull/6075 -- 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: boards/esp32_twai: Remove arm_arch.h from comment

2022-04-15 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 6041a2f0db boards/esp32_twai: Re

[GitHub] [incubator-nuttx-apps] pkarashchenko commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
pkarashchenko commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851150267 ## Application.mk: ## @@ -54,10 +54,13 @@ endif ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers ha

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1099959335 Commit https://github.com/apache/incubator-nuttx/pull/6057/commits/b16ad669bc6a6a1a7863082c33da01f48dc866cb should be definitely divided into smaller patches. There are man

[GitHub] [incubator-nuttx] raiden00pl commented on pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#issuecomment-1099954889 The CAN-utils library requires ALLOW_GPL_COMPONENTS=y. Should we allow this in upstream NuttX ? I'm not sure what is Apache policy in this matter. -- This is an automated mes

[GitHub] [incubator-nuttx] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851133901 ## arch/arm/src/stm32/hardware/stm32_fdcan.h: ## @@ -43,7 +43,6 @@ #define STM32_FDCAN_CREL_OFFSET0x /* FDCAN core release register */ #define ST

[incubator-nuttx-apps] branch master updated: netutils/usrsock_rpmsg: broadcast the client DNS server to server

2022-04-15 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 460f631fa netutils/usrsock_rpms

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1136: netutils/usrsock_rpmsg: broadcast the client DNS server to server

2022-04-15 Thread GitBox
xiaoxiang781216 merged PR #1136: URL: https://github.com/apache/incubator-nuttx-apps/pull/1136 -- 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] raiden00pl commented on a diff in pull request #6057: arch/stm32h7: Add SocketCAN FDCAN driver

2022-04-15 Thread GitBox
raiden00pl commented on code in PR #6057: URL: https://github.com/apache/incubator-nuttx/pull/6057#discussion_r851132617 ## arch/arm/src/stm32h7/stm32_fdcan_sock.c: ## @@ -0,0 +1,2422 @@ +/ + * arch/arm/

[GitHub] [incubator-nuttx-apps] anchao commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
anchao commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851132288 ## Application.mk: ## @@ -55,9 +55,12 @@ ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers have similar?

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6031: usrsock: Don't clear recvfrom available flag

2022-04-15 Thread GitBox
xiaoxiang781216 commented on PR #6031: URL: https://github.com/apache/incubator-nuttx/pull/6031#issuecomment-1099942577 @masayuki2009 can https://github.com/apache/incubator-nuttx-apps/pull/1139 fix this issue too? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-nuttx-apps] anchao commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
anchao commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851129073 ## Application.mk: ## @@ -55,9 +55,12 @@ ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers have similar?

[GitHub] [incubator-nuttx-apps] anchao commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
anchao commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851125730 ## Application.mk: ## @@ -54,10 +54,13 @@ endif ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers have simi

[GitHub] [incubator-nuttx-apps] pkarashchenko commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
pkarashchenko commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851125126 ## Application.mk: ## @@ -55,9 +55,12 @@ ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers have simil

[GitHub] [incubator-nuttx-apps] anchao commented on a diff in pull request #1140: Makefile/Application: correct libgcc PATH

2022-04-15 Thread GitBox
anchao commented on code in PR #1140: URL: https://github.com/apache/incubator-nuttx-apps/pull/1140#discussion_r851120449 ## Application.mk: ## @@ -54,10 +54,13 @@ endif ifeq ($(BUILD_MODULE),y) # Revisit: This only works for gcc and clang. # Do other compilers have simi

  1   2   >