Problem Interrupt

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

Ü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

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

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

Dazu fehlt noch Deine Antwort.

> Hinweis: Du brachst "volatile".

Gruß Dietrich

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

Ernst schrieb:
> Wofür genau volatile?

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

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 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.

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.

1

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

2

3

  switch(myKey)

4

  {

5

    case keyEnterMenuPressed:

6

      menuMarker = 1;

7

      ...   / do your stuff to enter menu

8

      break;

9

10

    case keyEnterMenuReleased:

11

      menuMarker = 0;

12

      ... // do your stuff to exit menu

13

      break;

14

15

    case keyChooseVariablePressed:

16

      if (menuMarker == 1)

17

      {

18

        ....

19

      }

20

      break;

21

22

    case keyzeitVerstellenPressed:

23

      if (menuMarker == 1)

24

      {

25

        ....

26

      }

27

      break;

28

  }

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

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

 Hmmm...

Dann nimm es so:

1

    case keyEnterMenuPressed:

2

      if (menuMarker == 0)

3

      {

4

        menuMarker = 1;

5

        ...   / do your stuff to enter menu

6

      }

7

      else

8

      {

9

        menuMarker = 0;

10

        ... // do your stuff to exit menu

11

      }

12

      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.