SPI mit ATMEGA328 als Master und X25020 als slave

[Aa B. (aaab)]

Ich habe das letztes mal vor 2-3 Jahre, SPI mit fertige Arduino
Libraries programmiert.
Heute habe ich mit ATMEGA328p als Master und X25020P(EEPROM) als Slave
konfiguriert. Ich würde sagen 'auf gut Deutsch'- das ist das billigste
von billigste SPI Programm. :)

Also, das Ziel war ein Byte (0x01) schreiben auf Adresse 0x20 und die
geschriebene Byte wieder lesen. Wenn lesen erfolgreich war, leuchtet ein
LED.

Ich wusste was wo angeschlossen werden muss, also nicht zu viel HW
Aufwand.
Datenblatt von x25020p chip:
http://pdf.datasheetcatalog.com/datasheet_pdf/xicor/X25020P_to_X25020SI-2,7.pdf


C code:

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



void spi_master_init()
{
   DDRB = ( (1<<PB2) | (1<<PB3)  | (1<<PB5) );   // /SS as output, MOSI and SCK as output
   SPCR = ( (1<<SPE) | (1<<SPR1) | (1<<MSTR)); //SPI Enable, fosc/64, Master mode
}


void spi_slave_init()
{
   DDRB = 0<<PB4;     //MOSI as input
   SPCR = 1<<SPE ;
}


void SPI_Tx(char data)
{
   SPDR=data;
   while( !(SPSR & (1<<SPIF)) );   //wait until SPIF flag is set, to indicate transmission is complete
}

char SPI_Rx()
{
   char Rx_char;   
   SPI_Tx(0);
   while( !(SPSR & (1<<SPIF)) );
     Rx_char= SPDR;
     
   return Rx_char;
}

void write_to_slave()
{
   PORTB|=1<<PB2;    //take /CS on slave LOW
   SPI_Tx(0x06);  //WREN write enable command
   PORTB&=~(1<<PB2);    //take /CS on slave HIGH
   
   PORTB|=1<<PB2;    //take /CS on slave LOW
   SPI_Tx(0x02);  //WRITE instruction
   SPI_Tx(0x20);  //Address to write data to
   SPI_Tx(0x01);  //Data to be written
   PORTB&=~(1<<PB2);    //take /CS  on slave HIGH
}

char read_from_slave()
{
   char recvd_char;
   PORTB|=1<<PB2;    //take /CS on slave LOW
   SPI_Tx(0x03);  //READ instruction
   SPI_Tx(0x20);  //Address to be read from
 
   recvd_char = SPI_Rx();
   return recvd_char;
}

int main (void)
{
   DDRC=0xff;   //PORTC as output
  
   
   char recvdchar=0;
   
   spi_master_init();
   spi_slave_init();
   
   write_to_slave();
   
   recvdchar=read_from_slave();

   if(recvdchar==0x01)
    PORTC=1<<PC0;




}

Das Problem ist nun, LED leuchtet nicht wie erwartet. Was habe ich 
falsch gemacht!?

[S. Landolt (Gast)]

Der uC ist der Master - wozu also dieses spi_slave_init()?

[Aa B. (aaab)]

S. Landolt schrieb:
> Der uC ist der Master - wozu also dieses spi_slave_init()?

um die MOSI pin in DDRB explizit als Input zu deklarieren, hätte ich in 
spi_master_init() schreiben können aber fand ich übersichtlicher es 
separat zu schreiben!

[S. Landolt (Gast)]

In spi_slave_init() steht
> SPCR = 1<<SPE ;
damit wird das
> SPCR = ( (1<<SPE) | (1<<SPR1) | (1<<MSTR));
aus spi_master_init() glatt überschrieben.

> PORTB|=1<<PB2;    //take /CS on slave LOW
> PORTB&=~(1<<PB2);    //take /CS on slave HIGH
Das ist doch jeweils genau umgekehrt, oder hängt ein Inverter 
dazwischen?

[Aa B. (aaab)]

S. Landolt schrieb:
> In spi_slave_init() steht
>> SPCR = 1<<SPE ;
> damit wird das
>> SPCR = ( (1<<SPE) | (1<<SPR1) | (1<<MSTR));
> aus spi_master_init() glatt überschrieben.

Das stimmt, muss ich korrigieren!

>> PORTB|=1<<PB2;    //take /CS on slave LOW
>> PORTB&=~(1<<PB2);    //take /CS on slave HIGH
> Das ist doch jeweils genau umgekehrt, oder hängt ein Inverter
> dazwischen?
/CS ist ein active low auf dem IC, und auf dem Master /SS ist active 
low.

[Aa B. (aaab)]

Das is auch quatsch!
DDRB = 0<<PB4;     //MOSI as input

Sollte so sein oder?
DDRB |= ~(1<<PB4);     //MOSI as input

[S. Landolt (Gast)]

Ich hätte es ganz weggelassen, der Reset-Wert ist ja 0, ansonsten
DDRB &= ~(1<<PB4);     //MOSI as input

[S. Landolt (Gast)]

Nicht nur der Reset-Wert, PB4 wird auch explizit auf 0 gesetzt mit dem
DDRB = ...
in spi_master_init().

[Aa B. (aaab)]

S. Landolt schrieb:
> Nicht nur der Reset-Wert, PB4 wird auch explizit auf 0 gesetzt mit
> dem
> DDRB = ...
> in spi_master_init().

Ja dass stimmt, danke für den Tipp!