Hilfe bei Fehlersuche

[Patrick P. (korlask)]

Hallo Community,
ich verwende für mein erstes Project einen ATmega32P. Die Taster(3 an 
der Zahl) sollen bei GND das 1-Signal geben(PORTB). Am Ausgang ist der 
PORTA(jeweils 8 LED´s) belegt.
Ich habe folgendes Problem mit der Switch & Case Variante im Programm. 
Wenn ich den Taster(S2) am PINB1 betätige springt er nicht im 2 Case. 
Ich weiss nicht wieso. Er bleibt immer im 1 Case. Könntet ihr mir 
vielleicht weiter helfen ?

#include <stdio.h>
#include <stdint.h>
#include <avr/io.h>
#include <util/delay.h>
#define F_CPU 16000000UL /* 16 Mhz-Takt; hier richtigen Wert eintragen */

int main(void)
{
  
    DDRA  = 0xFF; /*Ausgänge*/
    PORTA = 0x00;
    DDRB  = 0x00; /*Eingänge*/
    PORTB = 0xFF;
    
    uint8_t a;
    uint8_t x;
    uint8_t y;
    int z=0;
  
    while(1)
    {
  
  for (y=z; y < 10; y++){  
  

DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (1 << DDA3) | (1 << DDA4) | (1 << DDA5) | (1 << DDA6) | (1 << DDA7);
PORTA = DDRA;
_delay_ms(850.0);
DDRA = (0 << DDA0) | (0 << DDA1) | (0 << DDA2) | (0 << DDA3) | (0 << DDA4) | (0 << DDA5) | (0 << DDA6) | (0 << DDA7);
PORTA = DDRA;
_delay_ms(600.0);
DDRA = (1 << DDA0) | (1 << DDA1) | (1 << DDA2) | (1 << DDA3) | (1 << DDA4) | (1 << DDA5) | (1 << DDA6) | (1 << DDA7);
PORTA = DDRA;
_delay_ms(850.0);
  z = z+2;  
        }              
     
a=PORTB;
       
     if(a==0xFF)
     
      x = 1;
      
     if(a==0xFE)
     
      x = 2;

     if(a==0xFD)
     
      x = 3;

     if(a==0xFB)
     
      x = 4;    
            
    switch(x){
    
      case 1:
DDRA = (1 << DDA0) | (1 << DDA2) | (1 << DDA4) | (1 << DDA6) ;
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA1) | (1 << DDA3) | (1 << DDA5) | (1 << DDA7) ;
PORTA = DDRA;
_delay_ms(100.0);        
      break;
        
      case 2:
DDRA = (1 << DDA0);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA1);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA2);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA3);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA4);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA5);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA6);
PORTA = DDRA;
_delay_ms(100.0);
DDRA = (1 << DDA7);
PORTA = DDRA;
_delay_ms(100.0);
      break;
            
      case 3:
          .
          .
          .                    
      break;
      
      case 4:      
          .
          .  
          .     
      break;  }
      
      }     
    }

[nobi (Gast)]

Du liest das falsche Register;
mit a=PORTB die Konfiguration der Pull Ups wieder ein;
versuchs mal mit a=PINB :-)

mfg
Norbert

[Patrick P. (korlask)]

Das ist wirklich nur alles ?! :)

[nobi (Gast)]

Was verstehst Du nicht ?

einen Port liest man ein mit PINB,

wenn Du PORTB liest, dann liest Du das zurueck was Du zuvor in PORTB 
reingeschrieben hast, nähmlich die gesetzten PULL Up bits, und das 
entspricht einem Wert von 0xFF, egal, was Du aussen füe einen Pegel 
anlegst.

das heist, du gehst immer mit 0xFF in deine switch case anweisung!

[Patrick P. (korlask)]

Gerade ausprobiert , nur es geht leider immer noch nicht.
a=PORTB in a=PINB umgeschrieben.
:(

Der wechsel am Eingang kommt. 5V auf GND.Hardware mässig geht alles.

[Anfänger (Gast)]

Hallo,
wie soll denn deine Switchanweisung funktionieren?

Deine Sprünge sind abhängig von den Werten auf den anderen B-Eingängen. 
Es braucht nur ein weiterer Eingang nicht auf High liegen und dein 
Taster2 bleibt in der Abfrage unerfasst.

Du solltest die einzelnen Bits isolieren um sicherzustellen, dass ein 
bestimmtes Bit eine bestimmte Reaktion auslöst.