Need help on SPI code for Atmega16

[micropcb]

Hi everyone,

I am trying to establish communication between two Atmega16s using SPI
One Atmega16 is configured as master and the other as slave.
The connections are as follows :

Master                  Slave
SCK          <--->       SCK
MOSI        <--->     MOSI
MISO        <--->     MISO
SS            <--->        SS

I am expecting to get the binary 1,2,3....etc on the master's portA (common anode LEDs) every one second.

But it does not work. I am using SS from the master to synchronize .
I shall be obliged if somebody can help me find the bug.
Codes attached.

Thanks in advance.


    

[robotai]

The SPIF would clear after read both SPSR and SPDR. So the slave code may need to read out SPDR in order to clear the SPIF flag.

Try this code out on slave to see if it works,

Code:

while(1)
{
  SPDR = ++i;
  while ( !(SPSR & (1<<SPIF)));
  PORTA = ~SPDR;
}

[micropcb]

I tried that robotai but no response.
Attached proteus simulation and AVR studio files

[robotai]

Well, it seems you have a mistyping on register flag. ((1<SPE))

correction on master.c

Code:

//SPCR = (1<SPE) | (1<<MSTR) | (1<<SPR0)|(1<<SPR1);
SPCR = (1<<SPE) | (1<<MSTR) | (1<<SPR0)|(1<<SPR1);

correction on slave.c

Code:

//SPCR = (1<SPE) | (1<<SPR0) |(1<<SPR1);
SPCR = (1<<SPE)| (1<<SPR0) |(1<<SPR1);

[metal]

http://www.rocketnumbernine.com/2009/04/26/using-spi-on-an-avr-1

[micropcb]

Well, it seems you have a mistyping on register flag. ((1<SPE))

correction on master.c

Code:

//SPCR = (1<SPE) | (1<<MSTR) | (1<<SPR0)|(1<<SPR1);
SPCR = (1<<SPE) | (1<<MSTR) | (1<<SPR0)|(1<<SPR1);

correction on slave.c

Code:

//SPCR = (1<SPE) | (1<<SPR0) |(1<<SPR1);
SPCR = (1<<SPE)| (1<<SPR0) |(1<<SPR1);

How silly of me not to have noticed that !!
Now it works like a charm!!

[micropcb]

Here are the files of the working project for the use of anybody who may be interested.

Cheers!