]> git.baikalelectronics.ru Git - arm-tf.git/commit
delay_timer: Guarantee that delay time can never be undershot
authorJulius Werner <jwerner@chromium.org>
Mon, 22 Jan 2018 22:02:03 +0000 (14:02 -0800)
committerJulius Werner <jwerner@chromium.org>
Thu, 25 Jan 2018 19:03:07 +0000 (11:03 -0800)
commite2aec918d07237e4a6aa185ad2d267750e8f2883
treef27fb7c0f535052e771de1c683f349b9869c685d
parent7baa7bcaf58b7fe89aab6e5e7c0f46cc4adc2b93
delay_timer: Guarantee that delay time can never be undershot

Delay functions like udelay() are often used to ensure that the
necessary time passed to allow some asynchronous event to finish, such
as the stabilization delay for a power rail. For these use cases it is
not very problematic if the delay is slightly longer than requested,
but it is critical that the delay must never be shorter.

The current udelay() implementation contains two hazards that may cause
the delay to be slightly shorter than intended: Firstly, the amount of
ticks to wait is calculated with an integer division, which may cut off
the last fraction of ticks needed. Secondly, the delay may be short by a
fraction of a tick because we do not know whether the initial ("start")
sample of the timer was near the start or near the end of the current
tick. Thus, if the code intends to wait for one tick, it might read the
timer value close to the end of the current tick and then read it again
right after the start of the next tick, concluding that the duration of
a full tick has passed when it in fact was just a fraction of it.

This patch rounds up the division and always adds one extra tick to
counteract both problems and ensure that delays will always be larger
but never smaller than requested.

Change-Id: Ic5fe5f858b5cdf3c0dbf3e488d4d5702d9569433
Signed-off-by: Julius Werner <jwerner@chromium.org>
drivers/delay_timer/delay_timer.c