SPI-Kommunikation zwischen zwei ATTiny2313

[Jens K. (mister232)]

Guten Abend,

ich versuche gerade eine SPI-Kommunikation zwischen zwei ATTiny2313 
herzustellen. Ich nutze dafür die USI-Schnittstelle der Mikrocontroller. 
Ich verwende den internen RC-Oszillator. Später werde ich auf einen 
externen Quarz umsteigen, doch ich will erst einmal schauen ob ich 
überhaupt irgendetwas rüber bekomme.

Der Ablauf des Programms ist wie folgt:
1. Der Master bekommt einen Char über die UART-Schnittstelle vom 
Computer (das funktioniert auch soweit)
2. Der Master sendet das Char an den Slave
3. Der Slave lässt entsprechend der Zahl die er empfangen hat eine LED 
aufblinken

Hier mein Quellcode, zunächst nur vom Master:

/*
 * Communication_via_SPI_Master.c
 *
 * Created: 21.01.2014 15:02:29
 *  Author: Jens
 */ 


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

#define FCPU 1000000L
#define USART_BAUD 4800L
#define USART_UBBR_VALUE ((FCPU)/(USART_BAUD*16)-1)

// Initialization for the UART-Interface
void USART_vInit(void)
{
  // Set baudrate
  UBRRH = (uint8_t) (USART_UBBR_VALUE>>8);
  UBRRL = (uint8_t) USART_UBBR_VALUE;
  // Enable receiver and transmitter
  UCSRB = (1<<RXEN)|(1<<TXEN);
  // Set frame format to 8 data bits, no parity, 2 stop bits
  UCSRC = (1<<USBS)|(1<<UCSZ1)|(1<<UCSZ0);
}

// Initialization for the SPI-Interface
void SPI_init(void)
{
  //Output: PB7=USCK, PB6=DO, PB4=LED
  DDRB |= (1<<PB7)|(1<<PB6)|(1<<PB4);
  //Input: PB5=DI
  DDRB &= ~(1<<PB5);
  // Enable internal pull-up for PB5
  PORTB |= (1<<PB5);
  // Enable SPI in master mode
  USICR = (1<<USIWM0)|(1<<USICS1)|(1<<USICLK)|(1<<USITC);
}

// Function for receiving data via UART-Interface
unsigned char USART_ReceiveByte(void)
{
  // Wait until a byte has been received
  while(!(UCSRA&(1<<RXC)));
  // Return received data
  return UDR;
}

// Function for transmitting data via SPI-Interface
void SPI_Transfer(unsigned char data)
{
  // Load data into the data-register
  USIDR = data;
}

int main (void)
{
  unsigned char u8Data;
  DDRB = 0xFF;
  
  // Initialize USART and SPI
  USART_vInit();
  SPI_init();
  
  while(1)
  {
    // Receive data via UART
    u8Data = USART_ReceiveByte();
    while(!(UCSRA & (1<<UDRE)));
    UDR = u8Data;
    // Transmit the received data to the SPI-Slave
    SPI_Transfer(u8Data);
    // Led on
    PORTB |= (1<<PB4);
    _delay_ms(500);
    // Led off
    PORTB &= !(1<<PB4);
    _delay_ms(500);
  }
}

Leider kann ich beim senden der Daten keine Reaktion des Slave 
feststellen und auch ein Oszi zeigt nichts auf dem Bus an.

Könntet ihr vielleicht mal rüberschauen? Ich bin gerade erst dabei die 
SPI-Kommunikation kennzulernen, also bitte Verständnis für die unsaubere 
Programmierung :-)

[tictactoe (Gast)]

Jens K. schrieb:
> ich versuche gerade eine SPI-Kommunikation zwischen zwei ATTiny2313
> herzustellen. Ich nutze dafür die USI-Schnittstelle der Mikrocontroller.

Soweit ich im Datasheet gelesen habe, geht bei den ATTiny2313 nix 
automatisch. D.h., das hier:

void SPI_Transfer(unsigned char data)
{
  // Load data into the data-register
  USIDR = data;
}

bringt so gar nix. Du musst schon für die Clock-Pulses selber sorgen.
Zitat: "When operating as master, clock pulses are software generated by
toggling the PORTA register, while the data direction is set to output. 
The USITC bit in the USICR Register can be used for this purpose."

[c-hater (Gast)]

Jens K. schrieb:

> Leider kann ich beim senden der Daten keine Reaktion des Slave
> feststellen und auch ein Oszi zeigt nichts auf dem Bus an.

Was nicht anders zu erwarten ist.

> Ich bin gerade erst dabei die
> SPI-Kommunikation kennzulernen, also bitte Verständnis für die unsaubere
> Programmierung :-)

Das ist keine "unsaubere Programmierung", sondern ein deutliches 
Zeichen, daß du schlicht das Datenblatt zum Thema USI nicht gelesen hast 
und deswegen nicht einmal eine näherungsweise Vorstellung davon hast, 
was du da eigentlich tust.

Sonst wußtest du, daß das USI kaum mehr ist als ein dummes 
Schieberegister und ein genauso blöder Zähler. Da mußt du schon noch 
etwas mehr tun, damit sich da irgendwelche Bits bewegen, insbesondere 
z.B. für einen Takt für diese beiden Sachen sorgen...

Welche Möglichkeiten das Teil dazu bietet, steht, wer hätte das gedacht, 
im Datenblatt, also lies' das doch bitte einfach mal, bevor du anfängst, 
planlos drauflos zu programmieren.