Input Capture Atmega16 Problem

[Blubber (Gast)]

Hallo Leute,
Will zur Übung mit uCs und aus Spaß an der Freude eine DCF77-Uhr bauen.
Zuerst wollte ich mir dazu auf irgendeine Weise das Signal ausgeben 
lassen. Meine erster Ansatz: ich lasse mir die Pulsdauer (als einfacher 
Timerwert) auf ein 2x16 Display schreiben.

Conrad DCF Modul an PortD 
http://www.conrad.de/ce/de/product/641138/Conrad-DCF-Empfaengerplatine
P.Fleurys LCD-Bib, LCD an PORTA 
http://homepage.hispeed.ch/peterfleury/avr-software.html#libs
easyAVR6-Board (Pullups/downs am Board werden nicht genutzt)
Atmega16, 8Mhz int.RC, Timer1, Prescaler 1024.

Erwarten würde ich, dass ich, da Input Capture ja sowohl auf fallende 
als auch steigende Flanken reagieren sollte, für die 0,1-Sek.-Pulse 
ICR1-Werte um 780(für 0,2-Sek.-Pulse um 1550, für 0,9-Sek.-Pausen um 
7000, für 0,8-Sek.-Pausen 6250) bekommen würde.
Dem ist aber leider überhaupt nicht so. Die ausgegebenen Werte liegen 
alle etwa in den Bereichen 8800+-100, 8050+-100, 7200+-100.

Die einzige Erklärung die ich dafür finden konnte ist, dass nicht alle 
Flanken erkannt werden. Ist diese Erklärung evtl falsch? Wenn ja, wo ist 
mein Denkfehler? Wenn nein, woran könnte es liegen, bzw. was kann ich 
dagegen tun?

#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdio.h>
#include <string.h>

#include "lcd.h"
#define LCD_BL 2

volatile long int pulsewidth1=0;
volatile long int pulsewidth2=0;
volatile uint8_t a=0;
char LCD_string[10];


int main(void){  
  lcd_init(LCD_DISP_ON);
  DDRA|=(1<<LCD_BL);  // Pin unbelegt -> Ausgang
  PORTA|=(1<<LCD_BL);
  
  DDRD&=~(1<<DDD6);  // PD6 Eingang
  PORTD=(1<<PD6);    // Pullup, da DCF-Modul OpenCollector
  
  TCCR1B|=(1<<CS12)|(1<<CS10)|(1<<ICNC1);  // Normal-Mode, Prescaler 1024
  TIMSK|=(1<<TOIE1)|(1<<TICIE1);           // Timer 1 Ovf-Interrupt und Capt-Event-Interrupt
  sei();
  while(1){
    if(a){
      sprintf(LCD_string,"%#8ld", pulsewidth1);
      lcd_gotoxy(6,0);
      lcd_puts(LCD_string);
    }
    else{
      sprintf(LCD_string,"%#8ld", pulsewidth1);
      lcd_gotoxy(6,1);
      lcd_puts(LCD_string);
    }
  }
  cli();
}

ISR(TIMER1_CAPT_vect){
  pulsewidth1=ICR1;    // bei 8Mhz,Psc=1024: 781,25=0,1s | 1562,5=0,2s | 7812,5=1s
                       // 7031,25=0,9s | 6250=0,8s | 14843,75=1,9s | 14062,5=1,8s
  if(a==0){a=1;}
  else{a=0;}
  TCNT1=0;
}

ISR(TIMER1_OVF_vect){ // eigentlich irrelevant
  PORTC++;
}

Vielen Dank schonmal im voraus!

[Blubber (Gast)]

Nachtrag: Der einzige sinnvolle Wert kommt jede Minute beim 
ausgelassenen 59. Bit, welches einer Pause von 1,8 Sek bzw 1,9 Sek (also 
Timerwert um ca 15000) entspricht.

$$Wert=t \cdot \frac{Prescaler}{Taktfrequenz}$$

[c-hater (Gast)]

K. S. schrieb:

> Erwarten würde ich, dass ich, da Input Capture ja sowohl auf fallende
> als auch steigende Flanken reagieren sollte, für die 0,1-Sek.-Pulse
> ICR1-Werte um 780(für 0,2-Sek.-Pulse um 1550, für 0,9-Sek.-Pausen um
> 7000, für 0,8-Sek.-Pausen 6250) bekommen würde.

Soweit erstmal nachvollziehbare Überlegungen. Und da es lobens- und 
fördernswert ist, wenn noch jemand selber denkt, tatsächlich noch ein 
kleiner Hinweis:

> Dem ist aber leider überhaupt nicht so. Die ausgegebenen Werte liegen
> alle etwa in den Bereichen 8800+-100, 8050+-100, 7200+-100.

Und was passiert, wenn du vor jeder Ausgabe erstmal das Display komplett 
löschst (oder zumindest den Bereich, in dem die Ausgaben erfolgen, also 
wenigstens 4 Zeichen)...

Problem erkennt?

[test (Gast)]

K. S. schrieb:
> Erwarten würde ich, dass ich, da Input Capture ja sowohl auf fallende
> als auch steigende Flanken reagieren sollte...

Richtig!

Entweder wird auf die fallende oder auf die steigende Flanke reagiert. 
Welche davon, das wird durch Bit ICES1 in TCCR1B festgelegt.

[Blubber (Gast)]

c-hater schrieb:
> Und was passiert, wenn du vor jeder Ausgabe erstmal das Display komplett
> löschst (oder zumindest den Bereich, in dem die Ausgaben erfolgen, also
> wenigstens 4 Zeichen

Das stimmt natürlich, danke.

test schrieb:
> Entweder wird auf die fallende oder auf die steigende Flanke reagiert.
> Welche davon, das wird durch Bit ICES1 in TCCR1B festgelegt.

Dieses Bit ist mir überhaupt nicht aufgefallen, danke. Das heißt also, 
ich müsste das Bit jedes Mal im Interrupt Checken und entsprechend 
setzen bzw. löschen.

ISR(TIMER1_CAPT_vect){
  pulsewidth_new=ICR1;
  TCNT1=0;
  if(TCCR1B&&(1<<ICES1)){   // Wenn steigende Flanke aktiv
    TCCR1B&=~(1<<ICES1);    // fallende aktivieren
  }
  else{                      // Wenn fallende Flanke aktiv
    TCCR1B|=(1<<ICES1);     // steigende aktivieren
  }
}

Sehe ich das richtig?

[Blubber (Gast)]

K. S. schrieb:
>

if(TCCR1B&&(1<<ICES1))

Ist natürlich ein '&' zu viel. Vielen Dank für eure Hilfe. Klappt!