AVR Atmega 328 Hilfe bei LED Taster Schaltung

[Guido L. (egudo)]

Hey Leute,
ich als blutiger Anfänger hänge jetzt schon an einer sehr einfachen 
Schaltung fest. Da ich auch begreifen und lernen möchte habe ich nicht 
einfach Beispiele kopiert sondern an denen angelehnt meinen eigenen 
Sourcecode geschrieben. Sinn meiner Schaltung soll sein, dass eine LED 
leuchtet wenn man einen Taster drückt. Lässt man den Taster los soll die 
LED auch ausgehen. Das einschalten der LED funktioniert auch tadellos 
nur aus ausschalten geht nicht immer. Es scheint so als wäre die LED 
beim loslassen des Tasters in keinem definiertem Zustand da sie mal an 
bleibt und mal ausgeht. Leuchtet die LED weiter, leuchtet sie beim 
nächsten Betätigen des Tasters heller. Da ich schon seit Tagen suche 
weiß ich auch nicht ob der Fehler im Sourcecode oder in der Schaltung 
liegt.

#include <avr/io.h>
#define F_CPU 16000000UL
#include <util/delay.h>


int main(void)
{
DDRB = 0xFF; //Register B Ausgänge
DDRC = 0x00; // Register C Eingänge
PORTC = 0xFF; // Register C Pullups ein

  while (1)
  {
    if (!(PINC & ( 1<<PC4 ))) //wenn PC4 HIGH dann...
    {
      PINB |= ( 1<<PB0 );  // LED an PB0 an
    }
    else
    {
      PINB &= ~ ( 1<<PB0 ); // ansonste LED an PB0 aus
    }
    }
    return 0;

[Einer K. (Gast)]

int main(void)
{
  DDRB  = 0xFF; // Register B Ausgänge
  DDRC  = 0x00; // Register C Eingänge
  PORTC = 0xFF; // Register C Pullups ein

  while (1)
  {
    if ((PINC & ( 1<<PC4 ))) //wenn PC4 High dann...
    {
      PORTB |= ( 1<<PB0 );  // LED an PB0 an
    }
    else
    {
      PINB &= ~( 1<<PB0 ); // ansonste LED an PB0 aus
    }
  }
  return 0;
}

[Frank L. (hermastersvoice)]

entprell den Taster und Alles wird gut

[Einer K. (Gast)]

Ein PINB vergessen, in PORTB umzubauen.

int main(void)
{
  DDRB  = 0xFF; // Register B Ausgänge
  DDRC  = 0x00; // Register C Eingänge
  PORTC = 0xFF; // Register C Pullups ein

  while (1)
  {
    if (!(PINC & ( 1<<PC4 ))) //wenn PC4 Low dann...
    {
      PORTB |= ( 1<<PB0 );  // LED an PB0 an
    }
    else
    {
      PORTB &= ~( 1<<PB0 ); // ansonste LED an PB0 aus
    }
  }
  return 0;
}

[Guido L. (egudo)]

Vielen Dank, das umwandeln von Pin in Port brachte den gewünschten 
Erfolg aber warum verwendet man hier den Port-Befehl. Ich dachte wenn 
man einzelne Pins ansteuert oder liest ist der Befehl PIN angebracht und 
wenn man das Register definieren möchte (z.B. Pullups) dann nimmt man 
Ports?

[Einer K. (Gast)]

Ein Irrtum.

Dann solltest du nochmal das Datenblatt deines µC befragen.
Dort steht recht exakt beschrieben, wie es zusammen hängt.

[Guido L. (egudo)]

Besten Dank hab es jetzt gerafft.

[Karl M. (Gast)]

Guido L. schrieb:
> Hey Leute,
> ich als blutiger Anfänger hänge jetzt schon an einer sehr einfachen
> Schaltung fest. Da ich auch begreifen und lernen möchte habe ich nicht
> einfach Beispiele kopiert sondern an denen angelehnt meinen eigenen
> Sourcecode geschrieben. Sinn meiner Schaltung soll sein, dass eine LED
> leuchtet wenn man einen Taster drückt. Lässt man den Taster los soll die
> LED auch ausgehen. Das einschalten der LED funktioniert auch tadellos
> nur aus ausschalten geht nicht immer. Es scheint so als wäre die LED
> beim loslassen des Tasters in keinem definiertem Zustand da sie mal an
> bleibt und mal ausgeht. Leuchtet die LED weiter, leuchtet sie beim
> nächsten Betätigen des Tasters heller. Da ich schon seit Tagen suche
> weiß ich auch nicht ob der Fehler im Sourcecode oder in der Schaltung
> liegt.
>
Hallo,
Deine Schaltung hat noch einen kleinen Fehler, AREF liegt nicht an AVcc 
sondern,
Er erhält nur einen kleinen 100nF Kondensator nach Masse natürlich.

[Guido L. (egudo)]

Ah ja klar vielen Dank für den Hinweis, hat zwar in diesem Fall nicht 
die Funktion beeinträchtigt aber man will ja von vornherein saubere 
Schaltungen aufbauen sowie saubere Codes schreiben. Also vielen Dank.

Bin generell sehr begeistert wie einem so blutigen Anfänger hier 
geholfen wird, auch wenn er so "banale" Fragen stellt.

[Thomas G. (Firma: Frickelhauptquartier) (taximan)]

Haben alle mal klein angefangen.

[Jörn P. (jonnyp)]

Wenn du an den Taster Eingang auch noch einen pull up Widerstand hängst,
dann "sieht" der Eingang auch einen definierten Pegel.

[Jens M. (schuchkleisser)]

Jörn P. schrieb:
> Wenn du an den Taster Eingang auch noch einen pull up Widerstand hängst,
> dann "sieht" der Eingang auch einen definierten Pegel.

Aua:

Guido L. schrieb:
> DDRC = 0x00; // Register C Eingänge
> PORTC = 0xFF; // Register C Pullups ein

Frank L. schrieb:
> entprell den Taster und Alles wird gut

Nö. Die Lampe soll so funktonieren als wäre der µC da gar nicht drin: 
Knopp gedrückt=Licht, loslassen=Dunkelheit.
Da muss nichts entprellt werden.

[neuer PIC Freund (Gast)]

Hallo Guido.

Benutze ruhig das Atmel Studio. Dort simulierst du deinen Code und 
schaust im IO-Viewer nach den Ports. Da fällt dir in windeseile auf, 
dass sich ein Register ja gar nicht ändert. Und schon ist der Typo 
gefunden.

[Guido L. (egudo)]

Danke für den Tipp, Atmel Studio verwende ich sogar, habe nur noch 
nichts simuliert, werde mich wohl auch mal da mal mit beschäftigen.