ext. Interrupt löst nicht aus

[Dominique G. (dgoersch)]

Hallo zusammen,

ich habe das Problem, dass die externen Interrupts auf einem Atmega16 
nicht auslösen. Meiner Meinung nach sollte Softwareseitig soweit alles 
stimmen. An den Pins hängen Taster nach GND.

void init(void)
{
  DDRA = 0xff;    //PortA
  DDRB = 0xff;    //PortB
  DDRC = 0xff;    //PortC as Output
  DDRD = 0x00;    //PortD as Input
  PORTD = 0xff;    //PullUps on PortD

    TCCR1B = 1;               // Timer1 runs full system clock
    TIMSK |= (1<<OCIE1A);     // enable Timer1

    MCUCR |= (1<<ISC01);    //INT0 on falling edge
    MCUCR |= (1<<ISC11);    //INT1 on falling edge
    GICR &= (1<<INT0);      //enable INT0
    GICR &= (1<<INT1);      //enable INT1

    sei();
}

ISR(INT0_vect)
{
  pwm_setting[0] = 0xff;
}

ISR(INT1_vect)
{
  pwm_setting[0] = 0x00;
}

Die PWM an sich funktioniert, deswegen hier nur die relevanten Auszüge 
aus dem Code. Die Main-Loop ist zum Testen komplett leer. In der ISR 
soll testweise eine LED an- bzw. ausgeschaltet werden.

Jemand eine Idee, woran es hängt?

Danke und Gruß
Dominique Görsch

[holger (Gast)]

> Jemand eine Idee, woran es hängt?

Ja, das ist falsch

   GICR &= (1<<INT0);      //enable INT0
   GICR &= (1<<INT1);      //enable INT1

[Tritt-Ihn (Gast)]

Eine Kleinigkeit noch: Ist die Variable "pwm_setting" als volatile 
deklariert?

[Dominique G. (dgoersch)]

holger schrieb:
> Ja, das ist falsch
>
>    GICR &= (1<<INT0);      //enable INT0
>    GICR &= (1<<INT1);      //enable INT1

Verdammt, ich hab bestimmt 100mal über den Code geschaut, aber das immer 
übersehen. Danke, mit 'oder' klappts natürlich.

Tritt-Ihn schrieb:
> Eine Kleinigkeit noch: Ist die Variable "pwm_setting" als volatile
> deklariert?

Ja, das ist sie:

volatile uint8_t pwm_setting[24];

Danke an die Hilfe, x*2 Augen sehen eben doch mehr als 4 (jemand anderes 
hatte auch schon über den Code geschaut).