[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1005262042 ## include/nuttx/mutex.h: ## @@ -522,11 +523,11 @@ static inline int nxrmutex_breaklock(FAR rmutex_t *rmutex, pid_t tid = gettid(); int ret = -EPE

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request, #1386: Fix Error: ftpd.c:1773:9: error: variable 'pos' set but not used [-Werror,-Wunused-but-set-variable]

2022-10-25 Thread GitBox
xiaoxiang781216 opened a new pull request, #1386: URL: https://github.com/apache/incubator-nuttx-apps/pull/1386 ## Summary Report by: https://github.com/apache/incubator-nuttx/actions/runs/3318241409/jobs/5482179373 ## Impact Fix warning ## Testing Pass CI -- Th

[GitHub] [incubator-nuttx] robertobucher opened a new pull request, #7432: New files for pysimCoder

2022-10-25 Thread GitBox
robertobucher opened a new pull request, #7432: URL: https://github.com/apache/incubator-nuttx/pull/7432 ## Summary Files under board/arm/stm32f7/nucleo-144 f746-pysim/defconfig include/board.h src/nucleo-144.h src/stm32_appinitialize.c src//stm32_bringup.c src/stm32_gpio

[GitHub] [incubator-nuttx] robertobucher commented on pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
robertobucher commented on PR #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428#issuecomment-1291502074 I'll try to restart the full PR! -- 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

[GitHub] [incubator-nuttx] robertobucher closed pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
robertobucher closed pull request #7428: New files for pysimCoder URL: https://github.com/apache/incubator-nuttx/pull/7428 -- 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 un

[GitHub] [incubator-nuttx] XinStellaris commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1005189882 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s *

[GitHub] [incubator-nuttx] XinStellaris commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1005188835 ## include/nuttx/mutex.h: ## @@ -522,11 +523,11 @@ static inline int nxrmutex_breaklock(FAR rmutex_t *rmutex, pid_t tid = gettid(); int ret = -EPERM;

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7431: net: move IPv4/6BUF define to common header

2022-10-25 Thread GitBox
anchao opened a new pull request, #7431: URL: https://github.com/apache/incubator-nuttx/pull/7431 ## Summary net: move IPv4/6BUF define to common header Signed-off-by: chao an ## Impact N/A ## Testing tcp/udp iperf test -- This is an automated

[incubator-nuttx] branch master updated: CI: add example to sim

2022-10-25 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 f361d1cd41 CI: add example to sim f3

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7383: CI: add example to sim

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7383: URL: https://github.com/apache/incubator-nuttx/pull/7383 -- 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-unsubs

[incubator-nuttx] branch master updated: libs/libc: add obstack

2022-10-25 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 61b1791584 libs/libc: add obstack 61

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7198: libs/libc: add obstack

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7198: URL: https://github.com/apache/incubator-nuttx/pull/7198 -- 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-unsubs

[GitHub] [incubator-nuttx] ttnie commented on a diff in pull request #7383: CI: add example to sim

2022-10-25 Thread GitBox
ttnie commented on code in PR #7383: URL: https://github.com/apache/incubator-nuttx/pull/7383#discussion_r1005094312 ## tools/ci/testrun/script/test_example/test_example.py: ## @@ -0,0 +1,36 @@ +#!/usr/bin/python3 +# encoding: utf-8 +import pytest + +pytestmark = [pytest.mark.co

[incubator-nuttx-website] branch asf-site updated: Publishing web: d696c67810809c3f81a4dd87b09d9e940db854b4 docs: 739210708aa864d08e57544a9bf2e18fe106879d

2022-10-25 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 95ff002 Publishing web

[GitHub] [incubator-nuttx] acassis commented on pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
acassis commented on PR #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428#issuecomment-1291246549 @robertobucher you need to follow the workflow explained here: https://nuttx.apache.org/docs/latest/contributing/making-changes.html#git-workflow-with-an-upstream-repository

[incubator-nuttx] branch master updated: arch: Replace __builtin_bswapxx with bswapxx

2022-10-25 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 739210708a arch: Replace __builtin_bsw

[GitHub] [incubator-nuttx] acassis merged pull request #7430: arch: Replace __builtin_bswapxx with bswapxx

2022-10-25 Thread GitBox
acassis merged PR #7430: URL: https://github.com/apache/incubator-nuttx/pull/7430 -- 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...@nu

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7429: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7429: URL: https://github.com/apache/incubator-nuttx/pull/7429#discussion_r1004942723 ## include/nuttx/audio/i2s.h: ## @@ -286,6 +309,11 @@ struct i2s_ops_s FAR void *arg, uint32_t t

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7429: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7429: URL: https://github.com/apache/incubator-nuttx/pull/7429#discussion_r1004942723 ## include/nuttx/audio/i2s.h: ## @@ -286,6 +309,11 @@ struct i2s_ops_s FAR void *arg, uint32_t t

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7430: arch: Replace __builtin_bswapxx with bswapxx

2022-10-25 Thread GitBox
xiaoxiang781216 opened a new pull request, #7430: URL: https://github.com/apache/incubator-nuttx/pull/7430 ## Summary Code refactor only ## Impact same as before ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond to the mes

[incubator-nuttx] 05/08: drivers/video/isx019: Increase waiting time for HW initialization

2022-10-25 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 d57324473e12736f954abac1c318793fe6d28dc9 Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 04/08: drivers/video/isx012: Add missing FAR qualifier

2022-10-25 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 254139bcec607058f382b8d203205fb1b7ab8eec Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 08/08: drivers/video/isx019: Use clock function instead of gettimeofday

2022-10-25 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 6eb0ef618d3e15c812007f304f59788c400e5527 Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 07/08: drivers/video/isx019: Fix incorrect JPEG DQT table

2022-10-25 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 8a0ce3f0006d0cb1071f78d97587e6b78ba5 Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 01/08: drivers/video: Add get frame interval API

2022-10-25 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 1332bdb9a7aa1b97b384caf79082cc8d2f540f4e Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] branch master updated (8684814e09 -> 6eb0ef618d)

