STM32F103 SPI gibt nur 0 zurück

[Sean G. (atmega318)]

Hallo
Ich habe wieder mal ein kleines Problem, dass sich einfach nicht lösen 
lassen will:

Es geht um den SPI im STM32f103. TX funktioniert (Oszi) aber RX kommt 
nur 0 zurück. Elektrischer Fehler ist eigentlich ausgeschlossen: wenn 
ich MISO fest auf VCC hänge zeigt GPIOA->IDR zwar eine 1, aber Send() 
gibt 0 zurück!
Code ist nur das hier:

GPIO_InitTypeDef GPIO_InitStructure;
    SPI_InitTypeDef SPI_InitStructure;
    

    RCC_APB2PeriphClockCmd(RCC_APB2Periph_GPIOA, ENABLE);

    GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
    
    GPIO_InitStructure.GPIO_Pin = GPIO_Pin_5 | GPIO_Pin_7;
    GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AF_PP;           
    GPIO_Init(GPIOA, &GPIO_InitStructure);

    GPIO_InitStructure.GPIO_Pin = GPIO_Pin_6;
    GPIO_InitStructure.GPIO_Mode = GPIO_Mode_IN_FLOATING;
    GPIO_Init(GPIOA, &GPIO_InitStructure);

    RCC_APB2PeriphClockCmd(RCC_APB2Periph_SPI1, ENABLE);

    SPI_I2S_DeInit(SPI1);

    SPI_StructInit(&SPI_InitStructure);
    SPI_InitStructure.SPI_Direction = SPI_Direction_2Lines_FullDuplex;
    SPI_InitStructure.SPI_Mode = SPI_Mode_Master;
    SPI_InitStructure.SPI_DataSize = SPI_DataSize_8b;
    SPI_InitStructure.SPI_CPOL = SPI_CPOL_Low;
    SPI_InitStructure.SPI_CPHA = SPI_CPHA_1Edge;
    SPI_InitStructure.SPI_NSS = SPI_NSS_Soft;
    SPI_InitStructure.SPI_FirstBit = SPI_FirstBit_MSB;
    SPI_InitStructure.SPI_BaudRatePrescaler = SPI_BaudRatePrescaler_2;
    SPI_InitStructure.SPI_CRCPolynomial = 7;
    SPI_Init(SPI1, &SPI_InitStructure);

    SPI_Cmd(SPI1, ENABLE);

    while(1)
    {
        uint8_t x = Send(SPI1, 123);
    }

und die Funktion Send():

uint8_t Send(SPI_TypeDef* SPIx, uint8_t data){

  SPIx->DR = data; // write data to be transmitted to the SPI data register
  while( !(SPIx->SR & SPI_I2S_FLAG_TXE) ); // wait until transmit complete
  while( !(SPIx->SR & SPI_I2S_FLAG_RXNE) ); // wait until receive complete
  while( SPIx->SR & SPI_I2S_FLAG_BSY ); // wait until SPI is not busy anymore
  return SPIx->DR; // return received data from SPI data register
}

Weiss da jemand rat? SPI Slave funktioniert Perfekt (sogar mit DMA), nur 
Master RX zickt...
Vielen Dank, Sean

[holger (Gast)]

>aber Send() gibt 0 zurück!

Lass dir x ml per UART susgeben. Oder mach es volatile.
x wird nirgendwo verwendet und könnte wegoptimiert werden.

[Sean G. (atmega318)]

holger schrieb:
>>aber Send() gibt 0 zurück!
>
> Lass dir x ml per UART susgeben. Oder mach es volatile.
> x wird nirgendwo verwendet und könnte wegoptimiert werden.

Das ist es nicht, ich habe x mit dem Debugger angeschaut. Zudem habe ich 
es auch mal versucht mit mehreren zugriffen auf x, das ändert nichts. 
Auch SPI1->DR bleibt leer!

[Zur Höxxxx mit Flanders (Gast)]

Das ist Wasser auf meine Mühlen. Ich hatte genau das gleiche mit einem 
32F3xx. Ich hab irgendwann nach jedem Strohhalm gegriffen und 
angefangen, am Prescaler zu drehen. Bei einer einzigen Einstellung 
klappts. Einen drunter oder drüber ist das Ding tot...
Ich habs nicht weiter verfolgt, weil reines Hobby und keine Zeit.

Versuch mal "SPI_InitStructure.SPI_BaudRatePrescaler = 12;"

[holger (Gast)]

Nimm mal RCC_APB2Periph_AFIO mit rein und schalte die Clocks auf
BEVOR du irgendwas initialisierst.

    RCC_APB2PeriphClockCmd(RCC_APB2Periph_GPIOA | RCC_APB2Periph_AFIO, 
ENABLE);
    RCC_APB2PeriphClockCmd(RCC_APB2Periph_SPI1, ENABLE);

    GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
 .....

[Sean G. (atmega318)]

    RCC_APB2PeriphClockCmd(RCC_APB2Periph_GPIOA | RCC_APB2Periph_AFIO, ENABLE);
    RCC_APB2PeriphClockCmd(RCC_APB2Periph_SPI1, ENABLE);
    

    GPIO_InitTypeDef GPIO_InitStructure;
    SPI_InitTypeDef SPI_InitStructure;
    


    

    GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
    
    GPIO_InitStructure.GPIO_Pin = GPIO_Pin_5 | GPIO_Pin_7;
    GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AF_PP;           
    GPIO_Init(GPIOA, &GPIO_InitStructure);

    GPIO_InitStructure.GPIO_Pin = GPIO_Pin_6;
    GPIO_InitStructure.GPIO_Mode = GPIO_Mode_IN_FLOATING;
    GPIO_Init(GPIOA, &GPIO_InitStructure);



    SPI_I2S_DeInit(SPI1);

    SPI_StructInit(&SPI_InitStructure);
    SPI_InitStructure.SPI_Direction = SPI_Direction_2Lines_FullDuplex;
    SPI_InitStructure.SPI_Mode = SPI_Mode_Master;
    SPI_InitStructure.SPI_DataSize = SPI_DataSize_8b;
    SPI_InitStructure.SPI_CPOL = SPI_CPOL_Low;
    SPI_InitStructure.SPI_CPHA = SPI_CPHA_1Edge;
    SPI_InitStructure.SPI_NSS = SPI_NSS_Soft;
    SPI_InitStructure.SPI_FirstBit = SPI_FirstBit_MSB;
    SPI_InitStructure.SPI_BaudRatePrescaler = SPI_BaudRatePrescaler_2;
    SPI_InitStructure.SPI_CRCPolynomial = 7;
    SPI_Init(SPI1, &SPI_InitStructure);

    SPI_Cmd(SPI1, ENABLE);

    while(1)
    {
        uint8_t x = Send(SPI1, 123);
        delay = x / 2;  //dummy aktion, nur zur sicherheit!
    }

So siehts jetzt aus, hat sich aber nichts geändert. Das mit Prescaler = 
12 habe ich auch getestet, gleiches Ergebnis. Trotzdem danke :)

[holger (Gast)]

>So siehts jetzt aus, hat sich aber nichts geändert.

Ich werde das Gefühl immer noch nicht los das das
nur ein Missverständnis zwischen dir und deinem Debugger ist.

Gib x über UART aus. Da MOSI ja was sendet sollte
das SPI Modul funktionieren. SCK funktioniert auch?

[Marc V. (Firma: Vescomp) (logarithmus)]

Sean Goff schrieb:

 while( !(SPIx->SR & SPI_I2S_FLAG_TXE) ); // wait until transmit  complete
 while( !(SPIx->SR & SPI_I2S_FLAG_RXNE) ); // wait until receive complete

 Bist du sicher, dass receive complete nötig ist ?
 SPI ist eigentlich bidirectional, es könnte sein, dass du damit nur
 sein (leeres) SPI-Register empfängst...

[Sean G. (atmega318)]

Marc Vesely schrieb:
> Sean Goff schrieb:
>

>  while( !(SPIx->SR & SPI_I2S_FLAG_TXE) ); // wait until transmit 
> complete
>  while( !(SPIx->SR & SPI_I2S_FLAG_RXNE) ); // wait until receive 
> complete
>

>
>  Bist du sicher, dass receive complete nötig ist ?
>  SPI ist eigentlich bidirectional, es könnte sein, dass du damit nur
>  sein (leeres) SPI-Register empfängst...

Dachte ich auch schon, aber in allen Libs wird's so gemacht, und ohne 
ändert sich nichts!

holger schrieb:
>>So siehts jetzt aus, hat sich aber nichts geändert.
>
> Ich werde das Gefühl immer noch nicht los das das
> nur ein Missverständnis zwischen dir und deinem Debugger ist.
>
> Gib x über UART aus. Da MOSI ja was sendet sollte
> das SPI Modul funktionieren. SCK funktioniert auch?

SCK und MOSI funktionieren. Ich kanns mir einfach nicht vorstellen, ich 
habe EXAKT die gleiche Methode beim F4 angewendet. Funktioniert super 
(mit dem Debugger...)
UART müsste ich halt auch zuerst implementieren.

