Taster? Anfängerfrage Port Reagiert nicht?

[Randy T. (peroja)]

Hallo,

Ich bin am herumexperimentieren mit einem Taster. Hier erst mal mein 
Code. Danach das komische Problem

#ifndef F_CPU
#define F_CPU 16000000UL // 8 MHz clock speed
#endif

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

int main(void)
{
  
DDRA = 0x00; // pin 1-7 an port A auf eingang
  
  while(1)
  {
    if(PINA & (1<<PA0) == 1) //button an pin pb0 ???? 
    {
      PORTD |= (1<<PD0); //LED an PD0 ein
      _delay_ms(100); //kurzes delay
      PORTD &= ~(1<<PD0); //LED off
    }
  }
}

Wenn ich das tue dann Funktioniert es:

if(PINA & (1<<PA0) == 1)

Wenn ich aber den Taster auf Pin2 setze :

if(PINA & (1<<PA1) == 1)

Dann gehts nicht? Oder hab ich da etwas falsch verstanden? (taster 
danach natürlich am pin 2 angehängt)

Auf dieses Problem bin ich gestossen nachdem ich 2 Taster an Port B 
angeschlossen hab und 2 versch. LED's an PortD ansteuern wollte. Ich 
dachte erst ich mach etwas falsch mit einem bit setzen.

#ifndef F_CPU
#define F_CPU 16000000UL // 8 MHz clock speed
#endif

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



int main(void)
{
  
   DDRB = 0x00; //Makes all pins of PORTD input
  
  while(1)
  {
    if(PINB & (1<<PB0) == 1) //button an pin pb0
    {
      
      PORTD |= (1<<PD0); //LED an PD0 ein
      _delay_ms(100); //kurzes delay
      PORTD &= ~(1<<PD0); //LED off
    }

    if(PINB & (1<<PB1) == 1) //button an pin pb1 
    {
      
      PORTD |= (1<<PD7); //LED an PD7 ein
      _delay_ms(100); //delay
      PORTD &= ~(1<<PD7); //LED off
    }

    
  }
}

versteh ich das richtig dass ich im 2.ten IF ein anderes bit setzen 
muss?

if(PINB & (2<<PB1) == 1) ->anderes/falsches bit?
    {
      
      PORTD |= <<PD7); //LED an PD7 ein (und hier?)
      _delay_ms(100); //delay
      PORTD &= ~(1<<PD7); //LED off (und hier?)
    }

[Jörg W. (dl8dtl) (Moderator)]

Randy T. schrieb:

> #define F_CPU 16000000UL // 8 MHz clock speed

Schon falsch. ;-)  Der Makro sagt, es seien 16 MHz, der Kommentar
meint, es sind 8.


> if(PINA & (1<<PA0) == 1)
>
> Wenn ich aber den Taster auf Pin2 setze :
>
> if(PINA & (1<<PA1) == 1)
>
> Dann gehts nicht?

Weil (1 << PA1) niemals gleich 1 sein kann.

(Wenn du alle Compilerwarnungen einschaltest, dann sagt dir der
Compiler das auch.)

Lass einfach das blöde "== 1" da weg, dann wird es klappen (also
generell weglassen, auch bei PA0, obwohl es dort zufällig klappt).

[Randy T. (peroja)]

Falls mir jemand freundlicher weise den unteren Code (2 buttons) 
korrigieren kann so wie es richtig wäre, dann wäre ich mega happy. Auf 
diese Weise kann ich vergleichen und sehe dann WO ich Fehler mache. (ist 
meine art zu lernen)

Lieben Dank
Randy

[Karl M. (Gast)]

Hallo  Randy Tomlinson,

es ist immer besser den Kopf selbst an zu strengen,
die Frage lässt sich einfach beantworten, wenn Du die Werte

(1<<PA0) = ??
(1<<PA1) = ??
(1<<PA2) = ??
usw.

ausrechnen kannst.

Es wurde in <avr/io.h> definiert, bzw. diese Datei lädt dann die 
Definitionen für deinen ausgewählten AVR µC nach:

/* PORTA */
#define PA7     7
#define PA6     6
#define PA5     5
#define PA4     4
#define PA3     3
#define PA2     2
#define PA1     1
#define PA0     0

[DDRD (Gast)]

Wo wird den Port D als Ausgang konfiguriert?
DDRD = 0xFF; //PortD als Ausgang

Sollte man auch hinschreiben um das klar zu haben.

[fop (Gast)]

if(PINA & (1<<PA1) != 0)

bzw. da gab es doch auch schon ein Makro :

if(PINA & _BV(PA1)) != 0)

[Jörg W. (dl8dtl) (Moderator)]

Das "!= 0" ist aber überflüssig.  Kann man hinschreiben, aber wenn man
C einmal verstanden hat, kann man's genauso gut weglassen.

[Falk B. (falk)]

@ Randy Tomlinson (peroja)

>korrigieren kann so wie es richtig wäre, dann wäre ich mega happy. Auf

Siehe Bitmanipulation.

[Randy T. (peroja)]

Falk B. schrieb:
> @ Randy Tomlinson (peroja)
>
>>korrigieren kann so wie es richtig wäre, dann wäre ich mega happy. Auf
>
> Siehe Bitmanipulation.

super. danke. werd ich mir morgen mal ansehen und testen

im endeffekt geht es mir nur darum zu lernen wie ich 2 odere mehr 
buttons zuweise und jedem ein eigenen ablauf zuteile.

dankeschön an alle helfer :-)

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Randy T. schrieb:
> Wenn ich das tue dann Funktioniert es:
> if(PINA & (1<<PA0) == 1)
> Wenn ich aber den Taster auf Pin2 setze :
> if(PINA & (1<<PA1) == 1)
Wenn schon schön umständlich, dann aber so richtig umständlich:

if(PINA & (1<<PA0) == (1<<PA0)) ...

Da ist bei korrekter Formatierung dank Obfuscation schon fast der 
Kapierschutz mit eingebaut:

if(PINA & (1<<PA1)==(1<<PA1) ) ...

;-)

[Jörg W. (dl8dtl) (Moderator)]

Lothar M. schrieb:
> Kapierschutz

ROTFL :-)

Randy T. schrieb:
> im endeffekt geht es mir nur darum zu lernen wie ich 2 odere mehr
> buttons zuweise und jedem ein eigenen ablauf zuteile.

Buttons muss man sowieso entprellen und fragt sie daher im
Timer-Interrupt ab … Entprellung

Aber die Bitmanipulation ist so grundlegend, dass man sie trotzdem
vorher verstanden haben sollte.

[Reiner_Gast (Gast)]

Ich würde ja die internen Pull-Ups an den Input Pins einschalten, sonst 
floaten die Inputs, wenn kein Taster gedrückt wird....

Die Taster müssen dann natürlich gegen GND schließen und die 
Abfragelogic ist negiert

[Rolf M. (rmagnus)]

Reiner_Gast schrieb:
> Ich würde ja die internen Pull-Ups an den Input Pins einschalten, sonst
> floaten die Inputs, wenn kein Taster gedrückt wird....

Wir kennen die Schaltung nicht. Könnte ja sein, dass da schon 
Widerstände drauf sind, die irgendwo hin pullen.

fop schrieb:
> bzw. da gab es doch auch schon ein Makro :
> if(PINA & _BV(PA1)) != 0)

Da gibt's noch mehr Makros:

if (bit_is_set(PINA, PA1))