Arduino: Button drücken wird nicht erkennt. Nur der Interrupt wird ausgelöst

[Stefan S. (kami)]

Hi,

ich habe hier einen ATtiny84 mit einem simplen Taster dran:

http://www.mikrocontroller.net/wikifiles/8/8a/Taster.gif

Ich habe folgenden simplen Code zum Testen geschrieben und lasse mir das 
alles per Serial ausgeben:

#include <JeeLib.h>
#include <avr/sleep.h>
#include <avr/interrupt.h>  

#define DIO1 10  // d10 

static void setPrescaler (uint8_t mode) {
    cli();
    CLKPR = bit(CLKPCE);
    CLKPR = mode;
    sei();
}
int buttonState = 0;     
ISR(WDT_vect) { Sleepy::watchdogEvent(); }

ISR (PCINT0_vect) { 

Serial.println("Ubt"); 
     
}


void setup()   {
setPrescaler(0); // div 1, i.e. speed up to 8 MHz
         Serial.begin(38400);
      Serial.print("ack13 \n");
    
      pinMode(DIO1, INPUT);       //set the pin to input
PCMSK0 |= (1<<PCINT2); //  tell pin change mask to listen to (DIO1)
digitalWrite(DIO1, LOW); 
GIMSK  |= (1<<PCIE0); // enable PCINT interrupt in the general interrupt mask
 
      //sei(); 
  
}
 
void loop() {
  buttonState = digitalRead(DIO1);

  // check if the pushbutton is pressed.
  // if it is, the buttonState is HIGH:
  if (buttonState == HIGH) {     
    // turn LED on:    
     Serial.print("1 \n");  
  } 
  else {
    // turn LED off:
     Serial.print("2 \n");
  }
  //Serial.println(inttime);
  delay (500);
  
  

 
}

Mein Output in hterm sieht nun so aus:

ack13 
2 
2 
2 
2 
2 
2 
UUbt
bt
2 
2 
2 
2 
UUUUUbtUUbUUUUUUaUUbUUUbt
bt
bt
t
bt
2 
2

Was mich nun stört ist das egal ob der Schalter betätigt ist oder nicht 
er immer nur LOW ist und nie HIGH ist. Aber ich jeden Interrupt auf dem 
Pin angezeigt bekomme. Was mache ich falsch?

Gruß kami

[Thomas E. (thomase)]

Stefan S. schrieb:
> Was mache ich falsch?
So ziemlich alles.

Taster fragt man nicht per Interrupt ab.
Taster müssen entprellt werden.
In einer ISR schreibt man nichts auf den UART.
Delays sind nicht empfehlenswert.

mfg.

[Stefan S. (kami)]

Hi,

ja darum geht es nicht. Das ist auch nur Code zum testen. Ich schreibe 
das nachher anders. Ich will nur relativ simpel sehen, was geht und was 
nicht.
Der Pin 10  ist : PCINT2  AIN1  ADC2

könnte das ein Problem sein? Ich habe am Tiny keinen anderen mehr frei

Gruß kami

[Eumel (Gast)]

Stefan S. schrieb:
> ja darum geht es nicht. Das ist auch nur Code zum testen. Ich schreibe
> das nachher anders. Ich will nur relativ simpel sehen, was geht und was
> nicht.

Mach es gleich richtig.

[herb (Gast)]

ich habe auch mal sowas probiert und ohne entprellung kam da wirklich 
auch nur müll raus. und delay ist auch ganz fail, mach das mit millis

[Thomas E. (thomase)]

Stefan S. schrieb:
> ja darum geht es nicht. Das ist auch nur Code zum testen. Ich schreibe
> das nachher anders. Ich will nur relativ simpel sehen, was geht und was
> nicht.
Und jetzt bist du zu der Erkenntnis gekommen, daß das so nicht geht. Das 
ist ja auch schon was. Das musst du schon vernünftig machen. Sonst wird 
das nichts.

Stefan S. schrieb:
> könnte das ein Problem sein? Ich habe am Tiny keinen anderen mehr frei
Nein. Wenn der Pin I/O-Port ist, weiss der gar nicht, daß er auch noch 
ADC könnte.

mfg.

[Stefan S. (kami)]

Hi,

so Problem komplett gelöst. Es lag nur an der Zuordnung der Pins.

11 musste 8 sein und der DIO2 ist 0.

Damit hat alles hingehauen.

Vielen Dank nochmal.

Gruß kami