Commit 7691d931 authored by Adam Brooks's avatar Adam Brooks Committed by Russell King

[ARM] 2871/1: Fixes an issue with gettimeofday not working correctly on Intel IOP3xx processors

Patch from Adam Brooks

The current gettimeofday implementation for the IOP3xx processors reads the contents of the timer interrupt register and does math on the value to figure out exactly what time it is.  To do this it  multiplies the contents of the timer register with a large constant.  The result is then divided by a large constant.  Unfortunately the result of the first multiplication is often too large for the register to hold.  The solution is to combine the two large constants to a single smaller constant at compile time.  Then the timer value can be divided by single smaller constant without any overflow issues.
Signed-off-by: default avatarAdam Brooks <adam.j.brooks@intel.com>
Signed-off-by: default avatarDeepak Saxena <dsaxena@plexity.net>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent fd6480f2
...@@ -60,7 +60,7 @@ static unsigned long iop321_gettimeoffset(void) ...@@ -60,7 +60,7 @@ static unsigned long iop321_gettimeoffset(void)
/* /*
* Now convert them to usec. * Now convert them to usec.
*/ */
usec = (unsigned long)(elapsed * (tick_nsec / 1000)) / LATCH; usec = (unsigned long)(elapsed / (CLOCK_TICK_RATE/1000000));
return usec; return usec;
} }
......
...@@ -58,7 +58,7 @@ static unsigned long iop331_gettimeoffset(void) ...@@ -58,7 +58,7 @@ static unsigned long iop331_gettimeoffset(void)
/* /*
* Now convert them to usec. * Now convert them to usec.
*/ */
usec = (unsigned long)(elapsed * (tick_nsec / 1000)) / LATCH; usec = (unsigned long)(elapsed / (CLOCK_TICK_RATE/1000000));
return usec; return usec;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment