MSP430 SPI Daten verschicken, empfangen

[Benjamin H (Gast)]

Hallo liebes Forum,

ich beschäftige mich mit den MSP430. Als Chip benutze ich den G2553.

Als neues Kapitel, habe ich mir die Kommunikation via SPI vorgenommen.


Ich hab mich soweit durch den Family Guide - SPI Kapitel - und durch das 
Internet bzgl SPI durchgelesen.


Und stieß dann auf die Idee und den Code, die zwei 
Kommunikationsleitungen die der G2553 hat, miteinander zu verbinden, um 
so die LED anzusteuern.

Also habe ich die Pins: P1.1 mit P1.6, P1.2 mit P1.7 und P1.4 mit P1.5 
verbunden. (Siehe Bild)

Nun habe ich das im Programm so eingestellt, dass P1.6, P1.7 und P1.5 
vom Master angesteurt werden und die restlichen vom Slave.


Mein Vorhaben war, ein Zeichen - hier das A - per Master an den Slave zu 
schicken und wenn er es erhalten hat, die LED auszuschalten.

Zwar, gelingt es mir (wie auf dem zweiten Foto zu sehen), das Zeichen im 
TXBUF zu holen, aber gelangt es nicht in den RXBUF.

TESTWEISE, wie in den Kommentaren zu erkennen, habe ich in der ISR vom 
RX, vor der 'if' Anweisung die LED vorher angesteuert, ...

__interrupt void USCI0RX_ISR(void)
USCI0RX_ISR(void) 
{
  // P1OUT |= BIT0;     // Testweise, ob es bis hierhin kommt -> ja tut es
   
  if (UCA0RXBUF == 'A') 
  {

..., und es gelang mir erfolgreich die einzuschalten.
Nur, macht das Programm, beim Vergleich nicht weiter, bzw. geht nicht in 
die 'if' Anweisung rein.


Danke im voraus für jede Hilfe.

#include "msp430g2553.h"
#include <legacymsp430.h>



void main(void) {
  WDTCTL = WDTPW + WDTHOLD;                 // Stop watchdog timer
  
  DCOCTL = 0;                   // Select lowest DCOx and MODx settings<
  BCSCTL1 = CALBC1_1MHZ;        // Set DCO
  DCOCTL = CALDCO_1MHZ;
  
  P2DIR = 0xFF;                 // All P2.x outputs
  P2OUT &= 0x00;                // All P2.x reset
  
  P1DIR |= BIT0;
  P1OUT |= BIT0;
  
  //P1OUT &= ~BIT0;    // Testweise
  
  P1SEL = BIT1 + BIT2 + BIT4 + BIT5 + BIT6 + BIT7;
  P1SEL2 = BIT1 + BIT2 + BIT4 + BIT5 + BIT6 + BIT7;
  
//--------------Slave setup---------------------------------
  UCA0CTL1 = UCSWRST;                       // **Put state machine in reset**
  UCA0CTL0 |= UCCKPL + UCMSB + UCSYNC;      // 3-pin, 8-bit SPI slave
  UCA0CTL1 &= ~UCSWRST;                     // **Initialize USCI state machine**
  
  while(IFG2 & UCA0RXIFG);
  
  UC0IE |= UCA0RXIE;                          // Enable USCI0 RX interrupt
  
//--------------Slave setup done----------------------------------

  
//--------------Master setup--------------------------------------
  __delay_cycles(60000);
  __delay_cycles(60000);
  __delay_cycles(60000);
  __delay_cycles(60000);
  
  UCB0CTL0 |= UCCKPL + UCMSB + UCMST + UCSYNC;  // 3-pin, 8-bit SPI master
  UCB0CTL1 |= UCSSEL_2;                     // SMCLK
  UCB0BR0 |= 0x02;                          // :2  -> 1Mhz/2= 500 kHz
  UCB0BR1 = 0;                              //
  //UCA0MCTL = 0;                             // No modulation
  UCB0CTL1 &= ~UCSWRST;                     // **Initialize USCI state machine**
  
//  IE2 |= UCB0TXIE;               // Enable USCI0 TX interrupt
  
  UC0IE |= UCB0TXIE;
//--------------Master setup done----------------------------------
 
  
  
//-----------------transfer('A');---------------------------
  
  while(1) {
    __bis_SR_register(LPM4_bits + GIE);       // Enter LPM4, enable interrupts
  }
}


//-----------------Echo character--------------------------
#pragma vector=USCIAB0TX_VECTOR
__interrupt void USCI0TX_ISR(void)
//so funktioniert es nicht bei mir: interrupt(USCIAB0TX_VECTOR) USCI0TX_ISR(void) 
{
    UCB0TXBUF = 'A';                     // Transmit first character
}



#pragma vector=USCIAB0RX_VECTOR
__interrupt void USCI0RX_ISR(void)
//so funktioniert es nicht bei mir: interrupt(USCIAB0RX_VECTOR) USCI0RX_ISR(void) 
{
  // P1OUT |= BIT0;     // Testweise, ob es bis hierhin kommt -> ja tut es
   
  if (UCA0RXBUF == 'A') 
  {
    P1OUT &= ~BIT0;
  }
}

[Jim M. (turboj)]

Ich sehe nirgendwo das UCB RX interrupt Bit.
Ich sehe auch ein paar Fehler bei den Interrupt Bezeichnungen. Schau die 
Dir mal ganz in Ruhe an.


Edit: In diesem Falle braucht man Interrupts nicht unbedingt.

[full power (Gast)]

LPM4?
Bist du dir sicher, dass du den willst? ;-)

[Benjamin H (Gast)]

Hi, danke für die schnelle Antwort.

Jim M. schrieb:
> Ich sehe nirgendwo das UCB RX interrupt Bit.

Meinst du das hier?
Das ich das UCB RX ebenfalls aktivieren muss?

IE2 |= UCB0TXIE + UCB0RXIE + UCA0TXIE + UCA0RXIE;

Jim M. schrieb:
> Ich sehe auch ein paar Fehler bei den Interrupt Bezeichnungen. Schau die
> Dir mal ganz in Ruhe an.

Leider, habe ich da nichts finden können. Auch weiter im Internet 
geschaut, speziell zu den SPI Interrupts und mal im Beispiel Code von TI 
geschaut, aber die machen das auch so wie ich.

Ich mein, ich hab doch die Bezeichnungen richtig:

Von Master B0 TX ...

__interrupt void USCIB0TX_ISR(void)

... zu Slave A0 RX

__interrupt void USCIA0RX_ISR(void)

Oder stehe ich total auf den Schlauch und erkenne nicht, was du meinst?!


full power schrieb:
> LPM4?
> Bist du dir sicher, dass du den willst? ;-)

