Anfängerfehler IAR + MSP430F449

[Daniel (Gast)]

Hallo,

momentan versuche ich LED-Lichter zu programmieren. Allerdings gibt IAR 
mir den Fehler, dass in der Funktion P1_ISR in der Zeile

if ((P1IFG & FREIGABE) > 0)

eine Klammer ")" fehlt. In einem anderen Programm funktioniert diese 
Abfrage einwandfrei. Ich weiß nicht, welche Informationen ihr noch 
braucht (totaler Anfänger mit µC-Programmierung), aber hier mal der 
Code:

#include  <msp430x44x.h>

#define FREIGABE  0x01;
#define TASTER    0x02;

void xTASTER(void);

unsigned int AV = 0;
//unsigned char i = 0;

void main(void)
{
  WDTCTL = WDTPW + WDTHOLD; //WDT aus
  P1DIR = 0x00; //init
  P1SEL = 0x00; //komplett als I/O
  P1OUT = 0x00; //alle P1_OUT-Bits = 0
  P1IE |= FREIGABE; //Taster Freigabe für Interrupt aktivieren
  P6DIR = 0xFF; //komplett als Ausgang
  P1OUT = 0x00; //init
  
  _EINT(); //GIE setzen
  
  while(1) //warte bis sich was tut
  {
    _NOP();
  }
  
}

void xTASTER(void)
{
  AV = P6OUT;
  if((AV & 0x01) > 0)
  {
    AV >>= 1;
    AV &= ~(0x01);
  }
  else
  {
    AV = 1;
  }
  
  P6OUT = AV;
}

#pragma vector=PORT1_VECTOR
__interrupt void P1_ISR()
{
  if ((P1IFG & FREIGABE) > 0)
  {
    P1IE |= TASTER;
    P6OUT = 1;
    P1IFG &= ~FREIGABE;
  }
  else
  {
    xTASTER();
    P1IFG &= ~TASTER;
  }
}

PS: Ich denke, der Code ist so einfach, dass er nicht vieler Kommentare 
bedarf.

Vielen Dank,
Daniel

[Slava G. (slava)]

Ich bin auch ein anfänger.

Kanst du mir dan sagen wie du angefangen hast?

[Hans (Gast)]

Slava Gromov schrieb:
> Kanst du mir dan sagen wie du angefangen hast?

LOL

[Daniel (Gast)]

Slava Gromov schrieb:
> Kanst du mir dan sagen wie du angefangen hast?

Wie meinen?

[Slava G. (slava)]

Genau!

[Thomas K. (rlyeh_drifter)]

Daniel schrieb:
> #define FREIGABE  0x01;

durch das #define macht er dir aus

if ((P1IFG & FREIGABE) > 0)

ein

if ((P1IFG & 0x01;) > 0)

wobei das Semikolon echt störend ist ;-)

[Daniel (Gast)]

Thomas Klima schrieb:
> wobei das Semikolon echt störend ist ;-)

Ah, du Gott!!! Super! Vielen Dank... :-)

[Daniel (Gast)]

Gleich noch eine Frage hinterher.
Wenn ich jetzt die Simulation anwerfe und das Bit 0 vom Port 1 setze, 
sollte er ja aus unserer tollen Schleife

 while(1) //warte bis sich was tut
  {
    _NOP();
  }

in das P1_ISR springen, oder nicht? Tuts leider nicht.

[Daniel (Gast)]

ERLEDIGT. :-)

[Daniel (Gast)]

Nächste Frage... Der Interrupt vom zweite Bit (Bit1) des ersten Ports 
wird eigentlich erst aktiviert, wenn das erste Bit (Bit0) gesetzt wurde. 
Wenn ich aber ins Register schaue, wird es trotzdem im IFG gesetzt. :-( 
Also bevor ich Bit0 überhaupt berührt habe.