From 9e18f9577b4a838e98311afc59a34df4fc050075 Mon Sep 17 00:00:00 2001 From: Marti Bolivar Date: Thu, 7 Jun 2012 04:56:30 -0400 Subject: Move timer_dev definitions into libmaple/timer.c. The addition of STM32_HAVE_TIMER() allows us to avoid some repetition. There's still an issue with names on F1 preventing us from moving the IRQ handlers to libmaple/timer.c, but once that's resolved, we'll be able to remove even more. Signed-off-by: Marti Bolivar --- libmaple/stm32f1/timer.c | 125 ++++------------------------------------------- 1 file changed, 10 insertions(+), 115 deletions(-) (limited to 'libmaple/stm32f1') diff --git a/libmaple/stm32f1/timer.c b/libmaple/stm32f1/timer.c index 8671695..8b9e976 100644 --- a/libmaple/stm32f1/timer.c +++ b/libmaple/stm32f1/timer.c @@ -30,117 +30,10 @@ * @brief STM32F1 timer. */ -/* Notes: - * - * - We use STM32F1 density test macros throughout to avoid defining - * symbols or linking in code that would use timers that are - * unavailable in a given density. For example, TIM5 doesn't exist - * on medium-density, and TIM9 doesn't exist on high-density, so we - * don't define or use TIM5 when being compiled for medium-density, - * and similarly for TIM9 and high-density. - * - * This makes a mess, but helps avoid bloat and ensures backwards - * compatibility. Since the mess is manageable and there don't seem - * to be any plans on ST's part to add new STM32F1 lines or - * densities, we'll live with it. - */ - #include +#include #include "timer_private.h" -/* - * Devices - * - * Defer to the timer_private API. - */ - -static timer_dev timer1 = ADVANCED_TIMER(1); -static timer_dev timer2 = GENERAL_TIMER(2); -static timer_dev timer3 = GENERAL_TIMER(3); -static timer_dev timer4 = GENERAL_TIMER(4); - -/** Timer 1 device (advanced) */ -timer_dev *TIMER1 = &timer1; -/** Timer 2 device (general-purpose) */ -timer_dev *TIMER2 = &timer2; -/** Timer 3 device (general-purpose) */ -timer_dev *TIMER3 = &timer3; -/** Timer 4 device (general-purpose) */ -timer_dev *TIMER4 = &timer4; - -#if defined(STM32_HIGH_DENSITY) || defined(STM32_XL_DENSITY) -static timer_dev timer5 = GENERAL_TIMER(5); -static timer_dev timer6 = BASIC_TIMER(6); -static timer_dev timer7 = BASIC_TIMER(7); -static timer_dev timer8 = ADVANCED_TIMER(8); - -/** Timer 5 device (general-purpose) */ -timer_dev *TIMER5 = &timer5; -/** Timer 6 device (basic) */ -timer_dev *TIMER6 = &timer6; -/** Timer 7 device (basic) */ -timer_dev *TIMER7 = &timer7; -/** Timer 8 device (advanced) */ -timer_dev *TIMER8 = &timer8; -#endif /* defined(STM32_HIGH_DENSITY) || defined(STM32_XL_DENSITY) */ - -#ifdef STM32_XL_DENSITY -/* TIM9 has UIE, CC1IE, CC2IE, TIE bits in DIER. */ -static timer_dev timer9 = RESTRICTED_GENERAL_TIMER(9, TIMER_DIER_TIE_BIT); -/* TIM10 has UIE, CC1IE. */ -static timer_dev timer10 = RESTRICTED_GENERAL_TIMER(10, TIMER_DIER_CC1IE_BIT); -/* TIM11 has UIE, CC1IE. */ -static timer_dev timer11 = RESTRICTED_GENERAL_TIMER(11, TIMER_DIER_CC1IE_BIT); -/* TIM12 has UIE, CC1IE, CC2IE, TIE. */ -static timer_dev timer12 = RESTRICTED_GENERAL_TIMER(12, TIMER_DIER_TIE_BIT); -/* TIM13 has UIE, CC1IE. */ -static timer_dev timer13 = RESTRICTED_GENERAL_TIMER(13, TIMER_DIER_CC1IE_BIT); -/* TIM14 has UIE, CC1IE. */ -static timer_dev timer14 = RESTRICTED_GENERAL_TIMER(14, TIMER_DIER_CC1IE_BIT); - -/** Timer 9 device (general-purpose) */ -timer_dev *TIMER9 = &timer9; -/** Timer 10 device (general-purpose) */ -timer_dev *TIMER10 = &timer10; -/** Timer 11 device (general-purpose) */ -timer_dev *TIMER11 = &timer11; -/** Timer 12 device (general-purpose) */ -timer_dev *TIMER12 = &timer12; -/** Timer 13 device (general-purpose) */ -timer_dev *TIMER13 = &timer13; -/** Timer 14 device (general-purpose) */ -timer_dev *TIMER14 = &timer14; -#endif /* STM32_XL_DENSITY */ - -/* - * Routines - */ - -/** - * @brief Call a function on timer devices. - * @param fn Function to call on each timer device. - */ -void timer_foreach(void (*fn)(timer_dev*)) { - fn(TIMER1); - fn(TIMER2); - fn(TIMER3); - fn(TIMER4); -#if defined(STM32_HIGH_DENSITY) || defined(STM32_XL_DENSITY) - fn(TIMER5); - fn(TIMER6); - fn(TIMER7); - fn(TIMER8); -#endif -#ifdef STM32_XL_DENSITY - fn(TIMER9); - fn(TIMER10); - fn(TIMER11); - fn(TIMER12); - fn(TIMER13); - fn(TIMER14); -#endif -} - /* * IRQ handlers * @@ -149,26 +42,28 @@ void timer_foreach(void (*fn)(timer_dev*)) { * FIXME: The names of these handlers are inaccurate since XL-density * devices came out. Update these to match the STM32F2 names, maybe * using some weak symbol magic to preserve backwards compatibility if - * possible. + * possible. Once that's done, we can just move the IRQ handlers into + * the top-level libmaple/timer.c, and there will be no need for this + * file. */ void __irq_tim1_brk(void) { dispatch_adv_brk(TIMER1); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(9) dispatch_tim_9_12(TIMER9); #endif } void __irq_tim1_up(void) { dispatch_adv_up(TIMER1); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(10) dispatch_tim_10_11_13_14(TIMER10); #endif } void __irq_tim1_trg_com(void) { dispatch_adv_trg_com(TIMER1); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(11) dispatch_tim_10_11_13_14(TIMER11); #endif } @@ -204,21 +99,21 @@ void __irq_tim7(void) { void __irq_tim8_brk(void) { dispatch_adv_brk(TIMER8); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(12) dispatch_tim_9_12(TIMER12); #endif } void __irq_tim8_up(void) { dispatch_adv_up(TIMER8); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(13) dispatch_tim_10_11_13_14(TIMER13); #endif } void __irq_tim8_trg_com(void) { dispatch_adv_trg_com(TIMER8); -#ifdef STM32_XL_DENSITY +#if STM32_HAVE_TIMER(14) dispatch_tim_10_11_13_14(TIMER14); #endif } -- cgit v1.2.3