Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-26 Thread via GitHub
xiaoxiang781216 merged PR #15938: URL: https://github.com/apache/nuttx/pull/15938 -- 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

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-15 Thread via GitHub
lupyuen commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2710118218 @nuttxpr test qemu-armv8a:netnsh -- 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 specif

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-11 Thread via GitHub
jlaitine commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2703199570 Adding one example with real numbers how the timer drift causes the tick timer & watchdog timers to fail: Example: Counter frequency 2400 (24MHz) Tick frequency 100 H

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-10 Thread via GitHub
Fix-Point commented on code in PR #15938: URL: https://github.com/apache/nuttx/pull/15938#discussion_r1986948744 ## arch/arm64/src/common/arm64_arch_timer.c: ## @@ -265,34 +269,20 @@ static int arm64_tick_start(struct oneshot_lowerhalf_s *lower, { struct arm64_oneshot_lower

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-10 Thread via GitHub
nuttxpr commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2710131879 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+15779&type=issues) Build and Test Successful (qemu-armv8a:netnsh) https://gitla

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-07 Thread via GitHub
jlaitine commented on code in PR #15938: URL: https://github.com/apache/nuttx/pull/15938#discussion_r1984624911 ## arch/arm64/src/common/arm64_arch_timer.c: ## @@ -265,34 +269,20 @@ static int arm64_tick_start(struct oneshot_lowerhalf_s *lower, { struct arm64_oneshot_lowerh

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-07 Thread via GitHub
jlaitine commented on code in PR #15938: URL: https://github.com/apache/nuttx/pull/15938#discussion_r1984620387 ## arch/arm64/src/common/arm64_arch_timer.c: ## @@ -265,34 +269,20 @@ static int arm64_tick_start(struct oneshot_lowerhalf_s *lower, { struct arm64_oneshot_lowerh

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-06 Thread via GitHub
xiaoxiang781216 commented on code in PR #15938: URL: https://github.com/apache/nuttx/pull/15938#discussion_r1984406298 ## arch/arm64/src/common/arm64_arch_timer.c: ## @@ -265,34 +269,20 @@ static int arm64_tick_start(struct oneshot_lowerhalf_s *lower, { struct arm64_oneshot

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-06 Thread via GitHub
jlaitine commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2703630963 > Regarding the problem you described, the original implementation of aligning the tick scale also has this problem. Let's take an example. Assume that the frequency is `6250`, `

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-06 Thread via GitHub
Fix-Point commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2703593592 > Adding one example with real numbers how the timer drift causes the tick timer & watchdog timers to fail: > > Example: > > Counter frequency 2400 (24MHz) Tick frequen

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-05 Thread via GitHub
jlaitine commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2702997333 Could you please wait with this a bit, we'd like to test it on i.MX93 platform before merging. I will come back to this still today. -- This is an automated message from the Apache

Re: [PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-03 Thread via GitHub
nuttxpr commented on PR #15938: URL: https://github.com/apache/nuttx/pull/15938#issuecomment-2696110823 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No, this PR description does not fully meet the NuttX r

[PR] arch/arm64: Improvements on ARM generic timer drivers. [nuttx]

2025-03-03 Thread via GitHub
Fix-Point opened a new pull request, #15938: URL: https://github.com/apache/nuttx/pull/15938 ## Summary - Optimized the initialization process of percpu timer - Fixed the time drift problem caused by integer division precision loss when USEC_PER_TICKS is set to a low value #