Hab das auch mal testweise geändert, tut sich aber nicht viel.

 __bis_SR_register(GIE);       //enable interrupts

Hab es sogar mit den Flag löschen probiert, wie in einem anderen Thread 
hier im Forum ich gefunden habe.

IFG2 &= ~UCB0TXIFG;

Hilft leider aber nicht viel.

[Benjamin H (Gast)]

Für eine Antwort oder weitere Anregungen, wäre ich sehr dankbar.

[Rufus Τ. F. (rufus)]

Benjamin H schrieb:
> Als neues Kapitel, habe ich mir die Kommunikation via SPI vorgenommen.
>
> Ich hab mich soweit durch den Family Guide - SPI Kapitel - und durch das
> Internet bzgl SPI durchgelesen.
>
> Und stieß dann auf die Idee und den Code, die zwei
> Kommunikationsleitungen die der G2553 hat, miteinander zu verbinden, um
> so die LED anzusteuern.
> Also habe ich die Pins: P1.1 mit P1.6, P1.2 mit P1.7 und P1.4 mit P1.5
> verbunden. (Siehe Bild)

Das und auch das Bild beschreibt nicht wirklich, was Du da versuchst.

Du scheinst zwei 'G2553 zu verwenden, und an die ist irgendwie irgendwo 
'ne LED angeschlossen. Oder soll da ein Controller mit sich selbst 
reden?

Das ist alles nebulös und unklar.

Wie wäre es mit einem Bild, das tatsächlich zeigt, was Du machst? 
(Schaltplan)

Ansonsten hat TI funktionierende Codebeispiele für alle seine 
MSP430-Varianten, auch für den 'G2553. Und wenn das nicht reicht, kann 
man sich den Code ansehen, den "Grace" ausgibt, oder den Code, der als 
Laufzeitumgebung bei Energia dabei ist.

[Benjamin H (Gast)]

Hallo Rufus, danke für die Antwort.

Ich hab es jetzt hinbekommen.


Das einzige was ich nun geändert habe, war dass ich beim Slave die 
'Clock polarity' von 'low' zu 'high'
und die 'clock phase' von 'captured on the first UCKL edge' zu 'changed 
on the first UCKL edge'

Also: UCCKPH = 0 und UCCKPL = 1

UCA0CTL0 |= UCCKPL + UCMSB + UCSYNC;      // 3-pin, 8-bit SPI slave

Ich werde mich diesbezüglich weiter reinlesen, da ich es nicht ganz 
verstehe, wieso man es so machen muss.

[Benjamin H (Gast)]

Ach entschuldigung, ich meinte ich habe das beim MASTER geändert.


Also: UCCKPH = 1 und UCCKPL = 0

UCB0CTL0 |= UCCKPH + UCMSB + UCMST + UCSYNC;  // 3-pin, 8-bit SPI master

[Benjamin H (Gast)]