2022-10-25 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 8684814e09 strftime: Add support for the %T specifier new 1332bdb9a7 drivers/video: Add get frame int

[incubator-nuttx] 06/08: drivers/video/isx019: Fix white balance control and capability

2022-10-25 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 5e4e8f680978f9f01b84783bda3a08096e70971a Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 03/08: drivers/video/isx012: Support clip feature

2022-10-25 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 e48f8be8c539dc22a36a819a8969ab6be56f7869 Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[incubator-nuttx] 02/08: drivers/video: Update API to dump register for debug

2022-10-25 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 277c8e368bb05cd0e8cd65f5f0741e27389b414d Author: SPRESENSE <41312067+sprese...@users.noreply.github.com> Auth

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
xiaoxiang781216 commented on PR #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428#issuecomment-1290992429 please see here: https://github.com/apache/incubator-nuttx/pull/7428/commits the second commit should be dropped. -- This is an automated message from the Apache Gi

[GitHub] [incubator-nuttx] robertobucher commented on pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
robertobucher commented on PR #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428#issuecomment-1290896402 I can't understand exactly what you mean with "remove the merge patch"... I did a pull request of my master branch before putting my modifications and launch the PR... --

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7428: New files for pysimCoder

2022-10-25 Thread GitBox
xiaoxiang781216 commented on PR #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428#issuecomment-1290890555 @robertobucher could you rebase your change to remove the merge patch? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7389: Minor improvement for tools/ci/cibuild.sh

2022-10-25 Thread GitBox
pkarashchenko commented on PR #7389: URL: https://github.com/apache/incubator-nuttx/pull/7389#issuecomment-1290889129 Please fix MAC OS variant -- 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] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004752632 ## drivers/video/isx019.c: ## @@ -1076,34 +1086,42 @@ static int set_drive_mode(void) static bool try_repeat(int sec, int usec, CODE int (*trial_func)(vo

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7383: CI: add example to sim

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7383: URL: https://github.com/apache/incubator-nuttx/pull/7383#discussion_r1004742930 ## tools/ci/testrun/script/test_example/test_example.py: ## @@ -0,0 +1,44 @@ +#!/usr/bin/python3 +# encoding: utf-8 +import pytest + +pytestmark = [pyte

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7420: Update errno to match asm-generic

2022-10-25 Thread GitBox
xiaoxiang781216 commented on PR #7420: URL: https://github.com/apache/incubator-nuttx/pull/7420#issuecomment-1290868010 > What is the motivation for this PR? Why should we align with Linux and not Open/Free BSD for example? I will need some more time to look into these changes. Here

[incubator-nuttx] 02/02: strftime: Add support for the %T specifier

2022-10-25 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 8684814e09771012338eb1b39e69bd20a5c9aa1e Author: Xavier Del Campo AuthorDate: Tue Oct 25 08:54:28 2022 +0200

[incubator-nuttx] branch master updated (b5052a8475 -> 8684814e09)

2022-10-25 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 b5052a8475 board/stm32h7: update mcuboot-loader configuration new 097faac22f strftime: Add support fo

[incubator-nuttx] 01/02: strftime: Add support for the %R specifier

