Problem mit Interrupt

[Hlao L. (lli_howa)]

Hallo Zusammen,

ich bin ganz neu in der Welt der Programmierung mit Mikrocontroller. Ich 
habe ein Programm geschrieben, was ein Interrupt durch Drücken einen 
Taster auslöst. Dabei soll das mainProgramm unterbrochen werden und die 
LED leuchten. Leider bleibt das Hauptprogramm immer am Laufen und wird 
das Interrupt nicht ausgelöst.

Kann mir vielleicht da jemanden helfen, wie ich das Programm verbessern 
kann.

hier ist das geschriebene Code von mir:

#define F_CPU 2000000
#define BAUD 9600UL
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)

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

// Praeprozessor Makros fuer Ausgaenge
#define set_bit(port,mask) ((port) |= (1<<mask))
#define clr_bit(port,mask) ((port) &= ~(1<<mask))
#define tog_bit(port,mask) ((port) ^= (1<<mask))

// Praeprozessor Makros fuer Eingaenge
#define input_bit(port,mask) ((port) & (1<<mask))
void USART_Init(void)
{
  // Baudrate einstellen
  UBRR2H = (unsigned char)(UBRR_VAL>>8);
  UBRR2L = (unsigned char)UBRR_VAL;
  // receiver and transmitter einschalten
  UCSR2B = (1<<RXEN2)|(1<<TXEN2);
  
  //  frame format auf 8data, 1stop bit setzen
  UCSR2C = (1<<UCSZ21|1<<UCSZ20);
  CLKPR = (1<<CLKPCE); 
  CLKPR = (1<<CLKPS1); 
} // USART_Init

void USART_putChar( unsigned char data )
{
  // warte bis Sendepuffer leer ist
  while (!( UCSR2A & (1<<UDRE2)));
  
  UDR2 = data;
  
  return 0;
}
void USART_putStr(unsigned char *s)
{
  while (*s)
  {   
    USART_putChar(*s);
    s++;
  }
}


ISR(INT1_vect)
{
   set_bit(PORTB,PB6);
   USART_putStr("LED EIN \r\n");
}

int main(void)
{
  //PB3 Ausgang
  DDRB |= (1<<PB6);
  
  // PD1 als Eingang
  DDRD &=~(1<<PD1);
  
  // Pullup einschalten
  PORTD |=(1<<PD1);
  
  // fallende Flanke    
  EICRA |=(1<<ISC11);
  
  // INT1 einschalten 
  EIMSK |=(1<<INT1);
    
  // UART Init
  USART_Init();
  
  // Globale Interrupts einschalten
  sei();     
  
  while(1)
  
  {  
    USART_putStr("LED AUS \r\n");       
  }
  _delay_ms(500);
}

Für eure Hilfe bin euch sehr dankbar.

[Peter D. (peda)]

Es gibt auf der Welt nicht nur einen MC und von einer Familie nicht nur 
einen Typ.
Zu jeder Frage gehört daher die Angabe des Targets (komplette 
Bezeichnung).

Hlao Lajh schrieb:
> Leider bleibt das Hauptprogramm immer am Laufen und wird
> das Interrupt nicht ausgelöst.

Woran erkennst Du das?

Hlao Lajh schrieb:
> ISR(INT1_vect)
> {
>    set_bit(PORTB,PB6);
>    USART_putStr("LED EIN \r\n");

USART_putStr ist nicht reentrant. Du mußt Dich daher entscheiden, ob nur 
im Interrupt oder nur im Main. Beides zusammen ergibt Zeichensalat.

[Hlao L. (lli_howa)]

Peter Dannegger schrieb:
> Es gibt auf der Welt nicht nur einen MC und von einer Familie nicht nur
> einen Typ.
> Zu jeder Frage gehört daher die Angabe des Targets (komplette
> Bezeichnung).

Ich arbeite mit dem Atmega1280



> Woran erkennst Du das?

Auf dem Terminal wird immer LED AUS angezeigt.

Für Verbesserungsvorschläge wäre ich euch sehr dankbar.

[Daniel V. (danvet)]

Hallo,
zu deinem Programm gäbe es viel zu sagen.
Da du neu in der ControllerWelt bist schlage ich dir vor zunächst die 
UART-sachen weg zu lassen.

Versuche zuerst die LED ein- und auszuschalten und zwar bei jedem 
Tastendruck.
Dazu kannst du dir das hier durchlesen (und verstehen):
http://www.mikrocontroller.net/articles/Entprellung

Wenn das zuverlässig funktioniert kannst du mit der UART-Ausgabe 
weitermachen.

[Ralf G. (ralg)]

Hlao Lajh schrieb:
> set_bit(PORTB,PB6);

Geht die LED an? Ist die korrekt angeschlossen?

1. Kleiner Test zum Einfügen vor sei();

  set_bit(PORTB,PB6);
  _delay_ms(500);
  clr_bit(PORTB,PB6);
  _delay_ms(500);
  set_bit(PORTB,PB6);
  _delay_ms(500);
  clr_bit(PORTB,PB6);
  _delay_ms(500);

2. Lege einen Puffer an, der an die Funktion 'USART_putStr' übergeben 
wird.
In der ISR wird nur der Text dort reingeschrieben.