[incubator-nuttx] branch master updated: stm32f103re has a dac module.but the pinmap file has no corresponding definition

2022-02-26 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 fac10b6 stm32f103re has a dac modul

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5627: stm32f103re has a dac module.but the pinmap file has no corresponding…

2022-02-26 Thread GitBox
xiaoxiang781216 merged pull request #5627: URL: https://github.com/apache/incubator-nuttx/pull/5627 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815395056 ## File path: drivers/input/ajoystick.c ## @@ -686,7 +612,8 @@ static int ajoy_poll(FAR struct file *filep, FAR struct pollfd *fds, FA

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815394964 ## File path: drivers/input/ajoystick.c ## @@ -181,12 +162,6 @@ static void ajoy_enable(FAR struct ajoy_upperhalf_s *priv) lower = priv

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815394931 ## File path: drivers/input/djoystick.c ## @@ -181,12 +162,6 @@ static void djoy_enable(FAR struct djoy_upperhalf_s *priv) lower = priv

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815394817 ## File path: drivers/input/ajoystick.c ## @@ -526,12 +461,7 @@ static ssize_t ajoy_read(FAR struct file *filep, FAR char *buffer, /

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815394775 ## File path: drivers/input/djoystick.c ## @@ -524,12 +459,7 @@ static ssize_t djoy_read(FAR struct file *filep, FAR char *buffer, /

[GitHub] [incubator-nuttx] anchao opened a new pull request #5628: arm/armv7-a/r: handle swi on interrupt stack

2022-02-26 Thread GitBox
anchao opened a new pull request #5628: URL: https://github.com/apache/incubator-nuttx/pull/5628 ## Summary arm/armv7-a/r: handle swi on interrupt stack arm/armv7-a/r: do not set fiq bit on dataabort ## Impact armv7-a/r smp ## Testing sabre-6quad/smp , s

[GitHub] [incubator-nuttx] lipan118 opened a new pull request #5627: stm32f103re has a dac module.but the pinmap file has no corresponding…

2022-02-26 Thread GitBox
lipan118 opened a new pull request #5627: URL: https://github.com/apache/incubator-nuttx/pull/5627 … definition add dac pin definition in stm32f103r pinmap header file ## Summary stm32f103re has a dac module.but the pinmap file has no corresponding definition ## Impact stm32f

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

2022-02-26 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 3464358 Publishing web

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
pkarashchenko commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815356949 ## File path: drivers/input/ajoystick.c ## @@ -526,12 +461,7 @@ static ssize_t ajoy_read(FAR struct file *filep, FAR char *buffer, /*

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5625: joystick: Remove [a|d]joy_enable from [a|d]joy_sample

2022-02-26 Thread GitBox
xiaoxiang781216 commented on pull request #5625: URL: https://github.com/apache/incubator-nuttx/pull/5625#issuecomment-1052606100 > @xiaoxiang781216 will it support the case with a device have more than one joystick, like the joysticks with two analog XY potentiometer or even analog + digi

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #1032: pwm: Fix issue with directing to "> /dev/null"

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #1032: URL: https://github.com/apache/incubator-nuttx-apps/pull/1032#discussion_r815352954 ## File path: examples/pwm/pwm_main.c ## @@ -307,6 +307,13 @@ static void parse_args(FAR struct pwm_state_s *pwm, int argc, for

[GitHub] [incubator-nuttx] acassis commented on pull request #5624: joystick: Fix the event lose between the invocation of poll

2022-02-26 Thread GitBox
acassis commented on pull request #5624: URL: https://github.com/apache/incubator-nuttx/pull/5624#issuecomment-1052595635 @xiaoxiang781216 please do real test. It will avoid we receive complaint from users saying something is not working :-) -- This is an automated message from the Apach

[GitHub] [incubator-nuttx] acassis commented on pull request #5625: joystick: Remove [a|d]joy_enable from [a|d]joy_sample

2022-02-26 Thread GitBox
acassis commented on pull request #5625: URL: https://github.com/apache/incubator-nuttx/pull/5625#issuecomment-1052594450 @xiaoxiang781216 will it support the case with a device have more than one joystick, like the joysticks with two analog XY potentiometer or even analog + digital in the

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r815352567 ## File path: drivers/input/ajoystick.c ## @@ -412,45 +407,14 @@ static int ajoy_close(FAR struct file *filep) FAR struct ajoy_open_s *

[GitHub] [incubator-nuttx-apps] acassis opened a new pull request #1032: pwm: Fix issue with directing to "> /dev/null"

2022-02-26 Thread GitBox
acassis opened a new pull request #1032: URL: https://github.com/apache/incubator-nuttx-apps/pull/1032 ## Summary Fix issue with directing to "> /dev/null" ## Impact Solve issue: https://github.com/apache/incubator-nuttx/issues/5626 ## Testing Running: nsh> pwm -p /dev/pwm1

[GitHub] [incubator-nuttx] acassis opened a new issue #5626: Redirects doesn't work with programs like pwm

2022-02-26 Thread GitBox
acassis opened a new issue #5626: URL: https://github.com/apache/incubator-nuttx/issues/5626 I was debugging pwm and to get rid of mixed messages from pwm application and driver debug messages I decided to use "> /dev/null" but for some reason it just returns without executing: ```

[GitHub] [incubator-nuttx] acassis commented on issue #5586: Building System doesn't work case there are space or special characters in the directory name

