c-Files mit Timer "auslagern"

[Christian M. (krusti)]

Hallo,

ich bin mal wieder an meinem Projekt von hier dran: 
Beitrag "ATmega8 ADC-Durchschnittsberechnung"

Weil ich aber ein neues grundsätzliches Problem habe, will ich es in 
einem neuen Beitrag nachfragen.

Ich will in meinem 500-Zeilen-Code die einzelnen Funktionen "auslagern". 
Das klappt nur leider nicht mit allen Funktionen.
Ich habe die Beobachtung gemacht, dass ich Funktionen welche auf einen 
Timer zugreifen nicht separieren kann. Steht mein resetService mit in 
meiner main.c dann läuft es einwandfrei.

Ähnliches Problem wir in diesem Beitrag hier: 
Beitrag "Funktionen in mehrere c files auslagern funktioniert nicht"

Meinen Hauptcode hab ich jetzt erst mal nicht mit angehängt, weil wie 
gesagt 500 Zeilen. Bei Bedarf poste ich den gerne.

Timer läuft über einen ISR.

Ausgelagerter Code:
resetService.c

#include "resetService.h"

void resetService()
{
  arbeitstag = 0;
  eeprom_update_byte(&eeTage, arbeitstag);
  
  digit[2] = r;
  digit[1] = E;
  digit[0] = S;
  
///////////////////////////////////////////////////////
//Auf meiner 7-Seg wird nur rES angezeigt, weiter passiert nicht mehr

  timer2_wert = 0;
  
  //Timerschleife zur Anzeige der Ausgabe auf dem Display
  //Wartezeit 2 sec
  while (1) {
    if (timer2_wert >= 200) {
      break;
    }
  }
  digit[2] = ZERO;
  digit[1] = ZERO;
  digit[0] = ZERO;
}

resetService.h

#ifndef RESETSERVICE_H_
#define RESETSERVICE_H_

#include <stdint.h>
#include <avr/eeprom.h>
#include "SegementZeichen.h"

extern uint16_t arbeitstag;
extern uint8_t digit [3];
extern uint8_t eeTage;
extern uint16_t timer2_wert;

#endif /* RESETSERVICE_H_ */

[Frank M. (ukw) (Moderator)]

Christian M. schrieb:
> Ich habe die Beobachtung gemacht, dass ich Funktionen welche auf einen
> Timer zugreifen nicht separieren kann.

Fehlermeldung?

[linker linker (Gast)]

Was für eine Fehlermeldung bekommst Du denn ?
Welchen Compiler/IDE nimmst Du ?
Makefile bzw. Projekt auch angepaßt das Deine externen Dateien auch 
eingebunden sind ?

[Christian M. (krusti)]

Keine.

[Ähm... (Gast)]

Was mir auf anhieb auffällt:
- volatile bei Interruptvariablen!
- warum benutzt du einen Timer wenn du dein Programm an der while(1) eh 
zwangs-lahmlegst?

Grüße

[Stefan K. (stefan64)]

extern uint16_t timer2_wert;

ist nicht volatile definiert.
Selbst wenn timer2_wert irgendwo anders geändert wird: Deine while(1) 
Schleife bekommt davon nichts mit.

Gruß, Stefan

[linker linker (Gast)]

Ähm... schrieb:
> Was mir auf anhieb auffällt:
> - volatile bei Interruptvariablen!

Ist ja extern deklariert kann also volatile sein.

> - warum benutzt du einen Timer wenn du dein Programm an der while(1) eh
> zwangs-lahmlegst?
>

O_o das kann natürlich voll ins Auge gehen.
Sollte man mit zwei Variablen lösen, eine für gerade aktualisiert also 
erstmal nicht ändern und den Timestamp ab wann wieder geändert werden 
darf.
Gibt auch andere Lösungen ...
Nur ohne Fehlermeldung bzw. was nun sein eigentliches Problem ist kann 
nicht geholfen werden.

[Christian M. (krusti)]

