c++ Variable zählt nich hoch! Warum?

[Gert B. (gbirkner)]

Der Timer ISR wird alle 1ms aufgerufen und führt die entsprechende 
inline Methode auf.
Das funktioniert auch, aber aus irgendeinem Grund wird die Variable 
tcADCPower nicht hochgezählt und dementsprechend wird

if(tcADPower >= 10)

 nie wahr.

In der Klasse RobotMotor wird die Variable erstellt und im Konstruktor 
auf 0 gesetzt.
Die ISR ruft dann folgende private inline Methode auf:

void RobotMotor::isrTimer0COMPA()
{
  PORTD ^= (1<<PORTD3);
  PORTD |= (1 << PORTD2);
  tcADCPower++;
  
  if (tcADCPower >= 10)
  {
    PORTD &= ~(1<<PORTD2);
    tcADCPower = 0;
  }
}

PORTD3 wird alle 1ms getoggelt, PORTD2 bleibt aber immer gesetzt, sollte 
aber nach 10 Durchgängen immer wieder für 1ms gelöscht werden.
Ich weiß, der Code macht nicht viel Sinn, aber in der if sollte dann 
alle 10ms etwas "sinnvolles" ausgeführt werden.
Ich kontrolliere das mit dem Oszi, habe auch schon die Pins vertauscht 
usw. und komme in das if nicht hinein!
An der ISR kann es nicht direkt liegen, da ja PORTD3 brav getoggelt wird 
und wenn ich PORTD2 toggle wird das auch in der gewünschten Frequenz 
gemacht.
Wo kann hier das Problem liegen?
(Atmel Studio 6.2; Atmega328)

[Peter II (Gast)]

Gert Birkner schrieb:
> Wo kann hier das Problem liegen?

variable nicht volatile.

[Gert B. (gbirkner)]

Peter II schrieb:
> Gert Birkner schrieb:
>> Wo kann hier das Problem liegen?
>
> variable nicht volatile.
hab ich schon ausprobiert, hilft aber nichts.
Ich habe auch die variable global definiert (anstatt in der Klasse) und 
auch static in der Methode -> hat alles nichts geholfen. Kompiliert 
wurde es immer fehlerlos.

[Peter II (Gast)]

Gert Birkner schrieb:
> hab ich schon ausprobiert, hilft aber nichts.

dann bitte Quellcode zeigen, sonst wird wohl keiner helfen können.

[Martin (Gast)]

Und wenn du es so versuchst... den PD2 alle 10 ms toggeln?

void RobotMotor::isrTimer0COMPA()
{
  PORTD ^= (1<<PORTD3);
  
  if (++tcADCPower >= 10)
  {
    PORTD ^= (1<<PORTD2);
    tcADCPower = 0;
  }
}

[Gert B. (gbirkner)]

OK, die Klasse in der .h Datei sieht im Wesentlichen so aus:

extern "C" void TIMER0_COMPA_vect(void) __attribute__ ((signal));

class RobotMotor
{
//variables
private:
   uint16_t tcADCPower;  // Zähler für Timerverzögerung

   friend void TIMER0_COMPA_vect();
   inline void isrTimer0COMPA();

public:
  RobotMotor(); 

};

extern RobotMotor rmotor

die Implementierung in der .cpp:

#include "RobotMotor.h"

RobotMotor rmotor;

// default constructor
RobotMotor::RobotMotor()
{
   tcADCPower = 0;
   // .. Timer0 initialisierungen
}

void TIMER0_COMPA_vect(void)
{
  rmotor.isrTimer0COMPA();
}

void RobotMotor::isrTimer0COMPA()
{
  PORTD ^= (1<<PORTD3);
  PORTD |= (1 << PORTD2);
  tcADCPower++;
  
  if (tcADCPower >= 10)
  {
    PORTD &= ~(1<<PORTD2);
    tcADCPower = 0;
  }
}

Also: die isrTimer0COMPA wird ordnungsgemaess ausgeführt. PORTD3 
getoggelt und PORTD2 gesetzt. aber irgendwie wird der code in der if nie 
ausgeführt.

Danke für die schnellen Reaktionen
gert

[Gert B. (gbirkner)]

Martin schrieb:
> Und wenn du es so versuchst... den PD2 alle 10 ms toggeln?
>
>

> void RobotMotor::isrTimer0COMPA()
> {
>   PORTD ^= (1<<PORTD3);
> 
>   if (++tcADCPower >= 10)
>   {
>     PORTD ^= (1<<PORTD2);
>     tcADCPower = 0;
>   }
> }
>

Das ändert leider nichts!

[Gert B. (gbirkner)]

Problem teilweise gelöst!
Ich habe ein neues Projekt erstellt und es funktioniert. Ich habe keine 
Ahnung warum!
Das muss ich erst herausfinden.

