[GitHub] [incubator-nuttx] yamt commented on pull request #4881: libc: Add wctoxk and xktowc for the conversion between X11 keysym and wchar_t

2021-12-09 Thread GitBox
yamt commented on pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#issuecomment-989603660 > > > > > > it seems a bit awkward to have x11 stuff in libc. is there a precedent? (in nuttx or other systems) > > > > > > > > > > > > > > > The following item d

[GitHub] [incubator-nuttx-apps] OctopusHu0105 opened a new pull request #916: trace dump: add string and binary

2021-12-09 Thread GitBox
OctopusHu0105 opened a new pull request #916: URL: https://github.com/apache/incubator-nuttx-apps/pull/916 trace dump: add string and binary NOTE_DUMP_STRING NOTE_DUMP_BINARY And incubator-nuttx[id ddfdb3fc0] is together Change-Id: Id41072f161ad9d95d76915155ac33ab7240

[GitHub] [incubator-nuttx] anchao commented on a change in pull request #4957: cortex-m/hardfault: add usage/bus-fault handler

2021-12-09 Thread GitBox
anchao commented on a change in pull request #4957: URL: https://github.com/apache/incubator-nuttx/pull/4957#discussion_r765553106 ## File path: arch/arm/src/armv7-m/arm_busfault.c ## @@ -0,0 +1,107 @@ +/**

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-989647407 @YuuichiNakamura could you take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-nuttx] anchao opened a new pull request #4965: arm/cortex-m23: remove mem-fault select from armv8-m baseline

2021-12-09 Thread GitBox
anchao opened a new pull request #4965: URL: https://github.com/apache/incubator-nuttx/pull/4965 ## Summary arm/cortex-m23: armv8-m baseline do not support mem-fault Signed-off-by: chao.an ## Impact N/A ## Testing compile pass -- This is an automated me

[incubator-nuttx] branch master updated: cortex-m/fault: add CFSR(Configurable Fault Status Register) Definitions

2021-12-09 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 3e812dd cortex-m/fault: add CFSR(Co

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4955: cortex-m/fault: add CFSR(Configurable Fault Status Register) Definitions

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4955: URL: https://github.com/apache/incubator-nuttx/pull/4955 -- 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] masayuki2009 commented on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-989751797 @xiaoxiang781216 Hmm, the following commit has a problem with uSD card on Spresense. If I revert the commit locally, it works fine. ``` nsh> df

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-989756816 >@xiaoxiang781216 > >Hmm, the following commit has a problem with uSD card on Spresense. stm32f4discovery with uSD over SPI has the same problem too.

[GitHub] [incubator-nuttx] masayuki2009 edited a comment on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 edited a comment on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-989756816 >@xiaoxiang781216 > >Hmm, the following commit has a problem with uSD card on Spresense. @xiaoxiang781216 stm32f4discovery with uSD over SPI h

[incubator-nuttx] branch master updated: arm/cortex-m23: armv8-m baseline do not support mem-fault

2021-12-09 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 99fa58c arm/cortex-m23: armv8-m bas

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4965: arm/cortex-m23: remove mem-fault select from armv8-m baseline

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4965: URL: https://github.com/apache/incubator-nuttx/pull/4965 -- 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 pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-989776884 Ok, we will take a look tonight. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-nuttx] masayuki2009 commented on issue #3812: vfork() is broken in PROTECTED mode.

2021-12-09 Thread GitBox
masayuki2009 commented on issue #3812: URL: https://github.com/apache/incubator-nuttx/issues/3812#issuecomment-989787028 @patacongo I think the regression happened on the master branch around June 2021. Actually, both the latest master branch and the release/10.2 branch work fine.

[GitHub] [incubator-nuttx] patacongo commented on pull request #4881: libc: Add wctoxk and xktowc for the conversion between X11 keysym and wchar_t

2021-12-09 Thread GitBox
patacongo commented on pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#issuecomment-989874591 > it seems a bit awkward to have x11 stuff in libc. I agree. Not just awkward but wrong. The content of libc is controlled by POSIX/ANSI standards (plus glibc

[GitHub] [incubator-nuttx] patacongo commented on issue #3812: vfork() is broken in PROTECTED mode.

2021-12-09 Thread GitBox
patacongo commented on issue #3812: URL: https://github.com/apache/incubator-nuttx/issues/3812#issuecomment-989877372 > @patacongo I think the regression happened on the master branch around June 2021. Actually, both the latest master branch and the release/10.2 branch work fine. O

[GitHub] [incubator-nuttx] patacongo closed issue #3812: vfork() is broken in PROTECTED mode.

2021-12-09 Thread GitBox
patacongo closed issue #3812: URL: https://github.com/apache/incubator-nuttx/issues/3812 -- 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] davids5 commented on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
davids5 commented on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-989925286 @OctopusHu0105 - what it the use case and motivation for these? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-nuttx] jarivanewijk commented on pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#issuecomment-989972732 I finally was able to get back to this. I have addressed all of your comments. I have also added additional debugasserts to ensure everything is implemented as inte

