Attiny13A einfache Quizschaltung

[Marcel A. (marcel_lat)]

Hallo Liebe µC Gemeinde,

ich habe mir einen Atmel AVRISP MKII zugelegt, um einen Attiny13A zu 
programmieren. Rauskommen soll eine Quizschaltung, also 2 Spieler und 2 
LEDs, wer als erster drückt bei dem leuchtet es beim anderen nicht. Für 
die Spielerknöpfe verwende ich ganz normale Kurzhubtaster mit 22pF 
Kodensator parallel geschalten. Taster liegt auf GND und dem jeweilen 
PIN des µC.
Als Test habe ich einfach mal zwei LED abwechselnd Blinken lassen, das 
hat funktioniert. Aber die Quizschaltung möchte einfach nicht und ich 
weiß nicht mehr weiter. Vielleicht findet jemand das Problem.

Gruß Marcel

/*
 * Quiz.c
 *
 * Created: 16.03.2014 17:21:56
 *  Author: Marcel
 */ 

#define F_CPU 1000000
#include <avr/io.h>

//PB 2 LED A
//PB 1  LED B
//PB 5  SW A
//PB 3  SW B
//PB 4  SW RESET



int main(void)
{  
  DDRB = 0b000110;   //B2 und B1 auf Ausgang
  PORTB= 0b111000;  //B 0-2 auf low, B 3-6 auf High (SW Input)
  

    while(1)
    {  
      if ((PINB & (1<<5)) == 0){
        if((PINB & (1<<1)) == 0){ //Abfrage SW A, wenn dann LEDA schalten
          PORTB |= (1<<2);
        }
      }
    
      if ((PINB & (1<<3)) == 0){
        if((PINB & (1<<2)) == 0){ //Abfrage SW B, wenn dann LEDB schalten
          PORTB |= (1<<1);
        }
      }
      if ((PINB & (1<<4)) == 0){  //reset Button
        PORTB &=~(1<<2);
        PORTB &=~(1<<1);
      }
    }
}

[Karl H. (kbuchegg)]

Marcel Amthor schrieb:

> die Spielerknöpfe verwende ich ganz normale Kurzhubtaster mit 22pF
> Kodensator parallel geschalten.

Schlecht.
Solche Konstruktionen sind notorisch dafür bekannt, durch kurze 
Spannungseinbrüche auf der Versorgungsspannung, dieselbe zum Einsturz zu 
bringen, wodurch der µC abstürzen kann.

Löte die Kondensatoren aus. In deinem Beispiel brauchst du die nicht. 
Wer als erster drückt gewinnt, egal ob die Taste danach noch ein paar 
mal prellt oder nicht.

[gatsby (Gast)]

Hallo,

mit dieser Zeile

        if((PINB & (1<<1)) == 0){ //Abfrage SW A, wenn dann LEDA 
schalten

willst du prüfen ob die LED des Gegners schon eingeschaltet ist.
Da dies aber ein Ausgang ist, muss das heißen

        if((PORTB & (1<<1)) == 0){ //Abfrage SW A, wenn dann LEDA 
schalten

Außerdem verwirrt der Kommentar.


Gruß
gatsby

[Karl H. (kbuchegg)]

gatsby schrieb:
> Hallo,
>
> mit dieser Zeile
>
>         if((PINB & (1<<1)) == 0){ //Abfrage SW A, wenn dann LEDA
> schalten
>
> willst du prüfen ob die LED des Gegners schon eingeschaltet ist.
> Da dies aber ein Ausgang ist, muss das heißen
>
>         if((PORTB & (1<<1)) == 0){ //Abfrage SW A, wenn dann LEDA
> schalten

Es geht beides.
Mit PORT kriegt man den Pegel, den das Programm eingestellt hat., Mit 
PIN kriegt man den Pegel, den der Pin tatsächlich hat (nach einer kurzen 
Verzögerungszeit nach dem Setzen von PORT). Solange man keinen 
Kurzschluss hat und der Pin den am PORT eingestellten Pegel tatsächlich 
auch annehmen kann, stimmen die beiden Werte (nach 1 CPU Takt) überein.

Ich hätte es auch in der PORT Variante geschrieben, weil es meiner 
Meinung nach klarer ist, was hier die Absicht ist. Aber direkt falsch 
ist es nicht.

[Peter D. (peda)]

Marcel Amthor schrieb:
> Aber die Quizschaltung möchte einfach nicht

Das ist ja mal ne besonders ausführliche Fehlerbeschreibung, Hut ab.

Der Code sieht o.k. aus.
Ich würde noch ne Flankenerkennung einbauen gegen Schummeln (zu früh 
drücken).

[Marcel A. (marcel_lat)]

So vielen Dank für die Antworten!

Ich habs dann doch endlich hinbekommen. Der erste Fehler war PINB5 
überhaupt als IO port zu nutzen. Wenn nämlich Reset Fuse nicht gesetzt 
ist, wird das nichts. Also nutze ich nun die Pins 0-4 und verwende 
wieder die "bit_is_clear" Funktion zum abfragen der Pins. Zweite Sache 
die mir beim rumtesten auffiel, war dann die Tasterpins nach dem 
Abfragen wieder auf 1 zusetzen. Damits hats schlussendlich funktioniert. 
Anbei noch mal der neue Code und was fürs Auge :)

Vielen Danke nochmal an alle die mitgrübelten.

Grüße Marcel

/*
 * GccApplication1.c
 *
 * Created: 25.03.2014 12:26:07
 *  Author: Marcel
 */ 
#define F_CPU 1000000
#include <util/delay.h>
#include <avr/io.h>
int press=0;

int main(void)
{  DDRB=0b00000110;
  PORTB=0xFF;
    while(1)
    {   
    
    if (bit_is_clear(PINB, PINB0)  && press==0){
      PORTB=0b00011101;
      press=1;
    } 
    if (bit_is_clear(PINB, PINB3) && press==0){
      PORTB=0b00011011;
      press=1;
    }
    if (bit_is_clear(PINB, PINB4)){
      PORTB=0b00011001;
      press=0;
    }
  
    
     }  
    
}

[Falk B. (falk)]

@ Marcel Amthor (marcel_lat)

>wieder die "bit_is_clear" Funktion zum abfragen der Pins. Zweite Sache

Die ist eher veraltet, die gab es früher mal, als der avr gcc noch nicht 
so ganz fit war.

>die mir beim rumtesten auffiel, war dann die Tasterpins nach dem
>Abfragen wieder auf 1 zusetzen. Damits hats schlussendlich funktioniert.

Nicht gut. Einfach den Port als Ganzes setzen geht bestenfalls bei 
diesem sehr einfachen Programm. Sobald es ETWAS größer wird, versinkst 
du im Chaos. Also lerne etwas über Bitmanipulation und wie man 
einzelne Bits gezielt abfragen oder schreiben kann.

>Anbei noch mal der neue Code und was fürs Auge :)

Naja . . .
Beides eher hyperfluide . . .

[Karl H. (kbuchegg)]

> Naja . . .
> Beides eher hyperfluide . . .

Vor allen Dingen.
Die Bitzugriffe im Originalprogramm waren alle in Ordnung. Es gibt 
keinen Grund, warum ein bit_is_clear bzw. das Zuweisen des ganzen Ports 
irgendetwas mit dem Fehlersymptom zu tun haben sollte.

OK. Das du irrtümlich den Reset-Pin 'mitbenutzt' hast, das ist natürlich 
ein grober Schnitzer. Davon bin ich eigentlich nicht ausgegangen und das 
hab ich nicht mit dem Datenblatt kontrolliert.