Atmega328 SPI SPIF wird nicht gesetzt

[Philipp H. (philipp_ho)]

Hallo zusammen,

ich habe folgendes Problem: Ich möchte eine simple SPI Verbindung mit 
einem Atmega328 (Arduino UNI) testen. Zunächste einfach nur Master 
richtig initialisieren und auf Funktion testen. Byte in SPDR schreiben 
und warten bis die Flag gesetzt wird, LED an. Aber aus irgendeinem Grund 
wird die Flag nicht gesetzt. SS ist Output (anscheinend häufiger 
Fehler).

#include <stdint.h>
#include <stdlib.h>
#include <stdbool.h>
#include <avr/io.h>

#define SET_REG_BIT(a,b)  a = a | (1<<b)
#define DEL_REG_BIT(a,b)  a = a & ~(1<<b)
#define FLIP_REG_BIT(a,b)  a = a ^ (1<<b)
#define CHECK_REG_BIT(a,b)  (a = (a & (1<<b))>>b)





int main(void)
{
        //LED Pin -> Output
  SET_REG_BIT(DDRB, DDB0);
  //Set PB2(SS) to Output
  SET_REG_BIT(DDRB, DDB2);
  //Set PB5(SCK) to Output
  SET_REG_BIT(DDRB, DDB5);
  //Set PB3(MOSI) to Output
  SET_REG_BIT(DDRB, DDB3);
  //Enable SPI
  SET_REG_BIT(SPCR, SPE);
  //Set SPI to Master Mode
  SET_REG_BIT(SPCR, MSTR);
  //Set Clock Rate fck/16
  SET_REG_BIT(SPCR, SPR0);

  while(1)
  {
    //Pull SS LOW
    DEL_REG_BIT(PORTB, PORTB2);
    //Write Data to SPI Data Register
    SPDR = 0xFF;
    //Wait for SPI Flag = 1
    while(!(SPSR & (1<<SPIF)));
    //Activate LED
    SET_REG_BIT(PORTB, PORTB0);
    //Pull SS HIGH
    SET_REG_BIT(PORTB, PORTB2);
  }
}

[S. Landolt (Gast)]

Vielleicht mag SPI es nicht, häppchenweise eingeschaltet zu werden. Ich 
würde zumindest SPE erst nach MSTR (und SPR0) setzen.

[Veit D. (devil-elec)]

Hallo,

füge bitte einmal am Ende der Initialisierung, vor while, sei() ein.

[S. Landolt (Gast)]

> ... sei() ...
Es gibt doch gar keinen Interrupt!?

[jo mei (Gast)]

Philipp H. schrieb:
> Aber aus irgendeinem Grund wird die Flag nicht gesetzt.

SPI Enable Fuse gesetzt?

Zeige dein Projekt damit wir die Einstellungen kennen unter
denen du das Programm laufen lassen willst.

Es heisst übrigens allgemein das Flag, oder "die Flagge",
das sagt aber keine Schwein.

[S. Landolt (Gast)]

Es ist so, wie ich eingangs schrieb. Wenn MSTR erst nach SPE gesetzt 
wird, gerät SPI in den Slave-mode, SS (PB2) steht zu diesem Zeitpunkt ja 
auf low.

[S. Landolt (Gast)]

> SPI Enable Fuse gesetzt?

SPIEN betrifft ausschließlich die Programmierung: "SPIEN Enable Serial 
Program and Data Downloading". Außerdem ist ein Abschalten nur mit einem 
HV-Programmiergerät möglich: "The SPIEN Fuse is not accessible in serial 
programming mode".

[Philipp H. (philipp_ho)]

S. Landolt schrieb:
> Es ist so, wie ich eingangs schrieb. Wenn MSTR erst nach SPE gesetzt
> wird, gerät SPI in den Slave-mode, SS (PB2) steht zu diesem Zeitpunkt ja
> auf low.

Danke! Das war das Problem! Ist aber aus dem Datenblatt auch nicht 
unbedingt ersichtlich oder?

"If SS is configured as an output, the pin is a general output pin which 
does not affect the SPI
system. Typically, the pin will be driving the SS pin of the SPI Slave.
If SS is configured as an input, it must be held high to ensure Master 
SPI operation. If the SS pin
is driven low by peripheral circuitry when the SPI is configured as a 
Master with the SS pin
defined as an input, the SPI system interprets this as another master 
selecting the SPI as a
slave and starting to send data to it."

Klingt für mich als ob es kein Problem darstellen sollte wenn SS zuvor 
als Output gesetzt wurde. Wie dem auch sei, vielen Dank! :)

[S. Landolt (Gast)]

