ACSR - Bit7 setzen

> if(bit_is_set(ACSR,MEINBIT7) == 1)
Schau bitte mal nach, was bit_is_set macht. Und dann lass das "== 1" 
weg.

Außerdem werden LEDs auch gerne schon mal High-Side angeschlossen, so 
dass der betreffende Pin Low-Pegel haben muss, um die LED anzuschalten. 
Da Du nirgends erwähnst, was für eine Hardware und was für einen 
Controller Du verwendest (Ich habe jetzt allein wegen des ACSR und ACD 
und der avr/io.h mal auf AVR getippt), könnte es auch noch sein, dass an 
PORTC noch andere Dinge zu beachten sind, die aber dann nicht alle Pins 
betreffen.

BTW:
Warum benutzt Du eigentlich nicht die Definition für ACD und machst den 
MEINBIT7-Krampf?

J.Flender wrote:
> Oh, tut mir leid. Ich hatte leider vergessen zu erwähnen, was es für ein
> µC ist. (Ist auch mein erster Thread hier).
Dass man den verwendeten Controller im Betreff angeben soll, steht aber 
in den Forenregeln, deren Gelesen-haben man vor dem Absenden bestätigen 
muss...

> Der Atmega weigert sich halt, das Bit auf 1 zu schalten. Und ich habe
> keine Ahnung wieso...
Wie sieht denn jetzt Deine Überprüfung aus? Ich befürchte, Du hast nicht 
verstanden, dass nicht das Setzen des Bits falsch ist, sondern Deine 
Abfrage, ob es gesetzt ist! Die Bedingung hinter dem if kann so 
nicht wahr werden, weil auf "== 1" abgefragt wird. Der Term vorne kann 
aber nie gleich 1 werden. Du musst an dieser Stelle eine 
Interpretation des Ausdrucks "bit_is_set(ACSR, ACD)" bzw. der besseren 
(weil ohne irgendwelche nicht-Standard-Makros, deren Implementierung man 
erst nachschlagen muss, auskommenden) Alternative "ACSR & (1 << ACD)" 
als Wahrheitswert erzwingen.

Der Ausdruck "ACSR & (1 << ACD)" kann niemals gleich 1 sein, weil er 
nur die beiden Zustände "0" oder "1 << ACD" (was aufgelöst 0x80 oder 
dezimal 128 ist) annehmen kann! Wenn Du das "== 1" einfach weglässt, 
dann wird der restliche Ausdruck automatisch als Wahrheitswert 
interpretiert. Die Anweisung hinter dem if wird dann ausgeführt, wenn 
der Ausdruck in den Klammern () wahr, also von Null verschieden ist...

J.Flender wrote:
> Es funktioniert. Sehr schön.
> Ich habe, wie du gesagt hast, einfach die ==1 im if() weggelassen und es
> funktioniert.
Sag ich ja ;-) Ich hoffe, Du hast das nicht nur treudoof gemacht, 
sondern auch verstanden, warum das so sein muss. Das wird Dir 
nämlich noch des öfteren über den Weg laufen.

> Nochmal vielen Dank für deine Bemühungen.
> Wünsche noch einen schönen Abend!
Gleichfalls.

P.S.:
Wie oben schon erwähnt solltest Du Dir am besten gleich die 
Standard-konforme Schreibweise für das Setzen/Löschen von Bits bzw. das 
Auslesen ihres Zustandes angewöhnen. bit_is_set und Konsorten sind 
Makros, die einer bestimmten Bibliothek eines bestimmten Compilers 
entstammen und mit denen niemand, der einen anderen Compiler benutzt, 
etwas anfangen kann. Der Artikel Bitmanipulation auf dieser Seite 
behandelt die Möglichkeiten.