PIC 18F2685 - Ext. Interrupt funktioniert nicht

[Marco (Gast)]

Ich hab ein Problem.
Leider reagiert der besagte PIC nicht auf einen ext. Interrupt an Pin 
RB0. Der Timerinterrupt hingegen klappt wunderbar.
Wie man sieht toggelt der Timerinterrupt eine LED an Port RC0, der 
Interrupt von Pin RB1 soll das bei jeder steigenden Flanke an RC1 tun.

Hier mal mein Quellcode(MPLAB+C18):

/** I N C L U D E S **********************************************************/ 
#include <p18cxxx.h> 
#include "delays.h"                       // für die Warteschleife 
  
/** Configuration ********************************************************/ 
#pragma config OSC = HS   
#pragma config PWRT = ON 
#pragma config BOREN = OFF 
#pragma config WDT = OFF  //Watchdog Timer 
#pragma config LVP = OFF  //Low Voltage ICSP 
#pragma config PBADEN = OFF //PB0-PB4 digital inputs

void high_isr(void);

#pragma code high_vector=0x08
void interrupt_at_high_vector(void)
  {
    _asm goto high_isr _endasm
  }

#pragma code

#pragma interrupt high_isr
void high_isr (void)
{
TMR0L = 218;             // preset for timer register
  if(INTCONbits.TMR0IF)  //Timerinterrupt?
  {
      if(LATCbits.LATC0) //LED Toggeln
    {
        LATCbits.LATC0 = 0;
        }
        else
      {
          LATCbits.LATC0 = 1;
        }
    INTCONbits.TMR0IF = 0;  //Timer-Int-Flag rucksetzen
    INTCONbits.RBIE = 1;
    INTCONbits.INT0IE = 1;  //
    INTCONbits.GIE = 1;
  }


  if(INTCONbits.RBIF)
  {
    if(LATCbits.LATC1)
    {
        LATCbits.LATC1 = 0;
        }
        else
      {
          LATCbits.LATC1 = 1;
        }
    INTCONbits.RBIF = 0;
  INTCONbits.RBIE = 1;
    T0CONbits.TMR0ON = 0;
    INTCONbits.GIE = 1;
  INTCONbits.GIEL = 1;
    }
  if(INTCONbits.INT0IF)
  {
    LATCbits.LATC0 = 1;
    INTCONbits.INT0IF = 0;
    T0CONbits.TMR0ON = 1;
    INTCONbits.GIE = 1;
  INTCONbits.GIEL = 1;
    }
}

void main(void){
  TRISA = 0x00;
  LATA = 0x00;
  TRISB = 0xFF;
  LATC = 0x00;  
  TRISC = 0xF0;
  

  T0CONbits.T08BIT = 1;
  T0CONbits.T0CS = 0;
  T0CONbits.PSA = 1;
  T0CONbits.TMR0ON = 1;
  INTCONbits.TMR0IE = 1;
    INTCONbits.RBIE = 1;
  INTCON2bits.RBIP = 1;
  INTCON2bits.INTEDG0 = 1;

  INTCONbits.GIE = 1;
  INTCONbits.GIEL = 1;
  while(1);
}

Danke schonmal im Voraus für eure Hilfe.
Gruß

Marco

[René (Gast)]

Hallo Marco,

also wenn ich deinen Code richtig verstehe und mir das Datenblatt zum 
Pic ansehe, dann ziehst du beim externen Interrupt lediglich den RC0 auf 
High Pegel.

if(INTCONbits.INT0IF) {
    LATCbits.LATC0 = 1;
    INTCONbits.INT0IF = 0;
    T0CONbits.TMR0ON = 1;
    INTCONbits.GIE = 1;
    INTCONbits.GIEL = 1;
}

Das schalten des RC1 erzeugst du über den RB State change, der laut 
Datenblatt nur über die Pin RB7:RB4 funktioniert. Da du über die Port 
Pins RB7:RB4 nichts gesagt hast, gehe ich davon aus das du die auch 
nicht verwendest.

if(INTCONbits.INT0IF) {
    if(LATCbits.LATC1) {
        LATCbits.LATC1 = 0;
    }
    else {
        LATCbits.LATC1 = 1;
    }
    INTCONbits.INT0IF = 0;
}

Die Interrupt Enable bits müssen nicht noch einmal in der ISR gesetzt 
werden. Da du die an keiner Stelle im Code deaktivierst. Es muss 
lediglich das Interrupt Flag für den INT0 wieder gelöscht werden.

Gruß René

[Marco (Gast)]

So, ich hänge immer noch (wieder da dran)....habe jetzt mal den Code 
sehr simpel gestaltet. An RB0 liegt ein 1Hz Rechteck. Im Debugmodus 
lässt sich PortB ja anzeigen, da erkennt er auch wunderbar das Rechteck 
- allerdings setzt er nicht das Interrupt-Flag und geht nicht in die 
ISR...

#include <p18cxxx.h> 
  
/** Configuration ********************************************************/ 
#pragma config OSC = HS   
#pragma config PWRT = ON 
#pragma config BOREN = OFF 
#pragma config WDT = OFF  //Watchdog Timer 
#pragma config LVP = OFF  //Low Voltage ICSP 

void high_isr(void);

int temp;
#pragma code high_vector=0x08
void interrupt_at_high_vector(void)
  {
    _asm goto high_isr _endasm
  }

#pragma code

#pragma interrupt high_isr
void high_isr (void)
{
      if(LATCbits.LATC0==0) //LED Toggeln
    {
        LATCbits.LATC0 = 1;
        }
        else
      {
          LATCbits.LATC0 = 0;
        }
temp = PORTB;
INTCONbits.INT0IF = 0;
}

void main(void)
{
ADCON1 = 0xFF;
TRISB  = 0x01; 
RCONbits.IPEN = 1;    // Enable High Priority Interrupts
INTCONbits.GIEH  = 1;  
INTCON2bits.INTEDG0 = 1;
INTCONbits.INT0IE=1;
INTCONbits.GIE=1;


while(1)
{

}
}

[Marco (Gast)]

Äh, ich hab mein Problem entdeckt.

Es lag weder am Controller noch am Quellcode:
Es war der "Animate" Modus beim Debugen...wenn man den Controller normal 
laufen lässt (egal ob im debug oder nicht) geht es vollkommen 
problemlos. Blöderweise kann man dort halt nicht "zuschaun" was die Bits 
so tun...

[Andreas B. (biosniper)]

Marco schrieb:

>       if(LATCbits.LATC0==0) //LED Toggeln
>     {
>         LATCbits.LATC0 = 1;
>         }
>         else
>       {
>           LATCbits.LATC0 = 0;
>         }
>

Toggle macht man einfacher so:

LATCbits.LATC0 = !LATCbits.LATC0;