Was mache ich falsch

[Stefan (Gast)]

Hallo, ich bin AVR anfänger und versuche mich immoment mit der 
Bitmanipulation...wenn ich nun am pin pc5 ein signal bekomme soll der 
pin pb1 high gesetzt werden hier mal mein code :

#include <avr/io.h>
#include <stdint.h>

int main (void) {     

 /* alle Pins von Port D als Eingang */
 DDRD  = 0x00; 
 /* alle Pins von Port B als Ausgang */
 DDRB  = 0xff;  


  while(1) {              
         /* Wenn PC5==1 dann Led ein */
  
         if ( PINC & (1<<PINC5) ) {
         /* Led ein*/
         PORTB = 0b00000001;
               }
          }                       
 

   return 0;                
}

danke für jeden tipp!!

[Läubi .. (laeubi)]

Stefan schrieb:
> Was mache ich falsch

Keinen Sinnvollen Betreff zu wählen!

[Stefan (Gast)]

also ist der code richtig ??

[tork (Gast)]

Ich kann keinen Fehler entdecken, aber versuch doch mal de
n Pin ohne Abfrage zu setzen.

[Stefan (Gast)]

okay danke ich versuche es noch mal so^^

[avr (Gast)]

        if ( PINC & (1<<PINC5) ) {
         /* Led ein*/
         PORTB = 0b00000001;
               }
        else{ 
         /* Led aus*/
         PORTB = 0b00000000;
               }

Auch mal ausschalten ;)

avr

[Stefan (Gast)]

bringt es das wirklich??

[Rolf Magnus (Gast)]

Wenn er wieder ausgehen soll, ja.

[Stefan (Gast)]

ja aber else wird ansich doch für das einschalten nicht benötigt oder??

[Robert K. (mr_insanity)]

Stefan schrieb:
> ja aber else wird ansich doch für das einschalten nicht benötigt oder??

Richtig. Aber für das Ausschalten. Wenn Du dem Controller nicht sagst, 
was er machen soll wenn PC5 low ist, geht die LED nicht wieder aus.

[Bernd (Gast)]

Wie prüfst du denn, ob der Pin bzw. die LED an ist?
Im Simulator, durch die LED, durch Messen?
Vielleicht machst du da was falsch.
Ansonsten sehe ich ein "}" zu viel und das mit dem Return, ist das so 
richtig?
Probier doch mal ein void main(void) { ... } ohne return.

[Günter R. (galileo14)]

Im wesentlichen mußt du doch mal testen, ob deine LED auf das Setzen des 
Port-B-Pins überhaupt reagiert, d.h. ob du richtig gelötet hast.

Dazu solltest du einfach mal per Programm den PB0 auf low bzw. auf high 
setzen, dann muß die LED leuchten bzw. ausgehen (je nachdem ob du sie 
gegen GND oder Vcc geschaltet hast, Vorwiderstand nicht vergessen!). 
Wenn das funktioniert (und erst dann), kannst du mit dem Einlesen von 
Pins weitermachen und damit die LED steuern. Aber wie die Vorredner 
schon angesprochen haben: es wäre wohl nicht schlecht, wenn du sie auch 
wieder ausschalten könntest (else-Zweig), denn jetzt ginge sie ggf. 
durch den kleinsten Impuls an und leuchtet halt dann weiter, egal was an 
PC5 danach passiert.

[Bernd (Gast)]

Tschuldigung... ich meinte ein int main(void) { ... } aber ohne return 
in der while(1) !!

[Stefan (Gast)]

danke für die vielen infos, ich vermute mal stark dass ich auf die 
schnelle einen fehler in der lötung habe^^ Der else-Zweig wird jetzt 
noch ergänzt und dann versuche ich es mit nachgearbeiteter schaltung 
noch einmal...

[Sebastian R. (Gast)]

Hi!

Das return ist okay, auf dem uC aber nicht so interessant.
Die Anzahl der geschweiften Klammern ist korrekt, wenn auch seltsam 
eingerückt.

Bei diversen AVRs liegt auf PORTC das JTAG, das werksseitig aktiviert 
ist. Dann funktioniert einige der Pins des Ports nicht "einfach" als 
Ein-/Ausgang. Über eine Fuse lässt sich ggf. das JTAG abschalten.

Gruß

[Bernd (Gast)]

Jetzt habe ich es auch gesehen.
Klammern passen und das return steht unterhalb der while(1) Schleife, 
wird also nie ausgeführt.

[NV (Gast)]

Du definierst Port D als Eingang, oder nicht?

     /* alle Pins von Port D als Eingang */
     DDRD  = 0x00;

...fragst aber auf Port C ab!

    if ( PINC & (1<<PINC5) )

[Sebastian R. (Gast)]

Wenn das Programm so mit WinAVR übersetzt wurde und nicht noch extra ein 
Bootloader irgendwas verkurbelt, dann ist es okay.
(Ports sind erstmal alle Eingänge)

Ein Fehler außerhalb des Programmes wäre am Wahrscheinlichsten.
Wie tork schrieb, einfach mal nur

PORTB = 0b00000001;

in die while Schleife und gucken, am richtigen Pin ein high raus kommt.

[fliegermichl (Gast)]

Hallo,

Zunächst einmal möchte ich noch mal auf die Antwort von NV hinweisen. Da 
ist schonmal der erste Fehler.

Wenn Du an PortD.0 einen Taster hast, der nach Masse schaltet, dann ist 
das Bit in PIND.0 gesetzt, wenn der Taster nicht gedrückt ist und 
gelöscht, wenn der Taster gedrückt ist.

Die Abfrage muss dann also auf ein gelöschtes Bit hin geändert werden.

if ( ! (PIND & 1<<PD0)) {
 // LED anschalten
 PORTB |= 1<<PB0;
} else {
 // LED ausschalten
 PORTB &= ~(1<<PB0);
}

[Ben S. (theben)]

Was mir noch ein fallen würde ist das du einen eingang ab fragst und 
vorher den pullup nicht setzt

/* alle Pins von Port D als Eingang */
 DDRD  = 0x00; 
/* alle Pull up von Port D setzen */
 PortD  = 0xFF;

dies musst du machen wenn du deinen z.b. Taster diereckt an einem µC 
Eingang geschalten hast.