Atmega SPI will nicht

[Christoph H. (christoph_b)]

Hallo

Ich probiere mich gerade an der SPI Schnittstelle auf einem Atmega 2560 
für mein Projekt. Habe einen Quellcode vom Lehrer erhalten doch leider 
will die SPI Schnittstelle nicht so wie wir.

Vieleicht kann ja jemand einmal über den Code schauen.

/**************************************************************
Es soll alle halbe Sekunde im Wechsel 0 bzw. 1 gesendet werden.
Am korrespondierenden Slave soll zur Indikation jeweils die 
LEDs an bzw. aus gehen
Verdrahtung:  MISO(Master) --> MISO(Slave)
        MOSI(Master) --> MOSI(Slave)
        SCK(Master)  --> SCK(Slave)
        PB0(Master)   --> SS(Slave)
**************************************************************/

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/signal.h>

unsigned char status = 0;
volatile unsigned char count;

void timer1 (void);
void master_init (void);
void master_transmit (unsigned char data);

SIGNAL (SIG_SPI) {
  return;
}

SIGNAL (SIG_OVERFLOW1) {            //Senderoutine
  if (count == 1) {
    master_transmit ('1');
    count--;
    PORTL   = 0xFF;    
    return;
  }
  if (count == 0) {
    master_transmit ('0');
    PORTL   = 0x00;    
    count++;
  }
}

void timer1 (void) {
  TIMSK1 |= (1<<TOIE1);                   //Timer Overflow Interrupt enable
  TCNT1 = 0;                          //Rücksetzen des Timers
  TCCR1B = (1<<CS10) | (1<<CS11);      //8MHz/65536/64 = 1,91Hz --> 0,5s
}

void master_init (void) {
  DDRB = (1<<PB1) | (1<<PB2) | (1<<PB0);    // setze SCK,MOSI,PB0 (SS) als Ausgang
  PORTB = (1<<PB1) | (1<<PB0);        // SCK und PB0 high (ist mit SS am Slave verbunden)
  SPCR = (1<<SPE)|(1<<MSTR)|(1<<SPR0);  //Aktivierung des SPI, Master, Taktrate fck/16
  status = SPSR;                //Status löschen
}

void master_transmit (unsigned char data) {
  PORTB &= ~(1<<PB0);            //SS am Slave Low --> Beginn der Übertragung
  SPDR = data;                //Schreiben der Daten
  while (!(SPSR & (1<<SPIF)));
  PORTB |= (1<<PB0);              //SS High --> Ende der Übertragung
}

int main (void) {
  DDRL   = 0xFF;    
  master_init ();
  timer1 ();
  sei ();

  for (;;);
  return 0;
}

[Patrick B. (p51d)]

Christoph B. schrieb:
> will die SPI Schnittstelle nicht so wie wir.

Gute Fehlerbeschreibung... was will wie nicht, respektive passiert 
etwas...
Oszi vorhanden? Wenn ja, kannst du dir ja mal eine LED oder sowas in der 
TimerISR toggeln lassen (-> Timer OK). Ebenso das SPI

Ansonsten sieht es gut aus. Debugger schon versucht?

[Christoph H. (christoph_b)]

also Oszi und Logic Analyser sind vorhanden.

Der Timer läuft ( toggle eine LED). Trotzdem bekomme ich nur ein 5V 
Signal am Ausgang. Müsste da nicht ein "Schönes" Rechteck Signal 
anstehen?

Sende ja jede Sekunde eine 0 und 1.

[Martin (Gast)]

Muss MISO nicht auf MOSI gehen? (Serial Input/ Serial Output)

[Patrick B. (p51d)]

Ev. Triggerproblem?
SCK ist nur aktiv, wenn Daten gesendet werden.

Wofür ist das?

status = SPSR;                //Status löschen

Also bei mir läuft dies auf einem ATmega32 mit 4MHz:

