Atmega 8 löst undefiniert Tastendruck aus

[Chris T. (chris0086)]

Hallo Leute,
ich hab mal wieder ien Problem und zwar hab ich einen Atmega 8 hier.
Dieser soll gemäß unten stehender Software bei einem langen Tastendruck 
die blaue LED anschalten und dann immer bei einem kurzen die rote LED 
aufleuchten lassen ( und jeweils ein Relais schalten).
Das funktioniert bei einem Druck auf die Taste prima.
Das Problem ist bloß, es funktioniert auch manchmal wenn ich den 
Prozessor berühre oder in die Nähe komme.
Aber hier erstmal der Quellcode:

/************************************************************************/
/*                                                                      */
/*                      Debouncing 8 Keys                               */
/*                      Sampling 4 Times                                */
/*                      With Repeat Function                            */
/*                                                                      */
/*              Author: Peter Dannegger                                 */
/*                      [e-mail]                                  */
/*                                                                      */
/************************************************************************/

#include <stdint.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#ifndef F_CPU
#define F_CPU           7372800                   // processor clock frequency
#warning kein F_CPU definiert
#endif

#define KEY_DDR         DDRC
#define KEY_PORT        PORTC
#define KEY_PIN         PINC
#define KEY1            5

#define ALL_KEYS        ( 1<<KEY1 )

#define REPEAT_MASK     (1<<KEY1)       // repeat: key1, key2
#define REPEAT_START    100                        // after 500ms
#define REPEAT_NEXT     10                        // every 200ms

#define LED_DDR         DDRB
#define LED_PORT        PORTB

#define LED1            0
#define LED2            1

volatile uint8_t key_state;                                // debounced and inverted key state:
                                                  // bit = 1: key pressed
volatile uint8_t key_press;                                // key press detect

volatile uint8_t key_rpt;                                  // key long press and repeat


ISR( TIMER2_OVF_vect )                            // every 10ms
{
  static uint8_t ct0, ct1, rpt;
  uint8_t i;

  TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);  // preload for 10ms

  i = key_state ^ ~KEY_PIN;                       // key changed ?
  ct0 = ~( ct0 & i );                             // reset or count ct0
  ct1 = ct0 ^ (ct1 & i);                          // reset or count ct1
  i &= ct0 & ct1;                                 // count until roll over ?
  key_state ^= i;                                 // then toggle debounced state
  key_press |= key_state & i;                     // 0->1: key press detect

  if( (key_state & REPEAT_MASK) == 0 )            // check repeat function
     rpt = REPEAT_START;                          // start delay
  if( --rpt == 0 ){
    rpt = REPEAT_NEXT;                            // repeat delay
    key_rpt |= key_state & REPEAT_MASK;
  }
}

///////////////////////////////////////////////////////////////////
//
// check if a key has been pressed. Each pressed key is reported
// only once
//
uint8_t get_key_press( uint8_t key_mask )
{
  cli();                                          // read and clear atomic !
  key_mask &= key_press;                          // read key(s)
  key_press ^= key_mask;                          // clear key(s)
  sei();
  return key_mask;
}

///////////////////////////////////////////////////////////////////
//
// check if a key has been pressed long enough such that the
// key repeat functionality kicks in. After a small setup delay
// the key is reported being pressed in subsequent calls
// to this function. This simulates the user repeatedly
// pressing and releasing the key.
//
uint8_t get_key_rpt( uint8_t key_mask )
{
  cli();                                          // read and clear atomic !
  key_mask &= key_rpt;                            // read key(s)
  key_rpt ^= key_mask;                            // clear key(s)
  sei();
  return key_mask;
}

///////////////////////////////////////////////////////////////////
//
// check if a key is pressed right now
//
uint8_t get_key_state( uint8_t key_mask )

{
  key_mask &= key_state;
  return key_mask;
}

///////////////////////////////////////////////////////////////////
//
uint8_t get_key_short( uint8_t key_mask )
{
  cli();                                          // read key state and key press atomic !
  return get_key_press( ~key_state & key_mask );
}

///////////////////////////////////////////////////////////////////
//
uint8_t get_key_long( uint8_t key_mask )
{
  return get_key_press( get_key_rpt( key_mask ));
}

