C Timervariable verhält sich nicht wie gewünscht

Das Berechnen der 10ms, 100ms usw. machst du dann in der Main?

Zum Vergrößern anklicken....

kuddlmuddl schrieb:

Alleine die Grundidee für die 10 Sekunden Verzögerung ist murks: Du kannst nicht 1000x mit 1ms Timer eine ISR aufrufen und denken, dass dies dann genau 1 Sekunde dauert. Die Ausführung der ISR kostet auch Zeit! Erst recht wenn du so viel Mathe darin machst.
Wenn du willst, dass 'eine Zeitlang' gewartet wird, dann nimm doch dafür wieder einen Timer! Das hast du doch bereits für die 1ms Periode zum laufen gebracht.
Außerdem müssen generell Variablen, die zwischen ISR und anderen Funktionen geteilt verwerden werden volatile sein. Das trifft dann auch auf RS232_data zu. Ist GIO_UART_input auch eine ISR nur eben für rs232?

edit: Bagbags Lösung sieht schon 100x sauberer aus - wobei ich Polling auf Verzögerung komisch finde, wenn man schon Timer hat.
Dort muss time aber auch unbedingt volatile sein.
Wieso nicht einfach beim receive von 'c' einen bereits vorbereiteten Timer starten, der dann genau 1 Sekunde läuft?

Zum Vergrößern anklicken....

Bagbag schrieb:

Noch ein Vorteil von einer Lösung per Timer: Während dem warten könnte noch anderer Code ausgeführt werden.

Hab volatile hinzugefügt, danke.

Zum Vergrößern anklicken....

fanatiXalpha schrieb:

bis zu 1 Sekunde mach ich es auch mit Modulo
aber hinten raus hab ich es zunächst nicht gemacht weil ich da viel am rumwurschteln war
das dürfte ja aber nicht der Grund für die Probleme sein, sonst würde es in der Main ja auch nicht funktionieren, oder?

Naja, relevanter Code ist eigentlich nicht viel bis dahin.
Main:

Code:

*Hier ist der Aufruf der Initialisierungsroutinen*
while (1)
{
GIO_UART_input();

*anderer Kram der Momentan nicht benutzt wird*

}

dann der Timer:

Code:

#include <avr/io.h>
#include <avr/interrupt.h>

#include "Timer.h"

unsigned char uc1Ms;
unsigned char uc1Ms_2;
unsigned char uc10Ms;
unsigned char uc50Ms;
unsigned char uc100Ms;
unsigned char ucSec;
unsigned char uc2Sec;
volatile char uc1Min;
unsigned int  ui10MsCount;
unsigned int  ui1MsCount;
unsigned char uc1Micros;
unsigned char uc2Micros;
unsigned char uc3Micros;
unsigned int  ui1MicrosCount;

/*
* Timer INIT
*/
void init_Timer(void)
{
	uc1Ms = 0;
	uc1Ms_2 = 0;
	uc10Ms = 0;
	uc50Ms = 0;
	uc100Ms = 0;
	ucSec = 0;
	uc2Sec = 0;
	uc1Min = 0;
	ui10MsCount = 0;
	ui1MsCount = 0;
	// Timer 3 16-Bit eingestellt auf eine Millisekunde
	TCCR3B |= (1<<WGM32);													// Waveform Generation Mode 4 CTC
	TCCR3B |= (1<<CS31) ;													// Clock Select: prescaling 8 -> 16Mhz -> 2000000Hz
	TCCR3A |= (1<<COM3A1);													// Clear OC3A on Compare Match (Set output to low level)
	TIMSK3 |= (1<<OCIE3A);													// Output Compare Match A Interrupt Enable
	OCR3A = 2000;															// Alle 1 Ms
}

/**
* @ brief Interrupt Service Routine, Vektortabelle: 'TIMER3_COMPA_vect' wird jede Millisekunde ausgelöst
*/
ISR (TIMER3_COMPA_vect)
{
	ui1MsCount++;
	uc1Ms = 1;
	
	if((ui1MsCount%1) == 0)
	{
		uc1Ms_2 = 1;
	}
	
	if ((ui1MsCount%10) == 0)
	{
		uc10Ms = 1;
	}

	if ((ui1MsCount%50) == 0)					// ui10MsCount modulo 50 ohne Rest -> 50ms erreicht
	{
		uc50Ms = 1;
	}

	if((ui1MsCount%100) == 0)					// ui10MsCount modulo 100 ohne Rest -> 100ms erreicht
	{
		uc100Ms = 1;								
	}

	if(ui1MsCount == 1000)						// ui10MsCount nach 1sec zurücksetzen
	{
		ui10MsCount = 0;	
		ui1MsCount = 0;							
		ucSec = 1;
	}
	
	if(ucSec == 1)
	{
		ucSec = 0;
		uc2Sec++;
		if(uc2Sec == 10)
		{
			uc2Sec = 0;
			uc1Min = 1;
		}
		
	}
	
	
}

und dann noch die c-Datei die Probleme macht, GIO:

Code:

void GIO_UART_input(void)
{
	//beide UART-Ports werden geprüft wobei RS232-Schnittstelle priorisiert ist
	if(RS232_data_flag == 1 && Automatic_Mode_Flag == 0)
	{
		RS232_data_flag = 0;
		
		if(RS232_data == 'c')
		{
			serial_println_RS232("return state");	
 			uc1Min = 0;		
 			while(uc1Min == 0);

*anderer/weiterer Code der dann kommt*

An der Stelle nach "return state" soll er nachher entweder auf eine Eingabe warten oder eben nach einer Minute einen Timeout geben und der "Antrag" muss erneut gestellt werden.

Zum Vergrößern anklicken....

Und ja, ist umständlich ausgedrückt

Zum Vergrößern anklicken....