Problem Interrupt

[Ernst (Gast)]

Hey Guys,

ich hab ein Problem mit der ISR

while(marker)
{
   CODE
   PINX ((PINX & (Taster))==0)
   marker = 0; Goto(1);
}

ISR
{marker = 1;}

Nun möchte ich das ich mit dem selben Taster mit dem ich den Interrupt 
betätige wieder raus kommen aus der while Schleife.
Wie ist dies möglich ?

[Karl M. (Gast)]

Ist das so etwas wie virtueller Code?

Welcher Comtroller - Compiler etc. wird verwendet ?

Besser ist es ein richtiges um komplettes Programm mit der Fragestellung 
zu posten.

[Ernst (Gast)]

AVR 328
Atmel Studio

      while(menuMark)
      {
            lcd_clear();
            WeckerMenu();
            if (WeckerH == 24) WeckerH = 0;
            if (WeckerM == 60) WeckerM = 0;
            zeit(150);
            if ((PINC & (1<<ChooseVariable))==0) vari++;
            zeit(150);
            if (vari==0)
            {
              if ((PINC & (1<<ZeitverstellenPLUS))==0) WeckerH++;
              if ((PINC & (1<<ZeitverstellenMINUS))==0) WeckerH--;
            }
            else
            zeit(150);
            {
              if ((PINC & (1<<ZeitverstellenPLUS))==0) WeckerM++;
              if ((PINC & (1<<ZeitverstellenMINUS))==0) WeckerM--;
            }
            zeit(150);
            if (vari==2) vari=0;
            zeit(150);
            if ((PINC & (1<<Menu_Taster))==0) {menuMarker = 0; goto(1);}
      }

ISR(PCINT1_vect) // Interrupt Service Routine
{
  menuMark = 1;
}

Rest funktioniert des Progamms

[Peter II (Gast)]

ich denke es wird zeit das Programm komplett zu sehen, in einer ISR zu 
warten ist eine ganz schlechte Idee. Auch das

goto(1);

finde ich sehr bedenktlich, hast du eine Funktion mit dem namen goto?

[Ernst (Gast)]

was heißt goto eigentlich gotocase(1)...

Warte routinen sind zum einlesen der Taster das nicht mit einem 
Tastendruck 30 mal nach oben gezählt wird.


Rest des Programmes ist relativ irrelevant weil dies nichts mit dem 
Problem zu tun hat

[Ernst (Gast)]

Geht ledglich um eine Möglichkeit den Interrupt mit dem selben Taster 
wieder auszuschalten

[Dietrich L. (dietrichl)]

Wie ist "menuMark" deklariert?
Hinweis: Du brachst "volatile".

[Karl M. (Gast)]

Hallo,

Bei mir meckert der gcc an den Stellen, wo wir die Funktiondeklaraion 
und Variablen nicht kennen.

Was soll da genau passieren, in deinen Augen ?

[Oliver S. (oliverso)]

Üblicherweise merkst du dir dazu in einem Flag, ob du gerade drin oder 
draußen bist. Bei dem gezeigten Code ist allerdings "üblicherweise" 
eventuell nicht der passende Ansatz.

Die Meinung deines Compilers zu goto(1) würd mich allerdings schon 
interessieren.

Oliver

[Dietrich L. (dietrichl)]

Ernst schrieb:

> while(menuMark)
                ^^

> if ((PINC & (1<<Menu_Taster))==0) {menuMarker = 0; goto(1);}
                                             ^^
Ist das auch so programmiert oder ist das hier nur ein Schreibfehler?

Gruß Dietrich

[Ernst (Gast)]

int Phase_;
int vari = 0;
int menuMark = 0;
int clockMark = 0;

#define Menu_Taster PC0
#define ZeitverstellenPLUS PC1
#define ZeitverstellenMINUS PC2
#define ChooseVariable PC3
#define TurnClockoff PC4
#define Menu_Uhr PB5

Er sollte eigentlich aus dem Menü raus gehen, aber anstatt geht er nur 
raus wenn der Taster gedrückt wird, sprich die ISR setzt die variable 
dauerhaft 1 per Tastendruck und die 0 aus dem case wird ignoriert.

[Ernst (Gast)]

Dietrich L. schrieb:
> Ernst schrieb:
>
>> while(menuMark)
>                 ^^
>
>> if ((PINC & (1<<Menu_Taster))==0) {menuMarker = 0; goto(1);}
>                                              ^^
> Ist das auch so programmiert oder ist das hier nur ein Schreibfehler?
>
> Gruß Dietrich

Hab mehrfach rum probiert deshalb war der Code bisschen anderes und 
menuMarker wurde manuell eingefügt aber im Ursprung war es gleich

[Peter II (Gast)]

Ernst schrieb:
> die 0 aus dem case wird ignoriert.

welchem case? Ich code steht kein case.

[Ernst (Gast)]

Peter II schrieb:
> Ernst schrieb:
>> die 0 aus dem case wird ignoriert.
>
> welchem case? Ich code steht kein case.

Ist alles im switch programmiert.
Im case war damit gemeint in dem oberen Programmteil..

Wie gesagt der rest ist irrelevant...

[Dietrich L. (dietrichl)]

