Mal wieder Interrupts

[Lena R. (eny)]

Diesmal PIC18F2523, MPLAB X, XC 8, PicKit 3, kein Bootloader.

/* 
 * File:   main.c
 * Author: Lena
 *
 * Created on 4. August 2013, 23:25
 */
// PIC18F2523 Configuration Bit Settings

#include <xc.h>

// CONFIG1H
#pragma config OSC = HS         // Oscillator Selection bits (HS oscillator)
#pragma config FCMEN = OFF      // Fail-Safe Clock Monitor Enable bit (Fail-Safe Clock Monitor disabled)
#pragma config IESO = OFF       // Internal/External Oscillator Switchover bit (Oscillator Switchover mode disabled)

// CONFIG2L
#pragma config PWRT = OFF       // Power-up Timer Enable bit (PWRT disabled)
#pragma config BOREN = SBORDIS  // Brown-out Reset Enable bits (Brown-out Reset enabled in hardware only (SBOREN is disabled))
#pragma config BORV = 3         // Brown Out Reset Voltage bits (Minimum setting)

// CONFIG2H
#pragma config WDT = OFF        // Watchdog Timer Enable bit (WDT disabled (control is placed on the SWDTEN bit))
#pragma config WDTPS = 32768    // Watchdog Timer Postscale Select bits (1:32768)

// CONFIG3H
#pragma config CCP2MX = PORTC   // CCP2 MUX bit (CCP2 input/output is multiplexed with RC1)
#pragma config PBADEN = ON      // PORTB A/D Enable bit (PORTB<4:0> pins are configured as analog input channels on Reset)
#pragma config LPT1OSC = OFF    // Low-Power Timer1 Oscillator Enable bit (Timer1 configured for higher power operation)
#pragma config MCLRE = ON       // MCLR Pin Enable bit (MCLR pin enabled; RE3 input pin disabled)

// CONFIG4L
#pragma config STVREN = ON      // Stack Full/Underflow Reset Enable bit (Stack full/underflow will cause Reset)
#pragma config LVP = ON         // Single-Supply ICSP Enable bit (Single-Supply ICSP enabled)
#pragma config XINST = OFF      // Extended Instruction Set Enable bit (Instruction set extension and Indexed Addressing mode disabled (Legacy mode))

// CONFIG5L
#pragma config CP0 = OFF        // Code Protection bit (Block 0 (000800-001FFFh) not code-protected)
#pragma config CP1 = OFF        // Code Protection bit (Block 1 (002000-003FFFh) not code-protected)
#pragma config CP2 = OFF        // Code Protection bit (Block 2 (004000-005FFFh) not code-protected)
#pragma config CP3 = OFF        // Code Protection bit (Block 3 (006000-007FFFh) not code-protected)

// CONFIG5H
#pragma config CPB = OFF        // Boot Block Code Protection bit (Boot block (000000-0007FFh) not code-protected)
#pragma config CPD = OFF        // Data EEPROM Code Protection bit (Data EEPROM not code-protected)

// CONFIG6L
#pragma config WRT0 = OFF       // Write Protection bit (Block 0 (000800-001FFFh) not write-protected)
#pragma config WRT1 = OFF       // Write Protection bit (Block 1 (002000-003FFFh) not write-protected)
#pragma config WRT2 = OFF       // Write Protection bit (Block 2 (004000-005FFFh) not write-protected)
#pragma config WRT3 = OFF       // Write Protection bit (Block 3 (006000-007FFFh) not write-protected)

// CONFIG6H
#pragma config WRTC = OFF       // Configuration Register Write Protection bit (Configuration registers (300000-3000FFh) not write-protected)
#pragma config WRTB = OFF       // Boot Block Write Protection bit (Boot block (000000-0007FFh) not write-protected)
#pragma config WRTD = OFF       // Data EEPROM Write Protection bit (Data EEPROM not write-protected)

// CONFIG7L
#pragma config EBTR0 = OFF      // Table Read Protection bit (Block 0 (000800-001FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR1 = OFF      // Table Read Protection bit (Block 1 (002000-003FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR2 = OFF      // Table Read Protection bit (Block 2 (004000-005FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR3 = OFF      // Table Read Protection bit (Block 3 (006000-007FFFh) not protected from table reads executed in other blocks)

// CONFIG7H
#pragma config EBTRB = OFF      // Boot Block Table Read Protection bit (Boot block (000000-0007FFh) not protected from table reads executed in other blocks)




#define LED LATAbits.LA4
#define LED2 LATCbits.LC0

int led_state = 0;

void delay( int ms );

//Zeitverzögerung
void delay( int ms ) {
    int i=0;
    //1 ms = 4000 Befehlstakte (16 MHz Quarz / 4)
    for( i=0; i<ms; i++ ) {
            _delay(4000);
    }
}

//void interrupt isr_high(void);

void interrupt isr_high(void) {
   if(led_state == 0)
    {
       LED = 1;
       led_state = 1;
    } else {
       LED = 0;
       led_state = 0;
    }
   INTCONbits.INT0F = 0;
   INTCON3bits.INT2F = 0;
}


