Software SPI - Register lesen

[Jan H. (janiiix3)]

Moin Moin,

irgendwie stehe Ich gerade auf dem Schlauch.
Möchte anstatt einer "if" Abfrage die Bits so einsammeln..

uint8_t spiSoftRead( uint8_t byte )
{
  uint8_t n = 0;
  uint8_t ret = 0;

  /*
  *  Adresse zum lesen senden
  */
  spiSoftWrite( byte );

  /*
  *  Bits einsammeln..
  */
  for( n = 0 ; n < 8 ; n++ )
  {  
    ret |= ( ( SPI_MISO_PIN & SPI_MISO_BP ) >> SPI_MISO_BP );
    ret<<=1;          
  }
  
  return ret;
}

Aber das möchte nicht klappen. Was mache Ich falsch?

[Och nöö (Gast)]

Jan H. schrieb:
> Was mache Ich falsch?

Du zeigst nicht deinen kompletten Code.

Wie ist denn SPI_MISO_BP definiert?

[Jan H. (janiiix3)]

Folgend.

Der Port ist PINB.
Und die Bitposition 1<<4

[Och nöö (Gast)]

Jan H. schrieb:
> Und die Bitposition 1<<4

Du schaffst es ja nicht mal meine Frage zu beantworten.
Was eigentlich gaaaaaaanz einfach sein sollte.

Die lautete:

Och nöö schrieb:
> Wie ist denn SPI_MISO_BP definiert?

[holger (Gast)]

Wo ist dein SPI Clocksignal?

[Jan H. (janiiix3)]

/*
*uC = MEGA32
*/
SPI_MISO_PIN = PINA;
SPI_MISO_BP = (1<<4);

Es geht erstmal nur um die Abfrage was Ich gerade nicht so ganz blicke. 
Um das Clocksignal kümmere Ich mich später.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Jan H. schrieb:
> Um das Clocksignal kümmere Ich mich später.

Du kannst SPI keine Daten entlocken, wenn du es nicht taktest. Mit Glück 
liest du mit deinem Code immer das erste Bit, das aus der Peripherie 
tropft, aber kein bisschen mehr.

[Och nöö (Gast)]

Jan H. schrieb:
> SPI_MISO_BP = (1<<4);
>
> Es geht erstmal nur um die Abfrage was Ich gerade nicht so ganz blicke.

Wie matzetronics schon sagte musst du mit jedem Bit das du
liest auch einmal den Clock toggeln.

Um die If-Abfrage wirst du kaum herumkommen (was auch nicht
sinnvoll ist) da dein Datenbit im Port nicht auf Position
Null steht.

Also dann lieber so:

  
for( n = 0 ; n < 8 ; n++ )
{ 
  if  ( ( SPI_MISO_PIN & SPI_MISO_BP ) != 0) ret |= 1;
  ret<<=1;
  // hier noch den Clock betätigen
  // oder vorher, je nach Datenlage ....
}

[holger (Gast)]

SPI_MISO_BP = (1<<4);  // Das ist 16

 >> SPI_MISO_BP  // Du willst hier aber nur um 4 schieben

[Och nöö (Gast)]

Natürlich muss es heissen:

#define  SPI_MISO_BP (1<<4)

Aber das ist auch sonst noch einiges im Argen ...

[Jan H. (janiiix3)]

Jan H. schrieb:
> ( ( SPI_MISO_PIN & SPI_MISO_BP ) >> SPI_MISO_BP );
Damit wollte Ich bezwecken, dass das Bit (SPI_MISO_BP) mit dem Port 
(SPI_MISO_PIN) verUNDET wird. Wenn das Bit auf "high" liegt dann soll es 
ganz nach "rechts" geschoben werden, so dass das erste Bit gesetzt ist.
Dann wird weiter geschoben (das ganze 8x) sollte doch so klappen oder?

[Och nöö (Gast)]

Jan H. schrieb:
> sollte doch so klappen oder?

Nein.

[Jim M. (turboj)]

Jan H. schrieb:
> Dann wird weiter geschoben (das ganze 8x) sollte doch so klappen oder?

Nö. Denn da wird um 1<<4 geschoben und nicht nur um 4.

Lass Dir das Präprozessor Ergebnis (gcc -E) mal anzeigen, dann sieht man 
was passiert ist.