Pages: [1] 2   Go Down
Author Topic: <SOLVED> ATmega644 does not catch the interupt  (Read 2232 times)
0 Members and 1 Guest are viewing this topic.
The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Hi,

I'm trying to use the attachInterrupt() with the ATmega644 for sensing a rotary encoder.
For some reason the uC doesn't catch the interrupt. This is my code

Code:
// Rotary encoder interrupt service routine
boolean rotating=false;
void rotEncoder(){
rotating=true; // If motion is detected in the rotary encoder, set the flag to true
}

void setup(){
        // Attach Interrupts
attachInterrupt(2, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2

pinMode(4, INPUT);  // Button switch or Encoder pin for volume up
pinMode(2, INPUT);  // Button switch or Encoder pin for volume down
pinMode(3, INPUT);  // Button switch or pin for encoder built-in switch

digitalWrite(2, HIGH); // If H/W debouncing is implemented, set to LOW
digitalWrite(4, HIGH); // If H/W debouncing is implemented, set to LOW
digitalWrite(3, HIGH); // Enable pull-up resistor
}

void loop(){

        if (rotating){
            // do something
        }
}


When I take a look at the Sanguino website it states that the ATmega's interrupt pins are pin 2, 10 & 11. According to the site I should use int 2 to attach the interrupt routine to pin 2.
http://sanguino.cc/useit

Perhaps I'm getting it wrong. Some help is needed here  smiley-confuse
« Last Edit: February 09, 2013, 05:30:51 am by corprius » Logged

Global Moderator
Offline Offline
Brattain Member
*****
Karma: 452
Posts: 18694
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Interrupt variables need to be volatile, eg.

Code:
volatile boolean rotating=false;

Also your code doesn't actually do anything on the interrupt so how do you know it fails?

http://www.gammon.com.au/interrupts
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

I did not paste the code, because it is quite large. I know it fails because when I use the exact same code on my arduino Uno the interrupt gets fired. I can see the result on a display and I can hear the volume go up or down, so I know it's working on the Uno. I use it to control a DAC btw

For the Uno I use this line:
Code:
attachInterrupt(0, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2
instead of
Code:
attachInterrupt(2, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2
.

Logged

Left Coast, CA (USA)
Offline Offline
Brattain Member
*****
Karma: 331
Posts: 16459
Measurement changes behavior
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Are you reading digital input pins inside the ISR routines (most quadrature decoder ISR do have to), and you should be adjusting for the different pin mapping of the Sanguino then the Uno ISR routine uses?

What modified IDE version are you using, as the Sanguino folks haven't seemed to have kept up with the arduino IDE versions sense version 18 or so, not that would keep you from still using an older IDE modified to include the Sanguino 644P stuff.

Lefty
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

The have kept up to version 1.01

http://code.google.com/p/sanguino/downloads/list

Logged

Left Coast, CA (USA)
Offline Offline
Brattain Member
*****
Karma: 331
Posts: 16459
Measurement changes behavior
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

The have kept up to version 1.01

http://code.google.com/p/sanguino/downloads/list



Ah, good to see. So could you at least post your two ISR functions?

Lefty
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Of course, but they are they same. Assuming that you mean the ISR function that works and the one that doesn't...

This one works (Arduino Uno)

Code:
// Rotary encoder interrupt service routine
volatile boolean rotating=false;

void rotEncoder(){
rotating=true; // If motion is detected in the rotary encoder, set the flag to true
}
This one does not (ATmega644).
Code:
volatile boolean rotating=false;
void rotEncoder()
{
rotating=true; // If motion is detected in the rotary encoder, set the flag to true
}
« Last Edit: January 26, 2013, 03:15:34 pm by corprius » Logged

Left Coast, CA (USA)
Offline Offline
Brattain Member
*****
Karma: 331
Posts: 16459
Measurement changes behavior
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Code:
static volatile rotating=false;

Missing the variable type for that statement?

Lefty
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Sorry I messed up the code while pasting it. I corrected it. I was hoping that the missing variable type was the problem, but it's not. Messing up the pasted code was clearly a PEBKAC ("Problem Exists Between Keyboard And Chair")  smiley-roll-blue
Logged

Left Coast, CA (USA)
Offline Offline
Brattain Member
*****
Karma: 331
Posts: 16459
Measurement changes behavior
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Sorry I messed up the code while pasting it. I corrected it. I was hoping that the missing variable type was the problem, but it's not. Messing up the pasted code was clearly a PEBKAC ("Problem Exists Between Keyboard And Chair")  smiley-roll-blue

While in the Air Force we would have wrote it up as "found short between the headsets".  smiley-grin

Lefty
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

haha.., there are numerous variations to it.

Could it have something to do with the fuses? Mine differ from sanguino's fuses.



* Fuses.jpg (108.13 KB, 700x769 - viewed 12 times.)
Logged

Global Moderator
Offline Offline
Brattain Member
*****
Karma: 452
Posts: 18694
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

I'm trying to use the attachInterrupt() with the ATmega644 for sensing a rotary encoder.
For some reason the uC doesn't catch the interrupt. This is my code
...
When I take a look at the Sanguino website it states that the ATmega's interrupt pins are pin 2, 10 & 11. According to the site I should use int 2 to attach the interrupt routine to pin 2.

Do you actually have a Sanguino?
Logged

Offline Offline
Edison Member
*
Karma: 116
Posts: 2205
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Code:
attachInterrupt(2, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2

INT0 is typically on PD2.

With that corrected, your code works.
Logged

The Netherlands
Offline Offline
Newbie
*
Karma: 0
Posts: 38
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

I'm trying to use the attachInterrupt() with the ATmega644 for sensing a rotary encoder.
For some reason the uC doesn't catch the interrupt. This is my code
...
When I take a look at the Sanguino website it states that the ATmega's interrupt pins are pin 2, 10 & 11. According to the site I should use int 2 to attach the interrupt routine to pin 2.

Do you actually have a Sanguino?
Sorry, been away for a few days...

No I do not have the Sanguino, I was only inspired by it. I designed my own controller that can be mounted piggyback onto any 20x4 HD44780 based display. I use it to control my DAC. Here's al link to my creation. http://ce-designs.net/index.php/my-projects/other-builds/ce644-the-es9018-i2c-controller

Code:
attachInterrupt(2, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2

INT0 is typically on PD2.

With that corrected, your code works.

So you're saying I should change it to:
Code:
attachInterrupt(0, rotEncoder, CHANGE);  // ISR for rotary encoder on pin 2
???
Already tried that without success.
Logged

Global Moderator
Offline Offline
Brattain Member
*****
Karma: 452
Posts: 18694
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

To save me reading all that, can you please state which pin, on the Atmega644 chip, you are attaching the device which is to generate an interrupt?
Logged

Pages: [1] 2   Go Up
Jump to: