[GitHub] [incubator-nuttx] Ouss4 merged pull request #5178: arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S

2022-01-06 Thread GitBox
Ouss4 merged pull request #5178: URL: https://github.com/apache/incubator-nuttx/pull/5178 -- 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-

[incubator-nuttx] branch master updated: arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S

2022-01-06 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche 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 3982296 arch/sim: Rename up_vfork

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #5164: Add test to CI to verify that the `context` rule doesn't need to be rebuilt between runs

2022-01-06 Thread GitBox
Ouss4 commented on a change in pull request #5164: URL: https://github.com/apache/incubator-nuttx/pull/5164#discussion_r779385033 ## File path: tools/testbuild.sh ## @@ -258,6 +258,10 @@ function build { fail=1 fi + # Verify that the context rule does not need to be

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5175: Minor fix for input Kconfig

2022-01-06 Thread GitBox
xiaoxiang781216 merged pull request #5175: URL: https://github.com/apache/incubator-nuttx/pull/5175 -- 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 (3982296 -> afacc8d)

2022-01-06 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 3982296 arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S add 3c86a3c input: Remove prompt

[GitHub] [incubator-nuttx] pkarashchenko opened a new pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko opened a new pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180 ## Summary According to https://pubs.opengroup.org/onlinepubs/9699919799 the `The protocol attribute defines the protocol to be followed in utilizing mutexes.` and `The default value

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5070: libc/semaphore:sem_init change defult protocol

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1006443287 I started similar `pthread mutex` related activity in https://github.com/apache/incubator-nuttx/pull/5180 -- This is an automated message from the Apache Git Se

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006454958 Since phtread_mutex_t is always used as lock, the priority inherit isn't harmful as the sem_t used as signal. So an option to select the default value is reason

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779449804 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006459951 > Since phtread_mutex_t is always used as lock, the priority inherit isn't harmful as the sem_t used as signal. So an option to select the default value is reason

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006462476 @xiaoxiang781216 do you think it should be enabled by default? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] pkarashchenko edited a comment on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko edited a comment on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006462476 @xiaoxiang781216 do you think the priority inheritance should be enabled by default? -- This is an automated message from the Apache Git Service. To resp

[GitHub] [incubator-nuttx] pkarashchenko edited a comment on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko edited a comment on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006462476 @xiaoxiang781216 do you think the priority inheritance as default protocol should be enabled by default? -- This is an automated message from the Apache

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779457547 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779457573 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779457573 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779457547 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] acassis commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
acassis commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006498805 > @xiaoxiang781216 do you think the priority inheritance as default protocol should be enabled by default? @pkarashchenko I think enabling priority inheritance as

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779482914 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,17 +1011,52 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006521448 If the usage is fixed as lock(e.g. pthread_mutex_t, mutex_t), it's good to enable the priority inheritance by default. If the usage isn't fixed like sem_t, we h

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006523562 > > @xiaoxiang781216 do you think the priority inheritance as default protocol should be enabled by default? > > @pkarashchenko I think enabling priority in

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006525216 > If the usage is fixed as lock(e.g. pthread_mutex_t, mutex_t), it's good to enable the priority inheritance by default. If the usage isn't fixed like sem_t, we h

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779503516 ## File path: drivers/spi/qspi_flash.c ## @@ -178,22 +171,22 @@ struct sim_qspiflashdev_s /* QSPI methods */ -static int qspifl

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779510028 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006548893 > > If the usage is fixed as lock(e.g. pthread_mutex_t, mutex_t), it's good to enable the priority inheritance by default. If the usage isn't fixed like sem_t,

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006553929 > > > If the usage is fixed as lock(e.g. pthread_mutex_t, mutex_t), it's good to enable the priority inheritance by default. If the usage isn't fixed like sem_t,

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006558468 It's minor issue, I am fine with either:). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779530236 ## File path: arch/sim/src/sim/up_initialize.c ## @@ -125,12 +128,12 @@ static void up_init_smartfs(void) } } #endif -#endi

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779530569 ## File path: drivers/spi/qspi_flash.c ## @@ -178,22 +171,22 @@ struct sim_qspiflashdev_s /* QSPI methods */ -static int qspi

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779530975 ## File path: drivers/spi/qspi_flash.c ## @@ -178,22 +171,22 @@ struct sim_qspiflashdev_s /* QSPI methods */ -static int qspi

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779531126 ## File path: drivers/spi/qspi_flash.c ## @@ -178,22 +171,22 @@ struct sim_qspiflashdev_s /* QSPI methods */ -static int qspi

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779510028 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779510028 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779510028 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779510028 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] a-lunev opened a new pull request #5181: net/devif/devif_callback.c: devif_callback_free() call is not needed anymore in devif_callback_alloc()

