Code-Rätsel: SPI in Bascom und C mit Atmega8

[Samuel B. (ui999)]

Hallo zusammen,

ich steuere mit meinem Atmega8 einen Trinamic TMC429 über SPI an. Alles 
funktioniert wunderbar mit meinem kleinen Bascom-Programm:

$regfile = "M8def.dat"
$crystal = 16000000

Config Spi = Hard , Interrupt = Off , Data Order = Msb , Master = Yes , Polarity = High , Phase = 1 , Clockrate = 16 , Noss = 1

Config Portb.2 = Output   'SS as Output

Dim Re1 As Byte
Dim Re2 As Byte
Dim Re3 As Byte
Dim Re4 As Byte

Spiinit

Portb.2 = 0
Re1 = Spimove(126)
Re2 = Spimove(1)
Re3 = Spimove(7)
Re4 = Spimove(33)
Portb.2 = 1

Ich möchte nun das Ganze nach C übersetzen. Mein C-Code sieht so aus:

#include <avr/io.h>   

unsigned char spi_tranceiver (unsigned char data)
{
    SPDR = data;
    while(!(SPSR)&(1<<SPIF));
    return(SPDR);
}


int main (void)
{            
  DDRB |= (1<<DDB2)|(1<<DDB3)|(1<<DDB5);    // SS, MOSI and SCK as outputs
  DDRB &= ~(1<<DDB4);                     // MISO as input

  SPCR |= (1<<SPE)|(1<<MSTR)|(1<<SPR0)|(1<<CPOL)|(1<<CPHA);

  PORTB &= ~(1<<PB2);
  spi_tranceiver(126);
  spi_tranceiver(1);
  spi_tranceiver(7);
  spi_tranceiver(33);
  PORTB = (1<<PB2); 
}

In C scheint die SPI-Übertragung nicht zu funktionieren, da sich die 
angeschlossenen Schrittmotoren nicht rühren. Vielleicht kann mir jemand 
von euch auf die Sprünge helfen. Wo liegt der entscheidende Unterschied 
zum Bascom-Programm?

[Easylife (Gast)]

PORTB = (1<<PB2);

sollte eher

PORTB |= (1<<PB2);

heissen. Daran wird es aber vermutlich nicht liegen.
Hast du ein Speicher-Oszilloskop zur Verfügung?
Dann würde ich an deiner Stelle damit mal die Kommunikation 
(SS/MOSI/SCK) beider Programme mitschneiden und vergleichen.

[Rudolph (Gast)]

Samuel B. schrieb:

> spi_tranceiver(33);
>   PORTB = (1<<PB2);
> }

Bist Du sicher, dass an dieser Stelle das Programm aufhören soll?
Damit geht der Controller alle paar Dutzend µs durch den Reset.

[Rudolph (Gast)]

Rudolph schrieb:
> alle paar Dutzend µs

Hmm, der SPI läuft mit 8 MHz? Dann wären das so 4µs für die vier 
Transfers, also so alle <5µs ein Reset.

[Peter II (Gast)]

Rudolph schrieb:
> Bist Du sicher, dass an dieser Stelle das Programm aufhören soll?
> Damit geht der Controller alle paar Dutzend µs durch den Reset.

warum sollte er in den Reset gehen?

hinter der main baut der linker eine Endlosschleife ein, es passiert als 
nichts danach.

[Paul B. (paul_baumann)]

Dann laß doch das C-Zeug sein, wenn es mit Bascom läuft.
Man muß doch kein Masochist sein.

MfG Paul

[Frank M. (ukw) (Moderator)]

Paul Baumann schrieb:
> Dann laß doch das C-Zeug sein, wenn es mit Bascom läuft.

.... sagte der Strauß und steckte den Kopf tief in den Sand.

;-)

[Samuel B. (ui999)]

Vielen Dank Leute für die Antworten.

Ich habe gerade den Fehler selbst gefunden, eine Klammer in

while(!(SPSR)&(1<<SPIF));

war verrutscht. Es muss

while(!(SPSR&(1<<SPIF)));

lauten.

[Micael (Gast)]

C is nix für Leute die mit Basic rumspielen.

[Karl H. (kbuchegg)]

Micael schrieb:
> C is nix für Leute die mit Basic rumspielen.

C ist vor allen Dingen nix für Leute, die meinen man müsse alle 
Möglichkeiten ausnutzen, die einem die dichte Schreibweise von C bietet.

Mit ein paar Leerzeichen, auch wenn sie syntaktisch nicht notwendig 
sind, wird so mancher Fehler auch schon aus der Ferne sichtbar.

So geschrieben

   while( !(SPSR) & (1<<SPIF) )
     ;

wär wahrscheinlich früher aufgefallen, dass das ! den falschen Bezug 
hat.

Aber: irgendwann lernts jeder, dass Schreibweisen einen Einfluss auf die 
Fehlerhäufigkeit haben.
Genauso wie es die meisten früher oder später lernen, dass ein 'return' 
kein Funktionsaufruf ist, und daher die Klammerung des Rückgabewertes 
unnötig ist

   return(i);

   return i;

Zweiteres ist sogar trotz weniger Tippaufwand dann sogar noch einfacher 
zu lesen.
Oder schreibst du auch

   i = (j);

und klammerst die rechte Seite einer Zuweisung auf jeden Fall immer ein?
Nichts anderes beim return. Dei Syntax ist

   return Expression

und die Expression ist genau dieselbe Expression, die überall in C an 
den verschiedensten Stellen vorkommt und dort nirgends generell 
eingeklammet wird. Nur beim return ist irgendwie alles anders, wohl weil 
K&R in ihrem Buch das so vorgemacht haben.