Software-SPI-Slave mit Tiny13V

**sternst** · 18.01.2012, 13:28

Was ich so auf den ersten Blick sehe (also kein Anspruch auf Vollständigkeit):

1)

Code:

uint16_t MSB = 0b100000000000000; // Letzte bit

Für solche Konstanten die binäre Schreibweise zu verwenden, ist meist keine so gute Idee. Bei den vielen Nullen verzählt man sich schon mal, und genau das hast du auch.

2)

Code:

if(SPI_CLK_PB2 & (1<<SPI_CLK_PB2))


if(!(SPI_SS_PB1 & (1<<SPI_SS_PB1)))

Diese if-Bedingungen machen überhaupt keinen Sinn, und sind immer false.

3)

Code:

MSB <<=1;

Falsche Richtung.

4)
Wo ist der Rest von SPI_Write_Word?

**ExKluski** · 18.01.2012, 13:46

zu 1)

uint16_t MSB = 0x8000;

zu 2) Ich dachte als abfrage für die Pins.
SS Solange SS high nichts machen und sobald low ist mit dem senden anfangen
Und bei CLK fast genauso nur er soll schieben wenn CLK high ist.

zu 3) Er soll mit dem Höchsten bit anfangen?

zu 4) habe kein Rest nur die SW-SPI-Master Funktion von oben (1.post), da habe ich die sachen die ich nicht brauche (glaube ich zu mindestens) weg gelassen, Weil SS und CLK wird ja dem Tiny13 vom Tiny2313 geschickt.

MfG exkluski

**sternst** · 18.01.2012, 13:58

zu 2) Ich dachte als abfrage für die Pins.

Und wo in diesen Bedingungen wird irgendetwas abgefragt? Da steht einfach nur

Code:

if (2 & (1<<2)))

zu 3) Er soll mit dem Höchsten bit anfangen?

Und? Wie sieht MSB nach dieser Operation aus?

zu 4) habe kein Rest nur die SW-SPI-Master Funktion von oben (1.post)

Da wird also genau ein Bit ausgegeben. Was ist mit den anderen 15?

**ExKluski** · 18.01.2012, 14:45

Mit der abfrage habe ich mich verlesen sollte so heißen

Code:

 if(!(SPI_SS_PIN1 & (1<<SPI_SS_PB1)))

Und? Wie sieht MSB nach dieser Operation aus?

1 Bit weiter schieben und das dann ausgeben, mache ich das nicht mit

Code:

 MSB<<=1;

es ist doch in einer schleife die Solange durchlaufen wird bis 16 bit raus geschoben sind.

MfG exkluski

**sternst** · 18.01.2012, 14:54

Zitat von ExKluski

Mit der abfrage habe ich mich verlesen sollte so heißen

Code:

 if(!(SPI_SS_PIN1 & (1<<SPI_SS_PB1)))

Ob das jetzt korrekt ist, hängt davon ab, wie SPI_SS_PIN1 definiert ist. Die 1 darin lässt allerdings nichts Gutes vermuten.

Zitat von ExKluski

1 Bit weiter schieben und das dann ausgeben, mache ich das nicht mit

Code:

 MSB<<=1;

es ist doch in einer schleife die Solange durchlaufen wird bis 16 bit raus geschoben sind.

MfG exkluski

Nochmal: wenn MSB vorher 0x8000 ist, was ist es dann hinterher?
Und wo bitte ist diese Schleife?

**ExKluski** · 18.01.2012, 15:09

hinterher ist er 0, wäre es dann nicht besser es als Globale Variable "volatile" und die Richtung zu ändern

Die Schleife meine ich

Code:

for(i=0;i<15;i++)
{
if(mask | word) SPI_MOSI_PORT|=(1<<SPI_MOSI_PIN); 
else SPI_MOSI_PORT&=~(1<<SPI_MOSI_PIN); 
if(SPI_MISO_PIN == 1) temp |= mask;
SPI_CLK_PORT |=(1<<SPI_CLK_PIN); 		//CLK-Takt HIGH
SPI_CLK_PORT &=~(1<<SPI_CLK_PIN); 		//CLK LOW
mask >>= 1;
}