SPI Komunikation

Bin zwar kein Arduino-Profi, arbeite meistens direkt auf Register level 
mit Microcontrollern aber versuche mal hier zu helfen.

Der erste Tipp den ich habe ist, dass du das nächste Datenbit beim Slave 
schon im Interrupt in das SPDR schreibts. Denn sonst kann es passieren, 
dass der Slave in der main loop den SPDR setz code nicht zwischen 2 SPI 
transfers erreicht.

Aber das kann ja nicht das eigentliche Problem sein. Schau nochmal ob du 
beim Slave MISO als output gesetzt hast. Dass der Master das gleiche 
empfängt kann nämlich auch an der verkabelung liegen. Wenn zwei 
Leitungen nebeneinander laufen (was bei dir der Fall sein wird) dann 
gibt es dazwischen Übersprechen, besonders wenn eine auf beiden Seiten 
Eingänge (hochohmig) hängen hat, und die andere halbwegs hohe Frequenzen 
(sieht ja nach 2MHz aus) hat. Sonst auch noch mal die Steckverbindung 
selbst überprüfen.

So geht das nicht.

Beim Senden muß der Slave nach jedem Byte warten, bis der Master es 
abgeholt hat (Interrupt Flag).
Und der Master muß nach jedem Byte eine Wartezeit abwarten, damit der 
Slave das nächste Byte in SPDR schreiben kann. Die AVRs haben nämlich 
keinen Sendebuffer.

Ich komme langsam voran :-)

Aber hab momentan ein Problem mit der Datenübergabe.
Ich übergebe beim Start: char Paket_zum_Master [50] = "Starten vom 
Slave.";
und ändere in der SPI interrupt routine den Inhalt in "neuer Text"
Gesendet wird aber "Starten vom Slave."
Wie muss ich das ändern.

1

// Slave

2

char Paket_zum_Master [50] = "Starten vom Slave.";

3

volatile int pos;

4

volatile bool active;

5

String Send_Paket;

6

7

8

ISR (SPI_STC_vect)    // SPI interrupt routine

9

{

10

Send_Paket = "neuer Text";

11

char Sendung = ((Send_Paket + "\n").c_str());

12

Paket_zum_Master [50] = Sendung;  

13

  byte c = SPDR;

14

  if (c == 1)  // starting new sequence?

15

    {

    active = true;
    pos = 0;
    SPDR = Paket_zum_Master [pos++];   // sendet erste byte
    return;
    }
  if (!active)
    {
    SPDR = 0;
    return;
    }
  SPDR = Paket_zum_Master [pos];
  if (Paket_zum_Master [pos] == 0 || ++pos >= sizeof (Paket_zum_Master))
    active = false;
}  // end of interrupt service routine (ISR) SPI_STC_vect

he: Auh Weia "Bring erst mal Ordnung und Übersicht in dein Leben " ist 
ja wohl eine sehr persönliche Aussage, die muss ich mir von sone Wurst 
bestimmt nicht sagen lassen.

Bin 60 Jahre alt und denke, dass ich mein Leben sehr gut im Griff habe.
Zudem hab ich vor 30 Jahren schon einiges programmiert, das war da 
leider noch nicht so objekt gebunden, wie es heute ist.
Ich bin vor 1 Monat wieder angefangen mich mit Programmieren zu 
beschäftigen und momentan erschlagen mir förmlich die Mengen an 
Funktionen.
Aber ganz abgesehen davon:
Es gibt im Netz kein vernünftiges Beispiel einer SPI Übertragung, wo der 
Master ein String überträgt und gleichzeitig ein String empfängt.
Es bietet sich hier ja an, gemeinsam was vernümftiges zusammenzustricken 
und sei es nur dafür, das ein anderer es nutzen kann.

Und Stefanus F. mein Code ist außer die Variabel Namen ein Code von Nick 
Gammon und er funktioniert auch. Ich glaube so unübersichtlich 
programmiert  Nick Gammon auch nicht.
Was ich momentan ändern will, ist das der String der oben deklariert ist 
variabel änderbar ist.

