Probleme mit Switch Case

[Sebastian S. (sebastian_s25)]

Hallo

Hab da mal ne Frage. Kann es sein, dass man beim avr eine 
Switchanweisung so nicht programmieren kann?

switch(UDR0)
  {
    case 0xD0 :      // Set Retry
    case 0xD1 :      // Set RF Power
    case 0xD2 :      // Set Sensing Time
    case 0xE0 :      // Set PWM1 compare
    case 0xE1 :      // Set PWM2 compare
    case 0xE2 :      // Set PWM3 compare
    case 0xE3 :      // Set PWM4 compare
            Flag = UDR0;
      break;

    default  :
      break;
  }

Irgendwie bekomm ich da immer nur sch... raus...

mfg
Bastian

[avr (Gast)]

Wie ist UDR0 definiert?

[Sebastian S. (sebastian_s25)]

Ah sorry hatte ich vergessen zu erwaehnen

Dieser Codeteil steht in einer USART-ISR und UDR0 ist der Buffer!!

Da hab ich den AVR Standart verwendet.

habs schon getestet, das richtige wuerde ich bekommen...

mfg
Bastian

[Michael R. (dj_motionx)]

Doch sollte schon funktionieren.
Du musst dir nur klar sein dass Flag = UDR0; immer ausgeführt wird 
sobald UDR0 0xD0, 0xD1, 0xD2, 0xE0, 0xE1,0xE2 oder 0xE3 ist. Ansonsten 
sehe ich nichts verbotenes.

L.g.

[Magnus M. (magnetus)]

Und wenn du "Flag" auch außerhalb der ISR auswerten möchtest, musst du 
die Variable als volatile deklarieren.

[Krapao (Gast)]

Du kannst so nicht arbeiten. UDR0 ist ein SFR und verloren, wenn es 
einmal ausgelesen und nicht zugewiesen wurde. Die Zuweisung an Flag in 
den case-Anweisungen liest ein neues UDR0 aus und übergibt nicht 
unbedingt den Wert aus der switch-Bedingung.

  volatile uint8_t tmp;
  switch( (tmp = UDR0) )
  {
    case 0xD0 :      // Set Retry
    case 0xD1 :      // Set RF Power
    case 0xD2 :      // Set Sensing Time
    case 0xE0 :      // Set PWM1 compare
    case 0xE1 :      // Set PWM2 compare
    case 0xE2 :      // Set PWM3 compare
    case 0xE3 :      // Set PWM4 compare
      Flag = tmp;
      break;
    default  :
      break;
  }

[Rolf M. (rmagnus)]

Krapao schrieb:
> volatile uint8_t tmp;

Das volatile ist an dieser Stelle vollkommen überflüssig.

[Krapao (Gast)]

Technisch ja; mir hilft es beim Lesen des Codes.

[Rolf M. (rmagnus)]

Es macht den Code halt größer und langsamer und verbraucht mit hoher 
Wahrscheinlichkeit auch etwas mehr RAM.

[Krapao (Gast)]

Aus dem Bauch raus geschätzt: An der Stelle wird es nix machen.

Das hängt natürlich von der nicht gezeigten Umgebung ab, d.h. wie viele 
Register der Codegenerator gerade noch frei hat. In einer ISR sind 1) eh 
alle gerettet und 2) vielleicht noch nicht so viele in Benutzung.

[Sebastian S. (sebastian_s25)]

Hey danke fuer die Hilfe!!!

Jetzt weiss ich endlich warums nicht geht!!!

Danke!!

mfg
Bastian

[Magnus M. (magnetus)]

Sebastian S. schrieb:
> Jetzt weiss ich endlich warums nicht geht!!!

Klär uns auf...

[Tom (Gast)]

Wäre echt super wenn du uns erzählen würdest an was es jetzt gelegen 
hat.

[Karl H. (kbuchegg)]

Tom schrieb:
> Wäre echt super wenn du uns erzählen würdest an was es jetzt gelegen
> hat.

Falsche Frage. Woran es gelegen hat ist klar, Krapao hat es 
angesprochen. Die Frage ist, ob er es auch verstanden hat.

[Stefan E. (sternst)]

Krapao schrieb:
> Aus dem Bauch raus geschätzt: An der Stelle wird es nix machen.
>
> Das hängt natürlich von der nicht gezeigten Umgebung ab, d.h. wie viele
> Register der Codegenerator gerade noch frei hat. In einer ISR sind 1) eh
> alle gerettet und 2) vielleicht noch nicht so viele in Benutzung.

Doch, es wird was ausmachen.
Als volatile Variable wird sie ja eben nicht in einem Register gehalten, 
sondern sie existiert real im Speicher. Mit dem volatile zwingst du den 
Compiler dazu, extra für diese Variable einen Stack-Frame anzulegen. 
Ohne das volatile würde sie aller Wahrscheinlichkeit nach nur in einem 
Register existieren.

[Krapao (Gast)]

Du hast mich überzeugt!

[Sebastian S. (sebastian_s25)]

Hi
sorry dass ich mich jetzt erst wieder melde, aber hatte gestern keine 
Zeit mehr...

Also soweit ich das jetzt testen konnte, funktioniert es so wie es 
sollte...

und kapiert hab ich auch!! ;)

Wusste nur nicht, dass der Speicher nach dem auslesen wieder leer ist...

mfg
Bastian

[Fabian (Gast)]

Steht so aber im Datenblatt und ist daher kein Geheimnis mehr. Grade als 
Einsteiger sollte man dies ständig griffbereit haben...