[GitHub] [incubator-nuttx-apps] jarivanewijk opened a new pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

2021-12-09 Thread GitBox
jarivanewijk opened a new pull request #917: URL: https://github.com/apache/incubator-nuttx-apps/pull/917 ## Summary https://github.com/apache/incubator-nuttx/pull/4774 changes the naming of GPIO devices. Instead of separate gpout, gpin and gpint devices they are all named /dev/gpioN no

[GitHub] [incubator-nuttx-apps] jarivanewijk opened a new pull request #918: GPIO example: Add support for changing pintypes

2021-12-09 Thread GitBox
jarivanewijk opened a new pull request #918: URL: https://github.com/apache/incubator-nuttx-apps/pull/918 ## Summary Extension to the existing GPIO example to allow changing pintypes. This assumes that the lower half GPIO driver implements the setpintype command. ## Impact This

[GitHub] [incubator-nuttx] jarivanewijk opened a new pull request #4966: timer-gpout example renamed to timer-gpio

2021-12-09 Thread GitBox
jarivanewijk opened a new pull request #4966: URL: https://github.com/apache/incubator-nuttx/pull/4966 ## Summary See https://github.com/apache/incubator-nuttx-apps/pull/917 ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respo

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #918: GPIO example: Add support for changing pintypes

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #918: URL: https://github.com/apache/incubator-nuttx-apps/pull/918 -- 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-m

[incubator-nuttx-apps] branch master updated: GPIO example application: Add support for changing pintypes

2021-12-09 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 71f5af4 GPIO example applicati

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4957: cortex-m/hardfault: add usage/bus-fault handler

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4957: URL: https://github.com/apache/incubator-nuttx/pull/4957 -- 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

[incubator-nuttx] 02/03: cortex-m/hardfault: add usage-fault handler

2021-12-09 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 66e604b40e16e1a7b42c19617d598296f410bb5c Author: chao.an AuthorDate: Thu Dec 9 17:40:50 2021 +0800 cort

[incubator-nuttx] branch master updated (99fa58c -> 3d75c25)

2021-12-09 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 99fa58c arm/cortex-m23: armv8-m baseline do not support mem-fault new 2f44924 cortex-m/hardfault:

[incubator-nuttx] 01/03: cortex-m/hardfault: add bus-fault handler

2021-12-09 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 2f449245cc60217c1975db5344d19937ab829540 Author: chao.an AuthorDate: Tue Nov 30 21:55:48 2021 +0800 cor

[incubator-nuttx] 03/03: cortex-m/hardfault: enhance the dump information of mem/hard-fault

2021-12-09 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 3d75c25737dceb0caaefe3204c3e221ddbf7336f Author: chao.an AuthorDate: Tue Nov 30 22:04:05 2021 +0800 cor

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-990083507 > @OctopusHu0105 - what it the use case and motivation for these? Let's user add the custom info to the note stream, much like what Linux kernel do: htt

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-990085997 > @OctopusHu0105 - what it the use case and motivation for these? Let's user add the custom info to the note stream, much like what Linux kernel do: htt

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766028160 ## File path: drivers/ioexpander/gpio.c ## @@ -347,8 +349,9 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[incubator-nuttx] branch master updated: timer-gpout example renamed to timer-gpio

2021-12-09 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 bcf9b4e timer-gpout example renamed

[incubator-nuttx-apps] branch master updated (71f5af4 -> 21e0ff1)

2021-12-09 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 71f5af4 GPIO example application: Add support for changing pintypes new 128996f timer_gpout e

[incubator-nuttx-apps] 01/02: timer_gpout example: use /dev/gpioN instead of /dev/gpoutN

2021-12-09 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 commit 128996fb5f76a6c677587eed6fd68db64f12b4a0 Author: Jari van Ewijk AuthorDate: Thu Dec 9 14:44:28 2021 +01

[incubator-nuttx-apps] 02/02: timer_gpout example: rename to timer_gpio

2021-12-09 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 commit 21e0ff11534abd6662149ff06197afd1a6dd11d4 Author: Jari van Ewijk AuthorDate: Thu Dec 9 16:54:19 2021 +01

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4966: timer-gpout example renamed to timer-gpio

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4966: URL: https://github.com/apache/incubator-nuttx/pull/4966 -- 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-apps] xiaoxiang781216 merged pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #917: URL: https://github.com/apache/incubator-nuttx-apps/pull/917 -- 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-m

