ATmega 328P - SPI Interrupt Flag setzt sich nicht zurück

[Viktor B. (coldlogic)]

Hi Leute,
Ich hab hier ein Problem, wo ich langsam nicht mehr weiter weiß. Mein 
Ziel ist es, mit einem Drehencoder (über Interrupt angebunden, in 
sämtlichen anderen Projekten benutzt => zu 99,8% keine Fehlerquelle) 
8-Bit-Zahlen in einem Array zu speichern und das Array auf Knopfdruck 
per SPI auszugeben. Das Problem dabei: Das Programm bleibt da stecken, 
wo die Daten eigentlich gesendet werden. Interruptangebundene Version 
des Programms hatte den Interrupt genau einmal aufgerufen, und zwar zum 
Zeitpunkt des Hochfahrens - aber auch nicht mehr. Die Polling-Variante 
(beigelegt) steckt in der while(!SPCR&(1<<SPIF))-Schleife fest.
Also der Standardfehler mit dem CS-Pin ist es nicht. Abgesehen davon 
wüsste ich echt nicht, was da noch so falsch sein soll. Das Datenblatt 
dazu hab ich schon quasi als Desktophintergrund, werde aber auch daraus 
nicht schlauer.
Hat irgendjemand einen Ansatz?


-upd.: Ein paar Klammern hinzugefügt. Kein Effekt sichtbar
MfG, Viktor

/*
 * mainof_SPI_Tx.cpp
 *
 *  Created on: 08.03.2017
 *      Author: Victor
 */


#define SR_DATA PD1
#define SR_CLOCK PD0
#define SR_UPDATE PD2
#define SR_PORT PORTD

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

volatile unsigned char code, shift, argl = 0, argp = 0, byte = 0;
volatile char argv[42];
volatile bool tx_flag, sync_button = 0;

void out(unsigned char x){              // Function which shows a byte in binary on a shift register w. 8 LEDs
  unsigned int last = x;              //get actual parameters
  unsigned char i1 = 128;

  for(char z1=0; z1<8; z1++){            // 8 times decision if 1 or 0 should be written
    SR_PORT &= ~(1 << SR_CLOCK);                // PD0 as clock pin held down
    if(last>=i1){
      SR_PORT |= (1 << SR_DATA);        // PD1 as data pin
      last = last-i1;
    }
    else{
      SR_PORT &= ~(1 << SR_DATA);
    }
    SR_PORT |= (1 << SR_CLOCK);          // Clock pin pulled up to save the data pin position
    i1=i1/2;
  }
  SR_PORT &= ~(1 << SR_UPDATE);          // Pulse the update pin to get the actual value to LEDs
  SR_PORT |= (1 << SR_UPDATE);
}

ISR(PCINT2_vect){                  // ISR of port D; rotary encoder interface
  if(PIND&(1<<PIND5)){              // ISR called by pushing button?
    shift = 3;                  // Set flag
  }
  else{
    if(PIND&(1<<PIND6)){
      if(PIND&(1<<PIND7)){          // ISR called by change in pin 6?
        shift = 1;              // Encoder turned right if pin 7 already high,
      }                    // turned left if pin 7 still low
      else{
        shift = 2;              // Ze end
      }
    }
  }
}

int main(){
  /*Setup*/
  cli();                      // No interrupts
  DDRD  = 0x0F;                         // Port D i/o: 7--iiiioooo--0
  PORTD = 0xF0;                  // Pull-ups D: 7--11110000--0
  DDRB  = (1<<PB2)||(1<<PB3)||(1<<PB5);           // Port B i/o: 7--iioiooii--0
  PORTB = (1<<PB2)||(1<<PB3)||(1<<PB5);      // Pull-ups B: 7--11010011--0
  PCICR = 0b00000100;                // Enabling PCI on port D
  PCMSK2 = 0x60;                  // Enable PCI on pin 6 & 5 of port D
  SPCR = (1<<SPE)||(1<<MSTR)||(1<<DORD)||(1<<SPR0);//SPI w/o interrupots on; Master; LSB first; Fck/16
  sei();                      // Y u no interrupts
  /*Program as it is*/
  while (1){
    if (shift != 0){              // Encoder flag routine
      switch (shift) {
      default:
        break;
      case (1):
        shift = 0;
        byte++;
        break;
      case (2):
        shift = 0;
        byte--;
        break;
      case (3):
        shift = 0;
        argv[argl] = byte;
        argl++;
        byte = 0;
        break;
       }
    }
    if(!byte){
      out(code);
    }else{
      out(byte);
    }

    if(PIND&(1<<PIND4)){            // if pin D4 is set (standard pulse generator code)
      if (!sync_button){            // and its the first cycle being so
        tx_flag = 1;              // set transmission start
      }
      sync_button = 1;
    }else{
      sync_button = 0;
    }
    if(tx_flag){
      code = 0x01;
      PORTD |= (1 << PD3);            // Draw pin PD3 high
      while (argp < argl) {
        code = 0x02;
        SPDR = argv[argp];
        while (!(SPSR & (1<<SPIF))){
          code = 0x04;
          out(code);
        }
        argp++;
      }
      argp = 0;
      argl = 0;
      PORTD &= ~(1 << PD3);            // Draw pin PD3 low
      tx_flag = 0;
    }
  }
return 0;
}

