STM32F4 TIM2

[Gabriel J. (nyquist)]

Hallo,

stehe wieder vor einem Problem, bei dem ich nicht weiter weiß. Und zwar 
will ich mit dem Timer 2 nur raufzählen und beim Überlauf, soll eine LED 
toggeln. Irgendwie will das ganze nicht so recht bzw. das Programm geht 
nicht in die ISR.

Hier mal der Code

#include "ch.h"
#include "hal.h"
#include "test.h"

#define ARM_MATH_CM4
#include "arm_math.h"

#include "timconf.h"

static uint32_t tim_cnt = 0;

void TIM2_IRQHandler(void){
  if ((TIM2->SR & TIM_SR_UIF) != 0){
    palTogglePad(GPIOD, GPIOD_LED5);
    tim_cnt++;
    // Clear update interrupt flag
    TIM2->SR &= ~(TIM_SR_UIF);
  }
}

int main(void) {
  thread_t *shelltp = NULL;

  /*
   * System initializations.
   * - HAL initialization, this also initializes the configured device drivers
   *   and performs the board-specific initializations.
   * - Kernel initialization, the main() function becomes a thread and the
   *   RTOS is active.
   */
  halInit();
  chSysInit();

  tim_init();

  /*
   * Normal main() thread activity, in this demo it just performs
   * a shell respawn upon its termination.
   */
  while (true) {}
  }
}

In tim_conf.c ist folgender Code enthalten:

#include "ch.h"
#include "hal.h"

void tim_init(void)
{
  TIM2->CR1 &= ~(TIM_CR1_CEN);
  // TIM2 clock enable
  RCC->APB1ENR |= RCC_APB1ENR_TIM2EN;

  // Auto-reload value
  TIM2->ARR = 42000;
  // Clear update interrupt flag
  TIM2->SR &= ~(TIM_SR_UIF);
  // Update interrupt enable
  TIM2->DIER |= TIM_DIER_UIE;

  // Enable TIM2 for IRQ
  NVIC_EnableIRQ(TIM2_IRQn);

  // Enable timer counter
  TIM2->CR1 |= TIM_CR1_CEN;

  //END OF IRQ STUFF
}

In timconf.h

#ifndef TIM_H
#define TIM_H

void tim_init(void);

#endif /* TIM_H */

Sieht da einer von Euch den Fehler? Der Timer läuft über und fängt 
wieder von 0 an, nur wird die ISR nicht aufgerufen. Bin über jede Hilfe 
dankbar.

Lg Gabriel

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

Und wieder einmal: Wo wird die Clock für GPIOD angeschaltet?

[Ingo (Gast)]

PalTogglePad? Muss das evtl. HalTogglePad heissen? Keine den HAL nicht

[holger (Gast)]

halInit();
  chSysInit();

  tim_init();

Pack   tim_init(); doch mal vor chSysInit();.

[Gabriel J. (nyquist)]

Danke für die schnellen Rückmeldungen! Habe erst jetzt Zeit gefunden, 
die Vorschläge auszuprobieren.

Matthias S. schrieb:
> Und wieder einmal: Wo wird die Clock für GPIOD angeschaltet?

Hab den Code nun um folgende Zeile erweitert.

RCC->AHB1ENR |= RCC_AHB1ENR_GPIODEN;

Kannst du mir bitte sagen, warum es nötig ist, den Clock etxra 
einzuschalten, wenn ich lediglich die on-board LED toggeln lasse? Was 
mich auch noch etwas irritiert ist, dass immer alle GPIO's (von A bis I) 
enabled sind, auch wenn ich es im Code nicht explizit angebe.

Ingo;
> PalTogglePad? Muss das evtl. HalTogglePad heissen?

Nein palTogglePad funktioniert einwandfrei (zumindest in einem anderern 
Programmcode).

holger:
> Pack   tim_init(); doch mal vor chSysInit();

Hat ebenso zu keiner Verbesserung geführt. Was mir durch diese Änderung 
aufgefallen ist, ist, dass das Upadate Interrupt Enable (UIE) bit im 
DIER Register nicht gesetzt ist, obwohl ich es im Code enable.

@all

Beim Überlauf wird das Update Interrupt Flag (UIF) bit im SR Register 
als auch das CC2, CC3 und CC4 Interrupt Flag bit gesetzt. Kann mir da 
jemand sagen, warum die gesetzt werden und warum das CC1IF bit nicht 
gesetzt wird?

Danke schonmal im Voraus! Lg

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

Gabriel J. schrieb:
> Kannst du mir bitte sagen, warum es nötig ist, den Clock etxra
> einzuschalten, wenn ich lediglich die on-board LED toggeln lasse?

Weil sonst die I/O Logik des Ports nicht angeschubst wird. Es handelt 
sich (vereinfacht gesagt) um getaktete Flipflops, die ohne Taktsignal 
nix machen. Das steht aber auch in der Referenz.

Kannst du mir umgekehrt mal sagen, warum du mit HAL arbeitest, aber dann 
alle Register zu Fuss beschreibst? Das ist am Sinn und Zweck von HAL 
vorbei. Wenn du HAL benutzt, ist es nur konsequent, auch die Funktionen 
zu benutzen, um mit der Peripherie zu sprechen.

