Taster abfrage in C

[Dario (Gast)]

Hi

Ich habe diesen Code geschrieben:

#include <avr/io.h>

int main(void)
{
    DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (0 << DDA3) | (0 << DDA4) | (0 << DDA5) | (0 << DDA6) | (0 << DDA7);
  
    while(1)
    {


        if ( PINA & (1 << PINA3) ) {
          PORTA = (1 << PA0);
        }
        
        if ( PINA & (0 << PINA3))  {
          PORTA = (0 << PA0);
        }
  

    }
}

Die LED bleibt angeschaltet, wenn ich den Taster drücke.
Wenn ich den Code so abändere:

#include <avr/io.h>

#include <avr/io.h>

int main(void)
{
    DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (0 << DDA3) | (0 << DDA4) | (0 << DDA5) | (0 << DDA6) | (0 << DDA7);
  
    while(1)
    {


        if ( PINA & (1 << PINA3) ) {
          PORTA = (1 << PA0);
        }
        
        if (!( PINA & (0 << PINA3) )) {
          PORTA = (0 << PA0);
        }
  

    }
}

Funktioniert es. Warum ist das so?

lgd

[Peter II (Gast)]

Dario schrieb:
> Funktioniert es. Warum ist das so?

weil (0 << PINA3)

zu 99.99% Unsinn ist und keinen sinn ergibt. Denn ist es 0

  if ( PINA & (0 << PINA3))  {

und wenn man eine 0 mit irgendetwas verundet kommt immer 0 raus.

Auch das ist fehlerhaft:

>  PORTA = (0 << PA0);
auch wenn es hier funktioniert. Es macht aber viel mehr als ein Pin zu 
löschen und führt später Problemen.

[Dario (Gast)]

Der 2. Code muss natürlich:
#include <avr/io.h>

int main(void)
{
    DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (0 << DDA3) | (0 << 
DDA4) | (0 << DDA5) | (0 << DDA6) | (0 << DDA7);

    while(1)
    {


        if ( PINA & (1 << PINA3) ) {
          PORTA = (1 << PA0);
        }

        if (!( PINA & (1 << PINA3) )) {
          PORTA = (0 << PA0);
        }


    }
}
lauten.

[Der kleine Nils (Gast)]

Das Problem ist, dass du die Logik hinter der Schiebeoperation nicht 
verstanden hast.

Schau mal in diesen Artikel

http://www.mikrocontroller.net/articles/Bitmanipulation

in den Abschnitt "Standard C"

[Der kleine Nils (Gast)]

Der kleine Nils schrieb:
> in den Abschnitt "Standard C"

sollte natürlich heissen "jeweils in die Abschnitte Standard C"

[Dario (Gast)]

Ich verstehe das Problem immer noch nicht. Was ist denn bei meinem Code 
falsch?

[Max H. (hartl192)]

Dario schrieb:
> PORTA = (0 << PA0);
Das ist das gleiche wie PORTA=0;

Dario schrieb:
> !( PINA & (0 << PINA3)
Das ist immer TRUE

[Thomas E. (thomase)]

Dario schrieb:
> Der 2. Code muss natürlich:
> #include <avr/io.h>
>
> int main(void)
> {
>     DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (0 << DDA3) | (0 <<
> DDA4) | (0 << DDA5) | (0 << DDA6) | (0 << DDA7);
>
>     while(1)
>     {
>         if ( PINA & (1 << PINA3) ) {
>           PORTA = (1 << PA0);
>         }
>         if (!( PINA & (1 << PINA3) )) {
>           PORTA = (0 << PA0);
>         }
>     }
> }
> lauten.

Das funktioniert auch nicht. Und wenn doch, dann spätestens wenn an der 
Nordseeküste die nächste Flut einsetzt, nicht mehr. Es sei denn, du hast 
an PA3 einen Pullup gelötet.

#include <avr/io.h>
 
int main(void)
{
     DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2);
     PORTA |= (1 << 3);
     while(1)
     {
         if ( PINA & (1 << PINA3) ) 
         {
         //  PORTA = (1 << PA0);
             PORTA |= (1 << PA0);
         }
         if (!( PINA & (1 << PINA3) ))
         {
           //PORTA = (0 << PA0);
           PORTA &= ~(1 << 0);
         }
     }
}

Das ist aber immer noch nichts Dolles. Das lässt sich auch mit einem 
Taster alleine lösen. Ganz ohne Controller.
Für den nächsten Schritt, einmal Drücken EIN, nochmal Drücken AUS, muß 
dein Taster entprellt werden. Aber verinnerliche erstmal das Thema 
Bitmanipulation.

mfg.

[Dario (Gast)]

Ich habe einen 10k Pullup Wiederstand an PA3.

[Thomas E. (thomase)]

Dario schrieb:
> Ich habe einen 10k Pullup Wiederstand an PA3.
Wohl eher einen Widerstand. Dann hat sich das mit der Flut erledigt.
Der Widerstand ist aber unnötig, da der Controller Pullups eingebaut hat 
und der entsprechende mit

PORTA |= (1 << 3);

aktiviert wird.

Guck dir die letzte Zeile in meiner Version an und vergleiche sie mit 
deiner auskommentierten darüber. Wenn das nur Stirnrunzeln bei dir 
verursacht:

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Ausg.C3.A4nge

mfg.

[Karl H. (kbuchegg)]

Dario schrieb:
> Ich verstehe das Problem immer noch nicht.


Es gibt einen fundemantalen Unterschied zwischen

     1 << PA_irgendwas

und

     0 << PA_irgendwas

0 um eine ANzahl Bits nach links geschoeben, ist immer noch 0!

Das ist dasselbe wie in der Mathe

  1 * 1   = 1
  1 * 2   = 2
  1 * 3   = 3
  1 * 4   = 4
  ...

ABER

  0 * 1   = 0
  0 * 2   = 0
  0 * 3   = 0
  0 * 4   = 0
  ...

egal womit du 0 multiplizierst, das Ergebnis ist immer (in allen Fällen) 
0. Im Falle das du 1 multiplzierst kriegst du jeweils ein anderes 
Ergebnis. Aber nicht bei 0., Die Ergebnisse sind alle gleich, nämlich 0

Nimm den binären Wert 1 her

   0000 0001

und verschieb den um 5 Stellen nach links

   0000 0001   << 5

-> 0010 0000

das Ergebnis ist ein binärer Wert, der genau an der Bitposition 5 ein 
1-Bit hat. Genau darauf zielt man ja mit 1 << xxx ab: einen Wert zu 
kriegen, der genau an der Bitposition xxx ein 1 Bit aufweist.

Aber:
Das funktioniert, wenn man eine 1 schiebt.
Aber bei einer 0, so wie in 0 << PA0, ergibt die Operation keinen Sinn. 
Denn eine binäre 0 kannst du nach links schieben so oft du willst

   0000 0000   << 5

das Ergebnis ist wieder einfach nur

   0000 0000

Daher ist

   PORTA = (0 << PA0);

in allen Belangen völlig gleichwertig zu

   PORTA = 0;

Dass du da in deinem Ausdruck PA0 drinnen stehen hast, ist zwar nett, 
aber komplett irrelevant, weil es nichts bewirkt. De Facto hast du 
einfach nur an PORTA eine 0 zugewiesen.

[Dario (Gast)]

Danke für die tolle Erklärung!
Um das zu verinnerlichen wollte ich so eine LED zum Blinken bringen. 
Aber ich
finde den Fehler nicht :(
#include <avr/io.h>
#include <util/delay.h>

#define F_CPU 1000000

int main(void)
{
  DDRA = 0;
  DDRA |= (1 << 0) | (1 << 1) | (1 << 2);

    while(1)
    {
    PORTA |= (1 << 0);
    _delay_ms(1000)
    PORTA &= ~(1 << 0);
    }
}

[Felix P. (fixxl)]

Die Zeit vom Ausschalten zum Wiedereinschalten ist so kurz, dass du sie 
mit den Augen nie wahrnehmen kannst. Es fehlt noch ein Delay.

[Jens H. (jens_h19)]

Dario schrieb:
> while(1)
>     {
>     PORTA |= (1 << 0);
>     _delay_ms(1000)
>     PORTA &= ~(1 << 0);
>     }

vielleicht so:

    while(1)
    {
    PORTA |= (1 << 0);
    _delay_ms(1000);
    PORTA &= ~(1 << 0);
    _delay_ms(1000);
    }

bei dir fehlt ein ';' und die LED geht so schnell wieder an, dass du 
nichts siehst

[Dario (Gast)]

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

#define F_CPU 1000000

int main(void)
{
  DDRA = 0;
  DDRA |= (1 << 0) | (1 << 1) | (1 << 2);

    while(1)
    {
    PORTA |= (1 << 0);
    _delay_ms(1000);
    PORTA &= ~(1 << 0);
    }
}
Das ; bei _delay_ms hat gefehlt. Es blinkt trotzdem nicht :(

[Thomas E. (thomase)]

Dario schrieb:
> Danke für die tolle Erklärung!
> Um das zu verinnerlichen wollte ich so eine LED zum Blinken bringen.
> Aber ich
> finde den Fehler nicht :(
> #include <avr/io.h>
> #include <util/delay.h>
>
> #define F_CPU 1000000
>
> int main(void)
> {
>   DDRA = 0;
>   DDRA |= (1 << 0) | (1 << 1) | (1 << 2);
>
>     while(1)
>     {
>     PORTA |= (1 << 0);
>     _delay_ms(1000)
>     PORTA &= ~(1 << 0);
>     }
> }

     while(1)
     {
       PORTA |= (1 << 0);
       _delay_ms(1000);
       PORTA &= ~(1 << 0);
       //du mußt dir hier auch eine Sekunde gönnen
       _delay_ms(1000);
     }

mfg.

[Dario (Gast)]

Danke :)

[ich (Gast)]

oder auch noch einfacher:
> while(1)
>      {
>        PORTA ^= (1 << 0);
>        _delay_ms(1000);
>      }

[Dario (Gast)]

Ich stand gerade ein bisschen aufm Schlauch.

[Dario (Gast)]

Ich habe mein Ziel erreicht:

#include <avr/io.h>

#define F_CPU 1000000

int main(void)
{
  DDRA = 0;
  DDRA |= (1 << 0) | (1 << 1) | (1 << 2);
  
    while(1)
    {
    if ( PINA & (1 << PA3)) {
      PORTA |= (1 << PA0);
    }
    if ( PINA | (1 << PA3)) {
      PORTA &= ~(1 << PA0);
    }
      
    }
}

[Peter II (Gast)]

Dario schrieb:
> Ich habe mein Ziel erreicht:

was war das ziel? fehlerhafter Code?


>  if ( PINA | (1 << PA3)) {

ist zumindest schon mal nicht richtig.

[Karl H. (kbuchegg)]

Dario schrieb:
> Ich habe mein Ziel erreicht:

Nope

>

>
>     if ( PINA | (1 << PA3)) {
>
>

ganz sicher nicht.

Was ist denn so schwer dran

Abfrage, ob ein Pin auf 1 ist

  if( PINx & ( 1 << PinNr ) )
     ...

Abfrage ob der Pin auf 0 ist

  if( !( PINx & ( 1 << PinNr ) ) )

Oder in deinem speziellen Fall

    if ( PINA & (1 << PA3)) {
      PORTA |= (1 << PA0);
    }
    else {
      PORTA &= ~(1 << PA0);
    }

... da stellt sich dann die Fragestellung gar nichgt mehr. Denn wenn der 
Pin nicht auf 1 war (erstes if), dann ist automatisch klar, dass er auf 
0 sein muss. Eine andere Möglichkeit existiert gar nicht mehr. Der Pin 
ist entweder 1 oder er ist 0. Sobald klar ist, dass er nicht 1 ist, MUSS 
er 0 sein.

[Max H. (hartl192)]

Dario schrieb:
> PINA | (1 << PA3))
Das ist immer TRUE

[Dario (Gast)]

Okay

#include <avr/io.h>

#define F_CPU 1000000

int main(void)
{
  DDRA = 0;
  DDRA |= (1 << 0) | (1 << 1) | (1 << 2);

    while(1)
    {
    if ( PINA & (1 << 3)) {
      PORTA |= (1 << PA0);
    }
    if (!( PINA & (1 << 3)) ){
      PORTA &= ~(1 << PA0);
    }
    }
}
oder
#include <avr/io.h>

#define F_CPU 1000000

int main(void)
{
  DDRA = 0;
  DDRA |= (1 << 0) | (1 << 1) | (1 << 2);

    while(1)
    {
    if ( PINA & (1 << 3)) {
      PORTA |= (1 << PA0);
    }
    else{
      PORTA &= ~(1 << PA0);
    }
    }
}
Jetzt müsste es richtig sein. Wie kann ich den Code in ein Codefenster 
schreiben?(so wie Ihr)

[Max H. (hartl192)]

Dario schrieb:
> Wie kann ich den Code in ein Codefenster
> schreiben?(so wie Ihr)
[c*]C-Code[/c]
Nur ohne das '*'

@Mod: Sry, falls ich den oberen Beitrag aus Versehen gemeldet habe. Ich 
habe beim Zitieren vllt. den falschen Button angeklickt.