AVR-Studio 6 und keine Fehlermeldung. Die Dateien habe ich mit dem 
Solution Explorer erzeugt, entfernt, eingefügt, wieder entfernt,...

Ähm... schrieb:
> Was mir auf anhieb auffällt:
> - volatile bei Interruptvariablen!
> - warum benutzt du einen Timer wenn du dein Programm an der while(1) eh
> zwangs-lahmlegst?
>
> Grüße
Zur while(1). Ich will ja dass die 2sec auf jeden Fall durchgeführt 
werden und ich habe zu dem Zeitpunkt der programmierung keine andere 
Möglichkeit gefunden/gesehen zu warten bis mein Zähler auf 200 ist und 
ihn dann abzufragen. In der Zeit soll auch ruhig nichts anderes 
passieren. Quasi wie die _delay Funktion. Nicht gerade schön aber in 
meinem Fall funktional gewesen bis ich ein extra C machen wollte.

timer2_wert ist in der main voltaile

[Peter II (Gast)]

Christian M. schrieb:
> timer2_wert ist in der main voltaile

das klingt komisch, wieso ist er dort anders definiert? Hast du 
eventuell jetzt 2 Variablen mit dem gleichen Namen?

[Christian M. (krusti)]

Peter II schrieb:
> Christian M. schrieb:
>> timer2_wert ist in der main voltaile
>
> das klingt komisch, wieso ist er dort anders definiert? Hast du
> eventuell jetzt 2 Variablen mit dem gleichen Namen?

Vllt habe ich es auch falsch gemacht?
Kurz zum Verständnis: Muss ich die variablen beide exakt so nennen? Also 
volatile uint16_t timer2_wert? Und bei bedarf mit ergänze ich mit 
extern, z.B. bei einer neuen C-Datei? Oder darf/muss ich nur einmal 
voltaile schreiben?

[Peter II (Gast)]

Christian M. schrieb:
> Vllt habe ich es auch falsch gemacht?
> Kurz zum Verständnis: Muss ich die variablen beide exakt so nennen?

eigentlich muss du sie gar nicht neu benennen. Du machst ja ein include 
mit der Headerdatei. Damit ist sie ja bekannt.

[Mein grosses V. (vorbild)]

linker linker schrieb:
> Ist ja extern deklariert kann also volatile sein.

Nein, kann es nicht.

[Rufus Τ. F. (rufus)]

Christian M. schrieb:
> Oder darf/muss ich nur einmal voltaile schreiben?

Du musst jedesmal volatile schreiben. Woher soll sonst der Compiler 
beim Übersetzen der einzelnen Module wissen, daß die Variable volatile 
ist?

[Stefan K. (stefan64)]

Wenn der Compiler ResetService.c übersetzt, dann sieht er das volatile 
nicht. Und deswegen wird er an dieser Stelle:

  while (1) {
    if (timer2_wert >= 200) {
      break;
    }
  }

optimieren und timer2_wert EINMAL in ein Register einlesen und danach 
dieses Register forever auf >= 200 vergleichen. Kann sogar sein, daß der 
Vergleich nur einmal ausgeführt wird und der effektive Code dann so 
aussieht:

  if (timer2_wert >= 200)
  else
  {
    while (1);
  }

Gruß, Stefan

[Christian M. (krusti)]

Das war es. Volatile bei digit und timer2_wert dazu und schon läuft das 
ganze, wie urspünglich. Im Nachhinein eigentlich logisch das volatile 
benötigt wird, da ich ja sonst quasi nur einmal den Wert übergebe und 
zwar der der Initialisierung. So gesehen ist klar, dass ich nicht aus 
meiner Schleife raus komme.
Danke für die Hilfe. Hoffe so werden die anderen Funktionen auch 
funktionieren.

[Stefan K. (stefan64)]

Wenn Du dort, wo timer2_wert definiert wird, die .h Datei einbindest, in 
der timer2_wert als extern deklariert wird (also "resetService.h"), dann 
kann Dir der Fehler nicht mehr passieren, weil dann der Compiler einen 
Fehler wirft, sobald sich beide unterscheiden.

Gruß, Stefan