timer am avr startet nicht oder es wird nicht in die ISR gesprungen

[Kai (Gast)]

Hallo,
eigentlich wollte ich mir gestern ein DCF77-Modul besorg haben, aber es 
war keins mehr da. Daher würde ich dann jetzt mit dem Timer des 
Controllers anfangen.

#define F_CPU 16000000UL

#include <avr/io.h>
#include <avr/interrupt.h>

int main(void)
{
  DDRB |=  (1 << PB2);
  PORTB |= (1 << PB2);

  TCCR0A |= (1 << WGM01);
  TCCR0B |= (1 << CS00) | (1 << CS01);
  OCR0A = 250-1;
  TIMSK0 |= (1 << OCIE01);
  sei();

  while(1)
  {}
}

ISR(TIMER0_COMPA_vect)
{
    static uint8_t millis = 0;
    static uint8_t seconds = 0;
    static uint8_t minutes = 0;
    static uint8_t hours = 0;
    millis++;
    if(millis >= 1000 )
    {
        PORTB ^= ( 1 << PB2 );
        seconds++;
        millis = 0;
        if(seconds >= 60 )
        {
            minutes++;
            seconds = 0;
            if(minutes >= 60)
            {
                hours++;
                minutes = 0;
                if(hours >= 12)
                {
                    hours = 0;
                }
            }
        }
    }
}

habe einen Quarz mit 16Mhz. Das heisst, alle millisekunde sollte in die 
ISR gesprungen werden

Allerdings bleibt die LED immer am leuchten (noch soll diese erstmal 
blinken).

[Dietrich L. (dietrichl)]

Kai schrieb:
> static uint8_t millis = 0;
> ISR(TIMER0_COMPA_vect)
> {
>     static uint8_t seconds = 0;
>     static uint8_t minutes = 0;
>     static uint8_t hours = 0;
>     millis++;

Du löschst jedesmal bei Eintritt in die ISR die Variablen neu!

[Kai (Gast)]

Dietrich L. schrieb:
> Kai schrieb:
>> static uint8_t millis = 0;
>> ISR(TIMER0_COMPA_vect)
>> {
>>     static uint8_t seconds = 0;
>>     static uint8_t minutes = 0;
>>     static uint8_t hours = 0;
>>     millis++;
>
> Du löschst jedesmal bei Eintritt in die ISR die Variablen neu!

aber deswegen habe ich doch static, die werden doch nur einmalig mit 
null beschrieben.

Der fehler war, dass ich 1000 millis haben wollte, diese aber nur mit 
uint8_t deklariert habe
-> 1000 werden nie erreicht

[Dietrich L. (dietrichl)]

Kai schrieb:
>> Du löschst jedesmal bei Eintritt in die ISR die Variablen neu!
>
> aber deswegen habe ich doch static, die werden doch nur einmalig mit
> null beschrieben.

Wenn das so wirklich ist ziehe ich meine Aussage zurück. Ich bin kein 
C-Fachmann... ;-(

[Peter II (Gast)]

  static uint8_t millis = 0;
  millis++;
  if(millis >= 1000 )

fällt dir nichts auf? Vermutlich gibt es sogar eine Warnung.

in 8 bit passt keine 1000 rein.

[Dietrich L. (dietrichl)]

Peter II schrieb im Beitrag #4824155 um 09:40:
> fällt dir nichts auf? Vermutlich gibt es sogar eine Warnung.
>
> in 8 bit passt keine 1000 rein.

Kai schrieb im Beitrag #4824148 um 9:21:
> Der fehler war, dass ich 1000 millis haben wollte, diese aber nur mit
> uint8_t deklariert habe
> -> 1000 werden nie erreicht

Kai war selber viel schneller ;-)

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Da du allerdings deine Uhrzeit Bytes lokal in der ISR hältst, kommst du 
von aussen nicht ran, um sie z.B. anzuzeigen oder z.B. einzustellen. 
Erkläre sie besser als globale volatile Bytes ausserhalb der ISR, damit 
auch andere Programmteile drauf zugreifen können.

Die millis können ja lokal in der ISR bleiben.