Interrupt TIMER1_CAPT wird nicht ausgelöst

[X.P. (Gast)]

Hallo zusammen,

Ich versuche gerade einen Wert an einem Potentiometer einzulesen und 
anhand des ADCs ihn in digital umzuwandeln. Den digitalen Wert gebe ich 
an den OCR0B und den Timer0 habe ich im Fast-PWM-Modus mit einer 
Frequenz von 1 kHz eingestellt. Anhand des Timer1 (bzw. 
Timer1-Capture-Interrupt) versuche ich den Duty Cycle in Prozent 
auszugeben. Alles scheint ok zu funktionieren bis auf das Auslösen des 
Interrupts. Der Interrupt wird gar nicht ausgelöst und die Variable 
result hat keinen (vernünftigen) Wert am Ende. Kann mir bitte jemand 
sagen, was ich hier falsch mache?

#define F_CPU 16000000UL
#define BAUDRATE 9600
#define BAUD_CONST (((F_CPU/(BAUDRATE*16UL)))-1)

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <stdio.h>

volatile uint8_t ready = 0;
volatile uint16_t result;

// USART
void USART_Init() {
  UBRR0H = (BAUD_CONST >> 8);
  UBRR0L = BAUD_CONST;
  UCSR0B |= (1<<RXEN0)|(1<<TXEN0);
}

void USART_Transmit(unsigned char data) {
  while(!(UCSR0A & (1<<UDRE0))) ;
  UDR0 = data;
}

// ADC
void ADC_Init() {
  ADCSRA |= (1<<ADEN);    // ADC on
  ADCSRA |= (1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0);    // set prescaler to 128
}

uint8_t ReadValue(uint8_t adc_value) {
  uint8_t d_value;
  
  ADMUX = adc_value;
  _delay_ms(5);
  
  ADCSRA |= (1<<ADSC);    // start first conversion
  
  while (ADCSRA & (1<<ADSC)) ;    // wait until the conversion is over
  
  d_value = ADC >> 2;
  
  return (d_value < 250) ? d_value : 0;
}

// Timer1
ISR(TIMER1_CAPT_vect) {
  static uint16_t value1, value2;
  
  if(TCCR1B & (1<<ICES1)) {
    value1 = ICR1;
    TCCR1B &= ~(1<<ICES1);    // now waiting for falling edge
  }
  else {
    value2 = ICR1;
    TCCR1B |= (1<<ICES1);    // now waiting for raising edge
    result = value2 - value1;
    ready = 1;    // ready to print the duty cycle
  }
}


int main(void)
{  
  const unsigned char DutyCycle[] = "Duty cycle in percentage: ";
  unsigned char IntToString[3];
  int percentage;
  
  DDRD |= (1<<DDD5);    // OC0B set as output (LED)
  
  USART_Init();
  ADC_Init();
  
  // Timer0
  TCCR0A |= (1<<COM0B1)|(1<<WGM01)|(1<<WGM00);    // clear OC0A and OC0B on compare match, Fast PWM
  TCCR0B |= (1<<WGM02)|(1<<CS01)|(1<<CS00);    // prescaler set to 64
  OCR0A = 249;    // frequency is now 1 kHz
  
  // Timer1
  TCCR1B |= (1<<ICES1)|(1<<CS11)|(1<<CS10);    // waiting for raising edge, prescaler set to 64, 250 kHz frequency
  TIMSK1 |= (1<<ICIE1);    // Capture-Interrupt on
  
  sei();
  
    while (1) 
    {
    OCR0B = ReadValue(0b01000000);
    
    if(ready) {
      cli();
      
      percentage = (result / 1000.0) * 100;
      sprintf((char*)IntToString, "%d", percentage);
      
      for(int i = 0; DutyCycle[i] != '\0'; i++)
        USART_Transmit(DutyCycle[i]);
        
      for(int i = 0; IntToString[i] != '\0'; i++)
        USART_Transmit(IntToString[i]);
        
      USART_Transmit('%');
      USART_Transmit('\r');
      USART_Transmit('\n');
      
      sei();
      
      ready = 0;
    }
    
    //_delay_ms(500);
  }
}

[TLDR (Gast)]

Was ist unter dem Hinweis

"Wichtige Regeln - erst lesen, dann posten!
.............
 Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang
............."

nicht zu verstehen?

[S. Landolt (Gast)]

> Der Interrupt wird gar nicht ausgelöst und ...

? - ?
Es läuft doch alles!? Mal von einem ATmega328P ausgehend.
  Nur die Rechnerei sieht merkwürdig aus, Beispiel Ucc/2: dann ist der 
Tastgrad 0.5, bei 1 kHz werden also 500 us erzeugt, mit dem Timer1 bei 
250 kHz folglich 125 gemessen, was zur Ausgabe "Duty cycle in 
percentage: 12%" führt; bei Ucc*3/5 "Duty cycle in percentage: 15%".

[S. Landolt (Gast)]

PS:
D5 und B0 sind natürlich verbunden.

[X.P. (Gast)]

Danke für die Bemerkung. Hatte vergessen, den Wert von result nach dem 
Subtrahieren mit 4 zu multiplizieren, da bei 250 kHz ein Tick = 4 µs. 
Bei mir hat es gestern nicht funktioniert (duty cycle hat ständig 0 
ausgegeben), deswegen habe ich es gepostet (konnte es aber danach nicht 
testen, da das Gerät nicht im Labor der Uni liegt). Aber wenn du sagst, 
dass es funktioniert, dann sollte es. Vielen Dank nochmal :)

[X.P. (Gast)]

Hab's heute nochmal probiert und es hat funktioniert. Es hat 
höchstwahrscheinlich an die Verbindung der PINs gelegen.