Atmega als SPI Slave

[spi (Gast)]

Hallo, ich möchte mit einem Atmega als SPI Slave Daten von einem anderen 
IC empfangen, der als Master läuft. Den Atmega habe ich wie folgt 
konfiguriert:

SPI.c
#include <avr/io.h>
#include <avr/interrupt.h>
#include "uart.h"

unsigned char status;

void spi_slave_init (void) {
  DDRB |= (1<<PB6);              //MISO output
  //SPI on + Interrupt, MSB first, Slave select, CLK idle low, sample rising edge
  SPCR = (1<<SPE) | (1<<SPIE) | (0<<DORD) | (0<<MSTR) | (0<<CPOL) | (0<<CPHA);        
  status = SPSR;                //Status löschen
}

char spi_slave_receive(void)
{uint8_t dummy;
/* Wait for reception complete */
while(!(SPSR & (1<<SPIF)));
dummy=SPDR;
/* Return Data Register */
return SPDR;
}

Allerdings bin ich beim Empfang nicht sicher, wie das richtig gemacht 
wird.

SETREG_DATA   // grün
SETRXTX       // blau

SPDR=0x00;
byte2=spi_slave_receive();
if(byte2==0x13) uart_puts_P("byte2");  // Inhalt soll testweise per UART ausgegeben werden (code von Peter Fleury)
else uart_puts_P("xxx");
  
_delay_ms(20);
CLRREG_DATA
CLRRXTX

Auf dem Bild im Anhang ist zu sehen, wie der Master-IC bei Aktivierung 
der beiden Ports (grün & blau) kurz den Clock inaktiv setzt und dann 24 
Bit sendet. Ein SS vom Master aus gibt es nicht, deshalb habe ich den 
Atmega SS dauerhaft auf GND gelegt.

[J.-u. G. (juwe)]

spi schrieb:
> SPCR = (1<<SPE) | (1<<SPIE) | (0<<DORD) | (0<<MSTR) | (0<<CPOL) | (0<<CPHA);
Wenn Du den SPI-Interrupt nicht nutzt, brauchst Du auch nicht SPIE 
setzen.

> dummy=SPDR;
> /* Return Data Register */
> return SPDR;
Warum dieser doppelt hintereinander ausgeführte Zugriff auf SPDR?

> Allerdings bin ich beim Empfang nicht sicher, wie das richtig gemacht
> wird.
Falls Dein Code nicht funktioniert, gib doch bitte eine richtige 
Fehlerbeschreibung.

> if(byte2==0x13) uart_puts_P("byte2");  // Inhalt soll testweise per UART 
>ausgegeben werden
Hier wird nicht der Inhalt ausgegeben, sondern der Text "byte2".

[spi (Gast)]

J.-u. G. schrieb:
> spi schrieb:
>> SPCR = (1<<SPE) | (1<<SPIE) | (0<<DORD) | (0<<MSTR) | (0<<CPOL) | (0<<CPHA);
> Wenn Du den SPI-Interrupt nicht nutzt, brauchst Du auch nicht SPIE
> setzen.
>
OK, das hatte ich mal testweise auch versucht.
>> dummy=SPDR;
>> /* Return Data Register */
>> return SPDR;
> Warum dieser doppelt hintereinander ausgeführte Zugriff auf SPDR?
>
Hatte irgendwo gelesen, dass man das vorher nochmal auslesen müsse.
>> Allerdings bin ich beim Empfang nicht sicher, wie das richtig gemacht
>> wird.
> Falls Dein Code nicht funktioniert, gib doch bitte eine richtige
> Fehlerbeschreibung.
>
Naja, wenn ich spi_slave_receive() aufrufe bleibt er halt in der while 
hängen und es passiert nichts mehr.
>> if(byte2==0x13) uart_puts_P("byte2");  // Inhalt soll testweise per UART
>>ausgegeben werden
> Hier wird nicht der Inhalt ausgegeben, sondern der Text "byte2".
Ja zumindest sollte der Inhalt geprüft werden.