int main( void )
{
  LED_PORT = 0x00;
  LED_DDR = 0xFF;

  // Configure debouncing routines
  KEY_DDR = 0x00;                // configure key port for input
  KEY_PORT = 0xFF;                // and turn on pull up resistors

  TCCR2 |= (1<<CS22)| (1<<CS21) | (1<<CS20);         // divide by 1024
  TCNT2 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);  // preload for 10ms
  TIMSK |= 1<<TOIE2;                   // enable timer interrupt
  uint8_t setting = 0;
  uint8_t p_val = 0;

  sei();
  //PIN 3 Port D auf Ausgang Relais1
  DDRD |= (1<<DDD3);

  //PIN 4 Port D auf Ausgang Relais2
  DDRD |= (1<<DDD4);
  //PORTD |= (1<<PD4); //Einschalten
  DDRD  &= ~(1<<PD2);  /* Pin PC7 als Eingang */
  PORTD |= (1<<PD2);    /* internen Pull-Up an PC7 aktivieren */
  DDRD  &= ~(1<<PD5);  /* Pin PC7 als Eingang */
  PORTD |= (1<<PD5);    /* internen Pull-Up an PC7 aktivieren */
  DDRD  &= ~(1<<PD6);  /* Pin PC7 als Eingang */
  PORTD |= (1<<PD6);    /* internen Pull-Up an PC7 aktivieren */
  DDRD  &= ~(1<<PD7);  /* Pin PC7 als Eingang */
  PORTD |= (1<<PD7);    /* internen Pull-Up an PC7 aktivieren */


  while(1){
    if( (get_key_short( 1<<KEY1 )) && (setting == 1))
    {
      LED_PORT  &= ~(1<<LED1);

      if (p_val == 0)
      {
        LED_PORT ^= 1<<LED1;
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;

      }
      if (p_val == 1)
      {
        LED_PORT ^= 1<<LED1;
        PORTD ^= (1<<PD3); //schalten
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        PORTD ^= (1<<PD3); //schalten
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;

      }
      if (p_val == 2)
      {
        LED_PORT ^= 1<<LED1;
        PORTD ^= (1<<PD4); //schalten
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;
        PORTD ^= (1<<PD4); //schalten
        _delay_ms(250);
        LED_PORT ^= 1<<LED1;


      }
      p_val++;
      if(p_val == 3)
        p_val =0;
    }


    if( get_key_long( 1<<KEY1 ))
    {
      LED_PORT ^= 1<<LED2;
      setting = !setting;
      // PORTD ^= (1<<PD3); //schalten
    }



  }
}

Also die internen Pullup Widerstände sollten aktiviert sein ebenso hab 
ich versucht den ungenutzten Pins auch auf Eingang zu setzen und die 
Pullups zu aktivieren.
Weiterhin hab ich den Eingang vom Taster einen C mit 100nF gegen VCC 
spendiert.
Der Controller stürzt nicht ab da ja dann die blaue LED ausgehen würde.
Aber ich kann am Oszi am Tastereingang einen Spannungseinbruch 
feststellen wenn er mal wieder willkürlich schaltet. Also als ob jemand 
den Taster drücken würde.
Das einizge was ich am Aufbau wohl falsch gemacht hbe ist das ich AREF 
auf VCC gelgt habe, aber kann das denn auch daran liegen?
Hier noch in Video um die Sache zu verdeutlichen (auf 1 Minute spulen):
http://www.vidup.de/v/kWpFh/

Was kann ich machen damit er richtig funktioniert? Liegts an der 
Software?
Bei dem Video wurde derController über den Programmer versorgt, also 
nicht über den Schaltregler der auf dem Board ist.

[Felix P. (fixxl)]

In der ISR sollte in deinem Fall TCNT2 und nicht TCNT0 gesetzt werden.

AREF auf VCC zu setzen sollte lt. Datenblatt in Ordnung sein, solange du 
keine Referenz für den ADC einstellst (Bit REFS1 und REFS0 im 
ADMUX-Register müssen 0 sein).

