[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#discussion_r907041715 ## include/endian.h: ## @@ -51,23 +51,40 @@ /* Common byte swapping macros */ -#define __SWAP_UINT16_ISMACRO 1 -#undef __SWAP_UINT32_ISMACRO - -#ifde

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
xiaoxiang781216 opened a new pull request, #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529 ## Summary instead of -ETIMEOUT, as specify here: https://pubs.opengroup.org/onlinepubs/009604599/functions/sendmsg.html https://man7.org/linux/man-pages/man2/sendfile.2.html

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
pkarashchenko commented on PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#issuecomment-1166978738 @xiaoxiang781216 looking into CI I see some warnings like ``` warning: (NET_TCP_NO_STACK && NET_USRSOCK_TCP) selects NET_TCP which has unmet direct dependencies (NET &&

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907064003 ## net/sixlowpan/sixlowpan_tcpsend.c: ## @@ -660,6 +660,11 @@ static int sixlowpan_send_packet(FAR struct socket *psock, ret = net_timedwait

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6502: risc-v: Move common memory map to its own file from riscv_internal

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6502: URL: https://github.com/apache/incubator-nuttx/pull/6502#discussion_r907065957 ## arch/risc-v/src/common/riscv_common_memorymap.h: ## @@ -0,0 +1,105 @@ +/ + *

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko commented on PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#issuecomment-1166993995 @xiaoxiang781216 please fix ``` tcp/tcp_send_unbuffered.c: In function 'psock_tcp_send': Error: tcp/tcp_send_unbuffered.c:683:31: error: 'ETIMEOUT' undeclared (first

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907068209 ## net/sixlowpan/sixlowpan_tcpsend.c: ## @@ -660,6 +660,11 @@ static int sixlowpan_send_packet(FAR struct socket *psock, ret = net_timedwait

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907064003 ## net/sixlowpan/sixlowpan_tcpsend.c: ## @@ -660,6 +660,11 @@ static int sixlowpan_send_packet(FAR struct socket *psock, ret = net_timedwait

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #4335: arch/bl602:Add bl602 os adapter layer

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #4335: URL: https://github.com/apache/incubator-nuttx/pull/4335#discussion_r907070104 ## arch/risc-v/src/bl602/bl602_os_hal.h: ## @@ -0,0 +1,186 @@ +/ + * arch/risc-

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6492: Stm32wl55JC SPI driver

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6492: URL: https://github.com/apache/incubator-nuttx/pull/6492#discussion_r907082047 ## arch/arm/src/stm32wl5/hardware/stm32wl5_spi.h: ## @@ -0,0 +1,277 @@ +/ + * a

[GitHub] [incubator-nuttx] normanr commented on a diff in pull request #4335: arch/bl602:Add bl602 os adapter layer

2022-06-27 Thread GitBox
normanr commented on code in PR #4335: URL: https://github.com/apache/incubator-nuttx/pull/4335#discussion_r907084014 ## arch/risc-v/src/bl602/bl602_netdev.c: ## @@ -2120,10 +2122,12 @@ void bl602_net_event(int evt, int val) case CODE_WIFI_ON_DISCONNECT: do

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
xiaoxiang781216 commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907105001 ## net/sixlowpan/sixlowpan_tcpsend.c: ## @@ -660,6 +660,11 @@ static int sixlowpan_send_packet(FAR struct socket *psock, ret = net_timedwa

[GitHub] [incubator-nuttx] adamkaliszan commented on pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
adamkaliszan commented on PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#issuecomment-1167047984 The commit is fine. I have decided to withdrawn possibility of configuration LCD Command/Data pin because It is necessary to modify all boards.h files. Possibility of configu

[GitHub] [incubator-nuttx] SunJun8 commented on a diff in pull request #4335: arch/bl602:Add bl602 os adapter layer

2022-06-27 Thread GitBox
SunJun8 commented on code in PR #4335: URL: https://github.com/apache/incubator-nuttx/pull/4335#discussion_r907128491 ## arch/risc-v/src/bl602/bl602_netdev.c: ## @@ -2120,10 +2122,12 @@ void bl602_net_event(int evt, int val) case CODE_WIFI_ON_DISCONNECT: do

[GitHub] [incubator-nuttx] SunJun8 commented on a diff in pull request #4335: arch/bl602:Add bl602 os adapter layer

2022-06-27 Thread GitBox
SunJun8 commented on code in PR #4335: URL: https://github.com/apache/incubator-nuttx/pull/4335#discussion_r907129857 ## arch/risc-v/src/bl602/bl602_netdev.c: ## @@ -2120,10 +2122,12 @@ void bl602_net_event(int evt, int val) case CODE_WIFI_ON_DISCONNECT: do

[GitHub] [incubator-nuttx] SunJun8 commented on pull request #4335: arch/bl602:Add bl602 os adapter layer

2022-06-27 Thread GitBox
SunJun8 commented on PR #4335: URL: https://github.com/apache/incubator-nuttx/pull/4335#issuecomment-1167062787 @pkarashchenko thanks for your review, and I will update this part of the code in due course -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907187495 ## net/icmp/icmp_sendmsg.c: ## @@ -424,6 +424,8 @@ ssize_t icmp_sendmsg(FAR struct socket *psock, FAR struct msghdr *msg, { if (ret ==

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
xiaoxiang781216 commented on code in PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529#discussion_r907216655 ## net/icmpv6/icmpv6_sendmsg.c: ## @@ -415,6 +415,8 @@ ssize_t icmpv6_sendmsg(FAR struct socket *psock, FAR struct msghdr *msg, { if

[GitHub] [incubator-nuttx] pussuw opened a new pull request, #6530: mpfs: Fix IHC memory locations to native width type

2022-06-27 Thread GitBox
pussuw opened a new pull request, #6530: URL: https://github.com/apache/incubator-nuttx/pull/6530 Ne numeric type defaults to u32 which is not enough to represent a native memory location This fixes build error: https://github.com/apache/incubator-nuttx/runs/7067877053?check_sui

[GitHub] [incubator-nuttx] pussuw commented on pull request #6502: risc-v: Move common memory map to its own file from riscv_internal

2022-06-27 Thread GitBox
pussuw commented on PR #6502: URL: https://github.com/apache/incubator-nuttx/pull/6502#issuecomment-1167182707 The fix for the build error is here: https://github.com/apache/incubator-nuttx/pull/6530 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-nuttx] pussuw commented on a diff in pull request #6502: risc-v: Move common memory map to its own file from riscv_internal

2022-06-27 Thread GitBox
pussuw commented on code in PR #6502: URL: https://github.com/apache/incubator-nuttx/pull/6502#discussion_r907237841 ## arch/risc-v/src/common/riscv_common_memorymap.h: ## @@ -0,0 +1,105 @@ +/ + * arch/r

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6502: risc-v: Move common memory map to its own file from riscv_internal

2022-06-27 Thread GitBox
pkarashchenko commented on code in PR #6502: URL: https://github.com/apache/incubator-nuttx/pull/6502#discussion_r907258126 ## arch/risc-v/src/common/riscv_common_memorymap.h: ## @@ -0,0 +1,105 @@ +/ + *

[GitHub] [incubator-nuttx] curuvar commented on pull request #6528: Added RP2040 Boards: Adafruit KB2040 and Adafruit Feather RP2040

2022-06-27 Thread GitBox
curuvar commented on PR #6528: URL: https://github.com/apache/incubator-nuttx/pull/6528#issuecomment-1167273028 After thinking about this, I am going to change my driver so it works the same (from the users point of view) as ws2812 and combine the two into the same file. That way, any exis

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907324042 ## boards/xtensa/esp32/common/Kconfig: ## @@ -77,10 +77,6 @@ config SSD1680_GPIO_PIN_CS int "Pin that select the chip on SPI bus" default "5

[GitHub] [incubator-nuttx] gustavonihei commented on pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#issuecomment-1167289377 > The commit is fine. I have decided to withdrawn possibility of configuration LCD Command/Data pin. The reason is following: it is necessary to modify all boards.h files in

[GitHub] [incubator-nuttx] adamkaliszan commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
adamkaliszan commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907327484 ## boards/xtensa/esp32/common/src/esp32_board_spi.c: ## @@ -71,7 +71,7 @@ static inline uint8_t spi_status(struct spi_dev_s *dev, uint32_t devid) static i

[GitHub] [incubator-nuttx] adamkaliszan commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
adamkaliszan commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907329170 ## boards/xtensa/esp32/common/Kconfig: ## @@ -77,10 +77,6 @@ config SSD1680_GPIO_PIN_CS int "Pin that select the chip on SPI bus" default "5

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907336480 ## boards/xtensa/esp32/common/src/esp32_ssd1680.c: ## @@ -105,9 +105,8 @@ int board_lcd_initialize(void) /* Initialize additional I/O for e-ink display

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907336480 ## boards/xtensa/esp32/common/src/esp32_ssd1680.c: ## @@ -105,9 +105,8 @@ int board_lcd_initialize(void) /* Initialize additional I/O for e-ink display

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907344567 ## boards/xtensa/esp32/common/src/esp32_ssd1680.c: ## @@ -105,9 +105,8 @@ int board_lcd_initialize(void) /* Initialize additional I/O for e-ink display

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907344567 ## boards/xtensa/esp32/common/src/esp32_ssd1680.c: ## @@ -105,9 +105,8 @@ int board_lcd_initialize(void) /* Initialize additional I/O for e-ink display

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6530: mpfs: Fix IHC memory locations to native width type

2022-06-27 Thread GitBox
xiaoxiang781216 merged PR #6530: URL: https://github.com/apache/incubator-nuttx/pull/6530 -- 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: mpfs: Fix IHC memory locations to native width type

2022-06-27 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 77a01cfe52 mpfs: Fix IHC memory loca

[GitHub] [incubator-nuttx] hartmannathan commented on a diff in pull request #6512: driver/syslog:sched lock in syslog to prevent garbled print from multi-thread

2022-06-27 Thread GitBox
hartmannathan commented on code in PR #6512: URL: https://github.com/apache/incubator-nuttx/pull/6512#discussion_r907397184 ## drivers/syslog/Kconfig: ## @@ -373,4 +373,13 @@ config SYSLOG_CHARDEV byte output with no time-stamping or any other SYSLOG features

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907410201 ## .gitignore: ## @@ -31,3 +31,5 @@ Make.dep .context # nuttx/$(TOPDIR)/Makefile.[unix|win]::$(DIRLINKS_EXTERNAL_DIRS) .dirlinks +.vscode +.DS_Stor

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907410436 ## examples/neopixel/Make.defs: ## @@ -0,0 +1,23 @@ + +# apps/examples/hel

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907412611 ## examples/neopixel/Kconfig: ## @@ -0,0 +1,33 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfig-language.t

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907413606 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907415347 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907416260 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907417764 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907418201 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907418899 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907419232 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx-apps] gustavonihei commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907426496 ## examples/neopixel/neopixel_main.c: ## @@ -0,0 +1,301 @@ +/ + * apps/exa

