[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-20 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1082623300 ## fs/vfs/fs_symlink.c: ## @@ -180,4 +180,30 @@ int symlink(FAR const char *path1, FAR const char *path2) return ERROR; } +/

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8205: Implement more pthread API

2023-01-20 Thread GitBox
xiaoxiang781216 opened a new pull request, #8205: URL: https://github.com/apache/nuttx/pull/8205 ## Summary - Documentation: Remove the supported function from the unsupported list - libc/pthread: Implement pthread_rwlockattr API - libc/pthread: Implement pthread_attr_[set|get]s

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
gustavonihei commented on code in PR #8202: URL: https://github.com/apache/nuttx/pull/8202#discussion_r1082824622 ## arch/xtensa/src/esp32/esp32_touch.c: ## @@ -51,7 +51,7 @@ #define TOUCH_PAD_FILTER_FACTOR_DEFAULT (4) /* IIR filter coefficient */ #define TOUCH_PAD_SHIFT_DEFAU

[GitHub] [nuttx] lucasssvaz commented on a diff in pull request #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
lucasssvaz commented on code in PR #8202: URL: https://github.com/apache/nuttx/pull/8202#discussion_r1082737755 ## arch/xtensa/src/esp32/esp32_gpio.c: ## @@ -222,258 +220,152 @@ int esp32_configgpio(int pin, gpio_pinattr_t attr) func = 0; cntrl = 0; - if ((attr & FUNCT

[GitHub] [nuttx] raiden00pl opened a new pull request, #8204: drivers/foc: support for BEMF sensing

2023-01-20 Thread GitBox
raiden00pl opened a new pull request, #8204: URL: https://github.com/apache/nuttx/pull/8204 ## Summary drivers/foc: support for BEMF sensing ## Impact The next step towards BEMF sensing. Lower-half logic is not yet implemented. Releted issue: https://github.com/apache/nutt

[GitHub] [nuttx] raiden00pl opened a new pull request, #8203: drivers/foc: add interface that turn off all PWM switches

2023-01-20 Thread GitBox
raiden00pl opened a new pull request, #8203: URL: https://github.com/apache/nuttx/pull/8203 ## Summary - drivers/foc: add ioctl interface that turn off all PWM switches - stm32,stm32f7/foc: support for pwm_off() ## Impact Initial changes to support BEMF sensing when motor is n

[GitHub] [nuttx-apps] xiaoxiang781216 merged pull request #1496: media: fix race condition on nx_xx testcase

2023-01-20 Thread GitBox
xiaoxiang781216 merged PR #1496: URL: https://github.com/apache/nuttx-apps/pull/1496 -- 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] [nuttx] xiaoxiang781216 commented on pull request #8121: arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

2023-01-20 Thread GitBox
xiaoxiang781216 commented on PR #8121: URL: https://github.com/apache/nuttx/pull/8121#issuecomment-1398489955 Here https://en.cppreference.com/w/c/atomic/atomic_load -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8196: time.h: Add CLOCK_PROCESS_CPUTIME_ID and CLOCK_THREAD_CPUTIME_ID definition

2023-01-20 Thread GitBox
xiaoxiang781216 commented on code in PR #8196: URL: https://github.com/apache/nuttx/pull/8196#discussion_r1082636862 ## include/time.h: ## @@ -64,28 +64,38 @@ * forward and backward as the system time-of-day clock is changed. */ -#define CLOCK_REALTIME 0 +#define CLOCK

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
gustavonihei commented on code in PR #8202: URL: https://github.com/apache/nuttx/pull/8202#discussion_r1082635223 ## arch/xtensa/src/esp32/esp32_gpio.c: ## @@ -222,258 +220,152 @@ int esp32_configgpio(int pin, gpio_pinattr_t attr) func = 0; cntrl = 0; - if ((attr & FUN

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
gustavonihei commented on code in PR #8202: URL: https://github.com/apache/nuttx/pull/8202#discussion_r1082635223 ## arch/xtensa/src/esp32/esp32_gpio.c: ## @@ -222,258 +220,152 @@ int esp32_configgpio(int pin, gpio_pinattr_t attr) func = 0; cntrl = 0; - if ((attr & FUN

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
gustavonihei commented on code in PR #8202: URL: https://github.com/apache/nuttx/pull/8202#discussion_r1082631904 ## arch/xtensa/src/esp32/esp32_gpio.c: ## @@ -222,258 +220,152 @@ int esp32_configgpio(int pin, gpio_pinattr_t attr) func = 0; cntrl = 0; - if ((attr & FUN

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-20 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1398482685 Sorry, I don't see there is anything reason to revert this patch. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-20 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1082623300 ## fs/vfs/fs_symlink.c: ## @@ -180,4 +180,30 @@ int symlink(FAR const char *path1, FAR const char *path2) return ERROR; } +/

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-20 Thread GitBox
xiaoxiang781216 merged PR #8183: URL: https://github.com/apache/nuttx/pull/8183 -- 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...@nutt

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-20 Thread GitBox
xiaoxiang781216 merged PR #8194: URL: https://github.com/apache/nuttx/pull/8194 -- 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...@nutt

[GitHub] [nuttx] lucasssvaz opened a new pull request, #8202: arch/xtensa/esp32: Add support for RTC IRQs

2023-01-20 Thread GitBox
lucasssvaz opened a new pull request, #8202: URL: https://github.com/apache/nuttx/pull/8202 ## Summary Separate from GPIOs and add a handler for RTC interrupts. Also replaces the RTC watchdog interrupt handler with the new one. ## Impact None on existing features. Add

[GitHub] [nuttx] acassis commented on a diff in pull request #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-20 Thread GitBox
acassis commented on code in PR #8194: URL: https://github.com/apache/nuttx/pull/8194#discussion_r1082431352 ## fs/inode/fs_files.c: ## @@ -298,7 +301,10 @@ int files_duplist(FAR struct filelist *plist, FAR struct filelist *clist) #endif filep = &plist->fl_files[i

[GitHub] [nuttx] yamt opened a new pull request, #8201: Make 64-bit time_t unsigned

2023-01-20 Thread GitBox
yamt opened a new pull request, #8201: URL: https://github.com/apache/nuttx/pull/8201 ## Summary * 32-bit time_t should be unsigned because otherwise it wraps too soon. (in 2038) * 64-bit time_t should be unsigned because it should be consistent within NuttX. * While si

[GitHub] [nuttx] pkarashchenko merged pull request #8195: fs: Map madvice to posix_madvice

2023-01-20 Thread GitBox
pkarashchenko merged PR #8195: URL: https://github.com/apache/nuttx/pull/8195 -- 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...@nuttx.

[GitHub] [nuttx] pkarashchenko commented on pull request #8183: limits.h: add PTRDIFF_MAX and PTRDIFF_MIN

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8183: URL: https://github.com/apache/nuttx/pull/8183#issuecomment-1398136709 Please rebase this PR to fix the CI -- 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

[GitHub] [nuttx] pkarashchenko commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1398125566 Meanwhile I will try to run the test you provided on my end -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [nuttx] pkarashchenko commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1398122888 I also can't track a path that leads to pthread exiting while holding `group->tg_joinlock`. Is it possible to provide some backtrace information so we can better understand the flow?

[GitHub] [nuttx] pkarashchenko commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1398115501 So this PR is more like an attempt to fix some partial case while generic case is still left broken -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [nuttx] pkarashchenko commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1398110929 @xiaoxiang781216 @yamt so what is the path to move forward here? I read that this change brings regression to `yaffs2` and the natural step seems to be reverting of a change and re-ev

[GitHub] [nuttx] pkarashchenko commented on pull request #8121: arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

2023-01-20 Thread GitBox
pkarashchenko commented on PR #8121: URL: https://github.com/apache/nuttx/pull/8121#issuecomment-1398105755 @xiaoxiang781216 could you please point me to some good article describing atomic variables and accessing it? I need to educate myself as I was not expecting `atomic_load` to be used

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-20 Thread GitBox
pkarashchenko commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1082252596 ## libs/libxx/__config_site: ## @@ -0,0 +1,69 @@ +/ + * libs/libxx/__config_site + * + *

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8196: time.h: Add CLOCK_PROCESS_CPUTIME_ID and CLOCK_THREAD_CPUTIME_ID definition

2023-01-20 Thread GitBox
pkarashchenko commented on code in PR #8196: URL: https://github.com/apache/nuttx/pull/8196#discussion_r1082249826 ## include/time.h: ## @@ -64,28 +64,38 @@ * forward and backward as the system time-of-day clock is changed. */ -#define CLOCK_REALTIME 0 +#define CLOCK_R

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8198: esp32s2: Add support to efuse

2023-01-20 Thread GitBox
pkarashchenko commented on code in PR #8198: URL: https://github.com/apache/nuttx/pull/8198#discussion_r1082244712 ## arch/xtensa/src/esp32s2/esp32s2_efuse_table.c: ## @@ -0,0 +1,1347 @@ +/ + * arch/xten

[GitHub] [nuttx] pkarashchenko commented on a diff in pull request #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-20 Thread GitBox
pkarashchenko commented on code in PR #8194: URL: https://github.com/apache/nuttx/pull/8194#discussion_r1082239504 ## fs/inode/fs_files.c: ## @@ -298,7 +301,10 @@ int files_duplist(FAR struct filelist *plist, FAR struct filelist *clist) #endif filep = &plist->fl_f

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-19 Thread GitBox
xiaoxiang781216 commented on PR #8194: URL: https://github.com/apache/nuttx/pull/8194#issuecomment-1398020868 > @xiaoxiang781216 any idea why we still getting this error: > > ``` > make[1]: *** [uClibc++.defs:26: uClibc++] Error 6 > make[1]: Target 'context' not remade because of

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8198: esp32s2: Add support to efuse

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8198: URL: https://github.com/apache/nuttx/pull/8198 -- 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...@nutt

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8199: esp32s3: Add support to efuse

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8199: URL: https://github.com/apache/nuttx/pull/8199 -- 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...@nutt

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8200: esp32: fix lower half oneshot for usage with nxsched_oneshot_start

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8200: URL: https://github.com/apache/nuttx/pull/8200 -- 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...@nutt

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8196: time.h: Add CLOCK_PROCESS_CPUTIME_ID and CLOCK_THREAD_CPUTIME_ID definition

2023-01-19 Thread GitBox
xiaoxiang781216 commented on code in PR #8196: URL: https://github.com/apache/nuttx/pull/8196#discussion_r1082182720 ## include/time.h: ## @@ -64,28 +64,38 @@ * forward and backward as the system time-of-day clock is changed. */ -#define CLOCK_REALTIME 0 +#define CLOCK

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-19 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1398009453 > > Could you give a list of the function not implemented by NuttX yet. I can take time to implement them. > > The complete list of unimplemented functions is: > > ```c

[GitHub] [nuttx] g2gps commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-19 Thread GitBox
g2gps commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1397810269 > Could you give a list of the function not implemented by NuttX yet. I can take time to implement them. The complete list of unimplemented functions is: ```c int at_quick_exit(vo

[GitHub] [nuttx] maxikrie opened a new pull request, #8200: esp32: fix lower half oneshot for usage with nxsched_oneshot_start

2023-01-19 Thread GitBox
maxikrie opened a new pull request, #8200: URL: https://github.com/apache/nuttx/pull/8200 ## Summary This fix is required for using the oneshot driver for cpuload monitoring. Otherwise an assertion is thrown since nxsched_oneshot_start pass NULL as arg. ## Impact ## Testing T

[GitHub] [nuttx] acassis opened a new pull request, #8199: esp32s3: Add support to efuse

2023-01-19 Thread GitBox
acassis opened a new pull request, #8199: URL: https://github.com/apache/nuttx/pull/8199 ## Summary Add support to efuse ## Impact User will be able to use efuse ## Testing esp32-s3-devkit -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [nuttx] acassis opened a new pull request, #8198: esp32s2: Add support to efuse

2023-01-19 Thread GitBox
acassis opened a new pull request, #8198: URL: https://github.com/apache/nuttx/pull/8198 ## Summary Add support to efuse ## Impact User will be able to use efuse ## Testing esp32s2-saola-1 -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [nuttx] acassis commented on a diff in pull request #8196: time.h: Add CLOCK_PROCESS_CPUTIME_ID and CLOCK_THREAD_CPUTIME_ID definition

2023-01-19 Thread GitBox
acassis commented on code in PR #8196: URL: https://github.com/apache/nuttx/pull/8196#discussion_r1081845991 ## include/time.h: ## @@ -64,28 +64,38 @@ * forward and backward as the system time-of-day clock is changed. */ -#define CLOCK_REALTIME 0 +#define CLOCK_REALTIM

[GitHub] [nuttx] acassis commented on pull request #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-19 Thread GitBox
acassis commented on PR #8194: URL: https://github.com/apache/nuttx/pull/8194#issuecomment-1397572188 @xiaoxiang781216 any idea why we still getting this error: ``` make[1]: *** [uClibc++.defs:26: uClibc++] Error 6 make[1]: Target 'context' not remade because of errors. make: ***

[GitHub] [nuttx] haitomatic closed pull request #8197: Remove loopback check for CANFD driver Txpoll

2023-01-19 Thread GitBox
haitomatic closed pull request #8197: Remove loopback check for CANFD driver Txpoll URL: https://github.com/apache/nuttx/pull/8197 -- 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 commen

[GitHub] [nuttx] haitomatic opened a new pull request, #8197: Remove loopback check for CANFD driver Txpoll

2023-01-19 Thread GitBox
haitomatic opened a new pull request, #8197: URL: https://github.com/apache/nuttx/pull/8197 - Loopback check is not need in txpoll since it's now handled in caller `devif_poll` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8196: time.h: Add CLOCK_PROCESS_CPUTIME_ID and CLOCK_THREAD_CPUTIME_ID definition

2023-01-19 Thread GitBox
xiaoxiang781216 opened a new pull request, #8196: URL: https://github.com/apache/nuttx/pull/8196 ## Summary see the spec here: https://pubs.opengroup.org/onlinepubs/95399/functions/clock_getres.html ## Impact new macro ## Testing Pass CI -- This is

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8195: fs: Map madvice to posix_madvice

2023-01-19 Thread GitBox
xiaoxiang781216 opened a new pull request, #8195: URL: https://github.com/apache/nuttx/pull/8195 ## Summary https://man7.org/linux/man-pages/man2/madvise.2.html ## Impact New macro ## Testing Pass CI -- This is an automated message from the Apache Git Ser

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8191: arch/mpfs: Make selection of SBI boot or direct boot run-time configu…

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8191: URL: https://github.com/apache/nuttx/pull/8191 -- 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...@nutt

[GitHub] [nuttx] acassis opened a new pull request, #8194: fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

2023-01-19 Thread GitBox
acassis opened a new pull request, #8194: URL: https://github.com/apache/nuttx/pull/8194 ## Summary Avoid accessing filep fields if it is NULL & add DEBUGASSERTs ## Impact Avoid crashes ## Testing esp32-devkit -- This is an automated message from the Apache Git Service.

[GitHub] [nuttx] gustavonihei merged pull request #8177: build: Add STACK_USAGE(-fstack-usage) to assist the stack analysis

2023-01-19 Thread GitBox
gustavonihei merged PR #8177: URL: https://github.com/apache/nuttx/pull/8177 -- 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...@nuttx.a

[GitHub] [nuttx] gustavonihei commented on pull request #8121: arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

2023-01-19 Thread GitBox
gustavonihei commented on PR #8121: URL: https://github.com/apache/nuttx/pull/8121#issuecomment-1396982709 I honestly feel that adding this flag to this broad scope is too overkill, even because the occurrence is not even in upstream code. And this is a kind of warning that shouldn't be ove

[GitHub] [nuttx] jlaitine commented on pull request #8191: arch/mpfs: Make selection of SBI boot or direct boot run-time configu…

2023-01-19 Thread GitBox
jlaitine commented on PR #8191: URL: https://github.com/apache/nuttx/pull/8191#issuecomment-1396975928 Still removed "Private Functions" and "Public Data" sections from mpfs_entrypoints.c as they are now empty and not needed. -- This is an automated message from the Apache Git Service. To

[GitHub] [nuttx] davids5 commented on pull request #8192: backtrace: use CURRENT_REGS when in interrupt context

2023-01-19 Thread GitBox
davids5 commented on PR #8192: URL: https://github.com/apache/nuttx/pull/8192#issuecomment-1396832917 @zyfeier Can you add some background here explaining why and the value of the change? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [nuttx] dnschwa closed pull request #8185: Acknowledge interrupt before installing callback

2023-01-19 Thread GitBox
dnschwa closed pull request #8185: Acknowledge interrupt before installing callback URL: https://github.com/apache/nuttx/pull/8185 -- 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 commen

[GitHub] [nuttx] dnschwa commented on pull request #8185: Acknowledge interrupt before installing callback

2023-01-19 Thread GitBox
dnschwa commented on PR #8185: URL: https://github.com/apache/nuttx/pull/8185#issuecomment-1396798001 Didn't realize this is breaking existing code. Thanks, I will look at your example. -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8193: Documentation/p/a/rp2040: add docs for waveshare-rp2040-lcd-1.28

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8193: URL: https://github.com/apache/nuttx/pull/8193 -- 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...@nutt

[GitHub] [nuttx] dnschwa commented on pull request #8185: Acknowledge interrupt before installing callback

2023-01-19 Thread GitBox
dnschwa commented on PR #8185: URL: https://github.com/apache/nuttx/pull/8185#issuecomment-1396772931 Didn't realize this is breaking existing code. Thanks, I will look at your example. However, might `up_disable_irq(irq);` not work istead? (since it is not affecting HW state) -- This

[GitHub] [nuttx] PeterBee97 opened a new pull request, #8193: Documentation/p/a/rp2040: add docs for waveshare-rp2040-lcd-1.28

2023-01-19 Thread GitBox
PeterBee97 opened a new pull request, #8193: URL: https://github.com/apache/nuttx/pull/8193 ## Summary Add documentation for Waveshare RP2040 LCD 1.28 ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [nuttx] zyfeier opened a new pull request, #8192: backtrace: use CURRENT_REGS when in interrupt context

2023-01-19 Thread GitBox
zyfeier opened a new pull request, #8192: URL: https://github.com/apache/nuttx/pull/8192 ## Summary Backtrace CURRENT_REGS when in interrupt context. ## Impact NA ## Testing BES2003 -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [nuttx] davids5 commented on pull request #8185: Acknowledge interrupt before installing callback

2023-01-19 Thread GitBox
davids5 commented on PR #8185: URL: https://github.com/apache/nuttx/pull/8185#issuecomment-1396733765 It is service and device dependent. The normal sequence is to enable from NVIC outward. (Disable in the opposite direction) 1. attach 2. enable NVIC 3. enable device. 4.

[GitHub] [nuttx] jlaitine opened a new pull request, #8191: arch/mpfs: Make selection of SBI boot or direct boot run-time configu…

2023-01-19 Thread GitBox
jlaitine opened a new pull request, #8191: URL: https://github.com/apache/nuttx/pull/8191 …rable Allow bootloader to select run-time whether the payload binary is booted with SBI or directly by jumping to entrypoint address. - Use just one bitmask to select sbi or direct boot f

[GitHub] [nuttx] dnschwa commented on pull request #8185: Acknowledge interrupt before installing callback

2023-01-19 Thread GitBox
dnschwa commented on PR #8185: URL: https://github.com/apache/nuttx/pull/8185#issuecomment-1396627509 Does it make sense for the callback to service an event that happened *before* it the callback was installed? This is not theoretical either, I've encountered this in a real-world case.

[GitHub] [nuttx] xiaoxiang781216 merged pull request #8188: mm/mm_extend: Increase total heap size accordingly

2023-01-19 Thread GitBox
xiaoxiang781216 merged PR #8188: URL: https://github.com/apache/nuttx/pull/8188 -- 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...@nutt

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-19 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396608420 > > > Since iattr and ATTR_xxx isn't defined in nuttx. > > > > > > it's something you should provide as a part of nuttx glue logic, isn't it? > > Sorry, I don't think so. If N

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-19 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396600615 > > Since iattr and ATTR_xxx isn't defined in nuttx. > > it's something you should provide as a part of nuttx glue logic, isn't it? > Sorry, I don't think so. If Nut

[GitHub] [nuttx] anchao closed pull request #8189: arm/tiva: fix Kconfig warning

2023-01-19 Thread GitBox
anchao closed pull request #8189: arm/tiva: fix Kconfig warning URL: https://github.com/apache/nuttx/pull/8189 -- 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

[GitHub] [nuttx] anchao commented on pull request #8189: arm/tiva: fix Kconfig warning

2023-01-19 Thread GitBox
anchao commented on PR #8189: URL: https://github.com/apache/nuttx/pull/8189#issuecomment-1396590286 > @anchao it's already fixed by https://github.com/apache/nuttx/pull/8178 Ok, let me close this PR. -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [nuttx] anchao commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-19 Thread GitBox
anchao commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1396587897 I do not have pico W on hand, just placed an order, I will fix this issue later. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [nuttx] jlaitine closed pull request #8190: Fix ci build

2023-01-19 Thread GitBox
jlaitine closed pull request #8190: Fix ci build URL: https://github.com/apache/nuttx/pull/8190 -- 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-

[GitHub] [nuttx] jlaitine opened a new pull request, #8190: Fix ci build

2023-01-19 Thread GitBox
jlaitine opened a new pull request, #8190: URL: https://github.com/apache/nuttx/pull/8190 ## 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 URL above to go to the

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-19 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396584516 > Since iattr and ATTR_xxx isn't defined in nuttx. it's something you should provide as a part of nuttx glue logic, isn't it? anyway, if yaffs2 is broken, i'd suggest to fix it ins

[GitHub] [nuttx] zyfeier commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-19 Thread GitBox
zyfeier commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1396582225 > But I do not see that pholder code modifies sem count value. How is this different from the case without priority inheritance enabled? And is the same issue exists in non-SMP case?

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8189: arm/tiva: fix Kconfig warning

2023-01-19 Thread GitBox
xiaoxiang781216 commented on PR #8189: URL: https://github.com/apache/nuttx/pull/8189#issuecomment-1396578775 @anchao it's already fixed by https://github.com/apache/nuttx/pull/8178 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [nuttx] anchao opened a new pull request, #8189: arm/tiva: fix Kconfig warning

2023-01-19 Thread GitBox
anchao opened a new pull request, #8189: URL: https://github.com/apache/nuttx/pull/8189 ## Summary arm/tiva: fix Kconfig warning ``` boards/arm/tiva/tm4c1294-launchpad/Kconfig:90:warning: defaults for choice values not supported boards/arm/tiva/tm4c1294-launchpad/Kcon

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396575435 Since iattr and ATTR_xxx isn't defined in nuttx. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396572081 > > what's the point of this change? > > There is code which redefines DT_(e.g., http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blob;f=direct/yportenv.h;h=5a79f93583b963d715b0dee9a0b

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8121: arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8121: URL: https://github.com/apache/nuttx/pull/8121#issuecomment-1396564883 > @xiaoxiang781216 Could you please share the motivation for disabling this warning? In which scenario does it manifest? @gustavonihei here is the warning generated by Candanc

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396559754 > what's the point of this change? There is code which redefines DT_(e.g., http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blob;f=direct/yportenv.h;h=5a79f93583b963d715b0de

[GitHub] [nuttx] yamt commented on pull request #8163: fs: Define DT_xxx to number directly

2023-01-18 Thread GitBox
yamt commented on PR #8163: URL: https://github.com/apache/nuttx/pull/8163#issuecomment-1396556478 what's the point of this change? are you going to remove DTYPE_xxx? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [nuttx] pkarashchenko commented on pull request #8175: sched/semaphore: increase sem count when holder task exit

2023-01-18 Thread GitBox
pkarashchenko commented on PR #8175: URL: https://github.com/apache/nuttx/pull/8175#issuecomment-1396543499 But I do not see that pholder code modifies sem count value. How is this different from the case without priority inheritance enabled? -- This is an automated message from the Apach

[GitHub] [nuttx] jerpelea merged pull request #8187: c++: Change the default value of CXX_STANDARD from c++17 to gnu++17

2023-01-18 Thread GitBox
jerpelea merged PR #8187: URL: https://github.com/apache/nuttx/pull/8187 -- 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...@nuttx.apach

[GitHub] [nuttx] jerpelea merged pull request #8178: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread GitBox
jerpelea merged PR #8178: URL: https://github.com/apache/nuttx/pull/8178 -- 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...@nuttx.apach

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8178: tm4c1294-launchpad/Kconfig: Fix warning: defaults for choice values not supported

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8178: URL: https://github.com/apache/nuttx/pull/8178#issuecomment-1396541453 Can anybody merge this PR? The CI still spew many useless logs now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [nuttx-apps] jerpelea merged pull request #1498: Add *.su to .gitignore

2023-01-18 Thread GitBox
jerpelea merged PR #1498: URL: https://github.com/apache/nuttx-apps/pull/1498 -- 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...@nuttx.

[GitHub] [nuttx] jerpelea merged pull request #8186: boards: sabre-6quad: Adjust TCP and UDP configurations

2023-01-18 Thread GitBox
jerpelea merged PR #8186: URL: https://github.com/apache/nuttx/pull/8186 -- 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...@nuttx.apach

[GitHub] [nuttx] pussuw opened a new pull request, #8188: mm/mm_extend: Increase total heap size accordingly

2023-01-18 Thread GitBox
pussuw opened a new pull request, #8188: URL: https://github.com/apache/nuttx/pull/8188 ## Summary When adding more heap memory, the total heap size was not updated. This results in a crash in mm_mallinfo: DEBUGASSERT((size_t)info->uordblks + info->fordblks == heap->mm_heapsize);

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8176: Build warning fixes

2023-01-18 Thread GitBox
jlaitine commented on code in PR #8176: URL: https://github.com/apache/nuttx/pull/8176#discussion_r1080851962 ## mm/mempool/mempool_multiple.c: ## @@ -220,7 +220,7 @@ mempool_multiple_get_dict(FAR struct mempool_multiple_s *mpool, col = index - (row << mpool->dict_col_num_lo

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396494107 > For 15.0.0, it seems libcxx moved to using `__attribute__((using_if_exists))` in order to enable or disable features. See [this commit on LLVM](https://github.com/llvm/llvm-proje

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080846046 ## libs/libxx/Kconfig: ## @@ -35,10 +35,11 @@ config LIBCXXMINI config LIBCXX bool "LLVM libc++ C++ Standard Library" + depends on CXX_STANDARD="c++17" ||

[GitHub] [nuttx] g2gps commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396487086 > For 15.0.0, it seems libcxx moved to using `__attribute__((using_if_exists))` in order to enable or disable features. See [this commit on LLVM](https://github.com/llvm/llvm-project

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396485568 Please reorganize the patch set and remove the intermediate change, @g2gps . -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080842496 ## libs/libxx/Kconfig: ## @@ -35,10 +35,11 @@ config LIBCXXMINI config LIBCXX bool "LLVM libc++ C++ Standard Library" + depends on CXX_STANDARD="

[GitHub] [nuttx] PeterBee97 commented on issue #7475: RaspberryPi-Pico-W is not getting WiFi

2023-01-18 Thread GitBox
PeterBee97 commented on issue #7475: URL: https://github.com/apache/nuttx/issues/7475#issuecomment-1396484817 > @PeterBee97 Do I need to have the "Infineon 43439 firmware file" to get the WiFi working? There is not documentation about it in the README.tx file Sorry Alan, I only have t

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396479291 BTW, @g2gps do you plan to upgrade libcxx to 15.0.x -- 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] [nuttx] xiaoxiang781216 commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396470872 > Would you prefer that I squash the commits into one while applying the amendments? It's fine since github provide UI to compare the change between the new patch and the old

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080830955 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

[GitHub] [nuttx] g2gps commented on pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on PR #8173: URL: https://github.com/apache/nuttx/pull/8173#issuecomment-1396468518 Would you prefer that I squash the commits into one while applying the amendments? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #8187: c++: Change the default value of CXX_STANDARD from c++17 to gnu++17

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #8187: URL: https://github.com/apache/nuttx/pull/8187 ## Summary since many 3rd party code use some gnu c++ extension ## Impact C++ default compiler option ## Testing Pass CI -- This is an automated message from

[GitHub] [nuttx] g2gps commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
g2gps commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080829205 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..13bc65dfb30

[GitHub] [nuttx-apps] xiaoxiang781216 opened a new pull request, #1498: Add *.su to .gitignore

2023-01-18 Thread GitBox
xiaoxiang781216 opened a new pull request, #1498: URL: https://github.com/apache/nuttx-apps/pull/1498 ## Summary *.su is generated by gcc/clang when we pass -fstack-usage to them ## Impact Minor ## Testing Pass CI -- This is an automated message from the

[GitHub] [nuttx] xiaoxiang781216 commented on a diff in pull request #8173: LLVM libcxx upgrade from 12.0.0 to 14.0.6

2023-01-18 Thread GitBox
xiaoxiang781216 commented on code in PR #8173: URL: https://github.com/apache/nuttx/pull/8173#discussion_r1080817564 ## libs/libxx/0001-libcxx-enable-support-for-c++11.patch: ## @@ -0,0 +1,49 @@ +diff --git a/libcxx/src/barrier.cpp b/libcxx/src/barrier.cpp +index 54890b2d797e..1

  1   2   3   4   5   6   7   8   9   10   >