2022-01-06 Thread GitBox
a-lunev opened a new pull request #5181: URL: https://github.com/apache/incubator-nuttx/pull/5181 ## Summary As a result of PR #4575, devif_callback_free() call is not needed anymore in devif_callback_alloc(). Moreover, with NULL second argument devif_callback_free() does not do

[GitHub] [incubator-nuttx] a-lunev commented on pull request #4575: net/devif: Fix the memory leak in case of netdev isn't alive

2022-01-06 Thread GitBox
a-lunev commented on pull request #4575: URL: https://github.com/apache/incubator-nuttx/pull/4575#issuecomment-1006602259 > To fix this problem, it's better to add the lock in netdev_ifdown. In that case just turning the network interface down will be deferred. And since netdev_ifdow

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5182: Move simulated drivers from sim to common place

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new pull request #5182: URL: https://github.com/apache/incubator-nuttx/pull/5182 ## Summary Since it's good demo for driver writer: - arch/sim: Move the dummy foc driver to drivers/motor/foc - arch/sim: Move the dummy ioe driver to drivers/ioexpender

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5183: ioexpander/rpmsg: Select IOEXPANDER_INT_ENABLE instead depend on it

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new pull request #5183: URL: https://github.com/apache/incubator-nuttx/pull/5183 ## Summary follow other similiar drivers ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-nuttx-apps] acassis opened a new issue #962: Improvement: pwm application needs to support parameter for multichannel pwm

2022-01-06 Thread GitBox
acassis opened a new issue #962: URL: https://github.com/apache/incubator-nuttx-apps/issues/962 Currently the pwm test application doesn't have a parameter to let user to select what channel he/she want to use. -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue #5184: Replace all place which use semaphore as lock with mutex wrapper

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new issue #5184: URL: https://github.com/apache/incubator-nuttx/issues/5184 See the discussion: https://github.com/apache/incubator-nuttx/pull/5070 and https://www.mail-archive.com/dev@nuttx.apache.org/msg07312.html -- This is an automated message from the Apache

[incubator-nuttx] branch master updated: net/devif/devif_callback.c: devif_callback_free() call is not needed anymore in devif_callback_alloc()

2022-01-06 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 8042de1 net/devif/devif_callback.c:

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5181: net/devif/devif_callback.c: devif_callback_free() call is not needed anymore in devif_callback_alloc()

2022-01-06 Thread GitBox
xiaoxiang781216 merged pull request #5181: URL: https://github.com/apache/incubator-nuttx/pull/5181 -- 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] davids5 commented on pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
davids5 commented on pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#issuecomment-1006653867 Would a README with links to information (mfg, datasheets) add value here? -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#issuecomment-1006663170 Here is some introduction info: https://www.ceva-dsp.com/product/ceva-xm6/ https://www.ceva-dsp.com/product/ceva-xc16/ Basically, these DSPs are designed

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006736204 > It's minor issue, I am fine with either:). Done. @xiaoxiang781216 @acassis please take a look. -- This is an automated message from the Apache Git Se

[GitHub] [incubator-nuttx] acassis commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
acassis commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006758270 > > It's minor issue, I am fine with either:). > > Done. @xiaoxiang781216 @acassis please take a look. So, let see if I understood correctly: when CONFIG_PR

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5185: sched/Kconfig: Remove CONFIG_ prefix from SCHED_HPWORK

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new pull request #5185: URL: https://github.com/apache/incubator-nuttx/pull/5185 ## Summary Fix typo error ## Impact Make SIG_EVTHREAD_HPWORK selectable ## Testing Pass CI -- This is an automated message from the Apache Git Service. To r

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#issuecomment-1006803230 @pkarashchenko the new patch is used to verify the change work. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
xiaoxiang781216 edited a comment on pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#issuecomment-1006663170 Here is some introduction info: https://www.ceva-dsp.com/product/ceva-xm6/ https://www.ceva-dsp.com/product/ceva-xc16/ Basically, these DSPs are d

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006808808 > > > It's minor issue, I am fine with either:). > > > > > > Done. @xiaoxiang781216 @acassis please take a look. > > So, let see if I understood c

[GitHub] [incubator-nuttx] pkarashchenko edited a comment on pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko edited a comment on pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006808808 > > > It's minor issue, I am fine with either:). > > > > > > Done. @xiaoxiang781216 @acassis please take a look. > > So, let see if I under

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5186: board/sim: Turn on more rpmsg drivers in rpserver config

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new pull request #5186: URL: https://github.com/apache/incubator-nuttx/pull/5186 ## Summary to improve the build and test coverage ## Impact sim ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-nuttx] acassis opened a new pull request #5187: esp32: Add PWM support using the LEDC peripheral @acassis

