Druckbare Version
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.Zitat:
es ist doch in einer schleife die Solange durchlaufen wird bis 16 bit raus geschoben sind.
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.Zitat:
Zitat von ExKluski
jetzt habe ich es glaube verstanden wie du das meinst. Klar im slave code ist keine schleife also wird auch nur einmal geschoben.
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
Und das ist mein ganzer CodeCode:while(GIMSK & (1<<INT0));
//warte bis am Externen Interrupt etwas passiert
Ich bedanke mich schonmal für die Kritik :PCode:#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;
}
MfG Kluski
for(uint16_t a =15 ; a < 16 ; a--)
1)Da würde ich gerne mal die Definition von SPI_SS_PIN1 sehen.Code:if(!(SPI_SS_PIN1 & (1<<SPI_SS_PB1)))
2)"a = 0" ist eine Zuweisung, kein Vergleich. Aber auch wenn es ein Vergleich wäre, wäre es falsch, denn die Bedingung in einer for-Schleife ist eine "solange"-Bedingung, keine "bis"-Bedingung.Code:for(uint16_t a =15 ; a = 0 ; a--)
3)Da muss ich dich jetzt aber mal ernsthaft fragen: wie kommst du nur darauf? Was im Datenblatt lässt dich denken, dass dieses while das machen könnte, was im Kommentar steht?Code:while(GIMSK & (1<<INT0));
//warte bis am Externen Interrupt etwas passiert
4) Und last but not least, warum wird sowohl in einer Schleife in main, als auch in einer ISR etwas auf dem MISO-Pin ausgegeben?
5) Du willst 16 Bits aus einem Byte ausgeben?
Ich sehe gerade, dass du auch schon einen längeren Thread auf mikrocontroller.net hast. Damit bin ich raus. Ich habe schlicht keine Lust, zwei Threads zu checken, um zu sehen, ob ich meine Zeit nicht damit verschwende, Dinge zu schreiben, die vielleicht längst gesagt wurden.
Folgendes zum "Abschied":
Erst musst du dir mal darüber klar werden, ob das eigentliche Raustakten der Daten nun per Interrupt, oder per Polling erfolgen soll, und ob es 8 oder 16 Bit sind. Für 16 Bit Polling sieht der Pseudo-Code dann z.B so aus:
Code:mache 16 mal {
warte bis Takteingang high
gebe Bit aus
schiebe zum nächsten Bit
warte bis Takteingang low
}
zu 1)
zu 2) Mein fehler geändert inCode://DDRB
#define SPI_MOSI_DDR DDRB
#define SPI_CLK_DDR DDRB
#define SPI_MISO_DDR DDRB
#define SPI_SS_DDR DDRB
//PORTB
#define SPI_MOSI_PORT PORTB
#define SPI_CLK_PORT PORTB
#define SPI_MISO_PORT PORTB
#define SPI_SS_PORT PORTB
// PIN
#define SPI_MOSI_PIN0 PINB0
#define SPI_SS_PIN1 PINB1
#define SPI_CLK_PIN2 PINB2
#define SPI_MISO_PIN3 PINB3
//PB
#define SPI_MOSI_PB0 PB0
#define SPI_SS_PB1 PB1
#define SPI_CLK_PB2 PB2
#define SPI_MISO_PB3 PB3
//Slave-deklaration
void SPI_INIT(void)
{
// MOSI, CLK und SS auf Eingang setzen
SPI_MOSI_DDR = (1<<SPI_MOSI_PB0);
SPI_CLK_DDR = (1<<SPI_CLK_PB2);
SPI_SS_DDR = (1<<SPI_SS_PB1);
// MOSI, CLK und SS Internen Pull-Up aktivieren
SPI_MOSI_PORT |=(1<<SPI_MOSI_PB0);
SPI_CLK_PORT |=(1<<SPI_CLK_PB2);
SPI_SS_PORT |=(1<<SPI_SS_PB1);
// MISO auf Ausgang setzen
SPI_MISO_DDR |= (1<<SPI_MISO_PB3);
//MISO auf High setzten
SPI_MISO_PORT |= (1<<SPI_MISO_PB3);
}
zu3) Kennst du das nicht im Kopf Klang es logisch, eigentlich wollte ich damit erreichen das er an der stelle warten soll bist der externe Int0 ausgelöst wird.Code:for(uint16_t a =15 ; a > 0; a--)
zu4) Ne es soll nur etwas am miso-pin passieren wenn der der interrupt ausgelöst wird.
zu5) es werden 16-bit ausgegeben habe es nur byte genannt.
MfG Kluski
zu1 ) Bitte
zu 2) mein fehlerCode://DDRB
#define SPI_MOSI_DDR DDRB
#define SPI_CLK_DDR DDRB
#define SPI_MISO_DDR DDRB
#define SPI_SS_DDR DDRB
//PORTB
#define SPI_MOSI_PORT PORTB
#define SPI_CLK_PORT PORTB
#define SPI_MISO_PORT PORTB
#define SPI_SS_PORT PORTB
// PIN
#define SPI_MOSI_PIN0 PINB0
#define SPI_SS_PIN1 PINB1
#define SPI_CLK_PIN2 PINB2
#define SPI_MISO_PIN3 PINB3
//PB
#define SPI_MOSI_PB0 PB0
#define SPI_SS_PB1 PB1
#define SPI_CLK_PB2 PB2
#define SPI_MISO_PB3 PB3
//Slave-deklaration
void SPI_INIT(void)
{
// MOSI, CLK und SS auf Eingang setzen
SPI_MOSI_DDR = (1<<SPI_MOSI_PB0);
SPI_CLK_DDR = (1<<SPI_CLK_PB2);
SPI_SS_DDR = (1<<SPI_SS_PB1);
// MOSI, CLK und SS Internen Pull-Up aktivieren
SPI_MOSI_PORT |=(1<<SPI_MOSI_PB0);
SPI_CLK_PORT |=(1<<SPI_CLK_PB2);
SPI_SS_PORT |=(1<<SPI_SS_PB1);
// MISO auf Ausgang setzen
SPI_MISO_DDR |= (1<<SPI_MISO_PB3);
//MISO auf High setzten
SPI_MISO_PORT |= (1<<SPI_MISO_PB3);
}
zu3) Kennst du das nicht auch im Kopf Klang es Logisch :D damit meine ich er soll warten bis der Interrupt ausgelöst.Code:for(uint16_t a =15 ; a > 0; a--)
zu4) da wusste ich nicht wie ich das schreiben sollte. Ich will das er ein bit am MISO aus gibt wenn der Interrupt ausgelöst wird und ein bit weiter schiebt
zu5) er heisst nur Byte damit ich weiss es sind mehre bits
MfG Exkluski