Timer1 lässt sich nicht auf 0 stellen

[Manfred Schreier (Gast)]

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:

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
  USART_Transmit(TCNT1L);
  USART_Transmit(TCNT1H);
// set the prescaler s. Table 48 Datasheet
// 0,5µs= 1/(F_CPU/8)
  TCCR1B|=(1<<CS11);
}


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
  USART_Transmit_uint16(&temp);
// evaluate if it was a BCET or a WCET
  if (temp<BCET){
    BCET=temp;      // save the new BCET to the global variable
  }else if(temp>WCET){
    WCET=temp;      // save the new WCET to the global variable
  }  
  TCCR1B&=~(1<<CS11);
}

Leider hab ich festgestellt (mit den USART Ausgaben) das TCNT nicht auf 
0 gesetzt wird, warum ?

kann mir jemand helfen?

[Patrick B. (p51d)]

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

[Michael D. (etzen_michi)]

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?

[Karl H. (kbuchegg)]

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 ...

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;
...

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!

[Karl H. (kbuchegg)]

>   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.

[Manfred Schreier (Gast)]

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

[Krapao (Gast)]

> 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.

[Karl H. (kbuchegg)]

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.