void main(void) {

  

    //Port A
    LATA = 0x00;                //Pegel
    TRISA = 0x00;    //Alle Pins von Port A sind Ausgänge
//    ANSELA = 0x00;    //Alle Pins von Port A sind digitale I/O's
    //Port B
    LATB = 0x00;
    TRISB = 0xFF;    //Alle Pins von Port B sind Eingänge
//    ANSELB = 0x00;    //Alle Pins von Port A sind digitale I/O's
    //Port C
    LATC = 0x00;
    TRISC = 0xF0;    //RC0..RC3 = Ausgänge, RC4..RC7 = Eingänge
//    ANSELC = 0x30;    //nur RC4 und RC5 sind analoge Eingänge

    RCONbits.IPEN = 1;          //Interrupts können Prioritäten haben
    INTCONbits.GIE_GIEH = 1;     //Interrupts können hohe P. haben
    INTCONbits.GIEL = 1;
    INTCONbits.INT0IE = 1;
    INTCON3bits.INT2IE = 1;
    INTCON2bits.INTEDG0 = 1;    //Interrupt on rising edge
    INTCON2bits.INTEDG2 = 1;     //Interrupt on rising edge
    INTCON3bits.INT2IP = 1;



    while(1)
    {
        if(INTCON3bits.INT2F == 0) LED2 = 1;
        delay(20);
        LED2 = 0;
        delay(20);
    }
    


}

Irgendwie bekomm ich das nicht ans Laufen.
An Int0 und Int2 sind jeweils Funktionsgeneratoren angeschlossen, die 
ein Rechtecksignal liefern. Zum Testen einfach zwischen 1 und 50 Hz.

Es scheint, als ob er nicht in die ISR springt.
LED2 blinkt föhlich vor sich hin, LED macht garnix.

Meiner Meinung ist das Programm aber soweit richtig, oder hab ich 
irgendetwas vergessen?

[Julian B. (julinho)]

Ich kenne mich mit Pics nicht aus, aber in der I-Routine  änderst du nur 
die Variable led_state, aber der eigentliche LED-Port wird nicht 
beschrieben,oder?

[Lena R. (eny)]

oben:  #define LED LATAbits.LA4
in ISR: LED = 1;

Das wäre zu einfach gewesen ;)

[Michael S. (rbs_phoenix)]

Geht es denn, wenn du LED und LED2 vertauscht? Wenn dann die LED an RA4 
blinkt und die andere nichts macht, kann man schonmal davon ausgehen, 
dass die Konfig der Pins richtig ist. Ggf musst du mal gucken, obs geht, 
wenn der Komparator ausgeschaltet ist, der an RA4 ist.

Zudem: Bei einigen PIC16ern (gerade ältere) ist RA4 ein 
Open-Drain-Output, d.h. man benötigte dort einen Pull-Up-Widerstand. Im 
Datenblatt vom PIC18F2523 konnte ich auf die schnelle dazu nichts 
finden, aber vielleicht ist das bei diesem PIC auch noch so.

[Lena R. (eny)]

Wenn ich vertausche, gehts auch, dann blinkt die andere LED.

Die im Interrupt blinkt dann nicht.

Timer Interrupt funktioniert übrigens.
Nur wird kein Interrupt ausgelöst, auch bei fallender Flanke nicht.
Das Signal ist auch ok, 5V pp Rechteck.

Ich hab das Datenblatt jetzt 3 mal durch und finde kein Bit, was falsch 
gesetzt ist.

Von der Konfiguration her, ist auch nix was mit Interrupts 
zusammenhängt.

Glaube langsam, dass der PIC vielleicht defekt ist. Wobei das eigentlich 
unwarscheinlich ist.

Wenn ich die INTx Pins, als Ausgang belege, bekomm ich auch ein Signal.

[Michael S. (rbs_phoenix)]

Lena R. schrieb:
> Das Signal ist auch ok, 5V pp Rechteck.

Ich hoffe 0V - 5V und nicht -2.5V - 2.5V, die -2.5V wird er nicht mögen.

Ich würde nochmal probieren:

void interrupt isr_high(void) {
   if(INCONbitsINT0F){
       INTCONbits.INT0F = 0;
       LED = 1;
   }
   LED2 = 1;
}

Dazu die while(1) in der Main leer machen. Wenn LED leuchtet, war er 
zumindest einmal dort. Ich schätze aber, dass die nicht leuchten wird.

Es kann auch sein, dass "interrupt" (alleine stehend) die low-priority 
isr-Bezeichnung ist. Also entweder die (hier überflüssige) Prioritäten 
ausschalten oder:

void interrupt high_priority HighPriorityISRCode(){}

[Lena R. (eny)]

0V - 5V.

Leider bringt das alles keinen Erfolg.

void interrupt high_priority isr_high(void) {
   if(INTCON3bits.INT2F){
       INTCON3bits.INT2F = 0;
       LED = 1;
   }
   LED2 = 1;
}

Gleiches für INTCONbits.INT0F.

Soweit ich das verstanden habe, muss man nur low_priority deklarieren.
Aber wie gesagt, auch dein Code bringt keine LED zum leuchten :(

Werd wohl nacher mal den INT1 PIN testen, aber dürfte ja eigentlich kein 
Unterschied machen.

[Peter D. (peda)]

Füg dochmal nen isr_low Handler hinzu.

[Lena R. (eny)]

Leider auch keine Besserung :(

[Peter D. (peda)]

Ich kenn die PICs nicht, aber bei AVR, 8051 gibt es noch ne globale 
Interruptfreigabe.