xiaoxiang781216 commented on PR #1321:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/1321#issuecomment-1257169222
> @xiaoxiang781216 I think that you need to generate patch with
`--no-renames` option to get `patch` correctly interpreted on MAC
Done.
--
This is an automa
anchao opened a new pull request, #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181
## Summary
compiler.h: add MSVC(Microsoft Visual C++)-specific definitions
Signed-off-by: chao an
## Impact
N/A
## Testing
cmake + sim/windows
--
anchao opened a new pull request, #7182:
URL: https://github.com/apache/incubator-nuttx/pull/7182
## Summary
sim/windows: add more net configs
.
Log:
```
user_main:
hartmannathan commented on PR #6318:
URL: https://github.com/apache/incubator-nuttx/pull/6318#issuecomment-1257190965
By the way, how many `CONFIG_SEM_PREALLOCHOLDERS` "Number of pre-allocated
holders" are you using? (I used the default of 8.)
--
This is an automated message from the Apac
pkarashchenko commented on PR #6318:
URL: https://github.com/apache/incubator-nuttx/pull/6318#issuecomment-1257195775
I use default. Unfortunately I do not have the same bord as you. I just got
stm32f4 based board, but need to solder UART pins. Will test with that board on
upcoming week.
pkarashchenko commented on PR #1321:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/1321#issuecomment-1257196138
What about my comment in https://github.com/apache/incubator-nuttx/pull/6987
? Maybe we can move nuttx specific queue.h instead?
--
This is an automated message from
hartmannathan commented on PR #6318:
URL: https://github.com/apache/incubator-nuttx/pull/6318#issuecomment-1257196243
Sadly, no joy with `tm4c129e-launchpad:ostest` (+
`CONFIG_PRIORITY_INHERITANCE=y` added manually) either.
Although `b-g474e-dpow1` is STM32 and `tm4c129e-launchpad` is
pkarashchenko commented on code in PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181#discussion_r979408698
##
include/nuttx/compiler.h:
##
@@ -745,6 +745,69 @@
# define no_builtin(n)
+/* MSVC(Microsoft Visual C++)-specific definitions *
hartmannathan commented on PR #6318:
URL: https://github.com/apache/incubator-nuttx/pull/6318#issuecomment-1257196601
When you test with stm32f4, if the test passes, it will be a good idea to
compare defconfig and see if there are any differences that give a clue...
--
This is an automate
anchao commented on code in PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181#discussion_r979409478
##
include/nuttx/compiler.h:
##
@@ -745,6 +745,69 @@
# define no_builtin(n)
+/* MSVC(Microsoft Visual C++)-specific definitions **/
pkarashchenko commented on code in PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181#discussion_r979413120
##
include/nuttx/compiler.h:
##
@@ -745,6 +745,69 @@
# define no_builtin(n)
+/* MSVC(Microsoft Visual C++)-specific definitions *
hartmannathan opened a new pull request, #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183
## Summary
Expand the README.txt for `tm4c129e-launchpad`.
(The main motivation was to add the elusive `openocd` incantation to flash
program this board. However, I also add
hartmannathan commented on PR #7177:
URL: https://github.com/apache/incubator-nuttx/pull/7177#issuecomment-1257208645
@btashton, @gustavonihei, @pkarashchenko, thank you all for your input. The
changes I just pushed offer the following compromise:
(1) Removed 'sudo' from the 'openocd'
hartmannathan commented on PR #7177:
URL: https://github.com/apache/incubator-nuttx/pull/7177#issuecomment-1257210254
One moment, I forgot to squash commits...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
hartmannathan commented on PR #7177:
URL: https://github.com/apache/incubator-nuttx/pull/7177#issuecomment-1257210672
Okay, squashed. Ready for merge if you're happy with it. :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
This is an automated email from the ASF dual-hosted git repository.
btashton pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git
from 70290b6e38 arch: Change the linker generated symbols from uint32_t to
uint8_t *
add d129c95fd2 boards
btashton merged PR #7177:
URL: https://github.com/apache/incubator-nuttx/pull/7177
--
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...@n
anchao commented on code in PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181#discussion_r979419890
##
include/nuttx/compiler.h:
##
@@ -745,6 +745,69 @@
# define no_builtin(n)
+/* MSVC(Microsoft Visual C++)-specific definitions **/
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 d129c95fd2 boards/b-g474e-dpow1: Add note about udev/sudo regarding
openocd
add d71e8dacd4 sim/window
btashton commented on PR #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183#issuecomment-1257216244
You might find this PR of interest where I updated the docs for these and
then it got stuck
https://github.com/apache/incubator-nuttx/pull/3604
--
This is an automated mess
masayuki2009 merged PR #7182:
URL: https://github.com/apache/incubator-nuttx/pull/7182
--
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.
anchao opened a new pull request, #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184
## Summary
libs: fix visual studio Compiler Error C2124
D:\code\incubator-nuttx\libs\libc\stdlib\lib_strtod.c: error C2124: divide
or mod by zero
Windows MSVC restrictio
acassis commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257219647
Is it possible to change the CI script to try again when these kind of
errors show up:
```
Cloning into
'/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/tools'...
xiaoxiang781216 opened a new pull request, #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185
## Summary
to avoid the conflict with libuv's queue.h
## Impact
Code refactor
## Testing
Pass CI
--
This is an automated message from the Apache Git Service
xiaoxiang781216 commented on PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185#issuecomment-1257225358
Please ignore the mix case warning from arch/arm/src/phy62xx.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
xiaoxiang781216 commented on PR #1321:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/1321#issuecomment-1257227486
Let's close this PR to prefer
https://github.com/apache/incubator-nuttx/pull/7185
--
This is an automated message from the Apache Git Service.
To respond to the me
xiaoxiang781216 closed pull request #1321: libuv: Fix queue.h conflict with
nuttx verson
URL: https://github.com/apache/incubator-nuttx-apps/pull/1321
--
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
xiaoxiang781216 opened a new pull request, #1323:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1323
## Summary
## Impact
## Testing
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
pkarashchenko commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257255516
@acassis seems that it is not a matter of retry. I can't access that URL
from my laptop as well. Something is wrong with that repo
--
This is an automated message from the
pkarashchenko commented on PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185#issuecomment-1257256414
Seems like some files still need to be updated
```
In file included from wdog/wd_gettime.c:30:
Error: /github/workspace/sources/nuttx/sched/wdog/wdog.h:75:8: error:
btashton commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257257477
> @acassis seems that it is not a matter of retry. I can't access that URL
from my laptop as well. Something is wrong with that repo
Yeah looks like they are having a parti
pkarashchenko commented on code in PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185#discussion_r979451829
##
include/nuttx/wdog.h:
##
@@ -29,7 +29,6 @@
#include
#include
-#include
Review Comment:
Seems now we need to add `#include ` to `sched/wdog/w
pkarashchenko commented on code in PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185#discussion_r979451829
##
include/nuttx/wdog.h:
##
@@ -29,7 +29,6 @@
#include
#include
-#include
Review Comment:
Seems now we need to add `#include ` to `sched/wdog/w
xiaoxiang781216 commented on code in PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185#discussion_r979456101
##
include/nuttx/wdog.h:
##
@@ -29,7 +29,6 @@
#include
#include
-#include
Review Comment:
Done.
--
This is an automated message from the A
xiaoxiang781216 commented on PR #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183#issuecomment-1257274849
Let's ignore the false ci alarm.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
xiaoxiang781216 merged PR #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183
--
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
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 34192c07de boards/tm4c129e-launchpad
xiaoxiang781216 commented on PR #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183#issuecomment-1257275092
> You might find this PR of interest where I updated the docs for these and
then it got stuck #3604
@hartmannathan could you continue @btashton 's work?
--
This i
xiaoxiang781216 commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257275326
Let's ignore the false alarm.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
xiaoxiang781216 merged PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184
--
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
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 09c006b9c8 libs: fix visual studio C
xiaoxiang781216 commented on PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181#issuecomment-1257275435
Let's ignore the false alarm.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
xiaoxiang781216 merged PR #7181:
URL: https://github.com/apache/incubator-nuttx/pull/7181
--
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
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 74842880f9 compiler.h: add MSVC(Micr
hartmannathan commented on PR #7183:
URL: https://github.com/apache/incubator-nuttx/pull/7183#issuecomment-1257277429
> > You might find this PR of interest where I updated the docs for these
and then it got stuck #3604
>
> @hartmannathan could you continue @btashton 's work?
@
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 e93d78c Publishing web
ttnie opened a new pull request, #7186:
URL: https://github.com/apache/incubator-nuttx/pull/7186
Signed-off-by: nietingting
## Summary
enable pytest support on docker
## Impact
no
## Testing
test on local
--
This is an automated message from the Apache Git Servic
xiaoxiang781216 commented on PR #7186:
URL: https://github.com/apache/incubator-nuttx/pull/7186#issuecomment-1257382084
let's install in
https://github.com/apache/incubator-nuttx/blob/master/tools/ci/cibuild.sh too
--
This is an automated message from the Apache Git Service.
To respond to
CV-Bowen commented on PR #7131:
URL: https://github.com/apache/incubator-nuttx/pull/7131#issuecomment-1257384621
Delete `if (eventset)` and add this judgement to fs_poll.c
modified: drivers/net/tun.c
modified: drivers/sensors/sensor.c
modified: drivers/s
CV-Bowen commented on code in PR #7131:
URL: https://github.com/apache/incubator-nuttx/pull/7131#discussion_r979510556
##
drivers/serial/uart_bth4.c:
##
@@ -398,10 +383,10 @@ static int uart_bth4_poll(FAR struct file *filep, FAR
struct pollfd *fds,
if (!circbuf_is_empt
ttnie commented on PR #7186:
URL: https://github.com/apache/incubator-nuttx/pull/7186#issuecomment-1257429038
>
update done. pls help to review
--
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
SunJun8 opened a new pull request, #1324:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1324
## Summary
- Add client id field for dhcp discover and request
## Impact
- Maybe optimize dhcp server response
- unknown
## Testing
Pass CI
--
This is
xiaoxiang781216 commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257444381
How about we fork https://bitbucket.org/nuttx/tools.git to
https://github.com/NuttX?
--
This is an automated message from the Apache Git Service.
To respond to the messa
xiaoxiang781216 merged PR #7131:
URL: https://github.com/apache/incubator-nuttx/pull/7131
--
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
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 344c8be049 poll: add poll_notify() a
xiaoxiang781216 commented on PR #1324:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/1324#issuecomment-1257445523
@SunJun8 please fix the compiler error.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
btashton commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257453088
> How about we fork https://bitbucket.org/nuttx/tools.git to
https://github.com/NuttX? @btashton @pkarashchenko @acassis
This has come up before that https://github.com/Nu
xiaoxiang781216 commented on PR #7184:
URL: https://github.com/apache/incubator-nuttx/pull/7184#issuecomment-1257510475
But anyway we need one shared(not individual) and reliable place to host the
change can't upstream or tools which isn't maintained anymore.
--
This is an automated messa
xiaoxiang781216 merged PR #7186:
URL: https://github.com/apache/incubator-nuttx/pull/7186
--
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
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 8b3a936588 ci: enable pytest on dock
anchao opened a new pull request, #7187:
URL: https://github.com/apache/incubator-nuttx/pull/7187
## Summary
arm/armlink: add support of link time optimization(lto)
Optimization goal(Code size)
```
Smaller GCC(-Os) GCC(-flto -Os) CLANG(-flto -
pkarashchenko merged PR #1323:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1323
--
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
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 0dc5e9f46 Replace #include
pkarashchenko merged PR #7185:
URL: https://github.com/apache/incubator-nuttx/pull/7185
--
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
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 40ef5bc6db libc: Move queue.h fr
xiaoxiang781216 commented on PR #6987:
URL: https://github.com/apache/incubator-nuttx/pull/6987#issuecomment-1257543130
@zyfeier please rebase the patch to the last master.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
pkarashchenko commented on code in PR #7187:
URL: https://github.com/apache/incubator-nuttx/pull/7187#discussion_r979603370
##
arch/arm/src/common/Toolchain.defs:
##
@@ -220,9 +226,10 @@ ifneq ($(CONFIG_CXX_RTTI),y)
endif
ARCHOPTIMIZATION += -fno-common -Wall -Wshadow -Wunde
anchao commented on code in PR #7187:
URL: https://github.com/apache/incubator-nuttx/pull/7187#discussion_r979608681
##
arch/arm/src/common/Toolchain.defs:
##
@@ -220,9 +226,10 @@ ifneq ($(CONFIG_CXX_RTTI),y)
endif
ARCHOPTIMIZATION += -fno-common -Wall -Wshadow -Wundef
-ARCH
71 matches
Mail list logo