[GitHub] [incubator-nuttx] xiaoxiang781216 removed a comment on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 removed a comment on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-990085997 > @OctopusHu0105 - what it the use case and motivation for these? Let's user add the custom info to the note stream, much like what Linux kernel do

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-990091606 Here is the usage: https://github.com/apache/incubator-nuttx-apps/pull/916 -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [incubator-nuttx] xiaoxiang781216 removed a comment on pull request #4963: trace: and sched note dump

2021-12-09 Thread GitBox
xiaoxiang781216 removed a comment on pull request #4963: URL: https://github.com/apache/incubator-nuttx/pull/4963#issuecomment-990091606 Here is the usage: https://github.com/apache/incubator-nuttx-apps/pull/916 -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-nuttx] jarivanewijk commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766041514 ## File path: drivers/ioexpander/gpio.c ## @@ -347,8 +349,9 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] jarivanewijk commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766043072 ## File path: drivers/ioexpander/gpio.c ## @@ -489,7 +496,65 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx-apps] jarivanewijk commented on pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

2021-12-09 Thread GitBox
jarivanewijk commented on pull request #917: URL: https://github.com/apache/incubator-nuttx-apps/pull/917#issuecomment-990108282 This probably should not have been merged before https://github.com/apache/incubator-nuttx/pull/4774 ... -- This is an automated message from the Apache Git S

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766048429 ## File path: drivers/ioexpander/gpio.c ## @@ -347,8 +349,9 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766048429 ## File path: drivers/ioexpander/gpio.c ## @@ -347,8 +349,9 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766049681 ## File path: drivers/ioexpander/gpio.c ## @@ -489,7 +496,65 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] jarivanewijk commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766050340 ## File path: drivers/ioexpander/gpio.c ## @@ -562,13 +619,11 @@ int gpio_pin_register(FAR struct gpio_dev_s *dev, int minor) {

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #917: URL: https://github.com/apache/incubator-nuttx-apps/pull/917#issuecomment-990116060 Yes, but I think that patch will merge soon.:) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [incubator-nuttx-apps] pkarashchenko opened a new pull request #919: boot/mcuboot/Kconfig: extend configuration options

2021-12-09 Thread GitBox
pkarashchenko opened a new pull request #919: URL: https://github.com/apache/incubator-nuttx-apps/pull/919 ## Summary Extend configuration options for MCUboot ## Impact MCUboot based applications ## Testing PreCI pass -- This is an automated message from the Apache

[GitHub] [incubator-nuttx] jarivanewijk commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766075475 ## File path: drivers/ioexpander/gpio.c ## @@ -489,7 +496,65 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] jarivanewijk commented on a change in pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
jarivanewijk commented on a change in pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#discussion_r766075385 ## File path: drivers/ioexpander/gpio.c ## @@ -347,8 +349,9 @@ static int gpio_ioctl(FAR struct file *filep, int cmd, unsigned long arg)

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774#issuecomment-990141407 LGTM. -- 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 specifi

[incubator-nuttx-website] branch asf-site updated: Publishing web: adbba922a8890b650c27ecac31f9d26fe25e1ec4 docs: bcf9b4e5cd502e7270339c4c4f64f355f0c17870

2021-12-09 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 c7a9739 Publishing web

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-990440650 >Hmm, the following commit has a problem with uSD card on Spresense. >If I revert the commit locally, it works fine. @xiaoxiang781216 Because the nuttx/

[GitHub] [incubator-nuttx] masayuki2009 edited a comment on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 edited a comment on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-990440650 >Hmm, the following commit has a problem with uSD card on Spresense. >If I revert the commit locally, it works fine. @xiaoxiang781216 Because the

[GitHub] [incubator-nuttx] yamt commented on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
yamt commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-990471626 > > Hmm, the following commit has a problem with uSD card on Spresense. > > If I revert the commit locally, it works fine. > > @xiaoxiang781216 Because the nuttx/fs

[GitHub] [incubator-nuttx] yamt commented on pull request #4881: libc: Add wctoxk and xktowc for the conversion between X11 keysym and wchar_t

2021-12-09 Thread GitBox
yamt commented on pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#issuecomment-990473887 > > it seems a bit awkward to have x11 stuff in libc. > > I agree. Not just awkward but wrong. The content of libc is controlled by POSIX/ANSI standards (plus glibc c

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #4754: Revert "sim: Specify -fshort-wchar as NuttX wchar_t is 16-bit"

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4754: URL: https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-990488059 >i guess nuttx/fs/fat should use uint16_t instead. >using wchar_t to mean particular encoding (utf16, utf32, or whatever) is a bad practice. @yamt Than

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #4968: fs: fat: Use uin16_t instead of wchar_t in fs_fat32dirent.c

