Probleme mit Tastenabfrage ATMEGA8 Pollin Board

[Daniel S. (cyber)]

Hallo und guten Morgen,

Ich versuche gerade an PortB 6 Taster anzuschliessen doch leider wollen 
nur 4 Taster laufen und sie haben eine Verschiebung drin.

Nutze einen Mega8 auf dem Pollin Board inkl. Erweiterungsboard.

Angeschlossen sind nur die 6 Taster und gebe die Infos an der UART raus.
Der MEGA8 läuft mit 8Mhz.

Zur ansteuerung der UART nutze ich die Lib von Peter Fleury
Die Debounce kommt von Peter D.

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
//#include <lcd-routines.h>
#include "uart.h"


#define UART_BAUD_RATE      19200

// Defines der einzelnen Ports
// Buttons defines
#define BTN_PORT    PORTB
#define BTN_DDR      DDRB
#define BTN_SECOND_UP  PB5
#define BTN_SECOND_DOWN  PB4
#define BTN_MINUTE_UP  PB3
#define BTN_MINUTE_DOWN PB2
#define BTN_START    PB1
#define BTN_STOP    PB0
// Summer und Relais
#define OUT_POPRT    PORTD
#define OUT_DDR      DDRD
#define OUT_SUMMER1    PD5
#define OUT_SUMMER2    PD6
#define OUT_RELAIS    PD7
#define TEST_BLINC    PD6
#define TEST_POWER    PD5
#define TEST_SWITCH_LED  PD4
#define TEST_BTN    PD3
#define TEST_BTN2    PD2
#define TEST_LED2    PD1

#define debounce( port, pin )                          \
({                                        \
  static uint8_t flag = 0;      /* new variable on every macro usage */  \
  uint8_t i = 0;                                \
                                        \
  if( flag ){              /* check for key release: */      \
    for(;;){            /* loop ... */              \
      if( !(port & 1<<pin) ){    /* ... until key pressed or ... */    \
        i = 0;          /* 0 = bounce */            \
        break;                              \
      }                                  \
      _delay_us( 98 );      /* * 256 = 25ms */            \
      if( --i == 0 ){        /* ... until key >25ms released */    \
        flag = 0;        /* clear press flag */          \
        i = 0;          /* 0 = key release debounced */      \
        break;                              \
      }                                  \
    }                                    \
    }else{              /* else check for key press: */      \
    for(;;){            /* loop ... */              \
      if( (port & 1<<pin) ){    /* ... until key released or ... */    \
        i = 0;          /* 0 = bounce */            \
        break;                              \
      }                                  \
      _delay_us( 98 );      /* * 256 = 25ms */            \
      if( --i == 0 ){        /* ... until key >25ms pressed */    \
        flag = 1;        /* set press flag */          \
        i = 1;          /* 1 = key press debounced */      \
        break;                              \
      }                                  \
    }                                    \
  }                                      \
  i;        /* return value of Macro */                  \
})

ISR(TIMER0_OVF_vect)
{
  TCNT0 = 00;
  OUT_POPRT ^= (1 << TEST_BLINC);       
}

ISR(TIMER1_COMPA_vect)
{
  //Auflösung 1 Sekunde
  TCNT1 = 0;
  OUT_POPRT ^= (1 << OUT_RELAIS);  
}

