Timer0 problem

[Daniel L. (daniel_l49)]

hallo

Ich habe ein Programm geschrieben um den timer0 zu testen, aber es tut 
sich nichts

TCCR0  |= (1<<CS00) | (CS01);
TIMSK  |= (1<<TOIE0);

while(1)
{
  PORTB |= (1<<PB0);
  if(TCNT0 ==200)
     PORTB &= (1<<PB0);
}

[Motor (Gast)]

Daniel L. schrieb:
> while(1)
> {
>   PORTB |= (1<<PB0);
>   if(TCNT0 ==200)
>      PORTB &= (1<<PB0);
> }

Auf den ersten Blick hat die letzte Zeile einen Fehler:
>      PORTB &= (1<<PB0);
soll heißen
       PORTB &= ~(1<<PB0);

[Daniel L. (daniel_l49)]

ich hab das jetzt mal geändert, aber es hat nicht geholfen

[Motor (Gast)]

Wie stellst du das fest?
Ich vermute mal, mit einer LED am Ausgang oder sowas in der Art.
Da wirst du aber nicht viel sehen. Mit einem Oszilloskop schon.
Der Impuls wird nur sehr kurz sein.


> while(1)
> {
>   PORTB |= (1<<PB0);
PB0 wird gesetzt
>   if(TCNT0 ==200)
wenn TCNT0 = 200 ist,
>      PORTB &= (1<<PB0);
wird PB0 rückgesetzt
> }
und im nächsten Schleifendurchlauf sofort wieder gesetzt.

[Uwe (de0508)]

Hallo  Daniel,

sorry, das ist kein Programm, sondern nur ein Auszug und der lässt sich 
so nicht übersetzten !

Wo ist die Funktion main() und die prozessor spezifischen 
Include-Dateien?

[Ingo (Gast)]

Daniel L. schrieb:
> if(TCNT0 ==200)
>      PORTB &= (1<<PB0);

Du müsstest schon viel Glück haben das das Programm bei genau 200 die 
Abfrage macht.

if (TCNT0 >= 200){
 PORTB &= ~(1<<PB0);
}else{
 PORTB |= (1<<PB0);
}

wäre besser denke ich, auch wenn es immernoch Murks ist.

[Daniel L. (daniel_l49)]

Ich habe das jetzt mal außerhalb der while-Schleife versucht,aber die 
LED leuchtet durchgehend

[Karl H. (kbuchegg)]

Daniel L. schrieb:
> Ich habe das jetzt mal außerhalb der while-Schleife versucht,aber die
> LED leuchtet durchgehend


Was erwartest du?

Natürlich leuchtet die durchgehend (für dich langsamen Menschen).
Aber wenn du mal eine LED daneben auf Dauer-Ein einschaltest, dann wirst 
du sehen, dass sie etwas dunkler leuchtet (wenn auch nicht viel).

Kann es sein, dass du du ganz krass die Geschwindigkeit unterschätzt? 
Rechne mal nach, wie schnell deine LED eigentlich blinkt!

[Daniel L. (daniel_l49)]

Tut mir leid das habe ich jetzt nicht ganz verstanden

Wenn ein Code außerhalb der while-Schleife ist, wird dieser ja nur 
einmal durchlaufen

[Karl H. (kbuchegg)]

Und bitte: kompletten Code! Inklusive der Angabe der Taktfrequenz, mit 
der der µC läuft.
Ehe du Code in Prosa beschreibst, kopiere lieber den Code hier rein. Und 
zwar alles. Das ist für dich einfacher und für uns auch. Denn dann haben 
wir ein Bild davon, wie der Code wirklich aussieht und müssen nicht 
raten.

[Daniel L. (daniel_l49)]

Hier der Code:

#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1 
#define taste_nicht_gedruckt 0

volatile int k = 0;

int main (void)
{
  
  sei();
  
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher; 

  
  void blink1();
  void blink2();
  
  TCCR0  |= (1<<CS00) | (CS01);
  TIMSK  |= (1<<TOIE0);
  

  
  if (!(PIND & (1<<PD2)))
  {
    taste_vorher = taste_gedruckt;
  } 
  else
  {
    taste_vorher = taste_nicht_gedruckt;
  }
  
  
  
    
  while (1)
  {  
    if(!(PIND & (1<<PD2)))
    {
      taste_jetzt  = taste_gedruckt;  
    }
    else
    {
      taste_jetzt  = taste_nicht_gedruckt; 
    }      
  
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        i++;
        //switch(i)
        //{
          blink1(i, k);
        
        //blink2(i);
        //}
      }
    } taste_jetzt = taste_vorher;
    
    _delay_ms(50);
  }
}