[GitHub] [incubator-nuttx] edsut opened a new issue, #6531: IMXRT USB speed may need to be forced to FULL when CONFIG_USBDEV_DUALSPEED is not configured

2022-06-27 Thread GitBox
edsut opened a new issue, #6531: URL: https://github.com/apache/incubator-nuttx/issues/6531 I am working with a Teensy4.1 and had a consistent hardfault whenever I connected QGC to Mavlink over USB. I eventually realized that I did not have CONFIG_USBDEV_DUALSPEED enabled, but line 1975 of

[GitHub] [incubator-nuttx] acassis commented on pull request #6528: Added RP2040 Boards: Adafruit KB2040 and Adafruit Feather RP2040

2022-06-27 Thread GitBox
acassis commented on PR #6528: URL: https://github.com/apache/incubator-nuttx/pull/6528#issuecomment-1167454046 @curuvar currently there is only a board using @btashton original WS2812 driver, so I think you can modify his driver to include your improvement. Also because there are many RBG

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #6528: Added RP2040 Boards: Adafruit KB2040 and Adafruit Feather RP2040

2022-06-27 Thread GitBox
acassis commented on code in PR #6528: URL: https://github.com/apache/incubator-nuttx/pull/6528#discussion_r907485738 ## boards/arm/rp2040/adafruit-feather/README.txt: ## @@ -0,0 +1,180 @@ +README +== + +This directory contains the port of NuttX to the Adafruit Feather RP204

