Attiny2313, Timer 1 Starten, 1Minute, Fehler: left-hand operand of comma expression has no effect

[Matze (Gast)]

Hallo,

Ich möchte den Timer 1 (16Bit) eines Attiny 2313 benutzen um ca eine 
Minute zu messen.

Danach möchte ich die Ausgänge abschalten...

Nun bekomme ich oft den Fehler:
left-hand operand of comma expression has no effect

Dabei muss ich doch den Timer richtig einstellen.
Hier mein Programm:

#include <avr/io.h>
#define F_CPU 1000000UL    // 1 MHz (fuer delay.h)
#include <util/delay.h>


int main(void)
{
  PORTD = 0x00;      //Pin für Taster und ADC(Reset) als Eingang
  DDRD  = 0xff;
  PORTB = 0x00;
  DDRB  = 0x00;
  //-------TIMER-1----------------------------------
  TCCR1A  &= (0<<COM1A0,0<<COM1A1);  // CTC Modus
  TCCR0B &= (0<<ICNC1,0<<ICES1,0<<CS11);
  TCCR0B |= (1<<WGM12,1<<CS10,1<<CS12);  // Prescaler 1024
  OCR1A = 0x0400;                    //Zählen bis 1024;
  //------> Timer läuft nun
  int lauft=0;
    while(1)
    { 
          lauft = TIFR&(1<<OCF1A);
          if(lauft)
          {
            PORTD=0x00;
            while(1)
            {
              ;
            }
            TIFR |= (1<<OCF1A);
          }
          else
          {
            PORTD=0x55
          }      

    }
}

Da PORTD immer auf 0x55 geht, läuft der Timer wohl nicht.

Ich komm nicht auf den Fehler.

Danke dür eure Hilfe.

[Karl H. (kbuchegg)]

Matze schrieb:

> Nun bekomme ich oft den Fehler:
> left-hand operand of comma expression has no effect
>
> Dabei muss ich doch den Timer richtig einstellen.

Schon.
Aber nicht so

TCCR0B |= (1<<WGM12,1<<CS10,1<<CS12);

sondern so

TCCR0B |= (1<<WGM12) | (1<<CS10 ) | (1<<CS12);

und natürlich die anderen auch.

[Marc (gierig)]

Matze schrieb:
> TCCR1A  &= (0<<COM1A0,0<<COM1A1);  // CTC Modus
>   TCCR0B &= (0<<ICNC1,0<<ICES1,0<<CS11);
>   TCCR0B |= (1<<WGM12,1<<CS10,1<<CS12);  // Prescaler 1024
>   OCR1A = 0x0400;                    //Zählen bis 1024;


Was den das ? Das ist kompletter müll und kein C
Die einzelnen bits kannst du nicht einfach mit nem Komma trennen.

http://www.mikrocontroller.net/articles/Bitmanipulation#Standard_C
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Ver.C3.A4ndern_von_Registerinhalten

kennst du ? Da stehts wie du Bitmanipulationen richtig machst.

[Matze (Gast)]

Danke, die fehler sind nun weg.

Der Timer läuft wohl immer noch nicht, muss ich ihn irgendwie speziell 
starten?

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 1000000UL    // 1 MHz (fuer delay.h)
#include <util/delay.h>

int main(void)
{
  PORTD = 0x00;      //Pin für Taster und ADC(Reset) als Eingang
  DDRD  = 0xff;
  PORTB = 0x00;
  DDRB  = 0x00;
  //-------TIMER-1----------------------------------
  TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
  TCCR0B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024
  TCCR0B |= (1<<WGM12)|(1<<CS10)|(1<<CS12);  
  OCR1A = 0x0400;
  TIMSK |= (1<<OCIE1A);            //Compare Interrupt Enable
  sei();                    //Globale Interrupts ein
  //------> Timer läuft nun
  int lauft=0;
    while(1)
    {   
    lauft = TIFR&(1<<OCF1A);
    if(lauft)
    {
      PORTD=0x00;
      while(1)
      {
        ;
      }
      TIFR |= (1<<OCF1A);
    }
    else
    {
      PORTD=0x55;
    }      
    }
}

ISR(TIMER1_COMPA_vect) 
{
  PORTD=0x00;
  while(1)
  {
    ;
  }
}

Nun inst er hoffentlich auf Compare eingestellt.
Nach allem was ich gelesen habe müsste er bei einem Compare-Match das 
Bit OCF1A setzen, und einen Interrupt auslösen.

Der Interrupt soll durch

  TIMSK |= (1<<OCIE1A);            //Compare Interrupt Enable
  sei();                    //Globale Interrupts ein

ermöglicht werden.

Leider löst er nicht aus und ich kann die Minute so nicht messen.

Grüße
Matze

[Matze (Gast)]

Habe den Fehler gefunden.

Ich nutze Timer 1 und stelle das Kontroll-Register von Timer 0 ein.

  TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
  TCCR1B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024
  TCCR1B |= (1<<WGM12)|(1<<CS10)|(1<<CS12);

Grüße
Matze

[Peter II (Gast)]

Matze schrieb:
> TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
>   TCCR1B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024

das ist zwar kein fehler, aber unsinn. da kannst du auch gleich