Dieter T. schrieb:
> Gesendet wird aber "Starten vom Slave."
> Wie muss ich das ändern.

 Aufhören in der ISR irgendetwas zu ändern.
 In der ISR wird nur gesendet, bzw. empfangen, die Position wird
 geprüft und entsprechende Flags gesetzt.
 Alles andere führt meistens zu Fehlern.

Also ihr beiden Profis,
anstatt sich mal mit der Sache zu beschäftigen und mal sich mit ein 
Problem überhaupt mal auseinander zu setzen, sucht ihr glaub ich nur 
nach irgendwelchen Ausreden, um euer beklopptes Verhalten zu 
rechtfertigen.
Nochmal ich suche hier eigentlich nur nach einen Tipp, aber anscheinend 
seit ihr selber zu dumm, den Code zu lese.

Ich habe diese beiden Zeilen ausgetauscht und auch wenn ihr beiden es 
nicht glauben wollt, es läuft so wie es soll.

Send_Paket = "neuer Text";
strcpy(Paket_zum_Master, Send_Paket.c_str());
Serial.print ("Text vom Slave :---->  ");

Ich werde den Code noch ein wenig ausarbeiten und hier als Beispiel 
einsetzen, für diejenigen die nach einen guten Beispiel suchen.

Auf weitere Fachmeinungen von euch beiden kann ich gut verzichten.
MfG
      Dieter

Dieter T. schrieb:
> Ich werde den Code noch ein wenig ausarbeiten und hier als Beispiel
> einsetzen, für diejenigen die nach einen guten Beispiel suchen.

 Ich bin sicher, die weltweite Gemeinde der C-Programmierer wird mit
 Begeisterung ihre nun alten und wertlosen Bibliotheken und Programme
 wegwerfen und von da an nur deinen ausgezeichneten Code benutzen...

Ich glaube auch, daß hier ein grundlegende Verständnis-Problem vorliegt.
Ganz gut wird das in dieser Grafik erklärt.
Mit jedem Zyklus "tauschen" Master und Slave den Inhalt ihrer 
Datenregister.
Entnommen aus dem Ddatenblatt des ATMega328p

Ja stimmt natürlich der Master sendet natürlich ein Byte, und empfängt 
im Gegenzug ein Byte.
Allgemein gesagt Daten ausgetauscht.

Meinst du das wirklich ernst?

Stefanus für dich nehm es zurück, aber das läuft soo, nie!! stimmt ja 
auch nicht.
Beim durchlesen hätte ich von einen Profi erwartet, das er auf die 
Fehlerstelle hinweist und nicht "das wird nie laufen."
Mittlerweile läuft es super.
"Spiele erstmal mit einem ganz "normalen" Arduino Uno Board"
Also das Netz und die Gruppen bieten sich zusätzlich zum Lernen, an, 
auch Fragen zu stellen und so seine Kenntnisse zu erweitern.
Wer sagt denn, dass ich nicht zusätzlich mich auch mit C-Programmierung: 
Zeichenkettenfunktionen uvm. beschäftige.
Aber auf eine Frage die Antwort "Lern erst mal....." passt ja auch 
nicht.
Hilfreicher wäre einfach nur ein Hinweis in der Sache ein Tipp ...

Dieter T. schrieb:
> Ich übergebe beim Start: char Paket_zum_Master [50] = "Starten vom
> Slave.";
> und ändere in der SPI interrupt routine den Inhalt in "neuer Text"

Nein, tust Du nicht.
Du initialisierst mit "Starten vom Slave.", was ein großer Unterschied 
ist.
Dann mit "Paket_zum_Master [50] = Sendung;" schreibst Du ein Zeichen 
hinter die Variable, d.h. in einen ungültigen Bereich.
Ich kenne mich mit C++ nicht aus, aber in C kopiert man Strings mit 
strcpy(). Und die Länge ermittelt man mit strlen(). Schau mal ins Manual 
zur <string.h>.

sizeof() gibt Dir nur Anzahl Bytes einer Variablen, in diesem Fall 
konstant 50.

Dieter T. schrieb:
> ISR (SPI_STC_vect)    // SPI interrupt routine
> {
> Send_Paket = "neuer Text";
> char Sendung = ((Send_Paket + "\n").c_str());

Ich hab absolut keine Ahnung, was das bezwecken soll.