Wenn ich im Debugmodus in das SPDR was schreibe, müsste ich das doch in 
der Registeranzeige rechts sehen oder? Wenn ich z.B. SPDR=0x01; 
schreibe, tut sich da allerdings gar nichts.

[spi (Gast)]

Keiner ne Idee?

[J.-u. G. (juwe)]

spi schrieb:
> Naja, wenn ich spi_slave_receive() aufrufe bleibt er halt in der while
> hängen und es passiert nichts mehr.
Woran erkennst Du, dass er genau da hängen bleibt?

Zeig doch mal mehr von Deiner main()-Funktion.

[schwarz (Gast)]

Ist das Power Reduction Register richtig gesetzt?
  // Enable SPI (disable it in Power Reduction Register)
  PRR &= ~(1u << PRSPI);
Bin mir aber nicht sicher, ob das nicht schon standardmäßig gesetzt ist.

> Wenn ich im Debugmodus in das SPDR was schreibe, müsste ich das doch in
> der Registeranzeige rechts sehen oder? Wenn ich z.B. SPDR=0x01;
> schreibe, tut sich da allerdings gar nichts.

SPDR besteht aus zwei Registern, eins zum Lesen, eins zum Schreiben. 
Hier hatte ich auch schon das Problem, dass nicht das (für mich) 
richtige angezeigt wurde.

[spi (Gast)]

J.-u. G. schrieb:
> spi schrieb:
>> Naja, wenn ich spi_slave_receive() aufrufe bleibt er halt in der while
>> hängen und es passiert nichts mehr.
> Woran erkennst Du, dass er genau da hängen bleibt?
>
> Zeig doch mal mehr von Deiner main()-Funktion.

Zumindest kommt er aus der Funktion nicht mehr haeraus, da darauf 
folgende Aufrufe nicht mehr ausgeführt werden.

schwarz schrieb:
> Ist das Power Reduction Register richtig gesetzt?
>   // Enable SPI (disable it in Power Reduction Register)
>   PRR &= ~(1u << PRSPI);
> Bin mir aber nicht sicher, ob das nicht schon standardmäßig gesetzt ist.
>
Da habe ich gar nix mit gemacht, das ist wohl standardmäßig auch nicht 
von Bedeutung.

[spi (Gast)]

In main steht im Prinzip folgendes (UART-Kram mal entfernt):

/* define CPU frequency in Mhz here if not defined in Makefile */
#ifndef F_CPU
#define F_CPU 1000000UL
#endif

#include <stdlib.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <util/delay.h>

#include "uart.h"
#include "adc.h"
#include "timer.h"
#include "usartspi.h"
#include "spi.h"




/* 2400 baud */
#define UART_BAUD_RATE      2400      


/* ~SS line macros. (AD9833) */
#define SSPIN     PD6 //!< The pin used as ~SS to slave.
#define CSPIN     PD5 //!< The pin used as ~CS to slave.
#define SSDDR     DDRD //!< The data direction register for the ~SS line.
#define SSPORT    PORTD //!< The port register for the ~SS line.
#define SETSSDIR  SSDDR |= (1<<SSPIN); //!< Set output direction for ~SS line.
#define SETSSLINE SSPORT |= (1<<SSPIN); //!< Set ~SS line high.
#define CLRSSLINE SSPORT &= (0<<SSPIN); //!< Set ~SS line low.
#define SETCSLINE SSPORT |= (1<<CSPIN); //!< Set ~CS line high.
#define CLRCSLINE SSPORT &= (0<<CSPIN); //!< Set ~CS line low.

