Globale Variable ist irgendwie nicht global?!

[Sefco (Gast)]

Hallo!

Thema: RC-Auto
Folgendes soll mein Programm machen:
Ich möchte den Hebel meiner Fernsteuerung komplett auslenken und damit 
eine LED anschalten. Dann möchte ich die LED wieder ausschalten können, 
indem ich den Hebel zurück in Mittelstellung bringe und anschließend 
wieder auslenke.

Einfach erklärt:
start:
Hebel MITTE
Hebel HOCH: LED an
Hebel MITTE
Hebel HOCH: LED aus
goto start;


Es geht hierbei ja darum, mit einem Interrupt die steigende Flanke des 
Empfängers zu detektieren und dann durch Warten (max. 2 us) und erneutes 
abfragen des PINs, festzustellen, wie der Hebel ausgelenkt ist.

Das ganze klappt wunderbar, solange ist meine LED im Interrupt Vektor 
schalte. Nutze ich allerdings eine globale Variable, um den 
Schaltvorgang in die main-Funktion zu bringen, klappt das nicht. Die 
Variable on1 die global definiert ist wird irgendwie nicht verändert, 
oder bzw. wird verändert, aber die main Funktion scheint das nicht zu 
sehen. Weiß jemand warum?

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

uint8_t reset = 0;
uint8_t on1 = 0;

int main()
{

  DDRB = 0b11111111; //PORTB auf Ausgang schalten; LED angeschlossen
  PORTB = 0b11111111;
  DDRD = 0b00000000; //PORTD auf Eingang schalten
  
  GICR = (1 << INT1); //INT0-Interrupt aktivieren

  MCUCR = (1 << ISC11)|(1 << ISC10); //Aktiviere Sensitivität auf steigende Flanke

  sei(); //Generell Interrupts aktivieren


  while (1) //Hauptschleife
  {
    if(on1==1)
    {  
      PORTB =~ PORTB;
    }
  }

  return 0;

}


ISR(INT1_vect) //Interruptfunktion: Hier wird hingesprungen, wenn ein Interrupt ausgelöst werden soll.
{

  if(reset==0) //Muss Hebel in Mitte? reset=0 NEIN ; reset=1 JA
  {
    //Wird Fernsteuerungshebel ausgelenkt? --> Impulsbreite beträgt mind. 1,8us
    _delay_us(1800);  
    if(PIND & (1 << PD3)) //Ist der Pin nach 1,8us noch High?
    {
      on1=1;  //Ja ist er: Hebel ist fast ganz ausgelenkt --> LED einschalten
      reset = 1; //Hebel muss in Mitte!

    }
  }

  else //Hebel muss gesenkt werden, damit er durch erneutes Auslenken die LED wieder ausschalten kann
  {
    _delay_us(1600);  
    if( !(PIND & (1 << PD3)) ) //Wenn nach 1,6us kein High mehr an Pin anliegt --> Hebel ist in Mittelstellung
    {
      reset = 0; 
    }
  }


}

[Hugo (Gast)]

probier mal

volatile uint8_t on1 = 0;

[Simon G. (grosi)]

Hallo,
versuch es einmal mit volatile uint8_t on1 = 0.
Mfg

[Sefco (Gast)]

Das klappt schon viel besser! Kann mir einer erklären was das genau 
bedeutet? Ich hab das nicht verstanden was genau da passiert wenn ich 
das weglasse...

[Johann L. (gjlayde)]

Sefco schrieb:
> Ich hab das nicht verstanden was genau da passiert wenn ich
> das weglasse...

Ganz einfach: Ich setze eine globale Variable auf 2.
Preisfrage: Welchen Wert hat die Variable?

[Sefco (Gast)]

>Ganz einfach: Ich setze eine globale Variable auf 2.
>Preisfrage: Welchen Wert hat die Variable?

Den Wert 2. Wieso muss ich dann volatiel davor packen?

[Lukas K. (carrotindustries)]

Sefco schrieb:
> Das klappt schon viel besser! Kann mir einer erklären was das genau
> bedeutet? Ich hab das nicht verstanden was genau da passiert wenn ich
> das weglasse...

Ist in AVR-GCC-Tutorial: Datenaustausch mit Interrupt-Routinen 
erschöpfend ausgeführt.

[Johann L. (gjlayde)]

Sefco schrieb:
>>Ganz einfach: Ich setze eine globale Variable auf 2.
>>Preisfrage: Welchen Wert hat die Variable?
>
> Den Wert 2.

Nö, inzwischen hat sie den Wert 3, weil eine ISR den Wert verändert hat.

C-Funktionen bestehen aus Anweisungen, die sequentiell ausgeführt 
werden. Wenn nun eine ISR (oder die Hardware) einen Wert unter der Hand 
ändert, muss man das dem Compiler mittelen. Und das macht man mit 
volatile.

[Christopher G. (cbg)]

Das fehlende volatile wurde ja schon erwähnt und ebenfalls erklärt. Ich 
geh mal davon aus, dass du noch nicht soviel Erfahrung mit 
µC-Programmierung hast. Ich geb dir gleich mal einen Tipp, wie du dein 
Programm besser und vielseitiger gestalten kannst.

Das Busywaiting in der ISR solltest du dir schnell abegwöhnen. Das 
funktionniert zwar, ist aber wenn mal drüber nachdenkst einfach Pfui. 
Bei deinem Problem kannst du sehr schön die Nützlichkeit eines Timers 
erlernen. Ich geh mal davon aus, du hast in deinem AVR einen 16 Bit 
Timer mit Input Capture Funktion. Mit dem Input Capture Modus kannst du 
die Länge des Impulses sehr genau messen (genauer wirds mit einem AVR 
nicht) und dann abhängig von der Länge des Impulses darauf reagieren. 
Falls das Ding Batteriebetrieben werden soll (bzw auch wenn nicht, 
Stromsparen sollte man wo immer möglich) solltest du dir auch die Sleep 
Modes anschauen und den µC schlafen lassen, wenn er nichts zu tun hat.

Ist ein ganz gutes Anfängerprojekt um sich mit Timern zu beschäftigen.