timer problem atmega8

[Wastl F. (rescuetigerchen)]

Hallo Programmierer,

ich habe folgendes Problem. Es soll aus der main() eine Art 
Atemzugsgenerator aufgerufen werden, dessen Frequenz und Wiederholungen 
einstellbar ist.
Es wird ein Atmega8 mit 3.686400MHz verwendet.
Im nachfolgenden Code findet die Abarbeitung der zeitlichen Reihenfolge 
in der ISR statt, bis zu 27 Atemzüge pro Minute klappt das auch 
hervorragend. Aber ab dieser Frequenz leuchtet die LED entweder 
dauerhaft oder gar nicht.
Ich denke es könnte an der Berechnung liegen (klar, int ist nicht 
ganzzahlig).
Habt ihr einen Tip wonach ich suchen kann, eventuell ist ja auch mit den 
Timersettings etwas falsch.

Vielen Dank schon mal,
WastlB

#include <stdlib.h>
#include <stdio.h>
#include <avr\io.h>
#include <avr\interrupt.h>

volatile int exsp=0,insp=0,act=0,state=0,max=0;
volatile int timer;

ISR(TIMER1_OVF_vect)
{              
timer++;
  
    if(state==1)
    {
      if(timer<<insp)
      {
      PORTC |= (1<<0);
      }
      if((timer<<exsp) && (timer>=insp))
      {
      PORTC &= ~(1<<0);
      }
      if(timer==exsp)
      {
      act++;
      timer=0;
        if(act==max)
        {
        state=2;
        }  
      }
    }
}

void init_timer()
  {
    TCCR1A |= (0 << WGM11) | (1 << WGM10);
    TCCR1B |= (1 << WGM13) | (0 << WGM12);
    TCCR1B |= (1 << CS12)  | (0 << CS11)  | (1 << CS10);
    TIMSK |= (1<<TOIE1);
  OCR1A = 1;
  }

void init_front()
  {
  DDRC |= (1<<0)|(1<<1);
  PORTC |= (0<<0)|(0<<1);
  DDRB |= (0<<0)|(0<<1)|(0<<2);
  }

void calc_vent(int cyc,int ventf)
{
max=cyc;
exsp = (60000/ventf);
insp = exsp/3;    

}

int main(void)
{
  init_timer();
  init_front();
  sei();

  for(;;)
  {
      if(state==0)
      {
      calc_vent(5,27);
      state=1;
      }
  }

}

[Karl H. (kbuchegg)]

Bist du sicher, dass das hier

      if(timer<<insp)

tatsächlich ein Links-Schieben sein soll und kein Vergleich auf kleiner?

Persönlich finde ich es immer unklug, wenn man Programmtext so ganz ohne 
Leerzeichen Buchstaben an Buchstaben setzt. Man sieht dann so schlecht 
die kleinen Feinheiten. Dein Lesezentrum im Gehirn wurde seit du 6 Jahre 
alt bist, darauf trainiert, dass zwischen Wörtern ein Leerraum steht und 
wenn du etwas liest, dann sucht das Gehirn erst mal diese Leerräume um 
dort die gedankliche Trennung der Wörter reinzulegen. Kein Mensch käme 
auf die Idee,
deutschenTextkomplettohneLeerzeichenzuschreiben,weildanndasLeseneinfachn 
ur
furchtbarermüdendist.
Nur wenn dieselben Leute dann C Programmieren, dann müssen alle 
Buchstaben knirsch auf knirsch geschrieben werden, weil das ja beim 
Schreiben so wahsinnig viel Zeit spart.


   if( timer << insp )

Schon sticht das << heraus und es ist weit schwieriger so etwas beim 
Korrekturlesen des Codes irrtümlich als < zu lesen. Gerade in einer 
Sprache wie C, in der jedes falsche Zeichen über Sieg oder Niederlage 
entscheiden kann, ist es umso wichtiger, dass man seinen Code so 
gestaltet, dass er gut lesbar ist. Dazu gehören neben optischen Hilfen 
wie Leerzeichen auch korrekte Einrückungen.

