globale variable durch interrupt routine setzen geht nicht

Achja... wo ich grad schon amFragen bin...
Wie würdet ihr das Machen:
Globale Variable mit extern definieren oder nur eine globale Variable 
welche mit einer eigenen Funktion überprüft wird?

Wie könnte man hier diese Globale Variable vermeiden?

Warum Polling UND IRQ ?
Der Sinn eines IRQs ist doch das die Bearbeitung bedarfsgerecht 
weitergeht ohne in einer Warteschleife zu verharren.

Sieht für mich danach aus als ob die Funktion eine Kopie der globalen 
Variablen bei Aufruf bekommt, also den letzten Wert.

Wie wäre es mit dem ganzen Quellcode? Der Fehler liegt natürlich an 
einer Stelle, die du hier nicht zeigt. So wird dir keiner helfen können.

Ich polle ja nicht.
Die Interrupt Routine nimmt nur Werte auf und überprüft sie darauf hin 
ob ein Signal (ein Ultraschall Echo) empfangen wurde oder eben nur 
Rauschen. Ist also nur ein Flag was mir dann am Ende sagen soll ob ein 
Echo empfangen wurde oder eben nicht.

Ja ich denke ich schau jetz noch mal drüber und wenn ich jetz nichts 
finde (hatte gestern ein Tag Pause) lad ich den Code hoch.
Muss den halt erst was aufbereiten weil da viel Funktionen drin sind die 
nicht von interesse sind. Und dann kopier ich die ISR und die andere 
Funktion in eine Datei.

Sieht für mich auch so aus als würd er die globale Variable beim Start 
der Funktion in ein Work-Register laden und eben nicht das "original" 
benutzen.
Aber eigentlich ist doch um das zu verhindern volatile dar oder nicht?

Aber nicht nur Codefetzen sondern was lauffähiges bitte.

So ich habe keine Ahnung. Für mich sieht das so aus als ob der Compiler 
es entweder ignoriert oder ich volatile irgendwie Falsch verwende.

Gesetzt wird die globale Variable jeden falls. Das hab ich überprüft.

Was halt nicht funktionirt ist die Ausgabe in C-File-Finktion.txt ob die 
Messung erfolgreich war.

Gesetzt wird sie in C-File-ADC wo ich die zeile mit XXX kommentiert hab.
Hoffe ihr kommt mit dem Code zurrecht.

Nochmal den Ablauf beschrieben:
start_measurement() startet timer und ADC und wartet auf Beendigung von 
ADC
ADC-ISR setzt measurement_done auf 0 oder 1
start_measurement soll anhand der globalen Variable measurement_done 
eine Ausgabe machen.
measurement_done scheint nicht verändert zu sein.
Wenn ich in meiner main measurement_done überprüfe bekomm ich den 
richtigen Wert.

Danke schon mal für eure Hilfe.

Meine Güte. Aber Nase schnauben geht schon alleine, oder? :-)

Jetzt hängst Du mal schon ordentlich ein 'c' und ein 'h' an die Dateien 
und postest die nochmal hier. Vermutlich haben die meisten hier keine 
Lust die Dateien erst auf der Festplatte zu speichern.

Was übrigens auch merkwürdig ist, ist dass er die Ausgabe ob die Messung 
erfolgreich war nicht komplett weg optimiert.

Er kann beide Ausgaben machen nur halt versetzt als würd er die globale 
Variable nicht "updaten" also aus nem Register nehmen.

ups wusst nicht das das dann so läuft.

Au, Sch... Karl Heinz? Kommst Du mal, bitte?

Das klingt nach einer guten Erklärung Karl Heinz.
Ich probiers mal aus.

So habs ausprobiert indem ich eine weitere globale Variable hinzugefügt 
hab.
ADC_ended
Diese setze ich in der ADC-ISR auf eins an den stellen wo ich auch 
measurement_done auf 0 oder 1 setze.

in start_measurement() setze ich sie zu beginn auf 0 und überprüfe in 
der while-Schleife:
while(ADCON1bits.ADON==1 && ADC_ended==1){}     // wait for ADC

Das Problem besteht weiterhin.
Alle achtung wie schnell ihr durch einen fremden Code gelesen habt.

Du definierst die Variable gleich 2 mal in adc.c (über den Include adc.h 
und im Code) - davon 1 mal (im Include adc.h) als extern.

#include "adc.h"
#include "measurement.h"
#include "timer.h"
//#include "uart.h"
//#include "math.h"
//#include <stdlib.h>

/*** GLOBAL-VAR 
***************************************************************/

int adc_buffer[MEASUREMENTS*8/CHANNELS]={0};
int adc_time[MEASUREMENTS/CHANNELS][1]={{0},{0}};
volatile int measurement_done;

Ist es möglich, dass der Compiler da etwas "verschnupft" reagiert?

Hab ich mir noch ein Ei gelegt mit meiner art das zu überprüfen?

Ne hab doch eine conditional inclusion:

#ifndef adc_h                       // conditional inclusion
#define adc_h

...
extern volatile int measurement_done;
...

#endif

In der c-Datei definiere ich und in der h-Datei deklariere ich.

Tom schrieb:
> Ne hab doch eine conditional inclusion:

... die verhindert, dass das entsprechende Coding in adc.h 2 mal 
includiert wird. Das erste Mal wird aber schon funktionieren - oder?

Ok das mit das ich auf 0 prüfen muss hab ich grad auch gemerkt.

Verstehe aber nicht warum das Problem weiterhin besteht.

Setze die Variable vor der While-Schleife auf 0.
Dann warte ich bis die ADC-ISR diese auf 1 setzt.
Das tut sie wenn sie beendet ist.
Also müsste das doch gehen oder?

Das mit der 2 fachen definition verstehe ich auch nicht.
In der c-Datei definiere ich:
volatile int Measurement_done;
Und in der h-Datei deklariere ich mit extern:
extern volatile measurement_done;

Hab das öfter so gesehen, dass in der c-Datei auch zu zugehörige h-Datei 
includiert wird. Ist das Falsch?

Nein es geht ja eben nicht.

Ich setze vor der while auf null.
Die While schleife läuft solange diese Var 0 ist.

Da ich sehe das mein prog weiter läuft heist es ja die Variable wird von 
der ADC-ISR auf eins gesetzt und zwar an genau der stalle wo auch 
measurement_done gesetzt wird.

Das kann doch nicht sein?!?!

OK ich brauch Urlaub oder so.
Habs jetzt.

So funktionierts natürlich nicht:
while(ADCON1bits.ADON=1 && ADC_ended==0){}     // wait for ADC

Aber so:
while(ADC_ended==0){}     // wait for ADC

Das && ist der Fehler.

Leute ich danke euch mein Problem ist erledigt.
Da muss ich wohl noch viel üben. Toll wie schnell du dich in ein Problem 
hinen gedacht hast.

Zum glück baue ich keine Flugzeuge, Rakeuten oder Satelliten.

Karl Heinz schrieb:
Und Dieter empfehle ich, mal den Unterschied zwischen einer Definition 
und einer Deklaration zu lernen

Der Unterschied ist mir, denke ich, schon bekannt - nur wäre ich nie auf 
die Idee gekommen eine Variable in einer Code-Strecke als extern zu 
deklarieren und zusätzlich zu definieren.
Ich hätte zumindest einen Hinweis des Prä-Prozessors oder des Linkers 
erwartet ... tut sich aber nichts.
Insofern - vielen Dank, habe etwas gelernt.