Verlassen der Interrupt-Routine in PIC16F18877

Nach der Ausführung des Unterbrechungsblocks hält die Ausführung an und kehrt nie aus der Unterbrechungsroutine zum Block while(1) in der Hauptmethode zurück.

Um die Funktionalität zu überprüfen, habe ich den folgenden Code geschrieben, um den Interrupt zu testen.

 /*
 * File:   stepper.c
 * Author: vsathyan
 *
 * Created on August 30, 2019, 11:04 AM
 */

// PIC16F18877 Configuration Bit Settings

// 'C' source line config statements

// CONFIG1
#pragma config FEXTOSC = HS     // External Oscillator mode selection bits (HS (crystal oscillator) above 4MHz; PFM set to high power)
#pragma config RSTOSC = EXT1X   // Power-up default value for COSC bits (EXTOSC operating per FEXTOSC bits)
#pragma config CLKOUTEN = OFF   // Clock Out Enable bit (CLKOUT function is disabled; i/o or oscillator function on OSC2)
#pragma config CSWEN = ON       // Clock Switch Enable bit (Writing to NOSC and NDIV is allowed)
#pragma config FCMEN = ON       // Fail-Safe Clock Monitor Enable bit (FSCM timer enabled)

// CONFIG2
#pragma config MCLRE = ON       // Master Clear Enable bit (MCLR pin is Master Clear function)
#pragma config PWRTE = OFF      // Power-up Timer Enable bit (PWRT disabled)
#pragma config LPBOREN = OFF    // Low-Power BOR enable bit (ULPBOR disabled)
#pragma config BOREN = OFF      // Brown-out reset enable bits (Brown-out reset disabled)
#pragma config BORV = LO        // Brown-out Reset Voltage Selection (Brown-out Reset Voltage (VBOR) set to 1.9V on LF, and 2.45V on F Devices)
#pragma config ZCD = OFF        // Zero-cross detect disable (Zero-cross detect circuit is disabled at POR.)
#pragma config PPS1WAY = ON     // Peripheral Pin Select one-way control (The PPSLOCK bit can be cleared and set only once in software)
#pragma config STVREN = OFF     // Stack Overflow/Underflow Reset Enable bit (Stack Overflow or Underflow will not cause a reset)

// CONFIG3
#pragma config WDTCPS = WDTCPS_31// WDT Period Select bits (Divider ratio 1:65536; software control of WDTPS)
#pragma config WDTE = OFF       // WDT operating mode (WDT Disabled, SWDTEN is ignored)
#pragma config WDTCWS = WDTCWS_7// WDT Window Select bits (window always open (100%); software control; keyed access not required)
#pragma config WDTCCS = SC      // WDT input clock selector (Software Control)

// CONFIG4
#pragma config WRT = OFF        // UserNVM self-write protection bits (Write protection off)
#pragma config SCANE = not_available// Scanner Enable bit (Scanner module is not available for use)
#pragma config LVP = ON         // Low Voltage Programming Enable bit (Low Voltage programming enabled. MCLR/Vpp pin function is MCLR.)

// CONFIG5
#pragma config CP = ON          // UserNVM Program memory code protection bit (Program Memory code protection enabled)
#pragma config CPD = ON         // DataNVM code protection bit (Data EEPROM code protection enabled)

// #pragma config statements should precede project file includes.
// Use project enums instead of #define for ON and OFF.

#define _XTAL_FREQ 4000000

//LCD DISPLAY INTERFACE - PORTD & PORTE
#define RS RD2
#define EN RD3
#define D4 RC4
#define D5 RC3
#define D6 RD0
#define D7 RD1

#define TRIS_RS TRISD2
#define TRIS_EN TRISD3
#define TRIS_D4 TRISC4
#define TRIS_D5 TRISC3
#define TRIS_D6 TRISD0
#define TRIS_D7 TRISD1

#define BUZZER RC1
#define TRIS_BUZZER TRISC1

#define OUTPUT 0
#define INPUT 1

#define PRESSED 1
#define RELEASED 0
#define _PRESSED 0
#define _RELEASED 1

#define SET 1
#define CLEAR 0

#define HIGH 1
#define LOW 0

#define UP 1
#define DOWN 0

//INPUT CONFIGURATION
#define EMERGENCY_BUTTON RB0
#define MENU_OPTION RB1
#define VALUE_UP RB2
#define VALUE_DOWN RB3
#define SET_VALUE RB4
#define HOME_LIMIT_SWITCH RB5
#define END_LIMIT_SWITCH RB6

