[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7974: Improve the assert consistence

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7974: URL: https://github.com/apache/nuttx/pull/7974#issuecomment-1364661134 @pkarashchenko look like macOS CI still fail? -- 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

[GitHub] [nuttx] pkarashchenko commented on pull request #7974: Improve the assert consistence

2022-12-25 Thread GitBox
pkarashchenko commented on PR #7974: URL: https://github.com/apache/nuttx/pull/7974#issuecomment-1364675198 @xiaoxiang781216 I think you need to revise to fix macOS errors -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [nuttx] pkarashchenko commented on pull request #7974: Improve the assert consistence

2022-12-25 Thread GitBox
pkarashchenko commented on PR #7974: URL: https://github.com/apache/nuttx/pull/7974#issuecomment-1364678321 I see. Seems to be some new error with a similar root cause. I will take a look most probably tomorrow -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 opened a new pull request, #7975: URL: https://github.com/apache/nuttx/pull/7975 ## Summary ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364688983 @yamt here is the change which map time_4 to uint64_t instead int64_t -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [nuttx] xiaoxiang781216 closed pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 closed pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true URL: https://github.com/apache/nuttx/pull/7975 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364696147 libcxx report the following warning with this patch: ``` [2022-12-25T14:50:41.765Z] libcxx/src/condition_variable.cpp: In member function 'void std::__1::condition_variable::_

[GitHub] [nuttx] lupyuen opened a new pull request, #7977: arm64/pinephone: Add driver for LCD Panel (Xingbangda XBD599)

2022-12-25 Thread GitBox
lupyuen opened a new pull request, #7977: URL: https://github.com/apache/nuttx/pull/7977 ## Summary This PR adds the driver for Xingbangda XBD599 LCD Panel (based on Sitronix ST7703 LCD Controller) on PINE64 PinePhone. This PR also includes: - The driver for X-Powers AXP803 Pow

[GitHub] [nuttx] yamt commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
yamt commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364794044 > libcxx report the following warning with this patch: > > ``` > [2022-12-25T14:50:41.765Z] libcxx/src/condition_variable.cpp: In member function 'void std::__1::condition_variable::

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364826424 No, it's problem when we enable CONFIG_SYSTEM_TIME64. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [nuttx-apps] anchao merged pull request #1476: system/trace: Check NOTERAM_GETTASKNAME existence directly

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

[nuttx-apps] branch master updated: system/trace: Check NOTERAM_GETTASKNAME existence directly

2022-12-25 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 77ac0e483 system/trace: Check NOTERAM_GETTAS

[GitHub] [nuttx] yamt commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
yamt commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364856026 > No, it's problem when CONFIG_SYSTEM_TIME64 is enabled and time_t is map to uint64_t. So, if you don't have a good reason, it's better to map time_t to int64_t like other OS. For !CONFIG_SYST

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364874128 If so, why not change 32bit time_t to int32_t like other OS? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [nuttx] yamt commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
yamt commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364882663 > If so, why not change 32bit time_t to int32_t like other OS? because int32_t wraps too soon. -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [nuttx] robertlipe opened a new pull request, #7978: Create branch for ch32v

2022-12-25 Thread GitBox
robertlipe opened a new pull request, #7978: URL: https://github.com/apache/nuttx/pull/7978 ./tools/configure.sh ch32v307evt:nsh make Succeeds. Doubt it runs since my board won't connect. linker script and startup.S need nuttx-ified. -- This is an automated message fro

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364906298 > > No, it's problem when CONFIG_SYSTEM_TIME64 is enabled and time_t is map to uint64_t. So, if you don't have a good reason, it's better to map time_t to int64_t like other OS. For

[GitHub] [nuttx] robertlipe commented on pull request #7978: Create branch for ch32v

2022-12-25 Thread GitBox
robertlipe commented on PR #7978: URL: https://github.com/apache/nuttx/pull/7978#issuecomment-1364906602 Stupid Github. This is NOT where i asked you to branch... Sorry. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [nuttx] robertlipe closed pull request #7978: Create branch for ch32v

2022-12-25 Thread GitBox
robertlipe closed pull request #7978: Create branch for ch32v URL: https://github.com/apache/nuttx/pull/7978 -- 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] [nuttx] xiaoxiang781216 commented on a diff in pull request #7978: Create branch for ch32v

2022-12-25 Thread GitBox
xiaoxiang781216 commented on code in PR #7978: URL: https://github.com/apache/nuttx/pull/7978#discussion_r1057074937 ## arch/risc-v/include/ch32v/irq.h: ## @@ -0,0 +1,209 @@ +/ + * arch/risc-v/include/bl

[GitHub] [nuttx] yamt opened a new pull request, #7979: Suppress -Wsign-compare warning on a few places

2022-12-25 Thread GitBox
yamt opened a new pull request, #7979: URL: https://github.com/apache/nuttx/pull/7979 ## Summary Suppress -Wsign-compare warning on a few places ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [nuttx] yamt commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
yamt commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364926662 > > > No, it's problem when CONFIG_SYSTEM_TIME64 is enabled and time_t is map to uint64_t. So, if you don't have a good reason, it's better to map time_t to int64_t like other OS. For !CONFIG_

[GitHub] [nuttx] wengzhe opened a new pull request, #7980: net: Add netfilter compatible headers

2022-12-25 Thread GitBox
wengzhe opened a new pull request, #7980: URL: https://github.com/apache/nuttx/pull/7980 ## Summary We need an interface for user to configure our NAT (and maybe future firewall) function, the `set/getsockopt` interface and command line pattern of iptables might be the mostly widely

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #7975: include/time.h: Typedef time_t to uint64_t when CONFIG_SYSTEM_TIME64 is true

2022-12-25 Thread GitBox
xiaoxiang781216 commented on PR #7975: URL: https://github.com/apache/nuttx/pull/7975#issuecomment-1364935605 > > see #7979 i guess it's far from exhaustive. > I think 64bit signed/unsigned comparison is handled differently from 32bit signed/unsigned comparison by compiler. Si

[GitHub] [nuttx] xiaoxiang781216 merged pull request #7900: drivers/note: move taskname related functions to note_taskname.c

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

[nuttx] branch master updated (7322eafba7 -> dd32eccfc3)

2022-12-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/nuttx.git from 7322eafba7 drivers/usb: Fix the typo error in Kconfig add dd32eccfc3 drivers/note: Move taskname related functi

[GitHub] [nuttx] Gary-Hobson opened a new pull request, #7981: register notelog device

2022-12-25 Thread GitBox
Gary-Hobson opened a new pull request, #7981: URL: https://github.com/apache/nuttx/pull/7981 ## Summary Register notelog to the list of note drivers ## Impact still only one note driver can be used multi-channel functionality needs to wait for sysview changes to complete