msp430 C-Code Interrupt Problem

[Tester T. (darkstar999)]

Mein Problem ist das nach dem auslösen des Interrupts über einen Taster 
nicht zurück in die main() Funktion gesprungen wird bzw. das Programm 
irgendwo hängen bleibt. Oder habe ich da einen Denkfehler?

Idee des ganzen war eine LED blinken zu lassen und über betätigen des 
Tasters die Zeit zwischen dem blinken der LED zu verringern.

  
  1 #include<msp430.h>
  2 
  3 /*////////////////////////////////////////////////////////////////////////////////////////////
  4  *                                      
  5  *                                        MSP-EXP430G2
  6  *                                      
  7  *                              1       |iVCC           |GND    20
  8  *                              2       |RED            |XIN    19
  9  *                              3       |TxD            |XOUT   18
 10  *                              4       |RxD            |TEST   17
 11  *                              5       |S2             |S1 RST 16
 12  *                              6       |               |       14
 13  *                              7       |               |GREEN  14
 14  *                              8       |               |       13
 15  *                              9       |               |       12
 16  *                              10      |               |       11
 17  *
 47 */////////////////////////////////////////////////////////////////////////////////////////////
 48 
 49 
 50 #define LEDRED BIT0
 51 #define LEDGREEN BIT6
 52 #define BUTTON BIT3
 53 #define DEBOUNCE_MS 50 
 54 
 55 //***************************************
 56 void delay_ms(unsigned int ms){
 57         while(ms--){
 58                 __delay_cycles(10000);
 59         }
 60 }
 61 //***************************************
 62 char button_pressed(void){
 63         if((P1IN & BUTTON) == 0){
 64                 delay_ms(DEBOUNCE_MS);
 65                 if ((P1IN & BUTTON) == 0) return 1;
 66         }}
 67         return 0;
 68 }
 69 //***************************************
 70 
 71 
 72 unsigned int faktor = 1;
 73 
 74 int main(void){
 75         
 76         WDTCTL = WDTPW + WDTHOLD;                 
 77 
 78         P1DIR |= LEDRED;
 79         P1OUT |= BUTTON;
 80         P1DIR &= ~BUTTON;
 81         P1IE |= BUTTON;
 82         P1IFG &= ~BUTTON;
 83         P1REN |= BUTTON;
 84         __enable_interrupt();
 85  
 86         while(1){
 87                 delay_ms(1000/faktor);
 88                 P1OUT ^= LEDRED;         
 89         } 
 90 
 91 }
 92 
 93 void __attribute__ ((interrupt(PORT1_VECTOR))) watchdog_timer (void){
 94         if(button_pressed()){   
 95                 faktor+=1;
 96                 P1IFG &= ~BUTTON;
 97         }
 98 }

[Tester T. (darkstar999)]

EDIT:

button_pressed() Abfrage im interrupt scheint zu viel Code zu sein. Nach 
dem entfernen hängt sich der µC nicht mehr weg bzw. das Programm bleibt 
nicht mehr hängen. Allerdings scheint das Faktor hoch zählen noch 
wirkungslos zu bleiben.

  
  1 #include<msp430.h>
  2 
  3 /*////////////////////////////////////////////////////////////////////////////////////////////
  4  *                                      
  5  *                                        MSP-EXP430G2
  6  *                                      
  7  *                              1       |iVCC           |GND    20
  8  *                              2       |RED            |XIN    19
  9  *                              3       |TxD            |XOUT   18
 10  *                              4       |RxD            |TEST   17
 11  *                              5       |S2             |S1 RST 16
 12  *                              6       |               |       14
 13  *                              7       |               |GREEN  14
 14  *                              8       |               |       13
 15  *                              9       |               |       12
 16  *                              10      |               |       11
 17  *
 47 */////////////////////////////////////////////////////////////////////////////////////////////
 48 
 49 
 50 #define LEDRED BIT0
 51 #define LEDGREEN BIT6
 52 #define BUTTON BIT3
 53 #define DEBOUNCE_MS 50 
 54 
 55 //***************************************
 56 void delay_ms(unsigned int ms){
 57         while(ms--){
 58                 __delay_cycles(10000);
 59         }
 60 }
 61 //***************************************
 62 char button_pressed(void){
 63         if((P1IN & BUTTON) == 0){
 64                 delay_ms(DEBOUNCE_MS);
 65                 if ((P1IN & BUTTON) == 0) return 1;
 66         }}
 67         return 0;
 68 }
 69 //***************************************
 70 
 71 
 72 unsigned int faktor = 1;
 73 
 74 int main(void){
 75         
 76         WDTCTL = WDTPW + WDTHOLD;                 
 77 
 78         P1DIR |= LEDRED;
 79         P1OUT |= BUTTON;
 80         P1DIR &= ~BUTTON;
 81         P1IE |= BUTTON;
 82         P1IFG &= ~BUTTON;
 83         P1REN |= BUTTON;
 84         __enable_interrupt();
 85  
 86         while(1){
 87                 delay_ms(1000/faktor);
 88                 P1OUT ^= LEDRED;         
 89         } 
 90 
 91 }
 92 
 93 void __attribute__ ((interrupt(PORT1_VECTOR))) interrupt(void){  
 94                 faktor+=1;
 95                 P1IFG &= ~BUTTON;
 96 }

[Rufus Τ. F. (rufus)]

In einer Interruptroutine eine Funktion namens delay_ms aufzurufen ist 
keine gute Idee.

Interruptgesteuerte Tastenauswertung und -Entprellung macht man nicht 
mit Warteschleifen.

[Tester T. (darkstar999)]

Rufus Τ. F. schrieb:
> In einer Interruptroutine eine Funktion namens delay_ms aufzurufen ist
> keine gute Idee.

Hey danke für die Antwort jepp habe es gemerkt. Der Hauptfehler mit der 
Variable ist auch gelöst. Bin nen bisel eingerostet Erinnerung an DIR2 
Interrupts und Register ausm Studium. Damit der Interrupt auf die 
Variable zugreifen kann bzw. die Variable im Interrupt bereit steht muss 
dem Compiler das mitgeteilt werden

volatile

Da der Compiler ansonsten die Variable als Balast ansieht und sie im 
worst case sogar weg optimiert.

Rufus Τ. F. schrieb:
>Interruptgesteuerte Tastenauswertung und -Entprellung macht man nicht
>mit Warteschleifen.

Wie kann ich das den in Software lösen oder geht das nur in Hardware 
während des Interrupts?