Timer2 Atmega328

[Kai (Gast)]

Guten morgen,
ich habe ein Problem mit dem Timer2 bei einem Atmega328. Dieser hat 
einen 16MHz quarz angeschlossen.

TCCR0A & ~(1 << WGM00) | (1 << WGM01) & ~(1 << WGM02);
TCCR0B | (1 << CS00) | (1 << CS01) & ~(1 << CS02);
OCR0A = 250 - 1;
TIMSK0 | (1 << OCIE0A));

TCCR2A & ~(1 << WGM20) | (1 << WGM21) & ~(1 << WGM22);
TCCR2B & ~(1 << CS20) & ~(1 << CS21) | (1 << CS22);
OCR2A = 250-1;
TIMSK2 | (1 << OCIE2A));

void TIMER0_COMPA_vect(void)
{
    static uint16_t millis = 0;
    static uint8_t seconds = 0;
    static uint8_t minutes = 0;
    static uint8_t hours = 0;
    millis++;
    if(millis >= 1000)
    {
        //PORTB ^= ( 1 << PORTB5 );
        seconds++;
        millis = 0;
        if(seconds >= 10 )
        {
            minutes++;
            seconds = 0;
            if(minutes >= 10)
            {
                hours++;
                minutes = 0;
                if(hours >= 10)
                {
                    hours = 0;
                }
            }
        }
    }
}


void TIMER2_COMPA_vect(void)
{
    static uint16_t millis = 0;
    static uint8_t seconds = 0;
    millis++;
    if(millis >= 1000)
    {
        PORTB ^= ( 1 << PORTB5 );
        seconds++;
        millis = 0;
    }
}

eigentlich sollten so beide Timer gleich laufen. Wenn ich zur Probe eine 
LED in Timer0 blinken lasse, blinkt diese auch im Sekundentakt. Aber in 
der ISR vom Timer2 wird die LED nicht getoggelt. Hier bleibt diese aus 
(habe dazu natürlich das Togglen aus der ISR vom Timer0 herausgenommen).

Nach Beitrag "Atmega328p Timer2 will nicht" liegt es daran 
dass der Timer2 zwei Teiler mehr hat als Timer0. Aber im Datenblatt 
finde ich für Teiler nur das Register TCCR2B.

Wo finde ich den zweiten?

Gruß Kai

[Bastian W. (jackfrost)]

Du initialisierst die CS20 - 21 Bits mit null , da läuft der Timer ja 
nicht.
Du musst das ja wie bei TCRR0B machen mit |= 1<<CS2x

Wenn du mit dem Atmel Studio schreibst , simulier es , halte die CPU an 
und schau dir die Register an, dann siehst du ob die Initialisierung 
passt.

Gruß JackFrost

[Kai (Gast)]

Bastian W. schrieb:
> Du initialisierst die CS20 - 21 Bits mit null , da läuft der Timer ja
> nicht.

Aber ich setze doch CS22 auf 1
(1 << CS22) -> clkT2S/64 (From prescaler)

[sing sang song (Gast)]

Kai schrieb:
> [code]
> TCCR0A & ~(1 << WGM00) | (1 << WGM01) & ~(1 << WGM02);
> TCCR2A & ~(1 << WGM20) | (1 << WGM21) & ~(1 << WGM22);
> [code]

WGM02 und WGM22 ist schon mal Mist. Man sollte schon schauen, wo sich 
die Bits befinden. Wenn es auch nicht das Problem sein wird.

[Kai (Gast)]

Bzw. Die Initialierung muss anders sein

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

TCCR2A |= (1 << WGM21);
TCCR2B |= (1 << CS22);
OCR2A = 250-1;
TIMSK2 |= (1 << OCIE2A));

Im obigen Beispiel wurden die Register gar nicht gesetzt. Jetzt schreibe 
ich nur die Bits auf, die ich auch setzen möchte. Allerdings geht es 
immer noch nciht.

[Bastian W. (jackfrost)]

Kai schrieb:
> Bastian W. schrieb:
>> Du initialisierst die CS20 - 21 Bits mit null , da läuft der Timer ja
>> nicht.
>
> Aber ich setze doch CS22 auf 1
> (1 << CS22) -> clkT2S/64 (From prescaler)

Nein , mit deiner Zeile

TCCR2B & ~(1 << CS20) & ~(1 << CS21) | (1 << CS22);

Verundest du das Register mit dem Bit CS22. Das Ergebnis ist 0 da CS22 
im Register nicht gesetzt ist.

Ist die Zeile eine copy Paste Zeile von der darüber ?

[Bastian W. (jackfrost)]

Schau mal im Simulator ob der Timer zählt.

[Kai (Gast)]

Bastian W. schrieb:
> enn du mit dem Atmel Studio schreibst , simulier es , halte die CPU an
> und schau dir die Register an, dann siehst du ob die Initialisierung
> passt.

Das kann ich leider nicht machen. Der Atmega ist auf einem Arduino und 
ich Flashe diesen über USB. Habe versucht, den Mikrocontroller mit 
meinem Diamex-ProgS zu flashen, bekomme diesen aber nicht zu laufen.

Habe als Selected debugger STK500 mit dem Interface: ISP gewählt.
Bekomme aber den Error
Failed to open \\.\COM4. Error 0x2

