Pointerschreibweisen

OP #1287006
Ich habe eine Frage:

Ich muss den Port eines Atmels in einer Funktion übergeben (dient als CS 
bei SPI).
Das habe ich gemacht wie im GCC-Tutorial.
Also :

void fkt(volatile uint8_t *port, uint8_t pin)
{
  *port &= ~(1<<pin);
}

fkt(&PORTB, PB2);

So weit so gut...

Nun möchte ich der Übersicht wegen bei mehreren Slaves die Geräte mit 
structs initialisieren und nur noch diesen übergeben müssen.
Ungefähr so:

typedef struct
{
  volatile uint8_t *port;
  uint8_t pin;
} TsFlash;

void initTsFlash(TsFlash *sFlash, volatile uint8_t *port, uint8_t pin)
{
  sFlash -> port        = port;
  sFlash -> pin         = pin;
}

Main
{
  TsFlash serialFlash;
  initTsFlash(&serialFlash, &PORTB, PB2);
}

Jetzt die Frage, wie ich die Zeile programmiertechnisch in der Funktion 
fkt() richtig umsetzte.
Der Compiler akzeptiert folgende 4 Varianten:

void fkt(TsFlash *sFlash)
{
  *sFlash -> port   &= ~(1 << sFlash -> pin);
  (*sFlash -> port) &= ~(1 << sFlash -> pin);
  sFlash -> port[0] &= ~(1 << sFlash -> pin);
  (*(*sFlash).port) &= ~(1 << sFlash -> pin);
}

Sind die alle gleich und überhaupt richtig ?
Wenn ja, welche wäre die eleganteste ?

Gruß Micha
#1287016
Micha Obi schrieb:

> Sind die alle gleich und überhaupt richtig ?

Sind alle gleichwertig und auch alle richtig

> Wenn ja, welche wäre die eleganteste ?

Ich würds so schreiben

  *(sFlash->port) &= ~(1 << sFlash->pin);

Die Klammern sind zwar syntaktisch nicht notwendig, -> hat einen höheren 
Rang als *, klären aber beim schnellen Lesen darüber auf, wies gemeint 
ist ohne dass man sich lange den Kopf über Operatorenreihenfolge 
zerbrechen muss.
Persönliche Seite #1287302
Karl heinz Buchegger schrieb:
>
>   *(sFlash->port) &= ~(1 << sFlash->pin);
>

Hier ist zu beachten, daß das zu nicht-atomarem Code führt, was zu 
Race-Conditions führen kann, wenn man einen Port (zB PORTB) sowohl in 
einer ISR verändert (zB PortB.5 setzt) und in der Anwendung (zB effektiv 
über den indirekten Zugriff PortB.0 setzt).

Das geht auf AVR unter den gegeben Umständen nicht anders. Man sollte 
sich also immer darüber klar sein, daß hier möglicherweise ein fieser, 
sporadischer Bug lauert.

Johann

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 17 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.