Interrupt,Timer1

[Armer_Azubi (Gast)]

kann mir einer sagen wieso meine LED nicht toogelt ich bin echt am ende 
mit dem Programm für die Berufsschule :(

#define F_CPU 1000000UL  //1Mhz
#include <avr/io.h>
#include <avr/interrupt.h>



int main(void)
{ 
  DDRD=0xFF;
  PORTD= 0xFF;              //Alle LEDS ausschalten
  
    PORTC=0x0F;                //Aktivierung der Pull-up Widerstände
  
  TIMSK |= (1<<OCIE1A) ;    
  TCCR1B |= ((1<<WGM12)|(1<<CS10)|(1<<CS12));    //Auswahl des Prescaler Wert 1024 und des Zähler 16Bit
  sei();
  
  
  while (1)
  {
  switch(PINC)
  {
    case 0xFE: OCR1A = 4883;          //5s
          
          
    case 0xFD: OCR1A = 9766;          //10s
          
          
    case 0xFC:  OCR1A = 14648;          //15s
    
    default:  OCR1A = 14648;
          
      
    
  }
    
  }
}
  ISR(TIMER1_COMPA_vect)
 {
   
   while (1)
   {
     PORTD^= 0xFF;
     
   }
 }

[Armer_Azubi (Gast)]

bitte ignoriet das default das ist mir reingeruscht als ich meine Ocr1A 
werte überprüft habe um zu sehen ob meine Berechnungen stimmen sry :(

[holger (Gast)]

>kann mir einer sagen wieso meine LED nicht toogelt

Die toggelt. Aber verdammt schnell.
Was soll das bekloppte while(1) in der ISR?
Weisst du eigentlich was du tust?

[Andreas W. (geier99)]

Armer_Azubi schrieb:
ISR(TIMER1_COMPA_vect)
>  {
>
>    while (1)
>    {
>      PORTD^= 0xFF;
>
>    }
>  }

Hi ,

Du gehst nie aus dem Interrupt raus.

Und die "break;" bei den cases fehlen auch.

[Armer_Azubi (Gast)]

>Die toggelt. Aber verdammt schnell.
>Was soll das bekloppte while(1) in der ISR?
>Weisst du eigentlich was du tust?


Mit der jeweiligen Tastenkombination soll die LED in der ausgewählten 
zeit leuchten deswegen das while(1) in ISR

[c-hater (Gast)]

Armer_Azubi schrieb:

>>Weisst du eigentlich was du tust?
>
>
> Mit der jeweiligen Tastenkombination soll die LED in der ausgewählten
> zeit leuchten deswegen das while(1) in ISR

Die richtige Antwort auf die Frage wäre gewesen:

Nein, ich habe absolut keine Ahnung, was ich da tue.

[Armer_Azubi (Gast)]

>Die richtige Antwort auf die Frage wäre gewesen:

>Nein, ich habe absolut keine Ahnung, was ich da tue.


Deswegen bin ich auf der suche nach ahnung -.- absolut ist aber 
übertrieben

[Andreas W. (geier99)]

> Mit der jeweiligen Tastenkombination soll die LED in der ausgewählten
> zeit leuchten deswegen das while(1) in ISR

nochmal :-)
Du gehst nie mehr aus der ISR raus, d.h. beim ersten Auftreten des 
Interrupts bleibst Du in der Endlosschleife hängen, und weitere 
Interrupts werden keine mehr ausgelöst, da  Du noch in der ISR bist.

und vergesse bitte die "break;" nicht. Denn egal welcher Case zutrifft, 
du setzt die Zeit dann immer auf 15s.

Und schau mal in ein C-Buch :-)