Sehr seltsames Verhalten von mega1284p + 74hc595 beim Sprung in Funktion

[Nils S. (kruemeltee)]

Ich habe an einem Mega1284p 2 Shift Register (74hc595) angeschlossen und 
möchte die per HW-SPI ansprechen.

Die Funktion für die SRs habe ich aus einem anderen Projektordner von 
mir geholt, wo sie, in etwas anderer Form funktioniert. Bin das 
Datenblatt nochmal durchgegangen, kann aber keinen Fehler finden.

An PB1 ist RCK ("Enable")
PB5 Mosi ist SER
PB7 SCK ist SCK
Das Zweite Shift Register hängt mit SER auf QH' vom Ersten.

Ich habe mir mitm Oszi SCK und SER angesehen, wenn ich in einer Schleife 
die Funktion "srw" aufrufe, da tut sich gar nichts an den Pins.

Das sehr seltsame Verhalten jetzt ist:
1. Ich setzte Test LED auf AUS
2. Aufruf von srw()
3. Toggle LED (Led ist jetzt an)
4. Aufruf von srw()
5. Toggle LED (bis dahin kommt er nicht mehr, LED bleibt an, sollte aus)

Nun hab ich das Toggle direkt vor SPDR = data in der Funktion srw 
gesetzt und habe folgendes:

1. Setze Test auf AUS
2. srw() (LED jetzt an)
3. srw() LED geht nicht mehr an

Er bleibt also wohl beim Aufruf von srw hängen.

Hier der Code:

void srw(unsigned char data) {
  SPDR = data;
  while(!(SPSR & (1<<SPIF)));

  PORTB |= (1<<PB1);
  _delay_us(10);
  PORTB &= ~(1<<PB1);
  return;
}

int main(void) {
  DDRB |= (1<<PB1)|(1<<PB5)|(1<<PB7);
  PORTB &= ~(1<<PB1);  // disable SR
  
  DDRB |= (1<<PB0);  // test led
  PORTB |= (1<<PB0);  // led off
  
  SPCR = (1<<SPE)|(1<<MSTR);
  SPSR = (1<<SPI2X);
  

  srw(0x0f);
  srw(0xff);
    
  
  for(;;) {

  }
}

Ich hoffe meine Beobachtungen hierzu waren richtig, ich kann mir keinen 
Reim drauf machen und bin kurz vorm Verzweifeln...

[Christian (Gast)]

Da du das SPDR Register nicht ausliesst, wird das SPIF-Flag nicht von 
der Hardware zurückgesetzt. Wenn du die Funktion srw etwas änderst 
könnte es klappen:

unsigned char srw(unsigned char data) {
  SPDR = data;
  while(!(SPSR & (1<<SPIF)));

  PORTB |= (1<<PB1);
  _delay_us(10);
  PORTB &= ~(1<<PB1);
  return SPDR;
}

Ich hoffe das hilft. Ich habs nicht selbst getestet.

Mfg Christian

[Nils S. (kruemeltee)]

Ne, das wars nicht. Aber dabei fiel mir gleich eben dieser Fehler in 
mehreren anderen Programmen auf, gleich mal ändern :P

Der hängt ja noch bevor er das zweite mal in SPDR schreibt, zwischen dem 
"void srw(unsigned char data) {" und "SPDR = data" versuche ich die LED 
zu togglen, tut sich nichts...

[MWS (Gast)]

Lies Dir mal das DB durch, Du setzt zwar Master, kontrollierst den 
SS-Pin aber nicht.

[Nils S. (kruemeltee)]

>kontrollierst den SS-Pin aber nicht.
Den verwende ich doch aber gar nicht, ich habe PB1 als "SS" für die 
Shift Register.

Hab' jetzt noch PB4, was SS ist zur Sicherheit auf Ausgang gesetzt, 
immer noch der selbe Fehler.

[MWS (Gast)]

Nils S. schrieb:
> Den verwende ich doch aber gar nicht, ich habe PB1 als "SS" für die
> Shift Register.

Dann hast Du Dir das DB nicht angesehen, SS ist Teil der HW-SPI, Zitat:
"If SS is configured as an input, it must be held high to ensure Master 
SPI operation."

[Nils S. (kruemeltee)]

>"If SS is configured as an input, it must be held high to ensure Master
>SPI operation."
SS ist bei mir ja mittlerweile auch output.

Mit Pullup Int+Ext hab ichs an SS probiert, immer noch der selbe Fehler. 
Habs jetzt sogar aufm Steckbrett mit komplett leerem C-Projekt nur mit 
Code für die SRs probiert...

Die Shift Register funktionieren 100%ig, wenn ich von Hand Takt und 
Daten aufm Steckbrett anlege, gehts.

[MWS (Gast)]

Du hast den Code wie Christian geschrieben hat geändert ?
Ich hätte an Deiner Stelle probeweise die SPI-ISR eingebaut, denn dann 
wär' ich mir um das gelöschte SPIF sicher, außerdem böte so 'ne ISR die 
Möglichkeit zum Testen 'ne LED zu setzen um damit der Lösung näher zu 
kommen.

Und nochmal den tatsächlich verwendeten Code posten, vielleicht sieht 
dann jemand noch was.

[Peter D. (peda)]

Nils S. schrieb:
> SS ist bei mir ja mittlerweile auch output.

Aber nicht irgendwann, sondern vor dem Setzen als SPI-Master!


Peter