Salve,
come da titolo ho un problema con un bottone che non esegue l'istruzione che si trova all'interno dell'interrupt ad alta
priorità, l'interrupt a bassa priorità è gestito da un timer ed è funzionante, vorrei capire dov'è l'errore, inoltre sempre
nell'interrupt a bassa priorità ho un secondo problema, far visualizzare il valore zero sull'LCD, così come ho scritto il codice
che vi posto, lo zero viene visualizzato ma è tremolante, giustamente...
Il programma serve a visualizzare su un display i valori letti da un encoder:
Code:
#include <xc.h>
#include "LTlib.h"
#include "module_UART.h"
#include "module_UART.c"
#include "LTlib_utilities.h"
#include "LTlib_utilities.c"
#include "module_IO.h"
#include "module_IO.c"
#include "module_I2C.h"
#include "module_I2C.c"
#include "LTlib_delay.h"
#include "LTlib_delay.c"
#include "PCF8574.h"
#include "PCF8574.c"
#define PCF8574
#include "LCD_44780_I2C.h"
#include "LCD_44780_I2C.c"
#define Encoder_SW PORTBbits.RB3 //Reset
#define EncoderDT PORTBbits.RB5 //Encoder_DT
#define EncoderCLK PORTBbits.RB4 //Encoder_CLK
#define SPIKES_DELAY 1000
int NewPos=0;
int OldPos=0;
signed int count=0;
//char zero[]={'0','\0'};
unsigned int i;
//*************************************************
// Gestione Interrupt
//*************************************************
__interrupt (high_priority) void ISR_alta (void) {
if (INTCONbits.RBIF == 1 ) {
INTCONbits.RBIF = 0;
if(Encoder_SW == 0){
Reset();
}
}
}
__interrupt (low_priority) void ISR_bassa (void) {
// Controllo che l'interrupt sia stato generato dal Timer0
if (INTCONbits.TMR0IF == 1){
// Resetto il flag d'interrupt
INTCONbits.TMR0IF = 0;
NewPos = EncoderDT;
if(NewPos != OldPos){
if(EncoderCLK != NewPos){
count++;
}
else{
count--;
}
}
if(count == 0){
LCD_goto_xy(14,1);
LCD_write_integer(count,0,LCD_ZERO_CLEANING_OFF);
}
LCD_goto_xy(11,1);
LCD_write_integer(count,4,LCD_ZERO_CLEANING_ON);
}
UART1_write_byte(count);
OldPos = NewPos;
}
void main(void) {
// Setto le porte in input
IO_set_all_ports_as_inputs();
//Inizializzo il display
LCD_initialize (20);
//Accendo retroilluminazione LCD
LCD_backlight (LCD_TURN_ON_LED);
//Scrivo su LCD
LCD_write_message("Contatore:");
LCD_goto_xy(1,2);
LCD_write_message("Premi RESET x 0");
LCD_goto_xy(15,1);
LCD_write_char(223);
delay_ms(10);
//Avvio Seriale
UART1_open(UART_BAUDRATE_19200);
//Abilito PULL-UP
INTCON2bits.RBPU = 0x00;
// Abilito le interruzioni su PORTB
INTCONbits.RBIE = 1;
// Abilito le interruzioni su PORTB come alta priorità
INTCON2bits.RBIP = 1;
//************************************************************
// Abilito il Timer0 per funzionare a bassa priorità
//************************************************************
// Modalità a 8 bit
T0CONbits.T08BIT = 1;
// Clock interno
T0CONbits.T0CS = 0;
// Abilito Prescaler
T0CONbits.PSA = 0;
// Prescaler 256
T0CONbits.T0PS0 = 1;
T0CONbits.T0PS1 = 1;
T0CONbits.T0PS2 = 1;
// Abilito le interruzioni del Timer0
INTCONbits.TMR0IE = 1;
// Abilito le interruzioni del Timer0 come bassa priorità
INTCON2bits.TMR0IP = 0;
// Abilito il Timer0
T0CONbits.TMR0ON = 1;
// Abilito modalità interruzione a due livelli
RCONbits.IPEN = 1;
// Abilito l'interrupt ad alta priorità
INTCONbits.GIEH = 1;
// Abilito gli interrupt a bassa priorità
INTCONbits.GIEL = 1 ;
// Resetto il flag d'interrupt prima di attivare le interruzioni
INTCONbits.RBIF = 0;
// Abilito l'interrupt periferiche
INTCONbits.PEIE = 1 ;
OldPos = EncoderDT;
while(1){
}
}
da premettere che sono un autodidatta neofita e il codice potrebbe essere sicuramente migliorato quindi si accettano consigli
anche su questo.
Grazie
Autore della discussione
