Arduino 841 ISR SPI

[Herman Uh. (Gast)]

void spi_slave_init(void)
{
  SPI_DDR = (1<<SPI_MISO);
  SPCR = (1<<SPE) | (1<<SPIE);  // Enable SPI | Interrupt aktivieren
  SPDR = 0;
}

ISR(SPI_STC_vect)
{
  data = spi_slave_receive();
}

Hallo, ich möchte einen Attiny841 als SPI Slave nutzen, der reagieren 
soll, sobald sich etwas auf dem SPI Bus tut. Jedoch finde ich im 
Datenblatt kein Register, welches für den SPI Interrupt zu nutzen ist.

"21 0x0014 SPI SPI Serial Transfer Complete" ist was anderes.

[Karl H. (kbuchegg)]

Herman Uh. schrieb:

> Hallo, ich möchte einen Attiny841 als SPI Slave nutzen, der reagieren
> soll, sobald sich etwas auf dem SPI Bus tut.


Wenn du mit 'sobald sich etwas tut' meinst 'sobald der Master mit seinen 
Clock Pulsen anfängt', dann gibt es da auch nichts dazu.

Das einzige was es gibt, ist der Interrupt, so wie er in deinem 
Codeausschnitt ohnehin schon drinn ist, der feuert, wenn ein Transfer 
abgeschlossen ist.

[Herman Uh. (Gast)]

Ja das Problem ist, den Interrupt "SPI_STC_vect" gibt es beim Attiny841 
scheinbar nicht.

[Arduinoquäler (Gast)]

Auszug aus dem Manual Seite 158:
-----------------------------------------------
SPSR – SPI Status Register

Bit 7 – SPIF: SPI Interrupt Flag

This bit is set when a serial transfer is complete. An interrupt is 
generated if SPIE in SPCR is set and global interrupts are
enabled. If SS is an input and is driven low when the SPI is in Master 
mode, this will also set the SPIF Flag. SPIF is
cleared by hardware when executing the corresponding interrupt handling 
vector. Alternatively, the SPIF bit is cleared by
first reading the SPI Status Register with SPIF set, then accessing the 
SPI Data Register (SPDR).
-----------------------------------------------

"This bit is set when a serial transfer is complete"

gilt für Senden und Empfangen ("transfer")

[Karl H. (kbuchegg)]

Normalerweise ist es bei den AVR so, dass nur weil man eine der 
Alternativfunktionen für einen Pin aktiviert, die Basisfunktionalitäten 
deswegen nicht abgeschaltet werden.

Abhängig welchen SCK Pin du benutzt, könntest du mal probieren, ob der 
ganz normale PCINT3 bzw. PCINT4 Pin Change Interrupt (natürlich 
flankengetrieben) da noch durchkommt.

[Karl H. (kbuchegg)]

Herman Uh. schrieb:
> Ja das Problem ist, den Interrupt "SPI_STC_vect" gibt es beim Attiny841
> scheinbar nicht.

? Warum soll es den nicht geben

Die Bits in den Konfigurationsregistern gibt es ja auch.
Der heisst vielleicht beim Tiny841 nur ein bischen anders.

[Karl H. (kbuchegg)]

Herman Uh. schrieb:

> "21 0x0014 SPI SPI Serial Transfer Complete" ist was anderes.

Nö.
Das ist schon der richtige.

[Herman Uh. (Gast)]

ISR(SPI)
{
}

geht als Interrupt Vector nicht. avr/interrupt.h habe ich mit 
eingebunden.

Warning  1  'SPI' appears to be a misspelled signal handler [enabled by 
default]

[Karl H. (kbuchegg)]

Sieh dir mal die Dependency Liste in deinem Projekt an.
Dort müsste ein File namens iotn841.h aufgeführt sein (Oder so ähnlich. 
Mein Studio kennt den Tiny noch nicht, drum kann ich nichts genaueres 
sagen).

Machs auf und such nach "_vect". Irgendwo da drinn ist der korrekte Name 
für den Interrupt Handler, wie er im gcc heisst.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:
> Sieh dir mal die Dependency Liste in deinem Projekt an.
> Dort müsste ein File namens iotn841.h aufgeführt sein (Oder so ähnlich.
> Mein Studio kennt den Tiny noch nicht, drum kann ich nichts genaueres
> sagen).

Kann sein, dass das Header File auch anders heisst. Sein Name fängt 
normalerweise jedenfalls mit "io" an.

Noch was:
Damit die Dependency List vollständig ist, musst du das Projekt 
mindestens einmal compiliert haben. D.h. alles was Fehler wirft: 
gnadenlos wegkommentieren. Es geht nur darum, dass der Compiler 
durchkommt. Dann stimmt auch die Dependency List

[Karl H. (kbuchegg)]

Ach du Mist.
Du hast in deiner Überschrift ja das Stichwort 'Arduino' benutzt.
Na die IDE kann das nicht. Die ist dafür zu simpel gestrickt.