2022-02-26 Thread GitBox
acassis commented on issue #5586: URL: https://github.com/apache/incubator-nuttx/issues/5586#issuecomment-1052540536 Yes, maybe it is better to let configure.c to take care of it, instead of using bash, this way it also could work better among OSes that doesn't support bash. -- This is

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#issuecomment-1052491471 > I found a similar issue in `drivers/input/button_upper.c` Yes, I will fix button_upper too. -- This is an automated message from the Apache Git Servic

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5204: input/ajoystck: Always protect the au_open list by critical section

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5204: URL: https://github.com/apache/incubator-nuttx/pull/5204#discussion_r81538 ## File path: drivers/input/ajoystick.c ## @@ -331,10 +327,6 @@ static void ajoy_sample(FAR struct ajoy_upperhalf_s *priv) }

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5625: joystick: Remove [a|d]joy_enable from [a|d]joy_sample

2022-02-26 Thread GitBox
xiaoxiang781216 opened a new pull request #5625: URL: https://github.com/apache/incubator-nuttx/pull/5625 ## Summary Since the enable status keep stable in [a|d]joy_sample. - input/djoystick: Remove the unused config field from djoy_lowerhalf_s - input/ajoystck: Remove ajoy_enable

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5624: joystick: Fix the event lose between the invocation of poll

2022-02-26 Thread GitBox
xiaoxiang781216 opened a new pull request #5624: URL: https://github.com/apache/incubator-nuttx/pull/5624 ## Summary - input/ajoystick: Fix the event lose between the invocation of poll - input/djoystick: Fix the event lose between the invocation of poll ## Impact joysti

[GitHub] [incubator-nuttx] acassis commented on pull request #5373: tcp: added debug asserts and logging to investigate the rare (conn->dev == NULL) bug in callback handlers

2022-02-26 Thread GitBox
acassis commented on pull request #5373: URL: https://github.com/apache/incubator-nuttx/pull/5373#issuecomment-1052156695 @a-lunev sorry my delay, I missed your fix! -- 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] acassis merged pull request #5373: tcp: added debug asserts and logging to investigate the rare (conn->dev == NULL) bug in callback handlers

2022-02-26 Thread GitBox
acassis merged pull request #5373: URL: https://github.com/apache/incubator-nuttx/pull/5373 -- 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: commit

[incubator-nuttx] branch master updated: tcp: added debug asserts and logging to investigate the rare (conn->dev == NULL) bug in callback handlers

2022-02-26 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 404ceff tcp: added debug asserts and

[GitHub] [incubator-nuttx] acassis merged pull request #5499: Add EPSON RX8010SJ I2C RTC timer support

2022-02-26 Thread GitBox
acassis merged pull request #5499: URL: https://github.com/apache/incubator-nuttx/pull/5499 -- 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: commit

[incubator-nuttx] branch master updated: Add EPSON RX8010SJ I2C RTC timer support

2022-02-26 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 fb6f058 Add EPSON RX8010SJ I2C RTC ti

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5612: boards/arm/stm32: Update adb configs

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5612: URL: https://github.com/apache/incubator-nuttx/pull/5612#discussion_r815316138 ## File path: boards/arm/stm32/stm32f4discovery/scripts/Make.defs ## @@ -34,13 +34,15 @@ ifneq ($(CONFIG_DEBUG_NOOPT),y) ARCHOPTIMIZATI

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5612: boards/arm/stm32: Update adb configs

2022-02-26 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5612: URL: https://github.com/apache/incubator-nuttx/pull/5612#discussion_r815313571 ## File path: boards/arm/stm32/stm32f4discovery/scripts/Make.defs ## @@ -54,6 +54,9 @@ else endif +ARCHCFLAGS += -ffunction-sections

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5612: boards/arm/stm32: Update adb configs

2022-02-26 Thread GitBox
no1wudi commented on a change in pull request #5612: URL: https://github.com/apache/incubator-nuttx/pull/5612#discussion_r815312179 ## File path: boards/arm/stm32/stm32f4discovery/configs/adb/defconfig ## @@ -44,21 +44,25 @@ CONFIG_INTELHEX_BINARY=y CONFIG_LIBC_ENVPATH=y CONF

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #854: system/libuv: Bump to v1.42.0

2022-02-26 Thread GitBox
xiaoxiang781216 commented on pull request #854: URL: https://github.com/apache/incubator-nuttx-apps/pull/854#issuecomment-1052121764 This PR need the patch which remove -D__NUTTX__ from system/adb/Makefile to pass CI. -- This is an automated message from the Apache Git Service. To respo

[incubator-nuttx] branch master updated: arch/sim: Don't include execinfo.h since cygwin doesn't support it

2022-02-26 Thread gnutt
This is an automated email from the ASF dual-hosted git repository. gnutt 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 f634ea2 arch/sim: Don't include execinf

[GitHub] [incubator-nuttx] patacongo merged pull request #5623: arch/sim: Don't include execinfo.h since cygwin doesn't support it

2022-02-26 Thread GitBox
patacongo merged pull request #5623: URL: https://github.com/apache/incubator-nuttx/pull/5623 -- 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: comm

[GitHub] [incubator-nuttx] patacongo closed issue #5621: Simulator build fails on non-GLIBC platform

2022-02-26 Thread GitBox
patacongo closed issue #5621: URL: https://github.com/apache/incubator-nuttx/issues/5621 -- 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] patacongo commented on pull request #5623: arch/sim: Don't include execinfo.h since cygwin doesn't support it

2022-02-26 Thread GitBox
patacongo commented on pull request #5623: URL: https://github.com/apache/incubator-nuttx/pull/5623#issuecomment-1052119679 > @patacongo could you try this patch? Looks good. This is essential the same as my work around. I would have submitted a PR, but I wasn't sure if this was th