Arduino: Syntax von cli() sei()

[Furo (Gast)]

Hallo kurze Frage

Was gibt es für einen Grund, dass man das nicht so programmiert:

cli();
...
sei();

sondern so:

byte oldSREG = SREG;
cli();
...
SREG = oldSREG;

[Frank M. (ukw) (Moderator)]

Wenn man portabel programmiert, schreibt man seine Funktion so, dass sie 
möglichst wenige Bedingungen an die Umgebung stellt.

Die cli().... sei() Methode setzt voraus, dass vor dem cli() die 
Interrupts bereits eingeschaltet waren. Deshalb werden hier final die 
Interrupts eingeschaltet, obwohl sie vorher vielleicht abgeschaltet 
waren.

[Stefan F. (Gast)]

Liegt das nicht auf der Hand?

Im ersten Fall sind danach die Interrupts erlaubt. Im zweiten Fall hast 
du danach den gleichen zustand, wie davor.

In der Praxis macht das nur selten einen Unterschied, weil man 
Interrupts nur für wenige zusammenhängende Zeilen deaktiviert. Fertige 
Libraries und Codefragmente für die Copy-Paste Entwickler sollte man 
aber schon so weitsichtig wie möglich gestalten - koste es was es wolle 
(siehe Arduino, Cube HAL, ASF, ect).

[Furo (Gast)]

danke :)

[Veit D. (devil-elec)]

Wenn es um maximale Kompatibilität geht würde ich atomic verwenden und 
dann

ATOMIC_BLOCK (ATOMIC_RESTORESTATE) {
  // ... Code   
}

[Stefan F. (Gast)]

Veit D. schrieb:
> Wenn es um maximale Kompatibilität geht

Leider gibt es dieses Makro nur in der C Library für AVR, da endet die 
Kompatibilität schon.

[Veit D. (devil-elec)]

Aha, ich programmiere nur auf AVR.