SPI am ATMEGA644 - Master gibt nix aus

[SPIAnfänger (Gast)]

Hi Community

Ich wollte hier mal anfragen, ob mir evtl. jemand weiterhelfen kann..
Folgendes: Ich möchte gerne Daten per SPI über MOSI und SCK ausgeben an 
einem ATMEGA644.

Ich habe dazu im Datenblatt den Beispielcode gefunden, aber irgendwie 
tut sich da nichts.

Hier mein Programm:

/**************** INCLUDES ***************************************/
#include <avr/io.h>          //I/O library
#include "lcd-routines.h"    //LCD-Routine
#include <avr/interrupt.h>   //interrupt library
#include <stdint.h>          //Datentypen
#include <util/delay.h>      //delay library
/****************************************************************/

/**************** DEFINES ***************************************/
#ifndef F_CPU
#define F_CPU 20000000  //Frequenz -> 20MHZ
#endif 
/****************************************************************/

#define BLANK 0x10
#define VPRG  0x01
#define XLAT  0x08
#define GSCLK 0x40

#define MOSI  0x20
#define SCK   0x80
#define SPI  PORTB

#define DRIVER PORTB
#define TEST   PORTB
#define ON     0x01
#define OFF    0x00


int main(void)
{
    
  lcd_init();
  SPI_MasterInit();
  
  DDRC = 0xFF;
  DDRD = 0xFF;
  PORTC = 0x00;
  
  lcd_clear();
    lcd_setcursor(0,1);
  lcd_data(' '); 
  lcd_setcursor(3,1);
  lcd_string("Debug-Display");
  
  DRIVER = (1 << XLAT) | (1 << BLANK) | (1 << VPRG) | (1 << GSCLK);

  
  for(;;)
    {

    SPI_MasterTransmit(12);
  _delay_ms(19);
  
    }
}


void SPI_MasterInit(void)
{
   /* Set MOSI and SCK output, all others input */
   SPI = (1<<MOSI)|(1<<SCK);
   /* Enable SPI, Master, set clock rate fck/16 */
   SPCR = (1<<SPE)|(1<<MSTR)|(1<<SPR0);
}

void SPI_MasterTransmit(char cData)
{
   /* Start transmission */
   SPDR = cData;
   /* Wait for transmission complete */
   while(!(SPSR & (1<<SPIF)));
}

Mit einem DSO kann ich leider nicht mal ein ruckeln am MOSI oder am SCK 
Pin feststellen, der Controller funktioniert aber sonst (Display 
angeängt als Test).

Kann mir evtl. jemand einen Tipp geben, woran das liegen könnte?

Gruss und Danke fürs lesen :)

[Karl H. (kbuchegg)]

Du musst den SS Pin auf Ausgang programmieren.

Ist eine beliebte Falle.

[SPIAnfänger (Gast)]

Karl Heinz Buchegger schrieb:
> Du musst den SS Pin auf Ausgang programmieren.

Wow, danke für die schnelle Antwort :)

void SPI_MasterInit(void)
{
   /* Set MOSI and SCK and SS output, all others input */
   SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);
   /* Enable SPI, Master, set clock rate fck/16 */
   SPCR = (1<<SPE)|(1<<MSTR)|(1<<SPR0);
  
}

Tut sich leider immer noch nix :(

SPI = DDRB und MOSI, SCK und SS sind die richtigen Pins (PB4, PB5 und 
PB7)

Fehlt sonst noch was?

[Karl H. (kbuchegg)]

SPIAnfänger schrieb:
> Karl Heinz Buchegger schrieb:
>> Du musst den SS Pin auf Ausgang programmieren.
>
> Wow, danke für die schnelle Antwort :)
>

>    /* Set MOSI and SCK and SS output, all others input */
>    SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);

#define SPI  PORTB

Damit setzt du die Pins auf 1, wenn sie auf Ausgang wären.
Aber wo ist deine Programmierung des DDRB, um den Pin überhaupt mal auf 
Ausgang zu setzen!

[Karl H. (kbuchegg)]

Karl Heinz Buchegger schrieb:
> SPIAnfänger schrieb:
>> Karl Heinz Buchegger schrieb:
>>> Du musst den SS Pin auf Ausgang programmieren.
>>
>> Wow, danke für die schnelle Antwort :)
>>
>
>>    /* Set MOSI and SCK and SS output, all others input */
>>    SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);
>
> #define SPI  PORTB
>
> Damit setzt du die Pins auf 1, wenn sie auf Ausgang wären.
> Aber wo ist deine Programmierung des DDRB, um den Pin überhaupt mal auf
> Ausgang zu setzen!

