Re: agtimer(4/arm64): simplify agtimer_delay()

2023-08-10 Thread Mark Kettenis
> Date: Thu, 10 Aug 2023 16:01:10 -0500 > From: Scott Cheloha > > On Tue, Aug 08, 2023 at 08:00:47PM +0200, Mark Kettenis wrote: > > > From: Dale Rahn > > > Date: Tue, 8 Aug 2023 12:36:45 -0400 > > > > > > Switching the computation of cycles/delaycnt to a proper 64 value math > > > instead of

Re: agtimer(4/arm64): simplify agtimer_delay()

2023-08-10 Thread Scott Cheloha
On Tue, Aug 08, 2023 at 08:00:47PM +0200, Mark Kettenis wrote: > > From: Dale Rahn > > Date: Tue, 8 Aug 2023 12:36:45 -0400 > > > > Switching the computation of cycles/delaycnt to a proper 64 value math > > instead of the '32 bit safe' complex math is likely a good idea. > > However I am not

Re: agtimer(4/arm64): simplify agtimer_delay()

2023-08-08 Thread Mark Kettenis
> From: Dale Rahn > Date: Tue, 8 Aug 2023 12:36:45 -0400 > > Switching the computation of cycles/delaycnt to a proper 64 value math > instead of the '32 bit safe' complex math is likely a good idea. > However I am not completely convinced that switching to 'yield' (current > CPU_BUSY_CYCLE

Re: agtimer(4/arm64): simplify agtimer_delay()

2023-08-08 Thread Dale Rahn
Switching the computation of cycles/delaycnt to a proper 64 value math instead of the '32 bit safe' complex math is likely a good idea. However I am not completely convinced that switching to 'yield' (current CPU_BUSY_CYCLE implementation) for every loop of a 'short wait' in the wait loop makes

agtimer(4/arm64): simplify agtimer_delay()

2023-08-07 Thread Scott Cheloha
The agtimer(4/arm64) delay(9) implementation is quite complicated. This patch simplifies it. Am I missing something here? There's no reason to implement the busy-loop like this, right? ok? Index: agtimer.c === RCS file: