Atmega8 PS2 Protokoll

Zu allererst macht der Interrupt so eher wenig Sinn. Pro Interrupt 
solltest du genau ein Bit lesen, ohne das rumgetue mit den whiles. 
Vermutlich verpasst du mit deiner "Lösung" einige Bits.

Bei jedem low der Clock gibt es einen Bit. Wenn du dann nach dem low 
erst auf ein low, ein high und ein low wartest, ist schonmal mindestens 
ein Bit flöten, je nach Ausführungsgeschwindigkeit auch mehr.

Auch das Senden per UART sollte nicht im Interrupt geschehen. Das nimmt 
nämlich wieder Zeit in Anspruch, während derer der Interrupt nicht 
erneut feuern kann -> wieder verlorene Bits.

cygar schrieb:
> Kann mir da jemand Sagen was ich Falsch mache?
Vermutlich brauchen deine send_strings im Interrupt zu lange.
Du kannst entweder die Uart wesentlich schneller als die PS2 machen, 
oder dir überlegen, die du deine 0en und 1en am geschicktesten erst 
irgendwo ablegst und dann erst schickst, wenn die ganze Empfangsaktion 
schon gelaufen ist.

Siehe: Bitmanipulation

p.s.: KEIN array zum Speichern =)

p.p.s.: Wo hast du denn das mit dem SIG_INTERRUPT her? Das ist 
steinalter, sogar veralteter Kram. Falls das aus einem Tutorial ist, 
such dir ein anderes!
Zum Beispiel das AVR-GCC-Tutorial von hier.

Shifting per Variable sollte man vermeiden. Bei Fällen wie diesem hier 
kann man ganz leicht Zyklen sparen, indem man es so programmiert.

1

ISR(INT1_vect)

2

{

3

  static uint16_t tmp;

4

  PORTB = (1<<PB0);   //LED

5

6

  bit++;

7

  //read bit

8

  if (bit <= 11 && bit >= 1)

9

  {

10

     tmp <<= 1;

11

     if ( PIND & (1<<PIND0) )

12

       tmp |= 1;

13

  }

14

  if (bit == 11)

15

  {

16

    a = tmp;

17

    ausgabe=1;

18

    bit = 0;

19

    tmp = 0;

20

  }

21

22

}

tmp habe ich hinzugefügt, da ich annehme, dass a global ist und du 
damit die Datenübergabe ans Hauptprogramm realisierst. bit braucht 
übrigens nicht global sein, falls es sonst nicht verwendet wird. Wenn 
doch, dann aber volatile setzen, selbiges gilt für a und ausgabe.

Poste mal deinen aktuellen Code als Anhang.

Dass du a gleichzeitig als temporäre Variable in der ISR und auch als 
Übergabevariable verwendest ist schlecht. Je nachdem, wie schnell dein 
Signal ist, kanns sein, dass du mit /a = 0/ nach send_string dir einen 
Teil des ankommenden Signals löscht. Shiftest nun doch per Variable, 
nagut, von mir aus.
Hab mir PS2 noch nie angeschaut aber du empfängst 11 Bit, verundest mit 
Bits 9:1, shiftest dann um 1 nach rechts und hast dann Bits 9:1 des 
Signals in den untersten 8 Bit von a. Schaut merkwürdig aus, aber 
wenns passt, dann gut.

Was geht jetzt nicht?

cygar schrieb:
> Mein Code funktioniert jetzt soweit wie er soll. Shifte aber immer noch
> immer jedesmal wieder von vorne.

Den zweiten Satz kapier ich nicht.


PS: Übrigens wäre es besser, mit dem Input Capture eines Timers 
zusätzlich zu den Signalflanken auch noch die Zeit zwischen 2 Flanken zu 
messen. Somit kannst du schauen, ob das Signal von der Zeit her gültig 
ist.