Ein 100nF-Kondensator von RESET nach GND fehlt mir noch.

[Chris T. (chris0086)]

Hab das mit TCNT2 umgestellt und ihm noch einen C am Reset verpasst 
leider immernoch keine Besserung.

[Karl H. (kbuchegg)]

Chris tian schrieb:


> Also die internen Pullup Widerstände sollten aktiviert sein ebenso hab
> ich versucht den ungenutzten Pins auch auf Eingang zu setzen und die
> Pullups zu aktivieren.
> Weiterhin hab ich den Eingang vom Taster einen C mit 100nF gegen VCC
> spendiert.

Das war keine so prickelnde Idee. Das Pollin Board ist bekannt dafür 
unzuverlässig zu sein, eben weil da Kondensatoren kurz geschlossen 
werden und das kleine Spikes auf der Versorungungsspannung verursacht.

> Der Controller stürzt nicht ab da ja dann die blaue LED ausgehen würde.
> Aber ich kann am Oszi am Tastereingang einen Spannungseinbruch
> feststellen wenn er mal wieder willkürlich schaltet.

Hast du dir schon mal die Versorgungsspannung angesehen? Gibt es dort 
auch Glitsches, wenn du auf der Platine rumfingerst?

Du könntest mal probeweise einen externen Pullup-Widerstand an den 
Eingangspin anhängen. Größenordnung so um die 10k.


> Was kann ich machen damit er richtig funktioniert? Liegts an der
> Software?

Normalerweise würde ich bei solchen Symptomen sagen, du hast dir 
irgendwo die Pullups irrtümlich weggeschaltet. Allerdings kann ich im 
Code nichts entdecken. M.M nach ist der Code ok. Ich tippe auf ein 
Hardware Problem.

(Ok, die Sache mit TCNT2 bzw TCNT0. Ist aber uninteressant. Im Grunde 
kann man das Vorladen des Timers auch weg lassen. Ob die ISR jetzt alle 
5ms kommt oder alle 7ms oder 10 oder 15, spielt keine Rolle)

[Chris T. (chris0086)]

Was mich wundert ist das der µC ungewöhnlich warm wird. Aber wüsste 
nicht warum, die Relais die geschlatet werden sind aus. und sonst hängen 
ja nur die zwei LED dran.

[Karl H. (kbuchegg)]

Chris tian schrieb:
> Was mich wundert ist das der µC ungewöhnlich warm wird. Aber wüsste
> nicht warum

Irgendwo eine kleine Lötbrücke?
Die können sich fies verstecken

[Chris T. (chris0086)]

So Leute ich bin mir zu 99,9% sicher den Fehler gefunden zu haben.
Bin natürlich selbst schuld.
Im Anhang ist der Fehler zu sehen. Den konnte ich aber nicht entdecken 
als ich die einzelnen Pins gegenseitig auf Durchgang geprüft habe.
Eine Leitung ist die Tasterleitung, die andere RX auf der anderen Seite 
des Max232 der aber aktuell nicht gebraucht wird. Ich hab mich auch 
immer schon gewundert warum der MAx auch ungewöhnlich warm wird auf dem 
IR Bild.

Naja das  kommt davon wenn man zum schluss keinen ERC macht.
ICh hoffe das war der Fehler und bedanke mich trotzdem bei allen die mir 
hier geholfen haben.

Edit: Sorry irgendwie ist was mit dem Bils schief gegangen.
Ist aber nicht schlimm da ist eigentlich nur das Via zu sehen das leider 
beide Leitungen verbindet.

[Falk B. (falk)]

@ Chris tian (chris0086)

>Im Anhang ist der Fehler zu sehen.

Mein Monokel ist kaputt.

[Peter D. (peda)]

Falk Brunner schrieb:
> Mein Monokel ist kaputt.

Ich erkenne auch rein garnichts.

Wenn hier von Bildformat beachten die Rede ist, dann ist damit nicht die 
Pixelzahl gemeint, 90 Pixel ist wirklich ne Zumutung.

Sondern es ist die Dateigröße gemeint, d.h. die Komprimierung erhöhen.
Das Format kann (sollte) ruhig Bildschirm füllend sein.