STM32F SPI Slave Tx Interrupt Callback löst nicht aus

[StmGuy (Gast)]

Hallo,

ich habe Probleme beim Programmieren eines STM SPI-Slaves.
Dazu habe ich mir einen Testaufbau aus zwei Nucleo-Boards aufgebaut. 
Programmiert wird in STM32CubeIde, über dessen Oberfläche auch die 
Hardware-Konfiguration vorgenommen wird.

Das erste Board ist der SPI-Master (Nucleo-F411RE):
Dieser Master sendet erst ein Datenbyte auf der MOSI-Leitung aus (8 CLK 
Takte), danach gibt es eine kurze Pause, und anschließend erzeugt der 
Master 8 weitere Takte, um ein Byte vom Slave auf der MOSI-Leitung 
einzulesen.
Hier der zugehörige Code ausgeführt:

  //file: main.c (SPI Master)
  //... Setup functions generated by STM32CubeIde
  while (1)
  {
      //Send 1 byte 
      HAL_SPI_Transmit(&hspi1, msg, 1, HAL_MAX_DELAY);
        
        //short break
        for(int j=0; j<300; j++)
        { asm volatile("NOP"); }

        //receive 1 byte
        HAL_SPI_Receive(&hspi1, ans, 1, HAL_MAX_DELAY);

        //wait and repeat
        HAL_Delay(300);
  }

(Die Daten, die vom Slave kommen werden auf dem Master nicht weiter 
verarbeitet, das Master-Board ist hier nur "Taktgenerator" für die 
CKL-Leitung. Der Master erzeugt die CLK-Takte und sendet Daten auf der 
MOSI-Leitung, das konnte ich am Oszi überprüfen.)

Das zweite Board ist der SPI-Slave (Nucleo-F446RE). Dieses Board soll 
warten, bis ein Command Byte vom Master empfangen wird und beim nächsten 
Takt mit seinem eigenen Byte Antworten. (Eigentlich einfach, oder?)
Da der Slave noch andere (Mess-)Aufgaben ausführt und nicht blockieren 
kann, bis der Master ein Byte anfragt, sendet muss die Kommunikation auf 
dem Slave über Interrupts stattfinden. Dazu sollen die 
HAL_SPI_Receive_IT() bzw. HAL_SPI_Transmit_IT() Funktionen verwendet 
werden.

Hier ist der relevante Code der Slave-Main:

