Fehlersuche im Code für ATmega 8

[Dane (Gast)]

Hallo,

ich steige gerade wieder neu in die Programmierung von µCen ein und 
wollte einfach mal eine einfache Anwendung testen. Leider komme ich 
nicht darauf, was ich falsch mache - wahrscheinlich seh ich nur den Wald 
vor lauter Bäumen nicht - vielleicht kann mir jemand von euch kurz 
helfen.

Ich benutze das aTeVal Board von eHajo mit einem ATmega8A mit folgender 
Pin-Belegung: PD2, PD3, PD4 Taster, die auf Ground schalten, PD5, PD6 
LEDs

Was jetzt will ich einfach nur realisieren, dass die LED1 leuchtet, wenn 
ich den Taster1 drücke und die LED2 leuchtet, wenn ich den Taster2 
drücke.

Mein LED Test funktioniert ohne Probleme, ein anderes Beispiel mit den 
Tastern hat auch funktioniert. Nur dieses mal leuchten nach meinem LED 
Test beide LEDs dauerhaft.

Vielen Dank für euere Hilfe!

Vg Dane

 
#define F_CPU 8000000UL     /* 8MHz */
 
#include <avr/io.h>
#include <util/delay.h>
 
// Prototypen
void long_delay(uint16_t);
 
// Defines:
#define LED1    (1<<PD6)
#define LED2    (1<<PD5)

#define Tas1    (1<<PD2)
#define Tas2    (1<<PD3)
#define Tas3    (1<<PD4)
 
void long_delay(uint16_t ms)
{
  for(; ms>0; ms--)
  _delay_ms(1);
}
 
void main()
{

 
  // LED-Pins als Ausgang definieren
  DDRD |= LED1 | LED2;
  // Taster-Pins als Eingänge definieren (Pullups-Zuschalten)
  DDRD |= Tas1 | Tas2 | Tas3;
 
  //LED Test
  PORTD |= LED1;
  long_delay(100);
  PORTD &= ~LED1;
  
  PORTD |= LED2;
  long_delay(100);
    PORTD &= ~LED2;
  
  
  while(1)
  {
    
    
    if (!(PIND & Tas1))
    {
      PORTD |= LED1;
    }
    else
    {
      PORTD &= ~LED1;
    }
    
    
    if (!(PIND & Tas2))
    {
      PORTD |= LED2;
    }
    else
    {
      PORTD &= ~LED2;
    }
    
    
    }
}

[Karl H. (kbuchegg)]

Der erste:

>  // Taster-Pins als Eingänge definieren (Pullups-Zuschalten)
>  DDRD |= Tas1 | Tas2 | Tas3;

Nö.
Du schaltest die Pins auf Ausgang.
Pullups werden bei Eingangspins über PORTD zugeschaltet.

[Oliver (Gast)]

Dane schrieb:
> // Taster-Pins als Eingänge definieren (Pullups-Zuschalten)
>   DDRD |= Tas1 | Tas2 | Tas3;

Über die Zeile solltest du nochmals nachdenken.

Oliver

[Dane (Gast)]

Ohmann, ich Honk...

Vielen Dank, da hätte ich noch ewig gesucht - dummes Copy&Paste.

Dankeschön!