[SOLVED] Interrupt Not Running

joseMiguel said:

Hi,

put that data declaration defore the interrupt handler and after the interrupt handler use a #pragma code directive
I have not verified the bits about the GPIO interrupt.
char count = 0;
char segment[] = {0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F};
Hope, i help you
regards
JoseMiguel

Click to expand...

#pragma code My_HiPrio_Int = 0x08

void My_HiPrio_Int(void){
_asm
GOTO chk_isr
_endasm
}
#pragma code

Click to expand...

qwertyuiop415263 said:

you didn't cofigure the registers completely write these lines before your header file (if you are using external Crytsal oscillator)

#pragma config OSC=HS, OSCS=OFF
#pragma config PWRT=OFF, BOR=ON, BORV=45
#pragma config WDT =OFF
#pragma config DEBUG=OFF,LVP=OFF,STVR=OFF

Also you didn't declare your display_count fuction

Click to expand...

void display_count(void);

Click to expand...

darktangent said:

Hi, I am using the following code, but intterupts do not launch once the RB0 goes high.

Click to expand...

#include <p18f452.h>
#include <delays.h>
#pragma config WDT = OFF

void chk_isr(void); //Interrupt Handler
void inc_count(void);
void dec_count(void);

// Code to Be placed inthe Interrupt Handler
#pragma code My_HiPrio_Int = 0x08

	void My_HiPrio_Int(void){
		_asm
		GOTO chk_isr
		_endasm
	}
#pragma code	//end of interrupt vector table

#pragma interrupt chk_isr // Our Interrupt Service Routine

void chk_isr(void){
	if(INTCONbits.INT0IF == 1) inc_count();
	if(INTCON3bits.INT1IF == 1) dec_count();
	}

void display_count(void);
char count = 0;
char segment[] = {0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F};

	
void main(void){
	TRISC = 0;
	TRISD = 0;
	TRISBbits.RB0 = 1;
	TRISBbits.RB1 = 1;
	INTCONbits.INT0IF = 0;
	INTCONbits.INT0IE = 1;
                [COLOR="red"]INTCON3bits.INT1IF = 0;[/COLOR]
                INTCON3bits.INT1IE = 1;
	[COLOR="red"]INTCONbits.PEIE = 1;[/COLOR]
                INTCONbits.GIE = 1;

	while(1){
		display_count();
		
	}
	
}

void display_count(void){
	char MSB;
	char LSB;
                [COLOR="red"]INTCONbits.GIE = 0;[/COLOR]
                MSB = count / 10;
	LSB = count % 10;
                [COLOR="red"]INTCONbits.GIE = 1;[/COLOR]

	PORTC = segment[MSB];
	PORTD = segment[LSB];
	
	
}

void inc_count(void){
	count++;
	INTCONbits.INT0IF == 0;
}

void dec_count(void){
	count--;
	INTCON3bits.INT1IF == 0;
}

qwertyuiop415263 said:

you didn't cofigure the registers completely write these lines before your header file (if you are using external Crytsal oscillator)

#pragma config OSC=HS, OSCS=OFF
#pragma config PWRT=OFF, BOR=ON, BORV=45
#pragma config WDT =OFF
#pragma config DEBUG=OFF,LVP=OFF,STVR=OFF

Also you didn't declare your display_count fuction

Click to expand...

zuisti said:

Hi darktangent;
I don't know this proc nor this compiler exactly but ... try out this (look at the red lines):

Code:

#include <p18f452.h>
#include <delays.h>
#pragma config WDT = OFF

void chk_isr(void); //Interrupt Handler
void inc_count(void);
void dec_count(void);

// Code to Be placed inthe Interrupt Handler
#pragma code My_HiPrio_Int = 0x08

	void My_HiPrio_Int(void){
		_asm
		GOTO chk_isr
		_endasm
	}
#pragma code	//end of interrupt vector table

#pragma interrupt chk_isr // Our Interrupt Service Routine

void chk_isr(void){
	if(INTCONbits.INT0IF == 1) inc_count();
	if(INTCON3bits.INT1IF == 1) dec_count();
	}

void display_count(void);
char count = 0;
char segment[] = {0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F};

	
void main(void){
	TRISC = 0;
	TRISD = 0;
	TRISBbits.RB0 = 1;
	TRISBbits.RB1 = 1;
	INTCONbits.INT0IF = 0;
	INTCONbits.INT0IE = 1;
                [COLOR="red"]INTCON3bits.INT1IF = 0;[/COLOR]
                INTCON3bits.INT1IE = 1;
	[COLOR="red"]INTCONbits.PEIE = 1;[/COLOR]
                INTCONbits.GIE = 1;

	while(1){
		display_count();
		
	}
	
}

void display_count(void){
	char MSB;
	char LSB;
                [COLOR="red"]INTCONbits.GIE = 0;[/COLOR]
                MSB = count / 10;
	LSB = count % 10;
                [COLOR="red"]INTCONbits.GIE = 1;[/COLOR]

	PORTC = segment[MSB];
	PORTD = segment[LSB];
	
	
}

void inc_count(void){
	count++;
	INTCONbits.INT0IF == 0;
}

void dec_count(void){
	count--;
	INTCON3bits.INT1IF == 0;
}

Click to expand...

qwertyuiop415263 said:

you didn't initialize the Flag of interreupt RB1 to 0 in your main function. write (INTCON3bits.INT1IF == 0) in main fuction.

another thing you only initialize the Interuupt pins to input using tris, but you didnt write any default value to port such as
PORTBbits.RB0=1;
PORTBbits.RB1=1;
or
PORTBbits.RB0=0;
PORTBbits.RB1=0;

so that when interrupt comes from external harware (may be from push button) then RB0, RB1 changes its state. you need pullup and pulldown resistors for that.(depending upon whether you set RB0 and RB1 pin as 0 or 1).

Click to expand...