#define TRIS_EMERGENCY_BUTTON TRISB0
#define TRIS_MENU_OPTION TRISB1
#define TRIS_VALUE_UP TRISB2
#define TRIS_VALUE_DOWN TRISB3
#define TRIS_SET_VALUE TRISB4
#define TRIS_HOME_LIMIT_SWITCH TRISB5
#define TRIS_END_LIMIT_SWITCH TRISB6

//OUTPUT CONFIGURATION
#define PULSE RA0
#define DIRECTION RA1
#define ENABLE RA2

#define TRIS_PULSE TRISA0
#define TRIS_DIRECTION TRISA1
#define TRIS_ENABLE TRISA2

//function declarations
void showMessage(char *message1, char *message2);
void initializeVariables();

//program variables
char fValue[16];
int option;

#include <xc.h>
#include <stdio.h>
#include <string.h>
#include "stepper.h"
#include "lcd162.h"


//function declarations
void showMessage(char *message1, char *message2);
void showMessageWithDelay(char *message1, char *message2, int delay);
void initialize();
void initializeVariables();
void setStringValue(char stringValue, float floatValue);
void delay_ms(int delay);
void delay_us(int delay);

void interrupt isr()
{
    if(IOCIF) //if interrupt on change flag is set
    {
        if(IOCBF1) //if interrupt on port RB1 is set
        {
            showMessageWithDelay(" RB1 Interrupt  ", "                ", 3000); //display a message in LCD for 3 seconds
            IOCBF1 = 0;
        }
        IOCIF = 0;
    }
    showMessageWithDelay(" End ISR Method ", "                ", 3000); //display a message in LCD for 3 seconds
    return; //return to main program execution
}

void main(void) 
{
    initialize(); //initialize input pins, output pins, variables
    Lcd_Start(); //initialize 16x2 LCD
    Lcd_Clear(); //clear LCD;
    showMessageWithDelay("Automatic Speed ", "Controller V0.1 ", 3000); //display start message
    int i = 0;
    while(1)
    {
        i = i + 1;
        setStringValue(fValue, i);
        showMessageWithDelay("  Inside While  ", fValue, 1000); //message displayed when inside infinite while loop
    }
}
void showMessage(char *message1, char *message2)
{
    Lcd_Clear();
    Lcd_Set_Cursor(1,1);
    Lcd_Print_String(message1);
    Lcd_Set_Cursor(2,1);
    Lcd_Print_String(message2);
}
void showMessageWithDelay(char *message1, char *message2, int delay)
{
    Lcd_Clear();
    Lcd_Set_Cursor(1,1);
    Lcd_Print_String(message1);
    Lcd_Set_Cursor(2,1);
    Lcd_Print_String(message2);
    delay_ms(delay);
}
void initialize()
{
    ANSELA = 0x00;
    ANSELB = 0x00; //disable analog input on port B
    ANSELC = 0x00;
    ANSELD = 0x00;
    ANSE0 = 0;
    ANSE1 = 0;
    ANSE2 = 0;

    WPUB = 0xFF; //enable weak pull up resistors on all port B pins
    TRISB = 0xFF; //all port B pins are inputs
    IOCBP = 0xFF; //interrupt on change positive trigger enabled on port B;
    IOCBN = 0xFF; //interrupt on change negative trigger enabled on port B;
    IOCBF = 0x00; //clear RB1 interrupt flag

    TRISA = 0x00; //set PORTA as output port
    PORTA = 0x00; //set all PORTA bit values to 0

    TRIS_RS = OUTPUT;
    TRIS_EN = OUTPUT;
    TRIS_D4 = OUTPUT;
    TRIS_D5 = OUTPUT;
    TRIS_D6 = OUTPUT;
    TRIS_D7 = OUTPUT;

    TRIS_BUZZER = OUTPUT;

    TRIS_PULSE = OUTPUT;
    TRIS_DIRECTION = OUTPUT;
    TRIS_ENABLE = OUTPUT;

    PULSE = 0;
    ENABLE = 0;
    DIRECTION = 0;

    IOCIE = 1; //enable INTERRUPT ON CHANGE 
    PEIE = 1; //enable PERIPHERAL INTERRUPT
    GIE = 1; //enable GLOBAL INTERRUPT
    initializeVariables();

}
void initializeVariables()
{
    option = 0;
}
void setStringValue(char stringValue, float floatValue)
{
    sprintf(stringValue, "%.3f", floatValue);
}
void delay_ms(int delay)
{
    for(int i = 0; i < delay; i++)
    {
        __delay_ms(1);
    }
}
void delay_us(int delay)
{
    for(int i = 0; i < delay; i++)
    {
        __delay_us(1);
    }
}

Die Meldung, die ich auf der LED sehe, lautet "ISR-Methode beenden" und gelangt nach dem Ausführen der ISR nie ins Unendliche.

