AVR ATMEGA32 - SPI in GCC - CODEPROBLEM, erbitte Hilfe

[Daniel B. (dont_care)]

Hallo und schönen Sonntag allerseits,

ich versuche eine SPI Kommunikation zwischen einem Atmel ATMEGA32L (als 
Master) und einem bzw später mehreren Slaves zu implementieren.


Leider tat sich lt. Oszilloskop nicht das was ich mir erhofft habe. Habe 
ich etwas vergessen, übersehen oder falschgemacht?


Anbei mein C - Code mit der Bitte um Hilfe, Anregung und 
Lösungsvorschläge.

#include <avr/io.h>
#include <util/delay.h>
#include <stdbool.h>
#include <avr/interrupt.h>

// Prototypen

void SPI_MasterInit(void);
void SPI_MasterTransmit( int cData);
unsigned char SPI_MasterReceive(void);




int main(void)
{
    // Interrupts erlauben
    sei();
    //TestPin wechselt Zustand bei erfolgreichem Funktionsdurchlauf
    DDRD |= (1<<DDD7);
    
    unsigned char test= 0x54;
    unsigned char back;
    unsigned char end = false;
   
    while(end!=true)
    {
        SPI_MasterTransmit(test);
        back = SPI_MasterReceive();
     
    }
    
    
    return 0;               /* never reached */
}



void SPI_MasterInit(void)

{

    /* Set MOSI and SCK output, all others input */
    DDRB |= (1<<DDB5)|(1<<DDB7);
    // SET MISO als Eingang
    //DDRB &= ~(1<<DDB6);
    // MOSI, MISO und Takt auf LOW
    //PORTB &= ~((1<<PB5)|(1<<PB6)|(1<<PB7));
    // slave select als Ausgang
    DDRD |= (1<<DDD4);
    // SS auf High
    PORTD |= (1<<PD4);
    
    /* SPI Kontrollregister */
    
    // SPI Interrupt enable , SPI aktivieren, Mastermode
    SPCR |= ((1<<SPE)|(1<<MSTR));
    
   //Taktbasis low, steigende flanke MO, fallende flanke MI, SPI Takt = /4 -> 1 MHz / 4 = 250 KHZ = 250.000 Hz
    //SPCR &= ~((1<<SPIE)|(1<<CPOL)|(1<<CPHA)|(1<<SPR0)|(1<<SPR1));
   
    /*SPI  Status Register*/
    
    //Kontrollflag setzen
    
    PORTD |= (1 <<PD7);
    
}

void SPI_MasterTransmit( int cData)

{
    //testflag löschen
    PORTD &= ~(1<<PD7);

    //Slaveselect auf Masse
    PORTD &= ~(1<<PD4);
    
    
    /* Start transmission */
    SPDR = cData;
    /* Wait for transmission complete */
    while(!(SPSR & (1<<SPIF)));
    
    PORTD |= (1<<PD4);
    
    //testflag setzen
    PORTD |= (1 <<PD7);
    
    
    
    
}

  unsigned char SPI_MasterReceive(void)
{
    //testflag löschen
    PORTD &= ~(1<<PD7);
    
    _delay_ms(10);
    //testflag setzen
    PORTD |= (1 <<PD7);
    //
    return SPDR;
    
    
}

Fuse high byte:
# 0xB9 = 1 0 1 1   1 0 0 1
Fuse low byte:
# 0x81 = 1 0 0 0   0 0 0 1

[Martin (Gast)]

Der Slave Select ist doch PB4 und nicht PD4 oder?

#include <avr/io.h>
#include <util/delay.h>
#include <stdbool.h>
#include <avr/interrupt.h>

// Prototypen

void SPI_MasterInit(void);
uint8_t SPI_MasterTransmit(uint8_t cData);


int main(void)
{    
    uint8_t test= 0x54;
    uint8_t back;
    unsigned char end = false;
   
    while(end!=true)
    {
        back = SPI_MasterTransmit(test);   
    }
    
    
    return 0;               /* never reached */
}



void SPI_MasterInit(void)
{
    // SS auf High
    PORTD |= (1<<PB4);

    /* Set MOSI and SCK output, all others input */
    DDRB |= (1<<DDB4)|(1<<DDB5)|(1<<DDB7);
    
    // SPI Interrupt enable , SPI aktivieren, Mastermode
    SPCR |= ((1<<SPE)|(1<<MSTR)); 
}

uint8_t SPI_MasterTransmit(uint8_t cData)
{
    //Slaveselect auf Masse
    PORTB &= ~(1<<PB4);
    
    
    /* Start transmission */
    SPDR = cData;
    /* Wait for transmission complete */
    while(!(SPSR & (1<<SPIF)));
    
    PORTB |= (1<<PB4);
    
    return SPDR;
}

[Martin (Gast)]

OOh da hatte sich noch ein Fehler eingeschlichen

void SPI_MasterInit(void)
{
    // SS auf High
    PORTB |= (1<<PB4);

    /* Set MOSI and SCK output, all others input */
    DDRB |= (1<<DDB4)|(1<<DDB5)|(1<<DDB7);
    
    // SPI Interrupt enable , SPI aktivieren, Mastermode
    SPCR |= ((1<<SPE)|(1<<MSTR)); 
}

[Nil (nilsnilss)]

Du solltest deine Init Methode schon aufrufen.

[Daniel B. (dont_care)]

Hallo und vielen Dank. Es funktioniert jetzt so wie es soll!