void SPI_MasterInit(void){
  /* Set MOSI and SCK output, all others input */
  DDR_SPI = (1 << DD_MOSI) | (1 << DD_SCK) | (1 << DD_SS);
  /* Enable SPI, Master, set clock rate fck/2 */
  SPCR = (1 << SPE) | (1 << MSTR);
  SPSR = (1 << SPI2X);
}

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

Triggere mal auf SS (oder bleibt dieser immer auf High?).

Martin schrieb:
> Muss MISO nicht auf MOSI gehen? (Serial Input/ Serial Output)

Nö, passt schon. Master Out auf Master Out beim Slave. Die können gerade 
verbunden werden. Der Slave muss dann natürlich auf Slave eingestellt 
sein.

[Christoph H. (christoph_b)]

Hallo

Habe nun das SPI zwischen einem Atmega 2560 und einem Atmega 16 zum 
laufen bekommen. :-)

Was ich noch nicht hinbekomme ist das ich die Daten in einzelne Bytes 
aufteilen kann um sie so über den SPI bus zu senden.

[Yaro (Gast)]

Was war der Fehler?

Welche Daten möchtest du aufteilen?
Sowas lässt sich mit unions machen... Packst z.B. einen float rein und 
holst ein uint8_t array raus.

[Christoph H. (christoph_b)]

Es war irgendein komisches Problem mit dem AVR Studio 5. Habe zum Test 
nochmal Studio 4 installiert und siehe da SPI klappt.
Danach habe ich nochmal Version 5 installiert und auch da klappt nun 
SPI.

Ich kann ja max 255 verschiedene Werte übertragen. (8Bit) Würde jetzt 
gerne z.B. "PC4_1" für PortC LED1 On usw übertragen. Später muss ich 
über SPI die PWM des anderen Atmega steuern können.

Hast du zufällig ein Beispiel wie man das anstellt.

MFG Christoph

[Christoph H. (christoph_b)]

@ Yaro

Hast du zufällig ein Beispiel wie man das macht?

[Uwe (Gast)]

Denk dir nen Protokol aus z.B. :

1. Byte Adresse des Empfängers
2. Byte Befehl
3. Byte Länge der Daten bzw. Parameter
4. n Byte Daten
5. Checksum

Und zerlege alle Variablen in 8Bit und bau Sie danach wieder Zusammen.

[Uwe (Gast)]

einfach mal C Unions googlen

[Christoph H. (christoph_b)]

Hallo

Habe es jetzt hinbekommen das ich mehrer Bytes über SPI sende. ( 
verwende ein Array)

Das Problem ist das nicht genau weiß wie der Slave das auswertet.
Leider stimmt die Auswertung noch nicht.

Anbei auch ein Beispiel wie die Sende Daten ausschauen.

Quellcode.

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/signal.h>

volatile unsigned char data;
unsigned char status;
unsigned char data_IN[10];
unsigned int ADC_16;
i=0;

SIGNAL (SIG_SPI) {
  data_IN[i] = SPDR;
  i++;
  if (i=10)
  {
    ADC_16=data_IN[2]*256+data_IN[1];
    i=0;
  }
  
  
  if (ADC_16 == 'O','N')  PORTC = 0xFF;      //LEDs an
  if (ADC_16 == 'O','F')  PORTC = 0x00;      //LEDs aus
  
  if (ADC_16 == 'ON')  PORTC = 0xFF;      //LEDs an
  if (ADC_16 == 'OF')  PORTC = 0x00;      //LEDs aus
}



void slave_init (void) {
  DDRB |= _BV(PB6);              //MISO als Ausgang, der Rest als Eingang
  SPCR = _BV(SPE) | _BV(SPIE);        //Aktivierung des SPI + Interrupt
  status = SPSR;                //Status löschen
}

int main (void) {

  DDRC = 0xff;
  slave_init ();
  sei ();
  for (;;);
  
}