void init(void)
{
    TIMSK |= (1<<TOIE0);
    TCCR0 |= (1<<CS02) |  (1<<CS00);   
    TCNT0 = 00;  
  
  //TCCR1B = (1 << WGM12) | (1 << CS12);
  //OCR1A = 62500;
  
  //TIMSK = (1 << OCIE1A) | (1 << TOIE0);
  TIMSK = (1 << TOIE0);
  
  uart_init( UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
  uart_puts("Starting\r\n");  
}

int main(void)
{
  init();
  sei();
  
  OUT_DDR |= ((1 << OUT_RELAIS) | (1 << TEST_BLINC) | (1 << TEST_POWER)  | (1 << TEST_SWITCH_LED)  | (1 << TEST_LED2));
  OUT_DDR |= ((1 << OUT_RELAIS) | (1 << TEST_BLINC) | (1 << TEST_POWER)  | (1 << TEST_SWITCH_LED));
  //OUT_DDR &= ~((1 << TEST_BTN) | (1 << TEST_BTN2));
  //PORTD |= ((1 << TEST_BTN) | (1 << TEST_BTN2));
  
  BTN_DDR &= ~((1 << BTN_START) | (1 << BTN_STOP) | (1 << BTN_SECOND_UP) | (1 << BTN_SECOND_DOWN) | (1 << BTN_MINUTE_UP) | (1 << BTN_MINUTE_DOWN));
  BTN_PORT |= ((1 << BTN_START) | (1 << BTN_STOP) | (1 << BTN_SECOND_UP) | (1 << BTN_SECOND_DOWN) | (1 << BTN_MINUTE_UP) | (1 << BTN_MINUTE_DOWN));
  
  PORTD |= ((1 << TEST_POWER));
  
    while(1)
    {
    if (debounce(PINB, BTN_MINUTE_DOWN))
    {
      uart_puts("MIN_DOWN\r\n");
    }
    
    if (debounce(PINB, BTN_MINUTE_UP))
    {
      uart_puts("MIN_UP\r\n");
    }
    
    if (debounce(PINB, BTN_SECOND_DOWN))
    {
      uart_puts("SECOND_DOWN\r\n");
    }
    
    if (debounce(PINB, BTN_SECOND_UP))
    {
      uart_puts("SECOND_UP\r\n");
    }    
    
    if (debounce(PINB, BTN_START))
    {
      uart_puts("START\r\n");
    }
    
    if (debounce(PINB, BTN_STOP))
    {
      uart_puts("STOP\r\n");
    }
        
    }
}

Tastendruck enspricht aber links gedrückter Port, rechts die Ausgabe:
PB5 = MIN_UP(PB3)
PB4 = MIN_DOWN(PB2)
PB3 = START(PB1)
PB2 = STOP(PB0)
PB1 und PB0 sind ohne Funktion

Benutze externe Taster an dem Pollin Board gegen Masse geschaltet mit 
internem Pullup.
Der RS232 ISP ist nicht bestückt. Gejupmert ist nur die AURT sonst sind 
alle Jumper entfernt worden.

Könnte mir jemand helfen den Fehler zu lokalisieren? Habe ich in dem 
Source einen Fehler drin oder liegt es ggf. am Board?

[Hubert G. (hubertg)]

Wenn du das Schaltbild des Pollin-Boards betrachtest, dann wirst du 
feststellen das PB6 und PB7 des Mega8-Sockel nur zum Quarz gehen.
Am Erweiterungsboard liegt dann PB0 auf PB3(da dort mit PB1 begonnen 
wird) und so weiter aufsteigend.

Die Tasten hast du umgebaut? Standardmässig schalten sie nach +5V.

[Daniel S. (cyber)]

Guten Morgen,

bin gerade mal die Schaltplände durchgegangen und da hab ich es dann 
auch gesehen.
Das ist ja der letzte ... Wer tut so etwas...

Vielen Dank. Ich hätte sonst noch wer weiß wo den Fehler vermutet.

Ich bin hier schon einen ganzen Tag am suchen gewesen. Die Taster 
geprüft den Fehler an anderen Bauteilen gesucht und den Source x mal 
geprüft aber auf die Idee das die Pins nicht 1 zu 1 übernommen sind wäre 
ich nicht gekommen.

Umgebaut habe ich die Taste nicht, das war mir zu "dreckig". Ich habe 
sie ausgelötet und auf einer kleinen Lochraster Platine 8 Taster 
aufgelötet gegen Masse.

[Hubert G. (hubertg)]

Ich verwende das Board schnell mal zum testen recht gerne. Die kleinen 
Macken muss man halt akzeptieren.
Der Tastenumbau ist gar nicht so aufwändig, siehe hier:
http://www.schorsch.at/de/technik/mikrokontroller/35-atmel-evaluations-board.html

[Daniel S. (cyber)]

Naja das war mal schnell zusammen gelötet. Find ich jetzt etwas weniger 
Aufwand als am Board selber rum zu fuchteln.

[Daniel S. (cyber)]

Kleine Frage nebenbei...

Könnte mir jemand sagen wie ich den Doppelpunkt über die UART ausgebe?
Mein Controller stellt wohl seine Funktion ein wenn ich

  uart_puts("Starting\r\n");  
  uart_puts("Current Time Delay ");
  itoa(Minuten, buf, 3);
  uart_puts(buf);
  uart_puts(" min");
  //itoa(Sekunden, buf, 3);
  //uart_puts(buf);
  //uart_puts(" sec\r\n");

 versuche.
Das " min" wird nicht mehr geschrieben.

[Hubert G. (hubertg)]

Daniel S. schrieb:
> itoa(Minuten, buf, 3);

Wieso hier die 3? Sollte hier nicht das Zahlensystem sein?

[Bastler (Gast)]

Die Dezimalzahl 59 hat zur Basis 3 (0..2)  sicher mehr als 3Stellen.
Und buf ist nochmal wie groß?
Statt 3 eine 10 als Basis wählen, dann passen Minuten auch in 2 Zenichen 
(+\0).

[Daniel S. (cyber)]

Die 2 wird doch angezeigt es geht um die Zeile da da drunter.

Habe aber den Fehler nun gefunden... Alles in einer Zeile ist scheinbar 
zu lang. Current Time Delay + Minute + Min ist für eine Zeile zu lange. 
Wenn ich das Current weg lasse erscheint auch die min.

[Daniel S. (cyber)]

OK der Puffer der UART war zu klein.

#define UART_RX_BUFFER_SIZE 128

 Nun klappt es. :)