Kann es sein, dass du dich da vertan hast und du eigentlich

#define SPI   DDRB

haben wolltest?

[SPIAnfänger (Gast)]

Karl Heinz Buchegger schrieb:
> #define SPI   DDRB
>
> haben wolltest?

Ja, das habe ich in der Tat, habe das gerade gemerkt, als du geschrieben 
hast, dass ich SS als Ausgang setzen muss.

Ist also bereits bereinigt.

Hier nochmal der ganze Code, damit es keine Missverständnisse mehr gibt:

/**************** INCLUDES ***************************************/
#include <avr/io.h>          //I/O library
#include "lcd-routines.h"    //LCD-Routine
#include <avr/interrupt.h>   //interrupt library
#include <stdint.h>          //Datentypen
#include <util/delay.h>      //delay library
/****************************************************************/

/**************** DEFINES ***************************************/
#ifndef F_CPU
#define F_CPU 20000000  //Frequenz -> 20MHZ
#endif 
/****************************************************************/

#define BLANK 0x10
#define VPRG  0x01
#define XLAT  0x08
#define GSCLK 0x40

#define MOSI  0x20
#define SCK   0x80
#define SPI   DDRB
#define SS    0x10

#define DRIVER PORTB
#define TEST   PORTB
#define ON     0x01
#define OFF    0x00

int main(void)
{
    
  lcd_init();
  SPI_MasterInit();
  
  DDRC = 0xFF;
  DDRD = 0xFF;
  PORTC = 0x00;
  
  lcd_clear();
    lcd_setcursor(0,1);
  lcd_data(' '); 
  lcd_setcursor(3,1);
  lcd_string("Debug-Display");
  
  DRIVER = (1 << XLAT) | (1 << BLANK) | (1 << VPRG) | (1 << GSCLK);

  
  for(;;)
    {

    SPI_MasterTransmit(12);
  _delay_ms(20);
  
    }
}


void SPI_MasterInit(void)
{
   /* Set MOSI, SCK and SS output, all others input */
   SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);
   /* Enable SPI, Master, set clock rate fck/16 */
   SPCR = (1<<SPE)|(1<<MSTR)|(1<<SPR0); 
}

void SPI_MasterTransmit(char cData)
{
   /* Start transmission */
   SPDR = cData;
   /* Wait for transmission complete */
   while(!(SPSR & (1<<SPIF)));
}

[Spess53 (Gast)]

Hi

>#define MOSI  0x20  (32)
>#define SCK   0x80  (128)

>   SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);

Was meinst du, was bei 8 Bit herauskommt, wenn man eine 1 32 oder 128 
mal nach links schiebt?

MfG Spess

[SPIAnfänger (Gast)]

Spess53 schrieb:
> Was meinst du, was bei 8 Bit herauskommt, wenn man eine 1 32 oder 128
> mal nach links schiebt?

Oh, peinlich :S

DDRB = 0xB0 hat Abhilfe gebracht.
Die korrekte Schreibweise lese ich noch im Tut. auf der Seite nach, bin 
da immer ein wenig zerstreut.

Danke für eure Hilfe!

[SPIAnfänger (Gast)]

Wäre es möglich, dass die SCK Rate beim ATMEGA644 entgegen dem 
Datenblatt NICHT die fosc/2 schafft?

Bei:

SPCR = (1<<SPE)|(1<<MSTR);

Habe ich (wie im Datenblatt beschrieben -> siehe Anhang) einen Vorteiler 
von 4. Will ich nun aber

SPCR = (1<<SPE)|(1<<MSTR)| (1 << SPI2X);

Habe ich einen Vorteiler von 16?

Wo liegt mein Denkfehler?

[H.Joachim S. (crazyhorse)]

SPI2X liegt nicht im SPCR :-)

[Spess53 (Gast)]

Hi

>Wo liegt mein Denkfehler?

SPI2X ist in SPSR nicht in SPCR.

MfG Spess

[SPIAnfänger (Gast)]

Ach so.. das konnte ich jetzt irgendwie nicht rauslesen.
Habe mich allerdings schon gewundert, warum nur die Bits SPR1 und SPR0 
in der Beschreibung erwähnt werden.

Ein bisschen undurchsichtig, finde ich.

Danke nochmals für eure Hilfe, jetzt läuft alles wie ich wollte!
8 Bit in 800ns, perfekt.

[Karl H. (kbuchegg)]

