STM8S - Timer4 Overflow Interrupt führt zu unhandled exception

[Max M. (maxmicr)]

Ich versuche gerade, den Overflow Interrupt des Timer4 vom STM8S zum 
laufen zu kriegen. Nach dem der Timer voll ist, läuft das Programm in 
eine "_iar_unhandled_exception" (siehe Anhang). Meine Timer 
Initialisierung:

void initTimer(){
  CLK_PCKENR1 |= (1<<4); //Enable Clock for Timer 4
  TIM4_PSCR = (0x07);    //Prescaler, Divide Clock by 7
  TIM4_IER = 1;          //Enable Update Interrupt
  TIM4_CR1 |= 1;         //Enable Timer
  asm("rim");           //Enable Interrupts        
}

Meine Interrupt Funktion:

#pragma vector = 23
__interrupt void TIM4_UPD_OVF_IRQHandler(void){
  displayInit();
  show();
  TIM4_SR &=~(1<<0);    //Clear Update interrupt
}

Woran liegt das?

[STM8 (Gast)]

Max M. schrieb:
> #pragma vector = 23

Meine Meinung: Das ist falsch, der Interrupt hat die Nummer 11.

[STM8 (Gast)]

Oh sorry, Timer4 hab ich übersehen.

[Stefan F. (Gast)]

Die Initialisierung des Displays in der Interrupt Routine lässt mich 
ahnen, dass das Programm strukturelle Mängel hat. Falls du keinen 
wirklich wichtigen Grund dafür hast, schlage ich Dir vor, den gesamten 
Quelltext zu zeigen, dann können wir Dir helfen, ihn zu verbessern.

[Max M. (maxmicr)]

Wenn ich die Zeile:

asm("rim");

auskommentiere lande ich nach dem Overflow nicht in der Exception, 
allerdings auch nicht in der Interrupt Funktion. Der Timerwert erhöht 
sich nun nur nach jedem 21. Step im Debug-Modus (aber auch erst, nach 
dem er einmal mit +6 / Step durchgelaufen ist). Das UIF-Bit wird nach 
dem Overflow gesetzt.

[STM8 (Gast)]

Max M. schrieb:
> Wenn ich die Zeile:asm("rim");
>
> auskommentiere

Dann gibt es auch keine Interrupts ...

[STM8 (Gast)]

Stefan U. schrieb:
> schlage ich Dir vor, den gesamten
> Quelltext zu zeigen, dann können wir Dir helfen, ihn zu verbessern

[Max M. (maxmicr)]

Stefan U. schrieb:
> Falls du keinen
> wirklich wichtigen Grund dafür hast, schlage ich Dir vor, den gesamten
> Quelltext zu zeigen, dann können wir Dir helfen, ihn zu verbessern.

Ich hab die Funktionen von oben nochmal umbenannt, nicht wundern:

/*
* main.c - Datei
*/
#include "MAX7219SPI.c"
#include "stdbool.h"
#define H 0x0C
#define E 0x0B
#define L 0x0D
#define S 0x0A

bool didInit = false;

uint8_t buffer[] = {
  H,E,L,L,0,S,S,S,S,1,3,3,7,S,S,S,S
};

void applyBuffer(){
  uint8_t counter;
  for(counter = 0; counter < 8; counter++){
    SPISendData(counter+1,buffer[8-counter]);
  }
}

void shuffleRight(){
  uint8_t counter, temp;
  for(counter = 0; counter < sizeof(buffer); counter++){
    if(counter != 0){
      uint8_t temp2 = buffer[counter];
      buffer[counter] = temp;
      temp = temp2;
    } else {
      temp = buffer[counter];
      buffer[counter] = buffer[sizeof(buffer)-1];
    }
  }
}

#pragma vector = 23
__interrupt void TIM4_UPD_OVF_IRQHandler(void){
  if(!didInit){
    initMAX7219();
    didInit = true;
  }
  applyBuffer();
  shuffleRight();
  TIM4_SR &=~(1<<0);    //Clear Interrupt
}
void initGPIOs(){
  PD_DDR = (1<<MOSI) | (1<<SCK) | (1<<CS);
  PD_CR1 = (1<<MOSI) | (1<<SCK) | (1<<CS);
  PD_ODR |= (1<<MOSI) | (1<<CS);
  PD_ODR &=~(1<<SCK);   //set SCK to low otherwise first clock isn't generated
}