void blink1(int i , int k )
{
  while(1)
  {
    PORTB |= (1<<PD1);
    if (k  >= 440)
    {  
        PORTB |= (1<<PD2);
        PORTB &= ~ (1<<PD1);
    }
    if (k>= 440)
    {  
      PORTB &= ~ (1<<PD2);
      PORTB |= (1<<PD0);
    }
    if(k>=440)
      PORTB &= ~ (1<<PD0);    
  }
   
}
void blink2(int i)
{
  
}

ISR(TIMER0_OVF_vect)
{
  k++;
  if (k >= 440)
    k=0;
}

Hier ist das Programm

Das Problem liegt hier in der Funktion blink1

Es sollte hier einfach die LED's umschalten

[Karl H. (kbuchegg)]

Daniel L. schrieb:

> Es sollte hier einfach die LED's umschalten

Dann solltest du das auch machen.

ABer solange du den µC in der Funktion gefangen hältst

void blink1(int i , int k )
{
  while(1)
  {
    PORTB |= (1<<PD1);
    if (k  >= 440)
    {  
        PORTB |= (1<<PD2);
        PORTB &= ~ (1<<PD1);
    }
    if (k>= 440)
    {  
      PORTB &= ~ (1<<PD2);
      PORTB |= (1<<PD0);
    }
    if(k>=440)
      PORTB &= ~ (1<<PD0);    
  }
   
}

wird sich vom restlichen Programm da nichts mehr tun. Da ist eine 
Endlossschleife in der Funktion. Die Funktion wird nie mehr verlassen.
Ganz abgesehen davon, dass dein k auch mit ISR Hilfe nie größer/gleich 
440 werden kann.

Der ganze Ansatz ist Müll.
Mit der ISR hast du den perfekten Ort, um ein zeitliches Blinken zu 
realisieren.

[Daniel L. (daniel_l49)]

Ich habe den Code noch nicht geschrieben um die Funktion zu verlassen 
bzw.
Die schleife zu unterbrechen.

Wenn die Funktion aufgerufen wird sollten doch die LED's schon mal 
umgeschalten werden.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> Der ganze Ansatz ist Müll.
> Mit der ISR hast du den perfekten Ort, um ein zeitliches Blinken zu
> realisieren.

STeck den kompletten Blink-Code in die ISR.

ISR(TIMER0_OVF_vect)
{
  if( isBlinking )
  {
    k++;
    if (k >= 440)
    {
      k=0;

      if( k > 200 ) {
        PORTB |= (1<<PD2);
        PORTB &= ~ (1<<PD1);
      }
      else {
        PORTB &= ~ (1<<PD2);
        PORTB |= (1<<PD0);      
      }
    }
  }
}

und im Hauptprogramm schaltest du bei Tastendruck einfach nur das 
Blinken ein/aus, indem du die Variable 'isBlinking' auf 0 bzw. 1 setzt.

volatile uint8_t isBlinking;

....


    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        isBlinking = 1 - isBlinking;
      }
      ...

...

[Karl H. (kbuchegg)]

PS:
Allerdings müssen dazu auch die Interrupts mitels sei() freigegeben 
worden sein. Ansonsten wird das nichts mit dem AUfruf der ISR