[Tom (Gast)]

Wofür ist PD3 gedacht?

[Viktor B. (coldlogic)]

PD3 ist der CS-pin. Er sagt dem Slave, er sei angesprochen

[Tom (Gast)]

aber nur bei Low fühlt er sich angesprochen, oder?

[Viktor B. (coldlogic)]

Der Slave - Chip ist ein wenig komisch, was das angeht. Der wird 
angesprochen, wenn CS auf high ist. Das Problem liegt ja noch davor - 
wie eine Oszilloskopmessung gezeigt hatte, rührt sich weder auf SDA noch 
auf SCL was, und die SPI-Einheit des ATmega verweigert einfach mal den 
Dienst.

[Arno (Gast)]

Du schreibst die Daten per Software bitweise auf den Portpin, erwartest 
aber, dass die SPI-Hardware erkennt, wenn ein Byte fertig ist? Das kann 
nicht funktionieren.

Entweder du musst dir selbst ein SPIF-Flag generieren oder du musst, 
wenn du das Flag im SPSR nutzen willst, auch den Rest der SPI-Hardware 
nutzen. Also deine Daten senden, indem du sie nach SPDR schreibst. Dann 
kümmert sich die Hardware für dich darum, SR_DATA und SR_CLOCK richtig 
zu setzen. Natürlich bist du dann auf die Pins festgelegt, an die die 
SPI-Hardware angeschlossen ist.

Ein Beispiel dafür ist im Datenblatt enthalten, danach solltest du noch 
einmal genau schauen. Außerdem kannst du auch mal ein paar Beiträge 
weiter nach dem Thema SPI auf ATMega8 funktioniert nicht (oder so 
ähnlich) schauen, da wird das lang und breit diskutiert.

MfG, Arno

[Jim M. (turboj)]

Viktor B. schrieb:
> rührt sich weder auf SDA noch
> auf SCL was, und die SPI-Einheit des ATmega verweigert einfach mal den
> Dienst.

SDA und SCL wären aber I²C und nicht SPI. Du sprichts in Rätseln.

[Stefan E. (sternst)]

Viktor B. schrieb:
> Also der Standardfehler mit dem CS-Pin ist es nicht.

Doch, ist es vermutlich, denn ...

  DDRB  = (1<<PB2)||(1<<PB3)||(1<<PB5);

du setzt nur PB0 auf Ausgang. ;-)

PS: Ganz davon zu schweigen, dass natürlich auch die ganze 
SPI-Initialisierung auf Grund des gleichen Problems defekt ist.

[Viktor B. (coldlogic)]

Arno schrieb:
> Du schreibst die Daten per Software bitweise auf den Portpin

Sorry für die Verwechselung. Die Funktion out(); da oben sendet ein Byte 
an ein angeschlossenes Schieberegister; mit dem SPI hat es gar nichts zu 
tun.

Jim M. schrieb:
> SDA und SCL wären aber I²C und nicht SPI.

Jep, das sollten eig SCK und MOSI sein. Hatte die I^2C-Namen noch im 
Kopf

[Viktor B. (coldlogic)]

Stefan E. schrieb:
> du setzt nur PB0 auf Ausgang.

Setze ich damit nicht PB2 (SS), PB3 (MOSI) und PB5 (SCK) als Ausgang?

[Stefan E. (sternst)]

Viktor B. schrieb:
> Stefan E. schrieb:
>> du setzt nur PB0 auf Ausgang.
>
> Setze ich damit nicht PB2 (SS), PB3 (MOSI) und PB5 (SCK) als Ausgang?

Nein.

|| vs |

[Arno (Gast)]

Viktor B. schrieb:
> Arno schrieb:
>> Du schreibst die Daten per Software bitweise auf den Portpin
>
> Sorry für die Verwechselung. Die Funktion out(); da oben sendet ein Byte
> an ein angeschlossenes Schieberegister; mit dem SPI hat es gar nichts zu
> tun.

Ah, du hast Recht, und beim zweiten Versuch findet die Suchfunktion auch 
die Stelle in deinem Code, an der du SPDR setzt. Muss ich mich vorhin 
vertippt haben...

Viktor B. schrieb:
> Stefan E. schrieb:
>> du setzt nur PB0 auf Ausgang.
>
> Setze ich damit nicht PB2 (SS), PB3 (MOSI) und PB5 (SCK) als Ausgang?

...aber Stefan hat Recht. || ist logisches Oder, | ist bitweises Oder.

MfG, Arno

[Viktor B. (coldlogic)]

Wusste ich's doch, das der Fehler irgendwas unscheinbares sein wird. Auf 
jeden Fall vielen Dank, der µC scheint sich nicht mehr aufzuhängen! 
Später, wenn ich Zugriff aufs Oszi hab, werde ich weiter schauen. Kudos 
to you!