[GitHub] [incubator-nuttx] acassis commented on pull request #6528: Added RP2040 Boards: Adafruit KB2040 and Adafruit Feather RP2040

2022-06-27 Thread GitBox
acassis commented on PR #6528: URL: https://github.com/apache/incubator-nuttx/pull/6528#issuecomment-1167470025 Nice work @curuvar -- 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-apps] curuvar commented on a diff in pull request #1211: Added Neopixel Example

2022-06-27 Thread GitBox
curuvar commented on code in PR #1211: URL: https://github.com/apache/incubator-nuttx-apps/pull/1211#discussion_r907629441 ## .gitignore: ## @@ -31,3 +31,5 @@ Make.dep .context # nuttx/$(TOPDIR)/Makefile.[unix|win]::$(DIRLINKS_EXTERNAL_DIRS) .dirlinks +.vscode +.DS_Store Re

[GitHub] [incubator-nuttx] btashton commented on pull request #6528: Added RP2040 Boards: Adafruit KB2040 and Adafruit Feather RP2040

2022-06-27 Thread GitBox
btashton commented on PR #6528: URL: https://github.com/apache/incubator-nuttx/pull/6528#issuecomment-1167689285 Yeah there is at least one other person that modified it a bit to support a non in-tree board but I fully support making changes to improve this even if it does mean changing the