Dietrich L. schrieb:
> Wie ist "menuMark" deklariert?

Dazu fehlt noch Deine Antwort.

> Hinweis: Du brachst "volatile".

Gruß Dietrich

[Ernst (Gast)]

Dietrich L. schrieb:
> Dietrich L. schrieb:
>> Wie ist "menuMark" deklariert?
>
> Dazu fehlt noch Deine Antwort.
>
>> Hinweis: Du brachst "volatile".
>
> Gruß Dietrich

Zum zweiten Mal
int menuMark = 0;

Wofür genau volatile?

[Dietrich L. (dietrichl)]

Ernst schrieb:
> Ist alles im switch programmiert.
> Im case war damit gemeint in dem oberen Programmteil..

So wird das nichts. Du musst den kompletten und aktuellen Code zeigen, 
der den Fehler hat.

> Wie gesagt der rest ist irrelevant...

Bei Fehlersuche darf man nie davon ausgehen, man muss an allem 
zweifeln.

Ernst schrieb:
> Zum zweiten Mal

Das habe ich übersehen :-(

> int menuMark = 0;
>
> Wofür genau volatile?

Damit die Variable innerhalb und außerhalb der ISR gültig ist.

Gruß Dietrich

[Dietrich L. (dietrichl)]

Ernst schrieb:
> Wofür genau volatile?

Hier noch mehr dazu:
https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Datenaustausch_mit_Interrupt-Routinen

[Ernst (Gast)]

Hier der komplette Code in der C Datei

http://www.file-upload.net/download-11315566/Clock.c.html

[Dietrich L. (dietrichl)]

Ernst schrieb:
> Hier der komplette Code in der C Datei
>
> http://www.file-upload.net/download-11315566/Clock.c.html

Man das lieber als Dateianhang hier im Forum. Das ist für die Leser 
einfacher und sicherer.

[Ernst (Gast)]

Naja wie auch immer die Frage ist ja, wie ich den Taster für beides 
benutzen beenden und starten des Interrupts während der while Schleife

[Marc V. (Firma: Vescomp) (logarithmus)]

Ernst schrieb:
> Naja wie auch immer die Frage ist ja, wie ich den Taster für beides
> benutzen beenden und starten des Interrupts während der while Schleife

 Taster löst Interrupt aus.
 Interrupt toggelt einen Flag - nicht mehr und nicht weniger.
 In main() wird der Flag geprüft und das ist auch alles.

 Flag als volatile deklarieren.

[Karl M. (Gast)]

Hallo Betreuer,

bitte erklärt Ernst
wie er mit ISR(PCINT1_vect) - level interrupt umgehen muss.
Ich will mich nun nicht weiter einmischen.

[Stefan K. (stefan64)]

So wird das nichts.
Statt den Taster an einen Interrupt-Pin zu hängen, verwendest Du besser 
einen Timer-Interrupt, der alle paar ms ALLE Taster abfragt, das 
Entprellen übernimmt und die Tastendrücke entweder in einer volatile 
Variable oder besser noch in einem Fifo an main() weitergibt.

  char myKey = getKey(); // read key from fifo

  switch(myKey)
  {
    case keyEnterMenuPressed:
      menuMarker = 1;
      ...   / do your stuff to enter menu
      break;

    case keyEnterMenuReleased:
      menuMarker = 0;
      ... // do your stuff to exit menu
      break;

    case keyChooseVariablePressed:
      if (menuMarker == 1)
      {
        ....
      }
      break;

    case keyzeitVerstellenPressed:
      if (menuMarker == 1)
      {
        ....
      }
      break;
  }

Mit einer Timer-ISR kannst Du bei Bedarf nicht nur keyPressed sondern 
auch keyReleased Signale auswerten und an main() weiterleiten.

Vorteile:
* Du wertest Deine Tasten an EINER Stelle aus und nicht verstreut im
  Programm. Das erleichtert das Testen (was ist das?).
* Alle Tasten werden entprellt.
* Wenn main() mal ein paar ms beschäftigt ist (Graphik-Ausgabe, etc.),
  kann trotzdem kein tastendruck verlorengehen.
* ...

Wie man Tasten im Timer-ISR entprellt, findest Du hier an mehreren 
Stellen.

Gruß, Stefan

[Marc V. (Firma: Vescomp) (logarithmus)]

Stefan K. schrieb:
> case keyEnterMenuPressed:
>     case keyEnterMenuReleased:

 Hmmm...

[Stefan K. (stefan64)]

Dann nimm es so:

    case keyEnterMenuPressed:
      if (menuMarker == 0)
      {
        menuMarker = 1;
        ...   / do your stuff to enter menu
      }
      else
      {
        menuMarker = 0;
        ... // do your stuff to exit menu
      }
      break;

Was ich oben zeigen wollte:
* Entprellen ist essentiell.
* Klare Verteilung: ISR konvertiert Pin-Signale zu Tastenkommandos und
  main()steuert mit den Tastenkommandos das Menu, etc.
* Je nach Aufwand in der Timer-ISR lassen sich unterschiedliche
  Tastenkommandos generieren: pressed, released, longPressed... main() 
kann
  diese auswerten, muss das aber nicht.