ATmega128 und SPI Problem

[CG (Gast)]

Hallo

Vielleicht kann mir ja jemand helfen bei meinem Problem.
Ich habe einen ATmega128 und würde gerne die SPI-Schnittstelle testen.
Leider bleibt mein Programm an folgender Stelle immer hängen (kommt 
nicht mehr aus der while-Schleife raus)

while(!(SPSR & (1<<SPIF)));     //Wait until transmission complete

Habe schon einiges versucht doch leider alles ohne Erfolg.

Was mache ich denn falsch?
Habe keine gegenstelle angeschlossen, aber ich denke mal nicht das das 
was ausmacht oder? ich müsste ja zumindest einen Takt auf der SCK-Pin 
sehen.

Ich bedanke mich im Vorfeld für alle hilfreichen Antworten :-)

Zu den Eckdaten:
M103C Fuses ist nicht gesetzt
BOOTSZ ist auf 512 gesetzt
SUT_CKSEL ist auf Ext. Cystal/Resonator High Freq.: Start-up time: 16K 
CK +64ms
ein 16MHz Quarz ist angeschlossen

Ich habe einen Tastkopf an SCK von der SPI-Schnittstelle angeschlossen 
doch leider sehe ich dort auch gar keinen Tackt.



Zum TestCode:

main

#include "global.h"          // Globale Lib 

int main(void)
{
  char data;         //Received data stored here
    
  cli();            // Disable all interrupts
  spi_init_master();      // SPI Inizalisierung
//  spi_init_slave ();  
  initPort();          // Inizalisierung der Ports         
  sei();            // enable all interrupts

while(1)
    {    
  SWITCH_LED1;  
  _delay_ms(500);
  data = 0x00;                    //Reset ACK in "data"
    data = spi_tranceiver('Hallo', PIN_SPI_CS1);     
  SET_LED3;
  }
}

in der global.h

// SPI
#define PORT_SPI    PORTB
#define DDR_SPI      DDRB
#define PIN_SPI_MISO  3    // PB3
#define PIN_SPI_MOSI  2    // PB2
#define PIN_SPI_SCK    1    // PB1
#define PORT_SPI_CS    PORTC
#define DDR_SPI_CS    DDRC
#define PIN_SPI_CS1    7    // PC7
#define PIN_SPI_CS2    6    // PC6
#define PIN_SPI_CS3    5    // PC5

in der spi_master_slave.c
/*
 * spi_master_slave.c
 *
 * Created: 12.02.2015 14:08:46
 *  Author: u308488
 */ 

#include "spi_master_slave.h"

/*
// SPI Transmission/reception complete ISR
ISR(SPI_STC_vect)
{
    // Code to execute
    // whenever transmission/reception
    // is complete.
}
*/

void spi_init_slave (void)
{
   DDR_SPI |= (1<<PIN_SPI_MISO);    // MISO als Ausgang definieren
   SPCR |= (1<<SPE);                    //Enable SPI

}

//Initialize SPI Master Device
void spi_init_master (void)            
{
  DDR_SPI = (1<<PIN_SPI_MOSI) | (1<<PIN_SPI_SCK); // MOSI und SCK als Ausgang definieren
  DDR_SPI &= ~(1<<PIN_SPI_MISO);          // MISO als Eingang definieren
  PORT_SPI = (1<<PIN_SPI_MOSI) | (1<<PIN_SPI_SCK);
  
//  SPCR = (1<<SPR1)|(1<<SPR0);            // Prescaler: Fosc/128, Enable Interrupts
//  SPCR = (1<<SPI2X);                // Prescaler: Fosc/2,
  SPCR = (1<<MSTR);                // Set as Master
  SPCR = (1<<SPE);                // Enable SPI
  //  SPCR = (1<<SPIE);              // SPI Interrupt Enable
    
  SPSR |= (0 << SPIF); // SPIF zum Start löschen 
  _delay_ms(10);
}

