uint32_t zähler und vergleiche

[Jürgen S. (jsachs)]

Hallo,

bin eben beim testen darüber gestolpert, dass folgendes nicht ging 
(teilcode):

int main()
{
uint32_t count=0;

while(1)
{
  count++;
  if (count > 0x3ffff)
  {
    //tue was
    //toggle LED
    count=0;
  }
  // tue was anderes
}
}

ich hatte in der if eine LED die getoggelt wurde, diese ging nicht mehr 
ab einem Wertebereich von 0xffff.
Nur wieso ?

Ich habe das im Moment in 2 uint16_t Zähler gesplittet und es geht 
prima, nur wieso ging es nicht mit dem uint32_t ?

Bevor der Hinweis kommt. Interrupt gehen keine mehr, da ich per Software 
PWM Servoimpulse erzeuge, daher will ich es über solche Zähler lösen und 
unkritische Teile in main machen.

[2ter Gast (Gast)]

Jürgen Sachs schrieb:
> Ich habe das im Moment in 2 uint16_t Zähler gesplittet und es geht
> prima, nur wieso ging es nicht mit dem uint32_t ?

Wie sieht der Code denn aus? So kann man doch nur raten...

[Jürgen S. (jsachs)]

Die Frage verstehe ich nicht.
Das IST der Code.

Es fehlt nur das Setup des Ports
DDRB=0xff;

und das blinken der LED
PORTB^=_BV(PB0)

Ich habe diesen Code nicht mehr, da es nur zu Testzwecken war, und ich 
schon VIEL weiter bin. Mich interessiert das WARUM.

Wie gesagt, nachdem ich den uint32 durch 2 uint16 ersetzt habe, ging 
alles

int main()
{
uint16_t count1=0;
uint16_t count2=0;

while(1)
{
  count1++;
  if (count1 > 0xfe)
  {
    count1=0;
    count2++;
  }

  if (count2 > 0x07)
  {
    //tue was
    //toggle LED
    count2=0;
    count1=0;
  }
  // tue was anderes
}
}

Ich nehme daher an das hat was mit dem 32 Bit Datentyp zu tun. Nun habe 
ich gehofft, das die Wissenden eine Erklärung haben.

[Helfer (Gast)]

Der Code aus Beitrag "uint32_t zähler und vergleiche" kann 
von einem "agressiven" Optimierer komplett ersetzt werden durch diesen 
funktionsidentischen Code

int main(void)
{
  while(1) {}
}

Wenn du allerdings die Portschalterei hinzunimmst, kann der Optimierer 
das nicht mehr so vereinfachen. Deshalb ist die Frage nach dem exakten 
Code wichtig.

Hast du schon die Variante getestet, bei der die beiden Argumente des 
Vergleichs vom gleichen Datentyp sind?

int main()
{
  uint32_t count = 0;

  while(1)
  {
    count++;
    //                 vv
    if (count > 0x3ffffUL)
    {
      //tue was
      //toggle LED
      count = 0;
    }
    // tue was anderes
  }
}

[2ter Gast (Gast)]

Jürgen Sachs schrieb:
> Die Frage verstehe ich nicht.
> Das IST der Code.

So wie dein Code aussieht, toggelt da nichts.

Jürgen Sachs schrieb:
> Ich nehme daher an das hat was mit dem 32 Bit Datentyp zu tun. Nun habe
> ich gehofft, das die Wissenden eine Erklärung haben.
Nein. Es sei denn, der Compiler ist defekt. Es ist ein leichtes für dich 
das Programm mit dem Simulator zu prüfen.

[Karl H. (kbuchegg)]

Jürgen Sachs schrieb:

> ich gehofft, das die Wissenden eine Erklärung haben.

Es gibt keine spezielle.
Du hast irgendwo einen Fehler gemacht, denn du im Urpsrungsposting nicht 
gezeigt hast. Aus dem was du gezeigt hast, lässt sich kein Fehlerfall 
ablesen. Es gibt keinen Grund, warum der Zähler bei 32 Bit nicht zählen 
sollte.

[Jürgen S. (jsachs)]

Simulierer kann ich nicht. Ich arbeite unter Linux und da ist das etwas 
"kompliziert". Ich habe es jedenfalls noch nicht hin bekommen..

Ich habe nochmal versucht das nach zustellen. Da ich den Ursprungscode 
nicht mehr habe, durch neu programmieren.

#include <avr/io.h>

int main()
{
  uint32_t cnt = 0;
  DDRC = 0xFF;
  while(1)
  {
    cnt++;
    if (cnt > 0x3fffe)
    {
      PORTC ^= _BV(PC7);
      
      cnt = 0;
    }
  }
}

Und es tut wie erwartet.
Das Fuchst mich jetzt richtig. Über eine Stunde verbraten und dann aus 
Verzweiflung in unit16 geändert und tut.
Das nächste mal mache ich ein commit ins svn bevor ich das verändere.
Eventuell liegt es auch an der anderen Hardware (schnellerer Takt). Ich 
bin am grübeln ob ich die Zeit für das Zählen nicht unterschätzt habe 
(Ungeduld)

Ich danke Euch soweit.

Sorry
Juergen