Saliendo de la rutina de interrupción en PIC16F18877

Después de ejecutar el bloque de interrupción, la ejecución se detiene y nunca regresa de la rutina de interrupción al bloque while(1) en el método principal.

Para comprobar la funcionalidad, he escrito el siguiente código para probar la interrupción.

 /*
 * 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);
    }
}

El mensaje que veo en el LED es "Terminar método ISR" y nunca llega al infinito después de ejecutar el ISR.

Revisé las instrucciones RETFIE, pero estoy usando C y no lenguaje ensamblador. He usado la declaración normal de "retorno" de C al final de la rutina de interrupción, pero parece que la ejecución se atasca allí.

¿Qué estoy haciendo mal? ¿Alguna corrección a este código, por favor?

¿Con qué fuente de interrupción se adjunta la rutina isr?
No estoy seguro de lo que estás preguntando. ¿Debo adjuntar una interrupción en algún lugar? Solía ​​hacer lo mismo arriba para PIC16F887, y funcionaba. No estoy seguro de lo que me estoy perdiendo aquí. ¿Podría ayudar a explicar más sobre "adjuntar una interrupción a la fuente"? gracias.
bueno, quién causa la interrupción ... esa es la pregunta ... qué periférico y cuál es la bandera asociada con él
Tengo un interruptor de botón (activo bajo) conectado a RB1. Cuando presiono este botón, ingresa al método isr.
Estoy usando la interrupción en el cambio en el puerto B, por lo que las banderas son IOCIF (bandera de interrupción en el cambio) e IOCBF0, IOCBF1 y así sucesivamente hasta IOCBF7 para los pines individuales del puerto B. En mi caso, estoy usando RB1, por lo que la bandera responsable es IOCBF1.
Por lo tanto, en la rutina de interrupción, verifico si IOCIF está configurado e IOCBF1 está configurado, de modo que active el código para IOCBF1, lo que de hecho está sucediendo. Entra en el bloque de interrupción RB1 y muestra ese mensaje. También muestra el mensaje Finalizar método ISR y nunca vuelve al infinito mientras...
Activo bajo... Hm entonces, ¿entonces está configurado como una interrupción de borde descendente?
Demostración del problema en video: enlace de Google Drive: drive.google.com/file/d/1iTlWN2KDI0JNyaBzzmDoC7W0kHoJAjUz/…
¿Dónde inicializaste la interrupción? Supongo que hay una segunda interrupción en curso.
inicializar(); He inicializado la interrupción dentro de este método.
¿Podría ser que otra fuente de interrupción activó su rutina isr y no borra su bandera? Puede verlo si muestra algún contador en la pantalla, o puede intentar borrar todos los indicadores de interrupción y ver si comienza a funcionar.
Creo que el problema está en la declaración de devolución o en la función showmessage... Todavía no has mostrado cómo lo implementaste... Supongo que has usado interrupciones de temporizador dentro de esas funciones...
La función mostrar mensaje es simplemente mostrar el texto en la pantalla LCD para ver dónde se está ejecutando el código. Incluso si comento la función showmessage y no se incluye nada en el método isr, no sale del isr. Estoy atrapado aquí y no estoy seguro de lo que estoy haciendo mal... :(
Código de programa completo actualizado. Por favor verifique y aconseje qué/dónde estoy haciendo mal.
La hoja de datos dice que use la instrucción "RETFIE" para regresar de isr. No estoy escribiendo un programa ensamblador y es C. ¿Cómo hago esto ahora?

Respuestas (1)

Al escribir una ISR en C, nunca debe usar returna menos que sepa lo que está haciendo. La mayoría de las MCU mantienen instrucciones diferentes para "regresar de la subrutina" y "regresar de la interrupción".

Si su compilador se traduce returnpara regresar de la subrutina, obtendrá exactamente el tipo de error que describe. ¿Por qué escribe returnen primer lugar? Es una voidfunción, el retorno no tiene sentido.

Dado que se trata de un PIC, otra posibilidad probable es que su programa haya superado la profundidad de la pila o simplemente se haya quedado sin memoria de pila. Si su pila está dañada, el contador del programa saltará al medio de la nada al regresar de la interrupción. Observe el puntero de la pila mientras ejecuta un solo paso con un depurador. ¿Está el SP dentro de la pila asignada en todo momento?

No estoy de acuerdo con su última declaración, returna menudo se usa en voidfunciones para salir antes o simplemente para hacer que el código sea más legible.
@PrateekDhanuka Publicar el retorno al final de una función nula solo sirve para hacer que el código sea menos legible. El punto aquí es que no puede usarlo en ninguna parte, si el compilador traduce volver a "regresar de la subrutina".
@Prateek Tiene razón en el caso de las aplicaciones c que escribe para Linux o Windows. Pero no aquí.
Sí, entiendo que la devolución es lo que causó problemas aquí. Me refería a la última línea. Creo que 'devolver no tiene sentido' no es exacto.
1. Incluso si elimino la declaración de devolución del final de la subrutina isr, la ejecución no sale. 2. El mensaje de mostrar es solo para mostrar una cadena en la pantalla LCD para ver lo que se está ejecutando actualmente.
3. Espero que hayas visto el video en el enlace que compartí. Al presionar el botón, se activa el método isr y puede ver "Interrupción RB1" en la pantalla LCD. Luego, si sigue la secuencia de ejecución, espera allí durante 3 segundos al final del método show message y llega al final del isr. Aquí es donde muestra la cadena "Finalizar método ISR" en la pantalla LCD. 4. Tengo una cadena "Inside While" en el ciclo while infinito. Cuando se inicia el dispositivo, veo esto en la pantalla LCD. En el momento en que presiono el botón, va a isr y nunca vuelve a la ejecución del programa principal.
5. Si tener la declaración de devolución o no en el isr no hace ninguna diferencia. La ejecución del código todavía se atasca allí. No estoy seguro de dónde me estoy equivocando .. :(
@VinaySathyanarayana Desensamblar y ver qué instrucción de retorno recibe. No conozco el ensamblador PIC, pero debería explicarse por sí mismo. Dado que se trata de un PIC, otra posibilidad probable es que su programa haya superado la profundidad de la pila o simplemente se haya quedado sin memoria de pila. Si su pila está dañada, la PC saltará al medio de la nada al regresar de la interrupción.
@Lundin: eliminé la declaración de "retorno" del isr. Incluso entonces, se atasca dentro del isr y nunca sale. He actualizado el código completo en mi pregunta. El mismo código, con y sin declaración de retorno, funciona perfectamente bien en PIC16F887 en el mismo IDE y el mismo compilador...!!!
@VinaySathyanarayana O nunca funcionó a la perfección y siempre tuviste una corrupción de pila. Siempre que tenga problemas con un PIC, asuma que está relacionado con la pila.
Nunca funcionó en PIC16F18877, siempre funciona en PIC16F887. Probado incluso ahora ... Mucho frustrado por saber dónde está el problema y dónde me estoy equivocando ...
@VinaySathyanarayana No voy a decirle que verifique el puntero de la pila en otro momento. Lo más probable es que los diferentes PIC tengan un tamaño de pila diferente...
descubierto Nada que ver con pila o PC. Actualizaré el hilo con mi respuesta una vez que mi análisis esté completo. He estado trabajando en micros PIC durante más de 10 años, este problema me dio más información. que bueno que lo aprendi. :) Puede ver el problema y los videos resueltos aquí: drive.google.com/file/d/13V5qDM60Ek-MDTW5cefR5CIgmtKKbrN_/… Gracias por toda su ayuda y aportes hasta ahora.
@VinaySathyanarayana Puede responder su propia pregunta aquí, si esa es la solución exacta a su pregunta.