//Function to send and receive data
char spi_tranceiver (char data, unsigned char cs)
{
  PORT_SPI_CS &= ~(1<<cs);    //SS am Slave Low --> Beginn der Übertragung
  SWITCH_LED2;
    SPDR = data;                       //Load data into the buffer
    while(!(SPSR & (1<<SPIF)));     //Wait until transmission complete
  PORT_SPI_CS |= (1<<cs);      //SS High --> Ende der Übertragung
  return SPDR;                     //Return received data  
}

--

Wie schon von Mitforisten angemerkt: Quelltext in [ c ] [ /c ] - Tags 
einschließen.

-rufus

[Mitlesa (Gast)]

CG schrieb:
> Ich bedanke mich im Vorfeld für alle hilfreichen Antworten :-)

Die Leser würden sich bedanken wenn du übersichtlichen, sauber
tabulierten Code zeigen würdest. Insbesondere die C-Code
Formatierung beim Posten wäre hilfreich.

[CG (Gast)]

Okay
dafür entschuldige ich mich und werde dies fürs nächste mal beachten

[Jope (Gast)]

Sollte der SPI_CS-Pin in spi_init_master() nicht als Ausgang definiert 
werden?

[dummy (Gast)]

Der SS Pin MUSS beim SPI Master ein Ausgang sein.
Acu wenn er nicht benutzt wird.

[Mitlesa (Gast)]

dummy schrieb:
> Der SS Pin MUSS beim SPI Master ein Ausgang sein.
> Acu wenn er nicht benutzt wird.

Nö.

[N.G. (Gast)]

nein, er muss nicht auf Ausgang stehen, aber er darf niemals low werden, 
sonst geht der µC vom Master- in den Slave-mode über.

[dummy (Gast)]

>nein, er muss nicht auf Ausgang stehen, aber er darf niemals low werden,
>sonst geht der µC vom Master- in den Slave-mode über.

Und ist deshalb als Eingang nicht mehr sinvoll verwendbar.
Tolles "nein". Als Ausgang kann man den Pin wenigstens noch gebrauchen.

[N.G. (Gast)]

dummy schrieb:
> Und ist deshalb als Eingang nicht mehr sinvoll verwendbar.
> Tolles "nein". Als Ausgang kann man den Pin wenigstens noch gebrauchen.

Tolles "MUSS". Ich wollte nur andeuten, dass man den Pin schon auf Input 
haben kann, in manchen Schaltungen mag das auch gewollt sein, aber du 
hast mit deiner Aussage was den Sinn angeht natürlich recht :)

[Cyblord -. (cyblord)]

dummy schrieb:
> Als Ausgang kann man den Pin wenigstens noch gebrauchen.

Darum macht es Sinn ihn direkt als CS Pin zu nehmen. Dafür geht er gut, 
man hat keinen Pin verschwendet und man kommt später nicht in Versuchung 
den Pin als Eingang zu verwenden.

[Mitlesa (Gast)]

Hilft nicht bei seinem SPI Problem da die SPI Maschine im
Master Mode unabhängig von der Betätigung des SS Pin arbeitet.

[Debugger (Gast)]

  SPCR = (1<<MSTR);                // Set as Master
  SPCR = (1<<SPE);                // Enable SPI

bei enable SPI wird der vorher gesetzte Master mode wieder zurückgesetzt 
-> Slave

[CG (Gast)]

Mitlesa schrieb:
> Mitlesa

