[incubator-nuttx-apps] branch master updated: wapi/conf: convert mac address

2020-12-11 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 e4cddb1 wapi/conf: convert mac

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #517: wapi/conf: convert mac address

2020-12-11 Thread GitBox
xiaoxiang781216 merged pull request #517: URL: https://github.com/apache/incubator-nuttx-apps/pull/517 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-apps] xiaoxiang781216 commented on pull request #517: wapi/conf: convert mac address

2020-12-11 Thread GitBox
xiaoxiang781216 commented on pull request #517: URL: https://github.com/apache/incubator-nuttx-apps/pull/517#issuecomment-743711892 LGTM. This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2522: xtensa/esp32: Add ESP32 EFUSE char driver

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2522: URL: https://github.com/apache/incubator-nuttx/pull/2522#discussion_r541510110 ## File path: include/nuttx/efuse/esp_efuse.h ## @@ -0,0 +1,184 @@ +/*

[incubator-nuttx-website] branch asf-site updated: Publishing web: b5f9f15ea0a08ad7e52e24f580a88a7c2b9a085d docs: 3864912dc8f17d8ee0501c63662898493e9ec17c

2020-12-11 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 e36fb59 Publishing web

[GitHub] [incubator-nuttx] patacongo commented on pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
patacongo commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-743446674 There is some naming that should be made consistent too. In the TCB, the pid field should be called tid, for example. ---

[GitHub] [incubator-nuttx] hartmannathan commented on pull request #2473: [DRAFT] Stm32g4 add adc

2020-12-11 Thread GitBox
hartmannathan commented on pull request #2473: URL: https://github.com/apache/incubator-nuttx/pull/2473#issuecomment-743371393 @davids5 , @raiden00pl , Thank you for your input. I've finished this up and it's ready for review. --

[incubator-nuttx] branch master updated: arch/stm32: Fix nxstyle errors

2020-12-11 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 3864912 arch/stm32: Fix nxstyle error

[GitHub] [incubator-nuttx] acassis merged pull request #2521: arch/stm32: Fix nxstyle errors

2020-12-11 Thread GitBox
acassis merged pull request #2521: URL: https://github.com/apache/incubator-nuttx/pull/2521 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 t

[GitHub] [incubator-nuttx] acassis opened a new pull request #2522: xtensa/esp32: Add ESP32 EFUSE char driver

2020-12-11 Thread GitBox
acassis opened a new pull request #2522: URL: https://github.com/apache/incubator-nuttx/pull/2522 ## Summary Add ESP32 EFUSE char driver ## Impact ESP32 users will be able to read/burn the EFUSEs ## Testing Tested on esp32-wrover-kit board ---

[GitHub] [incubator-nuttx] hartmannathan opened a new pull request #2521: arch/stm32: Fix nxstyle errors

2020-12-11 Thread GitBox
hartmannathan opened a new pull request #2521: URL: https://github.com/apache/incubator-nuttx/pull/2521 ## Summary arch/arm/src/stm32/stm32l15xxx_rtcc.c: * Fix nxstyle errors. ## Impact Removes nxstyle errors. ## Testing nxstyle ---

[GitHub] [incubator-nuttx] hartmannathan commented on pull request #2488: libs/libc: correct the getrandom(2) prototype

2020-12-11 Thread GitBox
hartmannathan commented on pull request #2488: URL: https://github.com/apache/incubator-nuttx/pull/2488#issuecomment-743234459 > Have you checked, if the FreeBSD and Linux versions are different from each other in some subtle way? We cannot completely ignore Solaris precedent (and others w

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #518: fix build bread if CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 Thread GitBox
xiaoxiang781216 merged pull request #518: URL: https://github.com/apache/incubator-nuttx-apps/pull/518 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[incubator-nuttx-apps] branch master updated: fix build bread if CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 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 3d42c64 fix build bread if CON

[incubator-nuttx] branch master updated: fix build break with CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 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 7962179 fix build break with CONFIG

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2520: fix build break with CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 Thread GitBox
xiaoxiang781216 merged pull request #2520: URL: https://github.com/apache/incubator-nuttx/pull/2520 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] xiaoxiang781216 commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r540951299 ## File path: sched/task/task_getpid.c ## @@ -95,11 +95,17 @@ pid_t getpid(void) if (rtcb->task_state == TSTATE_TASK_RUNNING)

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r540951299 ## File path: sched/task/task_getpid.c ## @@ -95,11 +95,17 @@ pid_t getpid(void) if (rtcb->task_state == TSTATE_TASK_RUNNING)

