[STM32F407VG, Discovery-board] Timer-interrupt verwirrt mich

[Luigi (Gast)]

Servus zusammen,

ich steige gerade wieder auf den STM32 ein (hatte länger nix damit 
gemacht und fange bei den basics an) aber leider bin ich jetzt auf ein 
Problem gestoßen, dessen Hintergrund ich gerne Verstanden hätte. Sollte 
auch super leich zu reproduzieren sein, sofern ein STM32F4-discovery 
board und CooCox CoIDE vorliegen. (Damit hätte ich dann wohl auch meine 
Umgebung definiert)
Die veränderten Dateien habe ich angehängt, der Rest ist von den 
Standard-Libs, die man in CoIDE einfach aktiviert und die dann 
automatisch ins Projekt kopiert werden.

Folgende Aufgabe :
Ich will eine LED innerhalb einer Interruptroutine ein- und ausschalten. 
Etwa im 1s Zyklus.

Das Problem :
So gehts :

void TIM2_IRQHandler(void)
{
  static uint8_t i = 0;

  TIM_ClearITPendingBit(TIM2, TIM_IT_Update);

  GPIO_ToggleBits(GPIOD, GPIO_Pin_13); // LD3, orange

  if( 2 == ++i )
  {
    GPIO_ToggleBits(GPIOD, GPIO_Pin_14); // LD5, red
    i=0;
  }
}

So gehts nicht :
  (nur die rote LED blinkt, die orange nicht)

void TIM2_IRQHandler(void)
{
  static uint8_t i = 0;

  GPIO_ToggleBits(GPIOD, GPIO_Pin_13); // LD3, orange

  if( 2 == ++i )
  {
    GPIO_ToggleBits(GPIOD, GPIO_Pin_14); // LD5, red
    i=0;
  }

  TIM_ClearITPendingBit(TIM2, TIM_IT_Update);
}

Woran kann das liegen?
Ich stehe vollkommen auf dem Schlauch. Für mich ist das gerade völlig 
unerklärlich!

Vielen Dank für eure Hilfe!
Grüße
Luigi

[Leo B. (luigi)]

PS:

Hervorzuheben ist vielleicht, dass nur die Zeile

TIM_ClearITPendingBit(TIM2, TIM_IT_Update);

in der Position verändert wird. Das ist der einzuge Unterschlied 
zwischen "funktionier" und "funktioniert nicht".



PPS:
war nicht eingelogt, bin aber ich

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Da gabs mal was hier bei mc.net, man muss erst das Interrupt Flag 
löschen, bevor man an GPIOs rumfummelt. Ist mir auch mal beim STM32F100 
untergekommen.
Frag mich aber bitte nicht mehr nach dem Thread - ich habe nur diese 
Bemerkung in meinen Sourcen gefunden:

void TIM1_UP_TIM16_IRQHandler(void){
//uint8_t speedRegTicks = 0;
// microcontroller.net : Clear the IT pending bit before doing any GPIO stuff
TIM_ClearITPendingBit(TIM1,(TIM_IT_Update | TIM_IT_CC1 | TIM_IT_CC2 | TIM_IT_CC3));
  LED_GPIO_PORT->BSRR = ERROR_LED_PIN;  // debug
//....

[Jim M. (turboj)]

Luigi schrieb:
> Woran kann das liegen?
> Ich stehe vollkommen auf dem Schlauch. Für mich ist das gerade völlig
> unerklärlich!

TIM_ClearITPendingBit() kommt im 2. Falls zu spät. Der µC braucht ein 
paar Takte bis das Timer Peripherial die Interrupt Anforderung 
wirklich zurück nimmt - man bedenke z.B. den Schreib Puffer.

Dummerweise macht der µC dadurch beim Interrupt Return ein Tail-Chain 
wieder in denselben Handler, der dadruch 2x hintereinander aufgerufen 
wird. Du solltest bei der roten LED die doppelte Frequenz erkennen 
können.

Abhilfe: Entweder rechtzeitig Flag zurück setzen (1. Variante) oder 
nochmal ein Dummy Lesezugriff auf die Hardware ausführen (z.B. irgenden 
Register lesen).

[Leo B. (luigi)]

Jim M. schrieb:
> TIM_ClearITPendingBit() kommt im 2. Falls zu spät. Der µC braucht ein
> paar Takte bis das Timer Peripherial die Interrupt Anforderung
> wirklich zurück nimmt - man bedenke z.B. den Schreib Puffer.
>
> Dummerweise macht der µC dadurch beim Interrupt Return ein Tail-Chain
> wieder in denselben Handler, der dadruch 2x hintereinander aufgerufen
> wird. Du solltest bei der roten LED die doppelte Frequenz erkennen
> können.

Eindeutig, du hast recht. Vielen Dank. Das Erklärt zu 100% das Phänomen.
Kurz Warten (2*NOP) nach dem rücksetzten ist ebenfalls eine Lösung, zu 
kurz warten (1*NOP) löst das Problem nicht (ich sage mal: q.e.d.)



Der vollständigkeit halber, so gehts auch :

void TIM2_IRQHandler(void)
{
  static uint8_t i = 0;

  GPIO_ToggleBits(GPIOD, GPIO_Pin_13); // LD3, orange

  if( 2 == ++i )
  {
    GPIO_ToggleBits(GPIOD, GPIO_Pin_14); // LD5, red
    i=0;
  }

  TIM_ClearITPendingBit(TIM2, TIM_IT_Update);

  __asm volatile ("NOP");
  __asm volatile ("NOP");
}

Vielen Dank!