PIC18F46K80 CAN ISR

[Hans (Gast)]

Guten Tag und ein schönes Wochenende an alle Leser und Beantworter,

ich sitze seit einigen Tagen an einer CAN-Kommunikation mittels PIC 
18F46K80. Senden, Empfangen, klappt alles wunderbar - nur an der ISR 
scheitere ich. Ich nutze MPLABX und einen XC8-Compiler. Eigentlich ist 
mit dieser Kombination die Erstellung der ISR fast trivial. Nur für mich 
irgendwie nicht.

Den globalen Interrupt habe ich freigegeben, den Peripherial genauso, 
nur in die ISR will er einfach nicht - hat jemand auf die Schnelle eine 
Idee wo mein Fehler liegt?

Vielen Dank im Voraus für alle die eine passende Antwort parat haben.

// PIC18F46K80 Configuration Bit Settings

// 'C' source line config statements

#include "configuration.c"
#include <xc.h>
#include "p18f46k80.h"

//Funktionsprototypen
void delay();
void InitECAN();
void ECAN_Transmit();
void interrupt ISR(void);

void delay() {
    int counter = 0;
    for (counter = 0; counter<10000; counter++) {
        ;
    }
}

void main(void) {

    TRISBbits.TRISB2 = 0;
    TRISBbits.TRISB3 = 1;
    TRISCbits.TRISC6 = 0;
    TRISCbits.TRISC7 = 1;
    TRISBbits.TRISB5 = 0;

    InitECAN();
    INTCONbits.GIE = 1;
    PIE5bits.RXBnIE = 1;
    PIE5bits.IRXIE = 1;
    RXB0IF = 0;
    RXB1IF = 0;
    IRXIF = 0;

    while (1) {
        PORTBbits.RB0 = 0;
        delay();
        PORTBbits.RB0 = 1;
        delay();
}

}

void interrupt ISR(void)
{
    if(RXB0IF && PIE5bits.RXBnIE)
    {
        RXB0IF = 0;
        ECAN_Transmit();
    }
}

[Erich (Gast)]

Dein Programm kann niemals so funktionieren oder funktioniert haben.
Egal mit mit oder ohne Interrupt.

"RXBnIE"  haha

Gruss

[Stephan M. (multimeter90)]

Ich weiß nicht wie es mit dem xc8 aussieht. Ich benutze den C18. Da 
musste der ISR immer noch ein #pragma davor um dem Compiler zu sagen, wo 
die Funktion (im Speziellen _asm GOTO ISR _endasm) liegen soll, 
Ausschnitt:

#pragma code high_vector=0x08
void HightInterrupt(void) { _asm GOTO ISR _endasm }
#pragma code

#pragma code low_vector=0x18
void LowInterrupt(void) { _asm GOTO ISR _endasm }
#pragma code


volatile uint32_t os_SysTime = 0;


/**
 * @brief Interrupt-Service-Routine des Controllers
 *
 */
#pragma interruptlow ISR
void ISR(void) {

}

[Chris B. (dekatz)]

Ich weiss nicht was nach einem POR in den Registern steht, sehe jetzt 
auch nicht nach....
Aber vor der globale Interrupt Freigabe:
Mit RCON.IPEN festlegen ob mit oder ohne priorisierten Interrupt 
gearbeitet wird
Im IPRn Register festlegene ob HIGH oder LOW Interrupt.
Und sicherheitshalber die entsprechenden Flags im PIRn Register löschen.

[Hans (Gast)]

@Erich: Interessant, dass dein Post nur den Sinn hatte mich auszulachen! 
Mal den Befehl im MPLABX mit XC8 eingegeben? Der Comnpiler unterstützt 
diesen Befehl, aber kein Problem, nicht alle Menschen haben Interesse 
daran sich weiterzubilden - und doch, der Rest hat funktioniert!! Egal 
was du glauben magst.