Im Gerätemanager taucht dieser unter USB-Geräte auf. Bei Speicherort 
steht Port_#0001.Hub_#0004
Desweiteren ist die LED vom Programmer die ganze Zeit am Blinken

[Bastian W. (jackfrost)]

Kai schrieb:
> TIMSK0 |= (1 << OCIE0A));
>TIMSK2 |= (1 << OCIE2A));

Bei beiden Zeilen ist eine schließende Klammer zu viel. Compiliert die 
Arduion IDE dann überhaupt ?

Mit diesem Programm laufen beide Timer mit dem Simulator

#define F_CPU 8000000Ul

#include <avr/io.h>
#include <avr/interrupt.h>
#include <string.h>
#include <util/delay.h>
#include <stdlib.h>




ISR(TIMER0_COMPA_vect)
{
  static uint16_t millis = 0;
  static uint8_t seconds = 0;
  static uint8_t minutes = 0;
  static uint8_t hours = 0;
  millis++;
  if(millis >= 1000)
  {
    //PORTB ^= ( 1 << PORTB5 );
    seconds++;
    millis = 0;
    if(seconds >= 10 )
    {
      minutes++;
      seconds = 0;
      if(minutes >= 10)
      {
        hours++;
        minutes = 0;
        if(hours >= 10)
        {
          hours = 0;
        }
      }
    }
  }
}


ISR(TIMER2_COMPA_vect)
{
  static uint16_t millis = 0;
  static uint8_t seconds = 0;
  millis++;
  if(millis >= 1000)
  {
    PORTB ^= ( 1 << PORTB5 );
    seconds++;
    millis = 0;
  }
}

int main()
{
  TCCR0A |= (1 << WGM01);
  TCCR0B |= (1 << CS00) | (1 << CS01);
  OCR0A = 250 - 1;
  TIMSK0 |= (1 << OCIE0A);

  TCCR2A |= (1 << WGM21);
  TCCR2B |= (1 << CS22);
  OCR2A = 250-1;
  TIMSK2 |= (1 << OCIE2A);
  sei();
  while(1)
  {
    
  }

}

Gruß JackFrost

[Karl M. (Gast)]

Guten Morgen und ein gutes neues Jahr.

Es wurde schon versucht "mit Hilfe zur Selbsthilfe" dir weiter zu 
helfen.

Manchmal sieht man den "Wald vor lauter Bäumen nicht", so lautet ein 
Sprichwort.
Es ist dir nicht aufgefallen, dass z.B. diese Zeile nichts an TCCR0A 
ändert !

TCCR0A & ~(1 << WGM00) | (1 << WGM01) & ~(1 << WGM02)

Register löschen

TCCR0A = 0; // clean up

Bit setzen:

TCCR0A = TCCR0A | (1 << WGM00)

Bit löschen:

TCCR0A = TCCR0A & (uint8_t)~(1 << WGM00);

ps. ich habe bewusst die Anweisung in der Zeile nicht in der 
Kurzschreibweise geschrieben.

TCCR0A |= (1 << WGM00)

TCCR0A &= (uint8_t)~(1 << WGM00);

[Kai (Gast)]

Bastian W. schrieb:
>> TIMSK0 |= (1 << OCIE0A));
>>TIMSK2 |= (1 << OCIE2A));

Das kam daher, dass ich den Code aus meinem Ersten Post kopiert und 
angepasst habe. In AtmelStudio waren die Klammern weg.

Bastian W. schrieb:
> Bei beiden Zeilen ist eine schließende Klammer zu viel. Compiliert die
> Arduion IDE dann überhaupt

Ich Programmiere in AtmelStudio. Aber auch hier wäre es zu einem 
Compiler-Fehler gekommen.

Karl M. schrieb:
> Es ist dir nicht aufgefallen, dass z.B. diese Zeile nichts an TCCR0A
> ändert !

Daher habe ich das ja nochmal umgeschrieben.


Allerdings bekomme ich den Controller immer noch nicht über den 
Programmer geflasht. Das COM-Problem habe ich gelöst, allerdings bekomme 
ich jetzt den Error
Failed to launch program.
Error: Failed to start programming session before chip erase witz eeprom 
preserve: Failed to enter programming mode. ispEnterProgMode: Error 
status received: Got 0xc0, expected 0x00 (Command has failed to execute 
on the tool)

[Bastian W. (jackfrost)]

Wenn du im Atmelstudio schreibst,kannst du doch simulieren. Das hat 
nichts mit dem Programm zu tun. Schau mal ob dein Programm in der 
Simulation überhaupt läuft. Welches Studio verwendest du ? 4.19 , 6.x 
oder 7.x ?

Gruß JackFrost

[Kai (Gast)]

Bastian W. schrieb:
> Welches Studio verwendest du ? 4.19 , 6.x
> oder 7.x ?

Ich benutze Version 7.0.943

[Kai (Gast)]

Das mit der Simulation kannte ich so noch gar nicht.
Das Problem ist, TCCR2B gar nicht gesetzt wird.

TCCR2B |= (1 << CS22);

Alle anderen werden gesetzt. Wenn ich das Bit dann nachträglich setze, 
wird auch in die ISR gesprungen.

[Bastian W. (jackfrost)]

Poste mal den ganzen Code deiner main.c als Datei hier um Forum ?
Bei meinem Beispiel ist das Bit gesetzt.

Gruß JackFrost