Problem bei Programm mit Taster und LED

[Tim (Gast)]

Hallo Gemeinde,

ich bin blutiger Anfänger und habe daher einen für die meisten 
wahrscheinlich einfaches Problem. Ich hoffe aber, dass ihr mit helfen 
könnt/wollt.

Ich möchte nur ein einfaches Programm schreiben mit zwei Tastern und 
zwei LED`s.
Die eine LED soll "selbst" gehalten werden und die andere jeweils beim 
Tastendruck an gehen und anschließend wieder aus gehen.
Allerdings ist einer der LED`s von Beginn an, an und bei der anderen tut 
sich garnichts. Könntet Ihr bitte mal über meine Code rüber schauen und 
mit vllt verraten was ich dort falsch gemacht habe. Ich komme schon seit 
Tagen einfach nicht mehr weiter.

Vielen Dank im Vorfeld.

#define F_CPU 8000000
#include <avr/io.h>

#include <avr/delay.h>

int main(void)
{
  
  DDRB |=(1<<PB6); // PB 6 Ausgang 
  DDRB |=(1<<PB7); // PB7 Ausgang
  DDRD &=~ (1<<PD2); //PD 2 Eingang (Taster) 
  PORTD = (1<<PD2); // Pullup ein 
  DDRD &=~ (1<<PD3); //PD 3 Eingang (Taster) 
  PORTD = (1<<PD3); // Pullup ein
    
    while(1)
    {
  
  if((PIND & (1<<PD3))==0)
  
  {
    _delay_ms(50);
    
    
    
    if((PIND & (1<<PD3))==0)
    {
    PORTB |=(1<<PB7);
    }
    else 
    {
      PORTB &=~(1<<PB7);
    }
  }

  if((PIND & (1<<PD2))==0)
  
  {
    _delay_ms(50);
  
    if((PIND & (1<<PD2))==0)
    {
      
      PORTB ^=(1<<PB6);
    }
   while((PIND & (1<<PD2))==0);        
  }
  
  
    
  }
    
  }

[Rene H. (ballibou77)]

Hi,

#define F_CPU 8000000
#include <avr/io.h>
#include <avr/delay.h>

#define TASTER1_DDR DDRD
#define TASTER2_DDR DDRD
#define TASTER1_PIN PD2
#define TASTER2_PIN PD3
#define TASTER1_PORT PORTD
#define TASTER2_PORT PORTD
#define TASTER1_PIND PIND
#define TASTER2_PIND PIND

#define LED1_DDR DDRB
#define LED2_DDR DDRB
#define LED1_PIN PB6
#define LED2_PIN PB7
#define LED1_PORT PORTB
#define LED2_PORT PORTB

int onoff_var = 0;

LED1_PORT &= ~(1<<LED1_PIN); // schalte LED1 Aus
LED2_PORT &= ~(1<<LED2_PIN); // schalte LED2 Aus

inline uint8_t debounce(volatile uint8_t *port, uint8_t pin)
{
    if ( !(*port & (1 << pin)) )
    {
        /* Pin wurde auf Masse gezogen, 100ms warten   */
        _delay_ms(50);   // Maximalwert des Parameters an _delay_ms 
        _delay_ms(50);   // beachten, vgl. Dokumentation der avr-libc
        if ( *port & (1 << pin) )
        {
            /* Anwender Zeit zum Loslassen des Tasters geben */
            _delay_ms(50);
            _delay_ms(50); 
            return 1;
        }
    }
    return 0;
}
int main(void)
{

  LED1_DDR |=(1<<LED1_PIN); // PB 6 Ausgang
  LED2_DDR |=(1<<LED2_PIN); // PB7 Ausgang
  TASTER1_DDR &=~ (1<<TASTER1_PIN); //PD 2 Eingang (Taster)
  TASTER1_PORT = (1<<TASTER1_PIN); // Pullup ein
  TASTER2_DDR &=~ (1<<TASTER2_PIN); //PD 3 Eingang (Taster)
  TASTER2_PORT = (1<<TASTER2_PIN); // Pullup ein

while(1)
 {
    if (debounce(&TASTER1_PIND, TASTER1_PIN)) { // Taster 1 entprellen
      if (onoff_var == 0) { onoff_var = 1; } // setze LED Zustand auf 1 wenn 0 war
      if (onoff_var == 1) { onoff_var = 0; } // setze LED Zustand auf 0 wenn 1 war
    }

    if (debounce(&TASTER2_PIND, TASTER2_PIN)) { // Taster 2 entprellen
      LED2_PORT |= (1<<LED2_PIN); // LED 2 einschalten
      _delay_ms(1000); // 1 sek warten
      LED2_PORT &= ~(1<<LED2_PIN); // LED2 wieder ausschalten
    }

    if (onoff_var == 1)
      {
        LED1_PORT |= (1<<LED1_PIN); // schalte LED1 ein wenn Zustand 1 ist
      }
      else
      {
        LED1_PORT &= ~(1<<LED1_PIN); // schalte LED1 aus wenn Zustand 0 ist
      }
 }
}