Ich hab aber jetzt mal folgendes gemacht:

    WS2812_init();
    while(1)
    {
        uint8_t x = Send(SPI1, 123);
        if(x == 0)
        {
            led[0].r = 3;
            led[0].g = 0;
            led[0].b = 0;
        }
        else
        {
            led[0].r = 3;
            led[0].g = 3;
            led[0].b = 3;
        }
        WS2812_update();
    }

-leuchtet Rot, nix Weiss!

[holger (Gast)]

>SCK und MOSI funktionieren.

Bist du sicher das du MISO am richtigen Pin vermutest?
Vieleicht verzählt beim anschliessen?
Irgend sowas banales wird das sein;)

[Sean G. (atmega318)]

holger schrieb:
>>SCK und MOSI funktionieren.
>
> Bist du sicher das du MISO am richtigen Pin vermutest?
> Vieleicht verzählt beim anschliessen?
> Irgend sowas banales wird das sein;)

Sieht leider für mich nicht so aus, denn erstens ist das ne Platine, die 
garantiert korrekt ist und zweitens wenn ich den Pin auf VCC Brücke sehe 
ich ihn im GPIOA->IDR auf eins gehen (Debugger). Drittens habe ich ihn 
auch schon als Ausgang konfiguriert und zappeln lassen -> Oszi sagt es 
ist der richtige :(

[holger (Gast)]

Letzter Versuch:

Schalte MISO mal auf GPIO_Mode_IPU und lass ihn offen.

[Marc V. (Firma: Vescomp) (logarithmus)]

Sean Goff schrieb:
> Dachte ich auch schon, aber in allen Libs wird's so gemacht, und ohne
> ändert sich nichts!
 Probier es mal ganz ohne Flags, etwa so:

  SPIx->DR = data;  
  while (!(SPIx->SR & RXNE));  // 0x01
  return (SPIx->DR);

[Sean G. (atmega318)]

holger schrieb:
> Letzter Versuch:
>
> Schalte MISO mal auf GPIO_Mode_IPU und lass ihn offen.

Hat nichts gebracht :(

Marc Vesely schrieb:
> Sean Goff schrieb:
>> Dachte ich auch schon, aber in allen Libs wird's so gemacht, und ohne
>> ändert sich nichts!
>  Probier es mal ganz ohne Flags, etwa so:  SPIx->DR = data;
>   while (!(SPIx->SR & RXNE));  // 0x01
>   return (SPIx->DR);

Das akzeptiert der Compiler so nicht (er sagt RXNE sei nicht bekannt.) 
Wenn ich direkt 0x01 nehme, ändert sich am Ergebnis nichts (leuchtet 
rot.)

[Marc V. (Firma: Vescomp) (logarithmus)]

Sean Goff schrieb:
> Das akzeptiert der Compiler so nicht (er sagt RXNE sei nicht bekannt.)
> Wenn ich direkt 0x01 nehme, ändert sich am Ergebnis nichts (leuchtet
> rot.)
 Mein letzter Versuch:
 SPI-Mode ändern.

[mal so ins Blaue (Gast)]

> ich habe EXAKT die gleiche Methode beim F4 angewendet

Haben die beiden die gleiche BUS Matrix? Hast du beim F1 das richtige 
Target im Projekt ausgewählt?

[Sean G. (atmega318)]

mal so ins Blaue schrieb:
>> ich habe EXAKT die gleiche Methode beim F4 angewendet
>
> Haben die beiden die gleiche BUS Matrix? Hast du beim F1 das richtige
> Target im Projekt ausgewählt?

Mit Methode meinte ich eigentlich eher den Wert per debugger abzufragen, 
ohne ihn sonst zu benutzen. Prozessor stimmt!

[Sean G. (atmega318)]

Marc Vesely schrieb:
> Sean Goff schrieb:
>> Das akzeptiert der Compiler so nicht (er sagt RXNE sei nicht bekannt.)
>> Wenn ich direkt 0x01 nehme, ändert sich am Ergebnis nichts (leuchtet
>> rot.)
>  Mein letzter Versuch:
>  SPI-Mode ändern.

Wie meinst du das? Auf Slave? Dann funktionierts!

[Marc V. (Firma: Vescomp) (logarithmus)]

Sean Goff schrieb:
> Wie meinst du das? Auf Slave? Dann funktionierts!

 War auch so mal ins Blaue.
 Ich dachte wie die Daten eingelesen werden (welche Flanke).