void initTimer(){
  CLK_PCKENR1 |= (1<<4); //Enable Clock for Timer 4
  TIM4_PSCR = (0x07);    //Prescaler, Divide Clock by 7
  TIM4_IER = 1;          //Enable Update Interrupt
  TIM4_CR1 |= 1;         //Enable Timer
  asm("rim");           //Enable Interrupts        
}

int main(){
  initGPIOs();
  initTimer();
  while(1){
  }
}
/*
* MAX7219SPI.c Datei
*/
#include "iostm8s103f3.h"
#include "stdint.h"

#define MOSI    6
#define SCK     5
#define CS      4

void delay(uint16_t value){
  while(value != 0)
    value--;
}

void SPISendByte(uint8_t data){
  uint8_t mask;
  for(mask = 0x80; mask; mask >>=1){
    if(data & mask)
      PD_ODR |= (1<<MOSI);
    else
      PD_ODR &=~(1<<MOSI);
    delay(4);
    PD_ODR |= (1<<SCK);
    delay(12);
    PD_ODR &=~(1<<SCK);
    delay(2);
    PD_ODR &=~(1<<MOSI);
    delay(2);    
  }
}

void SPISendData(uint8_t regAddr, uint8_t data){
  delay(4);
  PD_ODR &=~(1<<CS);
  SPISendByte(regAddr);
  SPISendByte(data);
  PD_ODR |= (1<<CS);
  delay(4);
}

void initMAX7219(){
  SPISendData(0x0B,0x0F);               //Scan Limit
  delay(2);
  SPISendData(0x0A,0x0F);                //Max Intensity
  delay(2);
  SPISendData(0x0C,0x01);               //Normal Operation Mode
  delay(2);
  SPISendData(0x09,0xFF);               //Decode
  delay(2);
  SPISendData(0x0F,0x00);               //Normal Operation Mode
}

[STM8 (Gast)]

Das fehlerträchtige Konstrukt würde ich nochmals überdenken:

#include "MAX7219SPI.c"

Ich habe mal Deine Version übernommen und im IAR kompiliert. Laut 
map-file landet die Interrupt-Routine an der richtigen Stelle. Im 
Simulator passiert nichts. Damit kenne ich mich allerdings auch nicht 
gut aus. Was macht Dein Programm auf der realen Hardware?

[Max M. (maxmicr)]

Okay, ich habs gelöst. Etwas seltsam...
Die Interrupt-Funktion wurde als solche erkannt allerdings hat der 
Vector nicht gestimmt 
(http://www.st.com/content/ccc/resource/technical/document/datasheet/ce/13/13/03/a9/a4/42/8f/CD00226640.pdf/files/CD00226640.pdf/jcr:content/translations/en.CD00226640.pdf 
[S.43]). Im Datenblatt steht als Nummer "23", in der Headerdatei hat der 
define "TIM4_OVR_UIF_vector" den Wert "25", wahrscheinlich, weil die 2 
obersten Interrupts "Reset" und "TRAP" nicht mitgezählt wurden. So 
funktioniert es nun:

#pragma vector = TIM4_OVR_UIF_vector
__interrupt void TIM4_UPD_OVF_IRQHandler(void){
  if(!didInit){
    initMAX7219();
    didInit = true;
  }
  applyBuffer();
  shuffleRight();
  TIM4_SR &=~(1<<0);    //Clear Update interrupt flag
}

Allerdings hätte ich dann erwartet, dass an der Adresse des 
Interrupt-Vektors mit der Nr. 23 die Sprungadresse meiner Funktion 
steht, aber die ganze Interrupt-Tabelle zeigte im Disassembly auf die 
unhandled-exception. Das

asm("rim");

 ist auch schwer bis unmöglich im Datenblatt zu finden. Naja, STM 
typisch.

[STM8 (Gast)]

Max M. schrieb:
> asm("rim"); ist auch schwer bis unmöglich im Datenblatt zu finden. Naja,
> STM

schau Dir mal intrinsics.h an ...

[Stefan F. (Gast)]

Falls du später mal die Variable didInit  außerhalb der ISR verwenden 
wirst, musst du sie als volatile deklarieren.