Author: ian
Date: Wed Apr  2 18:32:27 2014
New Revision: 264049
URL: http://svnweb.freebsd.org/changeset/base/264049

Log:
  Disable the timer and clear any pending bit, then setup the new counter
  register values, then restart the timer.  This prevents a situation where
  an old event fires just as we're about to load a new value into the timer,
  when the start routine is called to change the time of the current event.
  
  Also re-nest the parens properly for casting the result of converting
  time and frequency to a count.  This doesn't actually change the result of
  the calcs, but will some day prevent a loss-of-precision warning on the
  assignment, if that warning gets enabled.

Modified:
  head/sys/arm/arm/mpcore_timer.c

Modified: head/sys/arm/arm/mpcore_timer.c
==============================================================================
--- head/sys/arm/arm/mpcore_timer.c     Wed Apr  2 17:34:17 2014        
(r264048)
+++ head/sys/arm/arm/mpcore_timer.c     Wed Apr  2 18:32:27 2014        
(r264049)
@@ -173,6 +173,9 @@ arm_tmr_start(struct eventtimer *et, sbi
        uint32_t load, count;
        uint32_t ctrl;
 
+       tmr_prv_write_4(PRV_TIMER_CTRL, 0);
+       tmr_prv_write_4(PRV_TIMER_INTR, PRV_TIMER_INTR_EVENT);
+
        ctrl = PRV_TIMER_CTRL_IRQ_ENABLE | PRV_TIMER_CTRL_TIMER_ENABLE;
 
        if (period != 0) {
@@ -182,14 +185,14 @@ arm_tmr_start(struct eventtimer *et, sbi
                load = 0;
 
        if (first != 0)
-               count = ((uint32_t)et->et_frequency * first) >> 32;
+               count = (uint32_t)((et->et_frequency * first) >> 32);
        else
                count = load;
 
        tmr_prv_write_4(PRV_TIMER_LOAD, load);
        tmr_prv_write_4(PRV_TIMER_COUNT, count);
-
        tmr_prv_write_4(PRV_TIMER_CTRL, ctrl);
+
        return (0);
 }
 
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to