Pin abfrage geht nicht

[John (Gast)]

Hallo

das ganze soll bei nem Mega32 eine Tasterturmatrix auslesen aber schon 
die einfache Pinabfrage über 1 Taste geht nicht!!

PINB2-4 Spalten
PINC2-5 Zeilen

Aktuell ist die Zeile 1 (PINC2) active und wenn dedrückt soll der PortB1 
da ganze auslesen (high sein) ! Stimmt was mit den Pull Up Einstellungen 
nicht.
Messe wunderbar 5V an PB1 wenn gedrückt und 0 V wenn nicht...

#define   F_CPU 1000000UL          // MP-Takt
#include  <avr/io.h>
#include  <stdio.h>
#include  <stdlib.h>
#include   <inttypes.h>
#include   <avr/interrupt.h>
#include   <avr/wdt.h>
#include    <util/delay.h>

#define BAUD     1200UL
#define UBRR_BAUD  ((F_CPU)/(16*(BAUD))-1)

int uart_putc(unsigned char c)
{
  while (!(UCSRA & (1<<UDRE)));

  UDR = c;
  return 0;
}
int uart_puts( char* str )
{
  while( *str )
    uart_putc( *str++ );
    while (!(UCSRA & (1<<UDRE)));

    UDR = 0x0A;
  return 0;
}

void long_delay(void);
    
int main(void)
{

// Eingang (Spalten)  
  DDRB  = 0x00;
  PORTB = 0x00;

// Ausgang (Zeilen)
  DDRC  = 0xFF;
  PORTC = 0xFF;

// RS232 aktivieren  
  UCSRB   |= (1 << TXEN) | ( 1 << RXEN ) | ( 1 << RXCIE ) ;          // UART TX, RX einschalten
   UCSRC   |= ( 1 << URSEL ) | ( 1<<UCSZ1 ) | ( 1<<UCSZ0);           // Asynchron 8N1,
    UBRRL  = (uint8_t) UBRR_BAUD;

  sei();   
  uart_puts("Start : ");
  while (1)
  {
    // Prüfe Zeile 1 ( 1 2 3)
    PORTC = (1<<PC2);

    if (PINB & (1<<PINB1))
    {
      _delay_ms(100);
      if (PINB & (1<<PINB1))
      {
        _delay_ms(100);
        if (PINB & (1<<PINB1))
        {
          uart_puts("  pressed");
        }
      }
    }
    else
    {
      uart_puts("not pressed");
    }
    long_delay();
    PORTC=0x00;
  }      
}

  
void long_delay(void)
{
       _delay_ms(200);
       _delay_ms(200);
       _delay_ms(200);
       _delay_ms(200);
}

[Wegstaben V. (wegstabenverbuchsler)]

> Stimmt was mit den Pull Up Einstellungen nicht.

Wenn das eine Frage sein soll, müßtest du ein Fragezeichen ans Satzende 
stellen.

[John (Gast)]

Hallo

ja das ist die Frage bzw. allg. warum es nicht funktioniert oder wo der 
Denkfehler ist.... Ich habe schon alles umgestellt aber finde nichts...

[Stefan E. (sternst)]

    PORTC = (1<<PC2);

    if (PINB & (1<<PINB1))

Zwischen diese beiden Zeilen muss ein NOP.

[Stefan E. (sternst)]

Und vom fehlenden NOP mal abgesehen, wenn du mit Pull-Ups arbeiten 
willst (die aber gar nicht aktiviert sind), ist doch deine ganze Logik 
verkehrt herum. Du musst dann 0 ausgeben und auch auf 0 testen.

[John (Gast)]

Besten Dank das wars!