Timer0 Lampe blinken

[Jan M. (pietus3)]

Hallo,

Ich habe das Programm schon für den 16 Bit-Timer geschrieben und befinde 
mich wieder einmal in einer Sackgasse.

Nun möchte ich den Timer 0 benutzen. Ich hab mir gedacht ich müsste nur 
die Register ändern und das wäre es. Aber irgendwo hab ich einen 
Denkfehler anscheinend gemacht. Ich programmier unter der Arduino IDE 
und mein µC hat einen atMega 328.

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

volatile boolean LampeAN;
volatile int TimerCount;

ISR( TIMER0_COMPA_vect)                // Interruptbehandlungsroutine
{
  TimerCount++;
  if(TimerCount == 100000)
  {
     if (LampeAN){
        LampeAN = false;    
        }
    else{
        LampeAN= true;
        }
    TimerCount = 0;
  }     
}

void setup() {
  DDRB = 0b11111111;
  PORTB |= (1<<PORTB1);           // Pullup für PB0 und PB1
 
  TCCR0A = (1<<WGM01);      // CTC-Mode;
  TCCR0B = (1<<CS00);      // Prescaler 1
  TIMSK0  = (1<<OCIE0A);                 // Timer-Compare Interrupt an
  LampeAN = true;
 
  OCR0A = 160;                         // Neutralposition ((2500-2312)*0.008ms)=1,5ms)
  sei();                                // Interrupts global an

}

void loop() {
  if (LampeAN) {       // Impuls-Zeit verlängern
      PORTB |= (1<<PORTB0); 
    } 
    else{       // Impuls-Zeit verkürzen
      PORTB &= ~(1<<PORTB0); 
    } 
}

Problem: Kein blinken bei 1 sec.

Vielen Dank für eure Bemühungen

Jan Peter Meyer

[Bitflüsterer (Gast)]

Ich hoffe Du hälst mich nicht für aufdringlich, aber willst Du uns das 
Problem nennen, oder dürfen wir uns ein beliebiges aussuchen?

[ich (Gast)]

Beim ersten groben Überfliegen würde ich schon mal diese Zeile
> if(TimerCount == 100000)
als Problem sehen. Der Vergleich wird niemals WAHR.

[ich (Gast)]

ich schrieb:
> Beim ersten groben Überfliegen würde ich schon mal diese Zeile
>> if(TimerCount == 100000)
> als Problem sehen. Der Vergleich wird niemals WAHR.

Da sehe ich ja noch viel mehr.
- Kein main()
- setup() wird niemals aufgerufen
- loop() wird niemals aufgerufen

Kurz gesagt, das ist kein vollständiges C-Programm.

[Jan M. (pietus3)]

Die main-Funktion ist Loop und Setup unter der Arduino-IDE main.

Warum wird TimerCount != 100000?

[ich (Gast)]

Jan Meyer schrieb:
> Die main-Funktion ist Loop und Setup unter der Arduino-IDE main.
>

Okay, mit Arduino kenne ich mich nicht aus, also halte ich da die Klappe 
:-)

> Warum wird TimerCount != 100000?

Schau dir mal den Variablen-Typ von TimerCount an.

[Jan M. (pietus3)]

Oh das ist bitter.

[Bitflüsterer (Gast)]

>Warum wird TimerCount != 100000?

Das geschieht an mehreren Stellen.

1. Bei der Definition der globalen Variablen wird mit 0 initialisiert.
2. Bei dem increment werden sich 99999 mal Werte ungleich 100000 
ergeben.
3. Nach dem Rücksetzen auf 0 ist die Variable notwendigerweise ungleich 
100000.

[Bitflüsterer (Gast)]

Ups. Das int ist mir durchgeschlüpft.

[ich (Gast)]

Jan Meyer schrieb:
> Oh das ist bitter.

Gehts denn jetzt?

[Jan M. (pietus3)]

ISR( TIMER0_COMPA_vect)                // Interruptbehandlungsroutine
{
  TimerCount++;
  if(TimerCount == 10000)
  {
    TimerCount2++;
     if(TimerCount2 == 10){ 
        if (LampeAN){
          LampeAN = false;    
          }
        else{
          LampeAN= true;
        }
      TimerCount2 = 0;
     }
     TimerCount = 0;
    
  }     
}

Hab das jetzt so abgeändert und nun läuft es.

Danke

[ich (Gast)]

Bitte :-)

[LostInMusic (Gast)]

        if (LampeAN){
          LampeAN = false;    
          }
        else{
          LampeAN= true;
        }

Wieso nicht einfach

    LampeAN = !LampeAN

?

[EGS_TI (Gast)]

Jan Meyer schrieb:
> Ich hab mir gedacht ich müsste nur
> die Register ändern und das wäre es.

Soso, gedacht....
Hier liegt vermutlich der Fehler. Wenn deine Annahme also nicht simmt, 
solltest du deine Vermutungen anhand des Datenblatts überprüfen.

[EGS_TI (Gast)]

LostInMusic schrieb:
> Wieso nicht einfach
>
>     LampeAN = !LampeAN
>
> ?

Wieso fragst du noch? Den Rest von ihm musst du doch auch schon gelesen 
haben.

[ich (Gast)]

EGS_TI schrieb:
> Jan Meyer schrieb:
>> Ich hab mir gedacht ich müsste nur
>> die Register ändern und das wäre es.
>
> Soso, gedacht....
> Hier liegt vermutlich der Fehler. Wenn deine Annahme also nicht simmt,
> solltest du deine Vermutungen anhand des Datenblatts überprüfen.

Du hast vermutlich die Posts überhaupt nicht gelesen, oder?
Was hat ein ungeeigneter Variablentyp mit dem Lesen eines Datenblattes 
zu tun?

Der Programmierstil ist wieder eine ganz andere Sache, daß man 
einfachere Lösungen findet usw. Aber nicht jeder ist als Meister vom 
Himmel gefallen.