SPIAnfänger schrieb:
> Ach so.. das konnte ich jetzt irgendwie nicht rauslesen.
> Habe mich allerdings schon gewundert, warum nur die Bits SPR1 und SPR0
> in der Beschreibung erwähnt werden.
>
> Ein bisschen undurchsichtig, finde ich.

Du darfst in dieser Beziehung NIE nach den Tabellen gehen. Egal ob SPI 
oder Timer oder ADC oder UART oder ...


Die Tabellen sind immer nur die logische Zusammenfassung von 
Configurationsbits. Aber das bedeutet nicht, dass sie auch im selben 
Register beheimatet sind.
Aus den Tabellen entnimmst du nur die Information, welche Bits du 
brauchst und dann gehst du alle Register durch, bis du dasjenige hast, 
in welchem das Bit beheimatet ist. Wenn du es in einem Register nicht 
findest, dann ist es eben in einem anderen. Aber so viele 
Konfigurationsregister gibt es ja bei einer Komponente nicht. Und die 
sind im Datenblatt dann ja auch beim Modul alle aufgeführt.


> Die korrekte Schreibweise lese ich noch im Tut.

Das schreibt sich eigentlich ganz einfach

#define MOSI  PB5
#define SCK   PB7
#define SS    PB4

also einfach die Pinnummern aus dem Datenblatt. Dann passt es auch mit

   SPI = (1<<MOSI)|(1<<SCK)|(1<<SS);

[SPIAnfänger (Gast)]

Karl Heinz Buchegger schrieb:
> Das schreibt sich eigentlich ganz einfach
>
> #define MOSI  PB5
> #define SCK   PB7
> #define SS    PB4

Ah, klasse, danke für den Tipp!
Meiner Schreibweise nach hätte es dann wohl so aussehen müssen?

SPI = MOSI + SCK + SS;

Spricht etwas gegen diese Schreibweise?

Karl Heinz Buchegger schrieb:
> Die Tabellen sind immer nur die logische Zusammenfassung von
> Configurationsbits. Aber das bedeutet nicht, dass sie auch im selben
> Register beheimatet sind.

Das wusste ich wirklich nicht. Habe ich bis jetzt noch nie so erlebt 
(bei Timer & ADC, UART habe ich noch nicht ausprobiert).

Werde mir das aber merken, gibts wieder eine Frage weniger im Forum :)

[Karl H. (kbuchegg)]

SPIAnfänger schrieb:
> Karl Heinz Buchegger schrieb:
>> Das schreibt sich eigentlich ganz einfach
>>
>> #define MOSI  PB5
>> #define SCK   PB7
>> #define SS    PB4
>
> Ah, klasse, danke für den Tipp!
> Meiner Schreibweise nach hätte es dann wohl so aussehen müssen?
>
> SPI = MOSI + SCK + SS;
>
> Spricht etwas gegen diese Schreibweise?

Im Prinzip nicht.
Es scheint so, als ob das die auf einem MSP gebräuchlichere Schreibweise 
ist, während die andere die eher auf AVR gebräuchlichere Schreibweise 
ist.

Wo du jetzt genau den Teil 1<< hinschiebst bleibt sich ja egal

#define XYZ   PB5
   Register |= ( 1 << XYZ )

und

#define XYZ  (1<<PB5)
   Register |= XYZ

laufen ja aufs gleiche hinaus.

Nur eines würde ich nicht tun

#define XYZ   0x20

denn: warum willst du dir selber ausrechnen, welches Bitmuster du 
brauchst um den Pin 5 auf 1 zu setzen. Lass das doch den Compiler 
machen. Der macht dabei keine Fehler :-)

[SPIAnfänger (Gast)]

Karl Heinz Buchegger schrieb:
> Im Prinzip nicht.
> Es scheint so, als ob das die auf einem MSP gebräuchlichere Schreibweise
> ist, während die andere die eher auf AVR gebräuchlichere Schreibweise
> ist.

Naja, gebräuchlich ist, was einem am leichtesten fällt, oder :)?

Karl Heinz Buchegger schrieb:
> Nur eines würde ich nicht tun
>
> #define XYZ   0x20
>
> denn: warum willst du dir selber ausrechnen, welches Bitmuster du
> brauchst um den Pin 5 auf 1 zu setzen. Lass das doch den Compiler
> machen. Der macht dabei keine Fehler :-)

Ja, da hast du natürlich recht. Bin mich irgendwie gewöhnt immer mit 
Ausmaskierungen zu arbeiten, weiss gar nicht recht woher das kommt.