Rufus Τ. F. schrieb:
> Du scheinst zwei 'G2553 zu verwenden, und an die ist irgendwie irgendwo
> 'ne LED angeschlossen. Oder soll da ein Controller mit sich selbst
> reden?

Genau, ich benutze nur einen Controller, der mit sich selbst redet.


Aber nur für den Anfang als experiment. Mein vorhaben ist, dass er mit 
einer SD-Karte kommuniziert.

Was ich jetzt beginnen werde ;-)

[Rufus Τ. F. (rufus)]

Benjamin H schrieb:
> Genau, ich benutze nur einen Controller, der mit sich selbst redet.

Dann ist die SPI-Schnittstelle ein sehr ungeeigneter Kandidat für 
derartige Tests.

[Benjamin H (Gast)]

Also hab das mal mit der SD Karte angeschlossen um zu testen, aber 
funktioniert leider nicht.

Als SD Karte benutze ich die SanDisk Ultra SDHC 8GB, die 9 Pins hat.


Hab mal wieder die verschiedensten Sachen ausprobiert, aber leider ohne 
positives Ergebnis.


Was mir klar wurde, ist dass das Timing eine große Rolle spielt beim 
SPI.
Ich will damit sagen, wann ich das Signal über die CS Leitung gebe ist 
von Bedeutung.  (Falls ich mich hier nicht irre)

Im Anhang findet ihr eine Skizze, wie ich den MC mit der SD-Karte 
angeschlossen habe.
Zudem habe ich bei der CS Leitung noch einen Pull-UP Widerstand benutzt. 
Da ich diesen Tipp mal nachgegangen bin, den ich im Internet gefunden 
habe.

#include "msp430g2553.h"
#include <legacymsp430.h>



void main(void) {
  WDTCTL = WDTPW + WDTHOLD;     // Stop watchdog timer
  
  P1OUT |= BIT5;                // für den ChipSelect
  P1DIR |= BIT5;
  
  DCOCTL = 0;                   // Select lowest DCOx and MODx settings<
  BCSCTL1 = CALBC1_1MHZ;        // Set DCO
  DCOCTL = CALDCO_1MHZ;
  
  P2DIR = 0xFF;                 // All P2.x outputs
  P2OUT &= 0x00;                // All P2.x reset
  
  P1DIR |= BIT0;
  P1OUT |= BIT0;
  
  
  P1SEL = BIT1 + BIT2 + BIT4;
  P1SEL2 = BIT1 + BIT2 + BIT4;
  
//--------------Slave setup---------------------------------
//  UCA0CTL1 = UCSWRST;                       // **Put state machine in reset**
//  UCA0CTL0 |= UCCKPL + UCMSB + UCSYNC;      // 3-pin, 8-bit SPI slave
//  UCA0CTL1 &= ~UCSWRST;                     // **Initialize USCI state machine**
//  
//   
//--------------Slave setup done----------------------------------

  
  
//--------------Master setup--------------------------------------
  __delay_cycles(60000);
  __delay_cycles(60000);
  __delay_cycles(60000);
  __delay_cycles(60000);
  
  UCA0CTL1 = UCSWRST;
  UCA0CTL0 |= UCCKPH + UCMSB + UCMST + UCSYNC;  // 3-pin, 8-bit SPI master
  UCA0CTL1 |= UCSSEL_2;                         // SMCLK
  UCA0BR0 |= 0x03;                              // :2  -> 1Mhz/2= 500 kHz
  UCA0BR1 = 0;                                  //
  UCA0MCTL = 0;                                 // No modulation
  UCA0CTL1 &= ~UCSWRST;                         // **Initialize USCI state machine**
  
  
  
  IE2 |= UCA0TXIE + UCA0RXIE;                   // Enable USCI0 TX interrupt
  
//--------------Master setup done----------------------------------
 
  P1OUT &= (~BIT5);                             // Select Device
  
//-----------------transfer('A');---------------------------
  while(!(IFG2 & UCA0TXIFG)); 
  
  while(1) {
    __bis_SR_register(GIE);         //enable interrupts
  }
}

//-----------------Echo character--------------------------

#pragma vector=USCIAB0TX_VECTOR
__interrupt void USCIA0TX_ISR(void)
//so funktioniert es nicht bei mir: interrupt(USCIAB0TX_VECTOR) USCI0TX_ISR(void) 
{
    UCA0TXBUF = 'A';                     // Transmit first character
}



#pragma vector=USCIAB0RX_VECTOR
__interrupt void USCIA0RX_ISR(void)
//so funktioniert es nicht bei mir: interrupt(USCIAB0RX_VECTOR) USCI0RX_ISR(void) 
{
  if (UCA0RXBUF == 'A') 
    P1OUT &= ~BIT0;
}

Im Anhang ist noch ein Bild darüber, was ich in den Registern auslese.