2022-10-25 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 097faac22f308a19d0ae2f61cc707ba3f99f8e67 Author: Xavier Del Campo AuthorDate: Tue Oct 25 08:54:23 2022 +0200

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7424: Add %T and %R conversion specifiers to strftime(3)

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7424: URL: https://github.com/apache/incubator-nuttx/pull/7424 -- 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-unsubs

[incubator-nuttx] branch master updated (f3256e5668 -> b5052a8475)

2022-10-25 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 f3256e5668 stm32fh7:Allow the use of the Network Monitor via polling add b5052a8475 board/stm32h7: up

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7427: board/stm32h7: update mcuboot-loader configuration

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7427: URL: https://github.com/apache/incubator-nuttx/pull/7427 -- 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-unsubs

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004703784 ## drivers/video/isx012.c: ## @@ -1827,7 +1977,7 @@ static int isx012_get_value(uint32_t id, uint8_tcnt; uint8_tthreea_enable; uint16_t rea

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004704105 ## drivers/video/isx012.c: ## @@ -825,7 +845,134 @@ static bool is_movie_needed(uint8_t fmt, uint8_t fps) return need; } -static int isx012_set_mode_par

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004703524 ## drivers/video/isx012.c: ## @@ -2150,14 +2300,57 @@ static int isx012_get_value(uint32_t id, return ret; } +static bool validate_clip_setting(uint32_t

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004703029 ## drivers/video/isx019.c: ## @@ -1076,34 +1086,42 @@ static int set_drive_mode(void) static bool try_repeat(int sec, int usec, CODE int (*trial_func)(void))

[GitHub] [incubator-nuttx] tmedicci commented on pull request #7429: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-25 Thread GitBox
tmedicci commented on PR #7429: URL: https://github.com/apache/incubator-nuttx/pull/7429#issuecomment-1290829333 Following https://github.com/apache/incubator-nuttx/pull/7421#issuecomment-1289502232 I've tested and found that the audio codec also supports 16KHz and 22,05KHz sample ra

[GitHub] [incubator-nuttx] tmedicci opened a new pull request, #7429: audio/i2s: add i2s_mclkfrequency method to the I2S lower half

2022-10-25 Thread GitBox
tmedicci opened a new pull request, #7429: URL: https://github.com/apache/incubator-nuttx/pull/7429 ## Summary This method allows the codec driver to set a specific mclk frequency for the I2S lower half, allowing to change it on run time according to the supported master clock frequency

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004701473 ## drivers/video/isx019.c: ## @@ -2128,7 +2226,53 @@ static int set_wbmode(imgsensor_value_t val) isx019_i2c_write(CAT_CATAWB, AWBUSER_NO, (FAR uint8_t *)&

[GitHub] [incubator-nuttx] SPRESENSE commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004701232 ## drivers/video/isx019.c: ## @@ -1503,30 +1528,27 @@ static int isx019_start_capture(imgsensor_stream_type_t type, } switch (fmt[IMGSENSOR_FMT_MAI

[GitHub] [incubator-nuttx] robertobucher opened a new pull request, #7428: New files for pysimCoder

2022-10-25 Thread GitBox
robertobucher opened a new pull request, #7428: URL: https://github.com/apache/incubator-nuttx/pull/7428 ## Summary Files under board/arm/stm32f7/nucleo-144 f746-pysim/defconfig include/board.h src/nucleo-144.h src/stm32_appinitialize.c src//stm32_bringup.c src/stm32_gpio

[incubator-nuttx-apps] branch master updated: examples/camera: Dynamically get the connected device

2022-10-25 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 c3f91b3a0 examples/camera: Dyna

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1384: examples/camera: Dynamically get the connected device

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #1384: URL: https://github.com/apache/incubator-nuttx-apps/pull/1384 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-u

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004578059 ## include/nuttx/mutex.h: ## @@ -522,11 +523,11 @@ static inline int nxrmutex_breaklock(FAR rmutex_t *rmutex, pid_t tid = gettid(); int ret = -EPE

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004573267 ## libs/libc/sched/sched_dumpstack.c: ## @@ -53,8 +53,28 @@ void sched_dumpstack(pid_t tid) { + FAR struct tcb_s *tcb = NULL; int size = DUMP_DE

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004570297 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004569154 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7417: stm32fh7:Allow the use of the Network Monitor via polling

2022-10-25 Thread GitBox
xiaoxiang781216 merged PR #7417: URL: https://github.com/apache/incubator-nuttx/pull/7417 -- 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-unsubs

