Blutiger Anfänger mit Timer Problem

Hallo,
ich geb mal einen Tip ohne Gewähr ab, led1on = delay ist kein atomarer 
Zugriff, weil delay ein uint16_t ist.

Mach mal vor der Zuweisung ein cli() und danach ein sei() rein.

Gruß
Frank

@Coder:
die led1on ist schon volatile! Hatte ich in dem kurzen Abschnitt nicht 
kopiert aber im main.c ist das zu sehen...

@Frank Link
ich hab jetzt mehrere Sachen versucht, aber irgendwie ist das Springen 
der LEDs immer noch da...

Meine main sieht jetzt so aus, oder hatte ich das falsch verstanden mit 
dem sei cli?

1

int main(void) {

2

  cli();

3

4

  uint16_t delay;

5

6

  systemInit();

7

  timerInit();

8

  sei();

9

  while (1) {

10

    if (IN(SWITCH_PORT, SWITCH_PIN)) {

11

      cli();

12

      delay = 500;

13

      sei();

14

    } else {

15

      cli();

16

      delay = 1000;

17

      sei();

18

    }

19

    if (!led1on) {

20

      leds();

21

      led1on = delay;

22

23

    }

24

  }

25

}

Jepp,
hast Du.

1

cli();

2

led1on = delay;

3

sei();

Alle anderen sind nutzlos.

Gruß
Frank

Ok, macht Sinn, sind ja 16bit Operationen...
Aber helfen tut es leider auch nicht :-(

Mario Dejung schrieb:
> Ok, macht Sinn, sind ja 16bit Operationen...
> Aber helfen tut es leider auch nicht :-(

Das reicht nicht.
Du hast ja noch einen Zugriff auf led1on, den du kapseln musst

1

    cli();

2

    uint16_t tmp = led1on;

3

    sei();

4

5

    if (!tmp) {

6

      leds();

7

      cli();

8

      led1on = delay;

9

      sei();

10

    }

Du hast natürlich dadurch, dass du die ganzen Zeitvariablen alle
unsigned int( also 16 Bit) gemacht hast, dem µC jede Menge unnötige
Fleissaufgabe aufgebürdet. Die solltest du (bis auf Millisekunden) alle
als uint8_t führen. Das langt dicke für Sekunden, Minuten, Stunden.

Ok, das scheint des Rätsels Lösung zu sein :-) Danke, auch wenn ich es 
nicht wirklich verstehe... Aber vielleicht kommt da noch bald die 
Routine...

Hätte man da dann nicht einfacher Weise die komplette if Abfrage in cli 
und sei setzen können, anstatt noch mal drumrum in ne andere Variable u 
speichern? So ungefähr:

1

  while (1) {

2

    if (IN(SWITCH_PORT, SWITCH_PIN)) {

3

      delay = 500;

4

    } else {

5

      delay = 1000;

6

    }

7

    cli();

8

    if (!led1on) {

9

      leds();

10

      led1on = delay;

11

    }

12

    sei();

13

  }

Mario Dejung schrieb:
> Ich hab nen externen Crystal mit 16GHz an nem Atmega644

16 GHz? Hoffentlich wird es da dem atmega644 nicht schwindlig :-)

Mega Giga, kann man doch mal verwechseln... :-)

By the Way, ich hab die Änderungen mit dem cli und sei gemacht wie in 
meinem letzten Post und das scheint zu funktionieren...

Mario Dejung schrieb:
> Hätte man da dann nicht einfacher Weise die komplette if Abfrage in cli
> und sei setzen können, anstatt noch mal drumrum in ne andere Variable u
> speichern? So ungefähr:

Natürlich kann man.
Wichtig ist nur, dass weder bei Abfrage noch beim Beschreiben der 
Variable ein Interrupt dazwischen funken kann.
Du musst dich eben fragen, wie lange du die Interrupts gesperrt haben 
willst.

Ein Schlüsselnotdienst, der sofort reagiert, hilft dir nun mal nichts, 
wenn wegen Telefonsperre keiner abhebt.

Karl Heinz Buchegger schrieb:
> Ein Schlüsselnotdienst, der sofort reagiert, hilft dir nun mal nichts,
> wenn wegen Telefonsperre keiner abhebt.

:-) Danke, den Vergleich hab ich gebraucht! :-)