Interrupt ATMEGA8

[Ronald E. (ronald_1)]

Hallo zusammen,

ich versuche über PortD Pin 2 und 3 ein externes Interrupt 
herbeizufüheren (4,7KOhm Pull-Down). Leider hab ich noch nie was mit 
Interrupts gemacht, und kriegs auch irgentwie nicht hin. Mein Versuch:

//External Interrupt Request 0
#define INT0_vect                       _VECTOR(1)
#define SIG_INTERRUPT0                  _VECTOR(1)

ISR(INT0_vect)       // veraltet: SIGNAL(SIG_INTERRUPT0) 
{
   //Ausgabe "Test an LCD"
}

Ist dies die richtige Vorgehensweise? Ich bitte um Hilfe.

MfG Ronald

[Karl H. (kbuchegg)]

Ronald E. schrieb:

>

> //External Interrupt Request 0
> #define INT0_vect                       _VECTOR(1)
> #define SIG_INTERRUPT0                  _VECTOR(1)
> 
> ISR(INT0_vect)       // veraltet: SIGNAL(SIG_INTERRUPT0)
> {
>    //Ausgabe "Test an LCD"
> }
>

>
> Ist dies die richtige Vorgehensweise? Ich bitte um Hilfe.

Nein.
Du definierst die Vektoren nicht selbst, sondern überlässt das dem

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

diese #include machen dann schon die richtigen #define für deinen 
Prozessor, so dass du

ISR(INT0_vect)
{
  // mach was
}

machen kannst.
Eine Ausgabe auf ein LCD sollte allerdings ausser für schnelle 
testzwecke tabu sein. Das dauert einfach zu lange und steht der 
Forderung, dass eine ISR möglichst schnell abgearbeitet werden soll im 
Wege.

[Ronald E. (ronald_1)]

Ach so, danke!
Die Ausgabe am LCD ist tatsächlich nur zum Testen, obs funktioniert.
Leider funktionierts nicht.

ISR(INT0_vect)       // SIGNAL(SIG_INTERRUPT0) 
{
        lcd_clear();
    lcd_setcursor(0,1);
    lcd_string("Test");
  _delay_ms(5000);
}

Die Interrupts habe ich global aktiviert (mit sei(); ). Das Display 
funktioniert (habs getestet). Mach ich nochwas falsch?

MfG Ronald

[Karl H. (kbuchegg)]

Die Pins sind auf Eingang gestellt?

Zeig doch der Einfachheit alles her, dann braucht man nicht raten

[Ronald E. (ronald_1)]

Anbei der Code. Gleichzeitig muss ich anmerken, dass ich noch nicht so 
erfahren bin. Das Display hängt an Port B, der Temperaturfühler an Port 
C. Port D ist also frei (bis auf den externen Interrupt).

[Tim T. (tim_taylor)]

Also ohne in deinen angehängten Quelltext geschaut zu haben, ist delay 
in einer ISR eigentlich immer Schrott und einer von 5000 mit Sicherheit.

[Karl H. (kbuchegg)]