Vergleich mal deine Variante

ISR(TIMER1_OVF_vect)
{              
timer++;
  
    if(state==1)
    {
      if(timer<<insp)
      {
      PORTC |= (1<<0);
      }
      if((timer<<exsp) && (timer>=insp))
      {
      PORTC &= ~(1<<0);
      }
      if(timer==exsp)
      {
      act++;
      timer=0;
        if(act==max)
        {
        state=2;
        }  
      }
    }
}

mit der hier

#define ATMEN  1
#define OFF    2

#define LED_PORT  PORTC
#define ATEM_LED  0

ISR(TIMER1_OVF_vect)
{              
  timer++;
  
  if( state == ATMEN )
  {
    if( timer < EinatemZeit )
      LED_PORT |= (1 << ATEM_LED);

    else if( timer < AusatemZeit )
      LED_PORT &= ~(1 << ATEM_LED);

    else {
      timer = 0;
      ZugNummer++;

      if( ZugNummer == AnzahlZuege )
        state = OFF;
    }
  }
}

Denk dir ein paar Füllwörter dazu und der Programmtext der ISR liest 
sich schon fast wie deutsche Sätze.

   if( timer < EinatemZeit )

Aha. Wenn die Zeitzählung als noch kleiner als die Zeit ist, die fürs 
einatmen vorgesehen ist, dann ....

      LED_PORT |= (1 << ATEM_LED);

... ja, logisch. Dann soll die LED welche einatmen anzeigt eingeschaltet 
werden.
Da brauch ich nur den Programmtext lesen, mir ein paar Wörter dazudenken 
und komm zum Schluss: passt. Die Logik kann stimmen.

[Wastl F. (rescuetigerchen)]

Gut, abgesehen davon hilft das meinem Problem aber auch nicht weiter.
Es sollte tatsächlich ein Vergleich sein, nur ging ich davon aus, daß es 
sich wie bei dem booleschen Vergleich wie zum Beispiel if (i==0) 
verhält.
Die ISR verhält sich nach wie vor ab einer Frequenz von 27 seltsam.

[Karl H. (kbuchegg)]

Wastl B. schrieb:
> Gut, abgesehen davon hilft das meinem Problem aber auch nicht weiter.

Inwiefern hilft das deinem Problem nicht weiter, wenn ich dir mitteile, 
dass du an dieser Stelle die völlig falsche Operation machst?

> Es sollte tatsächlich ein Vergleich sein, nur ging ich davon aus, daß es
> sich wie bei dem booleschen Vergleich wie zum Beispiel if (i==0)
> verhält.

Ähm.
Wie wäre es mit C-Grundlagen lernen, ehe man an ein Projekt geht. 
Wenigstens die einfachsten Sachen.

   <      ist kleiner
   <<     ist Links-Schieben

 if( time < zeit )

ist etwas komplett völlig anderes als

 if( time << zeit )

[Wastl F. (rescuetigerchen)]

Sorry.
Hatte (aus irgendwelchen Gründen) deinen Code nicht angezeigt bekommen. 
Erst beim erneuten Aufrufen der Seite war dein Post komplett.
Ich habe die Operatoren angepasst/korrigiert. Immer noch ist die 
Funktion gestört (ich nehme mal an, daß das Programm sich richtig 
verhält, ich es nur falsch programmiert habe).

[Karl H. (kbuchegg)]

Und du hast hoffentlich bemerkt, dass du denselben Fehler hier

     if((timer<<exsp) && (timer>=insp))

nocheinmal gemacht hast? Wie gesagt: Optische Hilfsmittel können einen 
wesentlichen Unterschied ausmachen.

[Wastl F. (rescuetigerchen)]

So, dein Hinweis auf die falschen Operatoren hat zum Ziel geführt.
Unglaublich aber, daß das dann überhaupt funktioniert hat.
Danke sehr, das mit den Operatoren bleibt mir beim nächsten Programm im 
Hinterkopf.