[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
patacongo commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r540950040 ## File path: sched/task/task_getpid.c ## @@ -95,11 +95,17 @@ pid_t getpid(void) if (rtcb->task_state == TSTATE_TASK_RUNNING)

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
xiaoxiang781216 commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-743190424 > This line in pthread.h needs to change too: > > #define pthread_self() ((pthread_t)getpid()) > > Please review all uses of getpid() in the OS and m

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r540943083 ## File path: sched/task/task_getpid.c ## @@ -95,11 +95,17 @@ pid_t getpid(void) if (rtcb->task_state == TSTATE_TASK_RUNNING)

[GitHub] [incubator-nuttx-apps] anchao commented on pull request #517: wapi/conf: convert mac address

2020-12-11 Thread GitBox
anchao commented on pull request #517: URL: https://github.com/apache/incubator-nuttx-apps/pull/517#issuecomment-743189621 apps/wireless/wapi/src/util.c:79:6: error: Mixed case identifier found not a real error, mixed case from cjson call ---

[GitHub] [incubator-nuttx] patacongo commented on issue #2499: getpid should return process id not thread id

2020-12-11 Thread GitBox
patacongo commented on issue #2499: URL: https://github.com/apache/incubator-nuttx/issues/2499#issuecomment-743187288 > > > @patacongo I provide a fix here: #2518, do you think it's reasonable? If it's OK, I will review all getpid caller and change to getthid as needed. It l

[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
patacongo commented on a change in pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#discussion_r540937984 ## File path: sched/task/task_getpid.c ## @@ -95,11 +95,17 @@ pid_t getpid(void) if (rtcb->task_state == TSTATE_TASK_RUNNING)

[GitHub] [incubator-nuttx] patacongo commented on pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
patacongo commented on pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518#issuecomment-743184614 This line in pthread.h needs to change too: #define pthread_self() ((pthread_t)getpid()) Please review all uses of getpid() in the OS and make sure that th

[GitHub] [incubator-nuttx-apps] Dan-guanghua opened a new pull request #518: fix build bread if CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 Thread GitBox
Dan-guanghua opened a new pull request #518: URL: https://github.com/apache/incubator-nuttx-apps/pull/518 Change-Id: I785f44a4381965df70cbd33faeb2d2e24f6e Signed-off-by: danguanghua ## Summary ## Impact ## Testing ---

[GitHub] [incubator-nuttx] Dan-guanghua opened a new pull request #2520: fix build break with CONFIG_AUDIO_MULTI_SESSION enabled

2020-12-11 Thread GitBox
Dan-guanghua opened a new pull request #2520: URL: https://github.com/apache/incubator-nuttx/pull/2520 N/A Change-Id: Idfa87031e09f26bd4ca57b5c220ce0ca849f80c4 Signed-off-by: danguanghua ## Summary ## Impact ## Testing ---

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on issue #2499: getpid should return process id not thread id

2020-12-11 Thread GitBox
xiaoxiang781216 edited a comment on issue #2499: URL: https://github.com/apache/incubator-nuttx/issues/2499#issuecomment-743160838 @patacongo I provide a fix here: https://github.com/apache/incubator-nuttx/pull/2518/, do you think it's reasonable? If it's OK, I will review all getpid call

[GitHub] [incubator-nuttx] davids5 merged pull request #2519: arch/arm/imxrt: replace clock_systimespec with clock_systime_timespec

2020-12-11 Thread GitBox
davids5 merged pull request #2519: URL: https://github.com/apache/incubator-nuttx/pull/2519 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 t

[incubator-nuttx] branch master updated: arch/arm/imxrt: replace clock_systimespec with clock_systime_timespec

2020-12-11 Thread davids5
This is an automated email from the ASF dual-hosted git repository. davids5 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 73d4832 arch/arm/imxrt: replace clock

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #2519: arch/arm/imxrt: replace clock_systimespec with clock_systime_timespec

2020-12-11 Thread GitBox
xiaoxiang781216 opened a new pull request #2519: URL: https://github.com/apache/incubator-nuttx/pull/2519 # Summary since clock_systimespec doesn't exist anymore ## Impact Fix the potential link break. ## Testing -

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #2499: getpid should return process id not thread id

2020-12-11 Thread GitBox
xiaoxiang781216 commented on issue #2499: URL: https://github.com/apache/incubator-nuttx/issues/2499#issuecomment-743160838 @patacongo I provide a fix here: https://github.com/apache/incubator-nuttx/pull/2518/, do you think it's resanable? If it's OK, I will review all getpid caller and c

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #2518: sched: Implement getpid and gettid correctly

2020-12-11 Thread GitBox
xiaoxiang781216 opened a new pull request #2518: URL: https://github.com/apache/incubator-nuttx/pull/2518 ## Summary getpid should return the main thread id gettid should return the current thread id See the discussion here: https://github.com/apache/incubator-nuttx/issues/2499

[incubator-nuttx] branch master updated (d9d6491 -> 6158b6b)

2020-12-11 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 d9d6491 drivers/leds/ws2812: optimize reset bytes new 21cb793 include: nuttx: Fix style warnings i

[incubator-nuttx] 01/02: include: nuttx: Fix style warnings in spinlock.h

2020-12-11 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 21cb7935c5eb173f6ba9b61522d85019893f54a2 Author: Masayuki Ishikawa AuthorDate: Mon Dec 7 21:01:28 2020 +0900

[incubator-nuttx] 02/02: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 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 6158b6b77bfb396a45789d0a8f4f0f9fae06ce98 Author: Masayuki Ishikawa AuthorDate: Mon Dec 7 21:04:16 2020 +0900

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 merged pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517 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] xiaoxiang781216 commented on pull request #2488: libs/libc: correct the getrandom(2) prototype

2020-12-11 Thread GitBox
xiaoxiang781216 commented on pull request #2488: URL: https://github.com/apache/incubator-nuttx/pull/2488#issuecomment-743152250 > > Because the current implementation only support the nonblocking behaviour, the right thing is checking GRND_NONBLOCK flag and return return error directly if

[GitHub] [incubator-nuttx] juniskane edited a comment on pull request #2488: libs/libc: correct the getrandom(2) prototype

2020-12-11 Thread GitBox
juniskane edited a comment on pull request #2488: URL: https://github.com/apache/incubator-nuttx/pull/2488#issuecomment-743120568 So the plan is rename current function as `arc4random_buf` and also add the three argument Linux version of `getrandom`? I guess I have to live with that. That

[GitHub] [incubator-nuttx] juniskane commented on pull request #2488: libs/libc: correct the getrandom(2) prototype

2020-12-11 Thread GitBox
juniskane commented on pull request #2488: URL: https://github.com/apache/incubator-nuttx/pull/2488#issuecomment-743120568 So the plan is rename current function as arc4random_buf and also add the three argument Linux version? I guess I have to live with that. That way we are compatible wi

[GitHub] [incubator-nuttx] juniskane commented on pull request #2488: libs/libc: correct the getrandom(2) prototype

2020-12-11 Thread GitBox
juniskane commented on pull request #2488: URL: https://github.com/apache/incubator-nuttx/pull/2488#issuecomment-743118817 > Because the current implementation only support the nonblocking behaviour, the right thing is checking GRND_NONBLOCK flag and return return error directly if user fo

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request #517: wapi/conf: convert mac address

2020-12-11 Thread GitBox
anchao opened a new pull request #517: URL: https://github.com/apache/incubator-nuttx-apps/pull/517 ## Summary wapi/conf: convert mac address ## Impact wapi save_config/reconnect ## Testing wapi wlan0 save_config wapi wlan0 reconnect -

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#issuecomment-743113388 @xiaoxiang781216 I've just pushed with -f. This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540845173 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE_SE

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540840327 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540821191 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE_SE

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540817299 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE_SE

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540806076 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540805791 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540805534 ## File path: arch/arm/Kconfig ## @@ -789,6 +789,13 @@ config ARM_THUMB default n depends on ARCH_ARMV7A +config ARM_WFE

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540789772 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define SP

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540787596 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540774339 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define SP

[GitHub] [incubator-nuttx] masayuki2009 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
masayuki2009 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540771664 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define SP

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540766144 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #2517: spinlock: Introduce SP_WFE() and SP_SEV()

2020-12-11 Thread GitBox
xiaoxiang781216 commented on a change in pull request #2517: URL: https://github.com/apache/incubator-nuttx/pull/2517#discussion_r540766144 ## File path: arch/arm/include/spinlock.h ## @@ -95,6 +95,11 @@ #define SP_DSB(n) __asm__ __volatile__ ("dsb sy" : : : "memory") #define