Da bleibt dir wohl nichts anderes übrig, als im Verzeichnis mit den 
Include Files zu suchen, welches für den Tiny 841 passt. Ausser jemand 
mit eine neuereren Atmel Studio als meines, erbarmt sich und sucht den 
Namen raus.

[Konrad S. (maybee)]

iotn841.h ist schon korrekt.

[Kaj G. (Firma: RUB) (bloody)]

Herman Uh. schrieb:
> ISR(SPI)
> {
> }
>
> geht als Interrupt Vector nicht. avr/interrupt.h habe ich mit
> eingebunden.
>
> Warning  1  'SPI' appears to be a misspelled signal handler [enabled by
> default]

Probiers mal mit:

#include <avr/interrupt.h>

ISR(SPI_vect)
{
}

[Herman Uh. (Gast)]

Karl Heinz schrieb:
> Ach du Mist.
> Du hast in deiner Überschrift ja das Stichwort 'Arduino' benutzt.
> Na die IDE kann das nicht. Die ist dafür zu simpel gestrickt.
Ähm, gute Frage wieso da Arduino steht, dass sollte Attiny heißen.

SPI_vect ist in den h-Files vorhanden gewesen

[Herman Uh. (Gast)]

So, hab nun ein bisschen rumgetestet, es wird zumindest schon etwas 
übermittelt. Nur noch nicht ganz so wie gewollt.

Zur Verdrahtung, die beiden Geräte sind wie folgt miteinander verbunden

1(M)  -  2(S)
SCK   -   SCK
MOSI  -  MOSI
SS    -    SS

Master

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

#define SPI_DDR    DDRA
#define SPI_PORT  PORTA
#define SPI_MOSI  PORTA6
#define SPI_MISO  PORTA5
#define SPI_SS    PORTA7
#define SPI_SCK    PORTA4

void spi_master_init(void)
{
  SPI_DDR |= (1<<SPI_SS) | (1<<SPI_MOSI) | (1<<SPI_SCK); //MOSI und SCK als Ausgang, Rest ist Eingang
  SPCR = (1<<SPE)|(1<<MSTR)|(1<<SPR0);    //Aktivierung des SPI, Master, Taktrate fck/16
}

void spi_master_transmit(uint8_t r, uint8_t g, uint8_t b)
{
  SPI_PORT &= ~(1<<SPI_SS);
  SPDR = r;                //starte Transmission
  while(!(SPSR & (1<<SPIF)));        //warte, bis Transmission abgeschlossen ist
  SPDR = g;                //starte Transmission
  while(!(SPSR & (1<<SPIF)));        //warte, bis Transmission abgeschlossen ist
  SPDR = b;                //starte Transmission
  while(!(SPSR & (1<<SPIF)));        //warte, bis Transmission abgeschlossen ist
  SPI_PORT |= (1<<SPI_SS);
}

int main(void)
{
  spi_master_init();
  sei();
  spi_master_transmit(0,0,0);

  while(1)
  {

    _delay_ms(1000);
    spi_master_transmit(255, 0, 0); // rot
    _delay_ms(1000);
    spi_master_transmit(0, 255, 0); // gruen
    _delay_ms(1000);
    spi_master_transmit(0, 0, 255); // blau
  }
}

Slave

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

#define SPI_DDR    DDRA
#define SPI_PORT  PORTA
#define SPI_MOSI  PORTA6
#define SPI_MISO  PORTA5
#define SPI_SS    PORTA7
#define SPI_SCK    PORTA4

volatile uint8_t data0 = 0xff;
volatile uint8_t data1 = 0xff;
volatile uint8_t data2 = 0xff;
volatile uint8_t data_out = 0;

void spi_slave_init(void)
{
  SPI_DDR |= (1<<SPI_MISO);
  SPCR = (1<<SPE) |(1<<SPIE);
}

uint8_t spi_slave_receive(void)
{
  while(!(SPSR & (1<<SPIF)));
  return SPDR;
}

ISR(SPI_vect)
{
  data2 = data1;
  data1 = data0;
  data0 = 255-spi_slave_receive();
  
  VAL_RED = data0;
  VAL_GREEN = data1;
  VAL_BLUE = data2;
}

int main(void)
{
spi_slave_init();
sei();
  while(1)
  {
  }
}

Im Master möchte ich mit 3 gesendeten 0en die Register data0 - data2 
zurücksetzen. Danach sende ich im Sekundentakt die Werte für eine RGB 
Led. Es soll einzeln rot, grün und blau blinken.

Jedoch geschieht das nicht. Es kommt dauerhaft zur Farbvermischung. Habe 
ich was vergessen?

[Themaverwirrter (Gast)]

Herman Uh. schrieb:
> Zur Verdrahtung, die beiden Geräte sind wie folgt miteinander
> verbunden1(M)  -  2(S)
> SCK   -   SCK
> MOSI  -  MOSI
> SS    -    SS

Auf den ersten Blick fällt auf dass MOSI an MISO muss.

[Themaverwirrter (Gast)]

Themaverwirrter schrieb:
> Auf den ersten Blick fällt auf dass MOSI an MISO muss.

Kann sein dass ich da etwas verwirrt war ...