Kann es sein, wenn in einer Solution mehrere Projekte erstellt werden, 
diese sich gegenseitig beeinflussen (stören)? - obwohl keine 
Querverweise vorhanden sind!

Danke für die Hilfe

gert

[Christian (Gast)]

Hallo,
ich hatte auch mal ein ähnliches Problem. Die Uhrsache war, dass das 
Compilieren/Erstellen die .hex Datei nicht überschrieben hat. War bei 
mir so im AVR Studio. Einfach .hex Datei manuell löschen und neu 
erstellen bzw. mal ausprobieren auf Release/Debug als Ausgabepfad 
umstellen.

MfG

Christian

[Gert B. (gbirkner)]

Das Problem habe ich auch schon vermutet, daher kontrolliere ich beim 
Überspielen der hex Datei immer die Uhrzeit der Erstellung.

Ich konnte aber jetzt folgendes feststellen:
Im Programm wird noch der Timer1 als fast PWM verwendet, der immer beim 
Erreichen von ICR1 seinen nächsten OCR1A und OCR1B berechnet. Schalte 
ich den per Prescaler aus, zählt meine Variable ordnungsgemäß hoch, 
sonst nicht.
Hier ist sich irgendwas im Weg.

[Norbert B. (norbert_b)]

uint8_t zaehler = 0;

static inline void initTimer1(void) { //Minutenzähler
  /* Normal mode (default), just counting */

  TCCR1A |= (1 << COM1A0);  //Toggle OCA1A oder PB1
  TCCR1B |= (1 << WGM12);   // CTC-Mode (Clear Timer on Compare Match)
  TCCR1B |= (1 << CS12) | (1 << CS10); /* Clock speed: 1 MHz / 1024,
                    each tick is 976,6 microseconds ~= 1 per ms  */
  TIMSK1 |= (1 << OCIE1A);  //Enable Interupt
  OCR1A = 1000;      // 0xE679 Tics entsprechen 1 Minute 
}

//ISR(TIMER1_COMPA_vect) {  //Zähler inkrementieren wenn OCR1A matches
//  zaehler++;
//}

static inline void initTimer0(void) { //Stundenzähler
/* Normal mode (default), just counting */
  TCCR0A |= (1 << COM0A0) | (1 << WGM01);  //Toggle OCA0A oder PD6/12 |  CTC-Mode (Clear Timer on Compare Match)
  TCCR0B |= (1 << CS02) | (1 << CS01); // External Clock on T0/PB4/6,
  TIMSK0 |= (1 << OCIE0A);  //Output Compare Match A Interrupt Enable

  OCR0A = 10                 ;      // 60 Tics entsprechen 1h 
}

ISR(TIMER0_COMPA_vect) {  //Zähler inkrementieren wenn OCR0A matches
        zaehler++;  
  LED_PORT = 0b00000100;
}

int main(void)
{
  DDRB = 0b00000111; //PB0 - PB2 als Output
  DDRD = 0b01000000; //PD6 als Output

  initTimer1(); //zählt bis 60 Sekunden dann Toggle auf PD6
  initTimer0(); //wird von PB1 getaktet (anPB4)
        initUSART_new(9600, 8, 'N');
  sei(); /* set (global) interrupt enable bit */
  
  while(1) 
  {

    if (zaehler > 5)
    {
      //cli();
      LED_PORT = 0b00000100;
      _delay_ms(1000);
      LED_PORT = 0b00000001;
      printString("zahler > 60 ");
      transmitByte(zaehler);
      zaehler = 0;
      //sei();
    }    
    _delay_ms(1000);
  }
}

Die ISR(TIMER1_COMPA_vect) zählt zaehler problemlos hoch, mit der 
ISR(TIMER0_COMPA_vect) ist die Variable zaehler immer 0 und das obwohl 
PB2 high wird. Hat irgend jemand eine Idee?
Programm läuft auf einem ATMEGA168PA.

mfg
ncb

[Georg G. (df2au)]

Norbert B. schrieb:
> Hat irgend jemand eine Idee?

"volatile" ist das Suchwort für dich.

volatile uint8_t zaehler = 0;

Der Interrupt zählt schon hoch, nur in main() wird das nicht bemerkt.

[Lutz H. (luhe)]

Ungefähr 62.900.000 Ergebnisse (0,42 Sekunden) :-)

[Norbert B. (norbert_b)]

Georg G. schrieb:
> Norbert B. schrieb:
>> Hat irgend jemand eine Idee?
>
> "volatile" ist das Suchwort für dich.
>
> volatile uint8_t zaehler = 0;
>
> Der Interrupt zählt schon hoch, nur in main() wird das nicht bemerkt.

Jooo.., sorry und ich habe mindestens 1 Stunde draufgeschaut und es 
nicht gesehen. Zu meiner Entschuldigung, irritierend war, das der Timer 
1 ein Ergebnis brachte.