@Stephan Meter: ja, so kenne ich die Routine auch noch als ich sie noch 
im alten MPLAB mit C18 geschrieben habe. Auf dieser Seite:
http://microchip.wikidot.com/faq:31
wird die Implementierung beschrieben wie sie eigentlich in XC8 
vorgesehen ist. Trotzdem vielen Dank für deine Bemühungen.

@Chris B: vielen Dank für den Tipp, werde morgen nochmal schauen ob es 
daran gelegen hat, jetzt ist eindeutig schon zu spät :) hab' gerade auch 
nochmal in das DB geschaut und auf S.147 wieder das gefunden bei dem ich 
mir sicher war, dass ich es schon mal gelesen hatte.

"When the IPEN bit is cleared (default state), the interrupt priority 
feature is disabled and interrupts are compatible with PIC® mid-range 
devices. In Compatibility mode, the
interrupt priority bits for each source have no effect."

Was ich gerade allerdings auch noch gelesen habe, ist, dass das PEIE-Bit 
im INTCON-Register gesetzt werden muss. Das habe ich vergessen.

Lösung kommt, wenn ich sie habe. Wenn natürlich jemand schneller drauf 
kommt - VIELEN Dank

[Hans (Gast)]

Auch wenn es Ignoranten wie Erich niemals glauben werden (ich hoffe 
nicht, dass ich niemals mit solchen Menschen zusammenarbeiten muss) - 
hier der funktionierende CAN-Quellcode. Geschrieben für MPLAB X mit 
XC8-Kompiler. Vielleicht hilft es jemandem weiter der im Rahmen seines 
Studiums (so wie ich) oder sonstwie damit zu tun hat. Ich wäre froh 
gewesen in diesem Forum etwas dergleichen zu finden, statt ausgelacht zu 
werden!! Gibt es hier keine Moderatoren die so einen Mist unterbinden?

// PIC18F46K80 Configuration Bit Settings

// 'C' source line config statements

#include "configuration.c"
#include <xc.h>
#include "p18f46k80.h"

//Funktionsprototypen
void delay();
void InitECAN();
void ECAN_Transmit(BYTE);
unsigned char ECAN_Receive();

void interrupt ISR(void)            // Interrupt-Service-Routine
{
    if(RXB1IF && PIE5bits.RXBnIE)   // bei Auslösen des CAN-Empfangen-Interrupts
    {
        RXB1IF = 0;                 // Interrupt-Flag zurücksetzen
        ECAN_Receive();             // Nachricht lesen
        ECAN_Transmit(0x02);        // Nachricht versenden
    }
}

void delay() {                      // Rudimentäre Warteschleife
    int counter = 0;                // Zähler=0 setzen
    for (counter = 0; counter<10000; counter++)
    {
        ;
    }
}

void main(void) {

    TRISBbits.TRISB2 = 0;           // RB2 als Output
    TRISBbits.TRISB3 = 1;           // RB3 als Input
    TRISCbits.TRISC6 = 0;           // RC6 als Output
    TRISCbits.TRISC7 = 1;           // RC7 als Input

    InitECAN();                     // Inititialisieren des ECAN-Moduls
    RXB0IF = 0;                     // Interrupt-Flag löschen
    RXB1IF = 0;                     // Interrupt-Flag löschen

    INTCONbits.GIE = 1;             // Global Interrupts freigeben
    INTCONbits.PEIE = 1;            // Peripheral Interrupts freigeben
    PIE5bits.RXBnIE = 1;            // CAN-Receive-Interrupts freigeben

    while (1) {                     // while-Schleife für Hauptprogramm
        PORTBbits.RB0 = 0;          // Blinkende LED zur Funktionsanzeige
        delay();
        PORTBbits.RB0 = 1;
        delay();
    }
}

[Stephan M. (multimeter90)]

Woran lag es denn nun genau? Also die IS-Routine sieht ja gleich aus. 
Was hat jetzt den Erfolg gebracht?