2022-01-06 Thread GitBox
acassis opened a new pull request #5187: URL: https://github.com/apache/incubator-nuttx/pull/5187 ## Summary This patch add support to PWM for ESP32 ## Impact User will be able to use PWM ## Testing ESP32-Devkitc -- This is an automated message from the Apache Git Servi

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
acassis commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779800516 ## File path: arch/ceva/include/spinlock.h ## @@ -0,0 +1,106 @@ +/ + *

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5187: esp32: Add PWM support using the LEDC peripheral

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5187: URL: https://github.com/apache/incubator-nuttx/pull/5187#discussion_r779828273 ## File path: arch/xtensa/src/esp32/Kconfig ## @@ -324,9 +324,10 @@ config ESP32_I2S1 No yet implemented config ESP32_LED

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5187: esp32: Add PWM support using the LEDC peripheral

2022-01-06 Thread GitBox
acassis commented on a change in pull request #5187: URL: https://github.com/apache/incubator-nuttx/pull/5187#discussion_r779845390 ## File path: arch/xtensa/src/esp32/Kconfig ## @@ -324,9 +324,10 @@ config ESP32_I2S1 No yet implemented config ESP32_LEDC -

[GitHub] [incubator-nuttx] Ouss4 merged pull request #5183: ioexpander/rpmsg: Select IOEXPANDER_INT_ENABLE instead depend on it

2022-01-06 Thread GitBox
Ouss4 merged pull request #5183: URL: https://github.com/apache/incubator-nuttx/pull/5183 -- 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-

[incubator-nuttx] branch master updated: ioexpander/rpmsg: Select IOEXPANDER_INT_ENABLE instead depend on it

2022-01-06 Thread aguettouche
This is an automated email from the ASF dual-hosted git repository. aguettouche 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 8dc2ce7 ioexpander/rpmsg: Select

[GitHub] [incubator-nuttx] Ouss4 commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
Ouss4 commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779885929 ## File path: arch/ceva/include/spinlock.h ## @@ -0,0 +1,106 @@ +/ + * a

[GitHub] [incubator-nuttx] acassis merged pull request #5185: sched/Kconfig: Remove CONFIG_ prefix from SCHED_HPWORK

2022-01-06 Thread GitBox
acassis merged pull request #5185: URL: https://github.com/apache/incubator-nuttx/pull/5185 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commit

[GitHub] [incubator-nuttx] acassis commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
acassis commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r779886637 ## File path: arch/ceva/include/spinlock.h ## @@ -0,0 +1,106 @@ +/ + *

[incubator-nuttx] branch master updated (8dc2ce7 -> 964f4b7)

2022-01-06 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from 8dc2ce7 ioexpander/rpmsg: Select IOEXPANDER_INT_ENABLE instead depend on it add 964f4b7 sched/Kconf

[GitHub] [incubator-nuttx-apps] a-lunev opened a new pull request #963: crypto/libtomcrypt: added CONFIG_LIBTOMCRYPT_HASHSUM option to build hashsum app

2022-01-06 Thread GitBox
a-lunev opened a new pull request #963: URL: https://github.com/apache/incubator-nuttx-apps/pull/963 ## Summary Added hashsum app build option based on libtomcrypt. ## Impact ## Testing ``` $ ./tools/configure.sh -l sim:tcpblaster $ make menuconfig (enable CONFI

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779927647 ## File path: drivers/spi/spi_flash.c ## @@ -642,15 +635,15 @@ static void spiflash_sectorerase(FAR struct sim_spiflashdev_s *priv) if (

[GitHub] [incubator-nuttx] yamt commented on pull request #5178: arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S

2022-01-06 Thread GitBox
yamt commented on pull request #5178: URL: https://github.com/apache/incubator-nuttx/pull/5178#issuecomment-1007029521 > to align with up_vfork_arm.S naming style why not `x86` and `x86_64` as we use in other places? -- This is an automated message from the Apache Git Service.

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

2022-01-06 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 77b9a69 Publishing web

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779978977 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779979899 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r779979899 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5179: arch/sim: Move spiflash simulation to drivers/spi instead

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5179: URL: https://github.com/apache/incubator-nuttx/pull/5179#discussion_r779989451 ## File path: drivers/spi/spi_flash.c ## @@ -642,15 +635,15 @@ static void spiflash_sectorerase(FAR struct sim_spiflashdev_s *priv) if

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #5188: arch/sim: Rename up_vfork_x[32|64].S to up_vfork_x86[_64].S

2022-01-06 Thread GitBox
xiaoxiang781216 opened a new pull request #5188: URL: https://github.com/apache/incubator-nuttx/pull/5188 ## Summary to follow other arch/x86 arch/x86_64 convention ## Impact Minor ## Testing Pass CI -- This is an automated message from the Apache Git Service. T

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5178: arch/sim: Rename up_vfork[32|64].S to up_vfork_x[32|64].S

