ATmega8 ignoriert if-Anweisung

[Leigh B. (launebaer)]

Hi Leute,

ich habe gerade damit begonnen mit einem ATmega8 meine ersten 
Erfahrungen in µC-Programmierung zu sammeln. Bisher bin ich ganz froh 
darüber eine LED zum Blinken gebracht zu haben, möchte jetzt aber dass 
sie nur blinkt wenn ein Input-Pin HIGH ist. Dafür habe ich eine 
if-Anweisung eingebaut, aber die wird einfach mal gepflegt ignoriert. 
Die LED blinkt unabhängig davon was am Pin anliegt, obwohl es in der 
Simulation funktioniert hat.

Ich habe das Test-Programm so simpel wie irgend möglich gehalten, hier 
mal der Code:

#define F_CPU 1000000

#include <avr/io.h>
#include <util/delay.h>

int main(void)
{
  DDRB = 0x02;
  DDRD = 0x00;
  
    while(1)
    {    
        if(PD2)
        {
            _delay_ms(1000);
            PORTB = 0x02;
            _delay_ms(618);
            PORTB = 0x00;
        }
    }
}

Also:
- Pin B1 wird auf Output gesetzt um die LED ansteuern zu können
- in der while-Schleife beginnt sofort die Abfrage, da das ständig 
gemacht werden soll

Wäre bitte jemand so nett mir zu helfen? Ich blick da nicht ganz durch.

Danke

[Peter II (Gast)]

Leigh Beyst schrieb:
> Wäre bitte jemand so nett mir zu helfen? Ich blick da nicht ganz durch.

> if(PD2)

PD2 ist ein define und steht für eine 2

also steht dort

if(2)

und das macht wenig sinn.

[Martin S. (tungl)]

Siehe auch: 
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Signalkopplung

[icke (Gast)]

Eingänge liest man mit PINX. Das liefert den kompletten Port zurück.

Dann kann man binäre Operationen drauf ausführen, um den entsprechenden 
Pin zu erhalten.

Bei Dir also sowas wie

 if (PINB & 0x02)
{
...
}

[Cyblord -. (cyblord)]

icke schrieb:

> Bei Dir also sowas wie
>

 if (PINB & 0x02)
> {
> ...
> }
>

Ne, so:

if (PINB & (1<<PD2))
{}

Und noch besser, wenn man per #define sowohl PINB als auch PD2 sinnvolle 
Namen gibt. Dann sieht man auf den ersten Blick was passieren soll.

[Leigh B. (launebaer)]

Ok.

Erstmal danke für diese schnellen Antworten.

Ich habe jetzt jede Variation ausprobiert, die mir in den Sinn kam:

...
if(PD2)
{
...
}
...

LED blinkt immer

...
if(PIND2)
{
...
}
...

LED blinkt immer

...
if(PIND2 == 1)
{
...
}
...

LED blinkt nie

...
if(PIND == 0b00000100) //nur mal auf Nummer Sicher
{
...
}
...

LED blinkt nie

In keinem der Fälle reagiert es auf eine Änderung des Inputs.

[Martin S. (tungl)]

Lies bitte die Antworten bzw. das Tutorial richtig. Keine Deiner 
Varianten entspricht den dort aufgefuehrten.

[Leigh B. (launebaer)]

icke schrieb:
> Bei Dir also sowas wie if (PINB & 0x02)
> {
> ...
> }

Ok das hat es tatsächlich getan. Scheint nur eine Frage des Maskierens 
gewesen zu sein. Daran habe ich natürlich nicht mehr gedacht.

Vielen vielen Dank!!!

[Martin (Gast)]

Keiner Deiner "Lösungs-Varianten"  scheint irgendeine der Hilfs-Postings
aufzugreifen.

Autismus?

[Andreas B. (bitverdreher)]

Es ist schon ein Kunststück, alles zu testen und dabei zu ignorieren was 
oben schon geschrieben wurde.

Glückwunsch!

[Leigh B. (launebaer)]

Zu meiner Verteidigung: Ich hatte es getestet und war gerade dabei es 
abzutippen als die Vorschläge eingingen.