Problem mit switch
-
Hi
Hast vielleicht nen break vergessen sodass er zum nächsten case weiter geht nach 16 ?
-
Ich poste euch mal den Code:
/******************************************************************************************** | Name: PWM_SET_SYNC | Called by: MIOS | Preconditions: none | Period Type : none | Inputs: PWM Channel, Dutycycle in Prozent | Return Value : none | Description : This Function Sets the Duty Cycle for the specified Port *********************************************************************************************/ void PWM_SetSync (IO_ChannelType channel, IO_ValueType dutycycle) { /* Temp Variables for dutycycle Calculations */ UINT16 var1,var2; /* Check for Invalid dutycycle value */ if ( dutycycle < 0 || dutycycle > 100) { ErrorHook(PWM,SET_ERROR,IO_E_INVALID_VALUE); } else { /* Set Dutycycle for specified channel or return Error for invalid Channel */ switch (channel) { case 0: var1 = MIOS14.MPWMSM0PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM0PULR.R = var2; break; case 1: var1 = MIOS14.MPWMSM1PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM1PULR.R = var2; break; case 2: var1 = MIOS14.MPWMSM2PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM2PULR.R = var2; break; case 3: var1 = MIOS14.MPWMSM3PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM3PULR.R = var2; break; case 4: var1 = MIOS14.MPWMSM4PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM4PULR.R = var2; break; case 5: var1 = MIOS14.MPWMSM5PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM5PULR.R = var2; break; case 16: var1 = MIOS14.MPWMSM16PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM16PULR.R = var2; break; case 17: var1 = MIOS14.MPWMSM17PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM17PULR.R = var2; break; case 18: var1 = MIOS14.MPWMSM18PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM18PULR.R = var2; break; case 19: var1 = MIOS14.MPWMSM19PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM19PULR.R = var2; break; case 20: var1 = MIOS14.MPWMSM20PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM20PULR.R = var2; break; case 21: var1 = MIOS14.MPWMSM21PERR.R; var2 = dutycycle * var1 / Exp (2,16); MIOS14.MPWMSM21PULR.R = var2; break; default: ErrorHook(PWM,SET_ERROR,IO_E_INVALID_CHANNEL_ID); break; } } }Also ich hab vor dem Switch schon eine Debugausgabe gemacht, das Switch erreicht er auf jeden Fall. Wenn ich die Variable channel mit einer 16 austausche gehts auch, wenn die Variable drinnsteht semmelt er ab.
-
bitte ein beispiel, dass wir kompilieren und selber testen können...
btw: bist du dir sicher, dass ein MIOS14.MPWMSM17PULR-Array nicht vielleicht besser wäre?
-
Nein, Arrays kann ich nicht nehmen, das sind direkt die namen der Register des Prozessors, dahinter verbirgt sich eine Speicheradresse. Das geht nicht mit arrays.
Ein Beispiel zum kompilieren und ausprobieren wird schwer sein, ausser ihr habt ein MPC564EVB Evaluation Board von der Firma Axiom.
-
Hast du schon mal versucht channel in einen normalen integer umzuwandeln und dann abzufragen? (Nur mal so ne Idee)
-
@AJ:
Ja hab ich auch schon, ich habs schon in einen char gecastet, in einen UIN16, in einen normalen int und in alle möglichen unsigned ints.
-
ich schlag vor du schreibst ein neues kleines
Testprogramm indem nur diese switch-Konstrukt
vorkommt. Auch ohne den Code innerhalb des switch.int main( void ) { unsigned short iVal = 16; switch ( iVal ) { case 16: iVal = 100; break; default: iVal = 0; }; return 0; };Wenn diese Code bei dir läuft liegt es auf keinen Fall
an deinem switch, sondern hakt an einer anderen Stelle
in deinem Code.Bye Peter.
-
Ich hab grad mal noch mal geschaut und voller schrecken mitbekommen, das er mir auch bei einem anderen Switch aussteigt wenn ich das aufrufe. Genau die gleichen Symptome. Wenn ich eine Variable einfüge stürzt das switch ab, wenn ich dei Zahl direkt reinschreibe läuft es durch.
-
Hi ich hab da so ne Vermutung.
MSDN schrieb:
Microsoft C does not limit the number of case values in a switch statement. The number is limited only by the available memory. ANSI C requires at least 257 case labels be allowed in a switch statement.
Wenn du dann einen festen wert in switch einträgs wird der Code optimiert und die also nur die eine Case anweisung kann ja ablaufen.
Mfg TheBender
-
Naja da steht das im Case 257 Stück erlaubt sind, ich hab grad mal 20 drinn. Das mit dem optimieren könnt ich mir schon vorstellen, aber wenn er das case ausführt wenn ichs direkt anspreche, wieso nicht wenn ich ne Variable einsetze. Irgendwas muss mit der Variable schiefgehen.
[EDIT]
Ich hab jetzt mal Spaßeshalber die ganzen switch rausgekickt und alles mit einzelnen if Anweisungen gemacht, und sofort geht es auch. Nur ist das halt ein schlechter Stil und ich will es eigentlich nicht so lassen, aber mir fällt echt langsam nix mehr ein.
-
MSDN schrieb:
Microsoft C does not limit the number of case values in a switch statement. The number is limited only by the available memory. ANSI C requires at least 257 case labels be allowed in a switch statement.
Der Satz könnte noch wichtig sein. Hast du schon mal überprüft, wieviel Speicher du überhaupt noch zur Verfügung hast?
-
Der Speicher reicht locker, ich hab 512 K und mein Programm hat alles in allem vielleicht mal grad 50 K
-
Arbeitsspeicher oder "Festplatten"speicher?
-
Ich glaube manche haben mein erstes Posting nicht gelesen oder ? Ich schrieb da das es um einen Power Pc auf einem Evaluation Board geht ? Da gibts keine Festplatten. Das ist ein Microcontroler mit Flash Speicher.
-
Ich wollte mit meinem Posting nicht auf die größe des Arbeitsspeicher aufmerksam machen sondern auf deinen Compiler vielleicht hatt der bei Switch() ne begrenzung von 8 oder so.
Du kannst ja mal probieren die switchanweisung zu begrenzen z.B:
case 1:
bla
case 2:
bla
default:
blaMfg TheBender
-
@Tom
Darum habe ich auch "Festplatte" in Anführungszeichen gesetzt. Ich meinte damit eigentlich deinen Speicher auf deinem Chip oder sonst was. Aber das ändert im Endeffekt nicht meine Frage. Meinst du nun mit den 512K deinen Flashspeicher oder deinen Arbeitsspeicher.
-
Ich würd mal nach Assembler compilieren und mir das angucken ...
Wenn alles nichts hilft, würd ich mich mit Usern dieser Plattform zusammensetzen, gibt bestimmt ne Newsgroup oder Mailingliste zum Thema. Es ist anscheinend auf jeden Fall irgendwas faul, auf dass du rein mit C-Mitteln einen Einfluss hast.
-
AJ:
Der Speicher ist es definitiv nicht, der langt noch ewig weit. 512 Flash und 4 MB Arbeitsspeicher. Laufen lass ich es im Moment im RAM also reichen die 4 MB dick.
@the Bender:
Das mit den Switches könnte schon sein, hab aber beim compiler nix gefunden das das begrenzt wäre, und den MISRA Spell Checker hab ich nicht aktiviert.
-
Hab jetzt kurzzeitig mal gedacht es könnte am Stack liegen, aber den hab ich jetzt auf 10 K erhöht, das dürfte wohl immer ausreichen. Also der wars dann wohl auch nicht.
-
Ich kann mich noch daran erinnern, dass es eine Funktion unter C gibt, mit der man den noch freien Speicher ausgeben kann. Nur der Name will mir jetzt nicht mehr einfallen. Stellt dein Programm eigentlich das Grundsystem auf dem Board dar oder ist da noch ein OS drunter?