2022-01-06 Thread GitBox
xiaoxiang781216 commented on pull request #5178: URL: https://github.com/apache/incubator-nuttx/pull/5178#issuecomment-1007099567 Yes, the proposal is better, fix here: https://github.com/apache/incubator-nuttx/pull/5188 -- This is an automated message from the Apache Git Service. To res

[GitHub] [incubator-nuttx] anchao opened a new pull request #5189: Kconfig: Remove CONFIG_ prefix from config definition

2022-01-06 Thread GitBox
anchao opened a new pull request #5189: URL: https://github.com/apache/incubator-nuttx/pull/5189 ## Summary Kconfig: Remove CONFIG_ prefix from config definition ## Impact N/A ## Testing CI check -- This is an automated message from the Apache Git Servic

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#discussion_r779998603 ## File path: libs/libc/pthread/pthread_mutexattr_setprotocol.c ## @@ -54,21 +54,24 @@ int pthread_mutexattr_setprotocol(FAR pthread_mutex

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#discussion_r779998603 ## File path: libs/libc/pthread/pthread_mutexattr_setprotocol.c ## @@ -54,21 +54,24 @@ int pthread_mutexattr_setprotocol(FAR pthread_mutex

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5165: CEVA: add ceva platform xc5 xm6 support

2022-01-06 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5165: URL: https://github.com/apache/incubator-nuttx/pull/5165#discussion_r78760 ## File path: arch/ceva/include/types.h ## @@ -0,0 +1,96 @@ +/

[incubator-nuttx-apps] branch master updated: crypto/libtomcrypt: added CONFIG_LIBTOMCRYPT_HASHSUM option to build hashsum app

2022-01-06 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 61fe9c4 crypto/libtomcrypt: ad

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #963: crypto/libtomcrypt: added CONFIG_LIBTOMCRYPT_HASHSUM option to build hashsum app

2022-01-06 Thread GitBox
xiaoxiang781216 merged pull request #963: URL: https://github.com/apache/incubator-nuttx-apps/pull/963 -- 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] zhaoxiu-zeng commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
zhaoxiu-zeng commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r780004722 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[incubator-nuttx] branch master updated (964f4b7 -> 8c35d31)

2022-01-06 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 964f4b7 sched/Kconfig: Remove CONFIG_ prefix from SCHED_HPWORK add 8c35d31 Kconfig: Remove CONFIG_

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5189: Kconfig: Remove CONFIG_ prefix from config definition

2022-01-06 Thread GitBox
xiaoxiang781216 merged pull request #5189: URL: https://github.com/apache/incubator-nuttx/pull/5189 -- 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] pkarashchenko commented on a change in pull request #5180: pthread: restore pthread mutex default protocol POSIX compatibility

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5180: URL: https://github.com/apache/incubator-nuttx/pull/5180#discussion_r780049007 ## File path: libs/libc/pthread/pthread_mutexattr_setprotocol.c ## @@ -54,21 +54,24 @@ int pthread_mutexattr_setprotocol(FAR pthread_mutexat

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r780051492 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5171: semphore: release all semphores' holder that the task held when exit

2022-01-06 Thread GitBox
pkarashchenko commented on a change in pull request #5171: URL: https://github.com/apache/incubator-nuttx/pull/5171#discussion_r780052416 ## File path: sched/semaphore/sem_holder.c ## @@ -1035,18 +1014,57 @@ void nxsem_release_holder(FAR sem_t *sem) { FAR struct tcb_s *rtcb

[GitHub] [incubator-nuttx-apps] anchao opened a new pull request #964: testing/ostest: set the default value to TESTING_OSTEST_FPUSIZE

2022-01-06 Thread GitBox
anchao opened a new pull request #964: URL: https://github.com/apache/incubator-nuttx-apps/pull/964 ## Summary testing/ostest: set the default value to TESTING_OSTEST_FPUSIZE .config:2053:warning: symbol value '' invalid for TESTING_OSTEST_FPUSIZE ## Impact ostes

[GitHub] [incubator-nuttx] anchao opened a new pull request #5190: drivers/ioexpander: fix the kconfig warning

2022-01-06 Thread GitBox
anchao opened a new pull request #5190: URL: https://github.com/apache/incubator-nuttx/pull/5190 ## Summary drivers/ioexpander: fix the kconfig warning drivers/ioexpander/Kconfig:28: warning: config symbol 'IOEXPANDER_RPMSG_INT_NCALLBACKS' uses select, but is not bo

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5188: arch/sim: Rename up_vfork_x[32|64].S to up_vfork_x86[_64].S

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