SPI funktioniert nicht

[Helge S. (freestila)]

Hallo zusammen,

ich versuche mich gerade an SPI. Dabei nutze ich einen Arduino Mega als 
Master und einen Atmega8 als Slave.

SS, SCK, MISO und MOSI sind angeschlossen. der Slave-code lautet

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/signal.h>
#ifndef F_CPU
#define F_CPU    16000000
#endif
#include <util/delay.h>

 
 //FUSE ATMEGA 8 16Mhz: hfsue C9, Lfuse FF
 
 volatile unsigned char data;
 unsigned char status;
   unsigned char receive(void){
     //warten
  //   SPDR = 1;
     while(!(SPSR & (1<<SPIF)));
     //im Sendepuffer steht jetzt empfangenes Byte
     return SPDR;
   }
   
   ISR (SIG_SPI) {
   //  PORTC ^= ( 1 << PC1 ); 
   if (status == 1){
     PORTC &=~(1<<PC1);
     status  = 0;
   } else {
     PORTC |=(1<<PC1);
     status  = 1;
   }       
     //data = SPDR;
     data = receive();
     if (data == '1'){  PORTC |= (1 << PC0) ;}    //LEDs an
   //  if (data == '0')  PORTC &= ~(1 << PC0) ;      //LEDs aus
   else {  PORTC &= ~(1 << PC0) ;}
   }
 

 void slave_init (void) {
  // DDRB |= _BV(PB4);              //MISO als Ausgang, der Rest als Eingang
  DDRB  |= ( 1 << PB4 );    
  // SPCR = _BV(SPE) | _BV(SPIE);        //Aktivierung des SPI + Interrupt
  //Einstellungen des SPI-Control Register

  SPCR = (1<<SPE)   // SPE=1 (Freigabe der SPI-Schnittstelle)
  |(0<<DORD) // DORD=0: Data Order MSB
  |(0<<MSTR)  //MSTR=1 (ATMega als Master definiert)
  |(0<<CPOL)   //SPI-Modus CPOL=0 und CPHA=1 (gesendete Bits werden bei fallender Flanke des SPI-Clocks SCK übernommen)
  |(0<<CPHA)   //SPI-Modus CPOL=0 und CPHA=1 (gesendete Bits werden bei fallender Flanke des SPI-Clocks SCK übernommen)
  //|(1<<SPR0);  //Übertragungsfrequenz der Daen wir auf fosc / 16 eingestellt
  |(1<<SPIE); // Interrupt aktivieren
   status = 1;                //Status löschen


//PD3 / Int1 als Interrupt für Slave select

 }
  

 


 int main (void) {
//return blink();
   DDRD = 0xff;
  

    DDRB |= ( 1 << PB0 );        // PB0 an PORTB als Ausgang setzen
    DDRC |= ( 1 << PC0 );        // PB0 an PORTB als Ausgang setzen
    DDRC |= ( 1 << PC1 );        // PB0 an PORTB als Ausgang setzen
     slave_init ();
     sei();
  //   
    
      PORTC |= (1 << PC1) ;
     PORTC |= (1 << PC0) ;
    // return blink();
  while( 1 ) {                // Endlosschleife
  PORTB ^= ( 1 << PB0 );  // Toggle PB0 z.B. angeschlossene LED
  _delay_ms(1000);        // Eine Sekunde +/-1/10000 Sekunde warten...
  }  
   return 0;
 }

Auf Arduino-Seite sende ich entweder eine 0 oder eine 1.

Leider klappt es nicht ganz. Ich muss merhmals eine 0 senden, dann erst 
geht pc0 aus. und erst ein oder zwei sendungen später wechselt PC1. 
Irgendwas stimmt da also nicht.
Auch das SS funktioniert nicht 100%. Ich hab SS direkt auf ground 
gesetzt, dann hat es am betsen funktioniert.

Hat einer eine Idee?

Der Arduino code ist:

//  50 (MISO), 51 (MOSI), 52 (SCK), and 53 (SS). 
//The full code is in library example file Quick_tour.ino
#include <Wire.h>
#include <LCD.h>
#include <LiquidCrystal.h>
#include <buttons.h>
#include <MENWIZ.h>
#include <EEPROM.h>
#include <SPI.h>

int  adc_key_val[5] ={30, 150, 360, 535, 760 };
//int button_codes[5]={MW_BTR,MW_BTU,MW_BTD,MW_BTL,MW_BTC};
int button_codes[5]={MW_BTR,MW_BTU,MW_BTD,MW_BTE,MW_BTC};
int NUM_KEYS = 5;
int adc_key_in;
int key=-1;
int oldkey=-1;

