Interrupt on Change am PIC 16F887

[David (Gast)]

Hi Leute,
wir haben ein Problem und kriegen es nicht gelöst. Das folgende Programm 
zeigt im Groben, was nicht funktioniert.
An einem PIC 16F887 soll an den Pins RB7-RB4 ein Interrupt on Change 
registriert werden, und der PORT B wird komplett auf PORTD (zu 
Debugging-Zwecken gegeben.) Nach dem ersten Interrupt sollen weitere 
Interrupts verboten werden.
Hier der Code:

#include <htc.h>
#include <pic16f887.h>

void pic_init(){
    TRISB = 0xff;

    INTCONbits.RBIE = 1;
    INTCONbits.GIE = 1;

    IOCBbits.IOCB7 = 1;          // Interrupt an Pin RB 4-7 ermoeglichen
    IOCBbits.IOCB7 = 1;
    IOCBbits.IOCB7 = 1;
    IOCBbits.IOCB7 = 1;

    ANSELH = 0x00;
    TRISD = 0x00;
}

void main (){
    pic_init();
    PORTD = 0;
    while(1)
    {
    }
}

void interrupt isr(void){

    _delay(10);
    PORTD = PORTB;
    INTCONbits.RBIE = 0;
    INTCONbits.GIE = 0;

    IOCBbits.IOCB7 = 0;          // Interrupt an Pin RB 4-7 verbieten
    IOCBbits.IOCB7 = 0;
    IOCBbits.IOCB7 = 0;
    IOCBbits.IOCB7 = 0;
}

Allerdings funktioniert das Programm nicht, es werden auch noch weitere 
Interrupts akzeptiert.

Wäre echt toll, wenn uns jemand helfen könnte.

MfG David

[Max H. (hartl192)]

Der "retfie" Befehl den der Compiler am Ende der ISR einbaut aktiviert 
die Interrupts wieder.
Normalerweise funktioniert es mit den Interrupts so: Mit dem Aufruf der 
ISR werden die Interrupts deaktiviert, und dann mit dem retfie Befehl 
wieder aktiviert.
Zitat Datenblatt:
>The “return from interrupt” instruction, RETFIE, exits
>the interrupt routine, as well as sets the GIE bit, which
>re-enables interrupts.
Du könntest alle Interrupts einzeln deaktivieren.

[David (Gast)]

Das mach ich doch mit:

 
INTCONbits.RBIE = 0;

Damit sollten zumindest die Interrupts von PORT B verboten werden....

[X-X (Gast)]

David schrieb:
> Wäre echt toll, wenn uns jemand helfen könnte.

Microchip hat sich das folgendermaßen gedacht.

Wenn die Interruptroutine aufgerufen wird soll das Programm abfragen 
welche Interrupts aufgetreten sind diese abarbeiten. Das macht die 
Abarbeitung meist einfach und übersichtlich so auch in deinem Fall.

Nachdem du die deinen Interrupt bearbeitet hast, löscht du am 
Funktionsende alle Flags. Das war's.

[Chris B. (dekatz)]

David schrieb:
> Allerdings funktioniert das Programm nicht, es werden auch noch weitere
> Interrupts akzeptiert.

Natürlich, denn mit
    IOCBbits.IOCB7 = 0;          // Interrupt an Pin RB 4-7 verbieten
    IOCBbits.IOCB7 = 0;
    IOCBbits.IOCB7 = 0;
    IOCBbits.IOCB7 = 0;

wird ja nur der RB7 gesperrt!!

[David (Gast)]

Danke...
Hab total übersehen, dass ich nur RB7 sperre^^

[Max H. (hartl192)]

INTCONbits.RBIF = 0;

Bau diese Zeile am Ende deiner ISR ein.

David schrieb:
> Allerdings funktioniert das Programm nicht, es werden auch noch weitere
> Interrupts akzeptiert.
Weiter IOC oder andere Interrupts?

[Chris B. (dekatz)]

David schrieb:
> Danke...
> Hab total übersehen, dass ich nur RB7 sperre^^

Und das selbe bei der Freigabe ;-)

[David (Gast)]

Ja, das passiert, wenn man Copy-Paste macht ;)

[michael "meikel" D. (meikel)]

David schrieb:
> Ja, das passiert, wenn man Copy-Paste macht ;)
 Stimmt. Was meinste, wieviel Arbeit Bugs machen, wenn man sie selber 
programmieren muß...