//file: main.c (SPI Slave)
//...
SPI_HandleTypeDef hspi1;
//...
int main(void
{
  //..
  MX_SPI1_Init();
  //...
  spi1_slave_init(&hspi1);
  while (1)
  {
     //leerer Loop, alles geschieht über Interrupts
  }
  //..
}
  
//..

static void MX_SPI1_Init(void)
{
  hspi1.Instance = SPI1;
  hspi1.Init.Mode = SPI_MODE_SLAVE;
  hspi1.Init.Direction = SPI_DIRECTION_2LINES;
  hspi1.Init.DataSize = SPI_DATASIZE_8BIT;
  hspi1.Init.CLKPolarity = SPI_POLARITY_LOW;
  hspi1.Init.CLKPhase = SPI_PHASE_2EDGE;
  hspi1.Init.NSS = SPI_NSS_SOFT;
  hspi1.Init.FirstBit = SPI_FIRSTBIT_MSB;
  hspi1.Init.TIMode = SPI_TIMODE_DISABLE;
  hspi1.Init.CRCCalculation = SPI_CRCCALCULATION_DISABLE;
  hspi1.Init.CRCPolynomial = 10;
  if (HAL_SPI_Init(&hspi1) != HAL_OK)
  {
    Error_Handler();
  }
}

//...

die Funktion spi1_slave_init() ist in der Datei spi-com.c definiert, 
genauso wie die entsprechenden Interrupt-Callbacks:

//file: spi-com.c
#include "spi-com.h"

extern UART_HandleTypeDef huart2;
extern SPI_HandleTypeDef hspi1;

uint8_t spi_command[2] = {0x00, 0};
uint8_t spi_tx_buf[2] = {0x3C, 0x55};

void spi1_slave_init()
{
  HAL_SPI_Receive_IT(&hspi1, spi_command, 1);
}

void HAL_SPI_TxCpltCallback (SPI_HandleTypeDef * hspi)
{
  /* THIS IS NEVER EXECUTED!!! WHY?*/

  //rearm SPI Rx Interrupt
  HAL_SPI_Receive_IT(&hspi1, spi_command, 1);
}


void HAL_SPI_RxCpltCallback(SPI_HandleTypeDef * hspi)
{
  char buf[30];
  int uart_buf_len = 0;
  
  //print received SPI command
  uart_buf_len = sprintf(buf, "%02x\r\n", spi_command[0]);
  HAL_UART_Transmit(&huart2, (uint8_t *)buf, uart_buf_len, 1000);
  
  //nachdem ein byte empfangen, beim nächsten Takt ein byte senden
  HAL_SPI_Transmit_IT(&hspi1, spi_tx_buf, 1);

}

PROBLEM:
Zwar wird der HAL_SPI_RxCpltCallback aufgerufen, wenn ein Byte vom 
Master empfangen wurde. Obwohl im dort der Tx-Interrupt scharf 
geschalten wird (mittels HAL_SPI_Transmit_IT()) wird der zugehörige 
Tx-Callback nie aufgerufen. also:

Warum wird HAL_SPI_TxCpltCallback nicht aufgerufen?

Anmerkungen:
1. SPI Interrupts wurden in CubeMX aktiviert
2. Kommentare des automatisch erzeugten Codes habe ich hier entfernt, 
sowie alle automatisch generierten Codeteile, die nichts mit SPI zu tun 
haben.
3. Wenn in HAL_SPI_RxCpltCallback() nicht HAL_SPI_Transmit_IT() sondern 
HAL_SPI_Receive_IT() aktiviert wird läuft das Programm ohne Probleme 
weiter und reagiert auf jedes eingehende Byte. Das Problem tritt also 
erst auf, wenn ich versuche die Tx-Funktionalität zu benutzen.



Viel Text, aber über Lösungsmöglichkeiten würde ich mich sehr freuen!

PS: Für dieses Projekt bin ich auf die HAL-Libraries beschränkt. Deshalb 
bitte keine Vorschläge à la "HAL wegwerfen und alles selber in die 
Register coden" ;)

[Peter (Gast)]

StmGuy schrieb:
> void HAL_SPI_RxCpltCallback(SPI_HandleTypeDef * hspi)
> {
>   char buf[30];
>   int uart_buf_len = 0;
>
>   //print received SPI command
>   uart_buf_len = sprintf(buf, "%02x\r\n", spi_command[0]);
>   HAL_UART_Transmit(&huart2, (uint8_t *)buf, uart_buf_len, 1000);
>
>   //nachdem ein byte empfangen, beim nächsten Takt ein byte senden
>   HAL_SPI_Transmit_IT(&hspi1, spi_tx_buf, 1);
> }

Das ist sehr optimistisch programmiert.

Du rufst hier in einem Interrupt-Callback (also im Interrupt selber) 
zwei Funktionen (HAL_UART_Transmit() und HAL_SPI_Transmit_IT()) auf, die 
eine Menge Code ausführen. Kein Mensch weiss, was genau da passiert. Es 
sei denn man hat sich die Funktionen mal angschaut. Hast Du das ?
Ausserdem ist es so, dass diese HAL-Funktionen ihre eigene 
State-Maschine mitbringen.

Mach eins nach dem anderen. Setz ein Flag und starte die Routinen in der 
Haupschleife.

Gruß Peter

[ergebnisseprüfen (Gast)]

Wie wäre erstmal, die Ergebnisse der HAL_ Funktionen zu prüfen.

[Harry L. (mysth)]

Peter schrieb:
> HAL_SPI_Transmit_IT()

Die darf man im Callback durchaus verwenden, da die nicht blockierend 
ist.

Anders ist das mit HAL_UART_Transmit(). Das ist blockierend und hat im 
Interrupt nichts zu suchen!
Peter schrieb:
> Setz ein Flag und starte die Routinen in der
> Haupschleife.

Genau so!

Ausserdem fehlt am Anfang des Callback die obligatorische Überprüfung ob 
das übergebene hspi (SPI-Handle) tatsächlich zu dem gewünschten Port 
gehört.

Also so:

void HAL_SPI_RxCpltCallback(SPI_HandleTypeDef * hspi)
{
  if (hspi == myspi)
    {
      // do something
    }
}