Выход из процедуры прерывания в PIC16F18877

После выполнения блока прерывания выполнение останавливается и никогда не возвращается из процедуры прерывания в блок while(1) в основном методе.

Чтобы проверить функциональность, я написал приведенный ниже код для проверки прерывания.

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

Сообщение, которое я вижу на светодиоде, называется «Конец метода ISR», и оно никогда не переходит в бесконечность после выполнения ISR.

Я проверил инструкцию RETFIE, но я использую C, а не язык ассемблера. Я использовал обычный оператор «возврата» C в конце процедуры прерывания, но, похоже, выполнение там застревает.

Что я делаю не так? Любые исправления в этом коде, пожалуйста?

подпрограмма isr связана с каким источником прерывания?
Я не уверен, что вы спрашиваете. Должен ли я где-то прикрепить прерывание? Раньше я делал то же самое выше для PIC16F887, и это работало. Не уверен, что мне здесь не хватает. Не могли бы вы помочь подробнее объяснить «присоединение прерывания к источнику»? Спасибо.
хорошо, кто вызывает прерывание ... вот в чем вопрос ... какое периферийное устройство и какой флаг связан с ним
У меня есть кнопочный переключатель (активный низкий уровень), подключенный к RB1. Когда я нажимаю эту кнопку, она попадает в метод isr.
Я использую прерывание при изменении на порту B, поэтому флаги IOCIF (флаг прерывания при изменении) и IOCBF0, IOCBF1 и так далее до IOCBF7 для отдельных контактов порта B. В моем случае я использую RB1, поэтому ответственным флагом является IOCBF1.
Следовательно, в подпрограмме прерывания я проверяю, установлен ли IOCIF и установлен ли IOCBF1, чтобы он запускал код для IOCBF1, что действительно происходит. Он входит в блок прерывания RB1 и отображает это сообщение. Он также отображает сообщение End ISR Method и никогда не возвращается к бесконечности, пока...
Активный низкий уровень ... Хм, значит, он настроен как прерывание по заднему фронту?
Демонстрация проблемы на видео - ссылка на гугл диск: drive.google.com/file/d/1iTlWN2KDI0JNyaBzzmDoC7W0kHoJAjUz/…
где вы инициализировали прерывание? Я думаю, это второе прерывание.
инициализировать(); Я инициализировал прерывание внутри этого метода.
Может ли быть так, что другой источник прерывания вызвал вашу процедуру isr, и вы не сбросили его флаг? Вы могли бы увидеть это, если бы вы вывели какой-нибудь счетчик на дисплей, или вы могли бы попытаться очистить все флаги прерывания и посмотреть, начнет ли он работать.
Я думаю, что проблема либо в операторе возврата, либо в вашей функции showmessage. Вы до сих пор не показали, как вы ее реализовали... Я думаю, вы использовали прерывания таймера внутри этих функций....
Функция showmessage просто отображает текст на ЖК-дисплее, чтобы увидеть, где выполняется код. Даже если я закомментирую функцию showmessage и ничего не будет включено в метод isr, он не выйдет из isr. Я застрял здесь и не знаю, что я делаю неправильно.. :(
Обновлен полный код программы. Пожалуйста, проверьте и посоветуйте, что / где я делаю неправильно.
В техническом описании сказано, что для возврата из isr следует использовать инструкцию RETFIE. Я не пишу программу на ассемблере и пишу на C. Как мне это сделать сейчас?

Ответы (1)

При написании ISR на C вы никогда не должны использовать, returnесли вы не знаете, что делаете. Большинство микроконтроллеров содержат разные инструкции для «возврата из подпрограммы» и «возврата из прерывания».

Если ваш компилятор транслирует returnвозврат из подпрограммы, вы получите именно ту ошибку, которую описываете. Почему вы пишете returnв первую очередь - это voidфункция, возврат не имеет смысла.

Поскольку это PIC, другая вероятная возможность заключается в том, что ваша программа взорвала глубину стека или просто закончилась память стека. Если ваш стек поврежден, счетчик программ прыгнет в никуда после возврата из прерывания. Наблюдайте за указателем стека при пошаговом выполнении с помощью отладчика. Всегда ли SP находится внутри выделенного стека?

Я не согласен с вашим последним утверждением return, которое часто используется в voidфункциях для более раннего выхода или просто для того, чтобы сделать код более читаемым.
@PrateekDhanuka Публикация возврата в конце функции void служит только для того, чтобы сделать код менее читаемым. Дело в том, что его нельзя нигде использовать, если компилятор переводит return как «возврат из подпрограммы».
@Prateek Вы правы в случае c приложений, которые вы пишете для Linux или Windows. Но не здесь.
Да, я понимаю, что возврат - это то, что вызвало здесь проблемы. Я имел в виду только последнюю строку. Я думаю, что «возврат не имеет смысла» не совсем точен.
1. Даже если я уберу оператор return из конца подпрограммы isr, выполнение не произойдет. 2. Сообщение show предназначено только для отображения строки на ЖК-дисплее, чтобы увидеть, что выполняется в данный момент.
3. Надеюсь, вы видели видео по ссылке, которой я поделился. Нажатие кнопки запускает метод isr, и вы можете увидеть «Прерывание RB1» на ЖК-экране. Затем, если вы следуете последовательности выполнения, он ждет 3 секунды, указанных в конце метода show message, и достигает конца isr. Именно здесь он отображает строку «End ISR Method» на ЖК-дисплее. 4. У меня есть строка "Inside While" в бесконечном цикле while. Когда устройство запускается, я вижу это на ЖК-дисплее. В тот момент, когда я нажимаю кнопку, она переходит в isr и никогда не возвращается к выполнению основной программы.
5. Наличие оператора return в isr не имеет значения. Выполнение кода все еще зависает там. Не знаю, где я ошибаюсь.. :(
@VinaySathyanarayana Разберите и посмотрите, какие инструкции по возврату вы получите. Я не знаю ассемблера PIC, но это должно быть довольно понятно. Поскольку это PIC, другая вероятная возможность заключается в том, что ваша программа взорвала глубину стека или просто закончилась память стека. Если ваш стек поврежден, компьютер прыгнет в глушь после возврата из прерывания.
@Lundin - я удалил оператор «возврат» из isr. Даже тогда он застревает внутри isr ​​и никогда не выходит. Я обновил полный код в своем вопросе. Один и тот же код с оператором возврата и без него отлично работает на PIC16F887 в той же IDE и с тем же компилятором..!!!
@VinaySathyanarayana Или это никогда не работало идеально, и у вас всегда было повреждение стека. Всякий раз, когда у вас возникают проблемы с PIC, предполагайте, что это связано со стеком.
Он никогда не работал на PIC16F18877, он всегда работает на PIC16F887. Проверено даже сейчас .. Очень расстроен, чтобы понять, в чем проблема и где я ошибаюсь ..
@VinaySathyanarayana Я не буду говорить вам проверять указатель стека еще раз. Скорее всего, разные ПОС имеют разный размер стека...
Разобрался. Ничего общего со стеком или ПК. Обновлю ветку своим ответом, как только мой анализ будет завершен. Работая над микропроцессорами PIC уже более 10 лет, эта проблема дала мне больше информации. Хорошо, что я этому научился. :) Вы можете проверить проблему и решить видео здесь - drive.google.com/file/d/13V5qDM60Ek-MDTW5cefR5CIgmtKKbrN_/… Спасибо за всю вашу помощь и вклад.
@VinaySathyanarayana Вы можете ответить на свой вопрос здесь, если это точное решение вашего вопроса.