[GitHub] [incubator-nuttx] nimish opened a new pull request, #6532: Remove '-g' LDFLAG

2022-06-27 Thread GitBox
nimish opened a new pull request, #6532: URL: https://github.com/apache/incubator-nuttx/pull/6532 ## Summary `-g` is either ignored (gnu linker, lld) or throws an error (mold). Either way, it doesn't do anything and it's unclear why it's being added. ## Impact `mold` doesn't

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
xiaoxiang781216 commented on PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#issuecomment-1167724409 @nimish please squash the second patch into the first. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6532: Remove '-g' LDFLAG

2022-06-27 Thread GitBox
xiaoxiang781216 commented on code in PR #6532: URL: https://github.com/apache/incubator-nuttx/pull/6532#discussion_r907680342 ## boards/renesas/m16c/skp16c26/scripts/Make.defs: ## @@ -50,5 +50,4 @@ CPPFLAGS = $(ARCHINCLUDES) $(ARCHDEFINES) $(EXTRAFLAGS) AFLAGS

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
xiaoxiang781216 commented on PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#issuecomment-1167728092 > @xiaoxiang781216 looking into CI I see some warnings like > > ``` > warning: (NET_TCP_NO_STACK && NET_USRSOCK_TCP) selects NET_TCP which has unmet direct depend

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
xiaoxiang781216 commented on PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#issuecomment-1167728916 > @nimish please squash the second patch into the first. and the style warning: ``` ../nuttx/tools/checkpatch.sh -m -g 4c2b7f3ac9052d50aa1c8fa20e643ce4bbf639c6

[GitHub] [incubator-nuttx] nimish commented on a diff in pull request #6532: Remove '-g' LDFLAG

2022-06-27 Thread GitBox
nimish commented on code in PR #6532: URL: https://github.com/apache/incubator-nuttx/pull/6532#discussion_r907687397 ## arch/xtensa/src/lx7/Toolchain.defs: ## @@ -101,8 +101,7 @@ ifeq ($(CONFIG_DEBUG_LINK_MAP),y) endif ifeq ($(CONFIG_DEBUG_SYMBOLS),y) - LDFLAGS +=

[GitHub] [incubator-nuttx] gustavonihei commented on a diff in pull request #6511: esp32_board_spi:Missing Data Command pin support

2022-06-27 Thread GitBox
gustavonihei commented on code in PR #6511: URL: https://github.com/apache/incubator-nuttx/pull/6511#discussion_r907691147 ## boards/xtensa/esp32/common/src/esp32_ssd1680.c: ## @@ -105,9 +105,8 @@ int board_lcd_initialize(void) /* Initialize additional I/O for e-ink display

[GitHub] [incubator-nuttx] acassis merged pull request #6532: Remove '-g' LDFLAG

2022-06-27 Thread GitBox
acassis merged PR #6532: URL: https://github.com/apache/incubator-nuttx/pull/6532 -- 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...@nu

[incubator-nuttx] branch master updated: this flag is meaningless for the linker

2022-06-27 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis 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 4afd25b567 this flag is meaningless fo

[GitHub] [incubator-nuttx] acassis commented on pull request #6525: drivers: add support of pwm capture driver

2022-06-27 Thread GitBox
acassis commented on PR #6525: URL: https://github.com/apache/incubator-nuttx/pull/6525#issuecomment-1168021659 Very nice @zouboan !!! Are you planing to add an app/ example to use /dev/capture0 ? Is it able to read PWM pulse used to control servo motors? I'm thinking about connecting a PWM

[GitHub] [incubator-nuttx] zouboan commented on pull request #6525: drivers: add support of pwm capture driver

2022-06-27 Thread GitBox
zouboan commented on PR #6525: URL: https://github.com/apache/incubator-nuttx/pull/6525#issuecomment-1168036803 Ok,i will add a capture example in apps ---Original--- From: "Alan Carvalho de ***@***.***> Date: Tue, Jun 28, 2022 07:10 AM To: ***@***.***>; Cc: ***@***

[incubator-nuttx-website] branch asf-site updated: Publishing web: 3c980bc717a3874c683a1bd2a48c64c33af05833 docs: 4afd25b567caeb477bace914535dbe11afd12597