Das Datenblattzitat bezieht sich im wesentlichen auf einen SPI-Master; 
bei der ursprünglich gewählten Reihenfolge der SPCR-bits startet SPI 
jedoch als Slave, dabei ist offenbar unerheblich, ob der SS-pin als Ein- 
oder Ausgang definiert ist. SS steht auf low, folglich geht der 
SPI-Slave direkt auf Empfang.
  (stelle ich mir zumindest so vor, ohne es näher untersucht zu haben)

[Veit D. (devil-elec)]

Hallo,

habe das auch nochmal nachvollzogen. SS wurde immer als Erstes 
konfiguriert.  Alle SS/CS Pins setze ich schon aus Gewohnheit als Erstes 
auf Ausgang und High Pegel. Entscheidend ist wirklich dass das SPE Bit 
als Letztes gesetzt wird falls man alle Bits einzeln in das Register 
schreibt. Interessante Erkenntnis. Danke.

[S. Landolt (Gast)]

> Entscheidend ist wirklich dass das SPE Bit als
> Letztes gesetzt wird

Das sieht hier anders aus: entscheidend ist der Zustand von SS beim 
Setzen von SPE; ich kann dieses durchaus als Erstes setzen, wenn nur SS 
zu diesem Zeitpunkt high ist.

Mit aller Zurückhaltung möchte ich behaupten, dass das beim Test nicht 
galt:

> Alle SS/CS Pins setze ich schon aus Gewohnheit als Erstes
> auf Ausgang und High Pegel.

[Oliver S. (oliverso)]

Das ganze Problem wäre gar nicht aufgetreten, wenn du diese bescheuerten 
Bit-Setz-Makros nicht benutzt hättest.

Was ist denn an

DDRB = (1<<DDB0) | (1<<DDB2) | (1<<DDB3) | (1<<DDB5);
SPCR = (1<<SPE) | (1<<MSTR) | (1<<SPR0);

so furchtbar kompliziert?

Oliver

[S. Landolt (Gast)]

Nun ja, mir gefallen sie auch nicht, aber ist Geschmackssache - warum 
nicht?

[Oliver S. (oliverso)]

Weil man sich, wie dieser Thread hier ja zeigt, damit unnötige 
Möglichkeiten einbaut, sich selber ins Knie zu schiessen.

Oliver

[S. Landolt (Gast)]

Schon - andererseits war das Hauptproblem ja unzureichende Kenntnis der 
SPI-Funktion.
  (mich persönlich stört eher die viele Schreibarbeit; und dass das 
Programm unnötig groß wird, stößt wohl nur Assemblerprogrammierern auf)

[Veit D. (devil-elec)]

Hallo,

ähm ne. Ich wollte zum Ausdruck bringen, dass es nicht egal ist wann das 
SPE Bit gesetzt wird. Ich hatte wie gesagt für meinen Test den SS Pin 
gleich am Anfang auf Ausgang und High Pegel gesetzt. Danach habe ich nur 
noch mit der Reihenfolge der Bits im SPCR Register gespielt. Alle 
einzeln wie der TO. Master funktioniert nur und gibt Daten raus wenn das 
SPE Bit als Letztes gesetzt wird. Also so wie deine Vermutung war, S. 
Landolt. Testobjekt war mein ATmega2560.

Ob einzeln oder in einem Rutsch kommt auch immer darauf was man machen 
möchte. Zum Bsp. beim Timer habe ich immer 3 Grundfunktionen. 
Konfiguration des Timern. Starten und Stoppen des Timers. Wer SPI 
zwischen Master und Slave umschalten möchte, muss das auch irgendwie 
sinnvoll aufdröseln.

[Peter D. (peda)]

Veit D. schrieb:
> Master funktioniert nur und gibt Daten raus wenn das
> SPE Bit als Letztes gesetzt wird.

Nein, das stimmt nicht. Man kann MSTR setzen, nachdem SPE gesetzt 
wurde.
Ich würde trotzdem immer die Bits in einem Rutsch setzen. Bei manchen 
IO-Registern haben die Bits verschiedene Bedeutung beim Lesen/Schreiben, 
d.h. ein Lesen/AND/OR/Schreiben kann zu Überraschungen führen.

[Veit D. (devil-elec)]

Hallo,

habe das nochmal durchgespielt. Man sollte das in Ruhe machen. Hatte 
mich oben etwas vertan, hatte das SS High setzen zum Test rausgenommen 
und vergessen wieder reinzunehmen. Sorry. Also nochmal.  :-)
Wenn der SS Pin Ausgang und High ist, dann spielt die Bit Reihenfolge 
keine Rolle. Wenn SS Ausgang und Low ist, dann muss SPE als Letztes 
gesetzt werden. Bedeutet der SS Pegel spielt eine entscheidende Rolle, 
wie S. Landolt schon geschrieben hatte. Da man eigentlich SS und alle 
anderen SS/CS Pins immer als Erstes konfiguriert, kommt man nicht in 
diese Verlegenheit.  :-)