2021-12-09 Thread GitBox
masayuki2009 opened a new pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968 ## Summary - Due to the recent changes of wchar_t, ls command always causes errors for the fat file system. - This commit fixes this issue by replacing wchar_t with uint

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #4968: fs: fat: Use uin16_t instead of wchar_t in fs_fat32dirent.c

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968#issuecomment-990519642 See https://github.com/apache/incubator-nuttx/pull/4754#issuecomment-990488059 -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx] yamt commented on pull request #4968: fs: fat: Use uin16_t instead of wchar_t in fs_fat32dirent.c

2021-12-09 Thread GitBox
yamt commented on pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968#issuecomment-990529519 i guess you need to change this too. https://github.com/apache/incubator-nuttx/blob/bcf9b4e5cd502e7270339c4c4f64f355f0c17870/fs/fat/fs_fat32.h#L948 -- This is an automate

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #4968: fs: fat: Use uin16_t instead of wchar_t in fs_fat32dirent.c

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968#issuecomment-990533958 >i guess you need to change this too. @yamt OK. I'll modify the file later. -- This is an automated message from the Apache Git Service. To respond

[incubator-nuttx-apps] branch master updated: boot/mcuboot/Kconfig: extend configuration options

2021-12-09 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 afbd132 boot/mcuboot/Kconfig:

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #919: boot/mcuboot/Kconfig: extend configuration options

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #919: URL: https://github.com/apache/incubator-nuttx-apps/pull/919 -- 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-m

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4968: fs: fat: Use uint16_t instead of wchar_t

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968#issuecomment-990568518 LGTM. -- 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 specifi

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4968: fs: fat: Use uint16_t instead of wchar_t

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4968: URL: https://github.com/apache/incubator-nuttx/pull/4968 -- 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

[incubator-nuttx] branch master updated: fs: fat: Use uint16_t instead of wchar_t

2021-12-09 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 a8d4468 fs: fat: Use uint16_t inste

[incubator-nuttx] branch master updated: arch/arm/src/stm32/Kconfig STM32_STM32F412 add SPI2 & SPI3

2021-12-09 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 efc949b arch/arm/src/stm32/Kconfig

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4967: arch/arm/src/stm32/Kconfig STM32_STM32F412 add SPI2 & SPI3

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4967: URL: https://github.com/apache/incubator-nuttx/pull/4967 -- 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] masayuki2009 commented on pull request #4953: sched/semaphore: fix race condition for timed wait

2021-12-09 Thread GitBox
masayuki2009 commented on pull request #4953: URL: https://github.com/apache/incubator-nuttx/pull/4953#issuecomment-990596073 @anchao >There is a potential problem that can lead to deadlock at >condition wait, if the timeout of watchdog is a very small value >(less than 1ms ?),

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4881: libc: Add wctoxk and xktowc for the conversion between X11 keysym and wchar_t

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#issuecomment-990609051 > > > it seems a bit awkward to have x11 stuff in libc. > > > > > > I agree. Not just awkward but wrong. The content of libc is controlled by POSIX/ANS

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #4881: libc: Update include/nuttx/input/x11_keysymdef.h to the latest version

2021-12-09 Thread GitBox
xiaoxiang781216 edited a comment on pull request #4881: URL: https://github.com/apache/incubator-nuttx/pull/4881#issuecomment-990609051 > > > it seems a bit awkward to have x11 stuff in libc. > > > > > > I agree. Not just awkward but wrong. The content of libc is controlled by PO

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #4774: GPIO driver: Use generic /dev/gpioN

2021-12-09 Thread GitBox
xiaoxiang781216 merged pull request #4774: URL: https://github.com/apache/incubator-nuttx/pull/4774 -- 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

[incubator-nuttx] branch master updated: GPIO driver: register all pintypes as generic /dev/gpioN

2021-12-09 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 2e47ef3 GPIO driver: register all p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4913: sched: Move argv from tcb_s to task_info_s

2021-12-09 Thread GitBox
xiaoxiang781216 commented on pull request #4913: URL: https://github.com/apache/incubator-nuttx/pull/4913#issuecomment-990640121 @Ouss4 could you look a look this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [incubator-nuttx] anchao commented on pull request #4953: sched/semaphore: fix race condition for timed wait

2021-12-09 Thread GitBox
anchao commented on pull request #4953: URL: https://github.com/apache/incubator-nuttx/pull/4953#issuecomment-990696182 @masayuki2009 san, I prepared a demo, which still difficult to reproduce this issue on test environment https://github.com/anchao/incubator-nuttx-apps/c