Attiny13 Lauflicht+Taster Problem

[Dex (Gast)]

Hallo alle zusammen ich sitze im moment an einem kleinen Lauflicht 
welches auf ein Drucktaster reagieren soll.

 #include <avr/io.h>
 #include <util/delay.h>
 
 int
 main (void)
 {
     DDRB=0b001111;         //PB0,PB1,PB2,PB3 als Ausgänge geschaltet    
   PORTB|=(1<<PORTB4);
   
   while(1) 
        {
        if(!(PORTB&(1<<PINB4)))
        {
          PORTB=0b000000;
          return 0;        
        }          
          
    _delay_ms(1000);
    PORTB|=(1<<PORTB0); //Oder-Verknüpfung mit 1 an PB0  // PORTB|=0b000001;
     
    
    
    _delay_ms(1000);
    PORTB|=(1<<PORTB1);//Oder-Verknüpfung mit 1 an PB1 //PORTB|=0b000010;
    
    
     _delay_ms(1000);
    PORTB|=(1<<PORTB2);//Oder-Verknüpfung mit 1 an PB2 //PORTB|=0b000100;
    
    
     _delay_ms(1000);
    PORTB|=(1<<PORTB3);//Oder-Verknüpfung mit 1 an PB3 //PORTB|=0b001000;       
    
    
    _delay_ms(1000);
    PORTB^=(1<<PORTB0); //Invertierung von PB0
     
    
    
    _delay_ms(1000);
    PORTB^=(1<<PORTB1);//Invertierung von PB1 
    
    
     _delay_ms(1000);
    PORTB^=(1<<PORTB2);//Invertierung von PB2 
    
    
     _delay_ms(1000);
    PORTB^=(1<<PORTB3);//Invertierung von PB3       
    
      }
 }

wie im Schematic hoffentlich zu erkennen habe ich an PB4 einen 
Drucktaster geschaltet und im den internen Pull up von PB4 
eingeschaltet. wird der taster gedrückt sollen alle 4 LEDs leuchten. 
beim Testen ist mir allerdings aufgefallen dass PB4 genauso wie die 
anderen Pins als ausgang fungiert und permanent getoggelt wird. Was kann 
ich falsch gemacht haben?

[Karl H. (kbuchegg)]

if(!(PORTB&(1<<PINB4)))
        {
          PORTB=0b000000;


Gratuliere. Du hast dir soeben den Pullup Widerstand ausgeschaltet.

[Karl H. (kbuchegg)]

>         if(!(PORTB&(1<<PINB4)))


zum Abfragen eines Eingangs wird PINB benutzt und nicht PORTB.
De facto fragst du hier ab, ob der Pullup eingeschaltet ist oder nicht.

[pitschu (Gast)]

Das Hauptproblem ist wohl das return 0 anstatt break;

ps

[Karl H. (kbuchegg)]

pitschu schrieb:
> Das Hauptproblem ist wohl das return 0 anstatt break;

Ah, danke.
das hab ich  noch gar nicht gesehen.

[Dex (Gast)]

Hab das Problem selbst gefunden nach dem ich ein paar minuten Pause 
gemacht habe. zu meiner Schande muss ich gestehen, dass ich die ganze 
zeit die falsche elf Datei zum flashen benutzt habe xD.

trotzdem danke ich euch allen für die verbesserungen, damit habt ihr mir 
die spätere fehlersuche erpsart xD

Eine Frage hätte ich aber noch: In den AVr tutorials werden öfters die 
Bezeichner PCx benutzt, steht es für port oder Pin? kann ich PCx und PBx 
usw auch benutzen? AVr Studio scheint es nämlich nicht zu kennen

[Martin K. (maart)]

Dex schrieb:
> Eine Frage hätte ich aber noch: In den AVr tutorials werden öfters die
> Bezeichner PCx benutzt, steht es für port oder Pin? kann ich PCx und PBx
> usw auch benutzen? AVr Studio scheint es nämlich nicht zu kennen

#include <avr/io.h>

 Dann kennt es das auch. Es steht für die jeweilige Pinnumer.

[Karl H. (kbuchegg)]

Dex schrieb:

> Eine Frage hätte ich aber noch: In den AVr tutorials werden öfters die
> Bezeichner PCx benutzt, steht es für port oder Pin?

Für keines von beiden.

Eigentlich schreibt man ja zb

   PORTB |= ( 1 << 4 );

wenn man den Pin 4 am Port B auf 1 setzen will.
Da aber die 4 da nicht besonders heraussticht, hat man dafür per Makro 
Bezeichnungen eingeführt

#define PB4   4

so dass man schreiben kann

   PORTB |= ( 1 << PB4 );

was die Sache insofern verbessert, als man in den Zahlen sofort erkennen 
kann, was denn nun die Pinnummer ist.

PC4 steht also weder für Port oder Pin, sondern es steht für 4.
Selbiges für PINB4, PORTB4, DDRB4, etc. Ist alles dasselbe: ein Synonym 
für 4

> kann ich PCx und PBx
> usw auch benutzen? AVr Studio scheint es nämlich nicht zu kennen

logisch. Dein Tiny hat ja auch keinen C Port. Daher gibt es auch kein 
PC4.