lese zustand PA5 gebe ihn auf PA4 aus

[Travolds]

hallo und erstmal danke für die antwort,

Den Widerstand R3 mit 100k kannst Du Dir normalerweise sparen,
Weil Du ja sowieso die internen Pullup-Widerstände des AVRs benutzt.

ok danke

Ich will Dir keinesfalls zu nahe treten, sind nur Verbesserungsvorschläge:
Ich würde anstatt:
DDRB |= _BV(LED);
lieber:
DDRB |= (1<<LED);
verwenden.
Ist "standardisierter", und lässt sich von den meisten besser lesen...

ich sehe das nicht als zu nahe treten bin für jeden tipp dankbar

Ich denke dein erster Fehler liegt schon in der ersten Zeile:
"if (bit_is_set(PINA, BUTTON))"
"Wenn Button-Pin High ist"

*am kopf hau* ist natürlich falsch ich habs aber nicht gesehen

Und ich wette, dass die LED bei manchen Tastenbetätigungen auch AUS ist.

if (bit_is_clear(PINA, BUTTON))

kannte ich noch nicht - wieder was gelernt

Das gaze ist aber auch kritisch!
Du müsstest die Tastenprellung berücksichtigen.
Mit dem Finger Tippst du zwatr nur einmal auf den taster,
aber der Kontakt prallt ein paar mal hin und her.
Also "Tippt" er ein paar mal mehr.
Daher kann es sein, dass die LED an und gleich wieder aus geht,
ohne dass man es sieht.

die entprellproblematik ist mir bewusst, mir ging es hier rein darum basics zu üben

vielen dank nochmal!

[Kaiser-F]

Kein Problem, dazu sind Wir ja da!

Noch ein Tipp:
Das "if (bit_is_set(PINA, BUTTON)){}" sieht man auch nicht mehr gerne...

Sieht man fast nur so:
if( PINA & (1<<BUTTON) ){}

Diese Sachen sind halt "klassischer", "standardisierter" für C.
Wenn Du Dich von Anfang an an die "normalen" C-Anweisungen hältst,
tuhst dich wesentlich leichter, wenn Du mal was außerhalb der
ATMEL-GCC-Umgebung machst.

Für den Anfang spielt es aber keine Rolle.

Viel Spaß noch! Du greifst es genau richtig an!