Musst nur noch oben die defines richtig einsetzen.

Ich möchte dir nicht das ganze C vorkauen. Guck dir die unterschiede im 
Code an und versuche sie zu verstehen. Ist eigentlich ganz simpel, in 
ein paar minuten geschrieben und funktioniert ;) (gerade mal getestet)

Die Taster werden entprellt. Wenn du nun den Taster an der LED drückst 
die nur kurz eingeschaltet werden soll dann geht die LED an, der µC 
wartet 1 sek (1000ms) und schaltet die LED wieder aus.

Bei der anderen LED setzt er jeweils die Zustand in die onoff_var. Diese 
ist von Start an 0. Wenn du nun den Taster drückst wird diese auf 1 
gesetzt. Steht dort bereits 1 drinne wird, wenn du den Taster wieder 
drückt dort eine 0 rein geschrieben usw. Am ende des Codes wird dann 
geschaut was in der variable steht. Steht dort eine 1 wird die LED 
eingeschaltet / bleibt eingeschaltet. Steht dort eine 0 wird die LED 
ausgeschaltet / bleibt ausgeschaltet.

[Michael B. (laberkopp)]

Tim schrieb:
> DDRD &=~ (1<<PD2); //PD 2 Eingang (Taster)
>   PORTD = (1<<PD2); // Pullup ein
>   DDRD &=~ (1<<PD3); //PD 3 Eingang (Taster)
>   PORTD = (1<<PD3); // Pullup ein

Der ist ja wohl falsch, macht dir deimem PD2 pull up kaputt.

> if((PIND & (1<<PD3))==0)
>
>   {
>     _delay_ms(50);
>
>     if((PIND & (1<<PD3))==0)
>     {
>     PORTB |=(1<<PB7);
>     }
>     else
>     {
>       PORTB &=~(1<<PB7);
>     }
>   }

Lass die doppelte Abfrage und den Delay-Kram in diesem Fall weg. Du 
prüfst nur wenn der Tasteneingang gedrückt ist, ob er gleich danach 
immer noch gedrückt ist und schaltet nur in dem unwahrscheinlichen Fall 
daß er prellt und nicht mehr gedrückt ist, wieder aus.

>
>   if((PIND & (1<<PD2))==0)
>
>   {
>     _delay_ms(50);
>
>     if((PIND & (1<<PD2))==0)
>     {
>
>       PORTB ^=(1<<PB6);
>     }
>    while((PIND & (1<<PD2))==0);
>   }

Das ist Murks. Die while Schleife wartet bis der Taster losgelassen 
wird, aber er kann dabei genau so prellen wie beim runterdrücken.

Insgesamt ist die Form der Entprellung eher murksig. Du behandelst 
runterdrücken und loslassen unterschiedlich. Das ist physikalischer 
Unsinn. Ausserdem verbrätst du 10 mal mehr Programm als nötig.

[Tim L. (tim1407)]

Vielen Dank für die schnellen Antworten. Ich werde am We versuchen dort 
durch zu steigen.

Aber kurze Frage: Wieso mache ich mir mit diesem Code den Pull-Up kaputt
?

[Karl H. (kbuchegg)]

Weil eine einfache Zuweisung an einen Port ALLE Bits neu setzt.
Wenn du also schreibst

PORTD = (1<<PD3); // Pullup ein

was passiert dann mit dem Bit PD2?

[Stefan F. (Gast)]

> PORTD = (1<<PD2);
> PORTD = (1<<PD3);

Das ergibt aufgelöst in Bits:

PORTD = 0b00000100;
PORTD = 0b00001000;

Also ist danach nur ein Pull-Up eingeschaltet, nämlich der von PD3. 
Richtig geht es so:

> PORTD |= (1<<PD2);
> PORTD |= (1<<PD3);

Initial ist PORTD auf 0b00000000.

PORTD = PORTD | 0b00000100;
PORTD = PORTD | 0b00001000;

Danach hat PORTD den Wert 0b00001100.

[Marc V. (Firma: Vescomp) (logarithmus)]

Stefan U. schrieb:
> Richtig geht es so:
>
>> PORTD |= (1<<PD2);
>> PORTD |= (1<<PD3);

    PORTD |= (1<<PD2) | (1<<PD3);

[Tim L. (tim1407)]

ahhhh ich verstehe.

super, danke für die schnelle und vor allem nutzbare Hilfe .