Problem mit ISR beim ext. Interrupt (PIC24)

[Hannes (Gast)]

Hallo Leute,

ich möchte einen externen Interrupt programmieren und bekomme meinen 
Code nicht kompiliert.
Fehlermeldung:

mainXC16.c:22:16: error: expected '=', ',', ';', 'asm' or 
'__attribute__' before 'ISR'

Ich bin schon etwas betriebsblind, aber vielleicht sieht jemand den 
Fehler sofort.

Danke!

Controller:  PIC24HJ128GP306
IDE:  MPLAB X mit Compiler XC16 (v1.25)

#define FOSC    7370000UL   
#define FCY     (FOSC/2)    

#include "xc.h"  
#include <stdio.h>
#include <stdlib.h>
#include <libpic30.h>

volatile uint8_t flag = 0;

void interrupt ISR(void){
 if(IFS3bits.INT4IF == 1){
        IFS3bits.INT4IF = 0;    // Reset the external interrupt flag  
        flag = 1;
 }
}

void interrupt_init(void){  //hierin sollte kein Fehler sein - kompiliert ohne Probleme
    INTCON1bits.NSTDIS = 1; //Interrupt nesting disabled 
    INTCON2bits.INT4EP = 1; //Ext. Interrupt falling edge 
    IFS3bits.INT4IF = 0;    // Reset the external interrupt flag
    IPC13bits.INT4IP = 3;   //Set ext. interrupt 4 priority to 3
    IEC3bits.INT4IE = 1;    //Enable ext. Interrupt 4
}


int main(void) {

TRISB = 0;
TRISD = 1;
interrupt_init();

    while(1){


        if(flag){
            flag=0;
            //mach was
        }
    }
    return 0;
}

[Pascal Würzer (Gast)]

Also im User Guide zum XC16 
(http://ww1.microchip.com/downloads/en/DeviceDoc/50002071E.pdf) steht 
das mit _attribute_ drin (Kapitel 14)

Verwechselst du die Schreibweise mit XC8?

[Klaus (Gast)]

Mal unabhängig von der Schreibweise, irgendwie muß für den Compiler zu 
erkennen sein, zu welchem Interrupt die Funktion gehören soll.

void _ISR _INT0Interrupt(void);

MfG Klaus

[Hannes (Gast)]

Danke schonmal.

Im Userguide steht aber auch:
Thus, interrupt and _interrupt_  are equivalent.

Ist damit nicht die Funktion "ISR" automatisch als ISR gekennzeichnet?
Ich habe das von hier:
[[http://picguides.com/beginner/interrupts.php]]


Klaus schrieb:
> Mal unabhängig von der Schreibweise, irgendwie muß für den Compiler zu
> erkennen sein, zu welchem Interrupt die Funktion gehören soll.

Das checke ich ja dann mit der if-Abfrage. Hier sollen später noch 
andere ISR rein. Mit der Codezele von Klaus kompiliert es zwar, es 
passiert aber nichts (mit LED und entsprechender zeile in der ISR 
getestet).

Ehrlich gesagt, komme ich mit diesen Unterstrichen überhaupt nicht klar 
:-(

[Hannes (Gast)]

Hannes schrieb:
> Mit der Codezele von Klaus kompiliert es zwar, es
> passiert aber nichts (mit LED und entsprechender zeile in der ISR
> getestet).

Sorry, hatte hier einen kleinen Fehler im Code. Der Interrupt funzt 
jetzt. Danke!

Das mit den Unterstrichen verstehe ich vielleicht auch noch irgendwann.

[Dieter W. (dds5)]

Hannes schrieb:
> Das checke ich ja dann mit der if-Abfrage. Hier sollen später noch
> andere ISR rein.

Das war bei den PIC 12, 16 und 18 so aber die 24er haben für jeden 
Interrupt einen eigenen Vector und brauchen deshalb auch jeweils eine 
eigene ISR.

Für den alten C-Compiler funktioniert bei mir z.B.

 void __attribute__ ((__interrupt__, no_auto_psv)) _INT1Interrupt(void)

Mit XC16 hab ich noch nicht gearbeitet.

[Klaus (Gast)]

Hannes schrieb:
> Das checke ich ja dann mit der if-Abfrage. Hier sollen später noch
> andere ISR rein. Mit der Codezele von Klaus kompiliert es zwar, es
> passiert aber nichts (mit LED und entsprechender zeile in der ISR
> getestet).

Wurde zwar schon gesagt, der PIC24 hat sehr viele Interrupte. Das können 
bei manchem Chip schon mal über 100 sein. Jeder und auch die Traps haben 
einen eigenen Vector und _ISR ist ein Systemmakro (daher der Unterstrich 
am Anfang) das erledigt das meisste. Geh mal mit gedrückter 
Control-Taste mit der Maus über das Symbol, dann kannst du sehen, wie es 
expandiert aussieht.

Das Interruptflag brauchst du nicht abzufragen, wenn du im 
Interrupthandler angekommen bist, ist es auf jeden Fall gesetzt. Nur 
löschen sollte man es.

Dieter W. schrieb:
> Mit XC16 hab ich noch nicht gearbeitet.

Ist nur eine neue Bezeichnung für den C30, damit die Namen einheitlicher 
sind: MPLABX, XC8, XC16, XC32, wobei die beiden großen der gcc ist.

MfG Klaus