Problem bei Fehlermeldung: invalid operands to binary |

Felix Gast #6048838 
Nabend,
ich habe eine Frage zu einer Fehlermeldung.
1
static uint8_t _dataPin;

2
static uint8_t _dataPort;

3


4
void WS2818(volatile uint8_t* ddr, uint8_t ddrNewSate, volatile uint8_t* port, uint8_t portNewState, uint8_t dataPin)

5
{

6
  *ddr |= ddrNewSate;

7
  *port |= portNewState;

8


9
  _dataPin = dataPin;

10
  _dataPort = port | (1 << dataPin);

11
}


hierbei bekomme ich die Fehlermeldung:
invalid operands to binary | (have 'volatile uint8_t * {aka volatile 
unsigned char *}' and 'int')

ich verstehe zwar die Worte, aber leider den Sinn nicht dahinter. Was 
genau bedeutet das? Und wieso kann ihc das nicht machen?
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke
Theor Gast #6048841 
Das hängt mit der Zeigernatur von port zusammen - besser ausgedrückt 
damit, dass port ein Zeiger ist und kein Integer. Das geht ja auch aus 
der Fehlermeldung hervor.

Die Ausdrücke:

1
  *ddr |= ddrNewSate;

2
  *port |= portNewState;


sind ausgeschrieben:

1
  *ddr = ddr | ddrNewSate;

2
  *port = port | portNewState;


und nicht, wie Du vermutlich annimmst

1
  *ddr = *ddr | ddrNewSate;

2
  *port = *port | portNewState;
Antwort Zitieren Bearbeiten Löschen Melden Danke
Nop Gast #6048853 
Felix schrieb:

>   _dataPort = port | (1 << dataPin);

In dieser Zeile ist port ein Pointer, den Du mit einem Int verordern 
willst, und das Ergebnis davon willst Du in einem uint8_t speichern? Mit 
ziemlicher Sicherheit meinst Du

[c]_dataPort = *port | (1 << dataPin);[c]

Und da das alles sowieso auf uint8_t läuft und vermutlich auf einem 
8-Bit-Prozessor, willst Du die Veroderung sicherlich nicht als Integer 
durchführen, sondern möchtest vielmehr das hier, weil es schneller geht:

[c]_dataPort = *port | (uint8_t)(((uint8_t) 1) << dataPin);[c]



Theor schrieb:

> Die Ausdrücke:
>   *ddr |= ddrNewSate;
>   *port |= portNewState;
>
> sind ausgeschrieben:
>   *ddr = ddr | ddrNewSate;
>   *port = port | portNewState;

So ein Unsinn.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Nop Gast #6048857 
Ah vergessen, die Tags zuzumachen.

ALso gemeint ist wohl

1
_dataPort = *port | (1 << dataPin);


Und performance-freundlich

1
_dataPort = *port | (uint8_t)(((uint8_t) 1) << dataPin);
Antwort Zitieren Bearbeiten Löschen Melden Danke
Theor Gast #6048868 
Ups. Meine Antwort war leider völlig daneben.


Es ist völlig korrekt anzunehmen, dass:

1
  *ddr |= ddrNewSate;

2
  *port |= portNewState;


äquivalent zu:

1
  *ddr = *ddr | ddrNewSate;

2
  *port = *port | portNewState;


ist.

Sorry.
Sollte erstmal was essen. :-)
Antwort Zitieren Bearbeiten Löschen Melden Danke
Jemand Gast #6048901 
Nop schrieb:
> Ah vergessen, die Tags zuzumachen.
>
> ALso gemeint ist wohl
> _dataPort = *port | (1 << dataPin);
> Und performance-freundlich
> _dataPort = *port | (uint8_t)(((uint8_t) 1) << dataPin);

Schnell geht auch mit -O1 oder höher, dann muss die Wartbarkeit des 
Codes nicht mit deiner Unkenntnis der C-Promotionsregeln 
verschlimmbessert werden.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Felix Gast #6048916 
Nop schrieb:
> [c]_dataPort = *port | (1 << dataPin);[c]

Ja, das war es.
Aber warum ist

Nop schrieb:
> [c]_dataPort = *port | (uint8_t)(((uint8_t) 1) << dataPin);[c]

schneller?

Du castest die 1 auf uint8_t, machst bitshift nach links und castest das 
Ergebnis wieder nach uint8_ t. Aber warum ist das jetzt schneller als 
nur die eins anf für sich um dataPin nach links zu shiften?
Antwort Zitieren Bearbeiten Löschen Melden Danke
Nop Gast #6048927 
Jemand schrieb:

> Schnell geht auch mit -O1 oder höher, dann muss die Wartbarkeit des
> Codes nicht mit deiner Unkenntnis der C-Promotionsregeln
> verschlimmbessert werden.

Dann erklär doch mal, wie Du den an sich unnötigen Cast auf mindestens 
16 Bit verhinderst, der allein deswegen schon ausgelöst wird, weil 1 ein 
Integer ist.

Verläßt Du Dich tatsächlich auf die Compiler-Optimierung?
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 6 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.

Dein Name und deine E-Mail-Adresse

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.