X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=drv%2Ftimer_avr.h;h=a530136414a231881a55b5d8508c55a804047626;hb=0ee1d8ff312b2b8fa07932cacc10d95ba3b23498;hp=85219236061bbddf554258b9f44d5b8e6f40c191;hpb=7afdef86655a7750a09d479d0d90f883417c0a12;p=bertos.git

diff --git a/drv/timer_avr.h b/drv/timer_avr.h
index 85219236..a5301364 100755
--- a/drv/timer_avr.h
+++ b/drv/timer_avr.h
@@ -13,30 +13,47 @@
  * \brief Low-level timer module for AVR
  */
 
-/*
- * $Log$
- * Revision 1.14  2004/08/24 16:27:01  bernie
- * Add missing headers.
- *
- * Revision 1.13  2004/08/24 14:30:11  bernie
- * Use new-style config macros for drv/timer.c
- *
- * Revision 1.12  2004/08/10 06:59:45  bernie
- * CONFIG_TIMER_STROBE: Define no-op default macros.
- *
- * Revision 1.11  2004/08/03 15:53:17  aleph
- * Fix spacing
- *
- * Revision 1.10  2004/08/02 20:20:29  aleph
- * Merge from project_ks
- *
- * Revision 1.9  2004/07/22 02:01:14  bernie
- * Use TIMER_PRESCALER consistently.
- */
+/*#*
+ *#* $Log$
+ *#* Revision 1.19  2004/10/19 08:56:41  bernie
+ *#* TIMER_STROBE_ON, TIMER_STROBE_OFF, TIMER_STROBE_INIT: Move from timer_avr.h to timer.h, where they really belong.
+ *#*
+ *#* Revision 1.18  2004/09/20 03:31:03  bernie
+ *#* Fix racy racy code.
+ *#*
+ *#* Revision 1.17  2004/09/14 21:07:09  bernie
+ *#* Include hw.h explicitly.
+ *#*
+ *#* Revision 1.16  2004/09/06 21:49:26  bernie
+ *#* CONFIG_TIMER_STROBE: be tolerant with missing optional macro.
+ *#*
+ *#* Revision 1.15  2004/08/25 14:12:08  rasky
+ *#* Aggiornato il comment block dei log RCS
+ *#*
+ *#* Revision 1.14  2004/08/24 16:27:01  bernie
+ *#* Add missing headers.
+ *#*
+ *#* Revision 1.13  2004/08/24 14:30:11  bernie
+ *#* Use new-style config macros for drv/timer.c
+ *#*
+ *#* Revision 1.12  2004/08/10 06:59:45  bernie
+ *#* CONFIG_TIMER_STROBE: Define no-op default macros.
+ *#*
+ *#* Revision 1.11  2004/08/03 15:53:17  aleph
+ *#* Fix spacing
+ *#*
+ *#* Revision 1.10  2004/08/02 20:20:29  aleph
+ *#* Merge from project_ks
+ *#*
+ *#* Revision 1.9  2004/07/22 02:01:14  bernie
+ *#* Use TIMER_PRESCALER consistently.
+ *#*/
 #ifndef DRV_TIMER_AVR_H
 #define DRV_TIMER_AVR_H
 
 #include <arch_config.h> // ARCH_BOARD_KC
+#include "hw.h"
+
 #include <avr/wdt.h>
 #include <avr/signal.h>
 
@@ -56,23 +73,6 @@
 #define TIMER_ON_OUTPUT_COMPARE2  3
 
 
-/*!
- * \def CONFIG_TIMER_STROBE
- *
- * This is a debug facility that can be used to
- * monitor timer interrupt activity on an external pin.
- *
- * To use strobes, redefine the macros TIMER_STROBE_ON,
- * TIMER_STROBE_OFF and TIMER_STROBE_INIT and set
- * CONFIG_TIMER_STROBE to 1.
- */
-#ifndef CONFIG_TIMER_STROBE
-	#define TIMER_STROBE_ON    do {/*nop*/} while(0)
-	#define TIMER_STROBE_OFF   do {/*nop*/} while(0)
-	#define TIMER_STROBE_INIT  do {/*nop*/} while(0)
-#endif
-
-
 /* Not needed, IRQ timer flag cleared automatically */
 #define timer_hw_irq() do {} while (0)
 
@@ -222,22 +222,6 @@
 	 */
 	SIGNAL(SIG_OVERFLOW1)
 	{
-		/*!
-		 * How many overflow we have to count before calling the true timer handler.
-		 * If timer overflow is at 24 kHz, with a value of 24 we have 1 ms between
-		 * each call.
-		 */
-		#define TIMER1_OVF_COUNT 24
-
-		static uint8_t count = TIMER1_OVF_COUNT;
-
-		count--;
-		if (!count)
-		{
-			timer_handler();
-			count = TIMER1_OVF_COUNT;
-		}
-
 	#if (ARCH & ARCH_BOARD_KC)
 		/*
 		 * Super-optimization-hack: switch CPU ADC mux here, ASAP after the start
@@ -248,18 +232,49 @@
 		 * of the handler.
 		 *
 		 * The switch is synchronized with the ADC handler using _adc_trigger_lock.
+		 *
+		 *      Mel (A Real Programmer)
 		 */
 		extern uint8_t _adc_idx_next;
 		extern bool _adc_trigger_lock;
 
 		if (!_adc_trigger_lock)
 		{
-			TIMER_STROBE_ON;
+			/*
+			 * Disable free-running mode to avoid starting a
+			 * new conversion before the ADC handler has read
+			 * the ongoing one.  This condition could occur
+			 * under very high interrupt load and would have the
+			 * unwanted effect of reading from the wrong ADC
+			 * channel.
+			 *
+			 * NOTE: writing 0 to ADSC and ADIF has no effect.
+			 */
+			ADCSRA = ADCSRA & ~(BV(ADFR) | BV(ADIF) | BV(ADSC));
+
 			ADC_SETCHN(_adc_idx_next);
-			TIMER_STROBE_OFF;
 			_adc_trigger_lock = true;
 		}
-	#endif
+	#endif // ARCH_BOARD_KC
+
+		/*!
+		 * How many timer overflows we must count before calling the real
+		 * timer handler.
+		 * When the timer is programmed to overflow at 24 kHz, a value of
+		 * 24 will result in 1ms between each call.
+		 */
+		#define TIMER1_OVF_COUNT 24
+		//#warning TIMER1_OVF_COUNT for timer at 12 kHz
+		//#define TIMER1_OVF_COUNT 12
+
+		static uint8_t count = TIMER1_OVF_COUNT;
+
+		count--;
+		if (!count)
+		{
+			timer_handler();
+			count = TIMER1_OVF_COUNT;
+		}
 	}
 
 #elif (CONFIG_TIMER == TIMER_ON_OUTPUT_COMPARE0)