const int slaveSelectPin = 53;

menwiz tree;
// create lcd obj using LiquidCrystal lib
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

int  list,sp=110;
int  mem;
 
void setup(){
  _menu *r,*s1,*s2;
mem=tree.freeRam();
  Serial.begin(9600);    
    Serial.println("ASCII Table ~ Character Map"); 
  tree.begin(&lcd,16,2); //declare lcd object and screen size to menwiz lib

  r=tree.addMenu(MW_ROOT,NULL,F("Einstellungen"));
    s1=tree.addMenu(MW_SUBMENU,r, F("Node 1"));
      s2=tree.addMenu(MW_VAR,s1, F("Node3"));
        s2->addVar(MW_LIST,&list);
        s2->addItem(MW_LIST, F("Option1"));
        s2->addItem(MW_LIST, F("Option2"));
        s2->addItem(MW_LIST, F("Option3"));
      s2=tree.addMenu(MW_VAR,s1, F("Node4"));
        s2->addVar(MW_AUTO_INT,&sp,0,120,10);  
    s1=tree.addMenu(MW_VAR,r, F("Sende an"));
      s1->addVar(MW_ACTION,sendeAn);
      s1=tree.addMenu(MW_VAR,r, F("Sende aus"));
      s1->addVar(MW_ACTION,sendeAus);
       tree.addUsrNav(&getPressedButton);
       tree.addUsrScreen(&menuMainCallBack, 5000);
         tree.addSplash((char *) tree.sbuf, 5000);
       
       sprintf(tree.sbuf,"Free mem. :%d\n.Used mem  :%d",tree.freeRam(),mem-tree.freeRam()); 
        // set the slaveSelectPin as an output:
  pinMode (slaveSelectPin, OUTPUT);

  // initialize SPI:
  SPI.begin(); 
    SPI.setBitOrder(MSBFIRST); 
SPI.setDataMode(1);
SPI.setClockDivider(SPI_CLOCK_DIV16);
  }
  
  int sendValue(int value){
    
     // take the SS pin low to select the chip:
  digitalWrite(slaveSelectPin,LOW);
  SPI.transfer(value);
  // take the SS pin high to de-select the chip:
  digitalWrite(slaveSelectPin,HIGH); 
}

void menuMainCallBack(){
 sprintf(tree.sbuf,"Uptime (s): %ld\nFree mem  : %d\n\n",millis()/1000,(int)tree.freeRam());
  tree.drawUsrScreen(tree.sbuf);
  
  
}

void loop(){
  tree.draw(); 
  }

void sendeAn(){
  Serial.println("Action an");
  sendValue(1);
  }
void sendeAus(){
  Serial.println("Action aus");
  sendValue(0);
  }
  
int getPressedButton(){
  
  adc_key_in = analogRead(0);    // read the value from the sensor  
  key = get_key(adc_key_in);            // convert into key press
  
  if (key != oldkey)            // if keypress is detected
  {
    delay(50);    // wait for debounce time
    adc_key_in = analogRead(0);    // read the value from the sensor  
    key = get_key(adc_key_in);            // convert into key press
    if (key != oldkey)        
    {      
      oldkey = key;
      if (key >=0){
        Serial.println("Pressed Button ");
        Serial.println(key);
        Serial.println(": ");
        Serial.println(button_codes[key]); 
      return button_codes[key];
     }
     
    }
  }
  return MW_BTNULL;

  
  
}

  
  
// Convert ADC value to key number
int get_key(unsigned int input)
{
  int k;
    
  for (k = 0; k < NUM_KEYS; k++)
  {
    if (input < adc_key_val[k])
    {
           
    return k;
        }
  }
    
    if (k >= NUM_KEYS)
        k = -1;     // No valid key pressed
    
    return k;
}

[Stefan F. (sfrings)]

Benutze als Master oder Slave ein Gerät, dass sich bisher bewährt hat.

Du probierts gerade zwei neue Programme und hast daher nichteinmal eine 
Ahnung, welches der beiden fehlerhaft ist.

Hast Du einen Hardwarefehler ausschließen können? Wenn nicht, würde ich 
mich erstmal darum kümmern. Und dann als zweiten Schritt ein bekanntest 
funktionierendes Bauteil verwenden, z.B. einen einfachen A/D Wandler 
oder ein Schieberegister.