Problem mit Schieberegister

[ben (Gast)]

Hallo,

möchte pber ein 74xx165 Schieberegister Daten einlesen und verwende dazu 
folgenden Code, der auch funktioniert (Ich möchte dafür kein SPI 
verwenden).:

//--Schnipp

M_LATCH_L;            //Paralell in 
M_LATCH_H; 

unsigned char in=0;
      
   for(unsigned char i=0; i< 8; i++)
   {
      M_CLK_H; //Clock high
      
      //read bit
      in = (in << 1);
      in |= ((M_IN_PIN >> M_IN_PINNR) & 1);
      
      M_CLK_L;
   }
              
//--Schnapp

Nun wollte ich noch ein weiteres neuntes bit einlesen und habe gedacht 
ich löse das so:

//--Schnipp

M_LATCH_L;            //Paralell in 
M_LATCH_H; 

unsigned char in=0;
unsigned char in2=0;

      
   for(unsigned char i=0; i< 9; i++)
   {
      M_CLK_H; //Clock high
      
      if(i < 8)
      {
         //read bit
         in = (in << 1);
         in |= ((M_IN_PIN >> M_IN_PINNR) & 1);
      }
      else
      {
         in2 = ((M_IN_PIN >> M_IN_PINNR) & 1);
      }
      M_CLK_L;
   }
              
//--Schnapp

Doch leider funktioniert es nicht richtig, das Problem ist nicht, dass 
das neunte Bit nicht richtig gelesen wird, sondern, dass die ersten 8 
bit nun derart geshifetet sind, dass das Höchstwertige fehlt.
Wahrscheinlich irgendein kleiner Denkfehler aber ich komme gerade 
einfach nicht drauf...
Danke

[Assemblerfan (Gast)]

Dieses Bitgemurkse in C ist doch wirklich widerlich.
Sowas sollte man in Assembler schreiben.

[Karl H. (kbuchegg)]

ben wrote:

> Doch leider funktioniert es nicht richtig, das Problem ist nicht, dass
> das neunte Bit nicht richtig gelesen wird, sondern, dass die ersten 8
> bit nun derart geshifetet sind, dass das Höchstwertige fehlt.
> Wahrscheinlich irgendein kleiner Denkfehler aber ich komme gerade
> einfach nicht drauf...

Klingt für mich nach:
Du hast nicht berücksichtigt, dass die Bits ja im Gänsemarsch
daherkommen. Das erste Bit, welches die liest, ist eigentlich
das Bit 9, und erst dann kommen die restlichen 8 Bit.

   for(unsigned char i=0; i< 9; i++)
   {
      M_CLK_H; //Clock high
      
      if( i == 0 )
      {
         in2 = ((M_IN_PIN >> M_IN_PINNR) & 1);
      }

      else
      {
         //read bit
         in = (in << 1);
         in |= ((M_IN_PIN >> M_IN_PINNR) & 1);
      }
      M_CLK_L;
   }

[ben (Gast)]

nein, leider nicht, das verhalten ist jetzt so, dass die werte sogar 
noch eines weiter geshiftet sind, also um 2 nach links.

ich möchte auch nur ein weiteres bit einlesen, dass am seriellen eingang 
des schieberegisters anliegt. die ersten 8 bits sind ja die selben

[Karl H. (kbuchegg)]

ben wrote:
> nein, leider nicht, das verhalten ist jetzt so, dass die werte sogar
> noch eines weiter geshiftet sind, also um 2 nach links.

OK. Dann ist deine Variante die Richtige.

Allerdings sehe ich nichts in deinem Code, was das beschriebene
Verhalten erklären könnte. Könnte es sein, dass der Fehler
nicht in dieser Funktion zu finden ist?

[ben (Gast)]

hatte ich auch gedacht, aber wenn ich den zweiten code gegen den ersten 
oben austausche funktionierts mit den ersten 9 bit