Hallo! Ich habe folgendes Problem. Ich möchte den Wert eines 16Bit Timers mit einer Methode auslesen und diesen dann zurück geben. Dazu muß ich aus dem High und Low Register einen short basteln. Ich hab mir folgenden Code ausgedacht. read_16Bit_Timer_Value(char Timern){ short value=0x0000; if(Timern==1){ char low = TCNT1L; char high = TCNT1H; value = value | high; value = value<<8; value = value | low; // hier geht scheinbar was schief return(value); } Ich arbeite mit AVRStudio und Zielplattform ist ein AT90CAN128. Bei meinem code passiert jetzt folgendes. Ich lese das high und das low Register jeweils in ein char aus. Die Werte stimmen laut AVRstudio. Danach kommt zuerst der high teil in die short variable. Das paßt auch noch. Der shift ist auch ok. Wenn ich dann aber die Low variable dazu haben will dann ist der "high"teil in der shortvariablen aufeinmal 0xFF! (vorher 0x01). Warum? Hab ich da einen Denkfehler? Gibt es für mein Problem eine bessere Lösung? Vielen Dank :-)
abgesehen davon dass man das so bastelt: short=(high<<8)+(low) bzw. low&0xff falls low keine 8bit ist. geht es einfach mit: value=TCNT1; das macht der Compiler dann schon richtig.
Die erste Idee dazu: Deine char sind nicht unsigned, also erfolgt eine Vorzeichenerweiterung falls beim TCNT1L (low) das Bit #7 gesetzt ist. Und mache Deinen Rückgabewert unsigned short. unsigned char low = TCNT1L; unsigned char high = TCNT1H; unsigned short value = high; value <<= 8; value |= low; return(value); Oder verwende (mit avr-gcc/WinAVR) ganz einfach. value = TCNT1; Schöne Feiertage noch Werner
Funktioniert! coole Sache! Da war ich ja mal wieder super umständlich dabei :-) Vielen Danke!
Bitte melde dich an um einen Beitrag zu schreiben. Anmeldung ist kostenlos und dauert nur eine Minute.
Bestehender Account
Schon ein Account bei Google/GoogleMail? Keine Anmeldung erforderlich!
Mit Google-Account einloggen
Mit Google-Account einloggen
Noch kein Account? Hier anmelden.