int main()
{
   ....


   sei();
   while( 1 )
   {
      ....

[Karl H. (kbuchegg)]

Daniel L. schrieb:

> Wenn die Funktion aufgerufen wird sollten doch die LED's schon mal
> umgeschalten werden.


Moment.

Hast du deine Tastenauswertung für sich alleine gestestet oder hast du 
das nicht?
Funktioniert die?

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> Hast du deine Tastenauswertung für sich alleine gestestet oder hast du
> das nicht?
> Funktioniert die?


Ehe da jetzt der nächste Schritt drauf gesetzt wird, sollte der Punkt 
erst mal geklärt sein. Nicht dass du/wir da jetzt einem Timer-Phantom 
nachjagen und das 'Problem' ganz woanders liegt.

+#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1 
#define taste_nicht_gedruckt 0

volatile int k = 0;

int main (void)
{
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher; 

  taste_vorher =  (PIND & (1<<PD2) ? taste_nicht_gedruckt : taste_gedruckt;

  while (1)
  {  
    taste_jetzt  = (PIND & (1<<PD2) ? taste_nicht_gedruckt : taste_gedruckt;  
  
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        PORTB ^= ( 1 << PB2 );
      }
    } taste_jetzt = taste_vorher;
    
    _delay_ms(50);
  }
}

die LED an PB2 muss mit jedem Tastendruck umschalten.
klappt?
klappt nicht?

[Karl H. (kbuchegg)]

Karl Heinz schrieb:
> PS:
> Allerdings müssen dazu auch die Interrupts mitels sei() freigegeben
> worden sein.

Seh gerade, du hast das am Anfang von main() gemacht.
Keine gute Idee.

Erst mal wird alles initialisiert, die Hardware eingestellt, Variablen 
auf ihre Anfangswerte begracht.
Und erst dann, wenn alles soweit fertig und grundsätzlich lauffähig ist, 
erst dann kommt das 'Feuer frei' für die Interrupts. Als letzte Aktion 
vor der Hauptschleife.
Denn was du nicht willst: Das die Interrupts sofort zu feuern beginnen, 
wenn du in einzelnen Komponenten einen spezifischen Interrupt während 
der Konfiguration frei gibst. Du kannst nicht alles parallel und zur 
gleichen Zeit konfigurieren. Du willst aber in komplexeren Programmen 
haben, dass das Interrupt-Konzert in geordneten Bahnen läuft. Und das 
tut es im Regelfall erst dann, wenn ALLE Komponenten initialisiert sind.

[Daniel L. (daniel_l49)]

if (!(PIND & (1<<PD2))) // wenn Taste gedrückt
  {
    taste_vorher = taste_gedruckt;
  } 
  else
  {
    taste_vorher = taste_nicht_gedruckt;
  }
  
  sei();  
  while (1)
  {  
    if(!(PIND & (1<<PD2))) // Wenn Taste gedrückt
    {
      taste_jetzt  = taste_gedruckt;  
    }
    else
    {
      taste_jetzt  = taste_nicht_gedruckt; 
    }      
  
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        
          PORTB ^= (1<<PB0); // Wird umgeshcalten
        
        
      }
    } taste_jetzt = taste_vorher;
    
    _delay_ms(50);
  }
}

[Daniel L. (daniel_l49)]

Habe gerade festgestellt, dass die LED nicht bei jedem Tastendruck 
umgeschalten wird.

[Karl H. (kbuchegg)]

>     } taste_jetzt = taste_vorher;

falsch rum

  taste_vorher = taste_jetzt;

(und ich Depp kopier das auch noch weiter)

[Daniel L. (daniel_l49)]

Ich habe das jetzt umgeändert, aber jetzt lässt sich die LED nur noch 
einschalten, aber nicht mehr ausschaltenb

[Karl H. (kbuchegg)]

Daniel L. schrieb:
> Ich habe das jetzt umgeändert, aber jetzt lässt sich die LED nur noch
> einschalten, aber nicht mehr ausschaltenb


Welches Programm?
(Nach DEM Fehler trau ich dir nicht mehr)

[Daniel L. (daniel_l49)]

#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1 
#define taste_nicht_gedruckt 0

volatile int k = 0;
volatile bool isblinking = false;

int main (void)
{
        
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher; 

  
  void blink1();
  void blink2();
  
  TCCR0  |= (1<<CS00) | (CS01);
  TIMSK  |= (1<<TOIE0);
  

  
  if (!(PIND & (1<<PD2)))
  {
    taste_vorher = taste_gedruckt;
  } 
  else
  {
    taste_vorher = taste_nicht_gedruckt;
  }
    
  sei();  
  while (1)
  {  
    if(!(PIND & (1<<PD2)))
    {
      taste_jetzt  = taste_gedruckt;  
    }
    else
    {
      taste_jetzt  = taste_nicht_gedruckt; 
    }      
  
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        
          PORTB ^= (1<<PB0);
        
      }
    } taste_vorher = taste_jetzt;
    
    _delay_ms(30);
  }
}

[Karl H. (kbuchegg)]

Daniel L. schrieb:

>   TIMSK  |= (1<<TOIE0);

Overflow Interrupt freigegeben


>   sei();

Interrupts generell freigegeben


> }

.... aber keine ISR.

Sowas wird mit einem µC-Reset bestraft.

Aus gutem Grund hab ich in meinem 'Testprogramm' weiter oben, den Timer 
komplett rausgelassen.

[Daniel L. (daniel_l49)]

Hier noch die ISR

ISR(TIMER0_OVF_vect)
{
  
  
  if (isblinking == true)
  {
    k++;
    if (k >= 440)
    k=0;
    
    PORTB |= (1<<PD1);
    if (k == 440)
    {
      PORTB |= (1<<PD2);
      PORTB &= ~(1<<PD1);
    }
    if (k>= 440)
    {
      PORTB &= ~(1<<PD2);
      PORTB |= (1<<PD0);
    }
    if(k>=440)
    PORTB &= ~(1<<PD0);
  }
  
}

[Karl H. (kbuchegg)]

Daniel L. schrieb:
> Hier noch die ISR

Schön.
Ich bin raus.

