RC-Reciever und das seltsame Timer Ergebniss

[Max D. (Firma: Hobby) (fmhweb)]

Hallo,

ich kann mir vorstellen, dass einige bei den folgenden Fragen ihr Gehirn 
sehen und bitte um Verständnis. Ich versuche etwas noch nie da Gewesenes 
zu machen, nämlich einen RC-Reciever auszulesen und kann zumindest einen 
Teilerfolg bieten. Ich wundere mich über die Ausgabe des Timers.

Ich verwende einen ATMega16 mit einem 8MHz Quartz und einen analogen 
Futuba Reciever, angeschlossen an INT0. Derzeit versuche ich, zum 
lernen, nur ein Servosignal auzuwerten und nicht das Summensignal. 
Soweit klappt das auch, doch der Timer gibt mir nicht das gewünschte 
Ergebniss auf meinem LCD aus. Die Fuses habe ich mit _delay_ms(1000) und 
LEDs getestet. Das übliche Servosignal ist 1ms – 2ms.

Hier meine (ausführliche) Rechnung:
8000000 Hz / 64 prescaler = 125000 Schritte pro Sekunde
1 sek / 12500 Schritte * 1000 = 0,008 ms pro Schritt
1 ms / 0,008 ms pro Schritt = 125 Schritte
2 ms / 0,008 ms pro Schritt = 250 Schritte
Auflösung: 125

Bei 1 ms erwarte ich also ein Countervalue (TCNT1) von 125 und bei 2 ms 
250. (Timer0 würde also ausreichen)

Auf dem LCD lese ich aber +/- 5:
1ms = 18608  (Knüppel unten)
1,5ms = 18680  (Knüppel mitte)
2ms = 18760  (Knüppel oben)

Bitte schaut euch dazu meinen Code an:

#define F_CPU 8000000UL

#include <avr/io.h>
#include "lcd-routines.c"
#include <avr/interrupt.h>
#include <stdlib.h>

//volatile uint16_t starttime = 0;
volatile uint16_t signal1 = 0;
volatile uint8_t edge_switch=0;

ISR(INT0_vect) {
  if(edge_switch == 0){
    //starttime = TCNT1;
    TCNT1 = 0;
    edge_switch = 1;
    MCUCR |= (0<<ISC00);          //Fallende Flanke
  }
  else{
    //signal1 = TCNT1 - starttime;
    signal1 = TCNT1;
    edge_switch = 0;
    MCUCR |= (1<<ISC00);          //Steigende Flanke
  }
}

void lcd_string_int(uint16_t tmp_int){
  char buffer[20];
  utoa(tmp_int, buffer, 10);
  lcd_string(buffer);
}

int main(void){
  GICR |= (1<<INT0);                     //INT0 enable
  MCUCR |= ((1<<ISC01) | (1<<ISC00));    //Steigende Flanke
  TCCR1B = ((1<<CS10) | (1<<CS11));      //Prescaler 64
  
  lcd_init();
  sei();
  
  while(1){
    lcd_clear();
    lcd_string_int(signal1);
    _delay_ms(1000);
  }
}

1) Kann jemand erkennen wo mein Fehler liegt?
2) Habe ich mich verrechnet?
3) Sind die Einstellungen richtig?

[L.A: (Gast)]

Max Durst schrieb:
> Reciever

Was ist das?

[Max D. (Firma: Hobby) (fmhweb)]

Empfänger?

Mit Reciever meine ich natürlich Receiver

[Max D. (Firma: Hobby) (fmhweb)]

Mit dem Betreff und dem eingedeutschtem Receiver habe ich wohl einiges 
an street credit eingebüsst. Dabei habe ich GCSE, A-Level und in England 
business information technology studiert:( Wie unangenehm.

[Stefan E. (sternst)]

    MCUCR |= (0<<ISC00);          //Fallende Flanke

Die Zeile macht rein gar nichts, also misst du ständig nur die Zeit 
zwischen zwei steigenden Flanken.

[Tom M. (tomm)]

Max Durst schrieb:
> MCUCR |= ((1<<ISC01) | (1<<ISC00));    //Steigende Flanke

Ich weiss nicht, ob das die Ursache ist, aber du setzt MCUCR an 
verschiedenen Stellen, auch in der ISR. Dort ver-oder-st du auch, also 
ändert das nichts an der Konfiguration.

MCUCR scheint mir etwas exotisch zu sein, passt da nicht eher EICRx 
(kuck nohcmals in Datenblatt).

Jedenfalls feuert die ISR bei beiden Flanken, und du weisst nicht, ob du 
gerade die steigende oder fallende beobachtet hast, das könnte eine 
Fehlerquelle sein.

[Max D. (Firma: Hobby) (fmhweb)]

Danke vielmals. Habe die Zeile geändert und jetzt geht es.

MCUCR ^= (1<<ISC00);

PS: EICRx ist habe ich mal bei einem Tiny AT gesehen. Wahrscheinlich 
kann man das bei den MCs nicht im Interrupt setzen. I dunno.

[Karl H. (kbuchegg)]

Max Durst schrieb:
> Danke vielmals. Habe die Zeile geändert und jetzt geht es.
>

> MCUCR ^= (1<<ISC00);
>

Kann man machen. Persönlich würdeich aber

  MCUCR &= ~( 1<<ISC00);

vorziehen.
Ich finde das klarer, dass hier das Bit dezidiert gelöscht werden soll. 
Bei deiner Lösung muss man ständig im Hinterkopf halten, dass es im 
anderen Zweig gesetzt wird um zu wissen, was hier eigentlich 
beabsichtigt ist.