PWM auslesen Duty-Cycle

[Benno (Gast)]

Hallo Leute,
Befasse mich jetzt seit geraumer Zeit mit der Programmierung von Atmel 
uC.
Jetzt befasse ich mich gerade mit Timer-Interrupts und versuche den 
Duty-Cycle eines 1khz PWM-Signals auszulesen.

Zum testen wollte ich einfach zu unterschiedlichen Duty-Cycle´n 
verschiedene Ausgänge schalten. Habe mich an einem Code aus einem 
anderen Beitrag orientiert und diesen leicht angepasst. Leider bekomme 
ich keine Ergebnisse und kein Ausgang schaltet. Vielleicht sieht ja 
einer den Fehler oder hat so etwas selber schon einmal gemacht.
Hier mein Code:


/*
 * PWM einlesen Interrupt.c
 *
 * Created: 22.03.2019 08:16:52
 * Author : Benno
 */


/*Beschreibung:

Ich habe den AT90CAN128 auf 16Mhz eingestellt, mit einem Vorteiler von 
64
ergibt das: 0,000004 Sekunden für einen Timerdurchlauf. Das PWM-Signal
hat 1kHz ==> 0.001 Sekunden
Daraus ergibt sich 0.001 / 0,000004 = 250

Der PortB sollen je nach Dutycycle geschaltet.

Nun zum Code:

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 16000000UL

volatile int flanke=0;
volatile int aktuell_pwm_l=0;
volatile int aktuell_pwm_h=0;

ISR(TIMER1_CAPT_vect)
{

  if (flanke==0)
  {
    TCNT1H = 0;             //Timer auf null zurück
    TCNT1L = 0;
    TCCR1B&= ~(1<<ICES1);  //ICP-Pin auf fallende Flanke einstellen
    flanke=1;
  }
  else
  {
    flanke=0;
    TCCR1B|=(1<<ICES1);     //ICP auf steigende Flanke wieder aktivieren
    aktuell_pwm_l=ICR1L;  //Zählerstand speichern
    aktuell_pwm_h=ICR1H;    //zählerstand speichern
  }
}

void init_prozedur(void)
{
  TCCR1B |=  (1<<ICNC1)|(1<<ICES1)|(1<<CS11)|(1<<CS10) ; // Mittelung ON, steigende flanke, vorteiler 64
  TIMSK1 |= (1<<ICIE1);//INTERRUPT ON

  sei();
}

int main (void)
{
  init_prozedur();
  DDRB=0b00001101;
  DDRD=0x00;
  
  while(1) 
  {
    
    if (aktuell_pwm_h >= 50 && aktuell_pwm_h < 100)
    {
      PORTB|=(1<<PB0);
    }
    if (aktuell_pwm_h >= 100 && aktuell_pwm_h <250)
    {
      PORTB|=(1<<PB2);
    }
    else
    {
      PORTB |= (0<<PB0)|(0<<PB2);
    }
  }

}

[Sebastian R. (sebastian_r569)]

Schau dir mal diese Appnote an:
http://ww1.microchip.com/downloads/en/AppNotes/Atmel-8014-Using-Timer-Capture-to-Measure-PWM-Duty-Cycle_ApplicationNote_AVR135.pdf

[Benno (Gast)]

Die habe ich mir auch schon angeschaut, hat mir nur leider nicht 
wirklich weiter geholfen.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Benno schrieb:
> Nun zum Code
Bitte künftig die Bedienungsanleitung über dem Texteingabefeld beachten:

Wichtige Regeln - erst lesen, dann posten!
    ...
Formatierung (mehr Informationen...)
    [c]C-Code[/c]
    ...

[foobar (Gast)]

Keine Ahnung bzgl des Interrupt-Kodes, aber in main sind zwei Fehler:

1) Die Ports werden nie gelöscht, ein Statement wie "y |= (0<<x)" macht 
nichts.  Wenn das Bit gelöscht werden soll, wäre "y ~= ~(1<<x)" richtig.

2) Du liest aktuell_pwm_h mehrmals aus.  Da sich der Wert durch einen 
Interrupt ändert, können beide if-Bedingungen wahr werden.

  while(1) 
  {
    uint8_t port = PORTB & ~((1<<PB0)|(1<<PB2));
    uint8_t pwm = aktuell_pwm_h;

    if (pwm >= 50 && pwm < 100)
       port |= 1<<PB0;
    if (pwm >= 100 && pwm < 250)
       port |= 1<<PB2;

    PORTB = port;
  }