[incubator-nuttx] branch master updated: stm32fh7:Allow the use of the Network Monitor via polling

2022-10-25 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 f3256e5668 stm32fh7:Allow the use of

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7389: Minor improvement for tools/ci/cibuild.sh

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7389: URL: https://github.com/apache/incubator-nuttx/pull/7389#discussion_r1004511122 ## tools/ci/cibuild.sh: ## @@ -55,50 +55,135 @@ function add_path { EXTRA_PATH=$1:${EXTRA_PATH} } -function python-tools { - # Python User Env -

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004495297 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s

[GitHub] [incubator-nuttx] XinStellaris commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004487107 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s *

[GitHub] [incubator-nuttx] XinStellaris commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004483651 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s *

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7389: Minor improvement for tools/ci/cibuild.sh

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7389: URL: https://github.com/apache/incubator-nuttx/pull/7389#discussion_r1004482163 ## tools/ci/cibuild.sh: ## @@ -55,50 +55,135 @@ function add_path { EXTRA_PATH=$1:${EXTRA_PATH} } -function python-tools { - # Python User Env - P

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004465123 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004464049 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_

[GitHub] [incubator-nuttx] GUIDINGLI commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
GUIDINGLI commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004436815 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s *tcb

[GitHub] [incubator-nuttx] tito97sp opened a new pull request, #7427: board/stm32h7: update mcuboot-loader configuration

2022-10-25 Thread GitBox
tito97sp opened a new pull request, #7427: URL: https://github.com/apache/incubator-nuttx/pull/7427 Signed-off-by: Andrés Sánchez Pascual ## Summary Update mcuboot-loader config for STM32H743ZI ## Impact STM32H743ZI board. Minor ## Testing -- This is an

[GitHub] [incubator-nuttx] midokura-xavi92 commented on a diff in pull request #7424: Add %T and %R conversion specifiers to strftime(3)

2022-10-25 Thread GitBox
midokura-xavi92 commented on code in PR #7424: URL: https://github.com/apache/incubator-nuttx/pull/7424#discussion_r1004455480 ## libs/libc/time/lib_strftime.c: ## @@ -368,6 +368,15 @@ size_t strftime(FAR char *s, size_t max, FAR const char *format, }

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
pkarashchenko commented on PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#issuecomment-1290501958 This change reminded me https://github.com/apache/incubator-nuttx/pull/6376#discussion_r893228330 discussion. -- This is an automated message from the Apache Git Service.

[GitHub] [incubator-nuttx] GUIDINGLI commented on pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
GUIDINGLI commented on PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#issuecomment-1290499226 > I am not sure whether we should print mutex holder in sched_dumpstack. It may be helpful when we lack coredump or gdb. In a real device and there is no JTAG (mainly for

[GitHub] [incubator-nuttx] GUIDINGLI commented on a diff in pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
GUIDINGLI commented on code in PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#discussion_r1004436815 ## libs/libc/sched/sched_dumpstack.c: ## @@ -55,6 +55,26 @@ void sched_dumpstack(pid_t tid) { int size = DUMP_DEPTH; int skip; + FAR struct tcb_s *tcb

[GitHub] [incubator-nuttx] XinStellaris commented on pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#issuecomment-1290484908 I am not sure whether we should print mutex holder in sched_dumpstack. It may be helpful when we lack coredump or gdb. -- This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] XinStellaris commented on pull request #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris commented on PR #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426#issuecomment-1290475970 @xiaoxiang781216 @liguiding please review -- 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] [incubator-nuttx] XinStellaris opened a new pull request, #7426: mutex:move mutex holder to struct mutex_s.

2022-10-25 Thread GitBox
XinStellaris opened a new pull request, #7426: URL: https://github.com/apache/incubator-nuttx/pull/7426 Signed-off-by: 田昕 ## Summary We move the mutex holder to struct mutex_s so that we can trace who holds the mutex when deadlock happens. We also print possible mutex holder in

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7383: CI: add example to sim

2022-10-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7383: URL: https://github.com/apache/incubator-nuttx/pull/7383#discussion_r1004372513 ## tools/ci/testrun/script/test_example/test_example.py: ## @@ -0,0 +1,36 @@ +#!/usr/bin/python3 +# encoding: utf-8 +import pytest + +pytestmark = [pyte

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#discussion_r1004277736 ## drivers/video/isx019.c: ## @@ -2128,7 +2226,53 @@ static int set_wbmode(imgsensor_value_t val) isx019_i2c_write(CAT_CATAWB, AWBUSER_NO, (FAR uint8_t

[GitHub] [incubator-nuttx] davids5 commented on pull request #7417: stm32fh7:Allow the use of the Network Monitor via polling