Ihr habt zwar alle recht aber leider hilft es wie Mitlesa schon schrieb 
es mir nicht bei meinem Problem :-(

[Mitlesa (Gast)]

CG schrieb:
> hilft es wie Mitlesa schon schrieb es mir nicht bei meinem Problem :-(

Aber was er schreibt sollte schon beachtet werden! :

Debugger schrieb:
> bei enable SPI wird der vorher gesetzte Master mode wieder zurückgesetzt
> -> Slave

[CG (Gast)]

Das habe ich erst gesehen als ich es schon abgeschickt hatte.
Werde es direkt mal ausprobieren und euch berichten :-)

[Martin (Gast)]

Das ist ja schrecklich ohne richtige Code Übersicht

#include "global.h"          // Globale Lib

int main(void)
{
  char data;         //Received data stored here

  cli();            // Disable all interrupts
  spi_init_master();      // SPI Inizalisierung
//  spi_init_slave ();
  initPort();          // Inizalisierung der Ports
  sei();            // enable all interrupts

while(1)
    {
  SWITCH_LED1;
  _delay_ms(500);
  data = 0x00;                    //Reset ACK in "data"
    data = spi_tranceiver('Hallo', PIN_SPI_CS1);
  SET_LED3;
  }
}

in der global.h

// SPI
#define PORT_SPI    PORTB
#define DDR_SPI      DDRB
#define PIN_SPI_MISO  3    // PB3
#define PIN_SPI_MOSI  2    // PB2
#define PIN_SPI_SCK    1    // PB1
#define PORT_SPI_CS    PORTC
#define DDR_SPI_CS    DDRC
#define PIN_SPI_CS1    7    // PC7
#define PIN_SPI_CS2    6    // PC6
#define PIN_SPI_CS3    5    // PC5

in der spi_master_slave.c
/*
 * spi_master_slave.c
 *
 * Created: 12.02.2015 14:08:46
 *  Author: u308488
 */

#include "spi_master_slave.h"

/*
// SPI Transmission/reception complete ISR
ISR(SPI_STC_vect)
{
    // Code to execute
    // whenever transmission/reception
    // is complete.
}
*/

void spi_init_slave (void)
{
   DDR_SPI |= (1<<PIN_SPI_MISO);    // MISO als Ausgang definieren
   SPCR |= (1<<SPE);                    //Enable SPI

}

//Initialize SPI Master Device
void spi_init_master (void)
{
  DDR_SPI = (1<<PIN_SPI_MOSI) | (1<<PIN_SPI_SCK); // MOSI und SCK als 
Ausgang definieren
  DDR_SPI &= ~(1<<PIN_SPI_MISO);          // MISO als Eingang definieren
  PORT_SPI = (1<<PIN_SPI_MOSI) | (1<<PIN_SPI_SCK);

//  SPCR = (1<<SPR1)|(1<<SPR0);            // Prescaler: Fosc/128, 
Enable Interrupts
//  SPCR = (1<<SPI2X);                // Prescaler: Fosc/2,
  SPCR = (1<<MSTR);                // Set as Master
  SPCR = (1<<SPE);                // Enable SPI
  //  SPCR = (1<<SPIE);              // SPI Interrupt Enable

  SPSR |= (0 << SPIF); // SPIF zum Start löschen
  _delay_ms(10);
}

//Function to send and receive data
char spi_tranceiver (char data, unsigned char cs)
{
  PORT_SPI_CS &= ~(1<<cs);    //SS am Slave Low --> Beginn der Übertragung
  SWITCH_LED2;
    SPDR = data;                       //Load data into the buffer
    while(!(SPSR & (1<<SPIF)));     //Wait until transmission complete
  PORT_SPI_CS |= (1<<cs);      //SS High --> Ende der Übertragung
  return SPDR;                     //Return received data
}

[CG (Gast)]

Debugger schrieb:
> SPCR = (1<<MSTR);                // Set as Master
>   SPCR = (1<<SPE);                // Enable SPI
>
> bei enable SPI wird der vorher gesetzte Master mode wieder zurückgesetzt
> -> Slave

Super es funktioniert :-)

Habe diesen Bereich einfach umgeschieben und siehe da Daten werden 
versendet.
SPCR = (1<<MSTR) | (1<<SPE); // Set as Master, Enable SPI

Dankeschön für eure Hilfe. :-)

[Peter D. (peda)]

Martin schrieb:
> spi_init_master();      // SPI Inizalisierung
>   initPort();          // Inizalisierung der Ports

Falsche Reihenfolge!

Der /CS Pin muß zuerst Ausgang werden!

[CG (Gast)]

Oh das habe ich übersehen danke.

Darf ich noch fragen ob die slave Routien richtig ist?