Weiterhin solltest du mal schauen, ob der NVIC sinnvoll initialisiert 
ist. Das betrifft vor allem die Prioritätsgruppen und -level.

[Little B. (lil-b)]

Ich sehe das ähnlich. Wenn du BareMetal arbeiten willst, solltest du 
auch konsequent keine HAL verwenden. (Oder umgekehrt, nur HAL verwenden)

Mit NVIC_EnableIRQ(TIM2_IRQn) schaltest du nur den Interrupt scharf, 
denke ich. Es fehlt noch die initialisierung der Priority Grouping und 
die Priorität deines Interrupts.

[Gabriel J. (nyquist)]

Danke für die Rückmeldung. Ok, das mit dem Clock für die GPIO's ist mir 
jetzt klar. Danke

Matthias S. schrieb:
> Kannst du mir umgekehrt mal sagen, warum du mit HAL arbeitest, aber dann
> alle Register zu Fuss beschreibst? Das ist am Sinn und Zweck von HAL
> vorbei. Wenn du HAL benutzt, ist es nur konsequent, auch die Funktionen
> zu benutzen, um mit der Peripherie zu sprechen.

Weil ich irgendwie nicht so recht weiß, wie ich mit GPTD2 den Timer 
genauso konfigurieren kann, wie mit den Registern. Wie man den clock 
einstellt und die callback function alle x GPT clock cycles aufruft, 
sehe ich schon raus.

static GPTConfig gpt2cfg =
{
    200000, //timer clock
    callback  //Timer callback
};
 // Configure the GPT timer
 gptStart(&GPTD2, &gpt2cfg);
 gptStartContinuous(&GPTD2, 100);

Aber wie stelle ich da zB. das Prescaler, Auto-Reload Register ein? Wie 
kann ich sagen, ob er rauf oder runterzählen soll, im Center-aligned 
oder Capture/Compare mode arbeiten soll. Da hab ich irgendwie nicht so 
recht den Durchblick. Gibts da weitere Strukturen, um das alles 
einzustellen? Wenn ich die Register zu Fuß beschreibe, ist es für mich 
"etwas" übersichtlicher. Da spreche ich auch nur jene Register an, die 
ich wirklich brauche. Gibts da nicht einen Vorteil in Sachen 
Geschwindigkeit? In weiterer Folge soll nämlich eine Regelung mit 20kHz 
betrieben werden.

> Weiterhin solltest du mal schauen, ob der NVIC sinnvoll initialisiert
> ist. Das betrifft vor allem die Prioritätsgruppen und -level.

Werd den NVIC mal entsprechend initialisieren. Vllt klappt es ja 
anschließend.

Lg

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

Gabriel J. schrieb:
> Aber wie stelle ich da zB. das Prescaler, Auto-Reload Register ein? Wie
> kann ich sagen, ob er rauf oder runterzählen soll, im Center-aligned
> oder Capture/Compare mode arbeiten soll.

Schau einfach mal hier:
Beitrag "STM32 Timer Callback wird nicht aufgerufen?"

Dort hat der TE das alles mit HAL gemacht. Es gab nur ein kleines 
Problem mit dem NVIC.
Und es ist eigentlich immer so, das man die Dokumentation vergessen kann 
und sich einfach die Libraries durchliest, die man benutzen will. Dort 
haben die eigentlichen Programmierer der Sachen meistens auch schon 
Anleitungen reingeschrieben, die die Doku toppen.

[Gabriel J. (nyquist)]

Matthias S. schrieb:

> Schau einfach mal hier:
> Beitrag "STM32 Timer Callback wird nicht aufgerufen?"
>
> Dort hat der TE das alles mit HAL gemacht. Es gab nur ein kleines
> Problem mit dem NVIC.
> Und es ist eigentlich immer so, das man die Dokumentation vergessen kann
> und sich einfach die Libraries durchliest, die man benutzen will. Dort
> haben die eigentlichen Programmierer der Sachen meistens auch schon
> Anleitungen reingeschrieben, die die Doku toppen.

Danke für die Information! Werd ich demnächst wirklich mit HAL machen. 
Hab meinen Fehler nun endlich gefunden. Es lag an den ST Driver System 
Settings. Anscheinend tretten mehrerer Interrupts gleichzeitig auf und 
irgendwie greift diese Struktur so auf den TIM2-Interrupt zu, dass bei 
meinem Code der Handler nicht aufgerufen wird.

Lösung:
In mcuconf.h:

#define STM32_ST_USE_TIMER        5// vorher 2

Da Timer 5 und 2 identisch sind.

In main.c:

void STM32_TIM2_HANDLER(void){
   if ((TIM2->SR & TIM_SR_UIF) != 0){
       palTogglePad(GPIOD, GPIOD_LED3);
       tim_cnt++;
     }
   // Clear update interrupt flag
   TIM2->SR &= ~(0xffff);
}

Danke für die sehr hilfreichen Antworten/Informationen. Hat für das 
Verständnis sehr geholfen!! Lg Gabriel