Probleme mit Timer0 und Timer1

[Lichti01]

Danke dir schon einmal für deine Antwort. Werde mein Code einmal entsprechend anpassen. Melde mich dann nochmal ob es dann funktioniert.

[Lichti01]

Hi

habe meinen Code jetzt so überarbeitet wie du es geschrieben hast. Leider hat es nicht funktioniert. Das Problem mit dem 16-Bit-Timer besteht weiterhin.

So sieht der COde jetzt aus:

#define F_CPU 1000000UL

#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdio.h>

#define LED1 1
volatile uint8_t i;

#define a 50			//Zeit in ms/10 für Blinkfrequenz der LED1
uint8_t b;

//variablen für entprellung
#define TASTERPORT PIND
#define TASTERBIT PIND3

//Entprellung
char taster(void)
{
	static unsigned char zustand;
	char rw = 0;

	if(zustand == 0 && !(TASTERPORT & (1<<TASTERBIT)))			//Taster wird gedrueckt (steigende Flanke)
	{
		zustand = 1;
		rw = 1;
	}
	else if (zustand == 1 && !(TASTERPORT & (1<<TASTERBIT)))   //Taster wird gehalten
	{
		zustand = 2;
		rw = 0;
	}
	else if (zustand == 2 && (TASTERPORT & (1<<TASTERBIT)))   //Taster wird losgelassen (fallende Flanke)
	{
		zustand = 3;
		rw = 0;
	}
	else if (zustand == 3 && (TASTERPORT & (1<<TASTERBIT)))   //Taster losgelassen
	{
		zustand = 0;
		rw = 0;
	}
	else if (((zustand == 1) || (zustand == 2)) && !(TASTERPORT & (1<<TASTERBIT)))   //Taster wird gehalten
	{
		zustand = 2;
		rw = 0;
	}
	return rw;
}

int main(void)
{

	DDRB = 0b00000010;
	
	//Einstellung Interrupt
	PORTD = 0b00000100;
	GIMSK = (1<<INT0);
	MCUCR = (1<<ISC01); //The falling edge of INT0 generates an Interrupt

	//Timerkonfiguration
	//TIMSK = (1<<OCIE0A);				//Ineterrupt aktivieren
	//TCCR0A = (1<<WGM01);				//CTC-Mode akrivieren
	//OCR0A = 155;						//Interrupt asulösen bei Zählerstand (10ms)
	//TCCR0B = ((1<<CS00) | (1<<CS01));	//Timer starten mit Prescaler 64
	
	i=0;
	b=2*a;
	
	//Globale interrupts aktivieren
	sei();
	
	while (1)
	{
		if (i==0)
		{
			PORTB &=~ (1<<LED1);
		}
		if ((i>=1) && (i<=a))
		{
			PORTB |= (1<<LED1);
		}
		if ((i>=1) && (i>a) && (i<=b))
		{
			PORTB &=~ (1<<LED1);
			if (i>=b)
			{
				i=1;
			}
		}
		
	}
}

ISR(INT0_vect)
{
	_delay_ms(50);
	
	if (bit_is_set (TIMSK,OCIE0A))
	{
		TIMSK &= ~(1<<OCIE0A);				//Ineterrupt aktivieren
		TCCR0A &= ~(1<<WGM01);				//CTC-Mode akrivieren
		OCR0A |= 0;						//Interrupt asulösen bei Zählerstand (10ms)
		TCCR0B &= ~((1<<CS00) | (1<<CS01));	//Timer starten mit Prescaler 64
	}
	if (bit_is_clear (TIMSK,OCIE0A))
	{
		TIMSK |= (1<<OCIE0A);				//Ineterrupt aktivieren
		TCCR0A |= (1<<WGM01);				//CTC-Mode akrivieren
		OCR0A |= 156;						//Interrupt asulösen bei Zählerstand (10ms)
		TCCR0B |= ((1<<CS00) | (1<<CS01));	//Timer starten mit Prescaler 64
	}	

	if (i==0)
	{
		i=1;
	}
	if (i>=1)
	{
		i=0;
	}

}

ISR(TIMER0_COMPA_vect)
{
	i++;
}

Bei meinem Anderen Problem habe ich den Fehler gefunden.

Über weitere Tipps bin ich sehr dankbar.

LG Lichti01

[Siro]

Hallo Lichti01

Ich habe noch nicht ganz begriffen was deine Software machen soll.
Wenn die Taste gedrückt wird soll eine LED blinken und wenn die Taste losgelassen wird soll sie aus gehen ?

Deine Abfrage

if (i==0)     // wenn i null ist, dann
{ 
    i=1;   // setzte i auf 1
}


if (i>=1)      // wenn i größer oder gleich 1 ist dann 
{
    i=0;   // setze i = null, dann hättest Du Dir das sparen können auf 1 zu setzen
}

wundert mich:

nehmen wir an i ist 0
dann wird ausgeführt i = 1
danach folgt die Abfrage wenn i größer oder gleich 1 ist, dann setze i = 0

das kommt irgendwie auf den gleichen Code als wenn Du den gesamten block weglässt und
i = 0 schreibst, oder verstehe ich da etwa falsch ?

evtl. ein "else if" oder so ?

[Lichti01]

Hallo Siro,

Ja du hast recht da fehlt ein else vor dem zweiten if (nur komisch, dass es beim 8-bit timer keine probleme damit gab).

Theorie ist:
Wenn der Taster gedrückt wird soll der Timer1 im CTC Mode gestartet werden. Wenn der angegebene Wert erreicht wird soll die Variable i immer um ein erhöht werden und somit der blinkintervall "generiert" werden. Wird der Taster erneut betätigt soll der Timer angehalten und die LED ausgeschaltet werden.

Das Problem das ich jetzt (hoffentlich) eigentlich nur noch hab ist, dass der Timer irgendwie viel zu langsam läuft.

Gruß
Lichti01

[seite5]

Timer zu langsam:
Fuses nicht eingestellt, läuft Dein Controller evtl. mit dem internen Clock?
Datenblatt, S. 23: "The device is shipped with CKSEL = “0100”, SUT = “10”, and CKDIV8 programmed."
mfg
Achim

[021aet04]

Wenn die Fuses nicht passen, würde das für beide Timer falsch sein.

MfG Hannes

[seite5]

Sorry, hat ich doch glatt überlesen.
Der code ist m.M.n. eh recht "unüblich". OCRA |= 0, was soll das bringen?
Jedes mal int ein/ aus halt ich auch für nicht nötig, reicht aus, den Timer-Takt
abzuschalten, wichtig: beim wiedereinschalten sicherheitshalber dann erstmal den Zähler nullen.
Würde wegen der Übersichtlichkeit überhaupt viel aus der ISR 'rausnehmen und über Flags
in der while(1) abarbeiten.
mfg
Achim