TCCR1A  = 0;
TCCR1B  = 0;

schreiben.

[Marc (gierig)]

Peter II schrieb:
> as ist zwar kein fehler, aber unsinn. da kannst du auch gleich
>
> TCCR1A  = 0;
> TCCR1B  = 0;


Dann kannst du die auch gleich komplett weglassen, da nach einem Reset 
diese
IO register eh auf Null sind. (jedenfalls beim dem tiny 2313)

[Karl H. (kbuchegg)]

Die while Schleife

ISR(TIMER1_COMPA_vect) 
{
  PORTD=0x00;
  while(1)
  {
    ;
  }
}

kommt da aber jetzt mal ganz schnell raus.

[Karl H. (kbuchegg)]

Es ist ziemlich unwahrscheinlich, dass dein Programm ausgerechnet hier

    while(1)
    {   
    lauft = TIFR&(1<<OCF1A);
    if(lauft)
    {
      PORTD=0x00;
      while(1)
      {
        ;
      }
      TIFR |= (1<<OCF1A);
    }
    else
    {
      PORTD=0x55;
    }      
    }

das OCF1A Flag als gesetzt antreffen wird. Sobald es vom Timer gesetzt 
wird, erfolgt der Aufruf der ISR innerhalb der nächsten 2 bis 3 
Taktzyklen, je nachdem welchen Befehl die CPU gerade in Arbeit hat. Dann 
passieren 2 Dinge: zum einen wird dadurch genau dieses Flag sofort 
wieder gelöscht, so dass die Hauptschleife es nicht zu Gesicht kriegen 
kann und zum zweiten hältst du das Programm durch die Endlosschleife 
innerhalb der ISR fest, so dass die Hauptschleife schon alleine aus 
diesem Grund nicht weiter arbeiten kann.


Und ich seh gerade, dass du da schon wieder eine Endlosschleife 
reingeschachtelt hast. Mit Endlosschleifen scheinst du es ganz im 
Speziellen zu haben. Lass den Unsinn! Es gibt nur EINE Endlosschleife 
und das ist die eine in main()!

[Karl H. (kbuchegg)]

machs doch nicht so kompliziert

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 1000000UL    // 1 MHz (fuer delay.h)
#include <util/delay.h>


volatile uint8_t tick;

int main(void)
{
  PORTD = 0x00;      //Pin für Taster und ADC(Reset) als Eingang
  DDRD  = 0xff;

  PORTB = 0x00;
  DDRB  = 0x00;

  //-------TIMER-1----------------------------------
  TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
  TCCR0B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024
  TCCR0B |= (1<<WGM12)|(1<<CS10)|(1<<CS12);  
  OCR1A = 0x0400;
  TIMSK |= (1<<OCIE1A);            //Compare Interrupt Enable
  sei();                    //Globale Interrupts ein
  //------> Timer läuft nun

  while(1)
  {   
    if( tick )
    {
      tick = 0;

      PORTD ^= 0x55;
    }
  }
}

ISR(TIMER1_COMPA_vect) 
{
  tick = 1;
}

Wenn deine Timereinstellungen stimmen, was ich nicht kontrolliert habe, 
dann müssten deine LED jetzt im Sekundentakt blinken.

[Karl H. (kbuchegg)]

Wie kommst du eigentlichg auf die Idee, dass dein Programm im 1 
Sekundenrhythmus die ISR aufruft? Wenn ich nachrechne, krieg ich da aber 
was ganz anderes raus. Mal abgesehen davon, dass deine 1Mhz vom internen 
Takt keine 1Mhz sein werden und die ganze Rechnerei schon alleine 
deswegen nicht stimmt. Oder hast du einen 1Mhz QUarz am Tiny hängen?

FAQ: Timer

[Karl H. (kbuchegg)]

Und ehe du da jetzt noch weiter rumhampelst:
Mal ausgehend davon, dass du den OCR1A Wert soweit korrigierst, dass du 
leidlich genau 1 Sekunden Abstände im ISR Aufruf hast, kann man Zeiten 
zb so abmessen, indem man einen Countdown-Zähler rückwärts zählen lässt, 
bis er bei 0 ist

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 1000000UL    // 1 MHz (fuer delay.h)
#include <util/delay.h>


volatile uint8_t tick;

int main(void)
{
  PORTD = 0x00;      //Pin für Taster und ADC(Reset) als Eingang
  DDRD  = 0xff;

  PORTB = 0x00;
  DDRB  = 0x00;

  //-------TIMER-1----------------------------------
  TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
  TCCR0B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024
  TCCR0B |= (1<<WGM12)|(1<<CS10)|(1<<CS12);  
  OCR1A = 0x0400;
  TIMSK |= (1<<OCIE1A);            //Compare Interrupt Enable

  sei();                    //Globale Interrupts ein
  //------> Timer läuft nun

  tick = 60;                // 60 Sekunden Countdown

  while(1)
  {   
    if( tick > 0 )          // läuft der Countdown noch?
      PORTD = 0x55;         //   Yep -> Led ein
    else
      PORTD = 0;            //   No -> Countdown ist fertig, Led aus
  }
}

ISR(TIMER1_COMPA_vect) 
{
  if( tick > 0 )
    tick--;
}