Ich habe nach RETFIE-Anweisungen gesucht, aber ich verwende C und keine Assemblersprache. Ich habe die normale C-Anweisung "return" am Ende der Interrupt-Routine verwendet, aber anscheinend bleibt die Ausführung dort hängen.

Was mache ich falsch? Irgendwelche Korrekturen an diesem Code bitte?

die routine isr ist mit welcher unterbrechungsquelle verbunden?
Ich bin mir nicht sicher, was Sie fragen. Soll ich irgendwo einen Interrupt anhängen? Ich habe das gleiche oben für PIC16F887 gemacht, und es würde funktionieren. Nicht sicher, was ich hier vermisse. Könnten Sie bitte helfen, mehr über "Anhängen eines Interrupts an die Quelle" zu erklären? Danke.
Nun, wer verursacht den Interrupt ... das ist die Frage ... welches Peripheriegerät und welches Flag damit verbunden ist
Ich habe einen Druckknopfschalter (aktiv niedrig) an RB1 angeschlossen. Wenn ich diese Taste drücke, gelangt es in die isr-Methode.
Ich verwende Interrupt on Change an Port B, also sind die Flags IOCIF (Interrupt on Change Flag) und IOCBF0, IOCBF1 usw. bis IOCBF7 für einzelne Pins von Port B. In meinem Fall verwende ich RB1, also ist das verantwortliche Flag IOCBF1.
Daher überprüfe ich in der Interrupt-Routine, ob IOCIF gesetzt ist und IOCBF1 gesetzt ist, sodass der Code für IOCBF1 ausgelöst wird, was tatsächlich geschieht. Es tritt in den RB1-Interrupt-Block ein und zeigt diese Nachricht an. Es zeigt auch die Meldung End ISR Method an und kehrt nie zum Unendlichen zurück, während ...
Aktiv niedrig ... Hm, also ist es dann als Interrupt mit fallender Flanke konfiguriert?
Demonstration des Problems im Video - Google Drive-Link: drive.google.com/file/d/1iTlWN2KDI0JNyaBzzmDoC7W0kHoJAjUz/…
Wo haben Sie den Interrupt initialisiert? Ich vermute, dass ein zweiter Interrupt im Gange ist.
initialisieren(); Ich habe den Interrupt innerhalb dieser Methode initialisiert.
Könnte es sein, dass eine andere Interrupt-Quelle Ihre isr-Routine ausgelöst hat und Sie ihr Flag nicht löschen? Sie könnten es sehen, wenn Sie einen Zähler auf dem Display anzeigen, oder Sie könnten versuchen, alle Interrupt-Flags zu löschen und zu sehen, ob es zu arbeiten beginnt.
Ich denke, das Problem liegt entweder in der Return-Anweisung oder in Ihrer Showmessage-Funktion. Sie haben immer noch nicht gezeigt, wie Sie es implementiert haben ... Ich denke, Sie haben Timer-Interrupts in dieser Funktion verwendet ....
Die Showmessage-Funktion zeigt einfach den Text auf dem LCD-Display an, um zu sehen, wo der Code ausgeführt wird. Selbst wenn ich die showmessage-Funktion auskommentiere und nichts in der isr-Methode enthalten ist, wird die isr nicht beendet. Ich stecke hier fest und bin mir nicht sicher, was ich falsch mache.. :(
Kompletter Programmcode aktualisiert. Bitte überprüfen Sie und geben Sie an, was / wo ich falsch mache.
Das Datenblatt besagt, dass die Anweisung "RETFIE" verwendet werden soll, um von isr zurückzukehren. Ich schreibe kein Assembler-Programm und ist C. Wie mache ich das jetzt?

Antworten (1)

Wenn Sie eine ISR in C schreiben, sollten Sie niemals verwenden return, es sei denn, Sie wissen, was Sie tun. Die meisten MCUs enthalten unterschiedliche Anweisungen für "Rückkehr von Unterprogramm" und "Rückkehr von Unterbrechung".

Wenn Ihr Compiler übersetzt, returnum von einer Subroutine zurückzukehren, würden Sie genau die Art von Fehler erhalten, die Sie beschreiben. Warum schreibst du returnüberhaupt - es ist eine voidFunktion, Return macht keinen Sinn.

Da dies ein PIC ist, besteht eine andere wahrscheinliche Möglichkeit darin, dass Ihr Programm die Stack-Tiefe gesprengt hat oder einfach nicht mehr genügend Stack-Speicher hat. Wenn Ihr Stack beschädigt ist, springt der Programmzähler bei der Rückkehr aus dem Interrupt ins Nirgendwo. Beobachten Sie den Stapelzeiger beim Einzelschrittbetrieb mit einem Debugger. Befindet sich der SP zu jeder Zeit im zugewiesenen Stack?

Ich stimme Ihrer letzten Aussage nicht zu, returndie häufig in voidFunktionen verwendet wird, um früher zu beenden oder den Code einfach lesbarer zu machen.
@PrateekDhanuka Das Posting return am Ende einer void-Funktion dient nur dazu, den Code weniger lesbar zu machen. Der Punkt hier ist, dass Sie es nirgendwo verwenden können, wenn der Compiler return in "return from subroutine" übersetzt.
@Prateek Sie haben Recht, wenn Sie C-Anwendungen für Linux oder Windows schreiben. Aber nicht hier.
Ja, ich verstehe, dass die Rückgabe hier Probleme verursacht hat. Ich bezog mich nur auf die letzte Zeile. Ich denke, "Rückgabe macht keinen Sinn" ist nicht richtig.
1. Selbst wenn ich die return-Anweisung vom Ende der isr-Subroutine entferne, kommt die Ausführung nicht heraus. 2. Die Show-Nachricht dient nur dazu, eine Zeichenfolge auf dem LCD anzuzeigen, um zu sehen, was gerade ausgeführt wird.
3. Ich hoffe, Sie haben das Video in dem von mir geteilten Link gesehen. Durch Drücken der Drucktaste wird die isr-Methode ausgelöst, und Sie können „RB1 Interrupt“ auf dem LCD-Bildschirm sehen. Wenn Sie dann der Ausführungssequenz folgen, wartet es dort für 3 Sekunden, die am Ende der Show-Message-Methode angegeben sind, und erreicht das Ende des isr. Hier wird die Zeichenfolge „ISR-Methode beenden“ auf dem LCD angezeigt. 4. Ich habe einen String „Inside While“ in der Endlosschleife. Wenn das Gerät gestartet wird, sehe ich dies im LCD. In dem Moment, in dem ich die Taste drücke, geht es zu isr und kehrt nie zur Ausführung des Hauptprogramms zurück.
5. Ob die Rückgabeerklärung im isr vorhanden ist oder nicht, spielt keine Rolle. Die Codeausführung bleibt dort immer noch hängen. Ich bin mir nicht sicher, wo ich falsch liege.. :(
@VinaySathyanarayana Demontieren Sie und sehen Sie, welche Rücksendeanweisungen Sie erhalten. Ich kenne PIC-Assembler nicht, aber es sollte ziemlich selbsterklärend sein. Da dies ein PIC ist, besteht eine andere wahrscheinliche Möglichkeit darin, dass Ihr Programm die Stack-Tiefe gesprengt hat oder einfach nicht mehr genügend Stack-Speicher hat. Wenn Ihr Stack beschädigt ist, springt der PC nach der Rückkehr aus dem Interrupt mitten ins Nirgendwo.
@Lundin - Ich habe die "Rückgabe" -Anweisung aus dem ISR entfernt. Selbst dann bleibt es im isr stecken und kommt nie heraus. Ich habe den vollständigen Code in meiner Frage aktualisiert. Derselbe Code, mit und ohne return-Anweisung, funktioniert einwandfrei auf PIC16F887 in derselben IDE und demselben Compiler..!!!
@VinaySathyanarayana Oder es hat nie perfekt funktioniert und Sie hatten immer eine Stack-Korruption. Wenn Sie Probleme mit einem PIC haben, gehen Sie davon aus, dass es mit dem Stack zusammenhängt.
Es hat nie auf PIC16F18877 funktioniert, es funktioniert immer auf PIC16F887. Selbst jetzt getestet. Viel frustriert, um herauszufinden, wo das Problem liegt und wo ich falsch liege.
@VinaySathyanarayana Ich werde Ihnen nicht sagen, dass Sie den Stapelzeiger noch ein weiteres Mal überprüfen sollen. Höchstwahrscheinlich haben die verschiedenen PICs unterschiedliche Stapelgrößen ...
Herausgefunden. Nichts mit Stack oder PC zu tun. Ich werde den Thread mit meiner Antwort aktualisieren, sobald meine Analyse abgeschlossen ist. Ich arbeite seit mehr als 10 Jahren an PIC-Mikros und diese Ausgabe hat mir weitere Einblicke gegeben. Gut, dass ich es gelernt habe. :) Sie können sich das Problem und die gelösten Videos hier ansehen - drive.google.com/file/d/13V5qDM60Ek-MDTW5cefR5CIgmtKKbrN_/… Vielen Dank für all Ihre bisherige Hilfe und Beiträge.
@VinaySathyanarayana Sie können hier Ihre eigene Frage beantworten, wenn dies die genaue Lösung für Ihre Frage ist.