Pin Change Interrupt - Controller hängt sich auf ab 40kHz - ATMega168

[Joshi (Gast)]

Hallo,

ich habe das Problem dass sich der Controller ab einer Frequenz von 
ca.40kHz am Eingangspin aufhängt. Kann mir einer den Grund dafür 
erklären?

Vielen Dank

[Thomas B. (nichtessbar)]

ohne dass du uns sagtst wie schnell der controller ist nicht...

und die Interruptroutine und deren Abarbeitungszeit wären auch nicht 
uninteressant...

[Peter D. (peda)]

Joshi schrieb:
> Kann mir einer den Grund dafür
> erklären?

Ja.
Da ist ein Fehler in Deinem Programm.

Peter

[Joshi (Gast)]

Der Controller läuft mit 8MHz und die Interruptroutine sieht 
folgendermaßen aus:

ISR(PCINT2_vect)
{
  PCINT_count++;
}

Alle 20ms wird dann der Counter abgefragt und wieder auf Null gesetzt.

Die Routine sieht folgendermaßen aus:

void read_PCINT(void)
{
  switch(PCINT_state)
  {
    case 0: 
    //disable PIN Change Interrupt
    PCICR &= ~(1 << PCIE2);
    
    //read the countet Interrupts
    PCINT_buffer[0] = PCINT_count;
            
    //set colour direction to red
    PORTC &= ~(1 << PC3);
    PORTC &= ~(1 << PC4);

    //restore counter value
    PCINT_count  = 0;

    //increase state
    PCINT_state++;

    //enable PIN Change Interrupt
    PCICR |= (1 << PCIE2);
    break;

    case 1:  
    //disable PIN Change Interrupt
    PCICR &= ~(1 << PCIE2);
    
    //read the countet Interrupts
    PCINT_buffer[1] = PCINT_count;
            
    //set colour direction to green
    PORTC |= (1 << PC3);
    PORTC &= ~(1 << PC4);

    //restore counter value
    PCINT_count  = 0;

    //increase state
    PCINT_state++;

    //enable PIN Change Interrupt
    PCICR |= (1 << PCIE2);
    break;

    case 2:      
    //disable PIN Change Interrupt
    PCICR &= ~(1 << PCIE2);
    
    //read the countet Interrupts
    PCINT_buffer[2] = PCINT_count;
            
    //set colour direction to blue
    PORTC &= ~(1 << PC3);
    PORTC |= (1 << PC4);

    //restore counter value
    PCINT_count  = 0;

    //increase state
    PCINT_state = 0;

    //enable PIN Change Interrupt
    PCICR |= (1 << PCIE2);
    break;
  }
}

[nicht "gast" (Gast)]

lässt sich deutlich kürzer schreiben

void read_PCINT(void)
{
    //disable PIN Change Interrupt
    PCICR &= ~(1 << PCIE2);
    
    //read the countet Interrupts
    PCINT_buffer[PCINT_STATE] = PCINT_count;

    PORTC = (PORTC & ~(1<<PC3 | 1<<PC4)) | PCINT_STATE++; 

    //restore counter value
    PCINT_count  = 0;

    //enable PIN Change Interrupt
    PCICR |= (1 << PCIE2);
}

[nicht "gast" (Gast)]

nicht "gast" schrieb:
> PORTC = (PORTC & ~(1<<PC3 | 1<<PC4)) | PCINT_STATE++;

PORTC = (PORTC & ~(1<<PC3 | 1<<PC4)) | PCINT_STATE++ << PC3;

[Markus W. (Firma: guloshop.de) (m-w)]

Hallo!

Joshi schrieb:
> Der Controller läuft mit 8MHz und die Interruptroutine sieht
> folgendermaßen aus:

Sicher, dass nicht das Fuse-Bit CKDIV8 auf 0 programmiert ist? In dem 
Fall würde dein Mikrocontroller nämlich mit 1 MHz laufen und den Fehler 
erklären.

> ISR(PCINT2_vect)
> {
>   PCINT_count++;
> }

Falls PCINT_count eine 16-Bit-Variable ist, braucht die Interruptroutine 
mehr als 50 Taktzyklen, wenn sie in C geschrieben ist.

Genaueres siehst du, wenn du dir den generierten Code disassemblieren 
lässt.

Mögliche Lösung: nimm Assembler, dann sind es 15 bis 17 Zyklen...

Zum PCI: Denk dran, dass der bei jedem Flankenwechsel ausgelöst wird, 
also beim Wechsel von 1 auf und und beim Wechsel von 0 auf 1. Falls dir 
eine Flanke reicht, nimm den INT0.

[Oliver J. (skriptkiddy)]

Joshi schrieb:
> Alle 20ms wird dann der Counter abgefragt und wieder auf Null gesetzt.
Warum nimmst du keinen Timer und taktest den extern über dein Signal? Da 
kannst du dann ohne Interruptlast zählen.

Ich tippe mal darauf, dass der µC ab einer bestimmten Signalfrequenz 
ständig in der ISR hängt, weil bevor die abgearbeitet ist, das 
Interruptflag schon wieder gesetzt wird und gleich nach dem reti ein 
neuer Aufruf der ISR stattfindet.

Gruß Oliver