Raspi Pico (RP2040): repeating timer callback

[B.W.H. (Gast)]

Hallo,

auf Basis des Beispiels in Kapitel 4.1.24 (hardware_timer) der 
"Raspberr-pi-pico-c-sdk"-Doku erzeuge ich mir Ticks für den Aufbau eines 
Zeitrasters.
Dazu erzeuge ich mir innerhalb "main" mittels "add_repeating_timer..." 
eine callback-Funktion ("repeating_timer_callback", oberhalb "main"), 
die während der Aufrufintervalle die "systemTimeInTicks" hochzählt (und 
zwecks Kontrolle des Durchlaufs einmalig die onboard-LED25 einschaltet).

Listing in Auszügen für den Fall ok:

#include .....
  .
  .
  .
  .

//void _initSystemTime(void);
  .
  .
  .

const uint8_t led_pin = 25;
volatile uint16_t systemTimeInTicks=0;
/*--------------------------------------------------*/
bool repeating_timer_callback(struct repeating_timer *t) 
{
    systemTimeInTicks++;
    gpio_put(led_pin, true);

    return true;
}
//********************************************************************
int main(void){   
    
    stdio_init_all();
    
    _initPortDirections();
    _initPorts();
    //_initSystemTime();   // <-------------------------------!!
    bool rm_art;               // Rueckmeldung add_repeating_timer

    systemTimeInTicks=0;
    struct repeating_timer timer;
    rm_art = add_repeating_timer_ms(-2, repeating_timer_callback, NULL, &timer);
  .
  .
  .
  .
}
  .
  .
  .

Nun möchte ich allerdings die Anweisungen für den callback (und im 
weiteren Verlauf auch weitere timing-bezogenen Routinen) in eine 
Funktion auslagern und diese dann in "main" aufrufen.
Leider wird jetzt die callback-Funktion nicht mehr durchlaufen, was sich 
im Ausbleiben der LED anzeigt und nach ca. 20 Sekunden auch durch das 
Aussteigen der USB-Verbindung (vermutlich weil der Pico intern 
"überläuft").

Listing in Auszügen für den Fall nicht ok:

#include .....
  .
  .
  .
  .

void _initSystemTime(void);
  .
  .
  .

const uint8_t led_pin = 25;
volatile uint16_t systemTimeInTicks=0;
/*--------------------------------------------------*/
bool repeating_timer_callback(struct repeating_timer *t) 
{
    systemTimeInTicks++;
    gpio_put(led_pin, true);

    return true;
}
//************************************************************************
int main(void){   
    
    stdio_init_all();
    
    _initPortDirections();
    _initPorts();
    _initSystemTime();   // <-------------------------------!!
    
  .
  .
  .
}
  .
  .
  .
    
void _initSystemTime(void)
{
    bool rm_art;               // Rueckmeldung add_repeating_timer

    systemTimeInTicks=0;
    struct repeating_timer timer;
    rm_art = add_repeating_timer_ms(-2, repeating_timer_callback, NULL, &timer); 
}

Wer kennt eine Lösung bzw. was mache ich falsch?

Danke für Tipps!

[temp (Gast)]

Dein "struct repeating_timer timer;" solltest entweder global machen 
oder static davor schreiben. So wie du das machst, wird die Struktur auf 
dem Stack angelegt und ist Geschichte wenn _initSystemTime() zu Ende 
ist. Die muss aber solange erhalten bleiben wie dein Timer laufen soll.

[B.W.H. (Gast)]

Super, genau das war es! Und Deine Begründung habe ich auch verstanden!

Besten Dank!!