Software-SPI-Slave mit Tiny13V

[sternst]

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

 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.

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

 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]

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

Die Schleife meine ich

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;
}

[sternst]

Die Schleife meine ich

Und was hat diese Schleife mit deinem Slave-Code zu tun? Wo ist da eine Schleife?

[ExKluski]

Sorry mein fehler habe mich mit der schleife falsch ausgedrückt.

Klar im slave habe ich keine schleife

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

damit meinte ich den master-code der in einer Zählschleife ist und 16 mal schiebt.

[sternst]

Sorry mein fehler habe mich mit der schleife falsch ausgedrückt.

Klar im slave habe ich keine schleife

damit meinte ich den master-code der in einer Zählschleife ist und 16 mal schiebt.

Das beantwortet aber nicht die Frage, was das jetzt mit dem Slave-Code zu tun haben soll? Dort wird nach wie vor nur ein Bit ausgegeben.

[ExKluski]

jetzt habe ich es glaube verstanden wie du das meinst. Klar im slave code ist keine schleife also wird auch nur einmal geschoben.

[ExKluski]

Nach längerem grübeln ist der Code entstanden ich würde mich freuen wenn sich jemand durchschaut und auf anhieb fehler erkennt bzw. ob es so richtig ist wie ich es mir vorgestellt habe.

bei der Warnung bin ich ratlos

../spi-tiny13.c:38: warning: suggest parentheses around assignment used as truth value

weiss aber nichts damit anzufangen.
Also das Programm soll ein SPI slave simulieren und ADC-werte schicken.
Am MISO(PB3) werden dann die Pegel Geschaltet,
Ich bin mir aber nicht Sicher ob der befehl das macht was ich will

while(GIMSK  & (1<<INT0));
//warte bis am Externen Interrupt etwas passiert

Und das ist mein ganzer Code

#include <avr/io.h>        // Namen der IO Register
#include <avr/Interrupt.h> // Interrupt register
#include "adc-init.h"
#include "SPI-init.h"

volatile char MISO_Byte;
volatile int MSB = 0x8000;

ISR(INT0_vect)
{
if(MISO_Byte == 1)SPI_MISO_PORT |= (1<<SPI_MISO_PB3); //Pin Miso high
else SPI_MISO_PORT &= ~(1<<SPI_MISO_PB3);
}

int main(void)
{ 
   		
  uint16_t adcval;
  ADC_Init();
/*Externen Interrupt-Init*/
  MCUCR = (1<<ISC01)|(1<<ISC00);
 //Interrupt auslösen wenn Steigende flanke ist
  GIMSK = (1<<INT0);	// Interrupt erlauben
  GIFR  = (1<<INTF0); 
 
while( 1 ) {
adcval = ADC_Read(0);  // Kanal 0
// mach was mit adcval
 
adcval = ADC_Read_Avg(2, 4);  // Kanal 2, Mittelwert aus 4 Messungen
// mach was mit adcval

MISO_Byte = MSB | adcval;
  
if(!(SPI_SS_PIN1 & (1<<SPI_SS_PB1)))// Wenn SS(PB1) auf low geht start der Übertragung
{
 ==>c35 for(uint16_t a =15 ; a = 0 ; a--)//Warning: suggest parentheses around assignment used as truth value
{
 if(MISO_Byte & MSB)SPI_MISO_PORT |= (1<<SPI_MISO_PB3);
 else SPI_MISO_PORT &= ~(1<<SPI_MISO_PB3);
 while(GIMSK  & (1<<INT0)); 
//warte bis am Externen Interrupt etwas passiert
 MISO_Byte<<=1;// Bit nach Links schieben
}
}
 else;
}
   return 0;
}

Ich bedanke mich schonmal für die Kritik :P

MfG Kluski