2022-10-25 Thread GitBox
davids5 commented on PR #7417: URL: https://github.com/apache/incubator-nuttx/pull/7417#issuecomment-1290305422 @acassis - Thank you for catching that, I fixed it in all the archs -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[incubator-nuttx] branch master updated (4c71075ea5 -> f018a9cb54)

2022-10-25 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 4c71075ea5 arch/arm/stm32h7: multiple fixes for stm32h7 flash interface add f018a9cb54 Makefile:

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #7419: Makefile: Remove INCDIROPT

2022-10-25 Thread GitBox
pkarashchenko merged PR #7419: URL: https://github.com/apache/incubator-nuttx/pull/7419 -- 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] [incubator-nuttx] pkarashchenko commented on a diff in pull request #7424: Add %T and %R conversion specifiers to strftime(3)

2022-10-25 Thread GitBox
pkarashchenko commented on code in PR #7424: URL: https://github.com/apache/incubator-nuttx/pull/7424#discussion_r1004269568 ## libs/libc/time/lib_strftime.c: ## @@ -368,6 +368,15 @@ size_t strftime(FAR char *s, size_t max, FAR const char *format, }

[incubator-nuttx] branch master updated: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 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 4c71075ea5 arch/arm/stm32h7: multiple

[GitHub] [incubator-nuttx] davids5 merged pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
davids5 merged PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401 -- 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...@nu

[GitHub] [incubator-nuttx] tito97sp commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
tito97sp commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290261126 Test passed on local. Ready to merge @pkarashchenko @xiaoxiang781216 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [incubator-nuttx] SPRESENSE commented on pull request #7416: drivers: video: Update some features

2022-10-25 Thread GitBox
SPRESENSE commented on PR #7416: URL: https://github.com/apache/incubator-nuttx/pull/7416#issuecomment-1290246800 I addd a commit changed not to use `timercmp()` that caused the CI error. please review again. -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-nuttx] tito97sp commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
tito97sp commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290224461 > > > > @tito97sp should we merge this patch instead yours(#7397)? > > > > > > > > > @xiaoxiang781216 Yes. This PR substitutes mine. > > > > > > Could you pl

[GitHub] [incubator-nuttx] tito97sp commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
tito97sp commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290219433 > > > @tito97sp should we merge this patch instead yours(#7397)? > > > > > > @xiaoxiang781216 Yes. This PR substitutes mine. > > Could you please approve the PR

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
pkarashchenko commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290190758 > > @tito97sp should we merge this patch instead yours(#7397)? > > @xiaoxiang781216 Yes. This PR substitutes mine. Could you please approve the PR if all the te

[GitHub] [incubator-nuttx] Gary-Hobson opened a new pull request, #7425: Optimize noteram speed, copy all data at once

2022-10-25 Thread GitBox
Gary-Hobson opened a new pull request, #7425: URL: https://github.com/apache/incubator-nuttx/pull/7425 ## Summary sched_note_add copies one byte at a time Refer to circbuff to copy the entire payload at once ## Impact ## Testing -- This is an automated message f

[GitHub] [incubator-nuttx] tito97sp closed pull request #7397: arch: stm32h7: Flush D-Cache on flash erase operation

2022-10-25 Thread GitBox
tito97sp closed pull request #7397: arch: stm32h7: Flush D-Cache on flash erase operation URL: https://github.com/apache/incubator-nuttx/pull/7397 -- 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] [incubator-nuttx] tito97sp commented on pull request #7397: arch: stm32h7: Flush D-Cache on flash erase operation

2022-10-25 Thread GitBox
tito97sp commented on PR #7397: URL: https://github.com/apache/incubator-nuttx/pull/7397#issuecomment-1290175614 Moved to: https://github.com/apache/incubator-nuttx/pull/7401 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] tito97sp commented on pull request #7401: arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

2022-10-25 Thread GitBox
tito97sp commented on PR #7401: URL: https://github.com/apache/incubator-nuttx/pull/7401#issuecomment-1290140269 > @tito97sp should we merge this patch instead yours(#7397)? Yes. This PR substitutes mine. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx] midokura-xavi92 opened a new pull request, #7424: Strftime new specifiers

2022-10-25 Thread GitBox
midokura-xavi92 opened a new pull request, #7424: URL: https://github.com/apache/incubator-nuttx/pull/7424 ## Summary This PR adds two missing conversion specifiers supplied by [`strftime(3)`](https://pubs.opengroup.org/onlinepubs/9699919799/functions/strftime.html) that are defined by P