void main(void)
{

  lcd_init();
  int8_t puffer[7];

  //Interrupts global aktivieren
  sei();

  DDRD = 0x0;  
  
  while (1)

Ähm.
Und wo erlaubst du den externen Interrupt 0?

Im Mega8 gibt es viele Interrupts. Behandelt werden aber nur die, die 
explizit freigegeben werden!

[Tim T. (tim_taylor)]

Nachdem ich deinen Quelltext gesehen habe, ist mir aufgefallen das du 
den INT0 garnicht aktiviert hast; also sind GICR und MCUCR deine 
Freunde.

Und im übrigen ist der Rückgabewert von main ein int...

[ronald_1 (Gast)]

Ahh da hab ich mal was gelesen.
Das probier ich gleich mal aus.

[Ronald E. (ronald_1)]

Also, erstmal danke für die Hilfe! Ich habs geschafft mittels Interrupt 
etwas am Display auszugeben. Nun versuche ich zu einer Zahl 
(Temperatur_Soll) mittels Interrupt 0.5 zu addieren. Leider bekomme ich 
einige Errors. Stimmt es, dass ich mit volatile eine Variabel global 
deffinieren kann?

[Verwirrter Anfänger (Gast)]

Wie willst du den auf einen Int 0.5 addieren? Ints sind immer nur 
Ganzzahlen.

Du bräuchtest also eigentlich float oder double. Allerdings würde ich 
dir zu Festkommaarithmetik raten.

Ciao,
VA

[Tim T. (tim_taylor)]

Nein, volatile sorgt nur dafür das der Compiler an der Variablen nichts 
rumoptimiert.

Du definierst eine globale Variable indem du sie ausserhalb einer 
Funktion definierst, wie du es auch fast richtig gemacht hast, nur die 
10 dahinter ist Murks. Wenn du ihr einen Startwert geben wolltest, muss 
noch ein = dazwischen.

Temperatur_Soll = Temperatur_Soll + 0.5 funktioniert nicht, da 0.5 keine 
Ganze Zahl ist, Temperatur_Soll jedoch schon.

Und statt:

//Interrupt wird ausgelöst bei fallender oder steigender Flanke (INT0,INT1)  
GIMSK = (1 << DDB6) | (1 << DDB7);

//Interrupts INT0, INT1 reagieren auf steigende Flanken
MCUCR = (1 << DDB0) | (1 << DDB1) | (1 << DDB2) | (1 << DDB3);

schreibst du besser:

//(INT0,INT1) Interrupt kann ausgelöst werden
GIMSK |= (1 << INT1) | (1 << INT0);

//Interrupts INT0, INT1 reagieren auf steigende Flanken
MCUCR |= (1 << ISC11) | (1 << ISC10) | (1 << ISC01) | (1 << ISC00);

Ist übersichtlicher und ändert nur was du auch ändern willst.

[Ronald E. (ronald_1)]

Vielen Dank nochmals für die guten Hinweise (hab gleich alles geändert).
Das mit der Kommastelle hab ich glaub ich jetzt gut gelöst. Probleme 
habe ich aber noch mit dem sprintf, da bekomme ich noch errors.

../avr_lcd.c:81: error: 'buf' undeclared (first use in this function)
../avr_lcd.c:81: error: 'Temperatur_Soll' undeclared (first use in this 
function)
../avr_lcd.c:81: error: 'Komma' undeclared (first use in this function)

Irgentwie scheint die Defintion nicht zu stimmen.

[abc (Gast)]

>Irgentwie scheint die Defintion nicht zu stimmen.
Das kann man so sagen:

#define uint8_t Temperatur_Soll = 10
#define uint8_t Komma = 0
#define uint8_t Speicher = 0

GRÖHL! So definiert man doch keine Variablen! Nimm die #define da weg!

Das Problem mit 'buf' sehe ich gerade nicht.

[abc (Gast)]

>Das Problem mit 'buf' sehe ich gerade nicht.

OK, gefunden: Der Präprozessor schießt dank deiner phantasievollen 
Deklarationen das uint8_t da weg und packt irgendwelchen Krempel an 
dessen Stelle. --> Problem.

[Tim T. (tim_taylor)]

#define uint8_t Temperatur_Soll = 10
#define uint8_t Komma = 0
#define uint8_t Speicher = 0

Das #define ist jedesmal zuviel.

Und häng mal die avr_lcd.c an, sonst ist das mit dem buf nicht klar.

[abc (Gast)]

>Deklarationen
Grrr, ich meinte Definitionen.

[Ronald E. (ronald_1)]

Ohne die define`s sind die errors weg, und es funktioniert. Nur leider 
etwas zu schnell, da muss ich mir noch was einfallen lassen. Danke für 
eure Hilfe.