|
Anzeige
|
Timer1 lässt sich nicht auf 0 stellen
Gast
#2539515
Hi, Ich hab den Atmega32. Ich wollte mir eine/zwei Funktionen schreiben, die mir es erlauben die WCET und die BCET einer funktion oder eines Codesegments zu bestimmen. Hier der Code:
Leider hab ich festgestellt (mit den USART Ausgaben) das TCNT nicht auf 0 gesetzt wird, warum ? kann mir jemand helfen? Den Timer würde ich bei ETE_stop als erstes stoppen, ansonsten stimmen dann die Register nicht mehr genau. Wie sieht denn die USART_Transmitt Funktion aus?? Im obigen Code scheints korrekt zu sein. Gruss Was passiert wenn du nur einen auf 0 und den anderen auf 1 schreibst? Welche Werte werden angezeigt? Bin mir grad nicht sicher über die Reihenfolge, aber was passiert wenn du zuerst H dann L schreibst? Michael D. schrieb: > Bin mir grad nicht sicher über die Reihenfolge, aber was passiert wenn > du zuerst H dann L schreibst? Geht mir genauso. Daher überlasse ich sowas dem Compiler. Der weiß das nämlich Anstelle von ...
... dann eben
Hat 2 Vorteile * zum einen brauch ich mir nicht merken, wie rum es nun korrekt ist * zum zweiten erspar ich mir das rumhampeln mit Bytes zusammensetzen bzw. auseinanderpfriemeln In C ist eben weniger oft mehr :-) Lass den Compiler für dich arbeiten! > USART_Transmit_uint16(&temp);
Bist du dir da ganz sicher, dass du hier eine Adresse übergeben musst?
Ungewöhnlich, da es an dieser Stelle nichts bringt einen Pointer auf
einen uint16_t zu übergeben. Da ein Pointer sicher nicht kleiner sein
wird, als ein uint16_t, verkompliziert das nur die Funktion, weil sie
über einen Pointer auf den Wert zugreifen muss. Das bringt also weder
geschwindigkeitsmässig etwas, noch ist es notwendig.
Gast
#2539671
Karl Heinz Buchegger schrieb: > Anstelle von ...void ETE_start(void){ > // set the counter value to 0 > TCNT1L = 0; // set the low part to 0 > TCNT1H = 0; // set the high part to 0 > > ... > > void ETE_stop(void){ > uint16_t temp=0; //auxiliary variable > // combine TCNT1L and TCNT1H > temp=TCNT1L; //save the lower part > temp|=(TCNT1H<<8); // save the higher part to the right place > ... > > ... dann eben > void ETE_start(void){ > // set the counter value to 0 > TCNT1 = 0; > > ... > > void ETE_stop(void){ > uint16_t temp = TCNT1; > ... Danke für die antwort --> war es aber nicht
Gast
#2539706
> Leider hab ich festgestellt (mit den USART Ausgaben) das TCNT nicht auf > 0 gesetzt wird, warum ? Es kann überhaupt nur in diesen Ausgaben aus ETE_start() festgestellt werden: > USART_Transmit(TCNT1L); > USART_Transmit(TCNT1H); wenn Timer1 nicht läuft, d.h. wenn z.B. zuvor ETE_stop() gelaufen ist und damit der Timertakt abgeschaltet wurde. Wenn im sonstigen nicht gezeigten Programm der Timertakt angeschaltet wurde oder ein anderer Timertakt als CS11 abgeschaltet wurde, ist eine Ausgabe von TCNT1 = 0 in ETE_start() Zufall. In ETE_stop() mit > USART_Transmit_uint16(&temp); ist es Zufall, wenn eine 0 für die Kopie des TCNT1 übermittelt wird. > TCCR1B&=~(1<<CS11); Am Ende von ETE_stop halte ich für eine logischen Unschönheit. Ich würde diese Anweisung am Anfang der Funktion machen, um den Timerzähler vor der Kopie an temp einzufrieren. Insgesamt würde ich mehr Debugausgaben einbauen, um zu sehen welche Funktion am Ausgeben ist und in welcher Reihenfolge die Funktionen aufgerufen werden. Den restlichen Quellcode würde ich auf Einstellungen des Timertakts scannen. Manfred Schreier schrieb: > Danke für die antwort > > --> war es aber nicht Was ist mit der USART_Transmit_uint16 Sache? Nur weil der Compiler das compiliert (und eine Warnung bringt) heißt das nicht, dass es korrekt ist! Kein vernünftiger Programmierer würde diese Funktion die Signatur void USART_Transmit_uint16( uint16_t * value ) verpassen. Antwort schreiben |
Anzeige
|
