MSP430 Interrupt Problem

[Philipp .. (philippg)]

Hi!
Ich habe ein kleines Problem mti einem der MSP430 Starter Kits.


Ich wollte ersteinmal ein ganz einfaches Toggeln einer LED per Button 
machen.
Das Togeln per vorinstalliertem Button funktioniert einwandfrei.
Will ich jedoch mittels eigenem angelöteten Schalter toggeln schlägt 
dies fehl.

Anschlüsse:
P10 und P16 LEDs
P13 ist ein aufgelöteter Taster.

Ich habe ein Kabel an GND befestigt. Sobald das Kabel P13 berührt, 
schalten die LEDs um.
Nach meinem Code oben müssten die LEDs aber auch umschalten, wenn ich 
P14 berühre... richtig?

void main(void){
  WDTCTL = WDTPW + WDTHOLD;  // Setup WDT
  
  P1DIR = BIT0 | BIT6;    // Setup In/Out
  P1OUT = BIT6;        // Flash green LED
  
  P1IE = BIT3;        // Interrupt on Bit3
  
  __bis_SR_register(GIE);

  P1IE |= BIT4;
  P1IES |= BIT4;
  
}


  /*
   * Button Interrupt
   */
#pragma vector=PORT1_VECTOR
__interrupt void PORT1_ISR(void){
  if( P1OUT & BIT6 )
    P1OUT = BIT0;
  else
    P1OUT = BIT6;
  P1IFG &= !BIT3;      // Reset Interrupt Flag Register
}

Bin für jede Hilfe dankbar!

Grüße, Philipp

[LuXXuS 9. (aichn)]

Dir fehlt ne Endlosschleife oder LPM.

[Philipp .. (philippg)]

Ich habe als letzte Zeile in der main()-Methode jetzt mal while(); 
eingefügt.
Bringt aber keine veränderung.

[LuXXuS 9. (aichn)]

While(1){}

[Philipp .. (philippg)]

Sry, hatte oben falsch gepostet!

while(1);

Stand im Code!

while(1); bzw. while(1){} bringt keine Veränderung!

[LuXXuS 9. (aichn)]

Poste bitte nochmal genau, was du überhaupt vorhast. Mit welchem Pin 
willst du jetzt was machen?

Wenn du einen Taster gegen GND schaltest, dann brauchst du auch nen 
Pull-Up.

[Philipp .. (philippg)]

Pin3 hat einen fertig verlöteten Taster.
Dieser löst den Interrupt korrekt aus.

Statt den Taster zu drücken, kann ich auch ein Kabel an GND und die 
Lötstelle am Taster halten. Funktioniert ebenso.

Wenn ich das Kabel von GND an Pin4 halte, wird der Interrupt allerdings 
nicht ausgelöst!
Das möchte ich aber.

Pin4 soll mittels eines Schalters (oder eben atm durch Kabel dran 
halten) den Interrupt auslösen.

[Philipp .. (philippg)]

Hier mal besser aufgeräumter Code:

So hier nochmal aufgeräumterer Code:

#include  "msp430x20x2.h"

#define     TASTER          BIT3
#define     LED             BIT6
#define     KABEL            BIT4


void main(void){
  WDTCTL = WDTPW + WDTHOLD;  // Setup WDT
  
  P1DIR = LED;  // Output in 0 and 6
  P1OUT = P1DIR;
  P1IN = !P1DIR;
  P1IFG = 0x00;
  
  P1IE = KABEL | TASTER;

  __bis_SR_register(GIE);
  while(1){}
  
}


  /*
   * Button Interrupt
   */
#pragma vector=PORT1_VECTOR
__interrupt void PORT1_ISR(void){
  P1OUT ^= LED;
  P1IFG = 0x00;          // Reset Interrupt Flag Register
}

[LuXXuS 9. (aichn)]

Schreiben ans P1IN-Register ist Blödsinn. Auch ist es nicht sinnvoll, 
Anweisungen wie P1OUT = P1DIR zu schreiben - das ist aber eher 
genereller Natur.

Benutzt du das Launchpad? Hast du einen Pull-Up-R (In Software 
jedenfalls nicht)? Evtl. hat der Tasterveinen in Hardware, das Kabel 
atürlich nicht.

[Philipp .. (philippg)]

Ja, ist das Launchpad.

Ich habe jetzt mal den Softwareseitigen Pull-Up-R aktiviert. Keine 
Veränderung.
hardwareseitig habe ich noch keinen. Muss gerade mal schauen was ich da 
an Wiederstand einsetzen muss.

#include  "msp430x20x2.h"

#define     TASTER          BIT3
#define     LED             BIT6
#define     KABEL            BIT4


void main(void){
  WDTCTL = WDTPW + WDTHOLD;  // Setup WDT
  
  P1DIR = LED;  // Output in 0 and 6
  P1REN |= KABEL;
  P1IFG = 0x00;
  P1IE = KABEL | TASTER;  
  __bis_SR_register(GIE);
  while(1){}
  
}

// Taster Interrupt
#pragma vector=PORT1_VECTOR
__interrupt void PORT1_ISR(void){
  P1OUT ^= LED;
  P1IFG = 0x00;          // Reset Interrupt Flag Register
}

[Stefan (Gast)]

Philipp ... schrieb:
> Ich habe jetzt mal den Softwareseitigen Pull-Up-R aktiviert.
Nein, du hast den Pull-Down-R eingeschaltet.  Wenn du den Pull-Up willst 
musst du noch das entsprechden P1OUT Bit setzen.

[Philipp .. (philippg)]

Und schon klappt es :-)

Wusste ich bisher so nicht..  großes Danke für die Hinweise und Tipps!
Es läuft wunderbar!

Hier nochmal der entgültig funktionierende Code:

#include  "msp430x20x2.h"

#define     TASTER          BIT3
#define     LED             BIT6
#define     KABEL            BIT4


void main(void){
  WDTCTL = WDTPW + WDTHOLD;  // Setup WDT
  
  P1DIR = LED;  // Output in 0 and 6
  P1REN |= KABEL; // Resistor enabled
  P1OUT |= KABEL; // out =1 for pullup-resistor
  P1IFG = 0x00; // Clear interrupt flags
  P1IE = KABEL | TASTER;  // Turn on Interrupts
  __bis_SR_register(GIE);
  while(1){}
  
}

// Taster Interrupt
#pragma vector=PORT1_VECTOR
__interrupt void PORT1_ISR(void){
  P1OUT ^= LED;
  P1IFG = 0x00;          // Reset Interrupt Flag Register
}