Ich sag dir einmal, du mögest den ganzen Code posten. Ich sags 
vielleicht noch ein zweitesmal ein paar Tage später. Aber nicht ein paar 
Minuten/Stunden später.
Such dir einen anderen Dummen. Ich bin das Rätselraten leid.

[Daniel L. (daniel_l49)]

Oke tut mir leid hier ist der ganze Code:

#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1
#define taste_nicht_gedruckt 0

volatile int k = 0;
volatile bool isblinking = false;

int main (void)
{
  
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher;

  
  void blink1();
  void blink2();
  
  TCCR0  |= (1<<CS00) | (CS01);
  TIMSK  |= (1<<TOIE0);
  

  
  if (!(PIND & (1<<PD2)))
  {
    taste_vorher = taste_gedruckt;
  }
  else
  {
    taste_vorher = taste_nicht_gedruckt;
  }
  
  
  
  
  sei();
  while (1)
  {
    if(!(PIND & (1<<PD2)))
    {
      taste_jetzt  = taste_gedruckt;
    }
    else
    {
      taste_jetzt  = taste_nicht_gedruckt;
    }
    
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
      
             isblinking = true;
        
        
      }
    } taste_vorher = taste_jetzt;
    
    _delay_ms(30);
  }
}
ISR(TIMER0_OVF_vect)
{
  
  
  if (isblinking == true)
  {
    k++;
    if (k >= 440)
    k=0;
    
    PORTB |= (1<<PD1);
    if (k == 440)
    {
      PORTB |= (1<<PD2);
      PORTB &= ~(1<<PD1);
    }
    if (k>= 440)
    {
      PORTB &= ~(1<<PD2);
      PORTB |= (1<<PD0);
    }
    if(k>=440)
    PORTB &= ~(1<<PD0);
  }
  
}

[Michael R. (Firma: Brainit GmbH) (fisa)]

ich versuch mal, dir zu helfen, dein Hirn einzuschalten:

Daniel L. schrieb:

   if (k >= 440)
       k=0;

wie groß ist k nun maximal?

   if (k == 440)

kann das jemals passieren?

   if (k>= 440)

oder kann das jemals passieren?

(dabei hatte ich erst gestern einen so vielversprechenden kandidaten für 
"useless code":

while (1) {
   break;
}

[LittleHelper (Gast)]

Daniel L. schrieb:
> Ich habe das jetzt umgeändert, aber jetzt lässt sich die LED nur
> noch
> einschalten, aber nicht mehr ausschaltenb

Daniel L. schrieb:
> if(taste_vorher != taste_jetzt)
>     {
>       if (taste_jetzt == taste_gedruckt)
>       {
>
>              isblinking = true;
>
>
>       }
>     } taste_vorher = taste_jetzt;

else if (taste_jetzt == taste_nicht_gedruckt)
{
  isBlinking = false;
}

[Daniel L. (daniel_l49)]

Oke Es kann nie passieren

[Martin (Gast)]

Hallo,

Kann mir jemand sagen warum die Led bei diesem Code nicht leuchtet ?

//---------------------CODE 1-----------------------------
#include <avr/io.h>
#include <avr/interrupt.h>

long cnt;
int main(void)
{
    TCCR0 = (1<<CS01);
    TIMSK |= (1<<TOIE0);
    DDRD = 0b11110111;
    sei();
    while(1)
    {
    if(cnt > 3900)
    {
      PORTD = 0x01;
      cnt = 0;
    }

    }
}
ISR (TIMER0_OVF_vect)
{
      cnt++;
}


Bei diesem Code leuchtet die Led jedoch!!!!!!!!!!!
//---------------------CODE 2-----------------------------
#include <avr/io.h>
#include <avr/interrupt.h>

long cnt;

int main(void)
{
    TCCR0 = (1<<CS01);
    TIMSK |= (1<<TOIE0);
    DDRD = 0b11110111;
    sei();

    while(1)
    {


    }
}

ISR (TIMER0_OVF_vect)
{
      cnt++;
      if(cnt > 3900)
      {
        PORTD = 0x01;
        cnt = 0;
      }
}


Im Grunde ist es das gleiche Programm aber es funktioniert nur der Code 
2.
Wenn ich das Programm im Atmel Studio 6 Simuliere sieht es aus als würde 
nur 1 mal die while schleife augeführt werden und danach nicht mehr, was 
auch erklären würden wieso die Led nicht leuchtet. Aber Wieso? Kann mir 
jemand weiter helfen!?

[Peter II (Gast)]

Martin schrieb:
> Kann mir jemand sagen warum die Led bei diesem Code nicht leuchtet ?

 cnt ist nicht volatile.

[Martin (Gast)]

Super Danke. Hat funktioniert!

Bin noch Anfänger im Microcontroller programmieren bzw. ist die 
Schulzeit schon sehr sehr lange her ;)