Speicherung von Pin-Zuständen funktioniert nicht

[Alexander (Gast)]

Hallo,

ich habe ein Programm geschrieben, das in durch den Timer vorgegebenen 
Zeitabständen die Pegel an den Pins PB1 und PB2 speichert. Um 
Speicherplatz zu sparen, wird in eine 8-Bit Variable vier mal der 
jeweilige Zustand gespeichert, danach kommt die nächste Variable.
Jetzt habe ich PB1 mit 5V und PB2 mit GND verbunden. Also müsste die 
VAriable doch eigentlich 0b01010101 sein? Es kommt bei mir über den UART 
aber immer 0b10101010 an. Das ganze ist also genau vertauscht.

Wenn ich PB1 mit GND und PB2 mit 5V verbinde kommt 0b10101000 raus.

Was mach ich nur falsch?


Hier der Code:

 
DDRB &= ~((1<<PB1) | (1<<PB2)); // PB1 und PB2 als Eingang 

if (SpeicherBits == 1) 
{ 
Ergebnis[AnzahlMessungen] |= (PINB & (1<<PINB1)); 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB2))<<1); 
SpeicherBits = 2; 
} 

else if (SpeicherBits == 2) 
{ 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB1))<<2); 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB2))<<3); 
SpeicherBits = 3; 
} 

else if (SpeicherBits == 3) 
{ 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB1))<<4); 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB2))<<5); 
SpeicherBits = 4; 
} 

else if (SpeicherBits == 4) 
{ 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB1))<<6); 
Ergebnis[AnzahlMessungen] |= ( (PINB & (1<<PINB2))<<7); 
AnzahlMessungen++; 
SpeicherBits = 1; 
}

Vielen Dank und Grüße, Alexander

[Gast (Gast)]

Nimm mal ein Blatt Papier und geh Schritt für Schritt nach was du da 
wohin schiebst und wo dieses Bit vor und nach dem Schieben steht.

[Oliver (Gast)]

>(PINB & (1<<PINB1)
und
>(PINB & (1<<PINB2)

tum wohl nicht das, was du erwartest, wenn der enztsprechende Pin 0 oder 
1 ist.

Also:
(PINB & (1<<PINB1)
ist ausgeschrieben
PINB & (1<<1)
also
PINB & 2

Ist der Pin PB1 auf GND, ergibt das
0 & 2 = 0
Ist der Pin auf VCC, ergibt das
2 & 2 = 2 = 0b00000010

Entsprechend für Pin PB2
0 & 4 = 0
bzw.
4 & 4 = 4 = 0b00000100

Die Einser sind also nicht da, wo du sie gerne hättest.

Oliver