/* ST7540 line macros */
#define RSTO        PA2 //!< 
#define WD          PD2 //!< 
#define BUTHERM     PB7 //!< 
#define RXTX        PB2 //!< 
#define SETRXTX     PORTB |= (1<<RXTX);
#define CLRRXTX     PORTB &= (0<<RXTX);
#define REG_DATA    PB1 //!< 
#define SETREG_DATA PORTB |= (1<<REG_DATA);
#define CLRREG_DATA PORTB &= (0<<REG_DATA);
#define CD_PD       PB0 //!< 
#define SETSS    PORTB |= (1<<PB4);
#define CLRSS     PORTB &= (0<<PB4);



int main(void)
{

  CLRREG_DATA
  SETRXTX

  DDRA |= (0<<RSTO);
  DDRB |= (1<<RXTX) | (1<<REG_DATA) | (0<<BUTHERM) | (0<<CD_PD);



volatile unsigned char byte2, byte1, byte0;

      
          //SPDR=0x01;
          
          SETREG_DATA
      
          spi_slave_init();
          //SETRXTX
      
          byte2=spi_slave_receive();
          byte1=spi_slave_receive();
          byte0=spi_slave_receive();

          uart_puts_P("byte");
      
          _delay_ms(20);
          CLRREG_DATA
          //CLRRXTX
  

//(...)
  
}

[J.-u. G. (juwe)]

spi schrieb:
> #define CLRSSLINE SSPORT &= (0<<SSPIN); //!< Set ~SS line low.
...
> #define CLRCSLINE SSPORT &= (0<<CSPIN); //!< Set ~CS line low.
...

> DDRA |= (0<<RSTO);
> DDRB |= (1<<RXTX) | (1<<REG_DATA) | (0<<BUTHERM) | (0<<CD_PD);

Nein, nein, nein! Bitte noch mal hier:
http://www.mikrocontroller.net/articles/Bitmanipulation
nachlesen, wie man Bits löscht.

Du scheinst ja einiges an Peripherie angeschlossen zu haben. Fehlerhaft 
beschriebene Konfigurationsregister können natürlich dann zu Hängern 
führen.

[spi (Gast)]

Toll Fehler lag dann doch an der Hardware, SPI SCK fehlerhaft 
angeschlossen.

[spi (Gast)]

J.-u. G. schrieb:
> spi schrieb:
>> #define CLRSSLINE SSPORT &= (0<<SSPIN); //!< Set ~SS line low.
> ...
>> #define CLRCSLINE SSPORT &= (0<<CSPIN); //!< Set ~CS line low.
> ...
>
>> DDRA |= (0<<RSTO);
>> DDRB |= (1<<RXTX) | (1<<REG_DATA) | (0<<BUTHERM) | (0<<CD_PD);
>
> Nein, nein, nein! Bitte noch mal hier:
> http://www.mikrocontroller.net/articles/Bitmanipulation
> nachlesen, wie man Bits löscht.
>
> Du scheinst ja einiges an Peripherie angeschlossen zu haben. Fehlerhaft
> beschriebene Konfigurationsregister können natürlich dann zu Hängern
> führen.

Wäre das so richtig?

DDRA |= (0<<RSTO);
DDRB |= (1<<RXTX) | (1<<REG_DATA);
DDRB &= ~((1<<BUTHERM) | (1<<CD_PD));

[J.-u. G. (juwe)]

spi schrieb:
> Wäre das so richtig?

> DDRA |= (0<<RSTO);
Nein. Diese Anweisung bewirkt nichts.

> DDRB |= (1<<RXTX) | (1<<REG_DATA);
> DDRB &= ~((1<<BUTHERM) | (1<<CD_PD));

Ja. So ist es richtig.

[spi (Gast)]

OK danke

[Mario S. (drm)]

Hallo!
Offenbar handelt es sich hier um den Code für ein Powerline Modem.
Bin selbger grade am Basteln, möchte so etwas basierend auf dem Arduino 
machen.
Bin leider Anfänger, in diesem Bereich und man findet nicht sehr viel 
Code rund um den ST7540.
Würdest du den Code mit mir teilen, damit könnte ich mal starten? Auf 
welcher Schaltung basiert dein Modem?