2022-06-27 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 3428560 Publishing web

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6482: Use builtins for byteswapping

2022-06-27 Thread GitBox
xiaoxiang781216 commented on PR #6482: URL: https://github.com/apache/incubator-nuttx/pull/6482#issuecomment-1168147311 @nimish The inclusion of endian/Make.defs need be removed too: ``` Makefile:30: endian/Make.defs: No such file or directory ``` -- This is an automated message f

[incubator-nuttx] branch master updated (4afd25b567 -> 00e8e4fa28)

2022-06-27 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 4afd25b567 this flag is meaningless for the linker new fd8eaf4f42 arch/stm32_capture_lowerhalf.c: add

[incubator-nuttx] 01/03: arch/stm32_capture_lowerhalf.c: add lower half support of capture

2022-06-27 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 fd8eaf4f42a1a230fb2af1de4502093b414ceeed Author: zouboan AuthorDate: Sun Jun 26 19:52:36 2022 +0800 arc

[incubator-nuttx] 02/03: drivers/timers/capture.c: add support of pwm capture driver

2022-06-27 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 b41929522c9d86d35cec9cf80665bfeab49750ee Author: zouboan AuthorDate: Sun Jun 26 20:00:38 2022 +0800 dri

[incubator-nuttx] 03/03: boards/stm32f4discovery: add setup of pwm capture device

2022-06-27 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 00e8e4fa28d957a257c94784235887e282220e01 Author: zouboan AuthorDate: Sun Jun 26 20:54:57 2022 +0800 boa

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6525: drivers: add support of pwm capture driver

2022-06-27 Thread GitBox
xiaoxiang781216 merged PR #6525: URL: https://github.com/apache/incubator-nuttx/pull/6525 -- 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 merged pull request #6492: Stm32wl55JC SPI driver

2022-06-27 Thread GitBox
xiaoxiang781216 merged PR #6492: URL: https://github.com/apache/incubator-nuttx/pull/6492 -- 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: Spi driver for Stm32wl55

2022-06-27 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 4f8c8815d6 Spi driver for Stm32wl55

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6510: Support for 2.13 inch display v2

2022-06-27 Thread GitBox
xiaoxiang781216 commented on PR #6510: URL: https://github.com/apache/incubator-nuttx/pull/6510#issuecomment-1168149374 @adamkaliszan will you fix the rest comment from @pkarashchenko ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-nuttx] adamkaliszan commented on a diff in pull request #6510: Support for 2.13 inch display v2

2022-06-27 Thread GitBox
adamkaliszan commented on code in PR #6510: URL: https://github.com/apache/incubator-nuttx/pull/6510#discussion_r907984008 ## drivers/lcd/ssd1680.c: ## @@ -878,19 +896,30 @@ static void ssd1680_select(FAR struct ssd1680_dev_s *priv, bool cs) } } -static void ssd1680_bus

[incubator-nuttx] branch master updated: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 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 abc72ad128 net: Ensure sendmsg a

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #6529: net: Ensure sendmsg and sendfile return -EAGAIN in case of timeout

2022-06-27 Thread GitBox
pkarashchenko merged PR #6529: URL: https://github.com/apache/incubator-nuttx/pull/6529 -- 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] anchao opened a new pull request, #6533: wireless/bcm43xxx: enable tx flow control to improve performance

2022-06-27 Thread GitBox
anchao opened a new pull request, #6533: URL: https://github.com/apache/incubator-nuttx/pull/6533 ## Summary 1. wireless/bcm43xxx: enable tx flow control to improve performance RX/TX shared free queue on bcmf implementation, if TX occupies the free queue completely, RX will t

[GitHub] [incubator-nuttx] XinStellaris commented on pull request #6512: driver/syslog:sched lock in syslog to prevent garbled print from multi-thread

2022-06-27 Thread GitBox
XinStellaris commented on PR #6512: URL: https://github.com/apache/incubator-nuttx/pull/6512#issuecomment-1168281601 After all the discussion, sched_lock or critical section in this pr will impact real-time behavior. So I will change how the application calls syslog, by spliting long str

[GitHub] [incubator-nuttx] XinStellaris closed pull request #6512: driver/syslog:sched lock in syslog to prevent garbled print from multi-thread

2022-06-27 Thread GitBox
XinStellaris closed pull request #6512: driver/syslog:sched lock in syslog to prevent garbled print from multi-thread URL: https://github.com/apache/incubator-nuttx/pull/6512 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us